Issue #3090
closedUploading an invalid rpm produces an error message: "unexpected error occurred importing uploaded file: 'primary'"
Description
This is a regression that was originally fixed in https://pulp.plan.io/issues/2543. See comments below for the commit that broke this.
Steps to reproduce:
touch meerkat-2.1.0.noarch.rpm
pulp-admin rpm repo uploads rpm -f meerkat-2.1.0.noarch.rpm --repo-id zoo
I'd expect an error message along the lines of "couldn't parse rpm". Instead, the error message is:
Task Failed
The importer yum_importer indicated a failed response when uploading rpm unit to
repository test.
unexpected error occurred importing uploaded file: 'primary'
Related issues
Updated by daviddavis almost 7 years ago
Looks like the cause is this commit:
https://github.com/pulp/pulp_rpm/commit/62db898a81679b40e3cc3a48af9f3b830f4521cb
I reverted and the error message I get is unexpected error occurred importing uploaded file: error reading package header
.
Updated by daviddavis almost 7 years ago
- Related to Issue #3072: Too generic "The importer yum_importer indicated a failed response" added
Updated by daviddavis almost 7 years ago
- Related to deleted (Issue #3072: Too generic "The importer yum_importer indicated a failed response")
Updated by daviddavis almost 7 years ago
- Related to Issue #3072: Too generic "The importer yum_importer indicated a failed response" added
Updated by daviddavis almost 7 years ago
Katello requested that we catch this error and raise a PulpCodedException so they can read the error details from the task[:error][:description]
. I think that seems reasonable and something that we can address with this bug fix. If not, let me know and I can open another redmine issue.
Updated by daviddavis almost 7 years ago
- Description updated (diff)
- Priority changed from Normal to High
Updated by dalley almost 7 years ago
- Sprint/Milestone set to 46
- Triaged changed from No to Yes
Updated by jortel@redhat.com almost 7 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to jortel@redhat.com
Updated by jortel@redhat.com almost 7 years ago
- Status changed from ASSIGNED to POST
Added by jortel@redhat.com almost 7 years ago
Added by jortel@redhat.com almost 7 years ago
Revision 474781bf | View on GitHub
Propagate coded exceptions raised during upload. re: #3090
Updated by jortel@redhat.com almost 7 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp_rpm:4492f888df6be5a4d4933ef66be88bf92d4eedd5.
Updated by pthomas@redhat.com almost 7 years ago
verified
[root@intel-sharkbay-dh-02 ~]# rpm -qa |grep pulp
python-pulp-common-2.14.3-0.1.beta.el7.noarch
python-kombu-3.0.33-8.pulp.el7.noarch
pulp-puppet-plugins-2.14.3-0.1.beta.el7.noarch
pulp-docker-admin-extensions-3.0.2-1.el7.noarch
pulp-ostree-plugins-1.3.0-1.el7.noarch
python-pulp-rpm-common-2.14.3-0.1.beta.el7.noarch
pulp-python-admin-extensions-2.0.2-1.el7.noarch
python-pulp-docker-common-3.0.2-1.el7.noarch
pulp-selinux-2.14.3-0.1.beta.el7.noarch
python-pulp-oid_validation-2.14.3-0.1.beta.el7.noarch
pulp-rpm-plugins-2.14.3-0.1.beta.el7.noarch
pulp-admin-client-2.14.3-0.1.beta.el7.noarch
python-pulp-ostree-common-1.3.0-1.el7.noarch
python-pulp-python-common-2.0.2-1.el7.noarch
pulp-python-plugins-2.0.2-1.el7.noarch
python-pulp-streamer-2.14.3-0.1.beta.el7.noarch
python-pulp-repoauth-2.14.3-0.1.beta.el7.noarch
pulp-server-2.14.3-0.1.beta.el7.noarch
python-pulp-bindings-2.14.3-0.1.beta.el7.noarch
pulp-puppet-admin-extensions-2.14.3-0.1.beta.el7.noarch
pulp-ostree-admin-extensions-1.3.0-1.el7.noarch
python-pulp-puppet-common-2.14.3-0.1.beta.el7.noarch
python-pulp-client-lib-2.14.3-0.1.beta.el7.noarch
python-isodate-0.5.0-4.pulp.el7.noarch
pulp-docker-plugins-3.0.2-1.el7.noarch
pulp-rpm-admin-extensions-2.14.3-0.1.beta.el7.noarch
[root@intel-sharkbay-dh-02 ~]#
[root@intel-sharkbay-dh-02 ~]# touch meerkat-2.1.0.noarch.rpm
[root@intel-sharkbay-dh-02 ~]# pulp-admin rpm repo create --repo-id zoo
Successfully created repository [zoo]
[root@intel-sharkbay-dh-02 ~]# pulp-admin rpm repo uploads rpm -f meerkat-2.1.0.noarch.rpm --repo-id zoo
+----------------------------------------------------------------------+
Unit Upload
+----------------------------------------------------------------------+
Extracting necessary metadata for each request...
[==================================================] 100%
Analyzing: meerkat-2.1.0.noarch.rpm
... completed
Creating upload requests on the server...
[==================================================] 100%
Initializing: meerkat-2.1.0.noarch.rpm
... completed
Starting upload of selected units. If this process is stopped through ctrl+c,
the uploads will be paused and may be resumed later using the resume command or
canceled entirely using the cancel command.
Uploading: meerkat-2.1.0.noarch.rpm
... completed
Importing into the repository...
This command may be exited via ctrl+c without affecting the request.
[\]
Running...
Task Failed
Error reading uploaded package header.
Deleting the upload request...
... completed
Updated by daviddavis over 6 years ago
- Related to Issue #2543: RPM Importer swallows exception when one is raised during upload added
Updated by jortel@redhat.com about 6 years ago
Failed QE. Additional PR: https://github.com/pulp/pulp_rpm/pull/1126
Updated by jortel@redhat.com about 6 years ago
- Status changed from 5 to MODIFIED
Applied in changeset pulp_rpm:f3e618b5eb0a2195ce5b31f37fe7f5ddc120bd8f.
Updated by dkliban@redhat.com over 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Raise coded exception for invalid package header. closes #3090