Project

Profile

Help

Issue #1073

wrong error msg on delete rpm from nonexisting repo

Added by igulina@redhat.com over 5 years ago. Updated almost 2 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.7.0 Beta
Platform Release:
2.8.0
OS:
RHEL 7
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Pulp 2
Sprint:
Quarter:

Description

If to delete a rpm from a nonexisting repo, there is no error message that such a repo doesn't exist. Instead it says "Nth was found". A user can take it as a rpm s/he wants to delete doesn't exist. I expect a msg that a repo is not found.

>> rpm -qa pulp-server
pulp-server-2.7.0-0.3.beta.el7.noarch

>> pulp-admin rpm repo list
+----------------------------------------------------------------------+
                            RPM Repositories
+----------------------------------------------------------------------+

Id:                  zoo_repo
Display Name:        zoo_repo
Description:         None
Content Unit Counts: 
  Erratum:          4
  Package Category: 1
  Package Group:    2
  Rpm:              32

>> pulp-admin rpm repo remove rpm --repo-id zoo --str-eq='name=dog'
This command may be exited via ctrl+c without affecting the request.

[\]
Running...

Nothing found that matches the given criteria.

I expect error that repo is not found as for

>> pulp-admin rpm repo sync run --repo-id zoo
+----------------------------------------------------------------------+
                     Synchronizing Repository [zoo]
+----------------------------------------------------------------------+

The following resource(s) could not be found:

  zoo (repo_id)

History

#2 Updated by amacdona@redhat.com over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to abradshaw@redhat.com

#3 Updated by amacdona@redhat.com over 5 years ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (abradshaw@redhat.com)

#4 Updated by bmbouter over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to amacdona@redhat.com
  • Triaged changed from No to Yes

#5 Updated by amacdona@redhat.com over 5 years ago

This bug can be fixed easily, but it should wait until https://github.com/pulp/pulp/pull/1931 has been merged to avoid fixing the same problem twice, once with pymongo, once with mongoengine.

#6 Updated by amacdona@redhat.com over 5 years ago

  • Status changed from ASSIGNED to MODIFIED

#7 Updated by amacdona@redhat.com over 5 years ago

  • Status changed from MODIFIED to NEW

#8 Updated by amacdona@redhat.com over 5 years ago

  • Assignee deleted (amacdona@redhat.com)

#9 Updated by amacdona@redhat.com over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to amacdona@redhat.com
  • Platform Release set to 2.8.0

#10 Updated by amacdona@redhat.com over 5 years ago

  • Status changed from ASSIGNED to MODIFIED

#11 Updated by dkliban@redhat.com almost 5 years ago

  • Status changed from MODIFIED to 5

#12 Updated by dkliban@redhat.com almost 5 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#13 Updated by bmbouter almost 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF