On Thu, Aug 04, 2011 at 02:23:06PM +0200, Michael Hanselmann wrote:
> Am 4. August 2011 09:48 schrieb Iustin Pop <ius...@google.com>:
> > On Wed, Aug 03, 2011 at 12:43:04PM +0200, Michael Hanselmann wrote:
> >> + inst_groups = self.cfg.GetInstanceNodeGroups(self.op.instance_name)
> >> + if not owned_groups.issuperset(inst_groups):
> >> + raise errors.OpPrereqError("Instance %s's node groups changed since"
> > […]
> >
> > I think I've seen this bit of code in about 5 different LUs already;
> > could you abstract it please?
>
> A separate patch for that is forthcoming, interdiff for this patch:
LGTM, thanks!
iustin
