feat: Enable ACM feature on hub

[cloud-pharaoh]

No description provided.

[comment-bot-dev]

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

[morgante]

/gcbrun

[morgante]

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

[bharathkkb]

We can also use the "gke-project-asm" which has hub already enabled

terraform-google-kubernetes-engine/test/setup/main.tf

Line 78 in c8096a4

module "gke-project-asm" {

[cloud-pharaoh]

@morgante @bharathkkb
I opted for adding enabling the hub API on project 2. I see a failure because of permissions. I need to grant the Hub Admin permission to the terraform SA. How can I do that?

[morgante]

@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

[bharathkkb]

Thanks for the PR!

[bharathkkb]

LGTM, just one nit that we should specify additional_components incase user does not have alpha installed