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
Improve nats reconciler when subscription filters are modified #13126
Conversation
Skipping CI for Draft Pull Request. |
95d356b
to
d652602
Compare
/test pre-main-kyma-skr-eventing |
2 similar comments
/test pre-main-kyma-skr-eventing |
/test pre-main-kyma-skr-eventing |
✔️ 🥰 Documentation preview ready! 🥰 🔨 Explore the source changes: 0929d5f 🔍 Inspect the deploy log: https://app.netlify.com/sites/kyma-project-docs-preview/deploys/61f11b070feb5c000859fde1 😎 Browse the preview: https://deploy-preview-13126--kyma-project-docs-preview.netlify.app |
|
||
// TestNatsSubAfterSync_FilterRemoved tests the SyncSubscription method | ||
// when a filter is removed from subscription | ||
func TestNatsSubAfterSync_FilterRemoved(t *testing.T) { |
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.
can we have a test with multiple subscriptions?
1 sub will not be modified
1 sub will be modified
both should work as expected before and after modification
if this test already exists, please ignore, i could not find it
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
cfc396d
to
a0fe2c2
Compare
Co-authored-by: Korbinian Stoemmer <github@k15r.dev>
/retest |
/retest |
@mfaizanse: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Description
Related issue(s)