Skip to content

feat: AKS: remove resource group on clean#283

Open
mook-as wants to merge 1 commit into
SUSE:masterfrom
mook-as:aks-cleanup-resource-group
Open

feat: AKS: remove resource group on clean#283
mook-as wants to merge 1 commit into
SUSE:masterfrom
mook-as:aks-cleanup-resource-group

Conversation

@mook-as

@mook-as mook-as commented Sep 15, 2020

Copy link
Copy Markdown
Member

We create a resource group in AKS on deployment; for mirroring, we should also remove it when cleaning up.

We create a resource group in AKS on deployment; for mirroring, we
should also remove it when cleaning up.
@mook-as mook-as added the type: enhancement New feature or request label Sep 15, 2020
@mook-as mook-as requested a review from viccuad September 15, 2020 17:39

@viccuad viccuad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure we need to remove the resource group.

From cap-terraform's readme, the resource group is a precondition for terraform, and it is personal:
https://github.com/SUSE/cap-terraform/blob/8831476c6c81758c54c162d780c5fcf039514b58/aks/README.md

This resource group will contain all resources of, and in support of your SUSE CAP cluster. Make note of the name and location which must be supplied in terraform variables.

And it seems that terraform doesn't create it either (following var https://github.com/SUSE/cap-terraform/blob/8831476c6c81758c54c162d780c5fcf039514b58/aks/variables.tf.json#L19).

So I see it as auth setup for the user, and in that case I think we should let the user decide when and how the resource group will be deleted, as it can be reused between deployments.

@mook-as

mook-as commented Sep 16, 2020

Copy link
Copy Markdown
Member Author

The problem was that, if we don't remove the resource group, the next deployment fails for me (because it already exists).

I'll try again later to get the exact error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants