Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Test build to check PR process #1989

Closed
wants to merge 3 commits into from

Conversation

dmurphy18
Copy link
Contributor

What does this PR do?

Check PR build process

What issues does this PR fix or reference?

Checking how PR build process works - test only

@dmurphy18 dmurphy18 self-assigned this Feb 5, 2024
@dmurphy18 dmurphy18 changed the title Test build to check PR process [WIP] Test build to check PR process Feb 8, 2024
@blu-base
Copy link

i believe, you just missed a single spot to bump 3005.1 to 3005.5, the respective suite name

https://github.com/dmurphy18/salt-bootstrap/blob/06c102b747c6a97f7186dbdf21d65597c4d5b1ae/kitchen.yml#L304

@@ -27,10 +27,10 @@ suites:
salt_call_command: c:\Program Files\Salt Project\Salt\salt-call.bat
- name: stable-3005
provisioner:
salt_version: 3005.1-1
salt_version: 3005.5-1
- name: stable-3005-1

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this likely need's to be changed to stable-3005-5, too

@@ -303,7 +303,7 @@ suites:
- openbsd-6
- name: stable-3005-1
Copy link

@blu-base blu-base Feb 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the valie need's to be changed to stable-3005-5

@dmurphy18
Copy link
Contributor Author

dmurphy18 commented Feb 15, 2024

@blu-base WIP Work In Progress :D , all of the 3005 support going away, 5 working days and it is EOL 🥳 .
Kitchen-salt is going too, moving it to GitHub Actions, so still a lot of major changes to go
This is just a test to see what is failing, only changed the date. Real PR is #1987

@blu-base
Copy link

blu-base commented Feb 15, 2024

Ok, nevermind. I was just trying to dive in a little bit, to see what I can help with in this project

@dmurphy18
Copy link
Contributor Author

@blu-base Quick comment back, updated mine with real PR #1987

@dmurphy18 dmurphy18 closed this Feb 22, 2024
@dmurphy18
Copy link
Contributor Author

Closing this since was a test of current build PR process, which has issues, see #1987 for actual PR doing the work

@dmurphy18 dmurphy18 deleted the test_build branch July 15, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants