Bug #31504: hammer content-import -: do not check for 'pulp' things - Katello - Foreman

Actions

Copy link

Bug #31504

closed

hammer content-import -: do not check for 'pulp' things

Added by Partha Aji about 4 years ago. Updated about 4 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Partha Aji

Category:

Inter Server Sync

Target version:

Katello 3.18.1

Difficulty:

Triaged:

Yes

Bugzilla link:

Pull request:

https://github.com/Katello/katello/pull/9077

Fixed in Releases:

Katello 4.0.0

Found in Releases:

Red Hat JIRA:

Description

When importing from a dump we see the following error

$ hammer content-import version --content-view="Test CV" --organization="Import Organization" --metadata-file=/home/vagrant/forklift-devel/bats/metadata-1.json --path=/var/lib/pulp/exports/Test_Organization/Test_CV/1.0/2020-12-11T22-46-12-00-00
Could not import the archive.:
  Invalid path specified: '/var/lib/pulp/exports/Test_Organization/Test_CV/1.0/2020-12-11T22-46-12-00-00' is not a directory

$ sudo ls -l /var/lib/pulp/exports/Test_Organization/Test_CV/1.0/2020-12-11T22-46-12-00-00
total 52
-rw-r--r--. 1 pulp pulp 46855 Dec 11 22:46 export-860c625d-53f4-4ada-a3a3-56f516cdadcf-20201211_2246.tar.gz
-rw-r--r--. 1 pulp pulp   333 Dec 11 22:46 export-860c625d-53f4-4ada-a3a3-56f516cdadcf-20201211_2246-toc.json

Even though the import path exists its not accessible by the foreman user and hence you get the invalid path error.
We need to instead make use of https://pulp.plan.io/issues/7549?pn=1#note-10 (when it becomes available) and remove the foreman server side checks for permissions in those directories