| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-11-03 | |||
| 12:46:38 | fried_rice | Wow | |
| 12:46:46 | leakypipes | fried_rice: :P | |
| 12:51:03 | fried_rice | leakypipes My take on alex_xu's comment is that he was after performance, not LOC. And what you suggest is the same execution-wise (plus a negligible amount of call stack overhead) so I don't see the benefit. | |
| 12:54:41 | mdbooth | Any cores not on a plane, could somebody please restore the +W to https://review.openstack.org/#/c/507202/ after a rebase? | |
| 12:55:42 | mdbooth | Gerrit complained of a merge conflict, but I was still able to use the rebase button... | |
| 12:59:55 | fried_rice | leakypipes ^ (FWIW, the rebase looks clean to me) | |
| 13:01:31 | openstackgerrit | guanzuoyu proposed openstack/nova master: Modify incorrect debug meaasge in _inject_data https://review.openstack.org/517227 | |
| 13:08:21 | gibi | fried_rice: removing __eq__ sound OK to me. I will check back to the review soon | |
| 13:08:31 | fried_rice | gibi Thanks! | |
| 13:10:27 | leakypipes | fried_rice: yeah, and I don't think the performance difference is worth adding the complexity to the object state. | |
| 13:10:40 | leakypipes | fried_rice: and needing to deal with cache coherence issues. | |
| 13:11:21 | fried_rice | leakypipes I can't speak to that (no idea about the performance of versioned objects), which is why I was staying out of it. alex_xu is +2 anyway, so... :) | |
| 13:12:33 | fried_rice | mdbooth - gibi gotcha covered | |
| 13:12:56 | gibi | mdbooth: yep, just +Wd it 10 seconds ago | |
| 13:12:56 | gibi | :) | |
| 13:12:58 | mdbooth | fried_rice: gibi Thanks | |
| 13:26:50 | openstackgerrit | Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151 | |
| 13:26:50 | openstackgerrit | Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223 | |
| 13:26:51 | openstackgerrit | Eric Fried proposed openstack/nova master: ResourceRequest.to_querystring() https://review.openstack.org/515811 | |
| 13:27:09 | fried_rice | leakypipes I just removed the traits check for now; let me know if you want me to muck with the ResourceClass check. ^ | |
| 13:30:31 | leakypipes | fried_rice: k. | |
| 13:30:54 | leakypipes | fried_rice: getting some breakfast and then still trying to fix the dang n-r-p failing func tests... | |
| 13:31:01 | fried_rice | ack | |
| 13:49:50 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/517489 | |
| 13:54:12 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/511035 | |
| 14:00:15 | openstackgerrit | Eric Fried proposed openstack/nova master: Update contributor microversion doc for compute https://review.openstack.org/517626 | |
| 14:00:22 | fried_rice | mriedem ^ | |
| 14:22:21 | openstackgerrit | Eric Fried proposed openstack/nova master: Update contributor microversion doc for compute https://review.openstack.org/517626 | |
| 14:23:39 | mriedem | mikal: just realized you have a specific session for privsep at the forum http://forumtopics.openstack.org/cfp/details/41 | |
| 14:23:48 | mriedem | so you have to start your own etherpad and add it to https://wiki.openstack.org/wiki/Forum/Sydney2017 | |
| 14:41:22 | leakypipes | fried_rice: +W on bottom two of the placement-side | |
| 14:41:35 | fried_rice | leakypipes Whee! Thanks. | |
| 14:42:43 | fried_rice | leakypipes I think I'm going to change the WIP that's on top there to use the new format like it's doing, but pull out the old-style data, so it stays doing what it's doing. Then do the actual API change with microversion later. (Much later - once the whole refactor and alex_xu's stack is ready.) | |
| 14:42:57 | leakypipes | ++ | |
| 14:54:52 | fried_rice | leakypipes Is there something still needed on https://review.openstack.org/#/c/515151/ ? | |
| 14:55:23 | leakypipes | fried_rice: yeah, gonna do what you suggested in PS3 | |
| 14:56:07 | kashyap | mdbooth: Why is Zuul -2 on this change? Or is it just its way of telling: "Waiting to merge"? | |
| 14:56:10 | kashyap | https://review.openstack.org/#/c/507202/ | |
| 14:56:54 | kashyap | Okay, I take that as a signal for: "Blocked on Verified Label". Ignore me. | |
| 14:56:56 | fried_rice | leakypipes You're talking about https://review.openstack.org/#/c/516781/ ? Cool. I was talking about https://review.openstack.org/#/c/515151/ though. | |
| 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] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | |
| 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:12:21 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 15:12:21 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 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 ProviderSummary objects in sep function https://review.openstack.org/516779 | |
| 16:01:16 | openstackgerrit | Jay Pipes proposed openstack/nova master: build alloc request resources for shared resources https://review.openstack.org/516780 | |
| 16:01:17 | openstackgerrit | Jay Pipes proposed openstack/nova master: create allocation request for single provider https://review.openstack.org/516781 | |
| 16:01:17 | openstackgerrit | Jay Pipes proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782 | |
| 16:01:18 | openstackgerrit | Jay Pipes proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | |
| 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: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | |