Skip to content

ci: build write-capable-build on ros-base too - #686

Merged
bburda merged 1 commit into
mainfrom
ci/write-capable-job-on-ros-base
Sep 13, 2026
Merged

ci: build write-capable-build on ros-base too#686
bburda merged 1 commit into
mainfrom
ci/write-capable-job-on-ros-base

Conversation

@bburda

@bburda bburda commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

opcua-plugin.yml's write-capable-build job still ran on ubuntu:noble with the "Install Git", "Pre-install ROS 2 apt source" and "Set up ROS 2 Jazzy" steps, and the ros-apt-source composite action the second step used no longer exists since the other container jobs moved to ros:<distro>-ros-base. The job fails at that step on every run of the workflow on main, and on every pull request that triggers it.

The job now runs on ros:jazzy-ros-base like its siblings, the three steps are gone, and "Install ccache" runs its own apt-get update first, because the ros-base images ship without an apt index.


Issue

No issue. This is a one-file repair of a job the ros-base migration on main left behind.


Type

  • Bug fix
  • New feature or tests
  • Breaking change
  • Documentation only

Testing

  • The write-capable-build job of the OPC-UA Plugin Integration workflow is the test: it is red on main and must be green on this branch.
  • Locally: the workflow parses, the job's container image is ros:jazzy-ros-base, its step list no longer contains the three removed steps, and no workflow file references ros-apt-source or setup-ros any more.

Checklist

  • Breaking changes are clearly described (and announced in docs / changelog if needed)
  • Tests were added or updated if needed
  • Docs were updated if behavior or public API changed

The write-capable-build job in opcua-plugin.yml was missed when the other
container jobs in this file moved from ubuntu:<codename> + setup-ros to
ros:<distro>-ros-base: it still ran on ubuntu:noble with the Install Git,
Pre-install ROS 2 apt source, and Set up ROS 2 Jazzy steps, and the
ros-apt-source composite action those steps used no longer exists. Switch
the job to ros:jazzy-ros-base, drop the three steps, and give Install
ccache its own apt-get update, matching every other job in this workflow.
@bburda bburda self-assigned this Sep 12, 2026
@bburda
bburda merged commit 5fbdded into main Sep 13, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants