opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | 00:25 |
---|---|---|
opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: WIP raw imagebackend support https://review.opendev.org/c/openstack/nova/+/884313 | 00:25 |
opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw https://review.opendev.org/c/openstack/nova/+/862416 | 00:32 |
opendevreview | Wenping Song proposed openstack/nova stable/victoria: Trival Change: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/796912 | 00:43 |
*** EugenMayer43 is now known as EugenMayer4 | 01:26 | |
opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw https://review.opendev.org/c/openstack/nova/+/862416 | 02:40 |
lowercase | sean-k-mooney: It appears we are hitting a bug where instances are not honoring availability zone placement even though we have cinder_Cross_az attach set to false. Is this ringing any bells or anything? | 14:37 |
lowercase | Xena release | 14:37 |
bauzas | lowercase: are you sure that the instances are not pinned ? | 15:01 |
lowercase | bauzas: yes, we don't do cpu pinning. | 15:05 |
bauzas | lowercase: no sorry, I mean, pinned to an AZ | 15:06 |
lowercase | how would we pin them to an AZ? | 15:07 |
lowercase | so what is happening is that a when we attempt to resize an instance, the nova scheduler can (in our experience) pick a host outside of the AZ where the instance is currently is, because we have cinder_cross_az attach set to false | 15:08 |
lowercase | because we have multiple ceph clusters, 1 ceph cluster per AZ | 15:08 |
lowercase | https://docs.openstack.org/nova/xena/admin/availability-zones.html#implications-for-moving-servers | 15:13 |
lowercase | #4 is our condition | 15:13 |
sean-k-mooney | lowercase: the schgduler is allowed ot pick an instance outside the current az | 15:30 |
sean-k-mooney | lowercase: it is only not allowe to do that if the instnace is create reqeusting a specific AZ | 15:31 |
lowercase | is there anyway I can retro-give an instance the paramter to keep the instance in a az | 15:31 |
sean-k-mooney | lowercase: implications-for-moving-servers is refering to moveing compute hosts not vms | 15:32 |
lowercase | interesting | 15:32 |
lowercase | the only way to pin a vm to an AZ is ONLY at creation time? | 15:32 |
sean-k-mooney | retoactivly on xena | 15:32 |
sean-k-mooney | i think vai shelve and then unshelve specifyign an az | 15:33 |
lowercase | hmmm.. that's not a bad idea actually. | 15:33 |
lowercase | is it possible to retroactively pin a vm to az by modifying the database? | 15:34 |
sean-k-mooney | yes... | 15:34 |
sean-k-mooney | you can set the az in teh request spec which is a json blob | 15:35 |
sean-k-mooney | https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id70 | 15:35 |
sean-k-mooney | unshlve to an az was added in train just an fyi | 15:35 |
sean-k-mooney | lowercase: https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L75 is what determins what the schduler requests | 15:36 |
sean-k-mooney | so you can just set the az in the json blob in the api db | 15:37 |
sean-k-mooney | but its error prone | 15:37 |
lowercase | shelving and unshelving requires to shutdown the instance. We would prefer a diff method that retroactively restrict vms to an az. | 15:37 |
lowercase | that's why I could script it out to alter the database would be preferable | 15:37 |
sean-k-mooney | i feel like this is one of those things that comes in phases. | 15:38 |
sean-k-mooney | we have had 2 or 3 diffent people ask about this in the last month | 15:38 |
sean-k-mooney | a nova-manage command to update the requested az in the request spec might be useful | 15:39 |
sean-k-mooney | i.e. one that woudl set it to whatever the instance currently has | 15:39 |
lowercase | We just entered a new market, where our openstck environment is still in flux. Typically, when vms are built they don't shutdown or get moved (or rarely). Since we have a new requirements in this new location, the customers vms have been in flux more than normal for us. | 15:40 |
sean-k-mooney | something like "nova-mange instance az sync <uuid>" | 15:40 |
lowercase | that would be very cool | 15:40 |
sean-k-mooney | its not nessisaly safe to set an az request that does not match the instance current value | 15:41 |
sean-k-mooney | but it shoudl always be safe to sync the current value to the request spec | 15:41 |
sean-k-mooney | well provided th instance is not in error | 15:41 |
sean-k-mooney | we can asusme that it its running in az 1 its safe to pin it to az1 going forward if you ask use too | 15:42 |
lowercase | ( we are having internal team dialogue, just a moment) | 15:48 |
lowercase | Okay, we are happy with your current suggestions. | 15:55 |
lowercase | A nova-mange instance az sync <uuid> would def be helpful in the future | 15:55 |
lowercase | I need to go afk, we are going to pick this up again after lunch. | 15:55 |
lowercase | thank you for your time. | 15:55 |
lowercase | We certainly appreciate it. | 15:55 |
opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882321 | 16:10 |
opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882322 | 16:10 |
bauzas | lowercase: sorry I forgot you :facepalm: | 17:42 |
bauzas | lowercase: sean-k-mooney: if an instance is created using a volume, then we pin the AZ by the RequestSpec | 17:43 |
bauzas | lowercase: but we have https://bugs.launchpad.net/nova/+bug/2018318 in case an enduser wants to create a 'boot-from-volume' instance by using an image that will be creating a volume | 17:44 |
lowercase | Sorry, internet went down. I receieved your :42 and :43 messages. | 17:51 |
lowercase | Tested the shelving/unshelving. It does fix the problem BUT it won't be a good solution for us | 17:53 |
lowercase | The way it works is it powers off the instance and snapshots. When you unshelve it boots the snapshot | 17:53 |
lowercase | That is very problematic because snapshots blow up qcows and it would not be practical at all with our instances | 17:53 |
lowercase | So looking at what updating the extra specs in the databse would be like | 17:53 |
lowercase | rebooting my computer, brb | 17:59 |
dansmith | thats what shelve does | 17:59 |
lowercase | back | 18:03 |
dansmith | [10:59:26] <dansmith> thats what shelve does | 18:05 |
*** ministry is now known as __ministry | 20:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!