Project

Profile

Help

Issue #1365

closed

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

Added by rbarlow over 8 years ago. Updated about 5 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 - CURRENTRELEASEipanova@redhat.comActions
Related to Python Support - Issue #1349: Deleting an python repo results in a TypeError on the serverCLOSED - CURRENTRELEASEipanova@redhat.comActions
Actions #1

Updated by rbarlow over 8 years ago

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

Updated by rbarlow over 8 years ago

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

Updated by rbarlow over 8 years ago

  • Description updated (diff)
Actions #4

Updated by rbarlow over 8 years ago

  • Description updated (diff)
Actions #5

Updated by rbarlow over 8 years ago

  • Target Release - Docker set to 2.0.0
Actions #6

Updated by ipanova@redhat.com over 8 years ago

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

Added by ipanova@redhat.com over 8 years ago

Revision 58b4a5dd | View on GitHub

Removing a Docker repository can cause a TypeError.

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

Added by ipanova@redhat.com over 8 years ago

Revision 58b4a5dd | View on GitHub

Removing a Docker repository can cause a TypeError.

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

Added by ipanova@redhat.com over 8 years ago

Revision 58b4a5dd | View on GitHub

Removing a Docker repository can cause a TypeError.

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

Added by ipanova@redhat.com over 8 years ago

Revision 58b4a5dd | View on GitHub

Removing a Docker repository can cause a TypeError.

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

Actions #7

Updated by ipanova@redhat.com over 8 years ago

  • Status changed from ASSIGNED to POST
Actions #8

Updated by mhrivnak over 8 years ago

  • Triaged changed from No to Yes
Actions #9

Updated by ipanova@redhat.com over 8 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #10

Updated by rbarlow about 8 years ago

  • Status changed from MODIFIED to 5
Actions #11

Updated by semyers almost 8 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #12

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF