Revision b12b9a0f
Added by Partha Aji about 4 years ago
Added by | lib/hammer_cli_foreman_docker/docker_manifest.rb | ||
|---|---|---|
| 15 | 15 |
field :schema_version, _("Schema Version")
|
| 16 | 16 |
field :digest, _("Digest")
|
| 17 | 17 |
field :downloaded, _("Downloaded"), Fields::Boolean
|
| 18 |
from :tag do |
|
| 19 |
field :name, _("Tag Name")
|
|
| 20 |
end |
|
| 18 |
field :_tags, _("Tags")
|
|
| 19 |
end |
|
| 20 |
|
|
| 21 |
def extend_data(manifest) |
|
| 22 |
manifest['_tags'] = manifest['tags'].map { |e| e["name"] }.join(", ")
|
|
| 23 |
manifest |
|
| 21 | 24 |
end |
| 22 | 25 |
|
| 23 | 26 |
build_options do |o| |
| ... | ... | |
| 32 | 35 |
field :schema_version, _("Schema Version")
|
| 33 | 36 |
field :digest, _("Digest")
|
| 34 | 37 |
field :downloaded, _("Downloaded"), Fields::Boolean
|
| 35 |
from :tag do
|
|
| 36 |
field :name, _("Tag Name")
|
|
| 38 |
collection :tags, _("Tags") do
|
|
| 39 |
field :name, _("Name")
|
|
| 37 | 40 |
end |
| 38 | 41 |
end |
| 39 | 42 |
|
| 43 |
def extend_data(manifest) |
|
| 44 |
manifest['_tags'] = manifest['tags'].map { |e| { name: e["name"] } }.join(", ")
|
|
| 45 |
manifest |
|
| 46 |
end |
|
| 47 |
|
|
| 40 | 48 |
build_options |
| 41 | 49 |
end |
| 42 | 50 |
|
Also available in: Unified diff
Fixes #25874 - Shows tag info in manifest list
Prior to this commit "hammer docker manifest list" would show empty
values in the "Tag Names" column. This is because Tags became a one to
many. i.e. manifest can have multiple tags.
This commit addresses both the "docker manifest list" and "docker
manifest info" cases appropriately