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

Add provider sensitivity propagation experiment #26706

Merged
merged 1 commit into from
Oct 26, 2020

Conversation

pselle
Copy link
Contributor

@pselle pselle commented Oct 26, 2020

Rolls back marking attributes providers mark as sensitive to an experiment and adds associated docs and adjustments
to the upgrade guide. This is making #26590 experimental, rather than default, and I submitting this for a 0.14 backport.

To propagate sensitivity to attributes providers mark as sensitive, activate the experiment in the terraform block:

terraform {
  experiments = [provider_sensitive_attrs]
}

@codecov
Copy link

codecov bot commented Oct 26, 2020

Codecov Report

Merging #26706 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted Files Coverage Δ
terraform/evaluate.go 53.11% <100.00%> (+0.19%) ⬆️
terraform/node_resource_plan.go 97.19% <0.00%> (+1.86%) ⬆️

@pselle pselle requested a review from a team October 26, 2020 16:24
@pselle pselle force-pushed the pselle/provider-sens-experiment branch from 403118b to 21d117b Compare October 26, 2020 16:26
Copy link
Member

@alisdair alisdair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a neat way to ensure there are no breaking changes! 🎉

I left a suggestion about expanding the upgrade guide a bit, but it's not a must-have. We might want to update the reporter of #26625 to let them know that the behaviour they were looking for will be marked experimental for 0.14.

website/upgrade-guides/0-14.html.markdown Show resolved Hide resolved
@pselle pselle force-pushed the pselle/provider-sens-experiment branch 2 times, most recently from fc1577e to 81c0bb6 Compare October 26, 2020 19:02
Rolls back marking attributes providers mark as sensitive
to an `experiment` and adds associated docs and adjustments
to the upgrade guide.
@ghost
Copy link

ghost commented Nov 26, 2020

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.

@ghost ghost locked as resolved and limited conversation to collaborators Nov 26, 2020
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