diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-02-21 08:47:59 -0500 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-02-21 08:47:59 -0500 |
commit | acb1dde9ba48b04d1ceb701ce849e96cef3d0070 (patch) | |
tree | 4754d07af191cc08dfc2be55420403fdb887ad7a /src/lib/Bcfg2/Client/Tools/Action.py | |
parent | d8002c666c6a450e99c9fe476a5a3dcfb23f05db (diff) | |
download | bcfg2-acb1dde9ba48b04d1ceb701ce849e96cef3d0070.tar.gz bcfg2-acb1dde9ba48b04d1ceb701ce849e96cef3d0070.tar.bz2 bcfg2-acb1dde9ba48b04d1ceb701ce849e96cef3d0070.zip |
removed in-place modification of "states" dict in client tools
Diffstat (limited to 'src/lib/Bcfg2/Client/Tools/Action.py')
-rw-r--r-- | src/lib/Bcfg2/Client/Tools/Action.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/Action.py b/src/lib/Bcfg2/Client/Tools/Action.py index 7e8366928..7b62f61c7 100644 --- a/src/lib/Bcfg2/Client/Tools/Action.py +++ b/src/lib/Bcfg2/Client/Tools/Action.py @@ -68,19 +68,23 @@ class Action(Bcfg2.Client.Tools.Tool): return self.RunAction(entry) return True - def BundleUpdated(self, bundle, states): + def BundleUpdated(self, bundle): """Run postinstalls when bundles have been updated.""" + states = dict() for action in bundle.findall("Action"): if action.get('timing') in ['post', 'both']: if not self._action_allowed(action): continue states[action] = self.RunAction(action) + return states - def BundleNotUpdated(self, bundle, states): + def BundleNotUpdated(self, bundle): """Run Actions when bundles have not been updated.""" + states = dict() for action in bundle.findall("Action"): if (action.get('timing') in ['post', 'both'] and action.get('when') != 'modified'): if not self._action_allowed(action): continue states[action] = self.RunAction(action) + return states |