| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-16 | |||
| 19:37:24 | mriedem | oh don't forget hostId: The ID of the host. | |
| 19:37:29 | mriedem | thanks compute api ref! | |
| 19:37:45 | melwitt | mriedem: yeah, I actually had a look at that. I wasn't sure if maybe it was intentional that they don't mention the distinction between host name and nodename | |
| 19:38:12 | melwitt | when you run for real, they the same unless you're running ironic | |
| 19:38:34 | mriedem | i'm not sure what the point in making the description unclear would be | |
| 19:38:42 | mriedem | do we get any security benefit from that? | |
| 19:39:02 | melwitt | I dunno. I was thinking maybe if there was concern about exposing implementation details like host and nodename. I'm not sure it matters | |
| 19:40:13 | melwitt | hypervisor_hostname: either the host name or the Ironic node name if ironic_driver | |
| 19:40:22 | melwitt | so, maybe that's reasonable to put there | |
| 19:40:27 | mriedem | i think so | |
| 19:40:38 | mriedem | if we can't actually explain wtf the fields are in our API response, our API sucks, right? | |
| 19:40:47 | mriedem | i'm pretty sure if cdent were around i'd get a checkmark for that | |
| 19:40:48 | melwitt | yeah. | |
| 19:40:58 | melwitt | hehe | |
| 19:41:05 | mriedem | i'll open a bug | |
| 19:41:10 | melwitt | k | |
| 19:42:32 | openstackgerrit | John Griffith proposed openstack/nova master: Add Cinder API version detection https://review.openstack.org/444465 | |
| 19:42:36 | lyarwood | mriedem: lol, I'm back for a while now | |
| 19:42:58 | lyarwood | mriedem: thanks for sorting the tests out, rushed things before dinner, always a mistake. | |
| 19:48:15 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | |
| 19:49:07 | mriedem | melwitt: dansmith: jaypipes: sdague: jroll: see if this makes sense https://bugs.launchpad.net/nova/+bug/1673593 | |
| 19:49:07 | openstack | Launchpad bug 1673593 in OpenStack Compute (nova) "api-ref: descriptions for the various host fields in server GET response are useless" [Low,Confirmed] | |
| 19:50:30 | melwitt | mriedem: makes sense to me | |
| 19:51:10 | jroll | mriedem: yeah, makes sense | |
| 19:51:13 | jroll | they are indeed useless | |
| 19:51:23 | mriedem | hostId: The ID of the host. | |
| 19:51:24 | mriedem | thanks | |
| 19:51:36 | jroll | # start the thread | |
| 19:51:38 | jroll | thread.start() | |
| 19:51:53 | jroll | love that stuff | |
| 19:52:12 | mriedem | # start it up kris | |
| 19:52:18 | mriedem | # it's what i was born to do | |
| 19:53:15 | melwitt | lol | |
| 19:53:41 | mriedem | *warm it up | |
| 19:53:43 | mriedem | my mistake | |
| 19:53:56 | dansmith | ooh, I was just about to catch you on that | |
| 19:54:02 | melwitt | let's wear our jeans backwards | |
| 19:54:39 | mriedem | that's wiggity wiggity wiggity whack | |
| 20:22:31 | winston-d | dansmith: ping | |
| 20:24:07 | winston-d | dansmith: Hi, Dan. I recall you mentioned Nova would have a 'force-shutdown' or sth similar API/feature to ensure hypervisor is truely down during PTG. | |
| 20:25:23 | winston-d | dansmith: Do you have any pointer to bp or spec? I quickly skim through nova-specs and couldn't find it. | |
| 20:30:51 | sdague | mriedem: I think the bug looks fine, push me a patch and lets get the api-ref fixed! | |
| 20:35:17 | mriedem | sdague: alright | |
| 20:35:52 | mriedem | sdague: while you're around, would be nice to +W this so i can include it in the backports for the related bug https://review.openstack.org/#/c/446685/ | |
| 20:36:04 | mriedem | retry on failed build is kind of important | |
| 20:36:27 | sdague | mriedem: +W | |
| 20:37:23 | sdague | mriedem: honestly, if you find a thing is wrong in api-ref, I would generally skip the bug and write the patch, and just be descriptive in the commit message. Definitely know a few of those things aren't quite right still. | |
| 20:40:07 | mriedem | i only wrote the bug because i wasn't planning on doing a patch right now | |
| 20:40:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: objects: Add attachment_id to BlockDeviceMapping https://review.openstack.org/437665 | |
| 20:42:55 | dansmith | winston-d: force-down | |
| 20:49:57 | dansmith | winston-d: https://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/mark-host-down.html | |
| 20:49:59 | openstackgerrit | John L. Villalovos proposed openstack/nova master: flake8: Specify 'nova' as name of app https://review.openstack.org/446736 | |
| 20:50:03 | bauzas | FWIW, folks I'll be off tomorrow | |
| 20:52:27 | mriedem | ok | |
| 20:52:39 | mriedem | i'll have a small human in the house, so i might be distracted | |
| 20:54:49 | mriedem | bauzas: did this get talked about in last week's nova meeting? https://blueprints.launchpad.net/nova/+spec/lvm-thin-pool | |
| 20:54:56 | mriedem | and this https://blueprints.launchpad.net/nova/+spec/remove-openstack-api-directory | |
| 20:55:17 | mriedem | logan-: edleafe: ^? | |
| 20:57:15 | bauzas | mriedem: the first one was not discussed given logan- wasn't there | |
| 20:57:32 | mriedem | ok | |
| 20:57:43 | bauzas | mriedem: the second one was discussed | |
| 20:57:52 | bauzas | mriedem: and we said possibly a specless BP | |
| 20:57:58 | bauzas | given only for tracking | |
| 20:58:00 | edleafe | mriedem: yeah, and I have some patches for that | |
| 20:58:15 | mriedem | but the bp wasn't approved | |
| 20:58:28 | edleafe | not yet :) | |
| 20:58:31 | mriedem | i guess i can look back at the logs, would have been nice if someone could have summarized in the bp whiteboard | |
| 20:59:46 | bauzas | http://eavesdrop.openstack.org/meetings/nova/2017/nova.2017-03-09-14.00.log.html | |
| 21:00:01 | bauzas | mmm, 1 min before the next one :) | |
| 21:00:10 | bauzas | actually.. | |
| 21:14:23 | efried | esberglu_ adreznec Could use a new +1 on https://review.openstack.org/438119 | |
| 21:17:55 | esberglu_ | ack | |
| 21:26:47 | openstackgerrit | Octave Orgeron proposed openstack/nova-specs master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446631 | |
| 21:34:32 | dtp | dansmith / melwitt: can i get a new task? | |
| 21:35:46 | jroll | mriedem: I don't see anything too crazy about it | |
| 21:35:54 | jroll | from ironic POV | |
| 21:36:17 | jroll | though I wonder if there's other ways to do it, hm | |
| 21:36:46 | mriedem | dtp: i will have something i think once i have a spec written | |
| 21:37:06 | dtp | ok | |
| 21:37:15 | dansmith | mriedem: the services id thing? | |
| 21:37:17 | mriedem | yes | |
| 21:37:20 | mriedem | dtp: this needs a spec https://blueprints.launchpad.net/nova/+spec/service-hyper-pci-uuid-in-api | |
| 21:37:31 | mriedem | dtp: if you can parse that and grok it, feel free to write the spec | |
| 21:37:33 | dansmith | mriedem: cool, I just tweaked the hostapi this morning to do the thing we said it should do | |
| 21:37:38 | mriedem | dansmith: ok | |
| 21:37:41 | mriedem | high five | |
| 21:37:47 | mriedem | \o\ | |
| 21:37:49 | mriedem | /o/ | |
| 21:37:51 | dansmith | o/* | |
| 21:37:55 | mriedem | o/. | |
| 21:38:08 | melwitt | .\o | |
| 21:38:10 | mriedem | i'm just going to steal the classics from melwitt | |
| 21:38:13 | jroll | o/' | |
| 21:38:17 | dtp | armpit bump? | |
| 21:38:18 | jroll | cheeeeeers | |
| 21:38:20 | dansmith | ~~.\o/.~~ | |
| 21:38:24 | melwitt | lol | |
| 21:38:25 | mriedem | odor lines? | |
| 21:38:30 | dansmith | smell lines yeah | |
| 21:38:32 | mriedem | ha | |