feat(asm): Adding CA option without providing CA_CERT maps and adding revision_name flag #952
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…t providing the four maps.
christopher-hendrich-sada
requested review from
bharathkkb,
Jberlinsky and
a team
as code owners
adding missed variable in line 207 and 211
correcting REVISION_NAME_COMMAND_SNIPPET variable in line 211
|
Thanks for the PR! 🚀 |
morgante
changed the title
Ca and rev name
Fixed a missing bracket
… --ca_certs flag, to use self-signed certificate
bharathkkb
requested changes
Jul 20, 2021
There was a problem hiding this comment.
Thanks for the PR @christopherhendrich
Some lint related issues #952 (comment) here
I assume we can also remove the bash CA_COMMAND* since with validation only all or none will be allowed?
|
Hi @bharathkkb, In regards to the CA_COMMAND, do you mean line 182-186 in install_asm.sh? I think that is still needed, because you want to be able to run just the '--ca citadel' flag or the flag and the ca_certs maps or no --ca flag to use meshca. |
bharathkkb
reviewed
Jul 21, 2021
…g {} around single digits numbers for line 24-32, adding echos for CA variables.
…aform-google-kubernetes-engine into ca-and-rev-name
bharathkkb
approved these changes
Jul 23, 2021
CPL-markus
pushed a commit
to WALTER-GROUP/terraform-google-kubernetes-engine
that referenced
this pull request
Jul 15, 2024
…n_name flag (terraform-google-modules#952) * modifying CA config options * Adding flag as an option and adding the ability for to be used without providing the four maps. * adding missed variable in line 207 and 211 * correcting REVISION_NAME_COMMAND_SNIPPET variable in line 211 * updated revision_label process * Made every CA CERTS map its own command snippet * Made every CA CERTS map its own command snippet * testing validations * testing validations * Fixed a missing bracket * Reverting * added revision_name flag and included option to run --ca flag without --ca_certs flag, to use self-signed certificate * Fixed whitespacing * removing another whitespace * ran make build * fixed Error: No newline at end of file ./modules/asm/variables.tf * testing * ran make build * In install_asm.sh: Consolidating IF statement for CA_CERT keys, adding {} around single digits numbers for line 24-32, adding echos for CA variables. Co-authored-by: Bharath KKB <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is to align the tf module with the install_asm script as used for Citadel to MeshCA in the official documentation: https://cloud.google.com/service-mesh/docs/ca-migration
It adds the --revision_name flag, so we can deploy a second ASM control plane of the same ASM version using a unique revision name.
It also lets you deploy ASM without specifying the maps for --ca_certs. If you only specify, --ca citadel, it will now deploy successfully.