Wednesday, 2025-04-30

opendevreviewSylvain Bauza proposed openstack/nova master: Add mtty/mdpy support for testing fake mdevs  https://review.opendev.org/c/openstack/nova/+/89810008:34
opendevreviewSylvain Bauza proposed openstack/nova master: WIP : Add mtty support to nova-next  https://review.opendev.org/c/openstack/nova/+/92214008:34
opendevreviewLiuWenYu proposed openstack/nova master: Implements: Fixed the issue where the code logic would throw an error when reading a variable after NUMA node 0’s CPUs were fully utilized. Related-Bug: #2109675  https://review.opendev.org/c/openstack/nova/+/94854009:38
gibisean-k-mooney: the generic env passing logic works nicely here https://review.opendev.org/c/openstack/nova/+/948450/2 with the devstack patch https://review.opendev.org/c/openstack/devstack/+/948436 So now we can selectively turn on threading mode per service from the zuul job config. 09:47
*** elodille1 is now known as elodilles10:11
elodillesauniyal bauzas Uggla : hi, sorry for pinging you, but could you take a look at the 2023.2 Bobcat EOL patch of nova please: https://review.opendev.org/c/openstack/releases/+/94819611:06
elodillesit would be nice to have an ACK from the team o:)11:07
elodillesi've checked and hashes are still all correct11:07
sean-k-mooneygibi: awsome that was also much smaller then i was expecting11:34
sean-k-mooney 'SYSTEMD_ENV_VARS["n-sch"]': OS_NOVA_DISABLE_EVENTLET_PATCHING=true11:35
sean-k-mooneya i see wll thats a little bit of a wart but i guess when your are rendering a local.conf via yaml11:36
sean-k-mooneyvia ansible11:36
sean-k-mooneythat less bad then it could have been11:36
sean-k-mooneyso in this version nova-next is the only job using the new code path11:37
sean-k-mooneyi guess it make sense to use that as our canary job11:37
gibiyeah11:47
gibiI think I will duplicate nova-multi-cell to have a threading variant eventually as well11:48
gibibut for now nova-next is good enough11:48
sean-k-mooneyzigo: i dont know if this is just a ubunut issue or also a debian issue but novnc is declared as requiring phty3-novnc which requires python3-oslo.confing. however as far as i can tell that depency was removed in 0.6.0 when they deleted there vendro copy of nova vnc proxy, novnc does not seam to have any python deps any more and is basiclly pure javascript12:48
bauzasgibi: sean-k-mooney: I had no time in between yesterday and today to look at the scheduler eventlet series, anything you want me to know ?12:48
sean-k-mooneybauzas: it now work well enough to pass ci12:49
zigosean-k-mooney: I guess you mean 1.6.0, no?12:49
sean-k-mooneyzigo: no i mean that dep was remove lik 10 years ago12:49
bauzassean-k-mooney: scatter gather works fine ?12:49
bauzaswhat about performance ?12:49
zigosean-k-mooney: Looks like the Ubuntu package is just the Debian one, so that one is on me indeed ! :)12:49
sean-k-mooneyzigo: the last release to use oslo was v0.5.112:49
zigosean-k-mooney: So, python3-novnc should just die, right?12:50
sean-k-mooneyzigo: context is we install novnc form disto package by default, and neutron have a oslo tip job that install oslo mater content adn for reason the venv we use in devstack has --site-packages12:51
sean-k-mooneyzigo: that also proably reasonable12:51
sean-k-mooneyi have no idea where its defiend and what its provideing 12:51
sean-k-mooneywell i sort of do https://ubuntu.pkgs.org/24.04/ubuntu-universe-amd64/python3-novnc_1.3.0-2_all.deb.html12:52
sean-k-mooneyzigo: it looks like its basicaly empty12:53
zigoYeah, doing the work now.12:53
zigoHopefully, this will reach Trixie in time for the release.12:53
* zigo is supposed to be 2 weeks on holidays in China right now... :P12:54
sean-k-mooneyzigo: oh sorry about that 12:54
zigoWell, nothing much to do, I'm at my familly in law place, booooorrrriiiinnnggg ... :)12:54
zigoOh, btw, got all of Epoxy in Debian 13 already, all ... but Neutron, which should migrate in 8 days.12:55
gibibauzas: in short the performance seems OK. I will have a blogpost out with more data in couple of hours. At the moment there is nothing to really review for landing as I need to clean up the patches12:55
zigoSo, looking good.12:55
zigoExcept Eventlet and Python 3.13 !12:55
zigoI hope we can find a solution.12:55
sean-k-mooneyzigo: i was going to ask12:55
zigoNot sure if Herve is still working on that.12:55
sean-k-mooneyi tought 3.13 was supper broken12:56
zigoI haven't had a chance to test it yet.12:56
sean-k-mooneywell maybe not super broken but the whoel thread id stuff was broken so...12:56
zigoThat's on my plate for when I return from holidays.12:56
zigoI need all in Trixie to be able to test though.12:56
sean-k-mooneyzigo: so as far as i know eventlet work well enough to pass our unit test and funcitonal12:57
zigoOh, ok.12:57
sean-k-mooneybut fails in reality when mokeypatching due to some changies in the threading module12:57
bauzasgibi: cool, thanks for the info, I'm now myself hardly working on mtty effort12:57
zigoI do unit tests for all packages at build time and autopkgtest, so I know that at least isn't broken.12:57
sean-k-mooneynot sure if that is still a the case12:57
zigoWhat I'm missing is a full setup + tempest run.12:57
sean-k-mooneybtu that is where it was at before the ptg12:57
zigoWith my Bookworm backports setup, I have 1900+ tempest tests passing.12:58
zigoI'll see how far I can go with Trixie.12:58
sean-k-mooneyzigo: ack. that all promising12:58
sean-k-mooneythat remind me i need to go update our pre-commit versions for py3.1312:58
sean-k-mooneyzigo:anyway thanks for looking at this while on vacation but please dont spend too much time on it13:00
zigoNo worries.13:00
zigoIf I put the novnc package somewhere, can you test it?13:01
zigoI did test 1.6.0 already...13:01
zigoWorked, though had to fix some CSS.13:02
sean-k-mooneyfrickler: ^ would you be able to test it in the test env you had set up13:02
fricklerI've now installed novnc from source already, better setup a new host I guess13:02
sean-k-mooneyzigo: i could proably install it into a devstack too i just dont have one set up for this at the moment13:03
zigohttp://shade.infomaniak.ch/novnc_1.6.0-2_all.deb13:03
frickler(that is a held node, not a local env)13:03
zigoIt's ok, I'll just upload like that ...13:03
zigouploaded13:05
zigoTo get the fixed novnc in Ubuntu, you may have to ping some Canonical guys ... jamespage (in here) for example! :)13:06
sean-k-mooneyack, it would be nice to fix in 24.0413:06
zigojamespage: Could you get novnc 1:1.6.0-2 pushed to Ubuntu?13:06
zigosean-k-mooney: Not sure they will push a new upstream release to an already released Ubuntu though, but maybe in their PPA.13:07
sean-k-mooneyya we have a workaround anywya13:07
sean-k-mooneywhich is just install form source13:07
sean-k-mooneyand it only affects jobs13:07
sean-k-mooneythat are using unreleased oslo13:07
sean-k-mooneywhich is a very small minority13:07
sean-k-mooneyso we are likey ok if they cant13:07
sean-k-mooneywe could also jsut swap to debian :)13:08
gibibauzas: blogpost with more data https://gibizer.github.io/posts/Eventlet-Removal-Scheduler-First-Run/13:11
bauzasta a lot13:13
* bauzas has some issues withe the libvirt nodedev for mtty but I'll fix it13:13
opendevreviewLiuWenYu proposed openstack/nova master: Implements: Fixed the issue where the code logic would throw an error when reading a variable after NUMA node 0’s CPUs were fully utilized. Related-Bug: #2109675  https://review.opendev.org/c/openstack/nova/+/94854013:23
zigosean-k-mooney: By all means, please get jobs running on Debian 13 and Py 3.13 ! :)13:24
sean-k-mooneyzigo: we moved nova-hybrid-plug to debian 12 to test with spice13:25
sean-k-mooneyzigo: so we will proably move that to debain 13 once thats avaiable provided spice is not compiled out13:26
fricklerzigo: fyi jamespage is no longer with canonical13:33
zigoOh, I didn't know.13:34
fricklerthat's why I mention it ;) happened pretty recently13:34
zigoWho's in charge of OpenStack stuff now at Canonical then?13:35
fricklerI'm not sure, I think all the contacts I knew earlier are now gone13:36
zigoOnly me stays ... forever ! :)13:40
opendevreviewSylvain Bauza proposed openstack/nova master: Add mtty/mdpy support for testing fake mdevs  https://review.opendev.org/c/openstack/nova/+/89810014:58
opendevreviewSylvain Bauza proposed openstack/nova master: WIP : Add mtty support to nova-next  https://review.opendev.org/c/openstack/nova/+/92214014:58
sean-k-mooneybauzas: so you can just install mdevctl on ubuntu16:20
sean-k-mooneyits not there by default but it should be packaged16:20
bauzasmelwitt: fun stuff, ubuntu noble doesn't have mdevctl https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_92e/openstack/92e4f13448c94210af9f36f145e4a58c/controller/logs/screen-n-cpu.txt16:20
sean-k-mooneyhttps://manpages.ubuntu.com/manpages/focal/man8/mdevctl.8.html16:20
sean-k-mooneybauzas: it does its just not instlaled by defualt16:20
bauzassean-k-mooney: yeah, we need to install it in nova-next16:20
sean-k-mooneyyou could add it to bindep in the test profile16:21
bauzasbut honestly, I hate that silent exception16:21
sean-k-mooneyto keep it optional16:21
sean-k-mooneybauzas: well the better fix would be to fall back to the old way of not using libvirt persitnace if mdevctl is not there16:21
bauzassean-k-mooney: I honestly don't know how to do it, examples would be welcomed16:22
bauzassean-k-mooney: yup, we should failback on a libvirt exception16:22
sean-k-mooneysure just add the package to https://github.com/openstack/nova/blob/master/bindep.txt16:22
bauzasI know what bindep is, I don't know how to make it optional16:22
sean-k-mooney[platform:dpkg test]16:22
sean-k-mooneyinstall on debing but only for the test profile16:23
sean-k-mooneywe use test in devstack16:23
bauzask16:23
sean-k-mooneywe could jsut list it as required16:23
sean-k-mooneyi.e. without test16:23
sean-k-mooneyif we want to always use the new persitance api16:23
bauzasthat's what we do 16:24
bauzasanyway, I'll quickly add it to bindep before I go off16:24
opendevreviewSylvain Bauza proposed openstack/nova master: Add mtty/mdpy support for testing fake mdevs  https://review.opendev.org/c/openstack/nova/+/89810016:26
opendevreviewSylvain Bauza proposed openstack/nova master: WIP : Add mtty support to nova-next  https://review.opendev.org/c/openstack/nova/+/92214016:26
*** __ministry is now known as Guest1484716:55
haleybzigo: regarding Canonical contact, i would start with Billy (wolsen) who isn't usually on irc, but you could try me as a proxy, i meet with them every Monday17:03
EugenMayer440180after my instance stopped restart properly, i restarted the cluster. Now i see that the docker container nova-compute did not start (restarted the controller/computes entirely), with the message '3 RLock(s) were not greened, to fix this error make sure you run eventlet.monkey_patch() before importing any other modules.'17:38
EugenMayer440180it seems like the controller shows all the VM 'active' and running, but they are not. Seems like the state cannot be updated 17:41
EugenMayer440180the actual issue seems to be 'My compute node 9946da6f-9a3d-45c7-a9e3-e7e8f99811fc has hypervisor_hostname compute1.cluster.kontextwork.net but virt driver reports it should be compute1. Possible rename detected, refusing to start!'17:49
EugenMayer440180we recently had issues with 2024.2 with hostname != hostname -f and thus we changed that the computes nor longer have a FQDN like 'compute1.cluster.kontextwork.net' but rather compute117:54
EugenMayer440180this seems to be an issue now. Is there any way to re-register the computes with the controller using the new hostname without loosing any data?17:55
dansmithEugenMayer440180: you cannot change the hostname of the computes, ever, for any reason18:13
EugenMayer440180well i just did.18:24
EugenMayer440180changed it in the database, restarted the computes and everything is up and running again18:29
sean-k-mooneyEugenMayer440180: the problem is the hostname is also in teh cinder and neutron dbs in several tables18:30
EugenMayer440180not using cinder, so this one i dodged. Neutron and the network agents already think about compute118:31
sean-k-mooneyyou would have to consitently change it in 10-20 tables18:31
EugenMayer440180at least when quering the network agents and so forth, they all use the hostname already (or lets say the hostname which is the new FQDN)18:31
sean-k-mooneychanging the hypervior_hostname will break all of the placement allcoations too18:31
sean-k-mooneyneutron port bindign if they are out of sync can be fixed by detachign the ports and reattachign them or live migration/shleve/resize18:32
sean-k-mooneyEugenMayer440180: the thing you hit preventign the agent form starting was likely https://specs.openstack.org/openstack/nova-specs/specs/2023.1/implemented/stable-compute-uuid.html18:33
sean-k-mooneyafter years fo saying renaming compute nodes is not supproted we finally started enforcing it18:34
EugenMayer440180checked the allocations tables in nova/nova_api/nova_cell0, they are all empty18:34
EugenMayer440180sean-k-mooney i did not choose to rename those18:34
sean-k-mooneythat not where they are stored18:34
sean-k-mooneyplacement has its own db18:34
sean-k-mooneyEugenMayer440180: has the value of hypervior_hostname changed in the db? if so that is the name used for the compute_node resouce provider in placement18:35
EugenMayer440180found them there, as you expected. Will try to fix them there too. Thanks for the hint18:37
EugenMayer440180sean-k-mooney i understand that i tapped into the internals. But there was no other choice for me here and i will clean up the mess that this created one by one until we migrate away i assume.18:41
EugenMayer440180The reason i changed the FQDN was the migration to 2024.2 where nova stopped scheduling instances since in my case, the compute nodes always had the 'hostname' (compute1) while the hypervisors had the FQDN (hostname -f) ... it's nothing i have chosen and worked since not sure, 5 years - but then stopped here. Then i had to change the FQDN work18:43
EugenMayer440180again, so i could e.g. restore instances from glance again. What i did not notice then is that this then broke the hypervisors / nova_libvirt (which happened now)18:43
EugenMayer440180i was told then 'openstack does not like when 'hostname != hostname -f' and this can lead to issues ... which it did with 2024. Kind of suprising that something of this size cannot handle what is a usual setup for me since decades. 18:44
sean-k-mooneywe have not chnage how the logic in nova works18:45
sean-k-mooneyyou do not have to have host and hypervior_hostname match18:45
sean-k-mooneythey can be differnt they jsut can change over time18:45
sean-k-mooneyhypervior hostname comes form libvirt 18:45
sean-k-mooneyyou can see what it returns with virish hostname18:45
EugenMayer440180well not sure who broke it, might have been neutron, since it was all about the chassis name18:46
sean-k-mooneythe host value comes form the config and defautl to socket.gethostname)(18:46
EugenMayer440180'due to no OVN chassis for host: compute3'18:46
sean-k-mooneyneutron have ways to cofnigure that via data in the ovs db in the case of ovn or config options for ml2/ovs18:46
EugenMayer440180with 2024.2 the chassis name (which was compute1 and so forth in my case) was no longer how nova expected (since what nova expected was compute1.cluster...) - this was the change that broke it18:47
sean-k-mooneythe value we pass for nova to neutron in the port biding is the valuce form conf.host18:48
sean-k-mooneywhich default to socket.gethostname()18:48
sean-k-mooneyso that implies either your dhcp server started handign out domians18:48
EugenMayer440180well then it either changed with neitron or kolla does no longer properly configure those - cannot say. 18:48
sean-k-mooneyyou /etc/host change teh fqdn to be the canonical hostname 18:48
sean-k-mooneyor /etc/hostname now has the fqdn18:48
EugenMayer440180My DNS was always resolving the FQDN properly, yes18:48
EugenMayer440180yes hosts where properly setup, as usual, and also the hostname. I mean, it's the 101 and it was not changed, and after it broke, i tripple checked it.18:49
sean-k-mooneyya so i know that the ovs-ctl script that is used to start ovs can also have a hostnaem parmater18:50
sean-k-mooneyi would assume it has to be either a change in that, in ovs or in how ovn is configured by kolla18:50
sean-k-mooneyEugenMayer440180: did you report an issue to the kolla team?18:50
EugenMayer440180yes, kolla might be a source for that trouble too.18:51
sean-k-mooneyim wondering if it could be a regresion on there end and if other will be impacted18:51
EugenMayer440180Yes / no. It basically ended up to be noones 'topic'. Neutron was insisting on 'that is how chasis are', nova was 'well it must much, this is how nova always looked for the chases' and kolla was 'well we did not change anything, do not look here'18:51
EugenMayer440180If i may ask, a different topic. The nova cli is deprecated as i have seen, but the openstack cli has no replacement for 'backup' - does this mean that it will be dropped in the (when?) near future?18:55
EugenMayer440180is there an official EOL for the nova cli?18:55
sean-k-mooneyyes and no. its been feature frozen for about 2 years18:57
sean-k-mooneythe python bindign are also frozen18:57
sean-k-mooneywe plan to remove it once heat/watcher/horizon stop usign it18:57
sean-k-mooneyEugenMayer440180: https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/server-backup.html18:58
EugenMayer440180any time horizon here? years, months? 2025.x or 2026?18:58
sean-k-mooneymy hope would be in the next 12-24 months but eventlet removal is a higher priority18:59
EugenMayer440180i tried to onboard the above, but it always failed with 'More than one Image exists with the name'18:59
sean-k-mooneyEugenMayer440180: osc has backup and we reached feature partiy with nova client a few years ago. teh only comamnd we did nto port are the host evacuate ones19:00
EugenMayer440180last time i tried was Nov 2023 though19:00
EugenMayer440180just tried again and:19:02
EugenMayer440180openstack --insecure server backup create --os-compute-api-version 2.64  --name "backup-vpn" --type daily --rotate 5  2ba904fb-47e3-4ee8-92b4-66e41bcf2a9619:02
EugenMayer440180More than one Image exists with the name 'backup-vpn'.19:02
sean-k-mooneywell you ment to provide unique names for each backup i belive19:02
sean-k-mooneywhen passing --name19:02
EugenMayer440180well it creates a backup, it just fails19:03
EugenMayer440180(and nova backup actually needs that name to rotate ..  so i thing osc will need that too;v19:03
EugenMayer440180i would love to use a name including a date, but this does not work for rotate - the name has to match the older backups.19:04
sean-k-mooneywell osc is using the sdk19:04
sean-k-mooneybut lo9oking at the api19:04
sean-k-mooneyhttps://docs.openstack.org/api-ref/compute/#id5719:04
sean-k-mooneythe name is not optional19:04
sean-k-mooneyEugenMayer440180: its not a bfv instance is it19:05
EugenMayer440180my point is, nova cli forced the name to be 'always the same' to be able to group the rotation - osc dropts that issue but ends up beeing in need for that too anyway. so with osc you are basically in a no-go spot19:05
EugenMayer440180bfv?19:05
sean-k-mooneyboot form volume19:05
sean-k-mooneyosc will default to using the server display name if you dont pass a name for the backup19:06
sean-k-mooneywich is reasonble but do you know if the error is coming form nova or form the client19:06
EugenMayer440180if osc defaults to that, it will drop an error, since the server-name should be the same every time 19:06
EugenMayer440180oh you mean i should try that once19:07
sean-k-mooneythe rotaion i belive is ment ot be server side19:07
sean-k-mooneybut im wonderign if your geting a error form nova or if its coming form the client or sdk19:08
EugenMayer440180the rotation is based on the name - that is no assumption. Tried and tested that, documented that19:08
sean-k-mooneyit might be form here https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server_backup.py#L10219:09
sean-k-mooneyso they should be saving the uuid that is returned form the backup call and then using that not the name to look upt the image19:09
EugenMayer440180well is that new on osc? because nova does not do that, it used the name19:09
sean-k-mooneythis is an osc bug19:10
EugenMayer440180or, nova changed that recently, i'am not reading the commits19:10
sean-k-mooneyits a regression form when they swaped form the novaclint to the sdk19:10
sean-k-mooneyEugenMayer440180: this has not change in nova in many many years19:10
EugenMayer440180well then i can assure you, the 'uuid lookup' is broken too.19:10
sean-k-mooneybut the openstackclinet moved all the nova command to the openstacksdk19:11
sean-k-mooneyEugenMayer440180: its not using uuid lookup at all19:11
sean-k-mooneyits finding the backup image by name19:11
EugenMayer440180ok you confused me / i got confused. If the name is required (you mean the server name or the backup name?)19:11
sean-k-mooneyin novas api the name of the backup image is required19:12
sean-k-mooneyosc is passing that to the backup_server functon19:12
sean-k-mooneybut its alos doing image_client.find_image(backup_name, ignore_missing=False)19:12
EugenMayer440180ok then we are actually on the same spot. The name is required and is used to find the older backups, the lookup group for the rotate, the PK or whatever19:12
sean-k-mooneyi htink your error is happening here https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server_backup.py#L10219:13
sean-k-mooneyi have an devstafck actully let me check19:13
EugenMayer440180so changing the name for every backup would be wrong. Thus wht osc requires of me (using a unique name) is wrong (and this is what you pointed out to be a bug)19:13
sean-k-mooneyya it would19:15
EugenMayer440180not sure how to take it, that the centralized, only way to backup an instance (openstack server backup) is broken for over 1.5 years and nobody has noticed that19:15
sean-k-mooneyit will need to be sable19:15
sean-k-mooneyEugenMayer440180: you realsise we condiered removing the backup api entirely 19:15
sean-k-mooneyinstend of porting it 19:15
sean-k-mooneyit is not really used much19:16
sean-k-mooneymost people just use snaphost directly instead of backup19:16
sean-k-mooneybecause backup has never workd form server booted form cinder volume19:16
sean-k-mooneywhere as the snapshot command do19:16
sean-k-mooneyim not saying this should not be fix im saying its a rearly used api/command19:17
EugenMayer440180yeah, this would explain that. Can you safely say that both 2025.x releases will not remove nova backup? or is this nothing that can be said19:18
sean-k-mooneymore or less yes. i expect it to still be a thing in 2026.119:19
sean-k-mooneyit might not exist in 2027.1 or 2027.2 if we mange to get everyone over to the sdk19:20
EugenMayer440180well i will not get to 2026 anyway, so this is not what i would worry then. I plan to migrate away from openstack in Dec 2025 and want to be sure the backup strategy works as long as that.19:21
sean-k-mooneyEugenMayer440180: https://paste.opendev.org/show/bZLdha8p7l5juf4jF8Dw/19:21
EugenMayer440180Again sean-k-mooney, as all the other times, you helped me so much. I would hand you just another medal for that :)19:21
sean-k-mooneyso with a clean vm on master the backup commadn works 19:21
sean-k-mooneyat least once19:21
sean-k-mooneylets see what happens if i do it again19:22
sean-k-mooneysean@devstack-ui:~$ openstack --os-cloud devstack-admin server backup create --name test --type daily --rotate 5 6a314bdf-66ba-42d5-908a-c13e6f67b21419:22
sean-k-mooneyCould not load 'message_list': module 'zaqarclient.queues.v2.cli' has no attribute 'OldListMessages'19:22
sean-k-mooneyCould not load 'message_post': module 'zaqarclient.queues.v2.cli' has no attribute 'OldPostMessages'19:22
sean-k-mooneyMore than one Image exists with the name 'test'.19:22
sean-k-mooneyso ya my guess is we only test creating 1 backup19:22
sean-k-mooneyand since that works we didnt notice19:22
EugenMayer440180for a backup supporting rotation? thats elaborated :)19:23
sean-k-mooneyhttps://paste.opendev.org/show/827657/19:23
sean-k-mooneyso this is a clinet bug19:24
sean-k-mooneynova responce fine and it fails exactly where i said it would19:24
sean-k-mooneysean@devstack-ui:~$ openstack --os-cloud devstack-admin image list19:25
sean-k-mooneyCould not load 'message_list': module 'zaqarclient.queues.v2.cli' has no attribute 'OldListMessages'19:25
sean-k-mooneyCould not load 'message_post': module 'zaqarclient.queues.v2.cli' has no attribute 'OldPostMessages'19:25
sean-k-mooney+--------------------------------------+--------------------------+--------+19:25
sean-k-mooney| ID                                   | Name                     | Status |19:25
sean-k-mooney+--------------------------------------+--------------------------+--------+19:25
sean-k-mooney| 6a1617c4-7b38-47ce-a197-c24932f1f358 | cirros-0.6.3-x86_64-disk | active |19:25
sean-k-mooney| e7feb98c-9eca-463b-a8d1-54d90b265c76 | test                     | active |19:25
sean-k-mooney| 7631698f-aac7-4ba0-93fd-c4ac7ccdc46f | test                     | active |19:25
sean-k-mooney| 19aaf0a5-2978-4a8f-ac69-a812e4805750 | test                     | active |19:25
sean-k-mooney+--------------------------------------+--------------------------+--------+19:25
sean-k-mooneyits not happy with that as it expecgin thte name to be unique19:25
sean-k-mooneyso we can see in the debug output on line 62 https://paste.opendev.org/show/827657/19:27
sean-k-mooneyRESP BODY: {"image_id": "e7feb98c-9eca-463b-a8d1-54d90b265c76"}19:28
sean-k-mooneyso the fix should be to save that to var and use it when geting the iamge https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server_backup.py#L94-L10219:28
EugenMayer440180good catch 19:31
sean-k-mooneyEugenMayer440180: so unfortunetly to fix this we need to fix the sdk first19:38
sean-k-mooneyhttps://github.com/openstack/openstacksdk/blob/df1b9e4f8ca877a7caddcbb9afc66de225a8dd21/openstack/compute/v2/server.py#L635-L65519:38
sean-k-mooneyit should return the id like create_image does https://github.com/openstack/openstacksdk/blob/df1b9e4f8ca877a7caddcbb9afc66de225a8dd21/openstack/compute/v2/server.py#L508-L55119:39
opendevreviewmelanie witt proposed openstack/nova master: DNM try to install mdevctl  https://review.opendev.org/c/openstack/nova/+/94858119:53
opendevreviewmelanie witt proposed openstack/nova master: DNM try to install mdevctl  https://review.opendev.org/c/openstack/nova/+/94858120:04
sean-k-mooneyEugenMayer440180: i have not witten any test but i have a version working locally20:07
sean-k-mooneyEugenMayer440180: https://review.opendev.org/c/openstack/python-openstackclient/+/948586 and https://review.opendev.org/c/openstack/openstacksdk/+/948584 are the fixes20:23
sean-k-mooneyEugenMayer440180: but it would be nice if there was a bug report. it will take a while to fully fix it becasue we will need a new sdk relase and test and then the client patch can merge after 20:24
sean-k-mooneyEugenMayer440180: im not sure if/when ill have time to do that but your welcome to try out the fix20:25
opendevreviewmelanie witt proposed openstack/nova master: DNM try to install mdevctl  https://review.opendev.org/c/openstack/nova/+/94858120:39
sean-k-mooneymelwitt: weried im not sure why bindep didnt work20:50
EugenMayer440180sean-k-mooney thanks!20:51
sean-k-mooneymelwitt: oh i see the issue "bindep_profiles="20:53
sean-k-mooneyi was wrgon we dont install the test profile in the devstack job we invoke it with no profile20:54
sean-k-mooneyso the fix is just to remvoe test form https://review.opendev.org/c/openstack/nova/+/922140/20/bindep.txt20:54
melwittsean-k-mooney: I dunno, I looked around for awhile in devstack and codesearch and it looked like it's pretty much just for tox envs like func and unit test jobs20:54
sean-k-mooneyright20:54
sean-k-mooneyi guess maybe we dont invoke it at all20:55
melwittsean-k-mooney: oh really? I looked for an invocation in the job output and didn't find it20:55
sean-k-mooneywell im not sure20:55
sean-k-mooneymelwitt: we do this https://paste.opendev.org/show/bdsCVZoGiydogkQcKlh8/20:56
sean-k-mooneyso im seing 2 things local bindep=020:56
sean-k-mooneyand local bindep_profiles=20:57
melwittyeah IIUC that is looking for a "-bindep" option in the install package args and if it's not found it won't invoke20:57
sean-k-mooneyso it looks like we install it without enabling bindep or any profile20:57
sean-k-mooneyok so the fix i guess is what your propsoing20:58
sean-k-mooneyjsut have the nova devstack plugin install it20:58
melwittI might be wrong, just wanted to try it out and see what happens20:58
sean-k-mooneyoh your doing it with a preplaybook20:58
sean-k-mooneyit would be nice to just do it in the devstack plug since its compiling the mtty module too20:58
sean-k-mooneyit might as well ensure mdevctl is install that way we can use it locally too20:59
melwittfrom what I could tell there is not a clean way to invoke bindep if it's not a tox job and then I thought well I could just run bindep and then I thought if bindep isn't already running then I don't need to install 50 packages just to get one package20:59
sean-k-mooneyright we dont need to use bindep20:59
sean-k-mooneywe can just do package_install mdevctl here https://github.com/openstack/nova/blob/master/devstack/plugin.sh#L920:59
melwittyeah, that's a good point it would be good to have it for devstack dev20:59
sean-k-mooneyoh its install_package not package install21:00
sean-k-mooneybut ya "install_package mdevctl" woudl fix it or you could do that in devstack propper21:01
sean-k-mooneyhere https://github.com/openstack/devstack/blob/master/lib/nova_plugins/hypervisor-libvirt#L90-L12521:02
melwittyeah I was looking at that too earlier21:02
sean-k-mooneyor here https://github.com/openstack/devstack/blob/master/lib/nova_plugins/functions-libvirt#L58-L10621:02
sean-k-mooneyi honestly dont really have a prefernce21:02
sean-k-mooneyother then via devstack in some from21:03
sean-k-mooneythe playbook works fine for ci too21:03
melwittthat's what I found first but didn't want to mess with it just to see nova-next run21:03
melwittme neither. I am not that expert about devstack and all that21:03
sean-k-mooneythe reson i sugggested here https://github.com/openstack/nova/blob/master/devstack/plugin.sh#L9  was it will only do it if we compile the mtty supprot 21:04
sean-k-mooneyso it would only be done in nova-next21:04
melwittI think that makes sense21:04
sean-k-mooneybut ya if it works i guess any of the options are fine21:04
sean-k-mooneyon ubuntu its a recommended package but we dont install those by default21:05
melwittyeah I noticed it in the "recommended packages" in the job output21:05
sean-k-mooneythe recommened packages are not insane we could enable those. the suggesetd ones are more porblematic21:06
melwittwelp, my thing didn't work. zuul job already angry 😝 21:07
sean-k-mooney:) it was worth a try21:07
sean-k-mooneyoh i see21:07
melwitt"/opt/stack/devstack/local.sh: line 18: install_package: command not found" yay21:07
sean-k-mooneyya that ran at the end but im not sure if its sourced21:08
melwittI thought devstack functions would be available in there. I dunno21:08
sean-k-mooneyif its just run in a subshell it wont have access to devtack functions21:08
melwittthat must be why then I guess21:09
sean-k-mooneyits odd because i tought it was actully sourced but i may have imagined that21:09
sean-k-mooneyok ya its not sourced https://github.com/openstack/devstack/blob/master/stack.sh#L148921:10
melwitta-ha21:10
sean-k-mooneyif you wanted to be really hacky not that i would encurage that :) local.conf is sourced....21:11
sean-k-mooneybut ya i guess plugin and then maybe squash with sylvian or waith thill they get back21:12
melwittme do hacks? I would never!21:12
melwitt😂 21:12
melwittyeah sylvain can add whatever way he prefers. I didn't want to mess up his patch21:13
sean-k-mooneyare you on PTO tomorow? im not sure who is around and who is off21:14
melwittno I'm not21:14
sean-k-mooneyah ok so may 1st isnt a day off in the US21:14
melwittoh yeah it's a holiday in france looks like21:15
sean-k-mooneyin ireland too kind of we move almost all our public holidays to the ned monday https://publicholidays.ie/21:16
melwittoh I see. that's better I think21:17
sean-k-mooneywhat confusitng is while we normally jsut call it the may bankholidy its somethime called labor day but that differnt in the us21:18
sean-k-mooneyso i avoid that name21:18
melwittmay day but not mayday21:19
sean-k-mooneyya so may1st is tradtionaly the 1st day of summer in ireland21:20
sean-k-mooneywell ikid of its a whole thing https://en.wikipedia.org/wiki/Irish_calendar the month of summer in teh irish language do not align to those in most english speaking contries21:23
opendevreviewmelanie witt proposed openstack/nova master: DNM try to install mdevctl  https://review.opendev.org/c/openstack/nova/+/94858121:24
melwittcool I didn't know that21:26
sean-k-mooneyits partly a genererational thing. while its still tought espically in irish speakign schools both calandars are actully used dependign on the context.21:27
sean-k-mooneymelwitt: when i was young i was alwasy confused why americans have spring break in summer21:28
sean-k-mooneyhum maybe  ye dont21:29
sean-k-mooneyhttps://www.voyista.com/blog/when-are-spring-break-dates-colleges-universities ?21:29
JayFSpring Break is usually in April, but it can depend on when Easter lands21:29
sean-k-mooneyi tought it was in may for some reason21:29
sean-k-mooneyah ok21:30
JayFearly may or late march is likely possible21:30
JayFplus it's not standard; in most places in the US school calendars are set at the district (city/county) level21:30
JayFalthough in practice they generally fall in close proximity21:30
melwittI was going to say similar, that they usually put it near easter21:30
sean-k-mooneythat must make thing harder to plan for21:30
sean-k-mooneyya its the same here we just call it midterm and it easter adjcent21:31
JayFI mean, really same thing for Summer break21:31
JayFmy wife has worked schools that start early August, she has worked schools that start early September (with summer start moving along with the start date)21:32
sean-k-mooneynot that the first sunday after the first full moon after the equanox relaly help nail it down21:32
melwittlol21:33
sean-k-mooneywe litrally get the modern Gregorian calendar form cristiantity and then dont even use it for easter...21:33
sean-k-mooneyit moves because it tied to the 13 month lunar calandar and you have to translate 21:33
sean-k-mooneyan this is why you thank the maintianer so time zone lib for there hard work and hope you never have to maintian them21:34
sean-k-mooneythat and 15min timezones that have non 1 hour DST adjustments21:35
sean-k-mooneyhttps://youtu.be/-5wpm-gesOY?si=ufuellJXKZcZBjQ721:38
melwittok so installing it in plugin.sh made it install to the controller only, need to also install on the subnode22:05
sean-k-mooneywell that implies the plugin is not runing on the compute22:06
sean-k-mooneywe need to compile the module in both places22:06
melwittyeah22:06
sean-k-mooneyso the plugin is probly just not enabled on the subnode22:06
sean-k-mooneyya looking at the job the devstack_plugins section is not defiend on the subnode22:07
sean-k-mooneywhich is a prexisting bug i guess22:08
sean-k-mooneymaybe also the libvirt section libvirt 22:08
melwittah ok22:09
sean-k-mooneyhum22:09
sean-k-mooneyhttps://718714dcc35fdedf2036-89f33ee168e8c2014b2d25c632c1d92a.ssl.cf1.rackcdn.com/openstack/0dc5d5810e004f078a7dd5cf7a968e79/compute1/logs/local_conf.txt22:09
sean-k-mooneyso no that not the problem22:09
sean-k-mooneythe problem is we are not setting the vard to do the compile?22:10
melwittoh huh22:10
sean-k-mooneyya NOVA_COMPILE_MDEV_SAMPLES is not defiend on the subnode22:11
melwittohh22:11
opendevreviewmelanie witt proposed openstack/nova master: DNM try to install mdevctl  https://review.opendev.org/c/openstack/nova/+/94858122:12
melwitt5th times a charm22:14
opendevreviewmelanie witt proposed openstack/nova master: DNM try to install mdevctl  https://review.opendev.org/c/openstack/nova/+/94858122:43
melwittartom_: I'm working on the vtpm live migration spec and I notice in the wip patches all of the config option names are "tpm" vs "vtpm" from the spec. for example, default_tpm_secret_security vs default_vtpm_secret_security. is this intentional? should I update the config option names in the spec while I am at it?23:41

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