Bug 1185011 - pulp-admin says sync schedule deleted succesfully for non-existing schedule id
Summary: pulp-admin says sync schedule deleted succesfully for non-existing schedule id
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Pulp
Classification: Retired
Component: async/tasks
Version: 2.6 Beta
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.6.0
Assignee: Ina Panova
QA Contact: Irina Gulina
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-22 17:19 UTC by Dennis Kliban
Modified: 2015-02-28 23:20 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-02-28 23:20:56 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Pulp Redmine 674 0 None None None Never

Description Dennis Kliban 2015-01-22 17:19:54 UTC
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. pulp-admin rpm repo sync schedules delete --schedule-id 54bff6f0421aa9725f7e0ab1 --repo-id zoo
2. pulp-admin rpm repo sync schedules delete --schedule-id 54bff6f0421aa9725f7e0ab1 --repo-id zoo
3. "Schedule successfully deleted" is displayed both times the command is executed with the same schedule-id.

Actual results:


Expected results:


Additional info:

Comment 1 Brian Bouterse 2015-01-26 17:13:54 UTC
The schedule id is used in the URL so it should return a 404. I think raising a MissingResource would cause this to be represented better (no ugly traceback) and give it a 404 response code.

Comment 2 Ina Panova 2015-02-05 17:18:12 UTC
https://github.com/pulp/pulp/pull/1614

Comment 3 Ina Panova 2015-02-09 17:42:34 UTC
this one https://github.com/pulp/pulp/pull/1618

Comment 4 Chris Duryee 2015-02-10 22:33:02 UTC
2.6.0-0.7.beta

Comment 5 Irina Gulina 2015-02-13 15:47:49 UTC
>> rpm -qa pulp-server
pulp-server-2.6.0-0.7.beta.el6.noarch

>> pulp-admin rpm repo sync schedules create --repo-id gena --schedule=2015-02-13T10:35:00Z/PT1M
Schedule successfully created

>> pulp-admin rpm repo sync schedules list --repo-id gena
+----------------------------------------------------------------------+
                               Schedules
+----------------------------------------------------------------------+

Schedule: 2015-02-13T10:35:00Z/PT1M
Id:       54de1bb5422f0704c03b99fe
Enabled:  True
Next Run: 2015-02-13T15:45:00Z

>> pulp-admin rpm repo sync schedules delete --schedule-id 54de1bb5422f0704c03b99fe --repo-id gena
Schedule successfully deleted

>> pulp-admin rpm repo sync schedules delete --schedule-id 54de1bb5422f0704c03b99fe --repo-id gena
The following resource(s) could not be found:

  54de1bb5422f0704c03b99fe (schedule_id)

>> pulp-admin rpm repo sync schedules list --repo-id gena+----------------------------------------------------------------------+
                               Schedules
+----------------------------------------------------------------------+

There are no schedules defined for this operation

Comment 6 Brian Bouterse 2015-02-28 23:20:56 UTC
Moved to https://pulp.plan.io/issues/674


Note You need to log in before you can comment on or make changes to this bug.