| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-10 | |||
| 14:09:58 | gibi | kashyap: but even if it clears CI I smell trouble | |
| 14:10:09 | kashyap | Yeah, I'm adding a quick review comment | |
| 14:11:52 | bauzas | I made a clear statement already | |
| 14:12:28 | bauzas | gibi: we shouldn't wait for this massive reqs update for os-traits | |
| 14:12:34 | kashyap | bauzas: Aaah, I missed your comment | |
| 14:12:47 | gibi | bauzas: we can try to propose a separate bump that only moves os-traits | |
| 14:12:50 | bauzas | gibi: want me to propose some u-c update for os-traits ? | |
| 14:12:53 | kashyap | Ah, we wrote the comment at the same moment (3:11 PM) | |
| 14:13:01 | gibi | bauzas: yeah, go ahead, I can +1 it | |
| 14:13:05 | bauzas | https://review.opendev.org/c/openstack/requirements/+/872065/4/upper-constraints.txt#361 | |
| 14:13:17 | bauzas | gibi: fwiw, os-traits isn't upgraded in this patch | |
| 14:13:35 | gibi | bauzas: yeah that patch wasn't regenerated since os-traits 2.10 landed | |
| 14:13:50 | gibi | I think it will be regenerated eventually but we don't need to wait for it | |
| 14:13:50 | bauzas | on it then | |
| 14:13:54 | gibi | thanks | |
| 14:14:16 | gibi | once we have 2.10 in global reqs we need a bump in placement too as we want to release placement with the latest os-traits | |
| 14:15:27 | bauzas | why isn't the bot proposing a new release ? https://review.opendev.org/c/openstack/requirements/+/854821 | |
| 14:16:58 | gibi | bauzas: maybe because it is waiting for the other bump to land? | |
| 14:17:00 | gibi | I'm not sure | |
| 14:17:02 | gibi | elodilles: ^^ | |
| 14:19:40 | gibi | bauzas: the spice image compression patch https://review.opendev.org/c/openstack/nova/+/828675 is nicely written and easy to review. I | |
| 14:19:43 | gibi | I | |
| 14:19:45 | gibi | I | |
| 14:19:45 | gibi | I'm +2 on it | |
| 14:19:55 | gibi | (too much coffee...) | |
| 14:20:11 | bauzas | gibi: ok, then, I'll take a look on it | |
| 14:20:22 | gibi | I think it is an easy win :) | |
| 14:20:29 | bauzas | I like easy wins | |
| 14:20:55 | bauzas | our antelope release will have bad numbers so anything that helps to improve our numbers is good :) | |
| 14:22:52 | gibi | and I jumped the gun on the manila series regarding the os-traits release. The manial series only needs os-traits 2.8.0 so that is OK. So only the scaphandre and the max_phy_bits series are blocked on os-traits 2.10.0 | |
| 14:23:43 | bauzas | that is correct | |
| 14:23:59 | bauzas | and since the scaphandre series actually relies on manila... | |
| 14:24:17 | bauzas | well, on virtiofs support coming from the manila series... | |
| 14:24:47 | gibi | yeah | |
| 14:28:05 | bauzas | gibi: as a wrap-up | |
| 14:28:13 | bauzas | I just discussed with the releases team | |
| 14:28:29 | bauzas | and the releases post-merge jobs are failing since Feb3 | |
| 14:29:00 | bauzas | in other words, os-vif and os-traits aren't formally released yet | |
| 14:29:08 | bauzas | despite the releases patch got merged | |
| 14:40:04 | gibi | ohh nice | |
| 14:46:10 | gibi | I see https://pypi.org/project/os-traits/ only has 2.9.0 | |
| 14:47:20 | bauzas | gibi: join the openstack-releases channel if you wanna get some fun | |
| 14:47:44 | gibi | I'm in :) but don't need more fun | |
| 15:24:16 | bauzas | gibi: +Wd the image compression patch | |
| 15:24:18 | bauzas | yay | |
| 15:24:53 | gibi | it won't land due to the ceph job block but still it is an achievement \o/ | |
| 15:28:27 | bauzas | yeah, and I'll chase the rechecks | |
| 15:50:46 | dansmith | I think zuul is down | |
| 15:52:33 | opendevreview | Merged openstack/python-novaclient master: Bump microversion to 2.95 https://review.opendev.org/c/openstack/python-novaclient/+/872418 | |
| 15:54:47 | dansmith | s'back | |
| 15:56:24 | bauzas | dansmith: there were problems with post-release pipelines | |
| 15:56:32 | bauzas | hence the zuul restart | |
| 15:56:41 | dansmith | yeah | |
| 15:58:07 | bauzas | dansmith: gmann: we'll get yet another failure for your tempest image fix https://zuul.openstack.org/status#873300 | |
| 15:58:17 | dansmith | yep | |
| 15:58:26 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Look for cpu controller on cgroups v2 https://review.opendev.org/c/openstack/nova/+/873127 | |
| 16:02:55 | bauzas | melwitt: could you please remove your -2 on https://review.opendev.org/c/openstack/nova/+/863177 (the ironic-vnc-console blueprint got accepted for the cycle) | |
| 16:06:41 | bauzas | gibi: I think we rounded on all the possible blueprints we have | |
| 16:07:13 | gibi | bauzas: yeah, I don't have brainpower to look at the ironic vnc one | |
| 16:07:16 | bauzas | we can try to take a look at https://review.opendev.org/c/openstack/nova/+/863177 possibly but sounds a bit optimistic | |
| 16:07:29 | gibi | yeah | |
| 16:08:14 | bauzas | gmann: planning to progress on https://review.opendev.org/c/openstack/nova/+/864594 ? | |
| 16:08:26 | gibi | I think I won't start anything big any more today but I'm still around for a bit if specific review is needed | |
| 16:08:34 | bauzas | me too, I'm done for today | |
| 16:08:50 | bauzas | I'll start reviewing the maxphysnet series today, but I'm not an expert in this | |
| 16:09:01 | bauzas | s//maxphysaddr | |
| 16:10:05 | gibi | ohh that one I can take a look | |
| 16:10:08 | gibi | I reviewd the spec there | |
| 16:12:43 | spatel | sean-k-mooney Hi, I have a question related HugePages, currently i am running SRIOV + CPU pinning + HugePages | |
| 16:13:27 | spatel | Lets say i don't want to use HugePages. Does that possible? | |
| 16:13:41 | bauzas | spatel: sean-k-mooney is on PTO until wed | |
| 16:13:49 | bauzas | lemme try to answer you | |
| 16:14:15 | bauzas | spatel: yes, it's possible to have CPU pinning without huge pages | |
| 16:14:26 | bauzas | but I guess you have running workloads ? | |
| 16:15:00 | spatel | Yes | |
| 16:15:39 | spatel | I am deploying new cloud and planning to not use HugePage. We had some incident in past related memory cause strange issue. | |
| 16:15:43 | bauzas | ok, so, do you want to tune off hugepages for all your computes but a subset ? | |
| 16:15:53 | bauzas | ah | |
| 16:16:00 | spatel | New cloud with no HugePage at all.. | |
| 16:16:17 | bauzas | ok, then you just need to use flavors that don't request hugepages | |
| 16:16:37 | spatel | This is what i have currently in my cloud - intel_iommu=on iommu=pt hugepagesz=2M hugepages=30000 transparent_hugepage=never | |
| 16:16:57 | spatel | Thinking to remove hugepages and change flavor | |
| 16:17:46 | bauzas | https://docs.openstack.org/nova/latest/admin/cpu-topologies.html | |
| 16:18:40 | bauzas | I need to verify one bit, sec | |
| 16:18:49 | spatel | sure! | |
| 16:19:18 | bauzas | https://docs.openstack.org/nova/latest/admin/huge-pages.html | |
| 16:19:53 | bauzas | so, say you no longer ask for hugepages, it won't request a NUMA topolgy | |
| 16:20:18 | spatel | That was my next question.. How numa play with HugePages? | |
| 16:20:38 | spatel | We want our workload schedule in single NUMA zone | |
| 16:21:54 | bauzas | so you want NUMA without hugepages | |
| 16:22:48 | spatel | Yes | |
| 16:23:27 | spatel | I believe openstack automatically schedule workload according NUNA correct? | |
| 16:23:33 | bauzas | see that doc https://docs.openstack.org/nova/latest/admin/cpu-topologies.html#customizing-instance-numa-placement-policies | |
| 16:23:40 | gibi | bauzas: the max_phy_address patch is just the start of the series. we will need more patches there. | |
| 16:24:00 | bauzas | either you explicitly specific a NUMA topology for your guest or you make it implicit with cpu pinning or hugepages flavor extra specs | |
| 16:24:09 | bauzas | gibi: ack | |
| 16:24:23 | spatel | This is my flavor properties - hw:cpu_policy='dedicated', hw:cpu_sockets='2', hw:cpu_threads='2', hw:mem_page_size='large', hw:pci_numa_affinity_policy='preferred', sriov='true' | |
| 16:24:57 | bauzas | spatel: so, you'd just remove the mention of using large pages | |
| 16:24:58 | spatel | I didn't tell my workload about where to schedule but it always put my VMs on single NUMA zone | |
| 16:25:27 | spatel | Yep! that is what i am thinking, remove from grub and flavor and it should be fine. | |