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

Adjust action to use v4-compatible version of upload/download artifact #124

Merged
merged 3 commits into from
Nov 21, 2024

Conversation

Koenvh1
Copy link
Contributor

@Koenvh1 Koenvh1 commented Nov 21, 2024

This release contains the following changes:

  • Update download-artifact and upload-artifact to v4-compatible due to GitHub changes:

Artifact actions v3 will be deprecated by December 5, 2024. You are receiving this email because you have GitHub Actions workflows using v3 of actions/upload-artifact or actions/download-artifact. After this date using v3 of these actions will result in a workflow failure. Artifacts within their retention period will remain accessible from the UI or REST API regardless of the version used to upload.

Successful test runs can be seen here:

Release checklist:

  • 1. Create a branch in the RELEASE repo, let's call this the RELEASE branch.
  • 2. Change RPM_MACROS_URL in the workflow to point to the new RELEASE branch.
  • 3. Create a PR in the RELEASE repo for the RELEASE branch.
  • 4. Create a matching branch in the TEST repo, let's call this the TEST branch.
  • 5. Make the desired changes to the RELEASE branch.
  • 6. In the TEST branch modify .github/workflows/pkg.yml so that instead of referring to pkg-rust.yml@vX it refers to pkg-rust.yml@<Git ref of HEAD commit on the TEST branch> or pkg-rust.yml@<test branch name>.
  • 7. Create a PR in the ploutos-testing repository from the TEST branch to main, let's call this the TEST PR.
  • 8. Repeat step 5 until the the Packaging workflow run in the TEST PR passes and behaves as desired.
  • 9. Merge the TEST PR to the main branch.
  • 10. Verify that the automatically invoked run of the Packaging workflow in the TEST repo against the main branch passes and behaves as desired. If not, repeat steps 4-9 until the new TEST PR passes and behaves as desired.
  • 11. Create a release tag in the TEST repo with the same release tag as will be used in the RELEASE repo, e.g. v1.2.3. Note: Remember to respect semantic versioning, i.e. if the changes being made are not backward compatible you will need to bump the MAJOR version (in MAJOR.MINOR.PATCH) and any workflows that invoke the reusable workflow will need to be manually edited to refer to the new MAJOR version.
  • 12. Verify that the automatically invoked run of the Packaging workflow in the TEST repo passes against the newly created release tag passes and behaves as desired. If not, delete the release tag in the TEST repo and repeat steps 4-11 until the new TEST PR passes and behaves as desired.
  • 13. Merge the RELEASE PR to the main branch.
  • 14. Change RPM_MACROS_URL in the workflow to point to vX.Y.Z tag (if your release branch has a different name).
  • 15. Create the new release vX.Y.Z tag in the RELEASE repo.
  • 16. Update the vX tag in the RELEASE repo to point to the new vX.Y.Z tag (howto).
  • 17. Edit .github/workflows/pkg.yml in the main branch of the TEST repo to refer again to @vX.
  • 18. Verify that the Packaging action in the TEST repo against the main branch passes and works as desired.
  • 19. (optional) If the MAJOR version was changed, update affected repositories that use the reusable workflow to use the new MAJOR version, including adjusting to any breaking changes introduced by the MAJOR version change.

@Koenvh1 Koenvh1 marked this pull request as ready for review November 21, 2024 14:45
@Koenvh1 Koenvh1 requested a review from ximon18 November 21, 2024 14:45
Copy link
Member

@ximon18 ximon18 left a comment

Choose a reason for hiding this comment

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

Great work @Koenvh1 !

@Koenvh1 Koenvh1 merged commit ffd58e8 into main Nov 21, 2024
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.

2 participants