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

Revert "chore: decommission push-gha-metrics-action" #14782

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Revert "chore: decommission push-gha-metrics-action" #14782

wants to merge 2 commits into from

Conversation

erikburt
Copy link
Collaborator

These metrics were preemptively removed. This adds them back for the time being.

Changes

@erikburt erikburt self-assigned this Oct 15, 2024
@erikburt erikburt requested review from a team as code owners October 15, 2024 21:19
@github-actions GitHub Actions
Copy link
Contributor

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

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

WF: CI Core#d5bfead

External service unavailable during test re-run:[Flakey Test Detection]

Source of Error: 
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6197638Z 2024/10/16 14:52:27 Error re-running flakey tests: push request failed: status=503, body=<html>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6200130Z <head><title>503 Service Temporarily Unavailable</title></head>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6201344Z <body>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6212098Z <center><h1>503 Service Temporarily Unavailable</h1></center>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6213512Z <hr><center>nginx</center>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6214445Z </body>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6215010Z </html>
Flakey Test Detection	Re-run tests	2024-10-16T14:52:27.6230349Z ##[error]Process completed with exit code 1.

Why: The error occurred because an external service was temporarily unavailable, as indicated by the HTTP 503 status code returned by the server.

Suggested fix: Implement retry logic with exponential backoff in the test runner to handle temporary unavailability of external services.

@erikburt erikburt force-pushed the revert/push-gha-metrics-action-removal branch from d5bfead to e84f019 Compare October 21, 2024 20:09
@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@momentmaker momentmaker momentmaker approved these changes

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

Assignees

@erikburt erikburt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@erikburt @momentmaker