Project

Profile

Help

Issue #3905

Purge for unlinked manifests is not working correctly

Added by ipanova@redhat.com 11 months ago. Updated 2 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Severity:
2. Medium
Version - Docker:
Platform Release:
2.17.0
Blocks Release:
Target Release - Docker:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 41

Description

Steps to reproduce:

1. pulp-admin docker repo create --repo-id=synctest --feed=https://registry-1.docker.io --upstream-name=busybox --tags=latest

2) sync repo

[ipanova@ina pulp_docker]$ pulp-admin docker repo list

--------------------------------------------------------------------
Docker Repositories
--------------------------------------------------------------------

Id: synctest
Display Name: None
Description: None
Content Unit Counts:
Docker Blob: 17
Docker Manifest: 9
Docker Manifest List: 1
Docker Tag: 2

3) remove 1 manifest list by specifying its digest

pulp-admin -vv docker repo remove manifest-list --repo-id synctest --str-eq='digest=sha256:cb63aa0641a885f54de20f61d152187419e8f6b159ed11a251a09d115fdff9bd'

4) observe that number of docker manifests did not change, that means manifests where not purged as they should been

[ipanova@ina pulp_docker]$ pulp-admin docker repo list

--------------------------------------------------------------------
Docker Repositories
--------------------------------------------------------------------

Id: synctest
Display Name: None
Description: None
Content Unit Counts:
Docker Blob: 17
Docker Manifest: 9
Docker Manifest List: 0
Docker Tag: 1

Expected behaviour:

4) observe that number of docker manifest decreased, because unlinked manifests were removed with manifest list removal

[ipanova@ina pulp_docker]$ pulp-admin docker repo list

--------------------------------------------------------------------
Docker Repositories
--------------------------------------------------------------------

Id: synctest
Display Name: None
Description: None
Content Unit Counts:
Docker Blob: 2
Docker Manifest: 1
Docker Manifest List: 0
Docker Tag: 1

Associated revisions

Revision 22209b17 View on GitHub
Added by ipanova@redhat.com 11 months ago

Fix removal of embedded manifests in manifest list.

closes#3905
https://pulp.plan.io/issues/3905

History

#1 Updated by ipanova@redhat.com 11 months ago

  • Project changed from Pulp to Docker Support
  • Status changed from NEW to POST
  • Sprint/Milestone set to 2.17.0

#2 Updated by ipanova@redhat.com 11 months ago

  • Description updated (diff)

#3 Updated by CodeHeeler 11 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 41

#4 Updated by ipanova@redhat.com 11 months ago

  • Status changed from POST to MODIFIED

#5 Updated by ipanova@redhat.com 11 months ago

  • Platform Release set to 2.17.0

#6 Updated by ipanova@redhat.com 11 months ago

  • Platform Release changed from 2.17.0 to master

#7 Updated by ipanova@redhat.com 11 months ago

  • Platform Release deleted (master)

#8 Updated by ipanova@redhat.com 11 months ago

  • Platform Release set to 2.17.0

#9 Updated by ipanova@redhat.com 11 months ago

  • Assignee set to ipanova@redhat.com

#10 Updated by ipanova@redhat.com 10 months ago

  • Status changed from MODIFIED to ON_QA

#11 Updated by ipanova@redhat.com 10 months ago

  • Status changed from ON_QA to CLOSED - CURRENTRELEASE

#12 Updated by bmbouter 2 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF