Project

Profile

Help

Issue #1793

closed

puppet repo doesn't support modules with names like "organization/module_name"

Added by cristi.falcas@gmail.com about 6 years ago. Updated about 3 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
2.8.0
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

We have some modules that have in metadata.json the name set to something like this: "name": "management/role".

Uploading those modules to pulp fail because it tries to copy them in /var/lib/pulp/uploads/management/role-version.tar.gz.

This is the error from server side:
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) [Errno 2] No such file or directory: u'/var/lib/pulp/uploads/management/role-0.9.0.tar.gz'
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) Traceback (most recent call last):
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) File "/usr/lib/python2.7/site-packages/pulp_puppet/plugins/importers/importer.py", line 82, in upload_unit
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) conduit)
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) File "/usr/lib/python2.7/site-packages/pulp_puppet/plugins/importers/upload.py", line 43, in handle_uploaded_unit
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) shutil.move(file_path, new_file_path)
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) File "/usr/lib64/python2.7/shutil.py", line 301, in move
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) copy2(src, real_dst)
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) File "/usr/lib64/python2.7/shutil.py", line 130, in copy2
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) copyfile(src, dst)
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) File "/usr/lib64/python2.7/shutil.py", line 83, in copyfile
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) with open(dst, 'wb') as fdst:
mar 24 09:58:09 v-so-repo-01.company.com pulp[2696]: pulp_puppet.plugins.importers.importer:ERROR: (2696-36288) IOError: [Errno 2] No such file or directory: u'/var/lib/pulp/uploads/management/role-0.9.0.tar.gz'

This error is hidden on the client side and the upload returns success.

I think that the names are valid on puppet, as at least zypprepo module has the same naming convention: https://github.com/deadpoint/puppet-zypprepo/blob/master/metadata.json.

Actions #1

Updated by mhrivnak about 6 years ago

Any idea if such a module would pass validation on puppet forge?

Actions #2

Updated by mhrivnak about 6 years ago

  • Triaged changed from No to Yes

The incorrect reporting of success by pulp-admin should be fixed here: https://pulp.plan.io/issues/1747

Actions #3

Updated by mhrivnak about 6 years ago

  • Severity changed from 2. Medium to 1. Low
Actions #4

Updated by bmbouter about 3 years ago

  • Status changed from NEW to CLOSED - WONTFIX
Actions #5

Updated by bmbouter about 3 years ago

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

Actions #6

Updated by bmbouter about 3 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF