| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-05-22 | |||
| 11:03:03 | sean-k-mooney | https://github.com/eventlet/eventlet/blob/master/eventlet/green/urllib/request.py | |
| 11:03:07 | gibi | (or we can rip out eventlet from nova and move to pure threads in the next coupe of years :)) | |
| 11:03:42 | sean-k-mooney | well nova-api didnt have any direct depency on eventlet until we added scater gateher | |
| 11:03:55 | sean-k-mooney | so if it was just that we could rip that out | |
| 11:04:15 | sean-k-mooney | but this could obvioulys happen for all teh other client invocations | |
| 11:04:20 | sean-k-mooney | in the other services so ya... | |
| 11:05:08 | gibi | hm, that seems like only monkey patching urllib but not urllib3 | |
| 11:05:44 | sean-k-mooney | maybe i didn't look too closely i assumed it was the same | |
| 11:05:48 | gibi | it seem for urllib3 the socket and select monkey considered enough | |
| 11:07:05 | opendevreview | Merged openstack/os-vif stable/wallaby: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841773 | |
| 11:07:07 | opendevreview | Merged openstack/os-vif stable/wallaby: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841774 | |
| 11:08:50 | sean-k-mooney | oh urllib is the standard libary | |
| 11:09:06 | sean-k-mooney | https://docs.python.org/3/library/urllib.html?highlight=urllib#module-urllib | |
| 11:32:46 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Reproducer for dangling volumes https://review.opendev.org/c/openstack/nova/+/881457 | |
| 11:32:46 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| 13:37:29 | opendevreview | Dan Smith proposed openstack/nova master: Populate ComputeNode.service_id https://review.opendev.org/c/openstack/nova/+/879904 | |
| 13:37:29 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id columns to instances, migrations https://review.opendev.org/c/openstack/nova/+/879499 | |
| 13:37:30 | opendevreview | Dan Smith proposed openstack/nova master: Add dest_compute_id to Migration object https://review.opendev.org/c/openstack/nova/+/879682 | |
| 13:37:30 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |
| 13:37:31 | opendevreview | Dan Smith proposed openstack/nova master: Online migrate missing Instance.compute_id fields https://review.opendev.org/c/openstack/nova/+/879905 | |
| 15:39:39 | opendevreview | Sylvain Bauza proposed openstack/nova stable/zed: Fix get_segments_id with subnets without segment_id https://review.opendev.org/c/openstack/nova/+/883723 | |
| 15:40:02 | opendevreview | Sylvain Bauza proposed openstack/nova stable/yoga: Fix get_segments_id with subnets without segment_id https://review.opendev.org/c/openstack/nova/+/883724 | |
| 15:41:08 | opendevreview | Sylvain Bauza proposed openstack/nova stable/xena: Fix get_segments_id with subnets without segment_id https://review.opendev.org/c/openstack/nova/+/883725 | |
| 15:42:29 | opendevreview | Sylvain Bauza proposed openstack/nova stable/wallaby: Fix get_segments_id with subnets without segment_id https://review.opendev.org/c/openstack/nova/+/883726 | |
| 15:50:58 | sean-k-mooney | o/ | |
| 15:53:33 | sean-k-mooney | melwitt: bauzas dansmith could ye take a look at https://review.opendev.org/c/openstack/nova/+/853269/1 and the follow ups | |
| 15:54:11 | melwitt | sure | |
| 15:54:34 | bauzas | done | |
| 15:55:16 | sean-k-mooney | thanks | |
| 15:55:20 | melwitt | sweet | |
| 16:20:07 | dansmith | so if powerkvm is gone, why do we still have its CI commenting on all our patches? Looks to me like it can't even devstack anymore | |
| 16:20:28 | dansmith | maybe it's just a bot in a cloud somewhere that someone forgot to turn off? | |
| 16:23:38 | opendevreview | Merged openstack/nova stable/xena: Reproducer for bug 1983753 https://review.opendev.org/c/openstack/nova/+/853269 | |
| 16:30:24 | clarkb | dansmith: there should be a contact email in the account as well as on the wiki for the third party ci. Asking them to stop would be the first step and if they don't then we (a gerrit admin) can disable the account | |
| 16:33:29 | dansmith | okay, the "lightbits" CI also seems superfluous, but I think we've asked before and nobody knows what it is? | |
| 16:33:44 | sean-k-mooney | i do | |
| 16:33:56 | sean-k-mooney | its a ci that test tehre os-brick integration | |
| 16:34:34 | sean-k-mooney | its testing changes to https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/lightos.py | |
| 16:34:53 | sean-k-mooney | https://github.com/openstack/nova/commit/b5e2128f3847d444a808a2b0f89e6f1e4ffb77fc | |
| 16:35:26 | sean-k-mooney | we asked them to set it up and maintain it whne https://www.lightbitslabs.com/ wanted to integrate with nova | |
| 16:35:54 | sean-k-mooney | which they did in yoga so its relitivly new | |
| 16:36:40 | sean-k-mooney | baiscally they upstream the integratio they had internally for several years | |
| 16:37:12 | dansmith | hmm, okay | |
| 16:39:42 | dansmith | the powerkvm contact either never made it to oftc or isn't around | |
| 16:40:06 | dansmith | never registered no oftc at least | |
| 16:40:13 | dansmith | I can email to ask | |
| 17:17:08 | opendevreview | Merged openstack/nova stable/xena: Update RequestSpec.pci_request for resize https://review.opendev.org/c/openstack/nova/+/853270 | |
| 17:17:14 | opendevreview | Merged openstack/nova stable/xena: Add reno for fixing bug 1941005 https://review.opendev.org/c/openstack/nova/+/853271 | |
| 17:24:47 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 17:24:48 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 17:24:48 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 17:24:50 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 17:24:51 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 17:24:52 | opendevreview | ribaudr proposed openstack/nova master: Check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 17:24:53 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 17:24:54 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 17:24:54 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 17:24:55 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 17:24:55 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | |
| 17:24:56 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 17:24:56 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 17:24:57 | 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 | |
| 17:24:57 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 17:24:59 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 17:24:59 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 17:25:01 | 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 | |
| 17:25:01 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 17:25:03 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 17:25:03 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 17:25:05 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 17:25:05 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 17:25:07 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 17:25:07 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 17:25:09 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 19:49:36 | opendevreview | Merged openstack/nova master: Fixes a typo in availability-zone doc https://review.opendev.org/c/openstack/nova/+/883474 | |
| 20:15:22 | opendevreview | Ghanshyam proposed openstack/nova master: DNM testing cirros 0.6.1 https://review.opendev.org/c/openstack/nova/+/883875 | |
| #openstack-nova - 2023-05-23 | |||
| 01:09:08 | opendevreview | melanie witt proposed openstack/nova master: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/872470 | |
| 01:09:08 | opendevreview | melanie witt proposed openstack/nova master: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/872471 | |
| 03:31:37 | opendevreview | Ghanshyam proposed openstack/nova master: Add integrated job to test the Nova RBAC old defaults https://review.opendev.org/c/openstack/nova/+/883557 | |
| 09:51:31 | WJeffs | Hey all, I have a unique case, where I need to allocate 380 cores to a x86 VM has anyone got around the i440fx or q35 limits? | |
| 10:24:40 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Reproducer for dangling volumes https://review.opendev.org/c/openstack/nova/+/881457 | |
| 10:24:41 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| 10:30:59 | sean-k-mooney | WJeffs: it should be possibel to do that in the newer releases of nova | |
| 10:31:17 | sean-k-mooney | if we mreged the patch im thinking of let me check | |
| 10:31:59 | WJeffs | sean-k-mooney: were already on Zed, I'm wondering if its due to the kernel setting "KVM_MAX_VCPU" | |
| 10:32:12 | WJeffs | sean-k-mooney: were already on Zed, I'm wondering if its due to the kernel setting "KVM_MAX_VCPU" | |
| 10:33:35 | sean-k-mooney | WJeffs: if you enable the viommu in the guest https://specs.openstack.org/openstack/nova-specs/specs/zed/implemented/libvirt-viommu-device.html#proposed-change | |
| 10:33:51 | sean-k-mooney | we also enable the eim feature | |
| 10:34:39 | sean-k-mooney | https://libvirt.org/formatdomain.html#iommu-devices | |
| 10:34:48 | sean-k-mooney | The eim attribute (with possible values on and off) can be used to configure Extended Interrupt Mode. A q35 domain with split I/O APIC (as described in Hypervisor features), and both interrupt remapping and EIM turned on for the IOMMU, will be able to use more than 255 vCPUs. Since 3.4.0 (QEMU/KVM only) | |
| 10:35:44 | sean-k-mooney | WJeffs: im not sure if that is enough to allow more then 255 vcpus or if you also need to have a machine type and kernel that is set up to deal with that | |
| 10:36:30 | sean-k-mooney | WJeffs: buy ya try adding hw:viommu_model=auto to the flavor or hw_viommu_model=auto to the image | |
| 10:36:47 | sean-k-mooney | this is new in zed | |
| 10:39:06 | sean-k-mooney | WJeffs: we havve not tested that by the way we just did not see any harm in enabling it when enabling the viommu so we made the feature as generic as possible | |
| 10:45:51 | WJeffs | Thanks, I will give it a try. No luck yet. I need to check this kernel has the supported part for more than 288 cores too - I'm pushing to get a 6.x kernel to support the new AMDs currently from my kernel team. | |
| 10:51:42 | sean-k-mooney | ack, we dont often have request to supprot that many cores so your kind of forging your own path here. | |
| 10:52:28 | WJeffs | Yea, I'm not surprised, its a pretty extreme case. Sometimes asking here can save me hours of work | |
| 10:52:36 | sean-k-mooney | if you figure out what combination of libvirt features we need to enable im sure we could actully add support for this usecause but i personlaly have not tested over 64 cores | |
| 10:53:41 | WJeffs | I will take a look and report back if its anything major. | |
| 10:53:46 | WJeffs | Cheers for your help | |