Re: rbac_playbook fix for RHEL7

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 29 Sep 2015 17:26:36 -0400
Ralph Bean <rbean@xxxxxxxxxx> wrote:

> On Tue, Sep 29, 2015 at 01:45:15PM -0600, Tim Flink wrote:
> > > It looks like this function used to return False if an invalid
> > > playbook_file was passed to it, but now it will raise a
> > > KeyError.  Is that intentional?  I see there is a test case for
> > > it, but the `rbac_playbook` function doesn't handle it.
> > 
> > KeyError now handled and test case added, thanks.
> > 
> > Tim
> 
> Looks good now.  +1, and thanks!

Yeah, seems good to me too. ;) 

kevin

Attachment: pgp9fVjqbSI12.pgp
Description: OpenPGP digital signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
http://lists.fedoraproject.org/postorius/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux