| *** tetsuro has joined #openstack-placement | 00:15 | |
| openstackgerrit | Merged openstack/nova master: Add functional test for forced live migration rollback allocs https://review.openstack.org/586636 | 00:24 |
|---|---|---|
| *** fanzhang has left #openstack-placement | 01:07 | |
| openstackgerrit | Merged openstack/nova stable/ocata: [stable only] Add functional regression test for bug 1783613 https://review.openstack.org/588416 | 01:36 |
| openstack | bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt) | 01:37 |
| *** mriedem has quit IRC | 02:08 | |
| openstackgerrit | Chen proposed openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068 | 02:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590070 | 02:13 |
| openstackgerrit | Chen proposed openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068 | 02:16 |
| openstackgerrit | Chen proposed openstack/nova stable/pike: Fix bad links for admin-guide https://review.openstack.org/590072 | 02:16 |
| openstackgerrit | Chen proposed openstack/nova master: Update ssh configuration doc https://review.openstack.org/589844 | 02:39 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Update the parameter explain when updating a volume attachment https://review.openstack.org/565181 | 02:45 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: add description for 1.29 https://review.openstack.org/589407 | 02:56 |
| openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081 | 03:12 |
| openstackgerrit | Chen proposed openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028 | 03:17 |
| *** e0ne has joined #openstack-placement | 04:11 | |
| openstackgerrit | Merged openstack/nova stable/queens: Fix bad links for admin-guide https://review.openstack.org/590068 | 04:13 |
| *** e0ne has quit IRC | 04:24 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081 | 05:32 |
| *** nicolasbock has joined #openstack-placement | 05:34 | |
| *** tetsuro has quit IRC | 05:35 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081 | 06:05 |
| *** nicolasbock has quit IRC | 06:35 | |
| *** nicolasbock has joined #openstack-placement | 06:41 | |
| openstackgerrit | Sergii Golovatiuk proposed openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548 | 06:55 |
| *** tetsuro has joined #openstack-placement | 07:16 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 07:23 |
| *** tetsuro has quit IRC | 08:11 | |
| *** tssurya has joined #openstack-placement | 08:19 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | 08:27 |
| openstackgerrit | Jose Castro Leon proposed openstack/nova master: Fix get_device_path from network mounted volume https://review.openstack.org/590188 | 08:36 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | 08:49 |
| *** cdent has joined #openstack-placement | 09:08 | |
| *** fanzhang has joined #openstack-placement | 09:38 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 10:11 |
| openstackgerrit | Sergii Golovatiuk proposed openstack/nova master: libvirt: Always escape IPv6 addresses when used in migration URI https://review.openstack.org/589548 | 10:18 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 10:40 |
| openstackgerrit | Chen proposed openstack/nova master: Add additional info to resource provider aggregates update API https://review.openstack.org/590243 | 11:07 |
| openstackgerrit | Merged openstack/nova master: Docs: Add guide to migrate instance with snapshot https://review.openstack.org/584442 | 11:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Improve performance of get_allocated_disk_size https://review.openstack.org/590253 | 11:33 |
| openstackgerrit | Merged openstack/nova master: Fix host validity check for live-migration https://review.openstack.org/401009 | 11:36 |
| *** s10 has joined #openstack-placement | 11:52 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova stable/queens: Fix host validity check for live-migration https://review.openstack.org/590262 | 11:55 |
| openstackgerrit | Rajesh Tailor proposed openstack/nova stable/pike: Fix host validity check for live-migration https://review.openstack.org/590263 | 11:58 |
| openstackgerrit | Merged openstack/nova master: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/382853 | 12:06 |
| jaypipes | cdent: are you "back near your puter" yet? :) | 12:39 |
| cdent | jaypipes: yeah, just now, I had babies crawling around my back yard for the morning | 12:40 |
| jaypipes | cdent: I'm curious about the perf impact (neg, pos or none) of https://review.openstack.org/#/c/590041/ | 12:40 |
| jaypipes | cdent: ewwwww. | 12:40 |
| jaypipes | cdent: did you call an exterminator? | 12:40 |
| cdent | no, they left of their own accord | 12:42 |
| cdent | i shall spin up the machines | 12:42 |
| jaypipes | cdent: I'm scratching my head about one thing though on that patch... | 12:45 |
| *** edmondsw has joined #openstack-placement | 12:45 | |
| jaypipes | cdent: I don't see any evidence that any of the LOG.debug() messages I put into _get_provider_ids_matching() are being output. | 12:46 |
| jaypipes | cdent: I've grepped for "providers with capacity" in placement-api logs for both the tempest-full and tempest-slow jobs and found nada | 12:46 |
| cdent | hmmm | 12:46 |
| * cdent looks | 12:46 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: rewrite _get_instance_disk_info_from_config https://review.openstack.org/589567 | 12:47 |
| cdent | jaypipes: presumably request.use_same_provider is never False? | 12:52 |
| cdent | line 3945 | 12:53 |
| cdent | if it isn't, _get_provider_ids_matching is never called | 12:53 |
| jaypipes | cdent: use_same_provider is always False. http://logs.openstack.org/41/590041/1/check/tempest-full/92d93e2/controller/logs/screen-placement-api.txt.gz?level=DEBUG#_Aug_08_22_20_59_590711 | 12:55 |
| cdent | which suggests that some logic somewhere is flawed | 12:55 |
| jaypipes | cdent: we don't yet send use_same_provider=True from the scheduler, right? | 12:55 |
| jaypipes | i.e. we don't yet send granular request groups | 12:55 |
| cdent | yes, but the only way I can see for your log messages to not show up is if the _get_provide is not caled | 12:56 |
| cdent | and the way for it not to be called is for the method it is called from to return early | 12:56 |
| cdent | once I have things spun up i'll see if I can confirm/deny | 12:56 |
| jaypipes | hmmm. | 12:56 |
| jaypipes | yeah, no worries, I'm concurrently looking at it too. | 12:57 |
| jaypipes | tia | 12:57 |
| cdent | everything was powered down, so I need to devstack from scratch | 12:57 |
| jaypipes | ack | 12:57 |
| cdent | (ever since the solar panels I keep turning things off) | 12:57 |
| jaypipes | cdent: shit. https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L4013 | 12:58 |
| jaypipes | cdent: we're now *always* processing with tree crap. | 12:59 |
| cdent | that's the line I pointed out above (line 3945 in your code) | 12:59 |
| jaypipes | cdent: oh, sorry | 12:59 |
| jaypipes | back to the same issue with the use_same_provider crap | 13:00 |
| jaypipes | oh, I long for the days when these code paths were legible. | 13:00 |
| jaypipes | cdent: you remember when I had code in objects/resource_provider.py that had a function called has_provider_trees()? | 13:01 |
| jaypipes | cdent: that was used to disable all the junk around tree processing when nested stuff wasn't in use. | 13:01 |
| cdent | I can't remember anything after adding nested and allocation candidates :) | 13:01 |
| cdent | since then it is all just haze and fog | 13:02 |
| jaypipes | cdent: here's yet another place we used to only call once: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/objects/resource_provider.py#L4023-L4027 | 13:05 |
| cdent | jaypipes: so given that your code doesn't seem to be reached (have we agreeed that's the case), what shall I do? Should I still confirm? | 13:11 |
| jaypipes | cdent: no, don't bother (it won't actually run the code paths). I'm pushing a new rev shortly. | 13:15 |
| cdent | i'll do some other inspecting | 13:19 |
| *** mriedem has joined #openstack-placement | 13:24 | |
| *** edmondsw has quit IRC | 13:29 | |
| *** edmondsw has joined #openstack-placement | 13:37 | |
| cdent | edleafe, efried, jaypipes: interesting observation which may lead to fruitful exploration: | 13:39 |
| cdent | time curl -s -H 'x-auth-token: admin' -H 'openstack-api-version: placement latest' http://192.168.1.76/placement/allocation_candidates?resources=VCPU:1,MEMORY_MB:256,DISK_GB:10 >/dev/null real0m3.637s | 13:40 |
| efried | Yeah, I've noticed setting up the connection is pretty slow. | 13:40 |
| cdent | time curl -s -H 'x-auth-token: admin' -H 'openstack-api-version: placement latest' http://192.168.1.76/placement/resource_providers?resources=VCPU:1,MEMORY_MB:256,DISK_GB:10 >/dev/null real0m0.288s | 13:40 |
| efried | oh, sorry, I misread that. | 13:40 |
| cdent | now, obvs getting a list of resource providers is doing _much_ less work, but that helps distinguish where the effort is | 13:41 |
| *** edmondsw has quit IRC | 13:41 | |
| efried | Are either of yesterday's fixes worthy of backporting? | 13:42 |
| cdent | efried: as far as I was able to discern they change code that was added this cycle | 13:42 |
| cdent | prior to this cycle the surrounding code was either not there, or not using get_by_uuid | 13:42 |
| jaypipes | I should have rev pushed shortly. | 13:42 |
| efried | Well, right, that would be hard to avoid; I guess I'm asking whether the concept could be backported. | 13:43 |
| jaypipes | electrician currentl installing ceili9ng fan over my desk... | 13:43 |
| efried | Hah, "electrician" for ceiling fan installation in my house is usually "wife". | 13:43 |
| cdent | efried: I don't think there is anywhere for a backport to land | 13:43 |
| edleafe | I have personally installed two ceiling fans in my house within the last month | 13:43 |
| efried | I think edleafe and I are basically calling jaypipes a wuss for bringing in outside help to install a ceiling fan. | 13:44 |
| cdent | in the UK a mere human is not allowed to install a ceiling fan, or if they do they then have to invite a certified electrician in to validate the work | 13:44 |
| efried | Yeah, I figured it was probably an issue of "permit". jaypipes must live in a city. | 13:44 |
| edleafe | cdent: that's true in the US too. Totally unenforced in 99.9% of cases | 13:50 |
| edleafe | I take solace in the fact that I *was* licensed (many years ago) | 13:51 |
| cdent | yeah, here it becomes a huge deal during home sales or lawsuits when there is a landlord | 13:51 |
| *** sean-k-mooney has joined #openstack-placement | 13:51 | |
| cdent | we just had our wiring re-checked and ciruit board replaced, etc | 13:51 |
| cdent | and the electrician was very careful becuase if he claims something is certified and then later it goes wack, he's liable | 13:52 |
| *** takashin has joined #openstack-placement | 13:53 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/589567 | 14:01 |
| *** jaypipes has quit IRC | 14:06 | |
| *** jaypipes has joined #openstack-placement | 14:10 | |
| jaypipes | electrician turned the wrong breaker off... shut off my puters. | 14:11 |
| cdent | ouch | 14:12 |
| efried | See, shutting off the breaker is for wusses. | 14:15 |
| efried | But if you MUST shut off the breaker, you make sure to get the right one by shorting it with a screwdriver. | 14:15 |
| edleafe | efried: stop giving away all our professional secrets! | 14:20 |
| * cdent hopes we don't get in the habit of calling people wusses | 14:21 | |
| efried | it's all in fun | 14:21 |
| efried | we all have our strengths. | 14:21 |
| efried | For instance, I'll bet jaypipes can drive a golf ball approx 200x further than I can. | 14:21 |
| jaypipes | efried: you'd be surprised. I suck at golf, even though I love it. :) | 14:22 |
| jaypipes | efried: just like coding. | 14:22 |
| cdent | I'm half serious efried: plenty of people walk into a room to see people who they don't know are jesting, see "wusses" and think "these people are jerks" and leave | 14:22 |
| * efried calculates 200 x 18" | 14:22 | |
| efried | cdent: Noted, I'll tone it down. | 14:23 |
| efried | can I talk about my Teriyaki bowls? | 14:23 |
| efried | Actually, I doubt that one made it over the pond. | 14:24 |
| cdent | I don't know the reference. But we can talk about Totoro if you want. | 14:25 |
| efried | cdent: This commercial is apparently offending people <rolls eyes>: https://www.youtube.com/watch?v=s0MgMAgfCqM | 14:32 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | 14:34 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP compute: Terminate volume connections during _shutdown_instance https://review.openstack.org/590348 | 14:34 |
| cdent | i'm offended by all commercials | 14:34 |
| cdent | jaypipes: did the electrician kill your code, or was it saved? | 14:38 |
| edleafe | efried: Making "balls" jokes should probably be limited to 4th graders | 14:41 |
| efried | edleafe: Which is clearly who wrote that commercial. | 14:41 |
| jaypipes | cdent: it was saved :) | 14:42 |
| jaypipes | cdent: just testing more locally now | 14:42 |
| cdent | ✔ | 14:42 |
| efried | jaypipes: Did you notice https://review.openstack.org/#/c/590150/ on top of your patch which exposes a bug? | 14:52 |
| efried | jaypipes: As you respin, can you reverse the order? ^ passes against master. | 14:58 |
| jaypipes | efried: looking at it locally now. reproduced it. | 15:11 |
| jaypipes | efried, cdent: added some diagnostic logging to the _get_provider_ids_matching() function... pretty useful: http://paste.openstack.org/show/727744/ | 15:12 |
| cdent | jaypipes: ux suggestion: when you're reporting from withing a chunk, add some ---> or whitespace or something? | 15:15 |
| cdent | I suppose that doesn't really matter in an interleaved log | 15:15 |
| efried | jaypipes: Whenever you get a 0, you can short out, nah? | 15:15 |
| cdent | but looking at your paste that was the first thing that came to mind | 15:15 |
| cdent | but yeah | 15:15 |
| jaypipes | efried: yes | 15:15 |
| jaypipes | cdent: ya, need to prefix the lines with the request id | 15:16 |
| *** s10 has quit IRC | 15:18 | |
| jaypipes | cdent: though I'm wondering how to get said request id... hint? | 15:20 |
| cdent | jaypipes: you mean the keystone request id thingie? That comes automagically: http://logs.openstack.org/41/590041/1/check/tempest-full/92d93e2/controller/logs/screen-placement-api.txt.gz?level=DEBUG#_Aug_08_22_21_07_872101 | 15:21 |
| cdent | when the logger has a context | 15:22 |
| jaypipes | ah | 15:22 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 15:22 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 15:23 |
| jaypipes | alrighty, cdent, if you can test both of the above patches that would be super useful. :) | 15:23 |
| cdent | let's implement "use simple code paths when possible" in everything!!! | 15:23 |
| cdent | aye aye | 15:23 |
| jaypipes | danke | 15:23 |
| *** alex_xu has quit IRC | 15:24 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Nix 'new in 1.19' from 1.19 sections for rp aggs https://review.openstack.org/590389 | 15:27 |
| cdent | I feel like such a traitor working on NFV related things. They are explicitly described as ways of making telcos be able to be smart pipes and that's pretty much the opposite of good. | 15:27 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013 | 15:29 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014 | 15:29 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/587071 | 15:29 |
| *** takashin has left #openstack-placement | 15:37 | |
| cdent | jaypipes: what kind of query do I need to make to tickle your changes | 15:38 |
| jaypipes | cdent: the original queries you were running. | 15:38 |
| jaypipes | cdent: basically, what you ran yesterday. | 15:38 |
| cdent | i'm not seeing any log messages | 15:38 |
| jaypipes | cdent: you won't on the first patch. the second you will though. | 15:38 |
| jaypipes | I suppose I should flip the order of those.. | 15:39 |
| cdent | i've got both patches in place | 15:39 |
| cdent | have restarted placement | 15:39 |
| cdent | lemme me add some debugs of my own | 15:39 |
| jaypipes | hmm. | 15:39 |
| jaypipes | cdent: you don't have sharing providers in place do you? | 15:41 |
| cdent | no | 15:42 |
| cdent | gimme a sec, | 15:42 |
| *** tssurya has quit IRC | 15:48 | |
| cdent | jaypipes: your code is still not being reached because sharing_providers: {0: [], 1: [], 2: []} | 15:50 |
| cdent | use_same_provider: False has_trees: False sharing_providers: {0: [], 1: [], 2: []} | 15:50 |
| cdent | so that conditional is True and is goes down the sharing/nested route | 15:51 |
| jaypipes | cdent: guh, thx. ok, lemme push a quick rev. | 15:51 |
| jaypipes | cdent: thx for picking that out. | 15:52 |
| * cdent bows | 15:52 | |
| cdent | i had to squint pretty hard | 15:52 |
| jaypipes | :) | 15:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 15:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 15:56 |
| jaypipes | cdent: k, re-ordered the patches and fixed that up. | 15:56 |
| cdent | roger | 15:57 |
| jaypipes | cdent: would be very cool to see the diff in perf between the two patches above... the first does the "use the non-tree, non-sharing code paths if possible" thing and the second reworks that _get_provider_ids_matching() function to be more diagnostic-friendly, so would be great to see any perf regression on the second compared to the first. | 15:57 |
| jaypipes | cdent: and thx very much in advance! | 15:58 |
| cdent | ✔ | 15:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 16:04 |
| efried | rebased Tetsuro's bug trap ^ but still think we should put it underneath. | 16:05 |
| cdent | jaypipes, efried : master time on the same request we've been doing: 3.688s | 16:10 |
| cdent | with smart path: 1.109 | 16:11 |
| efried | noyce | 16:11 |
| cdent | with split sql: 1.114 | 16:11 |
| efried | noyce noyce | 16:11 |
| efried | Are we sure smart path is still hitting split sql? | 16:11 |
| cdent | important to keep in mind here that this is no traits, no aggregates, etc | 16:11 |
| cdent | but yeah | 16:11 |
| efried | Awesome. | 16:11 |
| efried | Reckon it would be good to land this today for RC1 if possible. <== melwitt FYI | 16:13 |
| efried | jaypipes: If you like, I can fix my nits and reorder to put Tetsuro's on bottom. | 16:13 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Add a prelude release note for the 18.0.0 Rocky GA https://review.openstack.org/589303 | 16:16 |
| *** s10 has joined #openstack-placement | 16:38 | |
| openstackgerrit | Merged openstack/nova stable/pike: Fix bad links for admin-guide https://review.openstack.org/590072 | 16:48 |
| *** s10 has quit IRC | 16:57 | |
| cdent | jaypipes, efried : I'm done for the day but should be fully availalbe tomorrow. Anything in particular that wants attention? | 17:09 |
| efried | other than the stuff you know about. | 17:09 |
| efried | I pushed a teeny doc fix earlier. | 17:09 |
| efried | https://review.openstack.org/590389 | 17:09 |
| cdent | ayeaye | 17:11 |
| efried | cdent: Oh, and a reminder to look into making your perf stuff available via zuul somehow. | 17:11 |
| cdent | yeah, it's on my list | 17:12 |
| efried | cool mon | 17:12 |
| cdent | I think it is good idea, but I also think we need to make some kind of adjustment in our thinking and process so that the human experimenting I did which led to the discovery is more common | 17:12 |
| efried | That sounds like a nice ideal, but I doubt it's a realistic expectation. | 17:13 |
| cdent | well, realism is apparently lame and ill suited to creating good software, so it needs to change | 17:13 |
| efried | I'm not disagreeing that we should be trying to experiment and discover more; just trying to take a different angle on how that comes about. | 17:15 |
| efried | because getting our employers to pay us to "go off and play with stuff" is a tough sell. | 17:15 |
| efried | Particularly since getting our employers to pay us to "make these features come about" is getting to be a tough sell itself. | 17:15 |
| cdent | I know that it is hard, but that human element is the crucial piece of the pie and accepting what lame ass employers think is the right way to do shit is not going to change anything | 17:16 |
| efried | Falling on swords will, though. Not in a good way. | 17:18 |
| efried | Anyway, your experimenting was good and led to good work. You should punch out today on that positive note. | 17:19 |
| cdent | When I see more evidence that we, as employees, are trying to tell our employers what the right thing to do is, I'll be more willing to accept that reality is as harsh as it is. | 17:19 |
| cdent | But what I see is us reacting without fighting back. | 17:20 |
| cdent | Mind you: I have no idea what is going on inside your workplace so my apologies if there is action I cannot see. If there is, good luck and more power to you. | 17:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Rollback volumes to in-use on DeviceDetachFailed https://review.openstack.org/590439 | 17:21 |
| edleafe | Dunno about anyone else at IBM, but none of my bosses have ever told me to do anything other than what I thought was the best way to do it. | 17:28 |
| cdent | g'night all | 17:29 |
| *** cdent has quit IRC | 17:29 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/590445 | 17:34 |
| mriedem | jaypipes: lets see if this works ^ | 17:34 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/590445 | 17:38 |
| openstackgerrit | Merged openstack/nova master: Update the parameter explain when updating a volume attachment https://review.openstack.org/565181 | 17:44 |
| jaypipes | efried: hold off on pushing anything pls | 17:51 |
| efried | jaypipes: ack, holding. | 17:51 |
| jaypipes | efried: sorry, was at lunch | 17:51 |
| efried | no worries, guy's gotta eat. | 17:51 |
| * efried grabs sandwich | 17:51 | |
| *** mriedem has quit IRC | 19:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/590445 | 19:11 |
| *** mriedem has joined #openstack-placement | 19:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix image-defined numa claims during evacuate https://review.openstack.org/588657 | 19:15 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add encrypted volume support to feature matrix docs https://review.openstack.org/570255 | 19:18 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix GET /flavors?is_public description https://review.openstack.org/588092 | 19:28 |
| openstackgerrit | Merged openstack/nova stable/queens: Fix host validity check for live-migration https://review.openstack.org/590262 | 19:37 |
| openstackgerrit | Merged openstack/nova master: [placement] api-ref: add description for 1.29 https://review.openstack.org/589407 | 19:37 |
| openstackgerrit | Merged openstack/nova stable/queens: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590062 | 20:03 |
| *** nicolasbock has quit IRC | 20:11 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 20:17 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041 | 20:17 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 20:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle binding_failed vif plug errors on compute restart https://review.openstack.org/587498 | 20:22 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle binding_failed vif plug errors on compute restart https://review.openstack.org/587498 | 20:27 |
| openstackgerrit | Merged openstack/nova stable/queens: Reload oslo_context after calling monkey_patch() https://review.openstack.org/589249 | 20:39 |
| openstackgerrit | Merged openstack/nova stable/queens: Fix message for unexpected external event https://review.openstack.org/589505 | 20:39 |
| openstackgerrit | Merged openstack/nova master: Trivial fix on migration doc https://review.openstack.org/589028 | 20:39 |
| openstackgerrit | Merged openstack/nova master: Add a prelude release note for the 18.0.0 Rocky GA https://review.openstack.org/589303 | 20:39 |
| mriedem | jaypipes: scope warnings are gone amigo http://logs.openstack.org/45/590445/3/check/tempest-full-py3/b411332/controller/logs/screen-placement-api.txt.gz?level=WARNING | 20:49 |
| efried | woot | 20:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: placement: ignore policy scope check failures if not enforcing scope https://review.openstack.org/590445 | 20:54 |
| mriedem | ^ fixed cdent's comment | 20:55 |
| efried | mriedem: We're not hoping for this to land in RC1, I hope? | 20:55 |
| mriedem | i think we could backport it or do it in rc2 | 20:55 |
| mriedem | it's a shitload of spam | 20:55 |
| mriedem | and new in rocky | 20:55 |
| efried | mriedem: I agree it ought to go in rocky. | 20:55 |
| efried | just really want cdent's +1 before we merge it. | 20:56 |
| mriedem | we're doing an rc2 btw | 20:56 |
| efried | and lbragstad's too. | 20:56 |
| efried | okay, cool. | 20:56 |
| mriedem | i'll throw it in the rc etherpad | 20:56 |
| mriedem | melwitt: ^ | 20:56 |
| melwitt | ack | 20:56 |
| jaypipes | mriedem: ty for tackling that. | 20:58 |
| mriedem | np, should have done it months ago when i complained about the same thing | 20:59 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | 21:40 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | 22:07 |
| openstackgerrit | Merged openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413 | 22:31 |
| openstackgerrit | Merged openstack/nova master: Update ssh configuration doc https://review.openstack.org/589844 | 22:32 |
| openstackgerrit | Merged openstack/nova stable/queens: [placement] Retry allocation writes server side https://review.openstack.org/588569 | 22:35 |
| openstackgerrit | Merged openstack/nova stable/pike: Reload oslo_context after calling monkey_patch() https://review.openstack.org/589251 | 22:39 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add functional test for affinity with multiple cells https://review.openstack.org/585073 | 23:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258 | 23:02 |
| *** efried has quit IRC | 23:31 | |
| *** efried has joined #openstack-placement | 23:31 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!