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

remove workflowID from messageID due to gateway size limit #14737

Merged
merged 3 commits into from
Oct 16, 2024
Merged

remove workflowID from messageID due to gateway size limit #14737

merged 3 commits into from
Oct 16, 2024
Rate limit · GitHub

Access has been restricted

You have triggered a rate limit.

Please wait a few minutes before you try again;
in some cases this may take up to an hour.

Conversation

jinhoonbang
Copy link
Contributor

remove workflowID from messageID due to gateway size limit - MessageIdMaxLen = 128 (code)

@jinhoonbang jinhoonbang requested review from a team as code owners October 11, 2024 16:02
@github-actions GitHub Actions
Copy link
Contributor

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

WF: CI Core#3ac4b24

No errors found in this run. 🎉

@github-actions GitHub Actions
Copy link
Contributor

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

WF: CI Core#ec61416

No errors found in this run. 🎉

@jinhoonbang jinhoonbang added this pull request to the merge queue Oct 15, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 15, 2024
@jinhoonbang jinhoonbang added this pull request to the merge queue Oct 15, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 15, 2024
@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#cc3a88a

1. HTTP 503 Service Temporarily Unavailable error during push request:[Flakey Test Detection]

Source of Error: 
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3777984Z 2024/10/15 19:45:55 Error re-running flakey tests: push request failed: status=503, body=<html>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3779887Z <head><title>503 Service Temporarily Unavailable</title></head>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3784172Z <body>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3785956Z <center><h1>503 Service Temporarily Unavailable</h1></center>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3787014Z <hr><center>nginx</center>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3788536Z </body>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3789711Z </html>
Flakey Test Detection	Re-run tests	2024-10-15T19:45:55.3804045Z ##[error]Process completed with exit code 1.

Why: The error occurred because the server handling the push request was temporarily unavailable, likely due to maintenance or overload, indicated by the HTTP 503 status code.

Suggested fix: Retry the request after some time or implement a retry mechanism in the workflow to handle temporary server unavailability automatically.

@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

@jinhoonbang jinhoonbang added this pull request to the merge queue Oct 15, 2024
github-merge-queue bot pushed a commit that referenced this pull request Oct 15, 2024
@jinhoonbang
remove workflowID from messageID due to gateway size limit (#14737)
64cf1a3
github-merge-queue bot pushed a commit that referenced this pull request Oct 15, 2024
@jinhoonbang
remove workflowID from messageID due to gateway size limit (#14737)
a22c6dd
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 15, 2024
@jinhoonbang jinhoonbang added this pull request to the merge queue Oct 15, 2024
@kalverra kalverra removed this pull request from the merge queue due to a manual request Oct 16, 2024
@jinhoonbang jinhoonbang added this pull request to the merge queue Oct 16, 2024
Merged via the queue into develop with commit 0466e42 Oct 16, 2024
126 of 129 checks passed
@jinhoonbang jinhoonbang deleted the fix-for-http-target branch October 16, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DavidOrchard DavidOrchard

@justinkaseman justinkaseman

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

Successfully merging this pull request may close these issues.

None yet
3 participants
@jinhoonbang @DavidOrchard @justinkaseman