-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
feat: Enable ACM feature on hub #722
feat: Enable ACM feature on hub #722
Conversation
Thanks for the PR! 🚀 |
/gcbrun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we need to enable hub on our test project, defined here: https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/blob/master/test/setup/main.tf#L50
Operation "operations/acf.66e7d7f4-f02b-4b8e-8168-cf64d1e1d6fc" finished successfully.
API [gkehub.googleapis.com] not enabled on project [106532254158].
Would you like to enable and retry (this will take a few minutes)?
(y/N)?
ERROR: (gcloud.alpha.container.hub.config-management.enable) PERMISSION_DENIED: GKE Hub has not been used in project 106532254158 before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/gkehub.googleapis.com/overview?project=106532254158 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.
- '@type': type.googleapis.com/google.rpc.Help
links:
- description: Google developers console API activation
url: https://console.developers.google.com/apis/api/gkehub.googleapis.com/overview?project=106532254158
- '@type': type.googleapis.com/google.rpc.ErrorInfo
domain: googleapis.com
metadata:
consumer: projects/106532254158
service: gkehub.googleapis.com
reason: SERVICE_DISABLED
We can also use the "gke-project-asm" which has hub already enabled
|
@morgante @bharathkkb |
@cloud-pharaoh Permissions are also managed in test/setup: https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/blob/master/test/setup/iam.tf |
There was a problem hiding this 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!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one nit that we should specify additional_components
incase user does not have alpha installed
* enable acm * update README * enable gkehub api on test project * Add Hub Admin role to SA * add service account key file * add additional components
No description provided.