| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-25 | |||
| 13:36:54 | sean-k-mooney | hehe well this seams to be disto specific so not sure its entirly fair | |
| 13:37:05 | sean-k-mooney | at least its not nodejs | |
| 13:37:15 | sean-k-mooney | npm is just awfull | |
| 13:37:42 | sean-k-mooney | its like javascript libs compeet to see how many other libs they can use before they create a cycle | |
| 13:38:00 | dansmith | the js community is insane :) | |
| 13:38:19 | dansmith | I've never had problems resolving deps with npm, but probably just haven't done it enough | |
| 13:38:23 | kashyap | sean-k-mooney: Still, I hate this system pkgs vs PyPi interaction hell. It's been behind me for years; I keep hoping it gets better :D | |
| 13:38:36 | kashyap | dansmith: You're a brave man; this chanel is recorded, you know that, right :D | |
| 13:38:45 | sean-k-mooney | dansmith: depresovling is fine but packaging that as a distro is just not really possible | |
| 13:38:59 | dansmith | kashyap: I think they know they have a problem :) | |
| 13:39:10 | dansmith | sean-k-mooney: yeah totally | |
| 13:39:46 | sean-k-mooney | honestly the best that disto can do is provde a way to install npm and then let it do the rest | |
| 13:40:02 | dansmith | yup and I kinda expect that's where we're headed with python too | |
| 13:40:22 | sean-k-mooney | ya perhaps. go too for what its worth | |
| 13:40:30 | dansmith | I'm sure | |
| 13:40:41 | sean-k-mooney | in the past lanauges did not have a packagem manager but now its the default for them too | |
| 13:41:09 | sean-k-mooney | they had been always thrid party in c/c++/java | |
| 13:42:18 | sean-k-mooney | it does make cve much harder to manage but on the other hand if you fix it in the "comunity/language" package manage you fix it for all distos that use that | |
| 13:42:46 | dansmith | it totally makes the security landscape a disaster, | |
| 13:43:07 | dansmith | but I think npm integrates vulnerability tagging with the package manager itself, which is probably good | |
| 13:46:41 | sean-k-mooney | kashyap: by the way if this is blocking you crrently you might want to use a podman contianer or vm temporaly to workaround it | |
| 13:46:58 | sean-k-mooney | you could also try the pip-compile approch | |
| 13:47:17 | kashyap | sean-k-mooney: It's not quite blocking me; but annoying. Right, for now I'm resorting to a clean container approach. I'll check 'pip-compile' later | |
| 13:47:19 | sean-k-mooney | and perphaps protoype a tox change | |
| 13:47:20 | kashyap | Thakns! | |
| 13:50:08 | kashyap | For now, I'm just not using Py3.10. I have a row of yaks to shave before I head out for the long PTO | |
| 13:50:26 | kashyap | Thx for your debugging help. | |
| 14:08:12 | erlon | Hi folks, can you please give a push on these 2 patches? They are clean cherry-picks and just need 1 mode +2: | |
| 14:08:12 | erlon | https://review.opendev.org/c/openstack/nova/+/836015 | |
| 14:08:12 | erlon | https://review.opendev.org/c/openstack/nova/+/838550/3 | |
| 14:12:35 | sean-k-mooney | bauzas: dansmith ^ can you take a look at those or add me/gibi to the stable group wand we can. the backport is valid in my view and melwitt is +2 on both | |
| 14:15:04 | sean-k-mooney | actully elodilles if you are about you might be abel to help with those stabel reviews | |
| 14:19:36 | elodilles | sean-k-mooney: looking | |
| 14:28:56 | elodilles | erlon: they look good to me, +2+W'd them | |
| 14:29:20 | erlon | elodilles: thanks! | |
| 14:29:32 | elodilles | erlon: thanks for proposing the backport! | |
| 14:35:47 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: skeleton for asserting PCI resources in Placement https://review.opendev.org/c/openstack/nova/+/843291 | |
| 14:45:43 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: DNM: Testing live migration with local attach https://review.opendev.org/c/openstack/nova/+/843146 | |
| 15:25:06 | opendevreview | Merged openstack/nova stable/ussuri: Define new functional test tox env for placement gate to run https://review.opendev.org/c/openstack/nova/+/840771 | |
| 15:25:16 | opendevreview | Merged openstack/nova stable/xena: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838550 | |
| 15:49:23 | opendevreview | Ghanshyam proposed openstack/nova stable/victoria: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/nova/+/843301 | |
| 16:14:35 | opendevreview | Merged openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | |
| 16:33:37 | stephenfin | gibi: +2 on the PCI spec. Nice work | |
| 17:15:36 | sean-k-mooney | melwitt: im doing a fully review fo the pci spec which is why its taking a while not just the detals | |
| 17:15:42 | sean-k-mooney | so far all good | |
| 17:16:11 | sean-k-mooney | but do you want me to levae the +w to you or will i send it on its way assuming im happy with it when i get to the end | |
| 17:16:45 | melwitt | sean-k-mooney: no feel free to +W :) | |
| 17:26:00 | sean-k-mooney | ack it shoudl be sitting in the zuul pipeline now | |
| 17:28:45 | sean-k-mooney | ok im going to swap to my personal laptop and work on some minor patch updates so ill be off irc for the rest or the day. ill leave my client open but ill chat to people tomrrow | |
| 17:32:06 | opendevreview | Miguel Lavalle proposed openstack/os-vif master: Delete trunk bridges to avoid race with Neutron https://review.opendev.org/c/openstack/os-vif/+/841499 | |
| 17:33:09 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838332 | |
| 17:36:10 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838332 | |
| 17:38:04 | opendevreview | Merged openstack/nova-specs master: PCI device tracking in Placement https://review.opendev.org/c/openstack/nova-specs/+/791047 | |
| 18:12:56 | opendevreview | Ghanshyam proposed openstack/nova stable/ussuri: DNM: Testing stable/ussuri with tempest fix for constraints mismatch https://review.opendev.org/c/openstack/nova/+/843046 | |
| 18:51:51 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/843336 | |
| 18:52:01 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/843337 | |
| 18:52:37 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/843338 | |
| 18:52:38 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/843339 | |
| 21:33:11 | opendevreview | Miguel Lavalle proposed openstack/os-vif master: Delete trunk bridges to avoid race with Neutron https://review.opendev.org/c/openstack/os-vif/+/841499 | |
| #openstack-nova - 2022-05-26 | |||
| 07:56:11 | songwenping_ | sean-k-mooney:can we support multi-create vgpu vms? | |
| 09:35:02 | gibi | kashyap: I played with a fedora36 container but I cannot reproduce the slow pip resolving issue either there | |
| 09:35:45 | gibi | so I think you might have extra python3- packages installed that creates some version confusion | |
| 09:35:59 | gibi | it should now do it | |
| 09:36:04 | gibi | as venv is separate | |
| 09:36:08 | gibi | but I have no other ideas | |
| 09:40:46 | sean-k-mooney | songwenping: no that is not supported | |
| 09:41:00 | sean-k-mooney | songwenping: we do not suppport multi create with nested resouce providres | |
| 09:42:01 | sean-k-mooney | songwenping: https://bugs.launchpad.net/nova/+bug/1874664 | |
| 09:42:49 | sean-k-mooney | songwenping: actully it looks like the patches are merged | |
| 09:43:01 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/723884/ and https://review.opendev.org/c/openstack/nova/+/723858/ | |
| 09:45:20 | sean-k-mooney | songwenping: i dont think this fully resolves the issue bauzas ^ | |
| 09:46:28 | sean-k-mooney | songwenping: ya reading them we only added documentation and test to show that it does not work in some cases | |
| 09:46:44 | sean-k-mooney | so form the comit | |
| 09:46:47 | sean-k-mooney | - you can ask for a flavor with 2 VGPU with --max 2 | |
| 09:46:49 | sean-k-mooney | - but you can't ask for a flavor with 4 VGPU and --max 2 | |
| 09:47:23 | kashyap | gibi: Hey, thanks a ton for trying! Also, what do you mean by "it should now do it as venv is separate"? I can't quite parse it :) | |
| 09:47:31 | kashyap | Pls rephrase it | |
| 09:47:55 | gibi | tox should build a new virtual env that is sealed off from python packages on the host | |
| 09:49:02 | kashyap | Yeap; thanks! | |
| 09:49:18 | gibi | interestingly the dockerfile in the bugreport shows the issue to me but my dockerfile doesn't | |
| 09:49:24 | gibi | craxy | |
| 09:53:03 | opendevreview | Rajat Dhasmana proposed openstack/nova-specs master: Repropose volume backed server rebuild spec https://review.opendev.org/c/openstack/nova-specs/+/840155 | |
| 09:54:09 | whoami-rajat | sean-k-mooney, ^^ updated the spec | |
| 09:57:34 | sean-k-mooney | whoami-rajat: thanks ill review it shortly | |
| 09:57:52 | whoami-rajat | thanks! | |
| 10:28:07 | songwenping | sean-k-mooney: no, not this bug. | |
| 10:28:54 | sean-k-mooney | songwenping: are you refering to creating a vm with multipel vgpus? | |
| 10:28:57 | songwenping | i create two vms with flavor that has 1 vgpu metadata | |
| 10:29:09 | sean-k-mooney | ok | |
| 10:29:44 | songwenping | libvirt allocate the same vgpu for these two vms | |
| 10:29:47 | sean-k-mooney | we only support doing that via 2 differnt api request today | |
| 10:30:12 | sean-k-mooney | we do not supprot creating 2 vms that use vgpus in the same api request | |
| 10:30:37 | songwenping | does the api check the request? | |
| 10:30:44 | sean-k-mooney | no | |
| 10:30:54 | sean-k-mooney | we will not block it at the api | |
| 10:31:20 | sean-k-mooney | we docuement it in the api as part of the previous bug | |
| 10:31:57 | songwenping | why donot we block it at api? | |
| 10:31:59 | sean-k-mooney | gibi: speaking of which if you use pci device in placment for now at least multi create will not work for the same reason | |
| 10:32:12 | sean-k-mooney | songwenping: becuase at the api we dont know fi it uses nested resouce provder or not | |
| 10:32:27 | sean-k-mooney | so we documented the limistaion | |