Fixes time in container registry partial cleanup tooltip
What does this MR do and why?
Fixes time in container registry partial cleanup policies tooltip. Since this data is not available in group context, hide the icon which triggers the tooltip.
Fix for tooltip not displaying on all rows is being fixed as part of Fixes container registry popover not displaying... (!92244 - merged)
Screenshots or screen recordings
Before | After |
---|---|
How to set up and validate locally
- Setup & enable Container Registry https://gitlab.com/gitlab-org/gitlab-development-kit/-/blob/main/doc/howto/registry.md
- On a project with container registry images and clean up policies enabled.
- Push images to project/group so that you can view a list of images with multiple tags.
docker pull alpine:latest docker tag alpine:latest 127.0.0.1:5000/gitlab-org/gitlab-test:latest docker push 127.0.0.1:5000/gitlab-org/gitlab-test:latest docker tag alpine:latest 127.0.0.1:5000/gitlab-org/gitlab-test:temporary docker push 127.0.0.1:5000/gitlab-org/gitlab-test:temporary docker tag alpine:latest 127.0.0.1:5000/gitlab-org/gitlab-org/alpine/hello-world/hello-world docker push 127.0.0.1:5000/gitlab-org/gitlab-org/alpine/hello-world/hello-world
- Alternatively, you can also return
true
from both the computed properties https://gitlab.com/gitlab-org/gitlab/-/blob/364609-partial-cleanup-tooltip-does-not-display-time/app/assets/javascripts/packages_and_registries/container_registry/explorer/components/list_page/cleanup_status.vue#L47-52
- Alternatively, you can also return
- Hover on the information icon on any image with a number of tags that have been partially cleaned up.
- In group context the icon to trigger popover should not be present however in the project context, icon to trigger popover should be present & the popover component should mention the when cleanup will complete.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Closes #364609 (closed)
Edited by Rahul Chanila