-
Notifications
You must be signed in to change notification settings - Fork 120
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 an event filter with the resource.DesiredStateChanged predicate to filter status updates out #789
Add an event filter with the resource.DesiredStateChanged predicate to filter status updates out #789
Conversation
f28575f
to
a493060
Compare
ad69663
to
4ba8384
Compare
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 @ulucinar I left two comments that are related to copy-paste.
config/kendra/config.go
Outdated
"github.com/upbound/upjet/pkg/config" | ||
) | ||
|
||
// Configure adds configurations for kafka group. |
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.
// Configure adds configurations for kafka group. | |
// Configure adds configurations for kendra group. |
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 @sergenyalcin, done.
config/medialive/config.go
Outdated
"github.com/upbound/upjet/pkg/config" | ||
) | ||
|
||
// Configure adds configurations for kafka group. |
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.
// Configure adds configurations for kafka group. | |
// Configure adds configurations for medialive group. |
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.
Done.
4ba8384
to
37f8396
Compare
…o filter status updates out Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
37f8396
to
cf56c7c
Compare
Description of your changes
This PR adds an event filter using the resource.DesiredStateChanged to the generated resource controllers so that updates to only labels, annotations (Crossplane's create pending/failed annotations will be ignored) and to the
spec
will result in reconcile requests.I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
Tested with crossplane/upjet#231.