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

Add infra for deploy-gke CI workflows #65

Closed

Conversation

JeromeJu
Copy link
Contributor

This commit adds the infra variables for setting up the deploy-gke CI workflow inputs.

@JeromeJu
Copy link
Contributor Author

JeromeJu commented Apr 22, 2024

One question I want to confirm with is that we do not essentially need to have the subnetwork set up as in https://github.com/google-github-actions/test-infra/blob/main/project_get-gke-credentials.tf#L108-L125 IIUC (since that is to mock client network use case through a private cluster?).

project_deploy-gke.tf Outdated Show resolved Hide resolved
@sethvargo
Copy link
Member

One question I want to confirm with is that we do not essentially need to have the subnetwork set up as in https://github.com/google-github-actions/test-infra/blob/main/project_get-gke-credentials.tf#L108-L125 IIUC (since that is to mock client network use case through a private cluster?).

correct

@JeromeJu JeromeJu force-pushed the deploy-gke-infra branch 3 times, most recently from 6a6acea to e02621c Compare April 25, 2024 21:21
This commit adds the infra variables for setting up the deploy-gke CI
workflow inputs.
Copy link
Member

@sethvargo sethvargo left a comment

Choose a reason for hiding this comment

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

Seems reasonable - do you want me to try and apply these changes?

@JeromeJu
Copy link
Contributor Author

Seems reasonable - do you want me to try and apply these changes?

Thanks for your time! That would be much appreciated.

@sethvargo sethvargo closed this in #66 May 1, 2024
sethvargo added a commit that referenced this pull request May 1, 2024
Closes #65

Already applied

---------

Co-authored-by: JeromeJu <jeromeju@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants