Posted	Nick	Remark
#openstack-nova - 2017-11-03
14:57:59	fried_rice	kashyap It's a signal for "post a recheck" :) Which I've just done.
14:58:10	fried_rice	kashyap Unrelated moon phase timeout.
14:58:53	kashyap	fried_rice: Ah, thanks. (Your Friday nick is making me hungry)
14:59:11	fried_rice	kashyap :) I'm always hungry.
14:59:12	leakypipes	fried_rice: ah, yeah, sorry. I personally find "if (not valid_this_way and not invalid_that_way)" to be easier to read than "if not (valid_this_way or valid_that_way)" but it's a total preference.
14:59:24	leakypipes	fried_rice: and no, nothing left to do on that patch IMHO
14:59:56	leakypipes	fried_rice: +2df
15:00:07	fried_rice	leakypipes Thanks!
15:02:53	fried_rice	leakypipes I'm gonna take cdent's 'symmetric' series and at least resolve merge conflicts FYI.
15:03:04	leakypipes	fried_rice: the spec?
15:03:17	leakypipes	fried_rice: cuz mriedem had negative comments on the spec
15:03:34	fried_rice	leakypipes He updated the spec. I made some minor comments on it, but it's basically good to go.
15:03:53	fried_rice	leakypipes I'm talking about the code series starting at https://review.openstack.org/#/c/512420/
15:05:49	fried_rice	leakypipes I'll probably make those spec edits too when I'm done with the code series.
15:05:54	leakypipes	fried_rice: k, go for it then :)
15:06:42	fried_rice	(I was telling you in case you were looking to do the same, or had already started. But I take it from your reaction that it wasn't on your radar :)
15:09:43	leakypipes	fried_rice: correctomundo.
15:10:30	openstackgerrit	Eric Fried proposed openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id https://review.openstack.org/512420
15:10:31	openstackgerrit	Eric Fried proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073
15:10:31	openstackgerrit	Eric Fried proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626
15:12:21	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
15:12:21	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
15:12:42	leakypipes	fried_rice: fixed func test issues (friggin case-sensitive HTTP headers...)
15:12:45	leakypipes	^^
15:12:53	fried_rice	leakypipes Ack
15:14:12	alex_xu	fried_rice: leakypipes morning
15:14:22	leakypipes	alex_xu: good evening, sir!
15:14:24	fried_rice	alex_xu Howdy
15:15:00	alex_xu	leakypipes: yea, I already +2 on that patch, just throw an idea, actually I don't have mind how much time spend on running those small loop mulitple times, anyway that isn't a blocker
15:15:25	leakypipes	alex_xu: understood. I'm still going to add that @property because I think it makes the calling code more readable.
15:15:51	alex_xu	fried_rice: in normaly, what kind of response is for 'Howdy' in English?
15:16:02	alex_xu	leakypipes: ok, cool
15:16:13	fried_rice	alex_xu It's just like "Hello".
15:16:20	leakypipes	alex_xu: "howdy partner" is the appropriate response. :)
15:16:37	mriedem	or tip your cap and say "ma'am"
15:16:58	fried_rice	alex_xu I know if you look it up in the dictionary it says it means "How do you do?" But it's not a question, and doesn't require an answer.
15:17:27	alex_xu	fried_rice: ah, got it
15:17:31	fried_rice	alex_xu So more like 你好 than 你好吗
15:19:11	alex_xu	mriedem: or doing a chinese style as response https://qph.ec.quoracdn.net/main-qimg-954705de53626cf8a07dbab0464c6636.webp
15:19:56	fried_rice	Is it necessary to have a really serious look on your face too?
15:20:05	fried_rice	Or is that just in the movies?
15:21:14	openstackgerrit	Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512
15:21:27	alex_xu	fried_rice: that is in ancient :), or in Spring festive https://qph.ec.quoracdn.net/main-qimg-f860cf304b2c8a0cac5f00950202faed.webp, and with smile face
15:21:58	fried_rice	That guy on the right has a HUGE HEAD!
15:22:08	fried_rice	or a teeny body
15:23:47	alex_xu	fried_rice: good catch :) I found all the presenter on the CCTV new has huge head, i guess that is one of standard for choice presenter...
15:24:14	fried_rice	That and the perfectly-sculpted hair. But that's a universal TV anchor requirement.
15:25:03	alex_xu	yea
15:31:06	openstackgerrit	Merged openstack/nova master: Updated from global requirements https://review.openstack.org/517489
15:59:51	openstackgerrit	Eric Fried proposed openstack/nova master: placement: AllocCands.get_by_{filters => requests} https://review.openstack.org/514092
16:00:01	fried_rice	leakypipes As promised ^
16:00:43	leakypipes	fried_rice: guh, was just about to push latest refactoring series...
16:01:15	openstackgerrit	Jay Pipes proposed openstack/nova master: begin refactor AllocCandidates._get_by_filters() https://review.openstack.org/516778
16:01:16	openstackgerrit	Jay Pipes proposed openstack/nova master: build alloc request resources for shared resources https://review.openstack.org/516780
16:01:16	openstackgerrit	Jay Pipes proposed openstack/nova master: build ProviderSummary objects in sep function https://review.openstack.org/516779
16:01:17	openstackgerrit	Jay Pipes proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782
16:01:17	openstackgerrit	Jay Pipes proposed openstack/nova master: create allocation request for single provider https://review.openstack.org/516781
16:01:18	openstackgerrit	Jay Pipes proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646
16:01:18	openstackgerrit	Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119
16:01:18	openstackgerrit	Jay Pipes proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027
16:03:07	leakypipes	fried_rice: since you based yours off of 8c0188d8b0151daf31340d2a6ab2d932d8fa15b0 (the WIP: handle traits.. patch), you'll want to rebase from the latest version of that which I just pushed.
16:12:47	openstackgerrit	Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357
16:12:47	openstackgerrit	Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707
16:12:48	openstackgerrit	Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358
16:27:11	figleaf	Hey fried_rolls, if you're a Simpsons fan: https://getyarn.io/yarn-clip/9fa624dd-1cdc-4445-b95f-bb5d6aa3d178
16:31:05	openstackgerrit	Matt Riedemann proposed openstack/nova master: Avoid redundant security group queries in GET /servers/{id}/os-security-groups https://review.openstack.org/517648
17:06:50	openstackgerrit	Jay Pipes proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646
17:06:51	openstackgerrit	Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119
17:07:50	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
17:07:50	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
17:23:03	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
17:23:03	openstackgerrit	Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
17:34:08	evrardjp	doing the needful I see?
17:38:56	openstackgerrit	Merged openstack/nova master: RequestGroup class for placement & consumers https://review.openstack.org/517364
18:02:04	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
18:03:09	openstackgerrit	OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035
18:33:38	openstackgerrit	Merged openstack/nova master: libvirt: Don't VIR_MIGRATE_NON_SHARED_INC without migrate_disks https://review.openstack.org/507202
18:50:38	fried_rice	figleaf I don't get it :(
18:57:38	figleaf	fried_rice: then I guess you're not a fan. Fried rice, fried rolls... fried cheese!
18:58:08	fried_rice	figleaf I dig it.
18:59:09	fried_rice	figleaf I own a Brazilian jiu jitsu school, and just added a noon MWF class with a new instructor. So for the forseeable future I'll be going in to make sure everything's running smoothly. In BJJ, sparring is called "rolling". So, as delicious as "fried rolls" sound, it has nothing to do with food :)
19:02:12	figleaf	fried_rice: ah, I thought you were going for a greasy theme :)
19:02:29	fried_rice	figleaf Don't get me wrong, I love to eat fried stuff.
19:25:21	fried_rice	Fly safe edleafe
20:19:34	fried_rice	leakypipes Would I be correct in stating that GET /allocation_candidates affordance of n-r-p has not yet been started?
20:20:43	fried_rice	leakypipes ...but that that's the last major piece to be tackled after the series at https://review.openstack.org/#/c/377138/ is merged?
21:30:14	openstackgerrit	Matt Riedemann proposed openstack/nova stable/pike: libvirt: properly decode error message from qemu guest agent https://review.openstack.org/517727
21:35:17	cdent	fried_rice: nice work on the rp update, thanks for doing that
21:35:18	mriedem	edleafe: dansmith: melwitt: did you guys ever talk about an agenda for the nova onboarding session? https://etherpad.openstack.org/p/SYD-forum-nova-onboarding
21:35:52	fried_rice	cdent Sure thing. Hope I didn't eff anything up :) And I would like to get some tips for next week on how you produce that list at the bottom.
21:38:26	cdent	fried_rice: I don’t have a consistent strategy. It’s mostly based on stuff I’m aware of plus looking at anything written by the usual suspects and a general review of any bugs tagged placement
21:38:41	fried_rice	cdent Okay.
21:38:45	cdent	I don’t try to get everything as that would be painful
21:38:55	cdent	early on I did
21:39:12	cdent	but then it got out of hand, especially since if you squint anything is related to placement
21:39:20	fried_rice	heh, yeah.
21:39:55	fried_rice	Well, maybe I'll just do the same thing next week and let you fill the bellows again when you get back.
21:40:24	cdent	I’ll fix up the get/put spec later today (it’s early saturday morning apparently, but doesn’t feel like it) but if you want to mess with the code please feel free
21:40:59	cdent	yeah, bellows is a good metaphor. Probably worthwhile _not_ reporting everything as it gets overwhelming and there’s no way to care about everything.
21:42:18	fried_rice	cdent Okay, so IIUC on the code side one would need to modify GET /allocation_candidates to produce its allocation_requests in the new format; update the renos, api-ref, etc. accordingly; and... anything else?

Earlier   Later