Skip to content

Conversation

kota65535
Copy link

@kota65535 kota65535 commented Aug 27, 2025

Fixes #37497

Target Release

1.14.x

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.

Copy link

hashicorp-cla-app bot commented Aug 27, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@kota65535 kota65535 marked this pull request as ready for review August 27, 2025 06:44
@kota65535 kota65535 requested a review from a team as a code owner August 27, 2025 06:44
@crw
Copy link
Contributor

crw commented Sep 22, 2025

Hi @kota65535, thanks for this submission. I reviewed this with @jbardin and the feedback is that the long term plan is to revisit logging more generally. In this specific PR, you are implementing a new feature that is also partially implemented by providers, which is not ideal. Thanks for this submission, we really appreciate your contribution to the larger conversation!

@crw crw closed this Sep 22, 2025
Copy link
Contributor

Changelog Warning

Currently this PR would target a v1.15 release. Please add a changelog entry for in the .changes/v1.15 folder, or discuss which release you'd like to target with your reviewer. If you believe this change does not need a changelog entry, please add the 'no-changelog-needed' label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TF_LOG_PROVIDER_{PROVIDER_NAME} does not work as documented
2 participants