-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Fix aws_glue_job resource attribute #7340
Fix aws_glue_job resource attribute #7340
Conversation
@kterada0509 - you're a rockstar!! Thanks for getting this done so quick - fingers-crossed it will make 0.12 :). |
Why is this still open..? |
When will it be released? |
Ping (and thank you to the author and reviewers)! It's not possible to create |
@bflad - I wonder if anyone from your team can review that? |
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.
@kterada0509 my apologies for the long delay here, and thanks for pushing this change forward. The changes you proposed look good. I left a comment to add the ConflictsWith
property for the old and new attributes, and I would also like to ask that you rebase your branch onto the latest master to fix a number of linting issues.
Cheers!
aws/resource_aws_glue_job.go
Outdated
@@ -24,7 +24,8 @@ func resourceAwsGlueJob() *schema.Resource { | |||
"allocated_capacity": { | |||
Type: schema.TypeInt, | |||
Optional: true, | |||
Default: 10, | |||
Computed: true, | |||
Deprecated: "Please use attribute `max_capacity' instead. This attribute might be removed in future releases.", |
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.
When deprecating an attribute it is important to add ConflictsWith
to the resource schema for both the old and new attributes. More information can be found on our deprecation guide at https://www.terraform.io/docs/extend/best-practices/deprecations.html#renaming-an-optional-attribute
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.
@nywilken
Fix to add ConflictsWith
.
0c47022
to
52c765c
Compare
@nywilken
|
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.
@kterada0509 thanks for the quick updates. This is good to go 👍
* Fix aws_glue_job resource attribute * Add ConflictsWith param for deprecated/new attribute
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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Fixes #7325
Changes proposed in this pull request:
pythonshell
allocated_capacity
attribute to deprecatedallocated_capacity
Output from acceptance testing: