10BC0 Add warning when a `cloud` block is present in a child module by SarahFrench · Pull Request #37180 · hashicorp/terraform · GitHub
[go: up one dir, main page]

Skip to content

Conversation

SarahFrench
Copy link
Member
@SarahFrench SarahFrench commented May 29, 2025

Currently users only see warnings about ignored backend blocks in child modules, and there isn't an equivalent warning about cloud blocks. This PR (#29826) introduced the testdata/nested-cloud-warning testing fixtures for this, but that PR doesn't appear to include a test that uses them.

This PR adds the missing warning, and adds a new test asserting the warning is present.

Target Release

N/A

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@SarahFrench SarahFrench marked this pull request as ready for review May 29, 2025 15:42
@SarahFrench SarahFrench requested a review from a team as a code owner May 29, 2025 15:42
@SarahFrench SarahFrench enabled auto-merge (squash) May 29, 2025 15:47
@SarahFrench SarahFrench merged commit ab0947b into main May 29, 2025
13 checks passed
@SarahFrench SarahFrench deleted the sarah/add-submodule-warning-cloud-block branch May 29, 2025 15:47
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contrib 7DF0 utions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0