Bug 1185011

Summary:	pulp-admin says sync schedule deleted succesfully for non-existing schedule id
Product:	[Retired] Pulp	Reporter:	Dennis Kliban <dkliban>
Component:	async/tasks	Assignee:	Ina Panova <ipanova>
Status:	CLOSED UPSTREAM	QA Contact:	Irina Gulina <igulina>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	2.6 Beta	CC:	bmbouter, igulina, ipanova, skarmark
Target Milestone:	---	Keywords:	Triaged
Target Release:	2.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-28 23:20:56 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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