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

Replace deprecated terraform resources #184

Closed
timebertt opened this issue Oct 29, 2020 · 3 comments
Closed

Replace deprecated terraform resources #184

timebertt opened this issue Oct 29, 2020 · 3 comments
Assignees
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/enhancement Enhancement, improvement, extension platform/azure Microsoft Azure platform/infrastructure

Comments

@timebertt
Copy link
Member

How to categorize this issue?

/area quality
/kind enhancement
/priority normal
/platform azure

What would you like to be added:
The deprecated terraform resources network_security_group_id and route_table_id should be replaced by azurerm_subnet_network_security_group_association and azurerm_subnet_route_table_association respectively.

Why is this needed:
When creating an azure Infrastructure terraform gives out the following warnings:

Warning: "network_security_group_id": [DEPRECATED] Use the `azurerm_subnet_network_security_group_association` resource instead.

  on tf/main.tf line 24, in resource "azurerm_subnet" "workers":
  24: resource "azurerm_subnet" "workers" {



Warning: "route_table_id": [DEPRECATED] Use the `azurerm_subnet_route_table_association` resource instead.

  on tf/main.tf line 24, in resource "azurerm_subnet" "workers":
  24: resource "azurerm_subnet" "workers" {
@timebertt timebertt added the kind/enhancement Enhancement, improvement, extension label Oct 29, 2020
@gardener-robot gardener-robot added area/quality Output qualification (tests, checks, scans, automation in general, etc.) related platform/azure Microsoft Azure platform/infrastructure priority/normal labels Oct 29, 2020
@ialidzhikov
Copy link
Member

They should exist both until we upgrade our terraform-provider-azurerm to v2.0.0. We already had a major issue when we removed them in the past.

@dkistner
Copy link
Member

Yep this is already in preparation.
/assign

@ialidzhikov
Copy link
Member

ialidzhikov commented Nov 25, 2020

@dkistner , this can be closed after #192, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/quality Output qualification (tests, checks, scans, automation in general, etc.) related kind/enhancement Enhancement, improvement, extension platform/azure Microsoft Azure platform/infrastructure
Projects
None yet
Development

No branches or pull requests

4 participants