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.

Sign up for GitHub

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

Jump to bottom

fix: use provided service_account_name if available #1610

Merged
merged 4 commits into from
Apr 26, 2023
Merged

fix: use provided service_account_name if available #1610

merged 4 commits into from
Apr 26, 2023

Conversation

griseau
Copy link
Contributor

@griseau griseau commented Apr 17, 2023

Fixes #1609

@griseau griseau requested review from a team, Jberlinsky and ericyz as code owners April 17, 2023 08:40
@griseau griseau mentioned this pull request Apr 17, 2023
Merged
@apeabody
Copy link
Contributor

/gcbrun

@griseau
Copy link
Contributor Author

griseau commented Apr 17, 2023

Hey @apeabody,

I see that terraform-google-kubernetes-engine-int-trigger action failed but I don't have the possibility to see the reason it's failing, so can't fix it :/

@apeabody
Copy link
Contributor

Hey @apeabody,

I see that terraform-google-kubernetes-engine-int-trigger action failed but I don't have the possibility to see the reason it's failing, so can't fix it :/

Thanks @griseau - It appeared to be a flaky test run, no issues on the second run.

apeabody
apeabody requested changes Apr 17, 2023
View reviewed changes
Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @griseau!

Can you please run make build to build the terraform and docs.

autogen/main/sa.tf.tmpl Outdated Show resolved Hide resolved
@apeabody
Copy link
Contributor

/gcbrun

@apeabody
Copy link
Contributor

/gcbrun

@griseau
Copy link
Contributor Author

griseau commented Apr 20, 2023

Hey @apeabody,
Anything else required here ? Would we be able to have it in the next release ? :)

apeabody
apeabody approved these changes Apr 26, 2023
View reviewed changes
Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @griseau!

@apeabody apeabody merged commit a42ed88 into terraform-google-modules:master Apr 26, 2023
4 checks passed
@apeabody
Copy link
Contributor

Hey @apeabody, Anything else required here ? Would we be able to have it in the next release ? :)

Yup, will be included in the next release.

@michaellzc michaellzc mentioned this pull request Nov 23, 2023
Merged
@yorik yorik mentioned this pull request May 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apeabody apeabody apeabody approved these changes

@Jberlinsky Jberlinsky Awaiting requested review from Jberlinsky

@ericyz ericyz Awaiting requested review from ericyz ericyz is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Service account created doesn't have the correct name
2 participants
@griseau @apeabody