Bug 1791658

Summary: drop reports api endpoint
Product: Red Hat Satellite Reporter: Tomer Brisker <tbrisker>
Component: APIAssignee: Tomer Brisker <tbrisker>
Status: CLOSED ERRATA QA Contact: Lukáš Hellebrandt <lhellebr>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.7.0CC: apatel, egolov, gtalreja, spetrosi, ssainkar
Target Milestone: 6.8.0Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
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.
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-10-27 12:59:59 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 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