Skip to content

Conversation

k-a-il
Copy link
Contributor

@k-a-il k-a-il commented Jul 7, 2025

Motivation

This PR introduces archiving of CSV files with raw metrics generated during integration tests, which are required for parity docs updates

Changes

  • target/metric_reports path added to archive test results step

Testing

Checked correctness of the raw report path in a forked repository

@k-a-il k-a-il self-assigned this Jul 7, 2025
@k-a-il k-a-il added area: ci Running LocalStack in CI environments area: testing Testing Localstack semver: patch Non-breaking changes which can be included in patch releases labels Jul 7, 2025
Copy link

github-actions bot commented Jul 7, 2025

Test Results - Preflight, Unit

21 853 tests  ±0   20 196 ✅ ±0   6m 24s ⏱️ -26s
     1 suites ±0    1 657 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 7e6d7be. ± Comparison against base commit fa3e8c4.

Copy link

github-actions bot commented Jul 7, 2025

Test Results (MA/MR) - Preflight, Unit

21 853 tests   20 196 ✅  7m 4s ⏱️
     1 suites   1 657 💤
     1 files         0 ❌

Results for commit 7e6d7be.

Copy link

github-actions bot commented Jul 7, 2025

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 7s ⏱️ -8s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 7e6d7be. ± Comparison against base commit fa3e8c4.

Copy link

github-actions bot commented Jul 7, 2025

Test Results (amd64, MA/MR) - Acceptance

7 tests   5 ✅  3m 8s ⏱️
1 suites  2 💤
1 files    0 ❌

Results for commit 7e6d7be.

Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thanks for jumping on this! Nice and simple fix! 💯

Copy link

github-actions bot commented Jul 7, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 21m 16s ⏱️
5 271 tests 4 343 ✅ 928 💤 0 ❌
5 277 runs  4 343 ✅ 934 💤 0 ❌

Results for commit 7e6d7be.

Copy link

github-actions bot commented Jul 7, 2025

Test Results (amd64, MA/MR) - Integration, Bootstrap

    5 files      5 suites   2h 24m 13s ⏱️
5 271 tests 4 343 ✅ 928 💤 0 ❌
5 277 runs  4 343 ✅ 934 💤 0 ❌

Results for commit 7e6d7be.

@dfangl dfangl removed their request for review July 7, 2025 08:34
@k-a-il k-a-il merged commit 021614f into master Jul 7, 2025
58 of 59 checks passed
@k-a-il k-a-il deleted the ci/archive-metric-reports branch July 7, 2025 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ci Running LocalStack in CI environments area: testing Testing Localstack semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants