Bug 1208638

Summary: apipie docs incorrect for DELETE /subscriptions/:id
Product: Red Hat Satellite Reporter: Tom McKay <tomckay>
Component: Content ManagementAssignee: Tom McKay <tomckay>
Status: CLOSED CURRENTRELEASE QA Contact: Tazim Kolhar <tkolhar>
Severity: high Docs Contact:
Priority: unspecified    
Version: 6.0.4CC: bkearney, cwelton, tkolhar, tomckay
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/9984
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-12 13:57:15 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tom McKay 2015-04-02 18:15:12 UTC
The apipie docs for unassociating subscriptions from content hosts is incorrect: the 'subscriptions' array parameter is required and does not align with the other route suggestions.

Comment 1 Tom McKay 2015-04-02 18:15:14 UTC
Created from redmine issue http://projects.theforeman.org/issues/9984

Comment 2 Tom McKay 2015-04-02 18:18:45 UTC
I would like to see this in 6.1 as it prevents apipie from removing subscriptions from content-hosts (part of hammer-cli-csv for export from SAM-1 and import to Sat-6). No other hammer commands use this route so risk minimal for fix.

Comment 4 Bryan Kearney 2015-04-02 20:06:18 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/9984 has been closed
-------------
Thomas McKay
Applied in changeset commit:katello|2cf41d8e184f73d909a55ae89b34ac72d03f275f.

Comment 6 Tazim Kolhar 2015-04-29 07:06:55 UTC
hi

please provide verifications steps

thanks

Comment 7 Tom McKay 2015-05-05 11:52:18 UTC
To verify change is included, visit /apidoc/v2 and note that the subscriptions destroy does not have the 'required' flag on id
See https://github.com/Katello/katello/pull/5162/files

Comment 8 Tazim Kolhar 2015-05-06 09:57:14 UTC
VERIFIED:

 rpm -qa  | grep foreman
ibm-hs22-02.rhts.eng.brq.redhat.com-foreman-proxy-client-1.0-1.noarch
foreman-gce-1.7.2.18-1.el7sat.noarch
foreman-postgresql-1.7.2.18-1.el7sat.noarch
foreman-compute-1.7.2.18-1.el7sat.noarch
ruby193-rubygem-foreman_discovery-2.0.0.12-1.el7sat.noarch
rubygem-hammer_cli_foreman_discovery-0.0.1.8-1.el7sat.noarch
foreman-libvirt-1.7.2.18-1.el7sat.noarch
foreman-vmware-1.7.2.18-1.el7sat.noarch
ruby193-rubygem-foreman-redhat_access-0.1.0-1.el7sat.noarch
ruby193-rubygem-foreman-tasks-0.6.12.4-1.el7sat.noarch
rubygem-hammer_cli_foreman_bootdisk-0.1.2.6-1.el7sat.noarch
ruby193-rubygem-foreman_bootdisk-4.0.2.12-1.el7sat.noarch
ruby193-rubygem-foreman_docker-1.2.0.10-1.el7sat.noarch
rubygem-hammer_cli_foreman-0.1.4.10-1.el7sat.noarch
foreman-debug-1.7.2.18-1.el7sat.noarch
foreman-proxy-1.7.2.4-1.el7sat.noarch
ibm-hs22-02.rhts.eng.brq.redhat.com-foreman-client-1.0-1.noarch
ibm-hs22-02.rhts.eng.brq.redhat.com-foreman-proxy-1.0-2.noarch
foreman-1.7.2.18-1.el7sat.noarch
ruby193-rubygem-foreman_hooks-0.3.7-2.el7sat.noarch
rubygem-hammer_cli_foreman_tasks-0.0.3.4-1.el7sat.noarch
ruby193-rubygem-foreman_gutterball-0.0.1.9-1.el7sat.noarch
foreman-selinux-1.7.2.13-1.el7sat.noarch
puppet-foreman_scap_client-0.3.3-8.el7sat.noarch
foreman-ovirt-1.7.2.18-1.el7sat.noarch

steps
 visit /apidoc/v2 and note that the subscriptions destroy does not have the 'required' flag on id . id is optional

Comment 9 Bryan Kearney 2015-08-11 13:21:23 UTC
This bug is slated to be released with Satellite 6.1.

Comment 10 Bryan Kearney 2015-08-12 13:57:15 UTC
This bug was fixed in version 6.1.1 of Satellite which was released on 12 August, 2015.