Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(workflows/pr-review-companion): set environment on job #38383

Merged
merged 1 commit into from
Feb 28, 2025
Merged

fix(workflows/pr-review-companion): set environment on job #38383

merged 1 commit into from
Feb 28, 2025

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Feb 28, 2025

Description

Fixes an issue with GCP authentication in the pr-review-companion workflow.

Motivation

This run failed:

"Permission 'iam.serviceAccounts.getAccessToken' denied on resource (or it may not exist)."

Additional details

Related issues and pull requests

Follow-up to:

@caugner caugner requested a review from a team as a code owner February 28, 2025 18:54
@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/xs [PR only] 0-5 LoC changed labels Feb 28, 2025
caugner
caugner commented Feb 28, 2025
View reviewed changes
Copy link
Contributor Author

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Self-approval.

@caugner caugner merged commit 914dfdd into main Feb 28, 2025
10 of 11 checks passed
@caugner caugner deleted the MP-1889-set-review-environment branch February 28, 2025 18:55
@caugner caugner mentioned this pull request Feb 28, 2025
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size/xs [PR only] 0-5 LoC changed system [PR only] Infrastructure and configuration for the project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@caugner