-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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.
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
bump geth to v1.14.12 #15375
base: develop
Are you sure you want to change the base?
bump geth to v1.14.12 #15375
Conversation
AER Report: CI Coreaer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Core Tests (go_core_tests) , test-scripts , Core Tests (go_core_tests_integration) , GolangCI Lint (core/scripts) , Core Tests (go_core_ccip_deployment_tests) , GolangCI Lint (.) , Core Tests (go_core_fuzz) , GolangCI Lint (integration-tests/load) , Core Tests (go_core_race_tests) , GolangCI Lint (integration-tests) , GolangCI Lint (deployment) , lint , SonarQube Scan 1. Git diff detected changes:Job ID: Ensure clean after generate Source of Error:
Why: The Suggested fix: Ensure that the generate step does not introduce any changes or commit the changes before running the 2. TestLogPoller_PollAndSaveLogs failed:Job ID: Run tests Source of Error:
Why: The Suggested fix: Investigate the test cases to ensure that the log poller correctly handles finality depth and chain finality scenarios. Adjust the test setup or the log poller implementation as needed to pass the tests. 3. TestLogPoller_PollAndSaveLogsDeepReorg failed:Job ID: Run tests Source of Error:
Why: The Suggested fix: Investigate the test cases to ensure that the log poller correctly handles deep reorganization scenarios. Adjust the test setup or the log poller implementation as needed to pass the tests. AER Report: Operator UI CI ran successfully ✅ |
I thought that CI was just flaking, but this appears to legitimately break some log poller tests 🤔 |
0b92efb
to
425d984
Compare
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Flakeguard SummaryRan new or updated tests between View Flaky Detector Details | Compare Changes Found Flaky Tests ❌
ArtifactsFor detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json. |
Flakeguard SummaryRan new or updated tests between View Flaky Detector Details | Compare Changes Found Flaky Tests ❌
ArtifactsFor detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json. |
Quality Gate passedIssues Measures |
It looks like
params.Version
was removed, so we will have to find a work around.