LGTM, thanks On Mon, Jun 22, 2015 at 6:11 PM, 'Klaus Aehlig' via ganeti-devel < [email protected]> wrote:
> If a cluster allows sharedfile as only disk template, the amount of > total and free disk space might not be available. This is perfectly > normal, hence make the luxi backend handle it gracefully and just report > 0 available disk on 0 total disk. > > Signed-off-by: Klaus Aehlig <[email protected]> > Reviewed-by: Petr Pudlak <[email protected]> > > Cherry-picked-from: 49644203823562de0945de3feca5dfaa0cc2dc9c > Signed-off-by: Klaus Aehlig <[email protected]> > --- > src/Ganeti/HTools/Backend/Luxi.hs | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/src/Ganeti/HTools/Backend/Luxi.hs > b/src/Ganeti/HTools/Backend/Luxi.hs > index 94571fa..9ee80ca 100644 > --- a/src/Ganeti/HTools/Backend/Luxi.hs > +++ b/src/Ganeti/HTools/Backend/Luxi.hs > @@ -238,8 +238,12 @@ parseNode ktg [ name, mtotal, mnode, mfree, dtotal, > dfree > xmtotal <- lvconvert 0.0 "mtotal" mtotal > xmnode <- lvconvert 0 "mnode" mnode > xmfree <- lvconvert 0 "mfree" mfree > - xdtotal <- lvconvert 0.0 "dtotal" dtotal > - xdfree <- lvconvert 0 "dfree" dfree > + let xdtotal = genericResult (const 0.0) id > + $ lvconvert 0.0 "dtotal" dtotal > + xdfree = genericResult (const 0) id > + $ lvconvert 0 "dfree" dfree > + -- "dtotal" and "dfree" might be missing, e.g., if sharedfile > + -- is the only supported disk template > xctotal <- lvconvert 0.0 "ctotal" ctotal > xcnos <- lvconvert 0 "cnos" cnos > let node = flip Node.setCpuSpeed xcpu_speed . > -- > 2.4.3.573.g4eafbef > > Hrvoje Ribicic Ganeti Engineering Google Germany GmbH Dienerstr. 12, 80331, München Geschäftsführer: Graham Law, Christine Elizabeth Flores Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg
