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

Use deterministic OCR configuration #14775

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

Use deterministic OCR configuration #14775

wants to merge 9 commits into from
+55 −6

Conversation

connorwstein
Copy link
Contributor

@connorwstein connorwstein commented Oct 15, 2024 •
edited
Loading

  • Add support for shared secrets in setting OCR config. Use new libocr deterministic function and add OCR secrets as a piece of env specific deployment config.
  • Backwards compat with existing CCIPConfig contract
  • Add NonceAt for clearing pending txes CLI

@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#4c8aa47

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

Source of Error: 
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9168635Z 2024/10/15 23:45:48 Error re-running flakey tests: push request failed: status=503, body=<html>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9170206Z <head><title>503 Service Temporarily Unavailable</title></head>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9171110Z <body>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9172010Z <center><h1>503 Service Temporarily Unavailable</h1></center>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9173044Z <hr><center>nginx</center>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9173715Z </body>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9174258Z </html>
Flakey Test Detection	Re-run tests	2024-10-15T23:45:48.9185510Z ##[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 investigate server status and load to ensure it can handle incoming requests.

@connorwstein connorwstein marked this pull request as ready for review October 15, 2024 19:39
@connorwstein connorwstein requested review from a team as code owners October 15, 2024 19:39
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
3 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@connorwstein