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 enable_namespace_creation flag and start testing with 1.10 #968

Merged

Conversation

ZhengzheYang
Copy link
Contributor

ASM 1.10 has a new flag for install_asm to create the istio-system namespace for the user, if the user opts in.

@comment-bot-dev
Copy link

comment-bot-dev commented Aug 4, 2021

Thanks for the PR! 🚀
✅ Lint checks have passed.

@ZhengzheYang
Copy link
Contributor Author

ZhengzheYang commented Aug 4, 2021

Defaulting to ASM 1.10 for testing.
https://pantheon.corp.google.com/cloud-build/builds;region=global/1b6f4f57-a2ad-4eac-bea0-c1398aa9e940?project=cloud-foundation-cicd

The test has passed for 1.10. Switching back to 1.9

@ZhengzheYang ZhengzheYang force-pushed the enable-namespace branch 5 times, most recently from e18434e to 29ec234 Compare August 5, 2021 00:23
@ZhengzheYang ZhengzheYang changed the title [WIP] add enable_namespace_creation flag and start testing with 1.10 add enable_namespace_creation flag and start testing with 1.10 Aug 5, 2021
@ZhengzheYang ZhengzheYang marked this pull request as ready for review August 5, 2021 01:43
Copy link
Member

@bharathkkb bharathkkb 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 PR @ZhengzheYang

modules/asm/main.tf Outdated Show resolved Hide resolved
@ZhengzheYang ZhengzheYang force-pushed the enable-namespace branch 2 times, most recently from 94a4667 to e0621ed Compare August 5, 2021 17:52
modules/asm/main.tf Outdated Show resolved Hide resolved
modules/asm/variables.tf Outdated Show resolved Hide resolved
@ZhengzheYang ZhengzheYang force-pushed the enable-namespace branch 2 times, most recently from ef93f25 to 76fc426 Compare August 5, 2021 18:44
@bharathkkb bharathkkb merged commit 8764b76 into terraform-google-modules:master Aug 5, 2021
@ZhengzheYang ZhengzheYang deleted the enable-namespace branch August 5, 2021 20:51
CPL-markus pushed a commit to WALTER-GROUP/terraform-google-kubernetes-engine that referenced this pull request Jul 15, 2024
…ogle-modules#968)

* add enable_namespace_creation flag and start testing with 1.10

* address comments
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

4 participants