Page MenuHomePhabricator

sre.ganeti.makevm cook book only allows specifying RAM size in full gigabytes
Open, LowestPublic

Description

We have at least one service type which doesn't really need a full GB of RAM (T229903), but the makevm cook book uses an integer of gigabytes.

(I'm not sure if that's actually worth changing, mostly filing for posterity. Feel free to ignore/close)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMon, Aug 19, 11:05 AM
MoritzMuehlenhoff triaged this task as Lowest priority.Mon, Aug 19, 11:05 AM
Volans added a subscriber: elukey.Mon, Aug 19, 11:23 AM

That's because we pass memory={memory}g to the gnt-instance add command. We could instead accept a float in the cookbook, convert it to MB and use m in the command.
I'm fine either way. CCing @elukey that used it a lot and @crusnov