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

Re-Add private_cluster_config #669

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

hubatish
Copy link
Collaborator

@hubatish hubatish commented May 14, 2024

After initial PR was reverted, re-adding private_cluster-config setting so that public clusters can be created.

The initial commit had 2 private_cluster_configs defined & clearly didn't work, but I did testing prior to resolving some merge conflicts which I messed up.

Overall goal is allowing public clusters, which are much easier to run kubectl commands with.

@hubatish hubatish force-pushed the main branch 4 times, most recently from fe11d94 to 8fad61a Compare May 14, 2024 20:00
@hubatish hubatish changed the title Fix duplicate private_cluster_config Add private_cluster_config May 14, 2024
@hubatish hubatish changed the title Add private_cluster_config Re-Add private_cluster_config May 14, 2024
@hubatish
Copy link
Collaborator Author

Ran terraform destroy & apply on my existing setup successfully. From blank, terraform apply with private_cluster_config = null in tfvars creates a cluster with "Private cluster" disabled & terraform plan with private_cluster_config = {} from there tries to swap it. Oddly, an already spun up private cluster didn't offer to swap with terraform plan private_cluster_config = null

@annapendleton
Copy link
Collaborator

Ran terraform destroy & apply on my existing setup successfully. From blank, terraform apply with private_cluster_config = null in tfvars creates a cluster with "Private cluster" disabled & terraform plan with private_cluster_config = {} from there tries to swap it. Oddly, an already spun up private cluster didn't offer to swap with terraform plan private_cluster_config = null

oh interesting - I wonder if we should document this behavior as well? IMO it's likely an implementation detail in the modules we rely on for not changing private back to public

@hubatish
Copy link
Collaborator Author

Ran terraform destroy & apply on my existing setup successfully. From blank, terraform apply with private_cluster_config = null in tfvars creates a cluster with "Private cluster" disabled & terraform plan with private_cluster_config = {} from there tries to swap it. Oddly, an already spun up private cluster didn't offer to swap with terraform plan private_cluster_config = null

oh interesting - I wonder if we should document this behavior as well? IMO it's likely an implementation detail in the modules we rely on for not changing private back to public

Sure, added a note in the variables.tf description.

@hubatish hubatish merged commit 70fe82b into GoogleCloudPlatform:main May 16, 2024
5 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.

None yet

2 participants