Bug #2244
closed
avoid flapping os.release_name for Debian
Added by Florian Ernst almost 12 years ago.
Updated over 11 years ago.
Description
If some hosts provide facts[:lsbdistcodename] but other hosts don't,
os.release_name constantly changes when importing the facts, thus
filling your audit logs with useless massages.
The attached patch saved me a ton of audit logs, please consider including it.
Files
JFTR, this issue is still valid with 1.2.2, and my patch still fixes it.
- Description updated (diff)
Would you mind submitting this as a pull request to the project on GitHub? It's our normal way to get patches included in Foreman.
There are some instructions on the contributing page on the main website: http://theforeman.org/contribute.html
Thanks for the reply.
Well, for an occasional contributer that's an awful lot of hoops to jump through for single line patch:
I sent it via mail and was told to file it in an issue, so I needed to create a redmine account. And now,
after the issue remained untouched for months, I apparently need github as well to get things going.
Please bear in mind that I got in contact with the intention to help (lest my patch remains private), but
it seems I have to go to lengths to present my help (as little as it is) on a silver platter ...
Considering that, it'd probably be easier for me to just individually apply my patch after each foreman
upgrade, and that's a notion I don't like myself to think. So I'll see what I can do, but it might take a
while to get acquainted.
- Status changed from New to Assigned
- Assignee set to Sam Kottler
- Target version set to 1.3.0
- Status changed from Assigned to Ready For Testing
- Target version changed from 1.3.0 to 1.4.0
- Related to Tracker #3112: [TRACKER] Issues to be released in 1.3 RC or final added
- Target version changed from 1.4.0 to 1.3.0
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
- Related to deleted (Tracker #3112: [TRACKER] Issues to be released in 1.3 RC or final)
Also available in: Atom
PDF