| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-22 | |||
| 19:39:52 | melwitt | weird | |
| 19:40:05 | sean-k-mooney | so now its seeing two netwroks and failing | |
| 19:40:20 | sean-k-mooney | but i think before it might now have been seeing any networks and bootign without a network | |
| 19:40:42 | sean-k-mooney | thats just a guess but that was the only neutron patch that seamed to line up | |
| 19:41:18 | melwitt | yeah.. I didn't think that "auto" would include 'none' but I don't know that level of detail about it | |
| 19:42:00 | sean-k-mooney | i have never booted a nova instance wituout passsing a network or explictly passing none | |
| 19:42:22 | melwitt | same | |
| 19:42:26 | sean-k-mooney | so im not a good judge of what shoudl work in that regard | |
| 19:43:21 | melwitt | yeah. I was just thinking superficially, "get me a network" I would not have expected no network to be an option 😂 | |
| 19:43:34 | melwitt | get me a not a network | |
| 19:44:51 | sean-k-mooney | well this test is not testing that functionatliy | |
| 19:44:58 | dansmith | "get me an airgap" | |
| 19:45:42 | sean-k-mooney | also i didnt know "self.run_hooks('modify_body_for_create', body, **kwargs)" was a thing, now can i unlearn that the nova clinet allows you to supply hooks that change its behvior | |
| 19:46:24 | melwitt | lol | |
| 19:49:05 | sean-k-mooney | the 6% ginger beer i have in the fridge might help with that | |
| 19:49:18 | sean-k-mooney | i should go test this theory o/ | |
| 19:50:14 | sean-k-mooney | melwitt: by the way this test is usign the 2.1 microverison | |
| 19:50:27 | sean-k-mooney | so its not using the autoacocate behvior | |
| 19:50:35 | sean-k-mooney | and its falling back to the older behavior | |
| 19:51:05 | melwitt | hm ok | |
| 19:51:39 | sean-k-mooney | """Required parameter when there are multiple networks defined for the tenant. When you do not specify the networks parameter, the server attaches to the only network created for the current tenant. """ | |
| 19:52:02 | sean-k-mooney | so my theory is before they fixed the bug there was no network visiable and now there are 2 | |
| 19:55:13 | melwitt | ahhh | |
| 20:05:54 | sean-k-mooney | i think we are taking this return https://github.com/openstack/nova/blob/3bdb298e685417ed0ecabbba12062d148ad58826/nova/network/neutron.py#L1181-L1189 | |
| 20:06:17 | sean-k-mooney | i wanted ot see if i could find that quickly but i think that how it worked in the end | |
| 20:06:38 | sean-k-mooney | unfortunetly we dont have logs form the last succesfuly run | |
| 20:06:55 | sean-k-mooney | but if my revert passses we shoudl see that debug message for the instnace if im right | |
| 20:14:58 | sean-k-mooney | we have had that skip since quantom support was being added https://github.com/openstack/nova/commit/8f3b3db9b04bb5ff7b9486dc6fc211745b983855 | |
| 20:15:13 | melwitt | I see. I will watch for the result | |
| 20:16:36 | sean-k-mooney | i hit the same issue | |
| 20:16:37 | sean-k-mooney | https://zuul.openstack.org/stream/0effd51a31b34c03b8a9f8beef7a8cf8?logfile=console.log | |
| 20:16:47 | sean-k-mooney | so i guesss thats not it... | |
| 20:17:06 | sean-k-mooney | that of neutron is not a required proejct | |
| 20:19:46 | sean-k-mooney | oh wrong zuul job | |
| 20:20:05 | sean-k-mooney | that the non voting one i was waiting for it to pass/fail before i finsihed | |
| 20:20:14 | sean-k-mooney | that explains why it was using amster | |
| 20:21:44 | sean-k-mooney | it still used master in the otherone however so i dont think its set up to make depend on work | |
| 20:22:39 | sean-k-mooney | its eaiser to see that hwen the job finshes howeer so im going to leave it urn | |
| 20:23:24 | melwitt | and you think that might be bc the job definition doesn't required-projects neutron? | |
| 20:24:10 | melwitt | I hadn't thought that's needed though | |
| 21:35:07 | melwitt | dansmith: looks like compute correctly rejected the luks image, but glance skipped bc of config? | |
| 22:14:09 | opendevreview | melanie witt proposed openstack/python-novaclient master: [DNM] testing job failure https://review.opendev.org/c/openstack/python-novaclient/+/926950 | |
| 22:15:47 | melwitt | sean-k-mooney: I think you were right about required-projects, I did an edit on ^ I hope you don't mind | |
| 23:22:42 | melwitt | hm, that didn't help. it looks like the Depends-On worked regardless of required-projects. in the previous run: | |
| 23:22:52 | melwitt | 2024-08-22 19:25:30.213727 | controller | b007d1c19e9a6a9ea309dd414448030d9fa6fb72 Merge 'refs/changes/48/926948/1' | |
| 23:22:52 | melwitt | 2024-08-22 19:25:30.213604 | controller | opendev.org/openstack/neutron checked out to: | |
| 23:23:12 | melwitt | that's the proposed DNM change for neutron | |
| 23:26:04 | dansmith | melwitt: because you can't set disk_format=luks in glance so it knows it needs to skip it else it'll just fail outright | |
| 23:26:22 | melwitt | that's what I was thinking, ok | |
| #openstack-nova - 2024-08-23 | |||
| 06:58:34 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: Fix retried resize operation failing with FileNotFound https://review.opendev.org/c/openstack/nova/+/926251 | |
| 09:47:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 09:47:07 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 14:05:14 | frickler | Uggla: auniyal: bauzas: please check the os-vif release patch https://review.opendev.org/c/openstack/releases/+/926768 | |
| 14:48:58 | opendevreview | Rajesh Tailor proposed openstack/nova master: Rephrase the help message https://review.opendev.org/c/openstack/nova/+/925207 | |
| 18:26:10 | gmann | melwitt: sean-k-mooney: if you are around, can you please check this, need 2nd +2 https://review.opendev.org/c/openstack/placement/+/926870/2 | |
| 22:40:07 | opendevreview | Merged openstack/placement master: Update test to use service role https://review.opendev.org/c/openstack/placement/+/926870 | |
| #openstack-nova - 2024-08-24 | |||
| 02:04:30 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 02:04:31 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:56:19 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:56:19 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| #openstack-nova - 2024-08-25 | |||
| 21:50:28 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 21:50:28 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| #openstack-nova - 2024-08-26 | |||
| 00:06:04 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:39:12 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/926556 | |
| 07:39:13 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: Fix PCI passthrough cleanup on reschedule https://review.opendev.org/c/openstack/nova/+/926557 | |
| 08:17:06 | bauzas | morning nova | |
| 08:25:44 | frickler | good morning bauzas, in case you missed it last week: please check the os-vif release patch https://review.opendev.org/c/openstack/releases/+/926768 , we're now past the deadline already | |
| 08:26:18 | bauzas | frickler: ack, I was on PTO for the previous 3 weeks but I'll take a look at it very shortly | |
| 08:28:08 | frickler | thx, just wanted to make sure that you were aware of it with all the flapping that's still happening with your account :) | |
| 08:29:06 | frickler | (I made my client ignore joins and quits, but not name changes, so these kind of stand out to me) | |
| 08:36:17 | opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | |
| 08:36:17 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 08:36:18 | opendevreview | ribaudr proposed openstack/nova master: Use client token when talking to manila https://review.opendev.org/c/openstack/nova/+/925277 | |
| 08:36:18 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 08:36:19 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 08:36:20 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 08:36:20 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 08:36:23 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 08:36:24 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 08:36:26 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 08:36:28 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 08:36:29 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 08:36:30 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 08:36:31 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares https://review.opendev.org/c/openstack/nova/+/860287 | |
| 08:36:33 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 08:36:34 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 08:36:35 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 08:36:36 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 08:36:38 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 08:36:39 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 08:36:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 08:36:42 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 08:36:43 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 08:36:43 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 08:36:44 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 08:36:44 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 08:36:45 | opendevreview | ribaudr proposed openstack/nova master: Reports instance events to the DB regarding attaching and detaching a share https://review.opendev.org/c/openstack/nova/+/927088 | |
| 08:38:32 | auniyal | bauzas, frickler, yes, so I was not sure of https://review.opendev.org/c/openstack/releases/+/926768 | |
| 08:38:47 | auniyal | so was waiting for bauzas | |
| 09:01:25 | bauzas | no worries | |