Bug 1208638 - apipie docs incorrect for DELETE /subscriptions/:id
Summary: apipie docs incorrect for DELETE /subscriptions/:id
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Content Management
Version: 6.0.4
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: Unspecified
Assignee: Tom McKay
QA Contact: Tazim Kolhar
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-04-02 18:15 UTC by Tom McKay
Modified: 2017-02-23 20:11 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-12 13:57:15 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 9984 0 None None None 2016-04-22 16:15:34 UTC

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.


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