-
Notifications
You must be signed in to change notification settings - Fork 1.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: consider empty results set from new relic #5621
fix: consider empty results set from new relic #5621
Conversation
Signed-off-by: cyrilico <19289022+cyrilico@users.noreply.github.com>
Signed-off-by: cyrilico <19289022+cyrilico@users.noreply.github.com>
Signed-off-by: cyrilico <19289022+cyrilico@users.noreply.github.com>
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! Thanks for the fix ❤️
/run-e2e new_relic |
@JorTurFer something seemed to go wrong with provisioning for the tests, can you run them again? |
Could you resolve the changelog conflicts please? |
/run-e2e newrelic |
@cyrilico Mind fixing the conflict please? |
Signed-off-by: damas <19289022+cyrilico@users.noreply.github.com>
Head branch was pushed to by a user without write access
@tomkerkhove @JorTurFer fixed 🙏 |
/run-e2e newrelic |
Handle empty results set returned by New Relic query executor.
Checklist
Fixes #5619