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

RMN home reader test #14789

Merged
merged 29 commits into from
Oct 21, 2024
Merged

RMN home reader test #14789

merged 29 commits into from
Oct 21, 2024

Conversation

0xnogo
Copy link
Collaborator

@0xnogo 0xnogo commented Oct 16, 2024 •
edited
Loading

Added a RMNHome reader test in capabilities

@github-actions GitHub Actions
Copy link
Contributor

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

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

WF: CI Core#8afea0c

1. External service temporarily unavailable:Flakey Test Detection

Source of Error: 
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8667531Z 2024/10/16 11:08:06 Error re-running flakey tests: push request failed: status=503, body=<html>
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8669562Z <head><title>503 Service Temporarily Unavailable</title></head>
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8673183Z <body>
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8675035Z <center><h1>503 Service Temporarily Unavailable</h1></center>
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8675892Z <hr><center>nginx</center>
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8676455Z </body>
Flakey Test Detection	Re-run tests	2024-10-16T11:08:06.8676841Z </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: Implement retry logic with exponential backoff or check the service status and try the request at a later time.

@0xnogo 0xnogo changed the title rmn home reader test RMN home reader test Oct 16, 2024
@0xnogo 0xnogo marked this pull request as ready for review October 16, 2024 09:38
@0xnogo 0xnogo requested review from a team as code owners October 16, 2024 09:38
makramkd
makramkd previously approved these changes Oct 16, 2024
View reviewed changes
makramkd
makramkd previously approved these changes Oct 17, 2024
View reviewed changes
@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
0.0% Duplication on New Code

See analysis details on SonarQube

@0xnogo 0xnogo added this pull request to the merge queue Oct 21, 2024
Merged via the queue into develop with commit 5dfb130 Oct 21, 2024
147 checks passed
@0xnogo 0xnogo deleted the ng/rmn-home-test branch October 21, 2024 16:38
This was referenced Oct 21, 2024
Draft
Draft
Draft
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@makramkd makramkd makramkd approved these changes

@jmank88 jmank88 jmank88 approved these changes

@winder winder Awaiting requested review from winder winder is a code owner automatically assigned from smartcontractkit/ccip-offchain

@rstout rstout Awaiting requested review from rstout rstout is a code owner automatically assigned from smartcontractkit/ccip-offchain

@asoliman92 asoliman92 Awaiting requested review from asoliman92 asoliman92 is a code owner automatically assigned from smartcontractkit/ccip-offchain

@dimkouv dimkouv Awaiting requested review from dimkouv dimkouv is a code owner automatically assigned from smartcontractkit/ccip-offchain

@mateusz-sekara mateusz-sekara Awaiting requested review from mateusz-sekara mateusz-sekara is a code owner automatically assigned from smartcontractkit/ccip-offchain

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
@0xnogo @jmank88 @makramkd