Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-27
15:13:55 jaypipes dansmith: so it's really just the lone paren on a line that you don't like, right?
15:14:25 dansmith jaypipes: and the trailing comma in a call that goes with it
15:14:37 efried I'll restate dislike for the trailing comma, which loses its benefit (two-line change becomes one-line change) if you pull the paren back.
15:14:40 efried yeah, what he said.
15:15:07 dansmith jaypipes: I don't really see the need to break every call into a multiline one-per-line verbose list either, but.. that's less smelly than the lone paren and trailing comma thing
15:15:14 dansmith for long ones, or ones with a very long parameter name, sure
15:15:30 jaypipes different style, is all.
15:15:37 jaypipes but I'll work with you.
15:15:49 dansmith jaypipes: I know, and we do some of that in other code.. we do not, however, do the trailing comma and lone paren thing :)
15:16:03 openstackgerrit Eric Fried proposed openstack/nova master: Add formatting to scheduling activity diagram https://review.openstack.org/476204
15:16:55 jaypipes dansmith, efried: you guys would hate Golang then :)
15:16:57 efried I personally dig the one-per-line thing for really long param lists. I'll take the vspace hit for readability there.
15:17:18 dansmith jaypipes: it's consistency with the rest of our code that I care about
15:17:24 jaypipes understood.
15:17:29 dansmith jaypipes: if all our code looked like that it would be fine (ugly, but fine)
15:17:50 dansmith efried: for long param lists, I do it myself sometimes, yeah
15:25:35 jaypipes r you.
15:25:35 jaypipes ng like this just fo\
15:25:35 jaypipes g to start codi\
15:25:35 jaypipes m goin\
15:25:35 jaypipes dansmith: I'\
15:25:53 dansmith jaypipes: I don't hate that, but it's not our style, so I'll still -1 it :)
15:25:58 jaypipes lol
15:26:09 dansmith er wait, you probably didn't get that, let me try again:
15:26:11 dansmith jaypipes: I don't hate that, but it's not our style, so I'll still -1 it :
15:26:12 dansmith )
15:26:13 dansmith there
15:26:21 dansmith FTFY
15:26:54 jaypipes rofl
15:27:39 jaypipes dansmith: you missed the comma at the end of the first line.
15:28:49 jaypipes dansmith: you're my little exorcist today.
15:29:15 dansmith jaypipes: dude, https://review.openstack.org/#/c/476630/6/nova/tests/unit/scheduler/weights/test_weights_affinity.py on line 50 and below
15:29:22 dansmith why do you hate parens all the sudden?
15:29:30 dansmith stop torturing parens, jaypipes
15:30:35 jaypipes dansmith: alignment is very important to me.
15:31:26 openstackgerrit Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/478083
15:31:31 dansmith jaypipes: you can align those without putting parens on their own lines
15:31:37 jaypipes dansmith: but I'm gonna rework that patch actually to reduce the amount of code change as much as possible, even if it means some hackery.
15:31:41 dansmith you've actually increased indent there for no reason :)
15:32:22 dansmith jaypipes: okay, it is a lot of change, but I'm not super opposed to the way it is, FWIW
15:32:57 jaypipes dansmith: I see your suggestion about "just return the HostState objects". That's more along the lines of what I was thinking.
15:33:13 dansmith I'm glad you let the golang thing slip, as that completes the circle of me wondering what in your life happened in the last couple months that caused you to start doing this
15:33:21 dansmith jaypipes: yeah
15:33:22 jaypipes dansmith: :)
15:40:29 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971
16:26:11 evilpig_ hello, does anyone know if the xen backend is supported for vcpu pinning?
16:26:41 evilpig_ i think i have everything set up right, but the numatopologyfilter is rejecting my compute nodes
16:27:23 evilpig_ i can perform the pinning through virsh so it looks like libvirt support is there
16:28:00 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/477982
16:28:29 artom evilpig_, I don't think so
16:28:35 artom evilpig_, https://docs.openstack.org/developer/nova/feature_classification.html#matrix-nfv
16:28:46 artom That doesn't answer your exact question
16:29:32 evilpig_ artom, thank you for the pointer
18:09:16 dansmith jaypipes: this is a trivial object method add.. I didn't write it, but I did push the patch up first, so I haven't +2d: https://review.openstack.org/#/c/446244/29
18:09:44 dansmith jaypipes: you could increase universal goodwill by +Wing it to move things along, without needing to, um, look at it very deeply
18:11:37 jaypipes dansmith: it has been vertical synergized with the gate.
18:11:53 dansmith jaypipes: excellent solutioning, thanks
18:11:58 jaypipes np
18:13:09 melwitt I'm liking this hyperconvergence
18:23:08 dansmith I saw someone in a forum the other day asking where they could download some hyperconverged software
18:23:43 dansmith I can only expect they were either a CIO of a fortune 500 company, or a stooge recently tasked by the former
18:25:24 melwitt wow. I didn't think things like that really happened
18:27:21 jaypipes melwitt: I am currently experiencing things like that. live, on this conference call.
18:28:18 melwitt holy smokes
18:47:42 openstackgerrit Merged openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387
19:12:49 jaypipes dansmith: almost done recognizing my synergistic realities and integrating my vertical solutioning with the here-and-now. (tests almost finished)
19:13:15 dansmith oh I knew exactly what you were saying
19:43:37 openstackgerrit Merged openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244
20:32:04 openstack Launchpad bug 1688673 in OpenStack Compute (nova) "cpu_realtime_mask handling is not intuitive" [Wishlist,New]
20:32:04 cfriesen sdague: your scripts are going nuts: https://bugs.launchpad.net/nova/+bug/1688673
20:34:05 sdague cfriesen: gah, there is a bug in the base regexi
20:34:20 sdague sorry about that
20:34:49 sdague let me see if this is fixed now
20:37:28 melwitt my precious bugs!
20:45:10 openstackgerrit Merged openstack/nova master: return 400 Bad Request when empty string resources https://review.openstack.org/477019
20:47:16 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285
20:47:41 sdague cfriesen: I'm going to use that bug to make sure this is right, so ignore a few updates here
20:50:08 sdague oh cfriesen I see why, you used the js url
20:56:16 sdague cfriesen: on the upside, I think only your bug was impacted
20:59:01 cfriesen sdague: is there a better format for the URL?
21:00:00 sdague the official url for reviews is https://review.openstack.org/\d+
21:00:16 sdague the /#/c/ thing is all js magic
21:00:23 sdague single page js magic
21:00:38 sdague but, the tooling will do the other one now
21:30:00 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
21:40:58 openstackgerrit Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448
21:40:58 openstackgerrit Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631
21:40:59 openstackgerrit Jay Pipes proposed openstack/nova master: placement: refactor driver select_destinations() https://review.openstack.org/478291
21:40:59 openstackgerrit Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632
21:55:51 openstackgerrit Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
21:57:55 openstackgerrit Mike Perez proposed openstack/nova master: Remove conductor topic configuration option https://review.openstack.org/463193
22:04:33 openstackgerrit Mike Perez proposed openstack/nova master: Remove compute topic configuration option https://review.openstack.org/463197
22:09:18 openstackgerrit Mike Perez proposed openstack/nova master: Remove console topic configuration option https://review.openstack.org/463200
22:10:29 openstackgerrit Mike Perez proposed openstack/nova master: Remove wsdl_location configuration option https://review.openstack.org/463220
22:32:55 openstackgerrit melanie witt proposed openstack/nova master: Add scatter gather utilities for cells https://review.openstack.org/475957
22:32:55 openstackgerrit melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700
22:32:56 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
22:32:57 openstackgerrit melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699
22:32:58 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
22:33:02 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521

Earlier   Later