Skip to content

[DO NOT MERGE][ABI test] jazzy / compatible: expect "ABI compatible" + backport-humble - #3388

Closed
fujitatomoya wants to merge 1 commit into
jazzyfrom
abi-test/jazzy-compatible
Closed

fujitatomoya wants to merge 1 commit into
jazzyfrom
abi-test/jazzy-compatible

Conversation

@fujitatomoya

Copy link
Copy Markdown
Collaborator

Case: target jazzy, comment-only change in rclcpp/include/rclcpp/node.hpp.
Note: this branch has no abi.yaml yet, so the check only runs on this pull request because the
pull request adds a verbatim copy of the rolling abi.yaml. The sticky comment and labels come from
abi-report.yaml on the default branch (workflow_run), which needs nothing on this branch.


This is a test pull request for the ABI checker (abi.yaml / abi-report.yaml, backed by
fujitatomoya/ros2-abi-action) and the
backport-<distro> candidate labels (#2555). It exists only for fujitatomoya to verify the
action result for one specific case, and it is closed without merging once that is done.
Maintainers: nothing to review here, please ignore this pull request.

🤖 Generated with Claude Code

ABI checker / backport label test; this pull request is closed without merging.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@fujitatomoya

fujitatomoya commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator Author

Expected result

  • abi check: success (distro jazzy), all four libraries compatible
  • label ABI compatible added
  • exactly one backport label added: backport-humble

@github-actions

Copy link
Copy Markdown

ABI Compliance Check

✅ Verdict: compatible

Library Verdict Summary
libcomponent_manager.so ✅ compatible No ABI changes detected.
librclcpp.so ✅ compatible No ABI changes detected.
librclcpp_action.so ✅ compatible No ABI changes detected.
librclcpp_lifecycle.so ✅ compatible No ABI changes detected.
✅ libcomponent_manager.so — full abidiff report

Compared:

  • Base: lib-base/libcomponent_manager.so
  • Head: lib-pr/libcomponent_manager.so @ 4f70fb7
(empty report — no differences printed by abidiff)
✅ librclcpp.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp.so
  • Head: lib-pr/librclcpp.so @ 4f70fb7
(empty report — no differences printed by abidiff)
✅ librclcpp_action.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp_action.so
  • Head: lib-pr/librclcpp_action.so @ 4f70fb7
(empty report — no differences printed by abidiff)
✅ librclcpp_lifecycle.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp_lifecycle.so
  • Head: lib-pr/librclcpp_lifecycle.so @ 4f70fb7
(empty report — no differences printed by abidiff)

Updated for commit 4f70fb7 · check run · suppressions: /home/runner/work/_temp/ros2-abi-suppressions.txt

@fujitatomoya fujitatomoya left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

confirmed expected result, closing.

fujitatomoya added a commit that referenced this pull request Oct 28, 2026
Verbatim copy of .github/workflows/abi.yaml from rolling (#3262, #3276,
ABI check nor the "ABI compatible" / "ABI break" and backport-<distro>
labels, because pull_request workflows run from the workflow file of the
target branch.

Only abi.yaml is needed here: the sticky comment and the labels are
published by abi-report.yaml, a workflow_run workflow, which GitHub always
runs from the default branch (rolling). The distro and the REP-0009 policy
are resolved from the target branch (distro: auto, policy: auto), so the
check is strict on jazzy: an incompatible ABI change fails the check.

Verified with the same file on test pull request(s) #3388 (compatible) and #3389 (ABI break).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Signed-off-by: Tomoya.Fujita <fujita.tomoya@triorb.co.jp>
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.

1 participant