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

Upgrade actions/{upload,download}-artifact #565

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

YaSuenag
Copy link
Contributor

Pull Request

Issue Number: #564

Summary

Upgrade actions/upload-artifact and actions/download-artifact in GHA workflow.

See GitHub Blog for details: https://github.blog/changelog/2024-02-13-deprecation-notice-v1-and-v2-of-the-artifact-actions/

Changes

  • .github/workflows/1-pr.yaml
  • .github/workflows/dev_carbon-aware-api.yml
  • .github/workflows/2.a-deploy.yaml.ignore

All of lines in 2.a-deploy.yaml.ignore are commented out, but I upgraded because this workflow might be enabled again in future.

One of actions/download-artifact in 1-pr.yaml uses v3 (non-deprecated version), but I upgraded because I think it is better to use same version entire workflows.

Checklist

  • Local Tests Passing?
  • CICD and Pipeline Tests Passing?
  • Added any new Tests?
  • Documentation Updates Made?
  • Are there any API Changes? If yes, please describe below.
  • This is not a breaking change. If it is, please describe it below.

Are there API Changes?

No

Is this a breaking change?

No

This PR Closes #564

Signed-off-by: Yasumasa Suenaga <suenaga@oss.nttdata.com>
Signed-off-by: Yasumasa Suenaga <suenaga@oss.nttdata.com>
@YaSuenag YaSuenag changed the title Pr/upgrade upload artifact Upgrade actions/{upload,download}-artifact Sep 18, 2024
@YaSuenag
Copy link
Contributor Author

CC: @danuw

@danuw
Copy link
Collaborator

danuw commented Sep 18, 2024

@YaSuenag sorry missed that. I did not update the workflow that is disabled but I guess we can close this and use the other one I fixed in the branch directly. Did not get a chance to apply the change until now but looks all good now
Thank you for that btw

@YaSuenag
Copy link
Contributor Author

Changes for .github/workflows/1-pr.yaml does not seems to happen in #562 - I think we will see same error when other PR is filed.

@danuw
Should I revise this PR for 1-pr.yaml?

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.

[Bug]: 1-pr.yaml should upgrade actions/upload-artifact
2 participants