It was that way initially, and that created a gigantic set of race
conditions between runs and how the noderole state machine walks the graph.


On Tue, Oct 1, 2013 at 3:16 PM, Rob Hirschfeld <notifications@github.com>wrote:

> In crowbar_framework/app/models/node_role.rb:
>
> > @@ -389,6 +356,7 @@ def state=(val)
> >          end
> >          write_attribute("state",val)
> >          save!
> > +        role.send(meth,self) if snapshot.committed?
>
> if this is repeated can we DRY it to a single place?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/crowbar/barclamp-crowbar/pull/708/files#r6697993>
> .
>