Skip to content

Terraform provider configuration errors and role assignment output reference #1276

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

Open
wants to merge 181 commits into
base: main
Choose a base branch
from

Conversation

joseguzman1337
Copy link

Overview/Summary

This Pull Request resolves Terraform configuration issues that caused errors and warnings during terraform plan. It corrects provider-passing logic between the root module and child modules (management, identity, connectivity) and fixes an incorrect resource reference within the role_assignments_for_policy module's outputs. These changes ensure the Terraform configuration plans successfully and adheres to best practices for provider configuration.

This PR fixes/adds/changes/removes

  1. Fix: Corrected the providers block in main.tf for management_resources, identity_resources, and connectivity_resources modules to pass the default azurerm provider instead of attempting to pass a non-existent azurerm.subscription alias. Resolves Error: missing provider provider[".../azurerm"].subscription.
  2. Fix: Updated required_providers blocks within the management, identity, and connectivity child modules to explicitly declare expected provider aliases (e.g., azurerm.management, azurerm.subscription) using configuration_aliases. Resolves Warning: Reference to undefined provider.
  3. Fix: Corrected the resource reference in outputs.tf within the role_assignments_for_policy module to accurately point to the intended azurerm_role_assignment resource or handle its potential absence gracefully. Resolves Error: Reference to undeclared resource.

Breaking Changes

  1. None

Testing Evidence

image

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

