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.

Sign up for GitHub

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

Jump to bottom

ci: Update to actions/upload-artifact@v4 in cifuzz.yml #1554

Merged
merged 1 commit into from
Feb 10, 2025
Merged

ci: Update to actions/upload-artifact@v4 in cifuzz.yml #1554

merged 1 commit into from
Feb 10, 2025

Conversation

ac000
Copy link
Member

@ac000 ac000 commented Feb 4, 2025 

    ci: Update to actions/upload-artifact@v4 in cifuzz.yml
    
    We need to update to the latest version of actions/upload-artifact in
    cifuzz.yml due to the workflow failing because of
    
      Error: This request has been automatically failed because it uses a
      deprecated version of `actions/upload-artifact: v3`. Learn more:
      https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/
    
    Signed-off-by: Andrew Clayton <a.clayton@nginx.com>

@ac000 ac000 marked this pull request as ready for review February 4, 2025 04:53
@ac000 ac000 requested review from avahahn and hongzhidao February 4, 2025 04:53
hongzhidao
hongzhidao approved these changes Feb 10, 2025
View reviewed changes
Copy link
Contributor

@hongzhidao hongzhidao left a comment

Choose a reason for hiding this comment

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

Hi @ac000,
I'm fine with it if tests pass. Thanks.

@ac000
ci: Update to actions/upload-artifact@v4 in cifuzz.yml
7a0addd 
We need to update to the latest version of actions/upload-artifact in
cifuzz.yml due to the workflow failing because of

  Error: This request has been automatically failed because it uses a
  deprecated version of `actions/upload-artifact: v3`. Learn more:
  https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
@ac000
Copy link
Member Author

ac000 commented Feb 10, 2025

Rebase with master

$ git range-diff c78a54db...7a0addd7
-:  -------- > 1:  b2866409 otel, tools/unitctl: bump the openssl crate from 0.10.68 to 0.10.70
-:  -------- > 2:  df62a2a2 http: Fix WebSockets with Firefox
1:  c78a54db = 3:  7a0addd7 ci: Update to actions/upload-artifact@v4 in cifuzz.yml

@ac000 ac000 merged commit 7a0addd into nginx:master Feb 10, 2025
3 checks passed
@ac000 ac000 deleted the ci-fuzz-fix branch February 10, 2025 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hongzhidao hongzhidao hongzhidao approved these changes

@avahahn avahahn Awaiting requested review from avahahn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ac000 @hongzhidao