Bug #6911
closed
Unsupported puppet version should be logged as warning only
Added by Marek Hulán almost 11 years ago.
Updated about 10 years ago.
Description
After updating puppet, I get the dreadfull "something went wrong message."
After checking the log there is only the puppet version error...
Can the "Your puppet version does not support progress bar" be set to
WARNING instead of an ERROR? I want to check to output for Successful
messages when deploying foreman automatically in the future.
Your puppet version does not support progress bar
Preparing installation Done
Something went wrong! Check the log for ERROR-level output
The full log is at /var/log/foreman-installer/foreman-installer.log
cat /var/log/foreman-installer/foreman-installer.log|grep ERR
[ERROR 2014-08-04 11:17:28 main] Your puppet version does not
support progress bar
[ INFO 2014-08-04 11:17:44 main] Puppet has finished, bye!
[ INFO 2014-08-04 11:17:44 main] Executing hooks in group post
[ INFO 2014-08-04 11:17:44 main] All hooks in group post finished
[DEBUG 2014-08-04 11:17:44 main] Cleaning /tmp/d20140804-12665-yz1tdt
[DEBUG 2014-08-04 11:17:44 main] Cleaning /tmp/default_values.yaml
I'd argue this isn't even a warning but rather a notice.
Not that it would matter that much but I think it should be rather warning. I expect to see progress bar but it's impossible on my env. We automatically recover from this error by using some fallback. I understand info/notice as informative message like "Starting installation". What are pros of notice?
I have no strong opinion on this, but I think it mostly depends on the value you add to the progressbar. Personally I think it's not that important and not at all important to see in the log, but the user would want to see it when actually running the installer.
I reckon it should be debug (or whatever level isn't visible) :)
If a user runs an installer with a version of Puppet that doesn't display the progress bar, I don't think we should bother them with the information unless they specifically ask.
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/kafo/pull/104 added
- Pull request deleted (
)
- Assignee set to Marek Hulán
PR implements it as a WARN, user won't see it (only in log files) as the only errors in normal mode we print are in error and hihger log level
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Also available in: Atom
PDF