Project

General

Profile

Actions

Bug #31979

open

/api/v2/reports (used by Ansible) wants the client smart proxy to have 'Puppet' feature

Added by Dominique Quatravaux about 3 years ago. Updated about 3 years ago.

Status:
Ready For Testing
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

What I attempted:

  • set up a smart proxy with Ansible feature on, but Puppet feature off
  • ran Ansible role
What I expected to happen:
  • facts should appear in the Foreman database
  • report should appear in the Foreman database
What happened instead:
  • facts did appear in the Foreman database
  • I see a 403 error in the logs regarding /api/v2/reports
Actions #1

Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8361 added
Actions #2

Updated by Dominique Quatravaux about 3 years ago

So I made https://github.com/theforeman/foreman/pull/8361 and it does fix the problem at hand; however I'm sure there is room for improvement:
- I have no idea if and how to move this change into the foreman_ansible plugin
- I assume other consumers of /api/v2/reports (Chef? Salt?) will want something similar

Thanks for any suggestion, I'll be happy to revisit my PR!

Actions #3

Updated by Dominique Quatravaux about 3 years ago

  • Pull request https://github.com/theforeman/foreman_ansible/pull/388 added
  • Pull request deleted (https://github.com/theforeman/foreman/pull/8361)
Actions #4

Updated by Dominique Quatravaux about 3 years ago

Please close in favor of #31980

Actions

Also available in: Atom PDF