-
Notifications
You must be signed in to change notification settings - Fork 192
chore: Adds empty values when auto_scaling is not defined #3236
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
base: CLOUDP-308783_plan_modify_replication_specs
Are you sure you want to change the base?
chore: Adds empty values when auto_scaling is not defined #3236
Conversation
…ed in the config and not enabled in the state either
…P-308783_plan_modify_replication_specs_auto_scaling
…r tpf plan modifier
@@ -95,8 +95,16 @@ type rs struct { | |||
|
|||
// ModifyPlan is called before plan is shown to the user and right before the plan is applied. | |||
// Why do we need this? Why can't we use planmodifier.UseStateForUnknown in different fields? | |||
// 1. UseStateForUnknown always copies the state for unknown values. However, that leads to `Error: Provider produced inconsistent result after apply` in some cases (see implementation below). | |||
// 2. Adding the different UseStateForUnknown is very verbose. | |||
// 1. `UseStateForUnknown` always copies the state for unknown values. However, that leads to `Error: Provider produced inconsistent result after apply` in some cases (see implementation below). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Guide for new attributes and update
This PR has gone 7 days without any activity and meets the project’s definition of "stale". This will be auto-closed if there is no new activity over the next 7 days. If the issue is still relevant and active, you can simply comment with a "bump" to keep it open, or add the label "not_stale". Thanks for keeping our repository healthy! |
Description
TestAccClusterFlexCluster_basic
failing, will re-run itLink to any related issue(s): CLOUDP-308783
Type of change:
Required Checklist:
Further comments