Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-11-03
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
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: Change RPC for select_destinations() https://review.openstack.org/516707
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: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: adds REST API for nested providers https://review.openstack.org/384807

Earlier   Later