Bug #15797
closedRather than failing silently, need user friendly error message on GUI/hammer while falling to sync gitlab/third-party repository on the Satellite
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1294196
1. Proposed title of this feature request
[RFE]User friendly error message on GUI/hammer while falling to sync gitlab/third-part repository on the Satellite.
3. What is the nature and description of the request?
we should have user friendly error message while falling to sync gitlab/third-party repository on the Satellite as it fail silently without any error message.
4. Why does the customer need this?
For better customer experience.
5. How would the customer like to achieve this? (List the functional requirements here)
By reflecting the error message on GUI when gitlab/third-party repository fail to sync.
6. For each functional requirement listed, specify how Red Hat and the customer can test to confirm the requirement is successfully implemented.
Create a product and custom repository in satellite gui/hammer with following link to a repo: https://packages.gitlab.com/gitlab/gitlab-ce/el/7/x86_64 and then try to sync that repository and check if we can see any error message.
Note: Satellite requires repo metadata, specifically <checksum>-filelists.xml file to sync repo and gitlab repository doesn't provide a valid filelists.xml database therefore we won't able to sync gitlab repository on the Satellite. Please refer below Bugzilla for more information:
~
https://bugzilla.redhat.com/show_bug.cgi?id=1267579
~
Updated by Brad Buckingham over 8 years ago
- Blocks Tracker #15825: Sprint 1 : Content-Focused added
Updated by Andrew Kofink over 8 years ago
The error the user sees is "Katello::Errors::PulpError: PLP0000: Importer indicated a failed response". Below is the full output from pulp tasks:
{"pulp_tasks"=> [{"exception"=>nil, "task_type"=>"pulp.server.managers.repo.sync.sync", "_href"=>"/pulp/api/v2/tasks/4d5d89f1-130c-4d03-95d0-4138d2e5bddf/", "task_id"=>"4d5d89f1-130c-4d03-95d0-4138d2e5bddf", "tags"=> ["pulp:repository:Default_Organization-15798-nonexistant2", "pulp:action:sync"], "finish_time"=>"2016-08-04T12:20:12Z", "_ns"=>"task_status", "start_time"=>"2016-08-04T12:20:06Z", "traceback"=> "Traceback (most recent call last):\n File \"/usr/lib/python2.7/site-packages/celery/app/trace.py\", line 240, in trace_task\n R = retval = fun(*args, **kwargs)\n File \"/usr/lib/python2.7/site-packages/pulp/server/async/tasks.py\", line 473, in __call__\n return super(Task, self).__call__(*args, **kwargs)\n File \"/usr/lib/python2.7/site-packages/pulp/server/async/tasks.py\", line 103, in __call__\n return super(PulpTask, self).__call__(*args, **kwargs)\n File \"/usr/lib/python2.7/site-packages/celery/app/trace.py\", line 437, in __protected_call__\n return self.run(*args, **kwargs)\n File \"/usr/lib/python2.7/site-packages/pulp/server/controllers/repository.py\", line 810, in sync\n raise pulp_exceptions.PulpExecutionException(_('Importer indicated a failed response'))\nPulpExecutionException: Importer indicated a failed response\n", "spawned_tasks"=>[], "progress_report"=> {"yum_importer"=> {"content"=> {"size_total"=>43534014742, "items_left"=>140, "items_total"=>140, "state"=>"FAILED", "size_left"=>43534014742, "details"=> {"rpm_total"=>140, "rpm_done"=>0, "drpm_total"=>0, "drpm_done"=>0}, "error"=>"'\\xff\\xff\\xff\\xff'", "error_details"=>[]}, "comps"=>{"state"=>"NOT_STARTED"}, "purge_duplicates"=>{"state"=>"NOT_STARTED"}, "distribution"=> {"items_total"=>0, "state"=>"NOT_STARTED", "error_details"=>[], "items_left"=>0}, "errata"=>{"state"=>"NOT_STARTED"}, "metadata"=>{"state"=>"FINISHED"}}}, "queue"=>"reserved_resource_worker-1@akofink-dev.usersys.redhat.com.dq", "state"=>"error", "worker_name"=>"reserved_resource_worker-1@akofink-dev.usersys.redhat.com", "result"=>nil, "error"=> {"code"=>"PLP0000", "data"=>{}, "description"=>"Importer indicated a failed response", "sub_errors"=>[]}, "_id"=>{"$oid"=>"57a332f69321dd5b3a64f454"}, "id"=>"57a332f69321dd5b3a64f454"}], "contents_changed"=>true, "poll_attempts"=>{"total"=>6, "failed"=>1}}
Updated by Andrew Kofink over 8 years ago
- Status changed from New to Rejected
- Translation missing: en.field_release set to 166
Moved this issue to Pulp.
Updated by Brad Buckingham over 8 years ago
- Category changed from Repositories to 91