Skip to content

Add {tag_stem}-latest tags for ROS images built from main#12

Merged
cooperj merged 2 commits into
mainfrom
copilot/add-latest-tag-for-docker-images
Apr 27, 2026
Merged

Add {tag_stem}-latest tags for ROS images built from main#12
cooperj merged 2 commits into
mainfrom
copilot/add-latest-tag-for-docker-images

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 27, 2026

ROS and ROS-CUDA images built from main were never tagged as latest variants (e.g. lcas.lincoln.ac.uk/ros:humble-latest), only as humble-main. The standalone image workflow already handled latest correctly; the ROS reusable workflow did not.

Changes

  • .github/workflows/_build-ros-image.yaml — added a conditional type=raw tag entry to both the primary image and desktop image docker/metadata-action steps:
type=raw,enable=${{ inputs.image_context == 'main' }},value=${{ inputs.tag_stem }}-latest

Tag behaviour by trigger:

Trigger Tags produced
Push to main humble-main, humble-latest, semver tags
Tagged release humble-v1.2.3, humble-1.2, humble-1
PR / staging humble-staging

Uses inputs.image_context == 'main' (rather than github.ref_name) since image_context is the canonical abstraction already used throughout this reusable workflow and correctly distinguishes main-branch builds from tag-release builds.

Copilot AI linked an issue Apr 27, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add latest tag for docker images in workflows Add {tag_stem}-latest tags for ROS images built from main Apr 27, 2026
Copilot AI requested a review from cooperj April 27, 2026 10:37
@cooperj cooperj marked this pull request as ready for review April 27, 2026 10:52
@cooperj cooperj merged commit 7e9f675 into main Apr 27, 2026
6 checks passed
@cooperj cooperj deleted the copilot/add-latest-tag-for-docker-images branch April 27, 2026 10:59
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.

main branch containers should also be latest

2 participants