Friday, 2023-05-26

opendevreviewmelanie witt proposed openstack/nova master: Add encryption support to convert_image  https://review.opendev.org/c/openstack/nova/+/87093400:25
opendevreviewmelanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property  https://review.opendev.org/c/openstack/nova/+/87093500:25
opendevreviewmelanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase  https://review.opendev.org/c/openstack/nova/+/87093600:25
opendevreviewmelanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption  https://review.opendev.org/c/openstack/nova/+/87093700:25
opendevreviewmelanie witt proposed openstack/nova master: Support rescue with ephemeral encryption  https://review.opendev.org/c/openstack/nova/+/87367500:25
opendevreviewmelanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties  https://review.opendev.org/c/openstack/nova/+/87093900:25
opendevreviewmelanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS  https://review.opendev.org/c/openstack/nova/+/77227300:25
opendevreviewmelanie witt proposed openstack/nova master: WIP raw imagebackend support  https://review.opendev.org/c/openstack/nova/+/88431300:25
opendevreviewmelanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw  https://review.opendev.org/c/openstack/nova/+/86241600:32
opendevreviewWenping Song proposed openstack/nova stable/victoria: Trival Change: Remove redundant code in instance delete  https://review.opendev.org/c/openstack/nova/+/79691200:43
*** EugenMayer43 is now known as EugenMayer401:26
opendevreviewmelanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw  https://review.opendev.org/c/openstack/nova/+/86241602:40
lowercasesean-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
lowercaseXena release 14:37
bauzaslowercase: are you sure that the instances are not pinned ?15:01
lowercasebauzas: yes, we don't do cpu pinning.15:05
bauzaslowercase: no sorry, I mean, pinned to an AZ15:06
lowercasehow would we pin them to an AZ?15:07
lowercaseso 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 false15:08
lowercasebecause we have multiple ceph clusters, 1 ceph cluster per AZ15:08
lowercasehttps://docs.openstack.org/nova/xena/admin/availability-zones.html#implications-for-moving-servers15:13
lowercase#4 is our condition15:13
sean-k-mooneylowercase: the schgduler is allowed ot pick an instance outside the current az15:30
sean-k-mooneylowercase: it is only not allowe to do that if the instnace is create reqeusting a specific AZ15:31
lowercaseis there anyway I can retro-give an instance the paramter to keep the instance in a az15:31
sean-k-mooneylowercase: implications-for-moving-servers is refering to moveing compute hosts not vms15:32
lowercaseinteresting15:32
lowercasethe only way to pin a vm to an AZ is ONLY at creation time?15:32
sean-k-mooneyretoactivly on xena15:32
sean-k-mooneyi think vai shelve and then unshelve specifyign an az15:33
lowercasehmmm.. that's not a bad idea actually.15:33
lowercaseis it possible to retroactively pin a vm to az by modifying the database? 15:34
sean-k-mooneyyes...15:34
sean-k-mooneyyou can set the az in teh request spec which is a json blob15:35
sean-k-mooneyhttps://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id7015:35
sean-k-mooneyunshlve to an az was added in train just an fyi15:35
sean-k-mooneylowercase: https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L75 is what determins what the schduler requests15:36
sean-k-mooneyso you can just set the az in the json blob in the api db15:37
sean-k-mooneybut its error prone15:37
lowercaseshelving and unshelving requires to shutdown the instance. We would prefer a diff method that retroactively restrict vms to an az. 15:37
lowercasethat's why I could script it out to alter the database would be preferable 15:37
sean-k-mooneyi feel like this is one of those things that comes in phases.15:38
sean-k-mooneywe have had 2 or 3 diffent people ask about this in the last month15:38
sean-k-mooneya nova-manage command to update the requested az in the request spec might be useful 15:39
sean-k-mooneyi.e. one that woudl set it to whatever the instance currently has15:39
lowercaseWe 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-mooneysomething like "nova-mange instance az sync <uuid>"15:40
lowercasethat would be very cool15:40
sean-k-mooneyits not nessisaly safe to set an az request that does not match the instance current value15:41
sean-k-mooneybut it shoudl always be safe to sync the current value to the request spec15:41
sean-k-mooneywell provided th instance is not in error 15:41
sean-k-mooneywe can asusme that it its running in az 1 its safe to pin it to az1 going forward if you ask use too15:42
lowercase( we are having internal team dialogue, just a moment)15:48
lowercaseOkay, we are happy with your current suggestions. 15:55
lowercaseA nova-mange instance az sync <uuid> would def be helpful in the future 15:55
lowercaseI need to go afk, we are going to pick this up again after lunch.15:55
lowercasethank you for your time.15:55
lowercaseWe certainly appreciate it.15:55
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: Reproduce bug 1995153  https://review.opendev.org/c/openstack/nova/+/88232116:10
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: Save cell socket correctly when updating host NUMA topology  https://review.opendev.org/c/openstack/nova/+/88232216:10
bauzaslowercase: sorry I forgot you :facepalm:17:42
bauzaslowercase: sean-k-mooney: if an instance is created using a volume, then we pin the AZ by the RequestSpec 17:43
bauzaslowercase: 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
lowercaseSorry, internet went down. I receieved your :42 and :43 messages. 17:51
lowercaseTested the shelving/unshelving. It does fix the problem BUT it won't be a good solution for us17:53
lowercaseThe way it works is it powers off the instance and snapshots. When you unshelve it boots the snapshot17:53
lowercaseThat is very problematic because snapshots blow up qcows and it would not be practical at all with our instances17:53
lowercaseSo looking at what updating the extra specs in the databse would be like17:53
lowercaserebooting my computer, brb17:59
dansmiththats what shelve does17:59
lowercaseback18:03
dansmith[10:59:26]  <dansmith> thats what shelve does18:05
*** ministry is now known as __ministry20:38

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!