Skip to content

Commit fbb57a3

Browse files
committed
Remove annotations
1 parent 8f1849f commit fbb57a3

File tree

4 files changed

+13
-36
lines changed

4 files changed

+13
-36
lines changed

.github/actions/docker-bake/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,9 @@ runs:
7676
python-version: "3.x"
7777
cache: "pip"
7878

79-
- name: Install python deps
79+
- name: Install dependencies
8080
shell: bash
81-
run: pip install -r requirements.txt
81+
run: pip install -r "${{ github.action_path }}/requirements.txt"
8282

8383
- name: Detect build platform
8484
id: detect
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
actions_toolkit
2+
ruamel.yaml
3+
typing_extensions

.github/actions/docker-merge/action.yml

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -69,18 +69,14 @@ runs:
6969
shell: bash
7070
run: |
7171
set -euo pipefail
72-
cmd=(python .github/actions/docker-merge/merge_manifests.py
73-
--family "${{ inputs.family }}"
74-
--distro "${{ inputs.distro }}"
75-
--metadata-list "${{ inputs.metadata }}"
76-
${{ inputs.ghcr-password && format('--gh-owner "{0}"', inputs.ghcr-username) || '' }}
77-
${{ inputs.docker-password && format('--dockerhub-username "{0}"', inputs.docker-username) || '' }}
78-
${{ inputs.extra-tag != '' && format('--extra-tag "{0}"', inputs.extra-tag) || '' }}
79-
)
80-
if [[ "${{ inputs.dry-run }}" == "true" ]]; then
81-
cmd+=(--dry-run)
82-
fi
83-
"${cmd[@]}"
72+
python .github/actions/docker-merge/merge_manifests.py \
73+
--family "${{ inputs.family }}" \
74+
--distro "${{ inputs.distro }}" \
75+
--metadata-list "${{ inputs.metadata }}" \
76+
${{ inputs.ghcr-password && format('--gh-owner "{0}"', inputs.ghcr-username) || '' }} \
77+
${{ inputs.docker-password && format('--dockerhub-username "{0}"', inputs.docker-username) || '' }} \
78+
${{ inputs.extra-tag != '' && format('--extra-tag "{0}"', inputs.extra-tag) || '' }} \
79+
${{ inputs.dry-run == "true" && format('--dry-run') || ''}}
8480
8581
- name: Summary
8682
if: always()

.github/actions/docker-merge/merge_manifests.py

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,6 @@
1111
from typing import Dict, Iterable, List, Set
1212

1313
MetadataEntries = Dict[str, List[Dict[str, str]]]
14-
STAGE_DESCRIPTIONS = {
15-
"base": "Base image",
16-
"dev": "Developer image",
17-
"full": "Desktop image",
18-
"gazebo": "Gazebo image",
19-
"gazebo-nvidia": "Gazebo NVIDIA/CUDA image",
20-
"nvidia": "NVIDIA/CUDA image",
21-
}
2214

2315

2416
def parse_args() -> argparse.Namespace:
@@ -214,13 +206,6 @@ def compute_tags(
214206
return tags
215207

216208

217-
def manifest_description(family: str, distro: str, stage: str) -> str:
218-
"""Return an OCI description for the multi-arch manifest."""
219-
base = STAGE_DESCRIPTIONS.get(stage, stage.replace("-", " ").title())
220-
pretty_distro = distro.replace("-", " ").title()
221-
return f"{family.upper()} {pretty_distro} {base}".strip()
222-
223-
224209
def write_output(name: str, value: str) -> None:
225210
"""Emit a GitHub Actions output key/value pair."""
226211
output_path = os.environ.get("GITHUB_OUTPUT")
@@ -258,16 +243,9 @@ def main() -> int:
258243
)
259244
if not tags:
260245
raise ValueError("No destination tags specified for merge action.")
261-
annotation = manifest_description(args.family, args.distro, stage_name)
262246
cmd = ["docker", "buildx", "imagetools", "create"]
263247
for tag in tags:
264248
cmd.extend(["--tag", tag])
265-
cmd.extend(
266-
[
267-
"--annotation",
268-
f"org.opencontainers.image.description={annotation}",
269-
]
270-
)
271249
cmd.extend(refs)
272250
print(f"[merge] {target}: creating manifest with {len(refs)} refs")
273251
print(" tags:", ", ".join(tags))

0 commit comments

Comments
 (0)