Project

Profile

Help

Issue #1365

Removing a Docker repository can cause a TypeError if the repo has not been published

Added by rbarlow about 6 years ago. Updated over 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Start date:
Due date:
Estimated time:
Severity:
3. High
Version - Docker:
master
Platform Release:
Target Release - Docker:
2.0.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

Steps to reproduce:
0) $ pulp-admin docker repo create --repo-id busybox
1) $ pulp-admin docker repo delete --repo-id busybox

Actual results:

$ pulp-admin docker repo delete --repo-id busybox
This command may be exited via ctrl+c without affecting the request.

[\]
Running...

Task Failed

Pulp exception occurred: PulpExecutionException

coercing to Unicode: need string or buffer, NoneType found

coercing to Unicode: need string or buffer, NoneType found

In the log, this exception appears:

Error received removing distributor [docker_web_distributor_name_cli] from repo [busybox]
Traceback (most recent call last):
  File "/home/vagrant/devel/pulp/server/pulp/server/controllers/repository.py", line 311, in delete
    distributor_manager.remove_distributor(repo_id, repo_distributor['id'])
  File "/home/vagrant/devel/pulp/server/pulp/server/managers/repo/distributor.py", line 228, in remove_distributor
    distributor_instance.distributor_removed(transfer_repo, call_config)
  File "/home/vagrant/devel/pulp_docker/plugins/pulp_docker/plugins/distributors/distributor_web.py", line 162, in distributor_removed
    shutil.rmtree(repo_dir, ignore_errors=True)
  File "/usr/lib64/python2.7/shutil.py", line 228, in rmtree
    if os.path.islink(path):
  File "/usr/lib64/python2.7/posixpath.py", line 135, in islink
    st = os.lstat(path)
TypeError: coercing to Unicode: need string or buffer, NoneType found

Expected results:
The repo should be deleted without error.

Additonal information:
Similar errors have been reported against other plugins, and are linked in related issues.


Related issues

Related to OSTree Support - Issue #1096: Deleting an OSTree repo results in a TypeError on the serverCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Related to Python Support - Issue #1349: Deleting an python repo results in a TypeError on the serverCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 58b4a5dd View on GitHub
Added by ipanova@redhat.com about 6 years ago

Removing a Docker repository can cause a TypeError.

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

Revision 58b4a5dd View on GitHub
Added by ipanova@redhat.com about 6 years ago

Removing a Docker repository can cause a TypeError.

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

Revision 58b4a5dd View on GitHub
Added by ipanova@redhat.com about 6 years ago

Removing a Docker repository can cause a TypeError.

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

Revision 58b4a5dd View on GitHub
Added by ipanova@redhat.com about 6 years ago

Removing a Docker repository can cause a TypeError.

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

History

#1 Updated by rbarlow about 6 years ago

  • Related to Issue #1096: Deleting an OSTree repo results in a TypeError on the server added

#2 Updated by rbarlow about 6 years ago

  • Related to Issue #1349: Deleting an python repo results in a TypeError on the server added

#3 Updated by rbarlow about 6 years ago

  • Description updated (diff)

#4 Updated by rbarlow about 6 years ago

  • Description updated (diff)

#5 Updated by rbarlow about 6 years ago

  • Target Release - Docker set to 2.0.0

#6 Updated by ipanova@redhat.com about 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com

#7 Updated by ipanova@redhat.com about 6 years ago

  • Status changed from ASSIGNED to POST

#8 Updated by mhrivnak about 6 years ago

  • Triaged changed from No to Yes

#9 Updated by ipanova@redhat.com about 6 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#10 Updated by rbarlow almost 6 years ago

  • Status changed from MODIFIED to 5

#11 Updated by semyers over 5 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#12 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF