On Fri, 2010-11-26 at 12:52 -0500, Marcus Lauer wrote:
> There should be a better way of adding the checksumming code to both
> the Overlord and Minion copyfile.py files. I'll look into that next. Also,
> it should now be relatively easy to re-implement the --force argument (skip
> checksumming) and to create a new --backup argument (backup the target before
> overwriting like the original copyfile method did).
>
great.
> Based on the experience of writing this code I would like to make a
> feature suggestion. It would be useful if the "run" method in the "Overlord"
> class could take an optional "mask" argument which could instruct it to skip
> certain Minions. The mask could be a dictionary with minion names as the
> keys
> and something like True/False as the data. Coincidentally this is almost
> exactly the same structure which is returned by the "run" method in the
> "Overlord" class itself. This feature would make it easy and efficient to
> write functions which have multiple "run" calls which are dependent upon one
> another.
So - you'd setup an overlord client with a host-specification and then
you want to modify that specification on the fly for the run() method?
would it be easier/make more sense to just be able to modify the host
specification while the class exists?
ie:
fc = Client('myhost;list;of;doom')
fc.change_hosts('newspecification')
fc.run()
would that do what do what you want?
-sv
_______________________________________________
Func-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/func-list
