-
Notifications
You must be signed in to change notification settings - Fork 0
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
DEVOPS-12289: Move OIDC config to the open-turo #16
base: main
Are you sure you want to change the base?
DEVOPS-12289: Move OIDC config to the open-turo #16
Conversation
a4b5723
to
d7d130f
Compare
d7d130f
to
a3c6170
Compare
a3c6170
to
c43be76
Compare
BREAKING CHANGE: remove aws cli installation and credential configuration from consumer workflow
c43be76
to
edefcec
Compare
Release notes previewBelow is a preview of the release notes if your PR gets merged. 2.0.0 (2024-12-13)⚠ BREAKING CHANGES
Features
Continuous Integration
Breaking changes file
|
@@ -138,6 +138,12 @@ This action is a `composite` action. | |||
# Required: false | |||
# Default: ${{ github.sha }}-${{ github.run_number }}-${{ github.run_attempt }} | |||
|
|||
role-to-assume-with-oidc: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this just be role-to-assume
? And specify in the docs that it's expected that the agents where this run have the permissions to assume this role (using the OIDC or other means)?
- name: Install aws cli | ||
run: | | ||
pip install awscli==1.33.21 | ||
aws --version | ||
shell: bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this required? Or should we let the consumer decide how to install / if they need to install the AWS CLI?
Pinning the version here doesn't seem ok (what if a consumer wants to use v2 of the CLI?)
Description
Move OIDC config to open-turo
Part of the OIDC configuration has been moved to open-turo. This change enable developers to use the OIDC configuration in multiple actions without duplicating the configuration.
Centralizing setup logic in the reusable workflow is convenient and ensures consistency. It’s a trade-off between encapsulation (all logic in the reusable workflow) and optimization (doing setup only once from the consumer workflow).
The workflow has been tested in https://github.com/turo/platform-scratch-infrastructure/pull/176
Fixes #DEVOPS-12289
Changes
🚀 PR created with fotingo