-
Notifications
You must be signed in to change notification settings - Fork 30
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: smoke and sanity integration tests are failing. #2501
Conversation
lpegoraro
commented
Jul 14, 2023
•
edited
Loading
edited
- Fixes behavior when Sinks Partial Update was setting the configuration to null in Redis
- Fixes Delete Event not being read properly due to misconfigured owner id
- Fixes Sink Status After Partial Update not being Unknown.
- Add enhanced error messages in Sink.
- Fixed a wrong validation of custom headers not allowing un-restricted headers
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## develop #2501 +/- ##
===========================================
- Coverage 54.86% 54.84% -0.02%
===========================================
Files 74 74
Lines 5882 5887 +5
===========================================
+ Hits 3227 3229 +2
- Misses 2655 2658 +3
|
…appening when collector stays up even without a sink, wasting resources.
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