| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-03 | |||
| 18:03:33 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add functional tests to reproduce bug #1994983 https://review.opendev.org/c/openstack/nova/+/863416 | |
| 18:06:17 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add functional tests to reproduce bug #1994983 https://review.opendev.org/c/openstack/nova/+/863416 | |
| 18:10:27 | sean-k-mooney | gibi: i linked to this converation in the spec ill try and find time to follow up with rodolfo next week some time | |
| 18:10:45 | gibi | cool, thanks | |
| 18:19:16 | opendevreview | Alex Chan proposed openstack/nova master: record action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/862404 | |
| 19:30:52 | dansmith | remember when I volunteered to do the stable compute uuid thing at ptg? | |
| 19:30:58 | dansmith | why TF didn't anyone slap me? | |
| 20:19:48 | darkhorse | dansmith: I added the assert as you suggested. https://review.opendev.org/c/openstack/nova/+/862404 | |
| 20:20:08 | dansmith | darkhorse: yeah, just to the first one.. is the second one just as easy? | |
| 20:20:14 | dansmith | I was also waiting to see tests pass | |
| 20:22:09 | darkhorse | excuse me I did not know there was second one. it should be as easy. let me check and add assert to second too. | |
| 20:23:43 | dansmith | darkhorse: I said "same below" in my comment, but I should have been more explicit | |
| 20:24:17 | dansmith | darkhorse: https://review.opendev.org/c/openstack/nova/+/862404/4/nova/tests/unit/compute/test_api.py#7914 | |
| 20:24:22 | dansmith | that ^ one | |
| 20:25:29 | darkhorse | dansmith: thank you for clarification. | |
| 20:47:23 | opendevreview | Alex Chan proposed openstack/nova master: record action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/862404 | |
| 20:56:39 | dansmith | darkhorse: it's on its way | |
| 23:57:05 | opendevreview | Merged openstack/nova master: record action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/862404 | |
| #openstack-nova - 2022-11-04 | |||
| 00:46:58 | darkhorse | dansmith: thank you very much! | |
| 11:55:24 | elodilles | bauzas sean-k-mooney : if you are around: the final wallaby release patch has merged, so a formal +1 would be good for the transition patch: https://review.opendev.org/c/openstack/releases/+/862298 | |
| 12:52:07 | sean-k-mooney | this si EM os-vif too | |
| 12:52:32 | sean-k-mooney | elodilles: i think we need a release of os-vif to get the correct upperconstrati | |
| 12:52:54 | sean-k-mooney | i was going to do a release of that this week for that reason | |
| 12:53:39 | sean-k-mooney | elodilles: ya we still need to do that for wallaby we noticed it downstream last week | |
| 12:53:46 | sean-k-mooney | https://github.com/openstack/os-vif/blob/2.4.0/tox.ini#L12 | |
| 12:54:05 | sean-k-mooney | let me do a 2.4.1 release quickly then we can tansiation it to EM | |
| 12:56:39 | elodilles | sean-k-mooney: i heard that RH uses the releases a bit differently (i guess building the packages from based on tags) but most users tend to consume the released pypi packages and tarballs, and upper-constraints are not compiled in those packages | |
| 12:57:16 | sean-k-mooney | we actully dont use the tox env in general but it confused some | |
| 12:57:34 | elodilles | sean-k-mooney: so that's why it should not be released, because it would contain the very same content as the previous relese, and would just confuse operators | |
| 12:57:38 | sean-k-mooney | rdo only packages the released rpm in general | |
| 12:57:59 | sean-k-mooney | well are you saying we dont include tox.ini in the package | |
| 12:58:11 | sean-k-mooney | i woudl have expected that to be in the tar | |
| 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_ | |