Skip to content

[skip-changelog] fix release CI #2546

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 2 commits into from
Feb 20, 2024
Merged

[skip-changelog] fix release CI #2546

merged 2 commits into from
Feb 20, 2024

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Feb 19, 2024

Please check if the PR fulfills these requirements

See how to contribute

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
    - [ ] Tests for the changes have been added (for bug fixes / features)
    - [ ] Docs have been added / updated (for bug fixes / features)
    - [ ] UPGRADING.md has been updated with a migration guide (for breaking changes)
    - [ ] configuration.schema.json updated if new parameters are added.

What kind of change does this PR introduce?

Fix CI

What is the current behavior?

The create-release step is not able to authenticate with OIDC to properly obtain credentials.

What is the new behavior?

I've moved the permission in the proper section, under the last job. I've also replaced the plugin used to upload file to s3 with the standard s3 commandline

Does this PR introduce a breaking change, and is titled accordingly?

no

Other information

@umbynos umbynos added the topic: infrastructure Related to project infrastructure label Feb 19, 2024
@umbynos umbynos self-assigned this Feb 19, 2024
@umbynos umbynos added the type: imperfection Perceived defect in any part of project label Feb 19, 2024
Copy link

codecov bot commented Feb 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (56691a8) 69.39% compared to head (218c8e4) 69.39%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2546   +/-   ##
=======================================
  Coverage   69.39%   69.39%           
=======================================
  Files         205      205           
  Lines       20162    20162           
=======================================
  Hits        13991    13991           
  Misses       5044     5044           
  Partials     1127     1127           
Flag Coverage Δ
unit 69.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@alessio-perugini alessio-perugini left a comment

Choose a reason for hiding this comment

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

The archive content looks good too.

@umbynos
Copy link
Contributor Author

umbynos commented Feb 20, 2024

Sorry guys for the mess I did

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants