Bug #11704
closed
Configuration no report appears as "green" icon
Added by Ohad Levy over 9 years ago.
Updated over 6 years ago.
Description
After upgrading to the global host status, hosts who has no puppet report appear as green vs warning.
see attachment.
Files
- Category set to Reporting
- Translation missing: en.field_release set to 63
I think the intention is not to put hosts into a warning state when perhaps they don't even use configuration management.
However, we could implement #relevant? on ConfigurationStatus to be true when puppet? is true (and let other config management plugins to add their own conditions), then set it to warning if there are no reports.
So we should probably distinguish between a host with puppet master and the rest, or is there better way to resolve "#puppet?". Otherwise we'd mark discovered hosts or hosts not using puppet with warning flag.
I think checking puppet_proxy (sorry, not #puppet?) is the only thing we usually do, e.g. on the host#edit form for whether to show Puppet-related tabs. Host status shouldn't apply to discovered hosts, but I think it's worth distinguishing between a host that should be sending reports and one that shouldn't.
- Assignee set to Marek Hulán
OK, that makes sense to me and you're right about discovered hosts. I'll probably get to it this week.
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/2705 added
- Pull request deleted (
)
- % Done changed from 0 to 100
- Status changed from Ready For Testing to Closed
- Related to Bug #11791: N+1 query on hosts#index retrieving puppet_proxy added
Also available in: Atom
PDF