Project

General

Profile

Actions

Bug #2244

closed

avoid flapping os.release_name for Debian

Added by Florian Ernst over 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Sam Kottler
Category:
Audit Log
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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

Actions #1

Updated by Florian Ernst almost 11 years ago

JFTR, this issue is still valid with 1.2.2, and my patch still fixes it.

Actions #2

Updated by Dominic Cleal almost 11 years ago

  • 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

Actions #3

Updated by Florian Ernst almost 11 years ago

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.

Actions #4

Updated by Sam Kottler almost 11 years ago

  • Status changed from New to Assigned
  • Assignee set to Sam Kottler
  • Target version set to 1.3.0

Florian,

Sorry about the confusion - our contribution process has been the same for a while (outlined in http://projects.theforeman.org/projects/foreman/wiki/Contribute#Submit-Patches). The patch is appreciated!

I'll submit this as a pull request on your behalf and when it gets merged we can give you authorship in Git.

Actions #5

Updated by Sam Kottler almost 11 years ago

  • Status changed from Assigned to Ready For Testing
Actions #6

Updated by Lukas Zapletal almost 11 years ago

  • Target version changed from 1.3.0 to 1.4.0
Actions #7

Updated by Dominic Cleal over 10 years ago

  • Related to Tracker #3112: [TRACKER] Issues to be released in 1.3 RC or final added
Actions #8

Updated by Dominic Cleal over 10 years ago

  • Target version changed from 1.4.0 to 1.3.0
Actions #9

Updated by Anonymous over 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #10

Updated by Lukas Zapletal over 10 years ago

  • Related to deleted (Tracker #3112: [TRACKER] Issues to be released in 1.3 RC or final)
Actions

Also available in: Atom PDF