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.

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

Add missing backend.Commit() to log emitter deployment #15483

Closed
wants to merge 2 commits into from

Conversation

reductionista
Copy link
Contributor

@reductionista reductionista commented Dec 3, 2024

There have been a few LogPoller flakes lately due to this. Looks like calling Commit() between each call to a gethwrapper function is required for the latest version of simulated geth to avoid a race where it can send two tx's in a row with the same nonce before it's incremented.

BCFR-1081

This is required for the latest version of simulated geth to avoid a race
Copy link
Contributor

github-actions bot commented Dec 3, 2024

AER Report: CI Core

aer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Flakeguard Root Project / Get Tests To Run , lint , Core Tests (go_core_tests) , Flakeguard Deployment Project , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_race_tests) , Core Tests (go_core_fuzz) , Flakeguard Root Project / Run Tests (github.com/smartcontractkit/chainlink/v2/core/chains/evm/logpoller, ubuntu-latest) , Flakey Test Detection , SonarQube Scan , Flakeguard Root Project / Report

1. Error mapping test results to owners: open .github/CODEOWNERS: no such file or directory:[Set combined test results]

Source of Error:
2024/12/03 05:16:40 Error mapping test results to owners: open .github/CODEOWNERS: no such file or directory
**Why**: The error occurred because the script attempted to open the `.github/CODEOWNERS` file, which does not exist in the repository.

Suggested fix: Ensure that the .github/CODEOWNERS file is present in the repository. If the file is not needed, modify the script to skip this step.

2. Encountered test failures:[Run tests]

Source of Error:
2024-12-03T05:05:14.9760867Z Encountered test failures.
2024-12-03T05:05:14.9761306Z go_core_tests exiting with code 1
**Why**: The test suite encountered one or more test failures, causing the process to exit with a non-zero status.

Suggested fix: Review the test results to identify and fix the failing tests. Ensure that all dependencies and configurations are correctly set up for the tests.

3. Process completed with exit code 1:[Run tests with flakeguard]

Source of Error:
2024-12-03T05:16:10.5546529Z ##[error]Process completed with exit code 1.
**Why**: The flakeguard run encountered test failures, leading to a non-zero exit code.

Suggested fix: Investigate the specific tests that failed during the flakeguard run. Address any flaky tests or underlying issues causing the failures.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@reductionista
Copy link
Contributor Author

Closed after finding a cleaner way of fixing this:
#15579

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant