| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-03-10 | |||
| 18:37:01 | sean-k-mooney | libpcre3-dev: /usr/include/pcre.h | |
| 18:37:24 | sean-k-mooney | gmann: so no noble https://review.opendev.org/c/openstack/nova/+/935577/3/bindep.txt is not correct | |
| 18:37:43 | sean-k-mooney | line 53 should be all that is required | |
| 18:37:50 | sean-k-mooney | libpcre3-dev [platform:dpkg test] | |
| 18:39:24 | gmann | yeah, libpcre3-dev should be enough but it is not finding it when building wheel for python-pcre | |
| 18:39:58 | sean-k-mooney | i can try doing it in a clean vm and see what happens | |
| 18:40:11 | gmann | or you can see PS2 test run and it fails there too https://review.opendev.org/c/openstack/nova/+/935577/2 | |
| 18:40:29 | gmann | but log expire there | |
| 18:40:34 | gmann | expired | |
| 18:40:53 | gmann | oh, its there https://zuul.opendev.org/t/openstack/build/5ddcd6d9f0a94008a722346992cf499b | |
| 18:42:24 | sean-k-mooney | ya so this sound like somethign to do with the build chain | |
| 18:42:41 | sean-k-mooney | and by that i mean how the toll chain is set up in the vm | |
| 18:43:16 | sean-k-mooney | like its missing one of the devel packages like make or something is missing | |
| 18:47:02 | sean-k-mooney | i dont think this is it but one thing i do notice is i have a slightly old verion of virutal env locally then it used in the failed buil | |
| 18:47:13 | sean-k-mooney | virtualenv = 20.26.3 locally | |
| 18:58:13 | sean-k-mooney | gmann: so not a clean vm but https://paste.opendev.org/show/bv240D3BK3XVEKBywbMN/ | |
| 18:58:33 | sean-k-mooney | if i disable using all binaireas and force a build it definly works | |
| 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 | ralonsoh | yes, this vif plug method I was aware of it | |
| 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: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 | |