Hello,

Le 07/11/11 à 15:30, jcbollinger téléscripta :
> For the record, that is not an implementation of my recommendation.
> It is not safe for class sudo::apdater to include that new class,
> except in the most technical of senses.

Indeed.

> The point is to have a source of truth separate from apdater::client
> for that class and class sudo both to draw on, but you have that
> relationship reversed on one side.  That just lengthens the dependency
> chain you already had (now sudo depends on newclass depends on
> apdater::client), which doesn't gain you anything.  For this to work
> you need {sudo, apdater::client} depends on newclass.
> 
> Replace "newclass" with "global variable" or "external data" as you
> like.

OK.

> > > The separate class option is great, but it is distinct from the other
> > > two only if the new class can choose the variable's value based only
> > > on nodes' facts.
> >
> > It does not seem to be possible for me.
> 
> Whatever logic you use now either to set $sudo_allow_aptdater_user or
> to choose whether to include class apdater::client can certainly be
> put in a separate class.  Depending on what the logic is, it might be
> ugly / unmaintainable / painful to move it there, but not impossible.

For now the logic is that I manually add the adapter::client to the 
concerned nodes, so it is quite difficult to put this as it is in 
another class. (I am sure it would be possible, I can have the separate 
class that extlookup a file to see if the node should have the 
adapter::client class, but now, for my present need, it seems to be too 
much)

> > By the way I was able to order the module application using run stages,
> > so thanks for pointing me to this direction.
> > I don't find this to be the perfect/cleanest solution for my problem,
> > but at least it is working as needed and expected :)
> 
> I am glad you have found a solution.

Thanks for your time/help.

> Good luck,

Do I look so lost? ;)

> John

Regards,
Baptiste

-- 
\,,/_[-_-]_\,,/
http://asocial.ws/gwarf

Down to the Banana Republics,
Down to the tropical sun.
Go the expatriated Americans,
Hoping to find some fun.
Some of them go for the sailing,
Caught by the lure of the sea.
Trying to find what is ailing,
Living in the land of the free.
Some of them are running from lovers,
Leaving no forward address.
Some of them are running tons of ganja,
Some are running from the IRS.
Late at night you will find them,
In the cheap hotels and bars.
Hustling the senoritas,
While they dance beneath the stars.
                -- Jimmy Buffet, "Banana Republics"

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-users?hl=en.

Reply via email to