Project

General

Profile

Bug #15797

Rather than failing silently, need user friendly error message on GUI/hammer while falling to sync gitlab/third-party repository on the Satellite

Added by Andrew Kofink almost 6 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
Repositories
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

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
~


Related issues

Blocks Katello - Tracker #15825: Sprint 1 : Content-FocusedNew2016-07-24

History

#1 Updated by Brad Buckingham almost 6 years ago

#2 Updated by Justin Sherrill almost 6 years ago

  • Description updated (diff)

#3 Updated by Andrew Kofink almost 6 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}}

#4 Updated by Andrew Kofink almost 6 years ago

  • Status changed from New to Rejected
  • Legacy Backlogs Release (now unused) set to 166

Moved this issue to Pulp.

#5 Updated by Brad Buckingham almost 6 years ago

  • Category changed from Repositories to 91

Also available in: Atom PDF