| opendevreview | Ghanshyam proposed openstack/nova master: Introduce 'admin' policy base rule https://review.opendev.org/c/openstack/nova/+/819389 | 01:15 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/nova master: Introduce 'admin' policy base rule https://review.opendev.org/c/openstack/nova/+/819389 | 01:22 |
| opendevreview | Ghanshyam proposed openstack/nova master: Convert aggregate policies to admin and system scope https://review.opendev.org/c/openstack/nova/+/819390 | 01:44 |
| opendevreview | Ghanshyam proposed openstack/nova master: Introduce 'admin' policy base rule https://review.opendev.org/c/openstack/nova/+/819389 | 01:56 |
| gmann | dansmith: gibi bauzas I registered the nova BP for the RBAC goal. please let me know if I need to add spec or specless BP (with all direction defined in community wide goal) can be approved https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-2 | 02:00 |
| gmann | dansmith: gibi bauzas I mean all the direction are defined in goal itself so we can do audit of nova policy changes in proposed code itself or in wiki page before we code - https://wiki.openstack.org/wiki/Nova/rbac | 02:03 |
| gmann | johnthetubaguy[m]: ^^ | 02:03 |
| lyarwood | gibi / bauzas ; https://review.opendev.org/c/openstack/nova/+/819194 - would be good to get your thoughts on this btw | 08:25 |
| gibi | lyarwood: do we want nova-tox-functional-centos8-py36 to be changed to run on py38? or that would require centos9? | 08:32 |
| lyarwood | gibi: We can drop centos8-stream jobs entirely from master as the supported runtime is moving to 9 | 08:32 |
| * lyarwood needs to check on the devstack progress for that job | 08:33 | |
| lyarwood | gibi: and yeah 9 then brings py39 iirc | 08:33 |
| gibi | lyarwood: so then what I see in that change is consistent with the global testing runtime change | 08:34 |
| gibi | lyarwood: do you have some reservation about this change/ | 08:34 |
| gibi | ? | 08:34 |
| lyarwood | gibi: Nope I'm good with it, gmann just had reservations so I wanted more people to chime in | 08:34 |
| lyarwood | IMHO we should remove as much overhead as possible and that includes support/test runs for older runtimes | 08:35 |
| gibi | lyarwood: OK, I gave +2, I agree with you that we need to force py38+ in the setup.cfg | 08:40 |
| lyarwood | ack thanks, lets see what bauzas says as PTL before we +W | 08:41 |
| gibi | sure | 08:41 |
| gibi | stephenfin: as a fun note there is blue: https://blue.readthedocs.io/en/latest/ it is almost like black :D | 08:42 |
| lyarwood | flol I love that logo | 08:42 |
| gibi | I laught a lot the other day when somebody linked it in twitter | 08:43 |
| gibi | I went and +2d the other yoga testing runtime changes under our jurisdiction | 08:51 |
| *** brinzhang_ is now known as brinzhang | 08:53 | |
| kashyap | TIL; "blue" | 08:56 |
| kashyap | :D | 08:56 |
| gibi | somebody should start working on lightblue or even teal | 08:57 |
| kashyap | :) | 08:59 |
| gibi | gmann: re: RBAC: I don't have hard opinion. In one hand I see good dicsussion in https://review.opendev.org/c/openstack/nova-specs/+/793011 about a specific subset of the RBAC change, but on the other hand I don't think we need such discussion for all the policy during our audit. | 09:10 |
| gibi | gmann: so I would be OK with a specles BP and a promise that if we find something non trivial during the audit then we might raise a spec for that | 09:11 |
| bauzas | gibi: gmann: we can quickly discuss this BP for the next nova meeting | 09:23 |
| bauzas | like we do for the others | 09:23 |
| bauzas | lyarwood: looking | 09:23 |
| bauzas | lyarwood: ok, so I'm happy with https://review.opendev.org/c/openstack/nova/+/819194/ but I guess we'll need centos jobs once they run on top of centos9 ? | 09:33 |
| bauzas | I mean, devstack/centos9 | 09:33 |
| bauzas | if you say so, let's +W it | 09:34 |
| bauzas | but I'll add a comment | 09:34 |
| lyarwood | bauzas: ACK yeah it's being worked on https://review.opendev.org/c/openstack/devstack/+/800903 | 09:37 |
| * lyarwood will try to help out later today with that | 09:38 | |
| bauzas | OK, I'll mention it then | 09:38 |
| * bauzas was trying to find the devstack patch, appreciated. | 09:38 | |
| lyarwood | this isn't going to land until we drop the current centos 8 job from the integrated compute template btw | 09:38 |
| lyarwood | so actually hold off on +W | 09:38 |
| lyarwood | I'll throw something up to drop tempest-integrated-compute-centos-8-stream | 09:39 |
| bauzas | I see | 09:39 |
| bauzas | oh, saw the discussion on https://review.opendev.org/c/openstack/nova/+/819194/1/setup.cfg#b13 | 09:40 |
| bauzas | I'll chime into in | 09:40 |
| bauzas | it* | 09:40 |
| lyarwood | oh crap I thought that was updated in this PS | 09:43 |
| lyarwood | okay so -1 until that's actually blocked in setup.cfg | 09:43 |
| bauzas | oh, shit, I just sent it to the gate. | 09:47 |
| bauzas | lyarwood: I thought we had a consensus | 09:47 |
| lyarwood | yeah but setup.cfg hadn't changed to reflect it | 09:47 |
| lyarwood | I missed that earlier | 09:48 |
| lyarwood | I thought the latest PS had changed, we were discussing stuff on an older PS that confused things | 09:48 |
| bauzas | lyarwood: I eventually thought we said "OK but meh" | 09:50 |
| bauzas | lyarwood: what we can is to provide a FUP for modifying setup.cfg, no ? | 09:51 |
| bauzas | gibi: thoughts ? | 09:51 |
| lyarwood | bauzas: yeah sure I can do that now | 09:51 |
| bauzas | lyarwood: cool, appreciated | 09:52 |
| lyarwood | apologies, that was my fault | 09:52 |
| * bauzas prefers to have a FUP than stopping the gate run | 09:52 | |
| bauzas | lyarwood: no no it's me | 09:52 |
| bauzas | I saw gibi +2ing it | 09:52 |
| bauzas | and I thought the consensus was to say "nice talk, but let's just accept py36 to be unsupported but still used" | 09:53 |
| bauzas | hence my comment | 09:53 |
| opendevreview | Lee Yarwood proposed openstack/nova master: fup: Require python >= 3.8 from Yoga https://review.opendev.org/c/openstack/nova/+/819415 | 09:57 |
| lyarwood | Yeah no issues, there's the fup anyway | 09:57 |
| lyarwood | we can discuss more there | 09:57 |
| lyarwood | tbh thinking about it we might want to broadcast something like this on the ML | 09:59 |
| lyarwood | as it's going to break any centos8 stream jobs people have | 09:59 |
| bauzas | lyarwood: yeah, you're right | 10:01 |
| * gibi is on a looong call | 10:03 | |
| * kashyap gives Gibi the meeting ticker (which tracks the bleeding dollars): https://tobytripp.github.io/meeting-ticker/ | 10:12 | |
| kashyap | Plug in approximate numbers for the first entries; hit "Start" and watch it roll | 10:13 |
| lyarwood | https://zuul.opendev.org/t/openstack/build/4b3e8872abc54e96b992747b739b7d3b/log/job-output.txt looks like we need to update the LC job as well before this will work | 10:14 |
| gibi | kashyap: 200 hourly rate?! I need to talk to my manager .... | 10:15 |
| kashyap | gibi: It's usually managers and their managers that might have that hourly rate :D | 10:16 |
| stephenfin | gibi: Nice :-D | 10:16 |
| kashyap | gibi: The website is also US-based; as we know, the numbers won't be a 1-1 mapping for EU. | 10:17 |
| gibi | I can imagine it maps to some EU countries but not to where I sit. :D | 10:21 |
| bauzas | gibi: that depends on if you count the company payslip taxes too | 10:25 |
| bauzas | maybe folks don't know, but for 100€ in a French brute payslip, the person gets somethink like 70€ and the company pays *140€* | 10:26 |
| bauzas | ie. in between what the company pays, and what the person gets, it's around 100% taxes | 10:27 |
| gibi | bauzas: similar here | 10:33 |
| * kashyap doesn't even want to talk about Belgian taxes (50% flat out) | 10:34 | |
| gibi | from 100 I get 66 and company pays a total of 117 | 10:34 |
| gibi | numbers are nicer with kids. | 10:35 |
| bauzas | interesting comparison | 10:37 |
| sean-k-mooney[m] | bauzas i think irish takes work out at more. 40% income tax on aything earned over 39000 a year + 4.5% universal social charge (usc) on your gross income over ~19,000 + paye related social insurace (prsi) at 4% by employee and 11.5% by employer | 10:37 |
| bauzas | man, did I opened a can of worms :D | 10:37 |
| sean-k-mooney[m] | so for you t orecive 100 gross in ireland the comapany pay 115 and you will recive about 51 euro net | 10:38 |
| bauzas | again, interesting | 10:39 |
| sean-k-mooney[m] | at least if you are in the higher tage bracket for earning more then 39,000 a year | 10:39 |
| bauzas | looks like associates pay less for taxes than the company, but at the end, taxes are similar | 10:39 |
| bauzas | sean-k-mooney: sames goes here | 10:39 |
| bauzas | I reflected my level of pay | 10:40 |
| bauzas | and yeah, whether you're having kids, you pay less | 10:40 |
| bauzas | but this was limited to a discount of 1500€ per kid roughly | 10:41 |
| bauzas | (per year) | 10:41 |
| sean-k-mooney[m] | we do it slightly differnt in that you dont pay less but you get a tax credit which is deuected form your yearly tax | 10:41 |
| sean-k-mooney[m] | we also have paye pay as you earn | 10:42 |
| sean-k-mooney[m] | so you dont file your own tax returns | 10:42 |
| bauzas | if people read asimov's foundation (and don't look at the TV series which is a joke), they'll understand that any single govnerment can only generate more taxes over the time | 10:42 |
| sean-k-mooney[m] | your employer deducts tax at source and files it for you | 10:42 |
| sean-k-mooney[m] | which i like since for the most part you dont haver to worry about this | 10:43 |
| bauzas | very quickly, we have three levels of taxes for a single payslip | 10:44 |
| bauzas | 1/ the company taxes, payed by the company, which are roughly 47% except for minimum wages | 10:44 |
| bauzas | 2/ the social, retirement and insurance taxes, which are roughy 15-20%, and independent of your household | 10:45 |
| kashyap | bauzas: I've been wanting to read Asimov; thanks for the reminder! | 10:45 |
| bauzas | 3/ the income taxes, which were only collected once per year very late, and which changed recently and are now paid directly over the payslip | 10:46 |
| bauzas | kashyap: don't read my twitter account, you could get spoiled as I yelled multiple times at the series | 10:47 |
| kashyap | bauzas: Heh, don't worry; I'm social-media clean for 9 years now :-) | 10:47 |
| bauzas | kashyap: I also retired from Twitter | 10:47 |
| kashyap | More power to you. | 10:47 |
| kashyap | JOMO++ (joy of missing out) | 10:48 |
| bauzas | hell yeah, and Facebook is just very little used | 10:48 |
| bauzas | kashyap: never read Asimov N | 10:48 |
| bauzas | ? | 10:48 |
| bauzas | this is vast, I'd recomment you to start with Foundation's Prelude to get a chronological view instead of a publication view | 10:49 |
| bauzas | but you'd also have to read some other books | 10:49 |
| kashyap | bauzas: No. The only sci-fi I've ever read is Iain M. Banks' Culture series | 10:49 |
| bauzas | the Robots cycle, obviously | 10:49 |
| kashyap | Noted :) Thx for the reco | 10:50 |
| bauzas | but there are also some books very worthy between the Robots period and the Foundation perioid | 10:50 |
| * bauzas goes swating at the gym | 10:50 | |
| bauzas | sweating* even | 10:50 |
| sean-k-mooney[m] | kashyap your missing out. i “read” mostly via audio books on audible. i have complete abour 280 ish books in the last 6 years almost all are sifi/space opera/lit rpg or are fantasy. as someone with dislexia who has preverisly only read a hand full of books every audio books really work for me | 10:59 |
| kashyap | sean-k-mooney[m]: Yeah, I'll completely believe you. And I'm glad that you're able to enjoy audio books | 10:59 |
| sean-k-mooney[m] | that remiands me i think a new book in one of the series im following is out soon i should check | 11:01 |
| kashyap | sean-k-mooney[m]: On missing out; I hear ya. I mostly read (physical) books in the area of "cognitive science": neuroscience, psychology, primate/human behaviour, biology, ancient Greek/Roman philosophy, and linguistics :D | 11:01 |
| kashyap | I do read fiction; but have reduced it a lot. I should get back to it a bit more, to lighten up the mood :) | 11:02 |
| kashyap | sean-k-mooney[m]: For audio, I listen to 90-minute podcasts, but haven't yet tried a proper audiobook. I'll really need to give it a go | 11:02 |
| kashyap | Thanks for the reminder! | 11:03 |
| * kashyap today learnt of LitRPG genre; thx :-) | 11:03 | |
| lyarwood | bauzas / sean-k-mooney ; https://review.opendev.org/c/openstack/nova/+/815373 - I think I've got my head around this, it's weird but I guess it's valid as a bugfix. Would appreciate review and maybe some TODO fups from someone setting out a better solution that doesn't rely on the persistent domain XML. | 11:03 |
| kashyap | lyarwood: "The | 11:06 |
| kashyap | ... limitation has been lifted since then" -- is that in libvirt? | 11:06 |
| kashyap | (From the commit message) | 11:06 |
| sean-k-mooney[m] | i think in qemu | 11:06 |
| lyarwood | Yeah correct, they didn't list the version in the commit message but it's in the releasenote | 11:06 |
| kashyap | The hot-unplug of mediated device, i.e. | 11:06 |
| lyarwood | and yeah I'm sure it's both libvirt and QEMU | 11:07 |
| sean-k-mooney[m] | basicaly nova managed save did not support mdevs | 11:07 |
| kashyap | sean-k-mooney[m]: It's libvirt, the previous sentence talks of libvirt | 11:07 |
| kashyap | lyarwood: Yeah; that's also correct, actually. | 11:07 |
| sean-k-mooney[m] | *libvirt managed save | 11:07 |
| kashyap | Yea | 11:08 |
| sean-k-mooney[m] | kashyap well the libvirt limitation was becasue of a qemu limitation i belive | 11:08 |
| sean-k-mooney[m] | anyway now they do supprot | 11:08 |
| sean-k-mooney[m] | it so we dont need too or want too detach the mdevs right | 11:08 |
| sean-k-mooney[m] | but resume need to support both the new and old behavior | 11:08 |
| kashyap | (Yeah; I said it could be both a follow-up above.) | 11:09 |
| sean-k-mooney[m] | ack | 11:11 |
| sean-k-mooney[m] | i have not reviewd that patch closely by the way | 11:11 |
| sean-k-mooney[m] | at the end of the day i still think we should be tracking the mdevs in novas db and recreating form the db record instead of trying to use the guest xml to store the mdevs used | 11:13 |
| opendevreview | Merged openstack/nova master: Updating tests with Yoga testing runtime https://review.opendev.org/c/openstack/nova/+/819194 | 11:13 |
| sean-k-mooney[m] | e.g. right now we are relying on the domain to exist to prevent other vms from using the mdevs which is fragile | 11:14 |
| sean-k-mooney[m] | at least in comparison to storing them in the resoucs table or pci devices table. it has the advantage of being local i guess but it make reaning about this harder since we dont track the arent device ectra in other edge cases like host reboot | 11:16 |
| sean-k-mooney[m] | so i hope we can make this simpler and just recored it in the db eventurally | 11:16 |
| lyarwood | Yeah agreed, I think for now this is a valid backportable fix but really it needs to be replaced by something from our DB or placement or something | 11:18 |
| lyarwood | It's just super confusing given we detach from the active and then use the persistent domain to work out what we need to reattach before we then restore the managedSave state of the domain | 11:19 |
| lyarwood | I guess an alternative is for libvirt to support managedSave somehow for mdevs, likely by the caller telling it not to care about their state or something? | 11:20 |
| sean-k-mooney[m] | so i think it dose now right ? | 11:20 |
| sean-k-mooney[m] | oh no | 11:21 |
| sean-k-mooney[m] | what has changed is we blocked suspend before becaue we could not unplug and replug the mdev | 11:21 |
| sean-k-mooney[m] | now we can | 11:21 |
| sean-k-mooney[m] | oh there is another nasty bug there | 11:23 |
| sean-k-mooney[m] | so we previously detach the interface on suspend right | 11:24 |
| sean-k-mooney[m] | now we are reattaching them when we did not before which is good | 11:24 |
| sean-k-mooney[m] | but we sometimes suspend the guest when we do snapshot | 11:25 |
| sean-k-mooney[m] | so as a side effect if you could not use live snap shots snapshotting would remove the mdev and not reattach them | 11:25 |
| sean-k-mooney[m] | so this will fix that too | 11:25 |
| opendevreview | Lee Yarwood proposed openstack/nova master: fup: Require python >= 3.8 from Yoga https://review.opendev.org/c/openstack/nova/+/819415 | 11:26 |
| lyarwood | do we use suspend for that? | 11:27 |
| lyarwood | I thought that was a pause on the domain itself | 11:27 |
| lyarwood | ah it's suspend | 11:28 |
| lyarwood | TIL | 11:28 |
| lyarwood | okay yeah resume needs to be updated | 11:28 |
| sean-k-mooney[m] | ya i tought it would be pause too untile i read it again just now. | 11:29 |
| lyarwood | https://github.com/openstack/nova/blob/d630615a02469442fb50ed4aa7e092206a28166a/nova/virt/libvirt/driver.py#L3043-L3063 that even | 11:29 |
| lyarwood | nice catch | 11:29 |
| lyarwood | tbh the TODO from stephenfin is right, that could just be a single call to self.resume | 11:29 |
| sean-k-mooney[m] | actully i tought that just called resume like the suppoed_for_snappshot calls suspend | 11:30 |
| sean-k-mooney[m] | ya | 11:30 |
| sean-k-mooney[m] | it should be. | 11:30 |
| opendevreview | Lee Yarwood proposed openstack/nova master: DNM Testing python >= 3.8 requirement https://review.opendev.org/c/openstack/nova/+/819432 | 11:38 |
| sean-k-mooney | lyarwood:... unfortunetly we might need to keey requires_python>=3.6 for rdo this cycle due to upgrdes | 11:50 |
| sean-k-mooney | which makes me think we cant actully turn off testin 3.6 until z | 11:51 |
| lyarwood | Why would we need to keep it? | 11:51 |
| sean-k-mooney | because they will be supprot yoga on centos 8 and 9 for upgrades | 11:51 |
| lyarwood | Can't we just ship py38 in the Yoga containers? | 11:51 |
| sean-k-mooney | we can but they ahve to supprot rpm install. and to be fair 3.8 is avaiable on cents too | 11:52 |
| sean-k-mooney | * centos8 too | 11:52 |
| sean-k-mooney | its just not the default | 11:52 |
| lyarwood | Sigh why didn't the RDO folks raise that with the TC before they moved the runtimes for Yoga | 11:53 |
| lyarwood | brb | 11:53 |
| sean-k-mooney | ya i dont know i really dont like the idea of our testing and min requires being differnet so i dotn know what to do to be honest | 11:54 |
| lyarwood | Well for RDO at least they can workaround it easily enough | 11:55 |
| lyarwood | by installing py38 alongside py36 | 11:55 |
| gibi | lyarwood, bauzas: re py36 support: sorry I also missed that we are discussing that on an older PS. :/ | 11:55 |
| gibi | lyarwood bauzas: I agree to land the fup on setup.cfg | 11:56 |
| lyarwood | np my language here didn't help, I confused things massively | 11:56 |
| gibi | and I do believ that RDO missed the deadline not to raise the issue on the TC ruling | 12:03 |
| sean-k-mooney | im also kind of confused why RDO have not been testign with 3.8 as well | 12:09 |
| sean-k-mooney | it has been a testing requirement for some time | 12:10 |
| sean-k-mooney | i guess they were usign the lower requiremtn for centos of 3.6 | 12:10 |
| sean-k-mooney | but give our downstream will be using 3.9 form wallaby+ that feels like a gap someone form redhat would have been woking on fixing | 12:12 |
| sean-k-mooney | py 3.9.8 currrenlty py 3.8 will not be supproted on centos 9 | 12:13 |
| lyarwood | sorry back | 12:20 |
| lyarwood | just noticed I missed stephenfin 's thread about this on the ML | 12:20 |
| * lyarwood reads now | 12:20 | |
| * gibi replies to the recent posts of that tread | 12:28 | |
| gibi | thread even | 12:30 |
| artom | Huh, looks like at some point between... victoria? and master ovs+hybrid_plug became plug-time Neutron events... | 14:19 |
| artom | https://zuul.opendev.org/t/openstack/build/2f75f41da4df48f8a8abc5087fd29efb/log/controller/logs/screen-n-cpu.txt#9233 | 14:20 |
| tobias-urdin | lyarwood: should we ping in somebody from RDO on the above topic or did u take it already? | 14:23 |
| lyarwood | tobias-urdin: Looks like folks from RDO already raised this on the ML, I hadn't seen it until the end of the discussion earlier | 14:23 |
| tobias-urdin | lyarwood: ack, good to hear | 14:30 |
| sean-k-mooney | artom: ovs hybrid plug was always a plugtime event | 14:36 |
| artom | sean-k-mooney, not for revert resize | 14:36 |
| artom | Remember 7a7a223602ca5aa0aca8f65a6ab143f1d8f8ec1b? | 14:36 |
| sean-k-mooney | for revert its already plugged | 14:36 |
| artom | Looks like we don't need it anymore | 14:36 |
| sean-k-mooney | so its bind time | 14:36 |
| artom | Apparently that's no longer the case, see the log link | 14:37 |
| artom | It's from https://review.opendev.org/c/openstack/nova/+/819349 | 14:37 |
| sean-k-mooney | for resize we dont touch the netwroking on the souce host until you confrim | 14:38 |
| sean-k-mooney | we do update the port binding (there is only one) before we go to resize verify | 14:38 |
| sean-k-mooney | then on revert we update the host back to the soruce | 14:38 |
| artom | Well I have no idea what happened, but we now get plug-time events, and not bind-time events | 14:38 |
| sean-k-mooney | but we should not need to set up the networking on the host again | 14:39 |
| artom | I mean, I don't disagree | 14:39 |
| artom | But based on what my CI job run is telling me, revert resize is now plug-time like everything else (again) | 14:39 |
| sean-k-mooney | we are going to get plug time events as well i think | 14:39 |
| sean-k-mooney | but we should get bind time events first | 14:40 |
| artom | Never get the former | 14:40 |
| sean-k-mooney | we will call effectivly hard reboot as part of starting the instance again on the source | 14:40 |
| sean-k-mooney | that is where the plug time event comes form | 14:40 |
| sean-k-mooney | at least that is what im guessing | 14:40 |
| sean-k-mooney | have not looked at that code regently how are we starting the souce vm | 14:41 |
| artom | Tbh I'm sick of the whole thing, want to get CI coverage on it so when it changes again under us we at least notice it, and revert 7a7a223602ca5aa0aca8f65a6ab143f1d8f8ec1b because we apparently don't need it anymore | 14:42 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L11238-L11240 | 14:42 |
| sean-k-mooney | artom: im not sure we should | 14:43 |
| sean-k-mooney | revert 7a7a223602ca5aa0aca8f65a6ab143f1d8f8ec1b | 14:43 |
| sean-k-mooney | to me this is a neutron regresssion | 14:43 |
| sean-k-mooney | its a bug that it is not sendign a event at bind time when we change the host_id | 14:44 |
| artom | sean-k-mooney, hellz yeah, less work for us | 14:48 |
| artom | :P | 14:48 |
| sean-k-mooney | artom: so ya https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7262 we call plug vifs again when starting the ports on the source but that is idempotent and will only update the network backend (ovs) if the ports are not present | 14:48 |
| sean-k-mooney | artom: i suspect the issue is the l2 agent recice the port update event but does not set the port status as up because its already up and therefor does not trigger the event | 14:49 |
| artom | sean-k-mooney, huh, so I can actually hand this over to #neutron via the downstream CIX call, as originally it was a CIX | 14:52 |
| sean-k-mooney | it might be related to this logic https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L2696-L2699 | 14:54 |
| sean-k-mooney | that looks wrong to me as it will filter out host chagnes on the port | 14:54 |
| artom | sean-k-mooney, we can try a quick DNM change to verify that... what should we change it to? | 14:57 |
| sean-k-mooney | give me a minute to trace teh code | 14:58 |
| sean-k-mooney | i can push something but i feel like its ignoring the host chage or somehting like that since the port and flows hould alreuady be configured | 14:58 |
| sean-k-mooney | eighter that or its on the other side where setting the port status up is skipped if the port is up | 14:59 |
| sean-k-mooney | eiter case woudl lead to the provisioning blocks code not beeing called | 14:59 |
| sean-k-mooney | artom: we are not seeing this logged right https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py#L2069 | 15:02 |
| artom | sean-k-mooney, lemme check | 15:09 |
| sean-k-mooney | this i think is where the update shoudl come from https://github.com/openstack/neutron/blob/4b0a225e8011c904dfa7c4e41fac13cea0aa872b/neutron/plugins/ml2/rpc.py#L303-L327 | 15:11 |
| artom | (Sorry, teaching daughter albegra at the same time, aah COVID how we love thee) | 15:11 |
| artom | Though it is fun to figure out a way to make them understand, and see them get it | 15:12 |
| artom | They're essentially exploring the commutativity of division | 15:12 |
| artom | Without those terms, obviously, grade 2 :P | 15:12 |
| sean-k-mooney | i see | 15:13 |
| artom | So 14/2 = 10/2 + 5/2 | 15:13 |
| artom | For example | 15:13 |
| artom | Aaaaanyways | 15:13 |
| sean-k-mooney | ya out side of an acadmic context they are not really used | 15:14 |
| sean-k-mooney | well as written that should be != | 15:14 |
| sean-k-mooney | unless we are talking about integer aritmatic | 15:14 |
| sean-k-mooney | sicne 7 != 7.5 | 15:15 |
| artom | Err, 4/2 | 15:15 |
| artom | That was a typo :P | 15:15 |
| sean-k-mooney | :) | 15:15 |
| stephenfin | gmann: lyarwood: gibi: Looks like I opened an somewhat unrelated can of worms by bringing up that question about python_requires /o\ | 15:15 |
| stephenfin | reading back through your chat from earlier (been fighting with local auth issues all afternoon) | 15:16 |
| artom | sean-k-mooney, but to answer your question, no we do not see 'changing status to down' logged for our port that's resize reverting | 15:17 |
| sean-k-mooney | in the l2 agent | 15:18 |
| sean-k-mooney | ok | 15:18 |
| sean-k-mooney | do you have the link to the l2 agent log and the port uuid | 15:18 |
| sean-k-mooney | we shoudl see if we get the notificaiton for the put update with the host change | 15:18 |
| artom | sean-k-mooney, uh... wanna hop on a gmeet? I have the logs from the job run locally | 15:19 |
| artom | 'cuz half of those words I didn't understand :P | 15:19 |
| sean-k-mooney | sure | 15:20 |
| sean-k-mooney | also i wonder if we are hitting https://github.com/openstack/neutron/blob/1ad9ca56b07ffdc9f7e0bc6a62af61961b9128eb/neutron/plugins/ml2/drivers/agent/_common_agent.py#L302-L311 | 15:20 |
| sean-k-mooney | from _process_device_if_exists | 15:20 |
| sean-k-mooney | meet.google.com/oen-kxyp-cwa when it suits | 15:23 |
| gmann | gibi: bauzas on RBAC BP, ack. I will try to join next meeting but might not due to hospital appointment(let's see how long it takes me to drive in snow :)) but dansmith can answer the queries on that if he joining. | 15:48 |
| bauzas | ack | 15:48 |
| gmann | stephenfin: yeah :). I am not sure why people did not brought those py3.6 which we discussed since PTG and re-re-updated the testing runtime - https://review.opendev.org/c/openstack/governance/+/815851/3..6/reference/runtimes/yoga.rst | 15:49 |
| opendevreview | Merged openstack/nova stable/wallaby: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/818519 | 15:50 |
| gibi | stephenfin, gmann: I think it is a good discussion on the ML, it is probably a late discussion but a good one :) | 15:52 |
| gmann | gibi: IMO, it is not late but revised one :) I have kept centos-stream 8 in my updating the testing runtime because it was not clear for TC in PTG if centos-stream 9 is released or not and later in patch review one of TC checked with centos team and confirmed it is fine to move to centos stream9 | 15:55 |
| gmann | or different opinion from centos team may be | 15:55 |
| gibi | I think there is some missunderstanding / miscommunication around the centos 9 readyness | 15:56 |
| gmann | yeah | 15:57 |
| gmann | anyways let's see if we end up reverting then we can revert the changes inclduing https://review.opendev.org/c/openstack/nova/+/819415 | 15:57 |
| gmann | bauzas: I am +2 on lyarwood changing python_requires = >=3.8 just leaving +A in case you are ok - https://review.opendev.org/c/openstack/nova/+/819415 | 15:58 |
| lyarwood | do we want to see what happens with the thread first | 15:58 |
| lyarwood | in case the TC need to revert back to py36 and CS8 | 15:59 |
| bauzas | gmann: lyarwood: yeah we can hold | 15:59 |
| gmann | lyarwood: either is ok but if we revert to py36 then anyways we have to revert 819194 too | 15:59 |
| lyarwood | true | 15:59 |
| lyarwood | FWIW that depends on https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/819431 that I'm less sure about | 15:59 |
| lyarwood | I think the branches need to be listed there for this to work | 15:59 |
| lyarwood | and it obviously has an impact on other projects | 16:00 |
| lyarwood | let me -W the nova change for now | 16:00 |
| lyarwood | and that change | 16:00 |
| gibi | I agree ^^ | 16:00 |
| gibi | lets wait a bit | 16:00 |
| gibi | and try not to land things that breask py3.6 support in the meantime :) | 16:00 |
| gibi | breaks | 16:00 |
| bauzas | ++ | 16:01 |
| gmann | sure, I have -W on tempest one also | 16:01 |
| lyarwood | thanks | 16:02 |
| gmann | lyarwood: gibi bauzas stephenfin this is the proposal on Yoga testing runtime we agreed in TC channel with RDO team members, http://lists.openstack.org/pipermail/openstack-discuss/2021-November/026024.html | 17:28 |
| gmann | I will keep this proposal as open for next week before we conclude it as most of USA and other folks are on holiday today. meanwhile please add your opinion there | 17:29 |
| gibi | gmann: thanks I will check it | 17:42 |
| lyarwood | gmann: ack will do over the weekend, thanks! | 17:50 |
| opendevreview | Artom Lifshitz proposed openstack/nova master: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/819494 | 20:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!