Skip to content

Fujitatomoya/abi backport label - #3341

Merged
ahcorde merged 1 commit into
rollingfrom
fujitatomoya/abi-backport-label
Oct 20, 2026
Merged

ahcorde merged 1 commit into
rollingfrom
fujitatomoya/abi-backport-label

Conversation

@fujitatomoya

@fujitatomoya fujitatomoya commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

Description

follow up of #3340

this enables the action that labels the backport status based on the abi compliance checker result.
this was the original goal of #2555 not to miss the backports to the downstream.

Fixes # (issue)

Is this user-facing behavior change?

Yes, but mostly for maintainers.
After this PR, action can automatically label the backport-xxx based on the abi compliance checker result.
Be advised that this is not decision, decision will be made by maintainers.
This is because, even if that is ABI compatible change, it sometimes generates the huge behavior change.
In that case, it is unlikely to backport the feature to the downstream distribution.

Did you use Generative AI?

Yes, Claude Fable 5.1

Additional Information

@github-actions

github-actions Bot commented Oct 14, 2026 •

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 @ 30b27c1
(empty report — no differences printed by abidiff)
✅ librclcpp.so — full abidiff report

Compared:

  • Base: lib-base/librclcpp.so
  • Head: lib-pr/librclcpp.so @ 30b27c1
(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 @ 30b27c1
(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 @ 30b27c1
(empty report — no differences printed by abidiff)

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

@fujitatomoya

Copy link
Copy Markdown
Collaborator Author

@ahcorde can you review this?

Once abi-report publishes the ABI verdict of a pull request, add one
backport-<distro> label per distribution downstream of the PR's target
branch when every library is ABI compatible, and remove them when it is
not. Labels a maintainer removed by hand are not added again.

The job runs from the workflow_run token of this repository, so pull
requests from forks are labelled as well.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Signed-off-by: Tomoya.Fujita <fujita.tomoya@triorb.co.jp>
@fujitatomoya
fujitatomoya force-pushed the fujitatomoya/abi-backport-label branch from 2cac05b to 30b27c1 Compare October 19, 2026 14:09
@fujitatomoya

Copy link
Copy Markdown
Collaborator Author

@ahcorde this one is final PR to enable the auto-label backports, can you review this too?

after this PR is merged, i would like to create several test PRs to make sure that everything works fine. (and later on, i would like to enable this to other c/c++ projects.)

@ahcorde
ahcorde merged commit 2008bdd into rolling Oct 20, 2026
11 checks passed
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.

2 participants