Bug 1791658 - drop reports api endpoint
Summary: drop reports api endpoint
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: API
Version: 6.7.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: 6.8.0
Assignee: Tomer Brisker
QA Contact: Lukáš Hellebrandt
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-01-16 10:33 UTC by Tomer Brisker
Modified: 2021-12-01 10:47 UTC (History)
5 users (show)

Fixed In Version: foreman-2.0.0
Doc Type: Removed functionality
Doc Text:
The `/api/reports/` API endpoint was deprecated in Satellite 6.7 and is now removed in Satellite 6.8. Use the `/api/config_reports/` API endpoint instead.
Clone Of:
Environment:
Last Closed: 2020-10-27 12:59:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 28594 0 Normal Closed drop reports api endpoint 2020-10-28 17:22:17 UTC
Red Hat Product Errata RHSA-2020:4366 0 None None None 2020-10-27 13:00:13 UTC

Description Tomer Brisker 2020-01-16 10:33:29 UTC
it was renamed to config_reports in #4151.
We need to maintain the post endpoint since it is still used by ansible - https://github.com/ansible/ansible/pull/64955 is open to fix that

Comment 1 Tomer Brisker 2020-01-16 10:33:35 UTC
Created from redmine issue https://projects.theforeman.org/issues/28594

Comment 2 Tomer Brisker 2020-01-16 10:33:38 UTC
Upstream bug assigned to tbrisker

Comment 5 Bryan Kearney 2020-01-16 11:06:42 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/28594 has been resolved.

Comment 11 Lukáš Hellebrandt 2020-06-30 14:23:48 UTC
FailedQA with Sat 6.8 snap 6.0.

There is no /api/report API endopoint shown in apidoc. However, it should be there as per OP and due to the fact that the PR mentioned there is not in downstream yet:

# grep \"config_report\": /usr/lib/python2.7/site-packages/ansible/plugins/callback/foreman.py
=> empty

Comment 12 Tomer Brisker 2020-07-01 09:35:40 UTC
The endpoint was removed from the apidoc since it is no longer supported, however it is still there in the API exactly for the purpose of maintaining compatibility with older versions of Ansible (and other report sources which use it). You can check and see that POST to /api/report still works for sending a report to Satellite, despite the supported endpoint being /api/config_report.

Comment 13 Lukáš Hellebrandt 2020-07-01 11:51:59 UTC
Thanks, Tomer, that makes sense.

Verified with Sat 6.8 snap 6.0. All "config_reports" endpoints in apidoc work and their "reports" counterparts do not, except for the POST one which works.

Comment 16 errata-xmlrpc 2020-10-27 12:59:59 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: Satellite 6.8 release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:4366


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