Friday, 2023-10-27

gmannsean-k-mooney: bauzas: backport of HyperV, vmware driver deprecation backport https://review.opendev.org/q/topic:deprecate-virt-drivers+status:open 03:22
opendevreviewSylvain Bauza proposed openstack/nova master: DNM : Test a new compute RPC version with an error  https://review.opendev.org/c/openstack/nova/+/89939207:42
bauzasgmann: ack, looking07:45
sahiddansmith: o/ https://review.opendev.org/c/openstack/nova/+/896512 have you had a chance to look at my response to your comment? I guess no :-)08:17
opendevreviewJan Horstmann proposed openstack/nova master: Respect dedicated network during block migration with `iso9660` config drives  https://review.opendev.org/c/openstack/nova/+/89945809:23
bauzasgibi: sean-k-mooney: given how big will be the conversation on VGPUs, I think we should start discussing this afternoon about either the image type backend problem or the live migration URIs09:37
bauzasand constraint that first topic to be only for 30 mins09:37
bauzasso it would leave 1h30 for VGPUs09:38
bauzasgibi: sean-k-mooney: which one would you guys prefer to be discussed in 30 mins09:38
bauzas?09:38
gibibauzas: good question10:11
gibipas-ha had some comments to the migration_uri item on the etherpad where I would need more details. So if he is not around then we might not be able to full settle the migration_uri topic10:12
gibiso I suggest to by default go with the image type first to let pas-ha to come back with details10:13
opendevreviewPranali Deore proposed openstack/nova master: DNM: Test glance new location api  https://review.opendev.org/c/openstack/nova/+/89120710:44
bauzasgibi: ack12:22
* bauzas disappears for 20 mins 12:22
opendevreviewMerged openstack/os-vif master: Update master for stable/2023.2  https://review.opendev.org/c/openstack/os-vif/+/89406412:39
kashyapbauzas: Was this already discussed yesterday? "mdev live-migration in libvirt"?13:19
kashyapLooks like so; at least there's notes in the Etherpad13:19
bauzaskashyap: https://ptg.opendev.org/ptg.html13:20
bauzaswe're discussing image type backends13:20
kashyapYeah, I'm just looking at the minutes.  I think that image backend code is a bit delicate to do any deep surgery.13:21
kashyaps/minutes/Etherpad/13:21
kashyapBut deprecating the use_cow and force_raw is fine if they're causing more trouble than it's worth it.13:25
gibikashyap: vgpu stuff coming up soonish13:26
gibiin couple of minutes I guess13:26
kashyapgibi: Thanks for the heads-up!  I need to step out to the library as there's some construction noise near my apartment.13:27
kashyap(I'm interested in the live-migration related discussion on that topic.  Back in afew)13:27
kashyapsean-k-mooney: Heh, on your "90 seconds" remark!14:25
whoami-rajatbauzas, hey, here is the cinder meeting room https://www.openstack.org/ptg/rooms/cactus15:02
whoami-rajathope the nova team is available for the session15:03
bauzaswhoami-rajat: we're late, trying to join in a couple of mins15:05
whoami-rajatbauzas, sure, no problem15:06
bauzassee you folks either in the cinder room or at 4pm UTC back in the nova room15:15
pas-ha[m]bauzas: hi there, just to highlight my nick for you15:16
bauzaspas-ha[m]: sold !15:16
bauzaspas-ha[m]: and thanks for voluteering btw. !15:16
sean-k-mooneybauzas: ptg done?16:04
bauzassean-k-mooney: no, still on cinder room16:04
sean-k-mooneyoh cinder room16:04
sean-k-mooneycool ill join there16:05
sean-k-mooneyi just rejoined ours16:05
noonedeadpunkHey folks. Sorry for bothering with basic libvirt questions, but I'm slightly /o\. After upgrading some computes from 20.04 to 22.04 I've spotted they do have `LIBVIRT_PRT` tables in iptables that does `MASQUERADE  all  --  192.168.122.0/24    !192.168.122.0/24 `16:14
noonedeadpunkSo if VM is in network `192.168.122.0/24` - it get's SRCNAT-ed basically, and goes to the virtual namespace from mgmt IP of compute...16:15
sean-k-mooneywe dont use libvirt filtering in nova16:15
sean-k-mooneynova-networks may have but nova with neutron does not use libvirt for filtering16:16
noonedeadpunkdo you know from top of your head how to just disable that? As I don't see mentioning `open` or smth like that here: https://libvirt.org/firewall.html#the-virtual-network-driver16:18
noonedeadpunkLIke forward?16:19
opendevreviewHao Wang proposed openstack/nova stable/zed: Decorate only Flavor.get_* methods that execute queries  https://review.opendev.org/c/openstack/nova/+/89911916:21
sean-k-mooneyim not sure exactly but downstream we are just not deploying the libvirt networkign driver at all16:21
sean-k-mooneynoonedeadpunk: what backend are you useing16:22
sean-k-mooneyon the neutron side16:22
noonedeadpunkovs16:22
sean-k-mooneyovs with ml2/ovs with ovs firewal driver, iptable firewall driver, noop firewall driver or are you using ovs with ovn16:23
bauzaswe are back in the nova room16:23
noonedeadpunkah, firewall driver, iptables_hybrid16:24
noonedeadpunkml2/ovs16:24
noonedeadpunkbut yeah, that stuff is created by libvirt default network. Which is wiped as soon as it's dropped16:27
noonedeadpunkand I believe it's because of using iptables_hybrid actually... And what catched me is that before libvirt wasn't creating a default net on install16:40
sean-k-mooneyya you cand safely delete the default network16:54
sean-k-mooneywell default newtork in libvirt not in neutron :)16:55
noonedeadpunknot safely though, as then all instances loose network...16:55
noonedeadpunkbut can...16:55
sean-k-mooneyso really it should not be applying any rules ot the nova vms even with hybrid plug16:56
sean-k-mooneythe interface are type eterhenet or bridge and not assocated with the defautl network16:56
johnthetubaguybauzas: sorry, only just got your note on here. Thanks for the good discussion on the PCI topics, that was useful.17:26
sean-k-mooneyjohnthetubaguy: i wanted to ask you one other thing before you go17:27
johnthetubaguysure thing17:27
sean-k-mooneyjohnthetubaguy: we can chat more about pci in the future but do you know if the ironic-shard work will get completed in C17:27
sean-k-mooneybasically it would be nice to rever the revert but are you or JayF  going to work on completing that this cycle17:28
johnthetubaguyah, I am hoping we can re-propose that once the SDK is fixed up, and I think JayF has got good ways to test that now17:28
sean-k-mooneycool17:28
sean-k-mooneyya that is what i was hoping you were going to say17:28
johnthetubaguyI should re-propose the spec, etc.17:29
sean-k-mooneysure feel free to ping when its up. i dont think there is anythign contovertiol other then the testing section17:29
JayFsean-k-mooney: johnthetubaguy: The goal is, in my mind: 1) add better sharding tempest testing in ironic-proper 2) add sharding tempest tests at nova/ironic api interface (with 'bad' nodes in a shard  not under test) 3) merge!17:29
sean-k-mooneybefore we merge the code this time i think we are going to want to see this in an ironic devstack jobs17:29
JayFjohnthetubaguy: AIUI the SDK work is already done17:29
dansmithsean-k-mooney: ++17:30
JayFjohnthetubaguy: TheJulia: and I also are going to chat about looking over the Ironic driver to resolve some bugs, including the mixed use of ironicclient and openstacksdk17:30
JayFwhich leads to appearently inconsistent version negotiation17:30
johnthetubaguysean-k-mooney: ++17:30
JayFand completely muddied all this troubleshooting in the moment during bobcat17:30
sean-k-mooneyJayF: ya one of our unspoken goals is to complete moving form ironic client to sdk17:30
sean-k-mooneyJayF: stephenfin i belive had a series to do that17:31
sean-k-mooneywhich we just need to review and or update17:31
johnthetubaguyJayF: ace17:31
johnthetubaguysean-k-mooney: ah, I remembered we were half way there, I didn't realise the code had been up for that, nice.17:31
JayFTBH; PTG planning for Ironic+TC, plus I'm speaking at SeaGL a week from tomorrow, I'm just swamped, and detailed planning for this piece got pushed until after that talk was done :)17:32
sean-k-mooneyJayF: johnthetubaguy https://review.opendev.org/q/topic:bp%252Fopenstacksdk-in-nova17:32
sean-k-mooneyi belvie that fully move the ironic driver to the sdk17:32
JayFack; nice! 17:33
sean-k-mooneyjohnthetubaguy: JayF: stephenfin likely wont have time to continue workign on that but if either of ye do then im happy to review17:33
JayFdoes that man have useful patchsets for everything hidden somewhere17:34
JayFholy cow \o/17:34
johnthetubaguyJayF: lets catch up next week on who does what, and checking on getting the testing in place, etc.17:34
sean-k-mooneyJayF: we had a soft goal to start doing the convertion last cycle in general to using sdk in stead of project client and stephenfin volenteer to show how to do it for ironic17:34
JayFjohnthetubaguy: explicitly not next week. Week after next.17:35
johnthetubaguyJayF: cool17:35
JayFjohnthetubaguy: note previous message about being busy until a week from tomorrow :D 17:35
johnthetubaguysean-k-mooney: dumb question, is it best as a new patch or a revert revert in the commit message?17:38
sean-k-mooneyjohnthetubaguy: honestly either is fine i would proably revert the revert and or cleanup the commit message17:48

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!