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

Fix CI: Updated upload-artifact and download-artifact to v4 #22127

Merged
merged 5 commits into from
Feb 11, 2025

Conversation

abhishek-hashicorp
Copy link
Contributor

@abhishek-hashicorp abhishek-hashicorp commented Feb 10, 2025

Description

The current CI is failing due to the error This request has been automatically failed because it uses a deprecated version of actions/upload-artifact: 0b7f8abb1508181956e8e162db84b466c27e18ce.

Updated to v4 version now.

Deprecation Link

Testing & Reproduction steps

Links

Failed CI run

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@abhishek-hashicorp abhishek-hashicorp requested a review from a team as a code owner February 10, 2025 08:27
@github-actions github-actions bot added type/ci Relating to continuous integration (CI) tooling for testing or releases theme/contributing Additions and enhancements to community contributing materials labels Feb 10, 2025
@abhishek-hashicorp abhishek-hashicorp added pr/no-changelog PR does not need a corresponding .changelog entry pr/no-backport type/ci Relating to continuous integration (CI) tooling for testing or releases and removed type/ci Relating to continuous integration (CI) tooling for testing or releases theme/contributing Additions and enhancements to community contributing materials labels Feb 10, 2025
@abhishek-hashicorp abhishek-hashicorp changed the title Fix CI: Updated upload-artifact to v4 Fix CI: Updated upload-artifact and download-artifact to v4 Feb 10, 2025
Copy link
Contributor

@nitin-sachdev-29 nitin-sachdev-29 left a comment

Choose a reason for hiding this comment

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

LGTM

sreeram77
sreeram77 previously approved these changes Feb 10, 2025
Copy link
Member

@sreeram77 sreeram77 left a comment

Choose a reason for hiding this comment

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

LGTM!

@abhishek-hashicorp abhishek-hashicorp merged commit fcc972e into release/1.15.x Feb 11, 2025
146 of 148 checks passed
@abhishek-hashicorp abhishek-hashicorp deleted the abhishek/fix-ci-1-15-x branch February 11, 2025 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-backport pr/no-changelog PR does not need a corresponding .changelog entry type/ci Relating to continuous integration (CI) tooling for testing or releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants