Closed umbynos closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
20b9ba1
) 69.37% compared to head (1e0d26e
) 69.39%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Please check if the PR fulfills these requirements
See how to contribute
- [ ] 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?
infra change
What is the current behavior?
We currently use statically generated credentials to access to s3 buckets.
What is the new behavior?
OpenID Connect allows workflows to exchange short-lived tokens directly from your cloud provider (see here)
Does this PR introduce a breaking change, and is titled accordingly?
no
Other information
AWS_ACCESS_KEY_ID
andAWS_SECRET_ACCESS_KEY
from secretsAWS_ROLE_TO_ASSUME
to secrets