Skip to content

ignore SLOW SQL Query in OCR2 chaos tests#22143

Merged
Tofel merged 2 commits intodevelopfrom
slow-query-chaos-test-df1
May 4, 2026
Merged

ignore SLOW SQL Query in OCR2 chaos tests#22143
Tofel merged 2 commits intodevelopfrom
slow-query-chaos-test-df1

Conversation

@Tofel
Copy link
Copy Markdown
Contributor

@Tofel Tofel commented Apr 23, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

✅ No conflicts with other open PRs targeting develop

@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Apr 23, 2026

Static BadgeStatic BadgeStatic BadgeStatic Badge

Failed Test Failure Summary Logs
TestSample The test failed because a condition that should be false was true. Logs ↗︎
TestIntegration_Functions_MultipleV1Requests_ThresholdDecryptionSuccess The test failed without a specific error message, but the logs indicate an issue occurred during the integration test execution. Logs ↗︎

View Full Report ↗︎Docs

@Tofel Tofel marked this pull request as ready for review April 23, 2026 07:13
@Tofel Tofel requested a review from a team as a code owner April 23, 2026 07:13
Copilot AI review requested due to automatic review settings April 23, 2026 07:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Risk Rating: LOW — test-only change that adjusts log-scanning expectations during cleanup.

This PR updates the OCR2 chaos test cleanup log scanning to treat “SLOW SQL QUERY” log lines as expected during chaos (container disruption), preventing the test run from failing due to this known/benign message.

Changes:

  • Add zapcore import to support specifying an allowed log message severity.
  • Extend products.DefaultSettings(...) in TestOCR2Chaos cleanup to allow/ignore "SLOW SQL QUERY" during log scanning.

@Tofel Tofel enabled auto-merge April 23, 2026 09:01
@Tofel Tofel added this pull request to the merge queue May 4, 2026
@cl-sonarqube-production
Copy link
Copy Markdown

@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 4, 2026
@Tofel Tofel added this pull request to the merge queue May 4, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 4, 2026
@Tofel Tofel added this pull request to the merge queue May 4, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 4, 2026
@Tofel Tofel added this pull request to the merge queue May 4, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 4, 2026
@Tofel Tofel added this pull request to the merge queue May 4, 2026
Merged via the queue into develop with commit f4ea1d0 May 4, 2026
133 of 135 checks passed
@Tofel Tofel deleted the slow-query-chaos-test-df1 branch May 4, 2026 18:01
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.

4 participants