[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cobbler-devel
Subject:    Re: [PATCH] Setup to create a separate cobbler-puppet pacakge for
From:       Todd Zullinger <tmz () pobox ! com>
Date:       2011-01-28 22:16:04
Message-ID: 20110128221604.GR22947 () inocybe ! localdomain
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Scott Henson wrote:
> On Fri, 28 Jan 2011 14:37:08 -0600, Greg Swift <gregswift@gmail.com>
> wrote:
>> Hi all.  This is a patch I put together that might be useful for
>> letting puppet access cobbler using the ext-nodes script without
>> having to move files (binary and config) from your cobbler server
>> to install on your puppet server, most specifically with an RPM.  I
>> made the script search for the config file, so that should help
>> make it work in several use cases.
>>
>> I'm using this script in my environment now, and seems to be more
>> than adequate, however this is just one use case.
>
> I like it, a couple of things though. Does cobbler-ext-nodes
> actually require puppet? I would think it would be puppet-server
> that makes use of it, and even then, the cobbler-puppet package
> should not require puppet-server, me thinks.

And, just from a packaging perspective, if it doesn't require puppet
or puppet-server, it should also own /etc/puppet, to ensure the dir is
not left around should cobbler-puppet be installed without puppet.
E.g: %dir %{_sysconfdir}/puppet

--=20
Todd        OpenPGP -> KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
If a government were put in charge of the Sahara Desert, within five
years they'd have a shortage of sand.
    -- Dr. Milton Friedman


[Attachment #5 (application/pgp-signature)]

_______________________________________________
cobbler-devel mailing list
cobbler-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/cobbler-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic