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

github-bots: optionally use a pre-defined service account email #598

Merged
merged 4 commits into from
Oct 17, 2024

Conversation

hectorj2f
Copy link
Member

Allow passing a service account instead of always creating a service account.

@hectorj2f hectorj2f self-assigned this Oct 16, 2024
@hectorj2f hectorj2f force-pushed the optional_service_account branch 2 times, most recently from 7984c25 to c537c75 Compare October 16, 2024 18:50
@hectorj2f hectorj2f enabled auto-merge (squash) October 16, 2024 18:53
@hectorj2f hectorj2f force-pushed the optional_service_account branch 2 times, most recently from a9e9be9 to 799c0a5 Compare October 16, 2024 19:07
@hectorj2f hectorj2f force-pushed the optional_service_account branch from 799c0a5 to ac755f1 Compare October 16, 2024 19:16
@hectorj2f hectorj2f requested a review from k4leung4 October 16, 2024 19:21
Signed-off-by: hectorj2f <[email protected]>
@hectorj2f hectorj2f requested a review from k4leung4 October 17, 2024 14:48
@rawlingsj
Copy link
Member

I tested this patch against an existing bot + authorize service and the SA wasn't changed so AFAICT this won't affect existing deployments that use the default created SA.

@rawlingsj
Copy link
Member

@k4leung4 looks ok - you happy to merge?

hectorj2f and others added 2 commits October 17, 2024 19:24
Co-authored-by: Jason Hall <[email protected]>
Signed-off-by: Hector Fernandez <[email protected]>
Signed-off-by: hectorj2f <[email protected]>
Copy link
Contributor

@k4leung4 k4leung4 left a comment

Choose a reason for hiding this comment

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

lets try this
have a feeling this might not work

@hectorj2f hectorj2f merged commit feca306 into chainguard-dev:main Oct 17, 2024
83 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants