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

[CAPPL-474] Cleanup channels #15994

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

cedric-cordenier
Copy link
Contributor

Requires

Supports

@cedric-cordenier cedric-cordenier force-pushed the CAPPL-474-cleanup-channels branch from 7f8f492 to 3fd6d29 Compare January 20, 2025 16:22
Copy link
Contributor

github-actions bot commented Jan 20, 2025

AER Report: CI Core

aer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Core Tests (go_core_tests) , GolangCI Lint (.) , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , test-scripts , Core Tests (go_core_fuzz) , Core Tests (go_core_race_tests) , lint , SonarQube Scan

1. Duplicate message received for ID

Job ID: Run tests

Source of Error:
Run tests	2025-01-20T16:47:31.3436386Z 	 	duplicate message received for ID: 95ef5e32deb99a10ee6804bc4af13855687559d7ff6552ac6dbb2ce0abbadeed/web_api_target

Why: This error occurs when the system receives the same message ID more than once, which indicates a potential issue with message handling or ID generation logic.

Suggested fix: Ensure that each message ID is unique and that the system correctly handles message IDs to prevent duplicates. Review the message generation and handling logic for potential flaws.

2. Expectation not met in test

Job ID: Run tests

Source of Error:
Run tests	2025-01-20T16:47:31.3441170Z gateway_connector.go:665: FAIL: 6 out of 7 expectation(s) were met.
Run tests	2025-01-20T16:47:31.3442348Z 	The code you are testing needs to make 1 more call(s).

Why: The test expected 7 calls to be made, but only 6 were made. This indicates that the code under test did not fully execute all expected operations.

Suggested fix: Review the test case and the code to ensure that all expected calls are made. Check for any conditional logic that might prevent the final call from being executed.

3. Process completed with exit code 1

Job ID: Run tests

Source of Error:
Run tests	2025-01-20T16:49:26.3607250Z ##[error]Process completed with exit code 1.

Why: The process exited with a non-zero exit code, indicating that one or more tests failed during execution.

Suggested fix: Address the specific test failures identified in the logs. In this case, focus on resolving the duplicate message ID issue and ensuring all expected calls are made in the tests.

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@cedric-cordenier cedric-cordenier force-pushed the CAPPL-474-cleanup-channels branch from 3fd6d29 to 154145c Compare January 20, 2025 16:39
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