On Thu, Jun 09, 2016 at 11:13:57PM +0200, Iustin Pop wrote: > From: Iustin Pop <ius...@google.com> > > Do this in cases where this should already be the case: > - some functions are explicitly denoted to take UUIDs (documentation/parameter > name) > - some functions (e.g. clusterMasterNode, disk IDs, instance primary node) we > know return an UUID > > But still, this might be a behaviour change, although unlikely. > > Signed-off-by: Iustin Pop <ius...@google.com>
I'm reasonably convinced that there should be no behavioural change, the only possibly dubious case I could see was getInstPrimaryNode, and as far as I can see all the callers of that use exact instance names. LGTM.