| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-10 | |||
| 17:26:07 | kashyap | artom: `grep` for the terms: "conversation" and "wild" here :-) https://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg01455.html | |
| 17:26:11 | artom | kashyap, I am disappoint | |
| 17:26:21 | artom | They completely missed the "you've made me comma-tose" pun | |
| 17:26:45 | cdent | dansmith: yeah, you’re still working on it, or yeah, it’s dead, or maybe “yeah, it should exist but I’ve not got the time”? | |
| 17:26:48 | kashyap | artom: Ah, good one :-) | |
| 17:26:58 | dansmith | cdent: mostly the latter | |
| 17:27:21 | dansmith | cdent: why, you want to work on it? | |
| 17:27:36 | cdent | dansmith: cool, thanks | |
| 17:28:20 | cdent | dansmith: well, perhaps, but the immediate issue was that it’s in my to do list as a recurring thing to review, and I’ve been defering it over increasing long periods | |
| 17:28:43 | dansmith | okay | |
| 17:28:44 | cdent | so thought perhaps it was dead and I can get it off my radar. i’ll keep it there for now. | |
| 17:29:06 | cdent | I don’t have a clear understanding on what time or attention I’ve got… | |
| 17:33:07 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: [WIP] Add native implementation OVSDB API https://review.openstack.org/482226 | |
| 17:35:39 | artom | kashyap, since you nibbled at my bait | |
| 17:35:45 | artom | kashyap, https://review.openstack.org/#/c/481478/1//COMMIT_MSG | |
| 17:35:55 | artom | There's nothing *wrong* per se, it just reads weird | |
| 17:36:27 | artom | (Also, I think "taking the bait" is the correct expression, nibbling makes it sound, err, yeah) | |
| 17:37:00 | artom | Surely I can't just come in and "nothing's wrong here, but it's weird, here's what I'd write:" ? | |
| 17:38:29 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object https://review.openstack.org/474914 | |
| 17:38:48 | edleafe | artom: I usually acknowledge that while I understood the message, a native English speaker would phrase it differently | |
| 17:39:08 | kashyap | artom: Need to step out for a few; bbiab. Sorry | |
| 17:39:14 | kashyap | I'll look when I'm back | |
| 17:40:14 | artom | edleafe, ah, good technique :) | |
| 17:40:19 | artom | kashyap, sure, thanks :) | |
| 17:43:45 | artom | It comes down to reviewing something a person can't help | |
| 17:44:26 | artom | While we can hold pretty much everyone to the same coding standard, it's unreasonable to hold everyone to the same English standard | |
| 17:48:52 | mriedem | https://www.youtube.com/watch?v=KzyCi1BFATA | |
| 17:54:49 | artom | mriedem, better late than never ;) | |
| 17:56:03 | mriedem | gibi: was it intentional to not include the bdm.device_name in the versioned notifications? | |
| 18:03:52 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in scheduler https://review.openstack.org/476632 | |
| 18:03:56 | jaypipes | dansmith: corrected. ^ | |
| 18:07:11 | jaypipes | dansmith: isn't it a quill, you old fart? | |
| 18:08:41 | dansmith | jaypipes: of course | |
| 18:08:55 | mriedem | if a bdm is created with destination_type='local', does it get a volume_id? if that tree falls in the bdm woods, does it make a sound? | |
| 18:09:25 | dansmith | if a bear drops a bdm in the woods, does it smell? | |
| 18:09:31 | mriedem | i'm sure it does | |
| 18:09:44 | mriedem | <sniff> <sniff> "smells like....bdm" | |
| 18:12:48 | mriedem | so you can have an image/local bdm, which is the boot device on local storage for the image the server is created with? | |
| 18:13:03 | mriedem | which is not the same as boot from volume, since there is no volume? | |
| 18:13:50 | mriedem | and blank/local means swap or ephemeral disk, which is local, which is not a volume | |
| 18:13:52 | mriedem | yeah?! | |
| 19:02:16 | mriedem | ha, the instance.update notification doesn't actually included the updated_at field | |
| 19:07:10 | dansmith | jaypipes: I would +2 your top patch but mriedem usually sees that as an opportunity to screw up my stats, and I don't want to inflict that on you, so I'll circle back when he's had a look :) | |
| 19:07:18 | mriedem | gdi | |
| 19:07:43 | mriedem | you know i won't review until you +2 | |
| 19:07:56 | dansmith | well, then we have a problem here | |
| 19:08:12 | dansmith | I propose a dance-off | |
| 19:08:14 | mriedem | dansmith: how about i review that claims shenanigans and you see if the updates to https://review.openstack.org/#/c/416521/ make you happy | |
| 19:08:42 | dansmith | ugh | |
| 19:08:46 | mriedem | ha | |
| 19:08:56 | mriedem | or, see if my hypervisors/services uuid api changes make you happy | |
| 19:09:18 | mriedem | or any of the cinder v3 attachments patches | |
| 19:09:22 | mriedem | there are plenty to choose from | |
| 19:09:58 | ildikov | the Cinder v3 ones are very joyful :) | |
| 19:10:53 | ildikov | and if that statement would ruin my chances to get them reviewed please just ignore it... | |
| 19:11:01 | smcginnis | :) | |
| 19:11:49 | ildikov | although this one is really close: https://review.openstack.org/#/c/456971/ | |
| 19:13:26 | dansmith | mriedem: I thought I was all +2 on your services ones | |
| 19:13:42 | mriedem | you said "when the time comes" | |
| 19:14:28 | dansmith | oh I see I had a -1 recently on the top one | |
| 19:14:39 | mriedem | yeah i addressed your issues i think | |
| 19:14:41 | mriedem | and alex's | |
| 19:17:27 | dansmith | well, not the bit you said you didn't understand | |
| 19:17:30 | dansmith | does this help? https://review.openstack.org/#/c/471899/24/nova/api/openstack/compute/hypervisors.py | |
| 19:18:50 | mriedem | no | |
| 19:18:55 | mriedem | you want me to rename the parameter? | |
| 19:19:16 | mriedem | see https://review.openstack.org/#/c/471899/24/nova/api/openstack/compute/hypervisors.py@381 | |
| 19:23:31 | dansmith | I know, | |
| 19:23:58 | dansmith | I realize this is existing, I'm just saying it's super confusing/wrong to pass instances to a thing, calling them servers | |
| 19:24:42 | dansmith | just thought maybe we should clean that up since you're touching this code | |
| 19:24:44 | mriedem | so you want me to rename the formal parameter | |
| 19:24:53 | mriedem | i need you to tell me what you want me to do | |
| 19:24:54 | mriedem | :) | |
| 19:24:57 | mriedem | jaypipes: https://review.openstack.org/#/c/476632/ | |
| 19:25:42 | mriedem | i don't personally see how instances and servers is confusing, since instances feed the 'servers' resource in the API | |
| 19:25:47 | mriedem | are you thinking servers == hosts? | |
| 19:26:56 | dansmith | oh, I see, heh | |
| 19:26:58 | dansmith | yes | |
| 19:27:05 | mriedem | :/ | |
| 19:27:09 | dansmith | and also servers=services | |
| 19:27:14 | dansmith | more the latter I think | |
| 19:27:23 | dansmith | because I was already in the services frame of mind | |
| 19:27:30 | edleafe | ugh - I remember this naming confusion back from the early days with RAX | |
| 19:27:44 | mriedem | oh yeah | |
| 19:27:44 | mriedem | https://www.youtube.com/watch?v=D4O94hw3qt0 | |
| 19:27:47 | mriedem | that's the kind of mind i'm in | |
| 19:27:58 | edleafe | since they called their things "Cloud Servers", they insisted on calling an instance a "server" | |
| 19:28:11 | dansmith | servers is a stupid name for it, | |
| 19:28:26 | edleafe | dansmith: just one of the many arguments I lost in those days | |
| 19:28:42 | dansmith | but the ship has sailed, I just don't spend enough time in the api to remember all the time and the patch below is about _services_ | |
| 19:31:38 | mriedem | dansmith: if you're also good with the os-services change, can you +2 that so we can queue things up? | |
| 19:31:54 | mriedem | if i can get alex_xu to also be +2 on the series then we'll push it through | |
| 19:31:58 | dansmith | done | |
| 19:32:00 | mriedem | yay | |
| 19:36:04 | mriedem | now i can't get that song out of my head | |
| 19:37:56 | dansmith | mriedem: tbh I feel like I have a bit of review fatigue on this quota stuff | |
| 19:38:20 | dansmith | not sure I've seen an actual review from you on this one | |
| 19:38:29 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in scheduler https://review.openstack.org/476632 | |
| 19:38:30 | mriedem | you haven't | |
| 19:38:33 | jaypipes | mriedem: fixed :( ^ | |