Skip to content

chore: update agnocast to 2.4.0 - #7292

Merged
xmfcx merged 2 commits into
autowarefoundation:mainfrom
Koichi98:update_agnocast_to_2.4.0
Sep 2, 2026
Merged

chore: update agnocast to 2.4.0#7292
xmfcx merged 2 commits into
autowarefoundation:mainfrom
Koichi98:update_agnocast_to_2.4.0

Conversation

@Koichi98

@Koichi98 Koichi98 commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Description

Update agnocast from 2.3.5 to 2.4.0 (repositories/autoware.repos and the agnocast ansible role).

Also adds one .dockerignore exception. 2.4.0 introduces ros2agnocast_discovery_agent, whose setup.py installs systemd/README.md — the operating instructions for the agnocast-domain-bridge.service.example shipped next to it. src/**/*.md (added in #5009 to cut image size) excludes it from the Docker build context, so colcon build fails in the container with can't copy 'systemd/README.md' while a local build passes.

The exception restores a single 2 KB file. #5009's saving came mostly from .git — its own description says "the most effective one is .git" — and .svg/.asc/.gif stay excluded; all 636 .md files under src/ total 3.3 MB on a local checkout. Same shape as #5187 and #5263, which restored .csv/.png for the same reason.

How was this PR tested?

Confirmed the 2.4.0 tag exists in https://github.com/autowarefoundation/agnocast, and that agnocast-heaphook-v2.4.0 and agnocast-kmod-v2.4.0 are available from the agnocast PPA.

Health-check failed before the .dockerignore change with error: can't copy 'systemd/README.md' in ros2agnocast_discovery_agent on all five jobs (run).

Verified the exception with a scratch docker build over the same layout: systemd/README.md reaches the context while other src/**/*.md files stay excluded.

Signed-off-by: Koichi Imai <koichi.imai.2@tier4.jp>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@Koichi98
Koichi98 marked this pull request as ready for review September 1, 2026 07:28
@Koichi98 Koichi98 self-assigned this Sep 1, 2026
@Koichi98 Koichi98 added the run:health-check Run health-check label Sep 1, 2026

@xmfcx xmfcx 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.

Could you update here too?

agnocast_version="2.1.2"

…discovery_agent

agnocast 2.4.0 adds ros2agnocast_discovery_agent, whose setup.py data_files
installs systemd/README.md alongside agnocast-domain-bridge.service.example.
src/**/*.md kept it out of the Docker build context, so colcon build failed
in the container with "can't copy 'systemd/README.md'" while local builds
passed.

Signed-off-by: koichiimai <koichi.imai.2@tier4.jp>
@Koichi98
Koichi98 force-pushed the update_agnocast_to_2.4.0 branch from bd42b3c to e963f90 Compare September 1, 2026 21:58
@Koichi98

Koichi98 commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator Author

@xmfcx
That line was agnocast_version="2.1.2" in the manual snippet, which #7293 removed. The README no longer mentions a version — defaults/main.yaml is the only place left, and this PR updates it.

I also pushed a .dockerignore exception since your review: 2.4.0's new ros2agnocast_discovery_agent installs systemd/README.md via setup.py, which src/**/*.md kept out of the build context, failing health-check. Details in the description.

@Koichi98
Koichi98 marked this pull request as ready for review September 1, 2026 22:04
@Koichi98
Koichi98 requested a review from xmfcx September 2, 2026 13:12
@xmfcx
xmfcx merged commit bd86191 into autowarefoundation:main Sep 2, 2026
25 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:health-check Run health-check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants