Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EE images: initial manifest list support #1036

Merged
merged 7 commits into from
Oct 7, 2021
Merged

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Oct 4, 2021

Fixes AAH-965
API: ansible/galaxy_ng#997

This adds initial support for manifest list type images, which happen when pulling from remote registries.

The images list API returns all images, including manifest lists which just refer to the other images.

  • hide images in a manifest list from the main list
    • ❕ this won't work if the manifest list is not returned in the same page as the images (fixed by using exlcude_child_manifests)
  • digests and tags for manifests list should NOT be clickable
    • handle clicking on the activity screen
      • 20211005212125
      • (because we'd have to list all the images, and then use that to fillter links on the activities page; or disable all links)
  • set size, layers to --- for manifest lists
  • expand images under manifest list (expander only on manifest lists)
    • actually render manifest list details there
    • 20211006010451

@himdel himdel added the backport-4.4 This PR should be backported to stable-4.4 (2.1) label Oct 4, 2021
@himdel himdel force-pushed the manifest-list branch 2 times, most recently from 9880ea8 to 22330f0 Compare October 6, 2021 00:36
@himdel himdel marked this pull request as ready for review October 6, 2021 01:07
@himdel
Copy link
Collaborator Author

himdel commented Oct 6, 2021

TODO add &exclude_child_manifests=true to the api call to filter out images in a list (DONE)

Copy link
Member

@ZitaNemeckova ZitaNemeckova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@himdel himdel merged commit 9e771b0 into ansible:master Oct 7, 2021
@himdel himdel deleted the manifest-list branch October 7, 2021 12:26
@patchback
Copy link

patchback bot commented Oct 7, 2021

Backport to stable-4.4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.4/9e771b0dfcd6eb27e0a3e942353f4569af28a690/pr-1036

Backported as #1049

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Oct 7, 2021
* EE images: filter out layers already in a manifest list

* EE images: isManifestList -> don't link to detail screen

* EE manifest: handle API errors by showing a message about manifest lists, linking to Images

* EE detail Images - add expand/collapse button on manifest list table rows

and TODO pops up on the right line :)

* EE detail Images: renderManifestList

* EE Images - use exclude_child_manifests instead of UI filtering

* OS/Arch -> OS / Arch

(cherry picked from commit 9e771b0)
himdel added a commit that referenced this pull request Oct 7, 2021
* EE images: filter out layers already in a manifest list

* EE images: isManifestList -> don't link to detail screen

* EE manifest: handle API errors by showing a message about manifest lists, linking to Images

* EE detail Images - add expand/collapse button on manifest list table rows

and TODO pops up on the right line :)

* EE detail Images: renderManifestList

* EE Images - use exclude_child_manifests instead of UI filtering

* OS/Arch -> OS / Arch

(cherry picked from commit 9e771b0)

Co-authored-by: Martin Hradil <mhradil@redhat.com>
@github-actions github-actions bot added the backported-4.4 This PR has been backported to stable-4.4 (2.1) label Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.4 This PR should be backported to stable-4.4 (2.1) backported-4.4 This PR has been backported to stable-4.4 (2.1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants