Project

General

Profile

Bug #7093

Hammer remove-sync-plan adds sync-plan instead of removing it

Added by Adam Price about 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1121136
Description of problem:

1) afaik, product can have at most one syncplan assigned to itself

2) therefore I should be able to "hammer -d -v -u admin -p changeme product remove-sync-plan --id='95'" to remove any asociated syncplan

3) in reality, error "(option '--sync_plan_id' is required)" is returned

4) if I supply valid sync-plan-id it is not removed, but set as the products sync-plan

i.e: hammer -d -v -u admin -p changeme product remove-sync-plan --id='95' --sync-plan-id='14'

actually sends:

[ INFO 2014-07-18 08:49:11 API] PUT /katello/api/products/95
[DEBUG 2014-07-18 08:49:11 API] Params: {
"sync_plan_id" => "14"
}

Associated revisions

Revision 6f7e3d11 (diff)
Added by Adam Price about 6 years ago

fixes #7093 - properly remove sync-plan from a product, BZ 1121136

Revision d5f740e6
Added by Adam Price about 6 years ago

Merge pull request #222 from komidore64/rmi7093-remove-sync-plan-broken

fixes #7093 - properly remove sync-plan from a product, BZ 1121136

History

#1 Updated by The Foreman Bot about 6 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 54
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/222 added
  • Pull request deleted ()

#2 Updated by Anonymous about 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#3 Updated by Eric Helms about 6 years ago

  • Triaged changed from No to Yes

#4 Updated by Eric Helms about 6 years ago

  • Legacy Backlogs Release (now unused) set to 13

Also available in: Atom PDF