Skip to content
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

NSG gets created in Network Resource Group with no choice of creating in AKS resource group #384

Open
1 task done
stturc opened this issue May 17, 2024 · 1 comment
Open
1 task done
Labels
enhancement New feature or request new Added to an issue when it's new ;)

Comments

@stturc
Copy link
Contributor

stturc commented May 17, 2024

Is your feature request related to a problem? Please describe.

We just began using the code change from #328 and it broke behavior we had been relying on. We had been specifying the vnet_resource_group_name, vnet_name, and subnet_names, but were not specifying nsg_name and wanted it created in the AKS recource group and not in our vnet_resource group. It doesn't look like there is now a way for us to specify where the creation of the NSG takes place.

Describe the solution you'd like

We would like to see an option to create the NSG into the AKS resource group.

Describe alternatives you've considered

No response

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@stturc stturc added enhancement New feature or request new Added to an issue when it's new ;) labels May 17, 2024
@riragh
Copy link
Member

riragh commented May 20, 2024

@stturc, after internal investigation and discussion it was identified that all the network resources should be grouped together, which was not the case with the NSG that was being created when user provided vnet_resource_group_name. Hence the change in PR #328.
As this is the expected behavior, no further changes are expected. If you prefer to create the NSG in AKS resource group, I recommend you fork the repo and make the change here: https://github.com/sassoftware/viya4-iac-azure/blob/main/main.tf#L66
and specify resource_group_name = local.aks_rg.name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request new Added to an issue when it's new ;)
Projects
None yet
Development

No branches or pull requests

2 participants