-
Notifications
You must be signed in to change notification settings - Fork 363
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
enhance existing Glue resources #1380
Conversation
@wotolom can you cleanup the examples ? |
Hi @haarchri, are the examples ok now? |
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.
Thanks for this enhancement - the only open part are the fields with breaking changes for folks using glue stuff - for now it would be betternot changing so many existing fields - what do you think ? We can do this with migration hook support in the future
Thanks for the review! About the breaking changes: However I'm of course a bit biased. (contributing and future use) ;) |
@wotolom sounds great - can you add all breaking changes for release notes that would be great |
Signed-off-by: wotolom <car_929@yahoo.de> fix! some fieldnames Signed-off-by: wotolom <car_929@yahoo.de>
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.
thanks for Implementation - tested all glue resources
Description of your changes
changes for the existing AWS Glue resources SecurityConfiguration, Database, Connection, Classifier, Crawler and Job
Fixes #1304
Add
lateInitialize
,isUpToDate
andupdate
functionality where applicableAdd missing fields and referencers
StyleFix! several fieldnames
While adding missing fields and referencers/selectors, the opportunity arose to harmonize several existing fieldnames.
Here are these Breaking Changes:
roleArn
torole
(same forroleRef
,roleSelector
)cloudWatchEncryption
,jobBookmarksEncryption
,s3Encryption
):kmsKeyARN
tokmsKeyArn
(addkmsKeyArnRef
,kmsKeyArnSelector
)securityGroupIDList
tosecurityGroupIdList
(same forsecurityGroupIdRefs
,securityGroupIdSelector
)subnetID
tosubnetId
(same forsubnetIdRef
,subnetIdSelector
)roleArn
torole
(same forroleRef
,roleSelector
)s3Targets
:dlqEventQueueARN
todlqEventQueueArn
(adddlqEventQueueArnRef
/dlqEventQueueArnSelector
)eventQueueARN
toeventQueueArn
(addeventQueueArnRef
/eventQueueArnSelector
)People, who already used any of these Glue resources in an older crossplane version, would need to change the mentioned fieldnames in their Claims (XRCs), Compositions, XRDs etc.
Notes:
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
All 6 resources were tested manually (create, update, delete) with the objects in examples/glue.