Actions
Issue #4140
closedPuppet fails to find metadata file if there are multiple top-level files/dirs
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.18.0
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 45
Quarter:
Description
Steps to reproduce:
wget https://forge.puppet.com/v3/files/marcel-passenger-0.5.0.tar.gz
pulp-admin puppet repo create --repo-id marcelpuppet
pulp-admin puppet repo uploads upload --file marcel-passenger-0.5.0.tar.gz --repo-id marcelpuppet
pulp-admin puppet repo list
Result:
Task Failed
Could not find metadata file inside Puppet module
Related issues
Updated by daviddavis almost 6 years ago
- Related to Issue #1846: pulp does not sync puppetforge correctly added
Updated by daviddavis almost 6 years ago
- Status changed from ASSIGNED to POST
Added by daviddavis almost 6 years ago
Updated by daviddavis almost 6 years ago
- Subject changed from Puppet fails to find metadata file to Puppet fails to find metadata file if there are multiple top-level files/dirs
Updated by CodeHeeler almost 6 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 45
Updated by daviddavis almost 6 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp_puppet:e1123708f173e0579048927dd206013878c6f78a.
Updated by ttereshc over 5 years ago
- Project changed from Pulp to Puppet Support
Updated by ttereshc over 5 years ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
Actions
Fixed bug where path wasn't used and module_dir was set to non-directory
There were two bugs here at play. First the code wasn't using the full path including extraction_dir when checking if the module file was a directory and secondly, the loop meant that module_dir could be set to a file if it didn't find a directory.
Also, fixed some flake8 errors.
fixes #4140 https://pulp.plan.io/issues/4140