Project

General

Profile

Bug #24556

origin of config report is N/A

Added by Hart Mel 6 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Marek Hulán
Category:
Puppet Reports
Target version:
1.19.0
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/5952, https://github.com/theforeman/foreman/pull/5941
Team Backlog:
Fixed in Releases:
1.19.0
Found in Releases:
1.18.0, 1.18.0-RC1, 1.18.0-RC2, 1.18.0-RC3, 1.18.1, 1.18.2, 1.19.0, 1.19.0-RC1, 1.19.0-RC2

Description

When a puppet report contains only two lines like the following example, the column "Origin" of the report table shows N/A, though it s a report from Puppet.

content changed '{md5}085471495a8a34d24cfda019bccbe7ad' to '{md5}fc037522c6f4a510394eaeb352067ff6'
Puppet     Applied catalog in 0.32 seconds

Associated revisions

Revision 3af8b327 (diff)
Added by Marek Hulán 5 months ago

Fixes #24556 - correct puppet origin detection

Revision 684be3c7 (diff)
Added by Tomer Brisker 5 months ago

Refs #24556 - correct report scanner test (#5952)

History

#1 Updated by Hart Mel 6 months ago

replacing .first by .last in /usr/share/foreman/app/services/report_scanner/puppet_report_scanner.rb fixed it. I m not sure if this hack is the way to handle report logs..

#2 Updated by The Foreman Bot 5 months ago

  • Assignee set to Marek Hulán
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5941 added

#3 Updated by Marek Hulán 5 months ago

Thanks, I was able to reproduce. Could you please check whether the patch fixes the problem for you?

#4 Updated by Tomer Brisker 5 months ago

  • Fixed in Releases 1.19.0 added

#5 Updated by Tomer Brisker 5 months ago

  • Target version set to 1.19.0
  • Fixed in Releases deleted (1.19.0)

#6 Updated by Hart Mel 5 months ago

Yes, it fixes the problem.

#7 Updated by Tomer Brisker 5 months ago

  • Found in Releases 1.18.0, 1.18.0-RC1, 1.18.0-RC2, 1.18.0-RC3, 1.18.1, 1.18.2, 1.19.0, 1.19.0-RC2 added
  • Fixed in Releases 1.19.0, 1.20.0 added

#8 Updated by Marek Hulán 5 months ago

  • Status changed from Ready For Testing to Closed

#9 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/theforeman/foreman/pull/5952 added

Also available in: Atom PDF