yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #35252
[Bug 1445637] Re: Instance resource quota not observed for non-ephemeral storage
Cinder client had offer qos command. Those instance quota settings of
non-emphemeral disk should be set via cinder cli instead of inherit from
instance's flavor.
** Changed in: nova
Status: In Progress => Invalid
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1445637
Title:
Instance resource quota not observed for non-ephemeral storage
Status in OpenStack Compute (nova):
Invalid
Bug description:
I'm using a nova built from stable/kilo and trying to implement
instance IO resource quotas for disk as per
https://wiki.openstack.org/wiki/InstanceResourceQuota#IO.
While this works when building an instance from ephemeral storage, it
does not when booting from a bootable cinder volume. I realize I can
implement this using cinder quota but I want to apply the same
settings in nova regardless of the underlying disk.
Steps to produce:
nova flavor-create iolimited 1 8192 64 4
nova flavor-key 1 set quota:disk_read_iops_sec=10000
Boot an instance using the above flavor
Guest XML is missing <iotune> entries
Expected result:
<snip>
<target dev='vda' bus='virtio'/>
<iotune>
<read_iops_sec>10000</read_iops_sec>
</iotune>
</snip>
This relates somewhat to https://bugs.launchpad.net/nova/+bug/1405367
but that case is purely hit when booting from RBD-backed ephemeral
storage.
Essentially, for non-ephemeral disks, a call is made to
_get_volume_config() which creates a generic LibvirtConfigGuestDisk
object but no further processing is done to add extra-specs (if any).
I've essentially copied the disk_qos() method from the associated code
review (https://review.openstack.org/#/c/143939/) to implement my own
patch (attached).
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1445637/+subscriptions
References