| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-04 | |||
| 12:58:14 | sean-k-mooney | so you can run the tests | |
| 12:58:30 | sean-k-mooney | if its not then we dont need to do anything | |
| 12:58:46 | sean-k-mooney | i guess i can check | |
| 12:59:22 | elodilles | in pypi packages i'm sure we don't have it | |
| 12:59:55 | elodilles | besides, the code would be the very same either way | |
| 13:00:05 | sean-k-mooney | its in the tar | |
| 13:00:07 | sean-k-mooney | https://tarballs.opendev.org/openstack/os-vif/os_vif-2.4.0.tar.gz | |
| 13:00:20 | sean-k-mooney | and the tar is the primary release artifact | |
| 13:00:35 | sean-k-mooney | the tar is what the rpm and deb packages are built form | |
| 13:01:23 | sean-k-mooney | elodilles: you are correct that its not in the wheel | |
| 13:02:20 | sean-k-mooney | elodilles: so its in the tar published to pypi but not the whl | |
| 13:02:39 | sean-k-mooney | https://files.pythonhosted.org/packages/14/2a/dc1cac486333a725c3835b9e8d33ec4ab3bb0ed3e0f49d6354548c8cde1f/os_vif-2.4.0.tar.gz | |
| 13:03:11 | sean-k-mooney | elodilles: so i think we should like fix this unless you strongly disagree | |
| 13:05:41 | elodilles | sean-k-mooney: i'd rather not release this, but maybe it's just me :/ | |
| 13:10:09 | elodilles | sean-k-mooney: anyway, please propose the patch if you think it should be released, i'll comment on it, and let's see others opinion about it | |
| 13:11:00 | sean-k-mooney | https://review.opendev.org/c/openstack/releases/+/863644 | |
| 13:11:22 | sean-k-mooney | i had the patch ready locally i ment to do it last week | |
| 13:11:58 | sean-k-mooney | lets see what bauzas thinks | |
| 13:12:08 | sean-k-mooney | other then that im fine with movign things to em now | |
| 13:23:12 | sean-k-mooney | elodilles: i set +1 on the em transition patch so if others dont think we need the os-vif release you can just proceed with merging the em patch | |
| 13:26:16 | elodilles | sean-k-mooney: ack, thanks! | |
| 13:27:17 | sean-k-mooney | elodilles: part of why i at least wanted to push the patch is i told peole i would do it downstream in a ci escalation bug | |
| 13:27:55 | sean-k-mooney | this was not actully the cause of the downstream issue https://bugzilla.redhat.com/show_bug.cgi?id=2109536 | |
| 13:27:55 | elodilles | i see | |
| 13:28:04 | sean-k-mooney | stestr was just not installed in the correct location | |
| 13:30:34 | elodilles | i'll ask about this release on the weekly relmgmt meeting, which will start soon (14:00 UTC) | |
| 13:31:06 | sean-k-mooney | sure | |
| 15:23:09 | elodilles | sean-k-mooney: fyi, there were no strong objection about the os-vif release, so it is +2+W'd (and even released already) | |
| 15:23:27 | elodilles | i'll update the wallaby-em patch in a minute | |
| 15:23:33 | sean-k-mooney | ack i saw the email come in | |
| 15:23:40 | sean-k-mooney | cool | |
| 15:23:48 | sean-k-mooney | ill set +1 on it when done | |
| 15:27:21 | elodilles | sean-k-mooney: https://review.opendev.org/c/openstack/releases/+/862298 | |
| 15:34:55 | opendevreview | Jean-Sébastien Bevilacqua proposed openstack/nova master: Add Lustre support to nova https://review.opendev.org/c/openstack/nova/+/853786 | |
| 15:38:13 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Don't ignore InstanceNotFound exception by libvirt https://review.opendev.org/c/openstack/nova/+/863665 | |
| 18:45:10 | atmark | hello, is it posible to set `resume_guests_state_on_host_boot=true` in nova and then `virsh autostart $domainname --disable` a selected VMs that I don't want to start | |
| 18:45:23 | atmark | will two conflict with each other? | |
| 18:47:13 | dansmith | pretty sure they're unrelated and so the nova one will win, | |
| 18:47:17 | dansmith | but let me see if I can find it | |
| 18:51:05 | dansmith | atmark: by my reading, if the nova tunable is enabled, this is the only logic that might cause us not to take hard_reboot() action on the guest: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4150-L4172 | |
| 19:13:47 | atmark | dansmith: this it means i cannot mix the two. if leave nova tunable disabled and then virsh autostart selected VMs, will it work or power states will get messed up? | |
| 19:14:00 | atmark | btw, thx for pointing me to code | |
| 19:52:02 | dansmith | atmark: not sure tbh, nova might force it back off if it thinks it should be off, but if it was active and is active by the time nova starts I suspect it'll say "yep, as expected" | |
| #openstack-nova - 2022-11-05 | |||
| 15:09:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | |
| 15:11:10 | gibi_pto | sean-k-mooney: ^^ I updated ^^ based on your latest feedback. | |
| #openstack-nova - 2022-11-06 | |||
| 00:41:03 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2023.1 Antelope https://review.opendev.org/c/openstack/nova/+/858238 | |
| 13:52:56 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 14:28:25 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| #openstack-nova - 2022-11-07 | |||
| 08:30:35 | Uggla | Good morning nova | |
| 08:44:12 | sahid | o/ sean-k-mooney a gentle ping I should have fixed both patches based on the new spec that you were agree during ptg, https://review.opendev.org/c/openstack/nova/+/858383 https://review.opendev.org/c/openstack/nova/+/858384/ | |
| 08:44:21 | sahid | not sure if you have noticed that :-) | |
| 09:17:44 | opendevreview | Merged openstack/nova stable/zed: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/861856 | |
| 10:15:36 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 10:18:25 | opendevreview | Amit Uniyal proposed openstack/nova stable/xena: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863792 | |
| 10:31:55 | opendevreview | Amit Uniyal proposed openstack/nova stable/xena: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863792 | |
| 10:31:56 | opendevreview | Amit Uniyal proposed openstack/nova stable/xena: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863864 | |
| 10:48:47 | opendevreview | Merged openstack/nova stable/zed: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/861857 | |
| 12:24:45 | opendevreview | John Garbutt proposed openstack/nova-specs master: ronic shard_key to replace peer_list https://review.opendev.org/c/openstack/nova-specs/+/862833 | |
| 12:25:28 | opendevreview | John Garbutt proposed openstack/nova-specs master: Ironic shard_key to replace peer_list https://review.opendev.org/c/openstack/nova-specs/+/862833 | |
| 12:32:28 | opendevreview | John Garbutt proposed openstack/nova-specs master: Ironic shard_key to replace peer_list https://review.opendev.org/c/openstack/nova-specs/+/862833 | |
| 13:48:42 | ygk_12345 | Hi all | |
| 13:49:09 | ygk_12345 | can anyone help me with using pdb for tracing in the file https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py ? | |
| 13:50:06 | ygk_12345 | i am tryingto insert pdb at a place in this file. But while running the nova-api command , it is not showing the pdb shell | |
| 15:11:52 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: Repropose spec for updating user data https://review.opendev.org/c/openstack/nova-specs/+/863884 | |
| 15:35:03 | opendevreview | Arnaud Morin proposed openstack/nova master: Add os_vif in nova-config-generator https://review.opendev.org/c/openstack/nova/+/863891 | |
| 15:35:30 | amorin | bauzas and sean-k-mooney ^ | |
| 15:35:36 | amorin | we talked about that two weeks ago | |
| 15:36:00 | amorin | I expect this change to also do the necessary change on docs.o.o (config reference_ | |
| 15:36:03 | amorin | ) | |
| 15:36:09 | amorin | am I correct? | |
| 15:43:36 | opendevreview | Dan Smith proposed openstack/nova master: Test ceph-multistore with a real image https://review.opendev.org/c/openstack/nova/+/860864 | |
| 15:50:49 | opendevreview | Rajat Dhasmana proposed openstack/nova-specs master: Clarify client changes in rebuild spec https://review.opendev.org/c/openstack/nova-specs/+/856164 | |
| 15:54:02 | bauzas | amorin: should work | |
| 15:55:05 | amorin | ack | |
| 16:53:07 | opendevreview | Amit Uniyal proposed openstack/nova stable/xena: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863864 | |
| 16:53:08 | opendevreview | Amit Uniyal proposed openstack/nova stable/xena: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863792 | |
| 16:57:02 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863900 | |
| 16:57:03 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863901 | |
| 17:01:16 | opendevreview | Amit Uniyal proposed openstack/nova stable/victoria: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863902 | |
| 17:01:17 | opendevreview | Amit Uniyal proposed openstack/nova stable/victoria: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863903 | |
| 17:38:51 | opendevreview | Dan Smith proposed openstack/nova master: Test ceph-multistore with a real image https://review.opendev.org/c/openstack/nova/+/860864 | |
| 17:41:17 | darkhorse | Hi team - is there a standard way or recommended way of debugging under-development nova component under devstack environment? | |
| 17:42:38 | darkhorse | I tried to kill nova service from devstack and start my under-development nova in debug mode so that requests can be forwarded to the under-development nova. | |
| 17:44:04 | opendevreview | Stephen Finucane proposed openstack/nova master: hyperv: Mark driver as experimental https://review.opendev.org/c/openstack/nova/+/863910 | |
| 17:44:05 | opendevreview | Stephen Finucane proposed openstack/nova master: vmwareapi: Mark driver as experimental https://review.opendev.org/c/openstack/nova/+/863911 | |
| 17:44:42 | stephenfin | melwitt: dansmith: sean-k-mooney: I think that's what was suggested at the PTG. We don't deprecate, we just mark it experimental and remove if it starts costing us? ^ | |
| 17:45:56 | dansmith | stephenfin: yes, although I thought we were using other words like "untested" (or at least I was) | |
| 17:46:11 | dansmith | experimental means "not yet ready for prime time, but soon" to me in most software features | |
| 17:46:36 | dansmith | so I'd rather use something that implies it has degraded to the current state, not that it's working towards a better one | |
| 17:47:03 | dansmith | (although it's not a huge deal of course) | |
| 17:51:04 | melwitt | stephenfin: pretty much. my understanding was/is no formal deprecation, no plan for removal, just a log message that warns the driver is untested. ppl will use the driver at their own risk. there was also mention of updating the driver matrix doc to mention something similar to the log message for the drivers in question | |
| 17:51:40 | melwitt | *suggestion of updating driver matrix | |
| 18:19:57 | opendevreview | Dan Smith proposed openstack/nova master: Test ceph-multistore with a real image https://review.opendev.org/c/openstack/nova/+/860864 | |
| 18:32:12 | opendevreview | Dan Smith proposed openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 18:32:13 | opendevreview | Dan Smith proposed openstack/nova master: WIP Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 18:32:13 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 18:32:14 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 18:32:14 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 18:32:15 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 18:32:50 | dansmith | sean-k-mooney: all of this is marked WIP ^ because I haven't added more tests to cover any of the changes that aren't already covered (if any) | |