Fixes #7882 Added support for OracleLinux

[soumentrivedi]

Following on from earlier pull request #40 this is a clean and complete request now.

[ehelms]

@soumentrivedi do you think we should add a basic test for this similar to https://github.com/Katello/puppet-katello/blob/master/spec/classes/katello_spec.rb#L5 ?

[stbenjam]

Is there a reason to add these individually? There's a lot of Enterprise clones that may just work with Katello, and you can just catch them all:

$::osfamily == "RedHat" and $::operatingsystem != "Fedora"

[soumentrivedi]

Thanks @stbenjam I have changed that logic as it makes more sense. Also @ehelms I have updated the tests to include a context for OEL. Also squashed the commits into one :)

[soumentrivedi]

hello guys, would it be safe to assume that this pull request is just awaiting to be merged as I would like to start using the changes done in this fork of the branch for my tests till v2.0.1 is released and available through standard yum repositories?

[ekohl]

Adds trailing whitespace

[ehelms]

Everything looks good to me. ACK and merging after the trailing whitespace fix.

[soumentrivedi]

@ehelms I have done the changes and it is awaiting pull.

[ekohl]

👍

[ekohl]

In #44 I did some further cleaning and included these changes.

[ehelms]

@ekohl so both of these PRs are not needed correct? Your preference would be the style of #44 ?

[ekohl]

@ehelms yes, this PR in itself is good, but I was annoyed by the duplication of logic in install.pp.

[ehelms]

@soumentrivedi Thanks! Merging.