Re: [PATCH] Add checksumming feature to copyfile module

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

 



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
Func-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/func-list


[Index of Archives]     [Fedora Users]     [Linux Networking]     [Fedora Legacy List]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux