Ah, sorry that I did not read this earlier. I think I already fixed this, but only in 2.10. The merge should then be trivial, I guess. :)
On Mon, Jan 20, 2014 at 2:05 PM, Klaus Aehlig <[email protected]> wrote: > On Mon, Jan 20, 2014 at 02:55:26PM +0200, Apollon Oikonomopoulos wrote: > > Ganeti.Config.getNodeRole would rely on clusterMasterNode returning the > > master node name, however clusterMasterNode returns the master node's > > UUID. We fix this and a similar issue in Ganeti.Query.Node.nodeFields. > > > > Together with 1ec34e26, this fixes issue #687. > > > > Signed-off-by: Apollon Oikonomopoulos <[email protected]> > > --- > > src/Ganeti/Config.hs | 2 +- > > src/Ganeti/Query/Node.hs | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > LGTM. Thanks. > > -- > Klaus Aehlig > Google Germany GmbH, Dienerstr. 12, 80331 Muenchen > Registergericht und -nummer: Hamburg, HRB 86891 > Sitz der Gesellschaft: Hamburg > Geschaeftsfuehrer: Graham Law, Christine Elizabeth Flores > -- -- Helga Velroyen | Software Engineer | [email protected] | Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
