| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-30 | |||
| 14:06:00 | artom | Seems like just skipping the None case with an if and retrying the loop iteration would help... | |
| 14:07:14 | sean-k-mooney | manuvakery1: that feels like something i have heard fo before but its obviously a qemu bug | |
| 14:07:56 | sean-k-mooney | i guess it could be in libvirt but its defnintly not in nova | |
| 14:08:47 | sean-k-mooney | well cloud.get is presumable reading form the clouds.yaml which should nto be none | |
| 14:09:49 | artom | I think it's an API call... it's polling for stack events | |
| 14:10:17 | sean-k-mooney | i didnt look at the actual errror yet | |
| 14:10:19 | artom | Sorry, cloud.get_stack() :) | |
| 14:10:34 | artom | Which I assume calls Heat | |
| 14:10:44 | sean-k-mooney | presumeable | |
| 15:11:07 | opendevreview | Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627 | |
| 16:01:41 | dansmith | sean-k-mooney: +2 now ^ | |
| 17:04:35 | sean-k-mooney | dansmith: im out of energy now but i reveiwed that and approved. if you are going to request a FFE for the nova enablement of multiple locations support, given its size i think i woudl support that but if just want to leave that to next cycle it think thats also fine. it really depends n when the tempest stuff get resolved i guess so ill deffer to your judgement there. | |
| 17:05:10 | dansmith | well, tbh, I wasn't even going to consider it a feature | |
| 17:05:30 | dansmith | it's tiny and it's just using the glance api the right way, | |
| 17:05:39 | dansmith | but if you think it needs to be considered as such, then that's fine | |
| 17:05:52 | dansmith | not sure what that really means with bauzas gone next week though | |
| 17:06:09 | sean-k-mooney | well more i was suggeting its small enouch that we may want to consier not procedurlly blocking it | |
| 17:06:25 | sean-k-mooney | if the tempet issue can be resolved by the next team meeting | |
| 17:06:38 | sean-k-mooney | well FFE dont requrie PTL input | |
| 17:06:50 | sean-k-mooney | historically it only requried 3 cores to say yes | |
| 17:07:23 | sean-k-mooney | i only consider https://review.opendev.org/c/openstack/nova/+/891036 a feature in that we would not back port it | |
| 17:08:04 | sean-k-mooney | mainly because of the requirements bump but i agree otherwise its just using glances api the secure way | |
| 17:08:04 | dansmith | there's no point in backporting it | |
| 17:08:22 | sean-k-mooney | right because there is no supprot in older glance | |
| 17:08:27 | dansmith | I meant determining whether it was a feature or not, for the PTL buy-in.. I think that's bauzas' decision to make if it's unclear | |
| 17:08:58 | dansmith | anyway, we'll keep pushing on the tempest patch and can revisit on monday maybe | |
| 17:09:24 | sean-k-mooney | ack, either way i thin kthis could merge after rc1 if we decied to wait | |
| 17:09:29 | dansmith | the tempest patch isn't required, AFAIK, it just reduces a race that is already there just not hit because we don't actually use it yet, but it has passed without it | |
| 17:09:32 | sean-k-mooney | having it 2025.1 is the main thing | |
| 17:10:10 | sean-k-mooney | ah ok so its not strictly a dep but we would have some instablity without it | |
| 17:10:34 | dansmith | right | |
| 17:11:01 | sean-k-mooney | ok well im going to checkout for the weekend | |
| 17:11:07 | sean-k-mooney | catch you monday o/ | |
| 17:11:18 | dansmith | o/ | |
| 18:39:29 | opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/927639 | |
| 19:38:14 | opendevreview | Merged openstack/nova stable/2023.1: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/912511 | |
| 19:39:16 | bauzas | I'm still there, I have some slides to work on | |
| 19:59:18 | JayF | I think I'm going to need an FFE for 927639 if it's merging for C; I've been blocked on reviews and got some today and revised. I'd imagine it'll be ready to land very early next week. TBH, I'd like it in so it's off my plate, but it won't cause significant hardship if we have to push it. | |
| 19:59:29 | JayF | s/C/D/ | |
| 20:53:41 | opendevreview | Brian Haley proposed openstack/nova master: Support creating servers with RBAC SGs https://review.opendev.org/c/openstack/nova/+/811521 | |
| 20:57:27 | opendevreview | Merged openstack/nova master: Add hw_firmware_stateless image property https://review.opendev.org/c/openstack/nova/+/926590 | |
| 21:00:45 | opendevreview | Merged openstack/nova master: Cleanup setup.py https://review.opendev.org/c/openstack/nova/+/905419 | |
| 22:18:48 | melwitt | JayF: I think you accidentally added a new Change-Id to your next PS, so it did not go onto the review we've been doing. it made a new patch instead | |
| 22:19:18 | melwitt | *a new review instead | |
| 22:19:18 | JayF | egad | |
| 22:21:53 | opendevreview | Jay Faulkner proposed openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/923910 | |
| 22:22:27 | JayF | I had to completely blow away the commit hook to keep it from trying to add a new one everytime | |
| 22:23:16 | JayF | thanks melwitt :) | |
| 22:24:05 | melwitt | hm, I guess it really requires Change-Id to be the last line of the commit message. but apparently not all the time (?) | |
| 22:24:36 | JayF | yeah I don't really understand, but I'm just glad you noticed so I could fix it :) | |
| 23:52:00 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 23:52:01 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 23:52:01 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 23:52:02 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 23:52:02 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| #openstack-nova - 2024-08-31 | |||
| 01:46:16 | manuvakery1 | sean-k-mooney: it was due to newly added ceph mon host with a different weight. Eventhough the new mon hosts are not added in to ceph.conf it was impacting all vms which has the ceph backend/or ceph volume attached to it. | |
| 01:47:02 | sean-k-mooney[m] | the ceph mons are encoded in the libvirt xml | |
| 01:47:19 | sean-k-mooney[m] | so you cant change the ips or expand the set if you have active vms | |
| 02:52:38 | melwitt | bauzas: I just +W the ironic metadata patch with the understanding that FF was extended to today (friday) | |
| 04:42:24 | opendevreview | Merged openstack/nova master: [ironic] Factor out metadata and send to ironic https://review.opendev.org/c/openstack/nova/+/923910 | |
| 09:06:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: Ignore support-matrix.css https://review.opendev.org/c/openstack/nova/+/927659 | |
| 13:00:12 | opendevreview | Merged openstack/nova stable/2023.1: Separate OSError with ValueError https://review.opendev.org/c/openstack/nova/+/912512 | |
| 14:51:49 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop workaround for old keystoneauth https://review.opendev.org/c/openstack/nova/+/927669 | |
| #openstack-nova - 2024-09-01 | |||
| 04:19:11 | opendevreview | Merged openstack/nova master: update nova-next to use ubuntu 24.04 https://review.opendev.org/c/openstack/nova/+/922492 | |
| 07:20:55 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | |
| 07:20:56 | opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | |
| 07:20:56 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 07:20:57 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:20:57 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 07:23:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 07:23:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:23:07 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra specs for USB redirection. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 07:57:56 | opendevreview | Merged openstack/nova master: libvirt: Launch instances with stateless firmware https://review.opendev.org/c/openstack/nova/+/908890 | |
| 07:58:03 | opendevreview | Merged openstack/nova master: Fix s/addtional/additional/ typo https://review.opendev.org/c/openstack/nova/+/926083 | |
| 10:18:23 | opendevreview | Merged openstack/nova stable/2023.1: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/912513 | |
| 12:46:54 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop unused method https://review.opendev.org/c/openstack/nova/+/927680 | |
| 12:53:17 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop workaround for old keystoneauth https://review.opendev.org/c/openstack/nova/+/927669 | |
| #openstack-nova - 2024-09-02 | |||
| 02:27:00 | opendevreview | Merged openstack/nova master: Ignore support-matrix.css https://review.opendev.org/c/openstack/nova/+/927659 | |
| 05:52:40 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 05:52:40 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 05:52:41 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 05:53:08 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 05:54:47 | opendevreview | renliang proposed openstack/nova master: Update documentation link for openSUSE index. https://review.opendev.org/c/openstack/nova/+/927705 | |
| 06:03:37 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 06:03:37 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 07:26:10 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 07:26:10 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 07:48:55 | opendevreview | Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627 | |
| 09:28:16 | opendevreview | Merged openstack/nova master: Handle neutron-client conflict https://review.opendev.org/c/openstack/nova/+/918048 | |
| 10:44:53 | ykarel | is it known that https://review.opendev.org/c/openstack/nova/+/899753 broke 9-stream jobs atleast? | |
| 10:45:07 | ykarel | fails as chown: cannot access '/opt/stack/nova/*.egg-info': No such file or directory | |
| 10:46:27 | ykarel | sean-k-mooney, ^ | |
| 12:43:50 | opendevreview | Rajesh Tailor proposed openstack/nova stable/2024.1: Handle neutron-client conflict https://review.opendev.org/c/openstack/nova/+/927732 | |
| 12:48:16 | opendevreview | Merged openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | |
| 13:22:55 | opendevreview | chandan kumar proposed openstack/nova master: [DNM] RDO third party experimental job check https://review.opendev.org/c/openstack/nova/+/927736 | |
| 13:25:55 | ykarel | sean-k-mooney, for that egg-info issue the hack https://review.opendev.org/c/openstack/devstack/+/927730 worked fine | |
| 13:26:40 | ykarel | also refreshed your patch for global venv https://review.opendev.org/c/openstack/devstack/+/898845 but looks need more work as jobs failing there | |
| 14:57:57 | sean-k-mooney | ykarel: ya so it needs the wsgi stuff i think to be able to install in a venv | |
| 14:59:04 | sean-k-mooney | ykarel: but ya you could ignore safe_chown or you could jsut check for "is_fedora == false" | |
| 14:59:31 | sean-k-mooney | the wsgi stuff beign https://review.opendev.org/q/topic:%22remove-wsgi_scripts%22 | |