[Freeipa-devel] [PATCH] Allow ipa-getkeytab to find server name from config file

[Simo Sorce]

On Wed, 2015-11-25 at 14:34 +1000, Fraser Tweedale wrote:
> On Tue, Nov 24, 2015 at 02:36:17PM -0500, Simo Sorce wrote:
> > On Tue, 2015-11-24 at 17:34 +0100, Jan Cholasta wrote:
> > > On 24.11.2015 17:30, Simo Sorce wrote:
> > > > On Tue, 2015-11-24 at 09:14 +0100, Jan Cholasta wrote:
> > > >> On 24.11.2015 09:06, Petr Spacek wrote:
> > > >>> On 24.11.2015 07:32, Jan Cholasta wrote:
> > > >>>> On 23.11.2015 21:18, Simo Sorce wrote:
> > > >>>>> Fixes #2203 by reading the server name from /etc/ipa/default.conf if not
> > > >>>>> provided on the command line.
> > > >>>>>
> > > >>>>> Simo.
> > > >>>>
> > > >>>> Just a thought: it would be nice if we had libipaconfig and used it everywhere
> > > >>>> (the framework, ipa-getkeytab, certmonger, ...). I don't like that there are
> > > >>>> at least 3 ipa config parser implementations now, each with its own quirks.
> > > >>>
> > > >>> Yeah, define a Augeas grammar for IPA config and call it. There is C and
> > > >>> Python binding in RHEL 7.2 already.
> > > >>
> > > >> This won't be sufficient, as it should go beyond just parsing the config
> > > >> file - there are dynamic defaults and backward compatiblity support
> > > >> which should be the same everywhere too.
> > > >
> > > > This is a much bigger task, that someone keen on big refactoring may
> > > > undertake, but I do not think we should delay this useful feature for
> > > > that. It will be really easy to change this code to use whatever other
> > > > ipa library when it materializes.
> > > >
> > > > For now this patch stands IMHO.
> > > 
> > > Sure, I'm not suggesting otherwise.
> > 
> > Rebased patch that includes proper spec file buildrequires.
> > 
> > Simo.
> > 
> Tested and works as expected (great usability win IMO).
> 
> Man page should be updated accordingly.
> 
> Cheers,
> Fraser

Attached new patch that includes manpage changes.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-simo-555-3-Support-sourcing-the-IPA-server-name-from-config.patch
Type: text/x-patch
Size: 9815 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151125/fbcfa34a/attachment.bin>