-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
hdinsight
- fixup tests and deprecation for 4.0
#26784
Conversation
fixup deprecation to correctly reflect service change
…to target cleanups via tooling
}, | ||
}, | ||
} | ||
resource.Schema["roles"].Elem.(*pluginsdk.Resource).Schema["kafka_management_node"].Elem.(*pluginsdk.Resource).Schema["username"].Deprecated = "`username` will become Computed only in version 4.0 of the AzureRM Provider as the service auto-generates a value for this property" |
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.
Difficult to tell in this diff, but it looks like line 196-211 can be removed since that's now identical to roles
in the schema further up?
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.
the kafka_management_node
schema changes from SchemaHDInsightNodeDefinition
to SchemaHDInsightNodeDefinitionKafka
in 4.0 - the whole roles
block was processed previously so I kept that the same in the hope it made the change easier to see in the diff (apparently that didn't work out) and to remove later.
79cc573
to
930bdf8
Compare
hdinsight
- fixup tests for 4.0 via ignore_changes
hdinsight
- fixup tests and deprecation for 4.0
…t configured correctly for provisioning private link services. Please set PrivateLinkServiceNetworkPolicies to 'Disabled' on the target subnet.
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.
LGTM!
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. |
also fixes deprecation for
kafka_management_node.0.username
to correctly reflect service change.Community Note
Description
PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Testing
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
hdinsight
- fixup tests for 4.0 viaignore_changes
This is a (please select all that apply):