chore(docs): Add NOTE to direct ASM users using GKEHub TF Module

[wonjekang]

Currently, ASM Submodule is not supported / managed anymore by ASM team. Highly recommending customers using GKEHub TF module to install ASM.

[bharathkkb]

@wonjekang @djazayeri In this case could we remove the sub-module and add this note in our migration guide for next breaking release?

[wonjekang]

@bharathkkb I think adding this context in the migration guide is a good idea. Do we have a timeline to have it? However, we still have ASM customers depend on this ASM submodule and we cannot remove it at this time. Let's keep this change at this time.

[bharathkkb]

I think adding this context in the migration guide is a good idea. Do we have a timeline to have it?

If we are recommending to switch to the new resources, co-locating the migration guide in the PR would be good. Otherwise let's make it a fast follow.

We still have ASM customers depend on this ASM submodule and we cannot remove it at this time. Let's keep this change at this time.

Even if we remove the module, users can still use continue to use released versions. Only new module releases will not have this submodule.

[wonjekang]

@bharathkkb Could you share the instruction to write the migration guide? I don't have any context where I can start from. IIUC, your recommendation in this PR is:

1. Remove the ASM submodule package.
2. Add the context to use gkehub resources for ASM installation in migration guide document(?)

[bharathkkb]

@wonjekang Yeah for 1 and 2. Example guides are in https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/tree/master/docs and next major version would be 31. Here is a previous ASM example https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/blob/master/docs/upgrading_to_v20.0.md#upgrading-to-v200

[apeabody]

Thanks for the contribution @wonjekang!

Per the PR discussion in this (breaking) PR can you please remove the module and include a migration guide in docs/upgrading_to_v32.0.md

[github-actions]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

[apeabody]

Hi @wonjekang - Let's retool this as a deprecation message for v34, and remove entirely in a future version.