Skip to content

Fixing the integration test failure#152

Merged
rmaddikery merged 5 commits into
eclipse-score:mainfrom
RSingh1511:rs/fix-itf
Jun 23, 2026
Merged

Fixing the integration test failure#152
rmaddikery merged 5 commits into
eclipse-score:mainfrom
RSingh1511:rs/fix-itf

Conversation

@RSingh1511

@RSingh1511 RSingh1511 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

#153

Notes for Reviewer

Pre-Review Checklist for the PR Author

  • PR title is short, expressive and meaningful
  • Commits are properly organized
  • Relevant issues are linked in the References section
  • Tests are conducted
  • Unit tests are added

Checklist for the PR Reviewer

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes #

@RSingh1511 RSingh1511 temporarily deployed to workflow-approval June 17, 2026 10:58 — with GitHub Actions Inactive
@RSingh1511 RSingh1511 changed the title Fixing the integration test failure issu Fixing the integration test failure Jun 17, 2026
@github-actions

github-actions Bot commented Jun 17, 2026

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 6a887f5d-01aa-4af6-99f3-281bd22f327c
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 4 packages loaded
Loading: 4 packages loaded
    currently loading: 
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
INFO: Elapsed time: 5.521s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

rmaddikery
rmaddikery previously approved these changes Jun 17, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to On Hold in LOG - Logging FT Jun 17, 2026
@rmaddikery rmaddikery temporarily deployed to workflow-approval June 17, 2026 13:55 — with GitHub Actions Inactive
@rmaddikery rmaddikery self-requested a review June 17, 2026 13:56
@RSingh1511 RSingh1511 temporarily deployed to workflow-approval June 18, 2026 05:54 — with GitHub Actions Inactive
@RSingh1511 RSingh1511 requested a review from mobileinfo June 18, 2026 07:52
@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

- ensure QEMU reliably attaches and the guest is pingable
- gate the job on the integration_testing label
- make failures explicit
- minor opt
RSingh1511 and others added 2 commits June 23, 2026 09:59
- Remove stale issue and link recommended approach for
tackling gcc profile counter couter updates in multithreaded
applications
@rmaddikery

Copy link
Copy Markdown
Contributor

score_logging$ bazel coverage //score/mw/log/detail/wait_free_producer_queue:unit_test
INFO: Analyzed target //score/mw/log/detail/wait_free_producer_queue:unit_test (0 packages loaded, 0 targets configured).
INFO: LCOV coverage report is located at /home/user_xyz/.cache/bazel/_bazel_user_xyz/b76ec20db5213029581afcd2126529a5/execroot/_main/bazel-out/_coverage/_coverage_report.dat
and execpath is bazel-out/_coverage/_coverage_report.dat
INFO: Found 1 test target...
Target //score/mw/log/detail/wait_free_producer_queue:unit_test up-to-date:
bazel-bin/score/mw/log/detail/wait_free_producer_queue/unit_test
INFO: Elapsed time: 5.135s, Critical Path: 4.90s
INFO: 3 processes: 2 action cache hit, 1 internal, 2 linux-sandbox.
INFO: Build completed successfully, 3 total actions
//score/mw/log/detail/wait_free_producer_queue:unit_test PASSED in 4.7s


The bazel coverage runner runs successfully locally. It fails on CI because the coverage job is triggered by workflow defenition that defaults to BASE branch, not the PR head

@rmaddikery rmaddikery left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving with caveat: #152 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants