feat: add metadata through LABELs #103
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
domire8
approved these changes
Apr 29, 2024
eeberhard
reviewed
Apr 29, 2024
Comment on lines
+145
to
+146
| LABEL org.opencontainers.image.title="AICA ROS2 image" | ||
| LABEL org.opencontainers.image.description="AICA base ROS2 image (includes ros2_control)" |
There was a problem hiding this comment.
nitpick: follow ROS 2 branding guidelines, since this is a base public image it would be nice. In other place (code, comments, etc) the informal ros2 is acceptable.
Suggested change
| LABEL org.opencontainers.image.title="AICA ROS2 image" | |
| LABEL org.opencontainers.image.description="AICA base ROS2 image (includes ros2_control)" | |
| LABEL org.opencontainers.image.title="AICA ROS 2 image" | |
| LABEL org.opencontainers.image.description="AICA base ROS 2 image (includes ros2_control)" |
domire8
pushed a commit
to domire8/docker-images
that referenced
this pull request
Apr 30, 2024
domire8
pushed a commit
that referenced
this pull request
Apr 30, 2024
eeberhard
added a commit
that referenced
this pull request
May 10, 2024
This reverts commit 5cea82c.
eeberhard
added a commit
that referenced
this pull request
May 13, 2024
…s:iron (#109) * Revert "feat: create ros2-ws image that has ros2-control installed (#107)" This reverts commit a284a2d. * Revert "Revert "refactor: merge ros2-control image into ros2-ws (#100)"" This reverts commit be10502. * Revert "Revert "feat: add metadata through LABELs (#103)"" This reverts commit 5cea82c. * Revert "Revert "fix: correct user (#105)"" This reverts commit 52d2b8a. * fix: AICA image metadata base version as git tag, not ROS distro * Change tech.aica.image.metadata base.version to use VERSION (expected as semver vX.Y.Z-foo) instead of BASE_TAG (which was equiavlent to ROS distro (i.e iron). Otherwise, package-builder:v1 will fail to parse the metadata since it expects the base version as a semver string * feat: use specific SH256 digest for iron base image (2023.09.02) * Include more build args in Dockerfile to disambiguate ROS_DISTRO from BASE_TAG and to support different BASE_IMAGE * Update build push workflow to set new build args with condition based on ROS distro * Use specific SHA256 digest image and tag for ros:iron base image based on a build from September 02, 2023 * feat: add devcontainer metadata * Set default container user metadata for devcontainers (https://containers.dev/implementors/reference/#labels)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Also requires #102 so that
GIT_TAGis valid semverThis PR adds standard Docker metadata plus our internal one required for dependency management.
Review guidelines
Estimated Time of Review: 5 minutes