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

Adopt tflog #245

Closed
1 task done
bendbennett opened this issue Apr 26, 2022 · 2 comments
Closed
1 task done

Adopt tflog #245

bendbennett opened this issue Apr 26, 2022 · 2 comments
Milestone

Comments

@bendbennett
Copy link
Contributor

Terraform CLI and Provider Versions

Terraform v1.1.8
Provider 3.1.3

Use Cases or Problem Statement

Currently, the provider generates logs by using the log package within the standard library.

Proposal

Adopt tflog package.

How much impact is this issue causing?

Low

Additional Information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@bendbennett bendbennett added this to the v3.2.0 milestone Apr 26, 2022
bendbennett added a commit that referenced this issue Apr 26, 2022
@bendbennett
Copy link
Contributor Author

Closing as:

  • The state migration will be going away when the random provider is migrated from sdk/v2 to the framework.
  • The old MigrateState functions do not have the appropriate logging context available from the sdk.
  • tflog will drop logs that don’t have an appropriate logging context until Fallback to Generic Logger Output is done.

Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, 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 May 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant