Skip to content

(PA-6627) Update the first released version for released platforms in… #726

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

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

skyamgarp
Copy link
Contributor

@skyamgarp skyamgarp commented Jun 26, 2024

… spec

We add a new platform in 'latest_platform_list' method so that it considers builds from nightlies with the 'latest' tag before they are released (since the versioned release package won't be available for new platforms). The platforms which are no more new and have been released are removed from 'latest_platform_list' here along with their first released versions specified as needed.

We also test upgrade from the first released puppet7 version to the latest released puppet7 version. The new platforms only have one release, so this test is not applicable for them. Removed platforms which have had multiple puppet7 versions released as of now to make the test applicable for them. This is done by setting the 'multiple_puppet7_versions' parameter to true.

Tested this on local. Ad-hoc pipeline isn't available at this stage.

Updated release version according to https://perforce.atlassian.net/wiki/spaces/PUPPROD/pages/371851628/Currently+Supported+OS+Platforms

@skyamgarp skyamgarp requested review from bastelfreak and a team as code owners June 26, 2024 08:06
@skyamgarp skyamgarp marked this pull request as draft June 26, 2024 08:48
@skyamgarp skyamgarp marked this pull request as draft June 26, 2024 08:48
@skyamgarp skyamgarp marked this pull request as ready for review June 26, 2024 12:23
@skyamgarp skyamgarp marked this pull request as draft June 26, 2024 12:25
@bastelfreak
Copy link
Collaborator

can you elaborate a bit why you do this change?

@skyamgarp
Copy link
Contributor Author

skyamgarp commented Jun 26, 2024

can you elaborate a bit why you do this change?

We add a new platform in latest_platform_list method so it should consider build from nightlies before they are released. Now as they are released I have removed them from the method.

@skyamgarp skyamgarp marked this pull request as ready for review June 27, 2024 10:08
@donoghuc
Copy link
Contributor

That information belongs in the commit message. It is very unclear from the commit message what the refactor is doing.

@mhashizume mhashizume added the maintenance Maintenance chores are typically excluded from changelogs label Jun 27, 2024
@shubhamshinde360 shubhamshinde360 force-pushed the PA-6627 branch 3 times, most recently from 26daa5f to 5e34bf3 Compare June 28, 2024 07:03
… spec

 We add a new platform in 'latest_platform_list' method so that it considers builds from nightlies with the 'latest' tag before they are released (since the versioned release package won't be available for new platforms). The platforms which are no more new and have been released are removed from 'latest_platform_list' here along with their first released versions specified as needed.

 We also test upgrade from the first released puppet7 version to the latest released puppet7 version. The new platforms only have one release, so this test is not applicable for them. Removed platforms which have had multiple puppet7 versions released as of now to make the test applicable for them. This is done by setting the 'multiple_puppet7_versions' parameter to true.
@shubhamshinde360
Copy link
Contributor

@bastelfreak, @donoghuc Could you please take a look, added more info in the commit message. Thanks!

@joshcooper joshcooper merged commit 3ce04a2 into puppetlabs:main Jun 28, 2024
17 checks passed
@skyamgarp skyamgarp deleted the PA-6627 branch July 1, 2024 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Maintenance chores are typically excluded from changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants