| opendevreview | melanie witt proposed openstack/placement master: Add reproducer for Project creation race bug https://review.opendev.org/c/openstack/placement/+/806730 | 02:50 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/placement master: Narrow scope of set allocations database transaction https://review.opendev.org/c/openstack/placement/+/807014 | 02:50 |
| *** akekane__ is now known as abhishekk | 05:54 | |
| opendevreview | Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928 | 06:14 |
| opendevreview | MaAoyu proposed openstack/nova master: swap multiattach volume, report an error VolumeNotFound https://review.opendev.org/c/openstack/nova/+/807025 | 06:21 |
| gibi | o/ | 07:54 |
| gibi | the Asia friendly nova meeting starts in 5 minutes here in the channel | 07:55 |
| gibi | #startmeeting nova_extra | 08:00 |
| opendevmeet | Meeting started Thu Sep 2 08:00:38 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:00 |
| opendevmeet | The meeting name has been set to 'nova_extra' | 08:00 |
| gibi | who we have here to day for the meeting? | 08:00 |
| gibi | nobody? | 08:05 |
| gibi | not even bauzas? ;) | 08:05 |
| * gibi feels alone | 08:05 | |
| lyarwood | \o | 08:07 |
| lyarwood | sorry | 08:07 |
| lyarwood | has anyone from Asia actually joined this meeting recently? | 08:07 |
| gibi | lyarwood: thanks, it is better now :) | 08:07 |
| gibi | lyarwood: no. on the first occasion there was somebody telling us that the devs in his/her team will join next time | 08:08 |
| gibi | then nothing | 08:08 |
| lyarwood | ^_^ | 08:08 |
| gibi | so it is a huuuge success | 08:08 |
| lyarwood | something to discuss at the PTG | 08:08 |
| gibi | adding a topic ... | 08:08 |
| bauzas | \o | 08:09 |
| * bauzas waves late as his daughter was going for the first time to high school :cry: | 08:09 | |
| gibi | ohh | 08:09 |
| gibi | I accept your excuse :) | 08:10 |
| gibi | OK, I'm closing this | 08:12 |
| gibi | #endmeeting | 08:12 |
| opendevmeet | Meeting ended Thu Sep 2 08:12:14 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 08:12 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-09-02-08.00.html | 08:12 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-09-02-08.00.txt | 08:12 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova_extra/2021/nova_extra.2021-09-02-08.00.log.html | 08:12 |
| bauzas | yeah, actually I made a mistake, this is middle school | 08:12 |
| lyarwood | did you drop her at the wrong school?! ;D | 08:13 |
| bauzas | lyarwood: man, don't make me wish Bonaparte ruled your country or we could have had the same teaching system | 08:14 |
| lyarwood | we actually have middle schools in parts of the country, I don't recall why tbh | 08:14 |
| bauzas | we call it "college" but weirdly enough, your colleges are for post-graduate kids | 08:15 |
| lyarwood | but not in the area I grew up in | 08:15 |
| lyarwood | yeah college here is 16-18 | 08:15 |
| lyarwood | then unit | 08:15 |
| lyarwood | uni* | 08:15 |
| gibi | it is complicated | 08:16 |
| bauzas | anyway, she's changing from a classic single teacher with one program to multiple teachers with different programs and options | 08:16 |
| lyarwood | yeah that's a big step, hope it goes well :) | 08:16 |
| bauzas | we'll see :p | 08:17 |
| bauzas | (and fwiw, I was kidding for Bonaparte, I'm maybe one of the few French who consider it as a dictator) | 08:17 |
| lyarwood | I always berate my family about my first day at high school, thanks to my surname I was the last kid to be called forward out of ~250 in the year, honestly thought I was at the wrong school. | 08:18 |
| lyarwood | Final name in the final set (class), horrid way to start lol. | 08:19 |
| bauzas | hah | 08:20 |
| bauzas | I alwayts hated my surname because I was having no extra time for arriving compared to my other friends | 08:20 |
| lyarwood | ^_^ | 08:21 |
| lyarwood | Yeah I guess that part was always useful | 08:21 |
| kashyap | lyarwood: That's quite some etched memory about your first day at school :) | 08:22 |
| lyarwood | s/etched/chiseled/g | 08:22 |
| kashyap | Haha | 08:22 |
| gibi | being at the end of the name list would have other advanteges at least here. We had a big book with the stundents in the class ordered by surname containing basic information and the grades and marks for the student. And teachers tend to use this book to randomly generate a student for oral exam / quiz at the beginning of a class. But humans are bad at randomly opening book... | 08:32 |
| gibi | people in the middle of the book got selected a lot more than the first and the last person | 08:33 |
| gibi | and here in Hungary the primary and middle school are basically a single school, then we have high scool and then university | 08:35 |
| opendevreview | melanie witt proposed openstack/placement master: Add reproducer for Project creation race bug https://review.opendev.org/c/openstack/placement/+/806730 | 08:45 |
| opendevreview | melanie witt proposed openstack/placement master: Narrow scope of set allocations database transaction https://review.opendev.org/c/openstack/placement/+/807014 | 08:45 |
| melwitt | gibi: sorry for updating your test ^, I got carried away. feel free to revert it if desired. also not sure the fix approach is the best one but I couldn't think of anything better | 08:48 |
| gibi | add a topic on the ptg retrospective about the asia friendly meeting slot https://etherpad.opendev.org/p/nova-yoga-ptg L60 now | 08:48 |
| gibi | melwitt: ack, I will check | 08:48 |
| gibi | thanks for working on it | 08:49 |
| melwitt | tl;dr is I removed the helper method that was being called and thought "oh maybe we can test this higher up the stack" and then I got into the weeds | 08:49 |
| melwitt | no thanks needed. I'm sorry consumer types has been a pain | 08:50 |
| gibi | melwitt: don't be hard on yourself | 09:08 |
| bauzas | sean-k-mooney: just in case you haven't seen my -1 for a doc nit https://review.opendev.org/c/openstack/nova/+/792362/6 | 09:15 |
| stephenfin | lyarwood: Can we get a release note for this? https://review.opendev.org/c/openstack/python-openstackclient/+/804377 | 09:27 |
| lyarwood | ack I'll respin shortly | 09:29 |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tests: Add missing 'nova update' unit tests https://review.opendev.org/c/openstack/python-novaclient/+/806916 | 10:02 |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Microversion 2.90 - Configurable hostnames https://review.opendev.org/c/openstack/python-novaclient/+/806917 | 10:02 |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: trivial: Remove use of kwargs https://review.opendev.org/c/openstack/python-novaclient/+/806918 | 10:02 |
| stephenfin | gibi: lyarwood: Could we merge this ancient release notes patch? I still think the information contained within is useful https://review.opendev.org/c/openstack/nova/+/705667 | 10:09 |
| gibi | stephenfin: is it impacting an old release's reno? | 10:15 |
| gibi | does the tooling handle that properly or it will show up in our Xena renos? | 10:15 |
| stephenfin | nope, it'll just update the previous release note | 10:16 |
| gibi | cool | 10:16 |
| gibi | then I will +2 shortly | 10:16 |
| stephenfin | (the release note's "location" is determined by the initial commit) | 10:16 |
| gibi | gthiemonge: hi! mellwit proposed a fix for the placement issue you reported on Tuesday https://review.opendev.org/c/openstack/placement/+/807014 Would it be possible to trigger octavia jobs with a commit that depends-on the fix? | 10:19 |
| gthiemonge | gibi: oh cool! yes, I'll trigger it! | 10:20 |
| gibi | gthiemonge: let us know how it looks | 10:21 |
| gibi | and thanks | 10:21 |
| stephenfin | gibi: Oh, I also need eyes on https://review.opendev.org/c/openstack/nova/+/705666/ too, since it's below the release note change, please + thank you :) | 10:22 |
| stephenfin | I have a series to remove all that crud but I haven't rebased it in a while :-( | 10:22 |
| gthiemonge | gibi: melwitt: thanks | 10:22 |
| gibi | stephenfin: looking | 10:22 |
| stephenfin | ty | 10:24 |
| * gibi goes grab lunch and then look at the open client patches | 10:25 | |
| lyarwood | stephenfin: I'll have to look this afternoon sorry, getting sucked in by a downstream thing | 10:25 |
| opendevreview | sean mooney proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | 10:37 |
| opendevreview | sean mooney proposed openstack/nova master: Move 'hw:pmu', 'hw_pmu' parsing to nova.virt.hardware https://review.opendev.org/c/openstack/nova/+/792364 | 10:37 |
| sean-k-mooney | bauzas: done ^ | 10:37 |
| opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Rework how we check for extensions https://review.opendev.org/c/openstack/nova/+/705792 | 11:11 |
| opendevreview | Stephen Finucane proposed openstack/nova master: neutron: Remove references to dead 'qos-queue' extension https://review.opendev.org/c/openstack/nova/+/754448 | 11:11 |
| stephenfin | sean-k-mooney: Left comments on that patch ^ fwict, the changes you've made aren't complete. I'd suggest reverting back to the previous version and instead inserting a preceding patch to change the base exception for FlavorImageConflict instead | 11:24 |
| stephenfin | I can do that if you're swamped, given today is feature freeze (?) | 11:25 |
| sean-k-mooney | stephenfin: if you want to go for it | 11:29 |
| sean-k-mooney | i havent fully pasrced what you wrote | 11:30 |
| sean-k-mooney | are you concerned about the other places you have used that | 11:30 |
| sean-k-mooney | i noticed you did not add that excption in the first patch | 11:30 |
| sean-k-mooney | unless i missed it | 11:30 |
| stephenfin | I'm concerned about the other places _others_ have used it | 11:31 |
| sean-k-mooney | ah ok ya i tought about that but didnt have a chance to look | 11:31 |
| sean-k-mooney | stephenfin: you could also just make that a follow up change that change the base code | 11:32 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add 'hw:vif_multiqueue_enabled' flavor extra spec https://review.opendev.org/c/openstack/nova/+/792356 | 11:48 |
| opendevreview | Stephen Finucane proposed openstack/nova master: docs: Document virtio-net multiqueue https://review.opendev.org/c/openstack/nova/+/792362 | 11:48 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Move 'hw:pmu', 'hw_pmu' parsing to nova.virt.hardware https://review.opendev.org/c/openstack/nova/+/792364 | 11:48 |
| stephenfin | sean-k-mooney: Okay, new comments left. I've respin it to drop that code setting on the back of said comments. Let me know if I missed anything obvious | 11:48 |
| sean-k-mooney | stephenfin: so https://github.com/openstack/nova/blob/75f719e0/nova/api/openstack/compute/servers.py#L854-L855 will still change the code form a 403 to a 400 right | 11:48 |
| sean-k-mooney | so that is still not correct? | 11:49 |
| stephenfin | It will result in a HTTP 400. Where are you getting HTTP 403 from? | 11:49 |
| sean-k-mooney | the orginial expciton inherited form forbiden | 11:49 |
| sean-k-mooney | which returned a 403 | 11:49 |
| stephenfin | which original exception? | 11:49 |
| sean-k-mooney | this one | 11:51 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/792364/8/nova/exception.py | 11:51 |
| sean-k-mooney | in the followup | 11:51 |
| stephenfin | sean-k-mooney: that's a different review | 11:51 |
| sean-k-mooney | yes | 11:51 |
| sean-k-mooney | hum | 11:51 |
| stephenfin | ohhhhh | 11:51 |
| sean-k-mooney | ok its also in INVALID_FLAVOR_IMAGE_EXCEPTIONS | 11:51 |
| sean-k-mooney | so it was getting converted too | 11:52 |
| sean-k-mooney | ok it should be good | 11:52 |
| stephenfin | so because we were introducing this new helper in that change, you suggested changing it there | 11:52 |
| stephenfin | that was not at all obvious :-D context, lad | 11:52 |
| stephenfin | anyway, yeah, it was all getting translated to HTTP 400 in the end | 11:52 |
| stephenfin | and even if it wasn't, this would be an okay change IMO | 11:52 |
| sean-k-mooney | to a 400 | 11:53 |
| sean-k-mooney | yes to a 500 no | 11:53 |
| stephenfin | yup, fair | 11:53 |
| sean-k-mooney | anyway ok it should be good again | 11:53 |
| stephenfin | yup, should be | 11:53 |
| sean-k-mooney | and no i was chaning the code in hte bottom change becasue i tought it was not being caught there | 11:54 |
| stephenfin | gotcha | 11:55 |
| sean-k-mooney | we have a top level excption handeler above that that catuche anything that inherits form NovaException | 11:55 |
| sean-k-mooney | that is not caught else whwere i tought we were going all the way to that | 11:56 |
| stephenfin | The whole nova.exception module needs a rework to either double down on exception inheritance or remove it entirely. It's all very confusing. That rework would be seriously tedious though and I doubt anyone sane will want to do it :) | 11:56 |
| sean-k-mooney | ya espcially since we marshal between them in a few places too | 11:57 |
| stephenfin | yup | 11:57 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add --sleep option for archive_deleted_rows --until-complete https://review.opendev.org/c/openstack/nova/+/771749 | 11:57 |
| kashyap | sean-k-mooney: Heya, are you able to tell what's off with this negative test (only remaining failure from the CI for me) - It passed when I posted the patch locally; https://cpaste.org/?e49aa4ae738ca284#5xdtN3VCgvzp3P5fFPXDmYJ8gYxVRQvuGaKyfxsRjYfb | 12:10 |
| sean-k-mooney | do you have a link to the review | 12:11 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/794796? | 12:12 |
| kashyap | Yep | 12:12 |
| kashyap | Sorry, was just about to fetch it | 12:12 |
| sean-k-mooney | simple | 12:12 |
| sean-k-mooney | 1.30 is not less then 1.30 https://review.opendev.org/c/openstack/nova/+/794796/3/nova/objects/image_meta.py#190 | 12:12 |
| kashyap | (I fixed the trivial notification object part; so ignore that test, pls) | 12:12 |
| sean-k-mooney | so self.assertRaises(exception.ObjectActionError, | 12:12 |
| sean-k-mooney | obj.obj_to_primitive, '1.30') | 12:13 |
| sean-k-mooney | shoudl ne '1.29' | 12:13 |
| kashyap | sean-k-mooney: Urgh, <embarassed> :D | 12:13 |
| kashyap | sean-k-mooney: Thanks! | 12:14 |
| sean-k-mooney | There are two hard things in computer science: cache invalidation, naming things, and off-by-one errors. | 12:15 |
| kashyap | sean-k-mooney: LOL | 12:15 |
| opendevreview | Merged openstack/nova master: workarounds: Remove rbd_volume_local_attach https://review.opendev.org/c/openstack/nova/+/805648 | 12:16 |
| opendevreview | Federico Ressi proposed openstack/nova master: [WIP] Debug APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | 12:18 |
| opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | 12:18 |
| opendevreview | Federico Ressi proposed openstack/nova master: [WIP] Check Nova project changes with tobiko scenario test cases https://review.opendev.org/c/openstack/nova/+/806853 | 12:27 |
| opendevreview | Federico Ressi proposed openstack/nova master: Check Nova project changes with Tobiko scenario test cases https://review.opendev.org/c/openstack/nova/+/806853 | 12:31 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: Allow 'bochs' as a display device option https://review.opendev.org/c/openstack/nova/+/794796 | 12:40 |
| gibi | sean-k-mooney: :D | 12:51 |
| opendevreview | Takashi Natsume proposed openstack/nova master: Update compute rpc version alias for xena https://review.opendev.org/c/openstack/nova/+/807137 | 13:13 |
| kashyap | sean-k-mooney: Oh, duh. In my revision I missed the extra test that stephenfin asked above. Adding that too | 13:20 |
| opendevreview | Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928 | 13:50 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: Allow 'bochs' as a display device option https://review.opendev.org/c/openstack/nova/+/794796 | 14:00 |
| kashyap | stephenfin: Fixed --^ That should address all you asked for. Let me know if I missed anything. And thanks for the review. | 14:00 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Reproduce 404 when allocation queried with 1.38 https://review.opendev.org/c/openstack/placement/+/807154 | 14:04 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Fix adding 'unknown' to the ConsumerTypeCache https://review.opendev.org/c/openstack/placement/+/807155 | 14:04 |
| gibi | lyarwood, stephenfin, bauzas: can I get a second set of eyes on https://review.opendev.org/c/openstack/osc-placement/+/804458 (consumer_types osc-placement client support) | 14:14 |
| bauzas | gibi: doing it now | 14:15 |
| gibi | thanks you | 14:17 |
| kashyap | gibi: WHen you get 20 minutes, mind have a look at this? - https://review.opendev.org/c/openstack/nova/+/763928 | 14:24 |
| opendevreview | Merged openstack/osc-placement master: Add support for microversion 1.38 consumer types https://review.opendev.org/c/openstack/osc-placement/+/804458 | 14:31 |
| gibi | kashyap: are you OK with that patch? | 14:49 |
| kashyap | gibi: 80%; I just want to do a real migration test w/ that patch - that gives me more peace of mind. (I wanted to split it up for better reviewability but just couldn't get around to it.) | 15:01 |
| kashyap | gibi: How about this: | 15:02 |
| gibi | kashyap: I have no problem with the patch, as far as see it is specialized to arm64 so we cannot broke our main deployers with it :) | 15:03 |
| kashyap | gibi: I'll setup a local env to do a bunch more tests on KVM (our CI tests only QEMU) | 15:03 |
| kashyap | gibi: Oh, wait | 15:03 |
| kashyap | gibi: I gave you the wrong patch; damn | 15:03 |
| gibi | or you linked me something else :D | 15:03 |
| kashyap | gibi: Yeah, grrrr me. | 15:03 |
| gibi | no worries | 15:03 |
| kashyap | I meant this chunky one (hence the "20 minutes") - https://review.opendev.org/c/openstack/nova/+/763928 | 15:03 |
| kashyap | Feel free to say: "no, that is too much review at this point". | 15:04 |
| kashyap | But I can answer any questions about that patch. | 15:04 |
| kashyap | gibi: Incidentally, the commit message from the patch author is a verbatim copy/paste of what I wrote in one of the reviews :D | 15:04 |
| gibi | :) | 15:05 |
| gibi | you linked the wrong patch again https://review.opendev.org/c/openstack/nova/+/763928 | 15:05 |
| kashyap | gibi: The short summary is - this patch will be replacing the two older libvirt CPU APIs that are now causing more problems in the new world of hardware CVE flaws | 15:05 |
| kashyap | gibi: Gah, sorry, this one - https://review.opendev.org/c/openstack/nova/+/762330/ | 15:06 |
| kashyap | "libvirt: Use improved guest CPU config APIs" | 15:06 |
| gibi | ok that is chunky indeed :) | 15:06 |
| kashyap | gibi: It's a chunky one. Actually, this hour is not good to review it. And will only make you grumpy, I'm afraid | 15:06 |
| gibi | I will try and lets see how far I can get | 15:07 |
| kashyap | gibi: I can split it up, if you like. (That's also the right thing in my view, and told that on the review a few times) | 15:07 |
| kashyap | (And also in the original spec I wrote) | 15:07 |
| kashyap | gibi: I'll copy/paste from my notes, the "outline" of the patch: | 15:09 |
| kashyap | - libvirt: Introduce compare_hypervisor_cpu() method | 15:09 |
| kashyap | - libvirt: Introduce baseline_hypervisor_cpu() method | 15:09 |
| kashyap | - Rework _get_guest_cpu_model_config() method to take advantage of the new CPU APIs | 15:09 |
| kashyap | - Update the get_capabilities() method to make use of baseline_hypervisor_cpu() | 15:09 |
| kashyap | - Update the check_can_live_migrate_destination() method to use compare_hypervisor_cpu() | 15:09 |
| gibi | thanks | 15:10 |
| melwitt | gibi: thank you for reviewing my patch. I'm going to try a simpler fix and try to restore the original func test. if it works, I think reviewers might like it more | 15:38 |
| bauzas | melwitt: you're on my plate | 15:38 |
| bauzas | but I've seen gibi's concerns so I held a bit | 15:39 |
| melwitt | bauzas: yes thank you for your review too, I'm going to try a simpler approach bc as you said the current patch looks pretty hacky | 15:39 |
| gibi | melwitt: I have no problem with the logic of the current fix. Just had some nits about docs | 15:40 |
| gibi | melwitt: but sure, if you have a simpler solution then I'm happy to review that too later | 15:40 |
| melwitt | gibi: understood. I worry that it's too confusing | 15:40 |
| melwitt | (for maintainability) | 15:41 |
| gibi | OK | 15:41 |
| gibi | btw, there is another small fix targeting consumer_types here https://review.opendev.org/c/openstack/placement/+/807155 | 15:42 |
| gibi | but I think there will not be conflicts | 15:42 |
| melwitt | gibi: thanks for figuring that out, I will review | 15:43 |
| gibi | no problem | 15:44 |
| opendevreview | Merged openstack/python-novaclient master: tests: Add missing 'nova update' unit tests https://review.opendev.org/c/openstack/python-novaclient/+/806916 | 15:44 |
| opendevreview | Merged openstack/python-novaclient master: Microversion 2.90 - Configurable hostnames https://review.opendev.org/c/openstack/python-novaclient/+/806917 | 15:50 |
| gibi | kashyap: timed out on reading https://review.opendev.org/c/openstack/nova/+/762330, I left some comments / questions inline. | 16:06 |
| gibi | but I haven't read the driver part yet | 16:06 |
| * gibi goes updating python-novaclient release patch and the cyclehighlights | 16:07 | |
| gibi | all - please review the cycle highlights and lt me know if anything is wrong or missing https://review.opendev.org/c/openstack/releases/+/800755 | 16:29 |
| gibi | ending my day. see you tomorrow o/ | 16:30 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Allow 'bochs' as a display device option https://review.opendev.org/c/openstack/nova/+/794796 | 16:46 |
| sean-k-mooney | ok i think im going to call it a day o/ | 17:59 |
| opendevreview | Merged openstack/nova master: Add two new hacking rules https://review.opendev.org/c/openstack/nova/+/805668 | 19:03 |
| opendevreview | Merged openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | 19:03 |
| opendevreview | Merged openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | 19:04 |
| opendevreview | Merged openstack/nova master: Update compute rpc version alias for xena https://review.opendev.org/c/openstack/nova/+/807137 | 20:15 |
| opendevreview | melanie witt proposed openstack/placement master: Add reproducer for Project creation race bug https://review.opendev.org/c/openstack/placement/+/806730 | 20:18 |
| opendevreview | melanie witt proposed openstack/placement master: Narrow scope of set allocations database transaction https://review.opendev.org/c/openstack/placement/+/807014 | 20:18 |
| melwitt | gibi, bauzas: I rewrote the func tests and gibi I think I addressed all of your comments on the fix patch. I decided to just stay with the current approach, keeping the transaction as small as possible is likely most important | 20:24 |
| *** bnemec is now known as bnemec-pto | 21:45 | |
| opendevreview | melanie witt proposed openstack/placement master: Narrow scope of set allocations database transaction https://review.opendev.org/c/openstack/placement/+/807014 | 22:10 |
| opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | 22:18 |
| opendevreview | Federico Ressi proposed openstack/nova master: Check Nova project changes with Tobiko scenario test cases https://review.opendev.org/c/openstack/nova/+/806853 | 22:21 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!