| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-03-10 | |||
| 19:04:05 | sean-k-mooney | gmann: ok i create a clean ubuntu vm on our internal cloud. im gong to confirm pcre.h is not pre installed in the image and then only install the bindep package with the test profile | |
| 19:04:22 | sean-k-mooney | and ill see if mater doc target fail or not | |
| 19:06:57 | sean-k-mooney | ... cloning nova is painful aat 500KiB/s | |
| 19:12:29 | sean-k-mooney | gmann: so this is what im going to install | |
| 19:12:32 | sean-k-mooney | .venv) ubuntu@sean-upstream-docs:~/nova$ bindep -b test | nc termbin.com 9999 | |
| 19:12:34 | sean-k-mooney | https://termbin.com/hrrp | |
| 19:12:51 | sean-k-mooney | that is for master | |
| 19:12:58 | sean-k-mooney | i guess i should also include the docs target | |
| 19:23:26 | sean-k-mooney | gmann: so locally that works for me on a clean vm | |
| 19:23:50 | sean-k-mooney | gmann: what i did was as followes | |
| 19:23:53 | sean-k-mooney | sudo apt install python3.12-venv | |
| 19:24:03 | sean-k-mooney | python3 -m venv .venv | |
| 19:24:12 | sean-k-mooney | actully ill past it | |
| 19:25:03 | sean-k-mooney | https://paste.opendev.org/show/bq494eyZBhcRJTR1ATEm/ | |
| 19:27:15 | sean-k-mooney | this seams to be somethign either specific to the ci vms or to the playbooks we are using to do this in ci | |
| 19:29:35 | gmann | yeah, it seems so. something wit editable install with pyproject.toml ? but not sure | |
| 19:29:57 | opendevreview | Ghanshyam proposed openstack/nova master: Testing doc job on Ubuntu Noble https://review.opendev.org/c/openstack/nova/+/935577 | |
| 20:12:36 | opendevreview | Merged openstack/nova master: Update libvirt fixtures to support hostdevs https://review.opendev.org/c/openstack/nova/+/943207 | |
| 20:20:21 | opendevreview | Merged openstack/nova master: Update driver to map the targeted address for SR-IOV PCI devices https://review.opendev.org/c/openstack/nova/+/942147 | |
| #openstack-nova - 2025-03-11 | |||
| 01:12:17 | opendevreview | suiong ng proposed openstack/nova stable/2024.2: Fix parameter order in add_instance_info_to_node https://review.opendev.org/c/openstack/nova/+/943994 | |
| 06:19:40 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: enable NFS backend job https://review.opendev.org/c/openstack/nova/+/944008 | |
| 06:25:49 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: enable NFS backend job https://review.opendev.org/c/openstack/nova/+/944008 | |
| 08:02:20 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: enable NFS backend job https://review.opendev.org/c/openstack/nova/+/944008 | |
| 08:15:42 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: enable NFS backend job https://review.opendev.org/c/openstack/nova/+/944008 | |
| 08:28:34 | opendevreview | Amit Uniyal proposed openstack/nova master: DNM: enable NFS backend job https://review.opendev.org/c/openstack/nova/+/944008 | |
| 08:32:08 | ralonsoh | hello folks, where exactly in Nova compute the VF is created and the VLAN ID defined? | |
| 08:32:14 | ralonsoh | can you point me to the code? | |
| 08:32:28 | ralonsoh | I'm talking about SR-IOV ports | |
| 08:40:54 | ralonsoh | bauzas, sean-k-mooney ^ | |
| 08:41:36 | sean-k-mooney[m] | the vlan id is compute by neturon | |
| 08:41:50 | sean-k-mooney[m] | we just apply the value ye provide to the vf | |
| 08:41:52 | ralonsoh | sean-k-mooney[m], the VLAN ID is set by Neutron? | |
| 08:42:01 | sean-k-mooney[m] | no its set by nova | |
| 08:42:06 | ralonsoh | no no, I know the VLAN info is provided by Nova | |
| 08:42:14 | ralonsoh | but where this VLAN ID is set in the VF? | |
| 08:42:17 | sean-k-mooney[m] | but we use the value form the neutron network accosated with the neutron port | |
| 08:42:38 | ralonsoh | by Neutron* (not Nova) | |
| 08:42:51 | sean-k-mooney[m] | its set by nova in the libvirt xml | |
| 08:43:01 | ralonsoh | ok, this is what I was looking for | |
| 08:43:07 | sean-k-mooney[m] | ill find the code one sec | |
| 08:43:08 | ralonsoh | thanks! | |
| 08:46:22 | sean-k-mooney | so our LibvirtConfigGuestInterface has a vlan property https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L1912 which is rendered into the xml for type direct and hostdeve vnic_type=direct is hostdev in this case and vnic_type=macvtap is direct... | |
| 08:46:24 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L2015-L2019 | |
| 08:46:57 | ralonsoh | right, so the VLAN is set in the interface definition | |
| 08:47:00 | ralonsoh | thanks a lot! | |
| 08:48:11 | sean-k-mooney | that invloked form here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L630-L657 | |
| 08:49:01 | sean-k-mooney | ralonsoh: so yes it defiend in the libvirt defintion and it will program the VLAN filter via the PF before it assigned the VF to qemu | |
| 08:49:01 | ralonsoh | yes, this vif plug method I was aware of it | |
| 08:49:09 | ralonsoh | but I didn't see where the VLAN was defined | |
| 08:49:13 | ralonsoh | but is in the XML | |
| 08:50:09 | ralonsoh | sean-k-mooney, ups, now you are here | |
| 08:50:15 | ralonsoh | qq about resource providers | |
| 08:50:24 | sean-k-mooney | sure | |
| 08:50:44 | ralonsoh | in Neutron, if we add BW info for an agent (SRIOV, OVN, etc) and a network | |
| 08:50:57 | ralonsoh | Neutron will use the API to create the RP, the inventories, etc | |
| 08:51:06 | ralonsoh | what if we remove the RP? | |
| 08:51:18 | ralonsoh | I don't think we have considered that | |
| 08:51:43 | ralonsoh | at least in OVN, that reads the RP info not from the configuration but from the chassis (so there is no API restart) | |
| 08:51:48 | sean-k-mooney | so the design at the time considered removing an RP that has allcoation ot be operator error | |
| 08:52:25 | ralonsoh | so if there is an error in the configuration (for example, the physnet name is incorrect) | |
| 08:52:33 | ralonsoh | do we need to manually remove the RP ? | |
| 08:52:36 | opendevreview | Maksim Malchuk proposed openstack/nova stable/2024.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944009 | |
| 08:52:56 | ralonsoh | ah ah, so if there is allocation, it will fail | |
| 08:53:05 | sean-k-mooney | so it depends. we proposed but did not implemtne the idea of owner triats | |
| 08:53:34 | sean-k-mooney | the idea was that neutron or nvoa could add an owner trait to all RPs it owns | |
| 08:54:01 | ralonsoh | we use custom names for the physnets | |
| 08:54:02 | sean-k-mooney | that way it could check fi ther ewere any that were created that should not exist | |
| 08:54:54 | ralonsoh | hmmm I think this "deletion" operations are going to be more difficult than expected | |
| 08:54:56 | opendevreview | Maksim Malchuk proposed openstack/nova stable/2024.1: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944010 | |
| 08:54:57 | sean-k-mooney | ralonsoh: right so you use thing slik same subtree | |
| 08:55:12 | ralonsoh | yes, the host->agent subtree | |
| 08:55:19 | ralonsoh | so we "own" this RP | |
| 08:55:38 | sean-k-mooney | ya so you can get all of the sub RPs in the subtree that match your nameign convention | |
| 08:55:41 | opendevreview | Maksim Malchuk proposed openstack/nova stable/2023.2: Allow hyphen in cinder catalog_info service-type https://review.opendev.org/c/openstack/nova/+/944011 | |
| 08:55:45 | sean-k-mooney | and delete any that shoudl not exist | |
| 08:55:57 | sean-k-mooney | if you have a OWNER_NEUTRON trait on them its safer | |
| 08:56:19 | sean-k-mooney | as even if there are two service usign the same naming convetion you can look at only the ones you created | |
| 08:56:24 | sean-k-mooney | that makes deleting it simper | |
| 08:56:34 | ralonsoh | yeah, this could be a good improvement for sure | |
| 08:56:56 | sean-k-mooney | for now however you know they starte with or contain phsenet right | |
| 08:56:57 | ralonsoh | that will mark our RP with no doubt | |
| 08:57:16 | ralonsoh | sean-k-mooney, and the agent RP, that is handled by Neutron | |
| 08:57:30 | sean-k-mooney | so you can must look at the RPs in the subtree for the agent | |
| 08:57:43 | sean-k-mooney | and remove any that are not in you current config. | |
| 08:57:45 | ralonsoh | ok, I think I know (maybe) how to do this | |
| 08:57:47 | ralonsoh | reference: https://issues.redhat.com/browse/OSPRH-14597 | |
| 08:57:52 | ralonsoh | I'll open a LP bug too now | |
| 08:57:58 | sean-k-mooney | if they have allcoation it indicates that the removal migt be in error | |
| 08:58:09 | ralonsoh | yes, this is important: we can't remove used RP | |
| 08:58:39 | sean-k-mooney | i trhink placement blocks it by default but i dont recall | |
| 08:58:55 | sean-k-mooney | when we are removign compute nodes we loop overa ll the allcoation on the RP and delete them first | |
| 08:58:59 | sean-k-mooney | then delete the rp | |
| 08:58:59 | ralonsoh | yes, you are right | |
| 08:59:06 | ralonsoh | in any case, I'll try this first | |
| 09:00:27 | sean-k-mooney | ralonsoh: if you can tell there is a rename | |
| 09:00:34 | sean-k-mooney | you can also do what is called a reshape | |
| 09:00:44 | sean-k-mooney | whish is the operation of moving allcoation between RPs | |
| 09:01:08 | sean-k-mooney | they are non tivial but its how we evovled the stucture over time | |
| 09:01:30 | sean-k-mooney | https://docs.openstack.org/api-ref/placement/#id86 | |
| 09:02:58 | sean-k-mooney | so for a rename you would add the new rp, reshape the inventories/allocation from the old then delete the old RP | |