| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: wait for live migration vif plug short timeout https://review.openstack.org/590585 | 00:15 |
|---|---|---|
| *** mriedem has quit IRC | 00:52 | |
| *** Nel1x has joined #openstack-placement | 01:13 | |
| *** alex_xu has joined #openstack-placement | 01:28 | |
| *** fanzhang has left #openstack-placement | 01:44 | |
| openstackgerrit | Merged openstack/nova master: Update compute rpc version alias for rocky https://review.openstack.org/589972 | 02:05 |
| *** e0ne has joined #openstack-placement | 02:16 | |
| *** e0ne has quit IRC | 02:18 | |
| *** Nel1x has quit IRC | 03:04 | |
| openstackgerrit | Chen proposed openstack/nova stable/ocata: Fix host validity check for live-migration https://review.openstack.org/590611 | 03:06 |
| openstackgerrit | Chen proposed openstack/nova stable/ocata: Fix host validity check for live-migration https://review.openstack.org/590611 | 03:10 |
| openstackgerrit | Chen proposed openstack/nova master: Add additional info to resource provider aggregates update API https://review.openstack.org/590243 | 03:21 |
| openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081 | 05:04 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova stable/ocata: Return 400 when compute host is not found https://review.openstack.org/590649 | 05:50 |
| *** nicolasbock has joined #openstack-placement | 05:50 | |
| openstackgerrit | huanhongda proposed openstack/nova master: Destroy evacuated instance while unset nova-compute forced_down https://review.openstack.org/587807 | 05:56 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590439 | 07:18 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590439 | 07:20 |
| *** e0ne has joined #openstack-placement | 07:59 | |
| openstackgerrit | huanhongda proposed openstack/nova master: Destroy evacuated instance while unset nova-compute forced_down https://review.openstack.org/587807 | 08:41 |
| *** e0ne has quit IRC | 08:49 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove patching the mock lib https://review.openstack.org/590709 | 08:59 |
| *** e0ne has joined #openstack-placement | 09:14 | |
| *** tssurya has joined #openstack-placement | 09:20 | |
| *** e0ne has quit IRC | 09:27 | |
| *** cdent has joined #openstack-placement | 09:31 | |
| *** e0ne has joined #openstack-placement | 09:37 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590730 | 09:53 |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/590731 | 09:53 |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova master: Update reno for stable/rocky https://review.openstack.org/590732 | 09:54 |
| *** gibi is now known as giblet | 09:56 | |
| cdent | ooops. just push load average up to 2133.42 | 09:57 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP scheduler: check request_spec.instance_group before updating https://review.openstack.org/590733 | 09:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: add zvm into support matrix https://review.openstack.org/532720 | 10:20 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm admin intro and hypervisor information https://review.openstack.org/533125 | 10:20 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add zvm CI information https://review.openstack.org/533512 | 10:20 |
| *** e0ne has quit IRC | 10:21 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 10:30 |
| openstackgerrit | Merged openstack/nova stable/pike: Add recreate test for RT.stats bug 1784705 https://review.openstack.org/588036 | 10:30 |
| openstack | bug 1784705 in OpenStack Compute (nova) pike "ResourceTracker.stats can leak across multiple ironic nodes" [High,In progress] https://launchpad.net/bugs/1784705 - Assigned to Matt Riedemann (mriedem) | 10:30 |
| cdent | giblet: on the generation stuff, are you finding bugs in your changes only, or also other stuff? | 10:38 |
| giblet | cdent: so far I think it is mosly in my change. But I didn't spent much time analyzing them. I'm not affraid of the baseline as all the test I'm adding basicly a copy of an existing functional test and forcing some retry | 10:41 |
| giblet | cdent: also most of the problems I see is in case when we run out of retry and fail | 10:44 |
| giblet | cdent: when the retry succeeds nova seems solid | 10:45 |
| cdent | ✔ | 10:45 |
| openstackgerrit | Chris Dent proposed openstack/nova stable/pike: [placement] Retry allocation writes server side https://review.openstack.org/590745 | 10:57 |
| *** nicolasbock has quit IRC | 11:19 | |
| *** cdent has quit IRC | 11:58 | |
| *** cdent has joined #openstack-placement | 12:01 | |
| *** mriedem has joined #openstack-placement | 12:05 | |
| *** mriedem has quit IRC | 12:17 | |
| *** giblet is now known as giblet_ofr | 12:34 | |
| *** giblet_ofr is now known as giblet_off | 12:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix GET /flavors?is_public description https://review.openstack.org/588092 | 12:49 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Move static _get_power_off_values to compute_utils https://review.openstack.org/578822 | 13:13 |
| *** jaypipes is now known as leakypipes | 13:21 | |
| *** dansmith is now known as SteelyDan | 13:31 | |
| *** efried is now known as fried_rice | 13:35 | |
| fried_rice | cdent: You got https://review.openstack.org/#/c/590445/ on your radar for a re-look today? | 13:46 |
| cdent | fried_rice: weird, I thought I already had, but I guess I got distracted | 13:48 |
| cdent | leakypipes: is https://review.openstack.org/#/c/587510/ abandonable? | 14:02 |
| fried_rice | leakypipes: rocky is branched now, wanna +W https://review.openstack.org/#/c/590389/ since giblet_off is off? | 14:03 |
| leakypipes | cdent: done. sorry | 14:04 |
| leakypipes | fried_rice: done | 14:04 |
| cdent | leakypipes: no worries, just going through things as saw it was probably dead | 14:04 |
| fried_rice | thx | 14:04 |
| cdent | did giblet_off leave instructions on the care and feeding of his 1.28 report client changes? | 14:07 |
| fried_rice | is he gone for a long time? | 14:09 |
| fried_rice | are there multiple patches? | 14:10 |
| fried_rice | okay, I see them now. | 14:10 |
| fried_rice | Doesn't look like it's close to ready yet. Is there any urgency for others to pick it up while he's gone? | 14:11 |
| *** e0ne has joined #openstack-placement | 14:11 | |
| fried_rice | cdent, leakypipes: We want https://review.openstack.org/#/c/590388/ in RC2, yah? | 14:11 |
| cdent | fried_rice: I thought there wasn't going to be an rc2. But if there is, sure. | 14:12 |
| fried_rice | Well, "do we want it in Rocky" is I guess what I'm asking. Whether we make a fuss for it being in RC2 or wait until later to backport (I'm not really sure what the difference is). | 14:13 |
| fried_rice | What about Gigantor? That seems like something we should have in Rocky, so that when people move to Rocky, they have debuggability. | 14:13 |
| fried_rice | And btw, there will be a RC2 according to the ptb | 14:13 |
| cdent | oh, last I heard there was hope for not. But if there is: yes, both should go in | 14:14 |
| cdent | people will want the log info | 14:14 |
| fried_rice | okay, agree. So we want bugs on 'em. | 14:14 |
| cdent | an internal bug just showed up with "how do we debug where this went wrong, we used to..." | 14:15 |
| fried_rice | leakypipes: Do you have local edits or can I rebase the series (to put tetsuro's test on bottom) and add bug tags? | 14:15 |
| fried_rice | cdent: internal to whom? | 14:15 |
| cdent | in my experience "internal" in this context usually means "somewhere in the dungeons of my employer" | 14:16 |
| cdent | i pointed to the ml thread and the gigantor change | 14:16 |
| fried_rice | do we have a lp bug for that yet? | 14:17 |
| * cdent doesn't know | 14:17 | |
| leakypipes | fried_rice: nope, no local edits. have at it. | 14:22 |
| fried_rice | rgr | 14:22 |
| leakypipes | cdent: you saw I posted a ML response to bnemec looking for some feedback on that right? | 14:23 |
| cdent | yessir | 14:24 |
| fried_rice | I saw that too. I was going to quibble about the wording of the log messages a little bit, but for the most part it surely seems to be in the right vein. | 14:24 |
| cdent | leakypipes: (metadata service)++ | 14:30 |
| cdent | no obvious logic flaws in your articles, good read, not anywhere near as infuriating as you promised. lots of good idea. | 14:31 |
| fried_rice | cdent: You in the mood to seek-and-if-not-present-open bugs for those two patches? | 14:36 |
| cdent | fried_rice: just got sucked into a need-to-do-now thing for tc stuff | 14:37 |
| fried_rice | ight | 14:37 |
| cdent | but if it is not done in a few hours I could get to it | 14:37 |
| *** e0ne has quit IRC | 14:39 | |
| openstackgerrit | Merged openstack/nova stable/pike: Make ResourceTracker.stats node-specific https://review.openstack.org/588037 | 14:40 |
| *** tssurya has quit IRC | 14:52 | |
| *** e0ne has joined #openstack-placement | 15:03 | |
| fried_rice | cdent: Your issue with https://review.openstack.org/#/c/590445/ is just the commit message? | 15:03 |
| *** e0ne has quit IRC | 15:10 | |
| leakypipes | cdent: you didn't see earlier, angrier versions :) | 15:18 |
| * cdent is disappoint | 15:19 | |
| leakypipes | hehe, don't be. :) | 15:19 |
| cdent | fried_rice: i'm surfacing, point me please | 15:23 |
| cdent | fried_rice: i'll take care of the message and bug for https://review.openstack.org/#/c/590445/ first | 15:24 |
| fried_rice | cdent: It sounds like that's the only semi-urgent one. The other two are going to wait until release. Though I still don't get why ^ and not those. But whatever. | 15:27 |
| *** mriedem has joined #openstack-placement | 15:30 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/590445 | 15:31 |
| openstackgerrit | Merged openstack/nova master: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590439 | 15:35 |
| openstackgerrit | Merged openstack/nova master: Nix 'new in 1.19' from 1.19 sections for rp aggs https://review.openstack.org/590389 | 15:35 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590801 | 15:37 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590803 | 15:38 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590804 | 15:39 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | 15:50 |
| mriedem | cdent: thanks for updating that commit message | 15:59 |
| cdent | you're welcome | 15:59 |
| fried_rice | cdent: Are you working on this series yet? https://review.openstack.org/#/c/590388/3 | 16:02 |
| fried_rice | was gonna pull it down an shuffle it at least. | 16:03 |
| cdent | fried_rice: no, I got the impression we were stalled out on those, so got distracted to something else. go for it if you're in the mood. | 16:03 |
| fried_rice | ight | 16:03 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 16:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 16:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 16:07 |
| fried_rice | I didn't add bug tags; ran out of time. cdent if you happen to have the time and inclination in the next couple hours... | 16:07 |
| cdent | roger | 16:08 |
| fried_rice | Not sure if the test code is worth backporting | 16:08 |
| *** fried_rice is now known as fried_rolls | 16:08 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert "libvirt: add method to configure migration speed" https://review.openstack.org/590814 | 16:23 |
| *** mriedem has quit IRC | 16:27 | |
| *** purplerbot has quit IRC | 16:38 | |
| *** e0ne has joined #openstack-placement | 17:11 | |
| *** N3l1x has joined #openstack-placement | 17:14 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 17:35 |
| openstackgerrit | Chris Dent proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 17:35 |
| openstackgerrit | Chris Dent proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 17:35 |
| leakypipes | cdent: thx for creating that bug report and mod'ing the commit messages. appreciated. | 17:37 |
| *** purplerbot has joined #openstack-placement | 17:56 | |
| openstackgerrit | Merged openstack/nova master: Update reno for stable/rocky https://review.openstack.org/590732 | 17:57 |
| openstackgerrit | Merged openstack/nova stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/590730 | 17:57 |
| openstackgerrit | Merged openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081 | 17:57 |
| cdent | leakypipes: I figured by hanging it on the debug logging rather than the performance it would seem more important, based on recent discussions | 18:01 |
| leakypipes | ack, makes sense. | 18:03 |
| openstackgerrit | Marcus Furlong proposed openstack/nova master: improve migration script https://review.openstack.org/591030 | 18:09 |
| openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for PCI devices https://review.openstack.org/591037 | 18:31 |
| openstackgerrit | Chris Dent proposed openstack/nova stable/ocata: [placement] Retry allocation writes server side https://review.openstack.org/591042 | 18:36 |
| cdent | time for dins | 18:38 |
| fried_rolls | By cdent | 18:47 |
| *** fried_rolls is now known as fried_rice | 18:47 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | 18:48 |
| *** tssurya has joined #openstack-placement | 18:51 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | 19:18 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061 | 19:25 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070 | 19:43 |
| fried_rice | leakypipes: Run that on the debug-split patch and it fails ^ | 19:49 |
| fried_rice | comments forthcoming. | 19:49 |
| *** e0ne has quit IRC | 19:51 | |
| openstackgerrit | Merged openstack/nova master: Add additional info to resource provider aggregates update API https://review.openstack.org/590243 | 19:59 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061 | 20:04 |
| leakypipes | fried_rice: ack. I shall fix forthwith. | 20:08 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 20:11 |
| openstackgerrit | Eric Fried proposed openstack/nova master: DNM: demo https://review.openstack.org/591073 | 20:11 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 20:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 20:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 20:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070 | 20:13 |
| cdent | sigh. I consistently overestimate how much our tests are testing | 20:16 |
| cdent | I need like a shock bracelet or something | 20:17 |
| * cdent will check back tomorrow | 20:17 | |
| *** cdent has quit IRC | 20:17 | |
| fried_rice | cdent: That one was pretty tricky to create | 20:17 |
| fried_rice | cdent: I had to play with it for a while. | 20:18 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 20:19 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070 | 20:19 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 20:20 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 20:20 |
| openstackgerrit | melanie witt proposed openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080 | 20:36 |
| openstackgerrit | Merged openstack/nova master: Remove patching the mock lib https://review.openstack.org/590709 | 21:00 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 21:47 |
| *** tssurya has quit IRC | 22:21 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 22:31 |
| *** leakypipes has quit IRC | 22:31 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!