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.

Sign up for GitHub

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

Jump to bottom

Exclude capabilities tests and mocks from SonarQube coverage #14753

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Exclude capabilities tests and mocks from SonarQube coverage #14753

wants to merge 1 commit into from
+15 −10

Conversation

chudilka1
Copy link
Contributor

No description provided.

@chudilka1 chudilka1 marked this pull request as ready for review October 14, 2024 12:27
@chudilka1 chudilka1 requested a review from a team as a code owner October 14, 2024 12:27
@chudilka1 chudilka1 force-pushed the CAPPL-129-exclude-tests-from-SonarQube branch from 8753f68 to 5be8828 Compare October 14, 2024 16:30
@chudilka1 chudilka1 requested review from a team as code owners October 14, 2024 16:30
@chudilka1 chudilka1 force-pushed the CAPPL-129-exclude-tests-from-SonarQube branch from 5be8828 to ce565a2 Compare October 14, 2024 16:31
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 14, 2024 •
edited
Loading

Below is an analysis created by an LLM. Be mindful of hallucinations and verify accuracy.

WF: CI Core#d27c581

1. External service temporarily unavailable:Flakey Test Detection

Source of Error: 
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7990474Z 2024/10/15 20:56:27 Error re-running flakey tests: push request failed: status=503, body=<html>
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7992538Z <head><title>503 Service Temporarily Unavailable</title></head>
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7993439Z <body>
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7994207Z <center><h1>503 Service Temporarily Unavailable</h1></center>
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7994933Z <hr><center>nginx</center>
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7995383Z </body>
Flakey Test Detection	Re-run tests	2024-10-15T20:56:27.7995731Z </html>

Why: The error occurred due to a 503 Service Temporarily Unavailable response from an external service, likely caused by server downtime or maintenance.

Suggested fix: Retry the request after some time or implement a retry mechanism with exponential backoff to handle such temporary issues automatically.

kalanyuz
kalanyuz previously approved these changes Oct 15, 2024
View reviewed changes
@chudilka1 chudilka1 force-pushed the CAPPL-129-exclude-tests-from-SonarQube branch 2 times, most recently from 1a0d583 to 656da64 Compare October 15, 2024 10:10
ilija42
ilija42 previously approved these changes Oct 15, 2024
View reviewed changes
@chudilka1 chudilka1 force-pushed the CAPPL-129-exclude-tests-from-SonarQube branch 4 times, most recently from 779c63c to d27c581 Compare October 15, 2024 20:39
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@chudilka1 chudilka1 force-pushed the CAPPL-129-exclude-tests-from-SonarQube branch from d27c581 to c4604c5 Compare October 16, 2024 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmank88 jmank88 Awaiting requested review from jmank88

@ilija42 ilija42 Awaiting requested review from ilija42

@kalanyuz kalanyuz Awaiting requested review from kalanyuz

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chudilka1 @kalanyuz @ilija42