Skip to content

Backport #35110: Don't evaluate providers within overridden modules#35120

Merged
jbardin merged 1 commit intov1.8from
backport-35110
May 6, 2024
Merged

Backport #35110: Don't evaluate providers within overridden modules#35120
jbardin merged 1 commit intov1.8from
backport-35110

Conversation

@jbardin
Copy link
Copy Markdown
Member

@jbardin jbardin commented May 6, 2024

Manual backport of #35110


While we don't normally encounter providers within modules, they are technically still supported, and could exist within a module which has been overridden for testing. Since the module is not being evaluated, we cannot safely evaluate the provider config as variables will not exist within that module.

While we don't normally encounter providers within modules, they are
technically still supported, and could exist within a module which has
been overridden for testing. Since the module is not being evaluated, we
cannot safely evaluate the provider config as variables will not exist
within that module.
@jbardin jbardin merged commit 6fc6258 into v1.8 May 6, 2024
@jbardin jbardin deleted the backport-35110 branch May 6, 2024 20:32
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 6, 2024

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 6, 2024

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 contributions.
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 6, 2024
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.

1 participant