mend-bolt-for-github bot and others added 30 commits April 16, 2025 03:06
Bumps [github/super-linter](https://github.com/github/super-linter) from 6 to 7.
- [Release notes](https://github.com/github/super-linter/releases)
- [Changelog](https://github.com/github/super-linter/blob/main/CHANGELOG.md)
- [Commits](github/super-linter@v6...v7)

---
updated-dependencies:
- dependency-name: github/super-linter
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/create-github-app-token](https://github.com/actions/create-github-app-token) from 1 to 2.
- [Release notes](https://github.com/actions/create-github-app-token/releases)
- [Commits](actions/create-github-app-token@v1...v2)

---
updated-dependencies:
- dependency-name: actions/create-github-app-token
  dependency-version: '2'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ossf/scorecard-action](https://github.com/ossf/scorecard-action) from 2.3.1 to 2.4.1.
- [Release notes](https://github.com/ossf/scorecard-action/releases)
- [Changelog](https://github.com/ossf/scorecard-action/blob/main/RELEASE.md)
- [Commits](ossf/scorecard-action@0864cf1...f49aabe)

---
updated-dependencies:
- dependency-name: ossf/scorecard-action
  dependency-version: 2.4.1
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 97a0fba1372883ab732affbe8f94b823f91727db to c24449f33cd45d4826c6702db7e49f7cdb9b551d.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@97a0fba...c24449f)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: c24449f33cd45d4826c6702db7e49f7cdb9b551d
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…ithub/super-linter-7

chore(deps): bump github/super-linter from 6 to 7
…ctions/create-github-app-token-2

chore(deps): bump actions/create-github-app-token from 1 to 2
…ssf/scorecard-action-2.4.1

chore(deps): bump ossf/scorecard-action from 2.3.1 to 2.4.1
…ctions/upload-artifact-c24449f33cd45d4826c6702db7e49f7cdb9b551d

chore(deps): bump actions/upload-artifact from 97a0fba1372883ab732affbe8f94b823f91727db to c24449f33cd45d4826c6702db7e49f7cdb9b551d
Bumps the go_modules group with 2 updates in the /tests/terratest directory: [golang.org/x/crypto](https://github.com/golang/crypto) and [golang.org/x/net](https://github.com/golang/net).


Updates `golang.org/x/crypto` from 0.24.0 to 0.35.0
- [Commits](golang/crypto@v0.24.0...v0.35.0)

Updates `golang.org/x/net` from 0.26.0 to 0.36.0
- [Commits](golang/net@v0.26.0...v0.36.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-version: 0.35.0
  dependency-type: indirect
  dependency-group: go_modules
- dependency-name: golang.org/x/net
  dependency-version: 0.36.0
  dependency-type: indirect
  dependency-group: go_modules
...

Signed-off-by: dependabot[bot] <support@github.com>
…/terratest/go_modules-e862ff11fe

chore(deps): bump the go_modules group across 1 directory with 2 updates
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3.2.1.pre.node20 to 4.6.2.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@c24449f...ea165f8)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-version: 4.6.2
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
joseguzman1337 and others added 29 commits April 20, 2025 21:06
Bumps [google/osv-scanner-action](https://github.com/google/osv-scanner-action) from 1.7.1 to 2.0.1.
- [Release notes](https://github.com/google/osv-scanner-action/releases)
- [Commits](google/osv-scanner-action@1f12429...6fc7144)

---
updated-dependencies:
- dependency-name: google/osv-scanner-action
  dependency-version: 2.0.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.146.0 to 1.233.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb)
- [Commits](ruby/setup-ruby@55283cc...ca041f9)

---
updated-dependencies:
- dependency-name: ruby/setup-ruby
  dependency-version: 1.233.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [step-security/harden-runner](https://github.com/step-security/harden-runner) from 2.11.1 to 2.12.0.
- [Release notes](https://github.com/step-security/harden-runner/releases)
- [Commits](step-security/harden-runner@v2.11.1...v2.12.0)

---
updated-dependencies:
- dependency-name: step-security/harden-runner
  dependency-version: 2.12.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [bridgecrewio/checkov-action](https://github.com/bridgecrewio/checkov-action) from 12.2996.0 to 12.2997.0.
- [Release notes](https://github.com/bridgecrewio/checkov-action/releases)
- [Commits](bridgecrewio/checkov-action@v12.2996.0...v12.2997.0)

---
updated-dependencies:
- dependency-name: bridgecrewio/checkov-action
  dependency-version: 12.2997.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [jfrog/frogbot](https://github.com/jfrog/frogbot) from 2.21.2 to 2.26.0.
- [Release notes](https://github.com/jfrog/frogbot/releases)
- [Commits](jfrog/frogbot@5d9c42c...2d715d2)

---
updated-dependencies:
- dependency-name: jfrog/frogbot
  dependency-version: 2.26.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.146.0 to 1.233.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb)
- [Commits](ruby/setup-ruby@55283cc...ca041f9)

---
updated-dependencies:
- dependency-name: ruby/setup-ruby
  dependency-version: 1.233.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [apisec-inc/ethicalcheck-action](https://github.com/apisec-inc/ethicalcheck-action) from 005fac321dd843682b1af6b72f30caaf9952c641 to 6538d51caea53470bf8018e21f93414a70026f46.
- [Release notes](https://github.com/apisec-inc/ethicalcheck-action/releases)
- [Commits](https://github.com/apisec-inc/ethicalcheck-action/compare/005fac321dd843682b1af6b72f30caaf9952c641...6538d51caea53470bf8018e21f93414a70026f46)

---
updated-dependencies:
- dependency-name: apisec-inc/ethicalcheck-action
  dependency-version: 6538d51caea53470bf8018e21f93414a70026f46
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [sysdiglabs/scan-action](https://github.com/sysdiglabs/scan-action) from 3.2.0 to 3.6.0.
- [Release notes](https://github.com/sysdiglabs/scan-action/releases)
- [Commits](sysdiglabs/scan-action@768d762...d244738)

---
updated-dependencies:
- dependency-name: sysdiglabs/scan-action
  dependency-version: 3.6.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [checkmarx/ast-github-action](https://github.com/checkmarx/ast-github-action) from 2.3.18 to 2.3.19.
- [Release notes](https://github.com/checkmarx/ast-github-action/releases)
- [Commits](Checkmarx/ast-github-action@29f8e68...ef93013)

---
updated-dependencies:
- dependency-name: checkmarx/ast-github-action
  dependency-version: 2.3.19
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
…frog/frogbot-2.26.0

chore(deps): bump jfrog/frogbot from 2.21.2 to 2.26.0
…uby/setup-ruby-1.233.0

chore(deps): bump ruby/setup-ruby from 1.146.0 to 1.233.0
…check-action-6538d51caea53470bf8018e21f93414a70026f46
…apisec-inc/ethicalcheck-action-6538d51caea53470bf8018e21f93414a70026f46

chore(deps): bump apisec-inc/ethicalcheck-action from 005fac321dd843682b1af6b72f30caaf9952c641 to 6538d51caea53470bf8018e21f93414a70026f46
…sysdiglabs/scan-action-3.6.0

chore(deps): bump sysdiglabs/scan-action from 3.2.0 to 3.6.0
…checkmarx/ast-github-action-2.3.19

chore(deps): bump checkmarx/ast-github-action from 2.3.18 to 2.3.19
…google/osv-scanner-action-2.0.1

chore(deps): bump google/osv-scanner-action from 1.7.1 to 2.0.1
…ruby/setup-ruby-1.233.0

chore(deps): bump ruby/setup-ruby from 1.146.0 to 1.233.0
…step-security/harden-runner-2.12.0

chore(deps): bump step-security/harden-runner from 2.11.1 to 2.12.0
…bridgecrewio/checkov-action-12.2997.0

chore(deps): bump bridgecrewio/checkov-action from 12.2996.0 to 12.2997.0
@matt-FFFFFF
Copy link
Member

Hi this PR seems to have lots of files that we don't want in this repo. Please can you refactor?

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