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

Updating tools to newer versions #591

Merged
merged 5 commits into from
Jan 5, 2022
Merged

Conversation

Chambras
Copy link
Contributor

Description

Just a small maintenance update to upgrade the terraform tools to newer versions.

  • Upgrade azurerm provider to 2.90.0
  • Upgrade tflint to 0.34.0
  • Upgrade tflint azurerm ruleset to 0.14.0

Issue reference

NA

Checklist

Please make sure you've completed the relevant tasks for this PR out of the following list:

  • All acceptance criteria in the backlog item are met
  • The documentation is updated to cover any new or changed features
  • Manual tests have passed
  • Relevant issues are linked to this PR

Copy link
Contributor

@brooke-hamilton brooke-hamilton left a comment

Choose a reason for hiding this comment

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

@Chambras thank you! ❤️

@brooke-hamilton brooke-hamilton enabled auto-merge (squash) December 22, 2021 13:47
@brooke-hamilton brooke-hamilton self-assigned this Dec 22, 2021
@brooke-hamilton
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@glennmusa
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@glennmusa
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@brooke-hamilton
Copy link
Contributor

@Chambras I'm testing with Azure Cloud Shell on Azure Cloud (Azure commercial). I get the error below, which is the same as issue #528. But that was an issue we were previously seeing only in Azure US Government. Any ideas on this? I'm thinking we would want to merge the PR anyway if it's the same bug, but I want to get your input. (BTW my deployment from the dev container is working as expected.)

brooke@Azure:~/missionlz/src/terraform/mlz$ terraform apply
var.hub_subid
  Subscription ID for the Hub deployment

  Enter a value: < my subscription > 

╷
│ Warning: Provider azurerm is undefined
│
│   on main.tf line 221, in module "hub-network":
│  221:   providers  = { azurerm = azurerm.hub }
│
│ Module module.hub-network does not declare a provider named azurerm.
│ If you wish to specify a provider configuration for the module, add an entry for azurerm in the required_providers block within the module.
│
│ (and 11 more similar warnings elsewhere)
╵
╷
│ Error: expected "access_policy.0.object_id" to be a valid UUID, got
│
│   with module.jumpbox[0].azurerm_key_vault.jumpbox-keyvault,
│   on ../modules/jumpbox/main.tf line 18, in resource "azurerm_key_vault" "jumpbox-keyvault":
│   18:     object_id = var.object_id
│
╵
brooke@Azure:~/missionlz/src/terraform/mlz$ terraform version
Terraform v1.0.11
on linux_amd64
+ provider registry.terraform.io/hashicorp/azurerm v2.83.0
+ provider registry.terraform.io/hashicorp/random v3.1.0
+ provider registry.terraform.io/hashicorp/time v0.7.2

@Chambras
Copy link
Contributor Author

Chambras commented Jan 5, 2022

@Chambras I'm testing with Azure Cloud Shell on Azure Cloud (Azure commercial). I get the error below, which is the same as issue #528. But that was an issue we were previously seeing only in Azure US Government. Any ideas on this? I'm thinking we would want to merge the PR anyway if it's the same bug, but I want to get your input. (BTW my deployment from the dev container is working as expected.)

brooke@Azure:~/missionlz/src/terraform/mlz$ terraform apply
var.hub_subid
  Subscription ID for the Hub deployment

  Enter a value: < my subscription > 

╷
│ Warning: Provider azurerm is undefined
│
│   on main.tf line 221, in module "hub-network":
│  221:   providers  = { azurerm = azurerm.hub }
│
│ Module module.hub-network does not declare a provider named azurerm.
│ If you wish to specify a provider configuration for the module, add an entry for azurerm in the required_providers block within the module.
│
│ (and 11 more similar warnings elsewhere)
╵
╷
│ Error: expected "access_policy.0.object_id" to be a valid UUID, got
│
│   with module.jumpbox[0].azurerm_key_vault.jumpbox-keyvault,
│   on ../modules/jumpbox/main.tf line 18, in resource "azurerm_key_vault" "jumpbox-keyvault":
│   18:     object_id = var.object_id
│
╵
brooke@Azure:~/missionlz/src/terraform/mlz$ terraform version
Terraform v1.0.11
on linux_amd64
+ provider registry.terraform.io/hashicorp/azurerm v2.83.0
+ provider registry.terraform.io/hashicorp/random v3.1.0
+ provider registry.terraform.io/hashicorp/time v0.7.2

Hi @brooke-hamilton. Yes, it is the same bug. I did not get that because I use a service principal for all my deployments so if you test in cloud shell using a SP, it should work as well.

@brooke-hamilton brooke-hamilton merged commit c7ad427 into Azure:main Jan 5, 2022
@Chambras Chambras deleted the UpdateTools branch September 20, 2022 16:20
jwaltireland pushed a commit to ARPA-H/AzureMissionLZ that referenced this pull request Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants