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

[TT-1952] Write application logs for in-memory tests to file and safe in artifacts #16029

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

Conversation

Tofel
Copy link
Contributor

@Tofel Tofel commented Jan 22, 2025

Write the logs for the cl nodes used in the environment for the in-memory tests to log files located in ./integration-tests/smoke/ccip/logs/ . These files will be available as artifacts after running the in-memory tests in CI.

e.g. in this run https://github.com/smartcontractkit/chainlink/actions/runs/13176452557/job/36776884455 the smoke/ccip/ccip_messaging_test.go failed, so you can find the logs for the nodes that ran during the test in the smoke_ccip_ccip_messaging_test_go_-test-logs artifact.

@Tofel Tofel requested review from a team as code owners January 22, 2025 13:40
@Tofel Tofel changed the title [TT-1952] try to split logs by level in in-memory tests [TT-1952] try to split logs by level in in-memory tests [WIP] Jan 22, 2025

This comment was marked as spam.

@Tofel Tofel requested a review from a team as a code owner January 22, 2025 14:56
Copy link
Contributor

Flakeguard Summary

Ran new or updated tests between develop and a9a605c (tt-1952-split-in-memory-logs).

View Flaky Detector Details | Compare Changes

Found Flaky Tests ❌

Name Pass Ratio Panicked? Timed Out? Race? Runs Successes Failures Skips Package Package Panicked? Avg Duration Code Owners
TestInvalidOCR3Params 0.00% false false false 3 0 3 0 github.com/smartcontractkit/chainlink/deployment/ccip/changeset false 4.47s @smartcontractkit/ccip, @smartcontractkit/core, @smartcontractkit/deployment-automation
Test_ActiveCandidate 0.00% false false false 3 0 3 0 github.com/smartcontractkit/chainlink/deployment/ccip/changeset false 3m51.926666666s @smartcontractkit/ccip, @smartcontractkit/core, @smartcontractkit/deployment-automation

Artifacts

For detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json.

Copy link
Contributor

Flakeguard Summary

Ran new or updated tests between develop and b10043d (tt-1952-split-in-memory-logs).

View Flaky Detector Details | Compare Changes

Found Flaky Tests ❌

Name Pass Ratio Panicked? Timed Out? Race? Runs Successes Failures Skips Package Package Panicked? Avg Duration Code Owners
TestInvalidOCR3Params 0.00% false false false 3 0 3 0 github.com/smartcontractkit/chainlink/deployment/ccip/changeset false 4.446666666s @smartcontractkit/ccip, @smartcontractkit/core, @smartcontractkit/deployment-automation
Test_ActiveCandidate 0.00% false false false 3 0 3 0 github.com/smartcontractkit/chainlink/deployment/ccip/changeset false 3m50.103333333s @smartcontractkit/ccip, @smartcontractkit/core, @smartcontractkit/deployment-automation

Artifacts

For detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json.

Copy link
Contributor

Flakeguard Summary

Ran new or updated tests between develop and db92c9e (tt-1952-split-in-memory-logs).

View Flaky Detector Details | Compare Changes

Found Flaky Tests ❌

Name Pass Ratio Panicked? Timed Out? Race? Runs Successes Failures Skips Package Package Panicked? Avg Duration Code Owners
TestInvalidOCR3Params 0.00% false false false 3 0 3 0 github.com/smartcontractkit/chainlink/deployment/ccip/changeset false 4.583333333s @smartcontractkit/ccip, @smartcontractkit/core, @smartcontractkit/deployment-automation
Test_ActiveCandidate 0.00% false false false 3 0 3 0 github.com/smartcontractkit/chainlink/deployment/ccip/changeset false 3m49.37s @smartcontractkit/ccip, @smartcontractkit/core, @smartcontractkit/deployment-automation

Artifacts

For detailed logs of the failed tests, please refer to the artifact failed-test-results-with-logs.json.

@gheorghestrimtu gheorghestrimtu changed the title [TT-1952] try to split logs by level in in-memory tests [WIP] [TT-1952] Write application logs for in-memory tests to file and safe in artifacts Feb 5, 2025
@gheorghestrimtu gheorghestrimtu marked this pull request as ready for review February 6, 2025 12:28
makramkd
makramkd previously approved these changes Feb 6, 2025
@gheorghestrimtu gheorghestrimtu marked this pull request as draft February 6, 2025 13:07
@gheorghestrimtu gheorghestrimtu marked this pull request as ready for review February 6, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants