| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-10-27 | |||
| 15:03:26 | whoami-rajat | hope the nova team is available for the session | |
| 15:05:45 | bauzas | whoami-rajat: we're late, trying to join in a couple of mins | |
| 15:06:13 | whoami-rajat | bauzas, sure, no problem | |
| 15:15:00 | bauzas | see you folks either in the cinder room or at 4pm UTC back in the nova room | |
| 15:16:20 | pas-ha[m] | bauzas: hi there, just to highlight my nick for you | |
| 15:16:39 | bauzas | pas-ha[m]: sold ! | |
| 15:16:51 | bauzas | pas-ha[m]: and thanks for voluteering btw. ! | |
| 16:04:37 | sean-k-mooney | bauzas: ptg done? | |
| 16:04:52 | bauzas | sean-k-mooney: no, still on cinder room | |
| 16:04:58 | sean-k-mooney | oh cinder room | |
| 16:05:01 | sean-k-mooney | cool ill join there | |
| 16:05:04 | sean-k-mooney | i just rejoined ours | |
| 16:14:25 | noonedeadpunk | Hey 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:15:06 | noonedeadpunk | So 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:31 | sean-k-mooney | we dont use libvirt filtering in nova | |
| 16:16:45 | sean-k-mooney | nova-networks may have but nova with neutron does not use libvirt for filtering | |
| 16:18:27 | noonedeadpunk | do 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-driver | |
| 16:19:55 | noonedeadpunk | LIke forward? | |
| 16:21:44 | opendevreview | Hao Wang proposed openstack/nova stable/zed: Decorate only Flavor.get_* methods that execute queries https://review.opendev.org/c/openstack/nova/+/899119 | |
| 16:21:52 | sean-k-mooney | im not sure exactly but downstream we are just not deploying the libvirt networkign driver at all | |
| 16:22:12 | sean-k-mooney | noonedeadpunk: what backend are you useing | |
| 16:22:17 | sean-k-mooney | on the neutron side | |
| 16:22:40 | noonedeadpunk | ovs | |
| 16:23:21 | sean-k-mooney | ovs with ml2/ovs with ovs firewal driver, iptable firewall driver, noop firewall driver or are you using ovs with ovn | |
| 16:23:56 | bauzas | we are back in the nova room | |
| 16:24:19 | noonedeadpunk | ah, firewall driver, iptables_hybrid | |
| 16:24:36 | noonedeadpunk | ml2/ovs | |
| 16:27:49 | noonedeadpunk | but yeah, that stuff is created by libvirt default network. Which is wiped as soon as it's dropped | |
| 16:40:13 | noonedeadpunk | and 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 install | |
| 16:54:50 | sean-k-mooney | ya you cand safely delete the default network | |
| 16:55:22 | sean-k-mooney | well default newtork in libvirt not in neutron :) | |
| 16:55:30 | noonedeadpunk | not safely though, as then all instances loose network... | |
| 16:55:32 | noonedeadpunk | but can... | |
| 16:56:32 | sean-k-mooney | so really it should not be applying any rules ot the nova vms even with hybrid plug | |
| 16:56:51 | sean-k-mooney | the interface are type eterhenet or bridge and not assocated with the defautl network | |
| 17:26:28 | johnthetubaguy | bauzas: sorry, only just got your note on here. Thanks for the good discussion on the PCI topics, that was useful. | |
| 17:27:12 | sean-k-mooney | johnthetubaguy: i wanted to ask you one other thing before you go | |
| 17:27:31 | johnthetubaguy | sure thing | |
| 17:27:45 | sean-k-mooney | johnthetubaguy: we can chat more about pci in the future but do you know if the ironic-shard work will get completed in C | |
| 17:28:27 | sean-k-mooney | basically it would be nice to rever the revert but are you or JayF going to work on completing that this cycle | |
| 17:28:28 | johnthetubaguy | ah, 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 now | |
| 17:28:40 | sean-k-mooney | cool | |
| 17:28:57 | sean-k-mooney | ya that is what i was hoping you were going to say | |
| 17:29:03 | johnthetubaguy | I should re-propose the spec, etc. | |
| 17:29:37 | sean-k-mooney | sure feel free to ping when its up. i dont think there is anythign contovertiol other then the testing section | |
| 17:29:47 | JayF | sean-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:53 | sean-k-mooney | before we merge the code this time i think we are going to want to see this in an ironic devstack jobs | |
| 17:29:57 | JayF | johnthetubaguy: AIUI the SDK work is already done | |
| 17:30:03 | dansmith | sean-k-mooney: ++ | |
| 17:30:23 | JayF | johnthetubaguy: 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 openstacksdk | |
| 17:30:29 | JayF | which leads to appearently inconsistent version negotiation | |
| 17:30:30 | johnthetubaguy | sean-k-mooney: ++ | |
| 17:30:39 | JayF | and completely muddied all this troubleshooting in the moment during bobcat | |
| 17:30:58 | sean-k-mooney | JayF: ya one of our unspoken goals is to complete moving form ironic client to sdk | |
| 17:31:09 | sean-k-mooney | JayF: stephenfin i belive had a series to do that | |
| 17:31:17 | sean-k-mooney | which we just need to review and or update | |
| 17:31:28 | johnthetubaguy | JayF: ace | |
| 17:31:48 | johnthetubaguy | sean-k-mooney: ah, I remembered we were half way there, I didn't realise the code had been up for that, nice. | |
| 17:32:15 | JayF | TBH; 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:24 | sean-k-mooney | JayF: johnthetubaguy https://review.opendev.org/q/topic:bp%252Fopenstacksdk-in-nova | |
| 17:32:51 | sean-k-mooney | i belvie that fully move the ironic driver to the sdk | |
| 17:33:52 | JayF | ack; nice! | |
| 17:33:56 | sean-k-mooney | johnthetubaguy: JayF: stephenfin likely wont have time to continue workign on that but if either of ye do then im happy to review | |
| 17:34:07 | JayF | does that man have useful patchsets for everything hidden somewhere | |
| 17:34:11 | JayF | holy cow \o/ | |
| 17:34:50 | johnthetubaguy | JayF: lets catch up next week on who does what, and checking on getting the testing in place, etc. | |
| 17:34:57 | sean-k-mooney | JayF: 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 ironic | |
| 17:35:11 | JayF | johnthetubaguy: explicitly not next week. Week after next. | |
| 17:35:24 | johnthetubaguy | JayF: cool | |
| 17:35:29 | JayF | johnthetubaguy: note previous message about being busy until a week from tomorrow :D | |
| 17:38:13 | johnthetubaguy | sean-k-mooney: dumb question, is it best as a new patch or a revert revert in the commit message? | |
| 17:48:05 | sean-k-mooney | johnthetubaguy: honestly either is fine i would proably revert the revert and or cleanup the commit message | |
| #openstack-nova - 2023-10-29 | |||
| 16:40:12 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | |
| 21:06:23 | opendevreview | Merged openstack/nova master: objects: Stop fetching from security_groups table https://review.opendev.org/c/openstack/nova/+/860850 | |
| 21:06:32 | opendevreview | Merged openstack/nova master: disable ovn based testing of test_live_migration_with_trunk https://review.opendev.org/c/openstack/nova/+/895655 | |
| #openstack-nova - 2023-10-30 | |||
| 06:37:35 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix trivial doc issues https://review.opendev.org/c/openstack/nova/+/878779 | |
| 10:24:37 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix trivial doc issues https://review.opendev.org/c/openstack/nova/+/878779 | |
| 12:28:29 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix trivial doc issues https://review.opendev.org/c/openstack/nova/+/878779 | |
| 14:10:07 | opendevreview | Merged openstack/nova stable/zed: Decorate only Flavor.get_* methods that execute queries https://review.opendev.org/c/openstack/nova/+/899119 | |
| 15:59:41 | elodilles | Uggla auniyal bauzas : fyi, i've proposed nova release patches as i promised on last nova meeting. we can discuss if they are OK or we should wait any of them tomorrow @ nova meeting. https://review.opendev.org/q/project:openstack/releases+is:open+intopic:nova | |
| 17:14:16 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: Remove no longer used RPs https://review.opendev.org/c/openstack/nova/+/899625 | |
| 17:16:03 | dansmith | bauzas: I'm rebasing your rpc fix series.. do you mind if I also s/nullable/optional/ in the descriptions of the fix for those params/ | |
| 17:16:25 | dansmith | (per my comment here: https://review.opendev.org/c/openstack/nova/+/899234/4) | |
| 17:16:59 | bauzas | dansmith: ahah you're right | |
| 17:17:10 | bauzas | 'optional' should be the best name | |
| 17:17:22 | bauzas | I'll update my series | |
| 17:18:10 | opendevreview | Dan Smith proposed openstack/nova master: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/899234 | |
| 17:18:11 | opendevreview | Dan Smith proposed openstack/nova master: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/899235 | |
| 17:18:11 | opendevreview | Dan Smith proposed openstack/nova master: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/899375 | |
| 17:18:15 | dansmith | bauzas: ^ | |
| 17:18:21 | bauzas | cool thanks | |
| 17:18:59 | dansmith | melwitt: that series is pretty trivial reproduce, optionalize some args, include in tests ^ | |
| 17:19:01 | dansmith | ready for +W | |
| 17:21:04 | dansmith | (and fixes two cycles of broken RPC which basically breaks our SLURP guarantees) | |
| 17:22:24 | bauzas | I wish we had a reload method for a ProviderTree | |
| 17:22:41 | bauzas | anyway, I need to go off by now | |
| 17:23:12 | bauzas | I just discovered a race condition which I dislike | |
| 17:23:12 | bauzas | fuck nvidia | |
| 17:24:15 | melwitt | dansmith: ack, will look | |
| 17:25:06 | dansmith | melwitt: thanks, should be fairly easy I think :) | |