Skip to content
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

Exclude integration error logs by default. #1816

Merged
merged 2 commits into from
Feb 21, 2024

Conversation

alvarocabanas
Copy link
Contributor

  • Exclude integration error logs by default (Except the one causing to exit with status > 1)
  • Allow including the errors by integration using filters.

@alvarocabanas alvarocabanas requested a review from a team February 20, 2024 18:20
@alvarocabanas alvarocabanas force-pushed the exclude_integration_errors_by_default branch from 36d7751 to f1efb89 Compare February 20, 2024 18:22
…exit with status > 1) but allow including them using filters.
@alvarocabanas alvarocabanas force-pushed the exclude_integration_errors_by_default branch from f1efb89 to 2e8def8 Compare February 20, 2024 18:31
@rogercoll
Copy link
Contributor

@alvarocabanas
Copy link
Contributor Author

Copy link
Contributor

@rogercoll rogercoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alvarocabanas alvarocabanas merged commit 28d2929 into master Feb 21, 2024
23 checks passed
@alvarocabanas alvarocabanas deleted the exclude_integration_errors_by_default branch February 21, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants