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

Use OIDC to login to GCR #3838

Merged
merged 1 commit into from May 3, 2023
Merged

Use OIDC to login to GCR #3838

merged 1 commit into from May 3, 2023

Conversation

lucacome
Copy link
Member

@lucacome lucacome commented May 3, 2023

Proposed changes

Similarly too #3740 this removes static credentials and uses OIDC to login to GCR.

@lucacome lucacome self-assigned this May 3, 2023
@lucacome lucacome requested a review from a team as a code owner May 3, 2023 02:12
@github-actions github-actions bot added the chore Pull requests for routine tasks label May 3, 2023
@codecov
Copy link

codecov bot commented May 3, 2023

Codecov Report

Merging #3838 (19c8403) into main (186eec3) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3838      +/-   ##
==========================================
- Coverage   52.43%   52.39%   -0.04%     
==========================================
  Files          59       59              
  Lines       16931    16931              
==========================================
- Hits         8877     8871       -6     
- Misses       7757     7761       +4     
- Partials      297      299       +2     

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@lucacome lucacome merged commit f22a1e2 into main May 3, 2023
60 checks passed
@lucacome lucacome deleted the chore/gcr-login branch May 3, 2023 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Pull requests for routine tasks
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

None yet

3 participants