| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-13 | |||
| 20:18:47 | mriedem | i've done the libvirtd log digging | |
| 20:19:04 | mriedem | not sure if this is a problem in libvirt/qemu or if it's expected to take this long | |
| 20:19:09 | mriedem | not a problem for ovs | |
| 20:19:37 | sgordon | mmm, and if i am perfectly honest most of the testing on our end focuses on ovs these days | |
| 20:22:15 | mriedem | we could bump this timeout for tap devices, but that's just a workaround, and i believe the tempest tests would actually timeout before that | |
| 20:23:18 | mriedem | tempest times out just at 5 minutes | |
| 20:23:26 | mriedem | so it'd be a craptacular race to the bottom | |
| 20:24:27 | sgordon | mriedem, i would like to think it should not in fact take > 1 minute but i will track down someone who is actually intelligent to look at it | |
| 20:24:32 | sgordon | :) | |
| 20:25:05 | mriedem | sgordon: fwiw the neutron side reverted the change to use vif_type='tap' with linuxbridge https://review.openstack.org/#/c/472365/ | |
| 20:25:12 | mriedem | interested to know why that would make a difference | |
| 20:25:41 | mriedem | the vif_type is now using 'bridge' again | |
| 20:26:39 | mriedem | which goes through os-vif | |
| 20:26:42 | mriedem | tap does not yet | |
| 20:43:28 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: Use CONF.host for powervm nodename https://review.openstack.org/473975 | |
| 20:48:33 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773 | |
| 21:07:19 | mriedem | claudiub|2: i thought hyper-v supported fibrechannel? https://docs.openstack.org/developer/nova/support-matrix.html#storage_block_backend_fibrechannel | |
| 21:08:05 | mriedem | https://github.com/openstack/nova/blob/master/nova/virt/hyperv/volumeops.py#L364 | |
| 21:09:30 | mriedem | efried: questions in the powervm support matrix change https://review.openstack.org/#/c/470999/ | |
| 21:10:03 | efried | esberglu ^^ | |
| 21:10:07 | efried | Thanks mriedem, looking. | |
| 21:12:30 | openstackgerrit | Merged openstack/nova master: remove ugly local import https://review.openstack.org/450787 | |
| 21:12:33 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Indicate Hyper-v supports fibre channel in support matrix https://review.openstack.org/473986 | |
| 21:15:36 | claudiub|2 | mriedem: yep, it does. | |
| 21:15:57 | mriedem | claudiub|2: ok patched above | |
| 21:15:58 | claudiub|2 | i guess we have to update the matrix. | |
| 21:17:33 | mriedem | already did ^ | |
| 21:19:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove service control from feature support matrix https://review.openstack.org/473989 | |
| 21:20:10 | mriedem | sdague: dansmith: do you happen to remember what "service control" meant? ^ | |
| 21:20:39 | mriedem | maybe it meant QoS? | |
| 21:20:48 | dansmith | mriedem: literally bringing up and down services, I think. not sure if that meant nova-compute itself, or things like turning xen on and off for a given node | |
| 21:21:01 | dansmith | before my time I think, but never widely supported AFAIK | |
| 21:21:21 | mriedem | oh like the libvirt driver auto-disabling nova-compute... | |
| 21:21:36 | dansmith | well, I think it was more that that | |
| 21:21:41 | mriedem | either way, | |
| 21:21:49 | dansmith | like if libvirt supported it, it might be system("service libvirt-bin stop") kind of thing | |
| 21:22:00 | mriedem | if we don't know wtf it means, probably best to not doc it and if we care about documenting specifics we do that separately | |
| 21:22:06 | dansmith | yeah | |
| 21:22:17 | dansmith | here's a good description: | |
| 21:22:21 | dansmith | "not a thing we do" | |
| 21:23:10 | mriedem | heh, reminds me of https://developer.openstack.org/api-ref/compute/?expanded=start-host-detail#start-host | |
| 21:23:19 | mriedem | "This is not implemented by any in-tree compute drivers and therefore will always fail with a 501 NotImplemented error. Needless to say, it is not recommended to use this API and it is deprecated as of the 2.43 microversion." | |
| 21:23:35 | mriedem | "here is our middle finger, have a nice day" | |
| 21:25:16 | dansmith | heh | |
| 21:25:36 | dansmith | maybe: | |
| 21:32:33 | mriedem | efried: OS-DCF:diskConfig is the auto disk configstuff | |
| 21:32:43 | mriedem | old xen thing that predated cloud-init i think | |
| 21:32:52 | mriedem | johnthetubaguy could probably explain it | |
| 21:33:14 | mriedem | i'd love to remove it from our api code since it's messy | |
| 21:35:28 | efried | hyperv seems to think it has it also. | |
| 21:36:10 | mriedem | well, i don't think it means anything to other drivers | |
| 21:36:25 | mriedem | tempest tests against that request parameter in our gate, with libvirt | |
| 21:36:32 | mriedem | but i don't think it does anything | |
| 21:37:36 | mriedem | yeah grep auto_disk_config | |
| 21:37:43 | efried | Looks like sdague might've been on the road to removing it. | |
| 21:37:43 | mriedem | it's an image property only used by the xen driver | |
| 21:37:45 | efried | https://review.openstack.org/#/c/327188/ | |
| 21:37:52 | efried | Tomorrow will be the 1y anniversary of that change ;-) | |
| 21:38:02 | mriedem | efried: that's not removing it from the api, | |
| 21:38:11 | mriedem | that's just folding the extension controller code in | |
| 21:38:32 | mriedem | https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/api-no-more-extensions.html | |
| 21:41:14 | efried | Είναι όλα ελληνικά για μένα | |
| 21:47:19 | mriedem | efried: btw this is where the auto_configure_disk magic happens https://github.com/openstack/nova/blob/master/nova/virt/xenapi/vm_utils.py#L929 | |
| 21:47:35 | mriedem | btw, setting independent_compute=True apparently is a super fun NoValidHost :) | |
| 21:48:46 | mriedem | so i assume any xen operator using that is setting up host aggregates with metadata to tie images with that property to them | |
| 21:54:30 | mriedem | http://git.openstack.org/cgit/openstack/glance/tree/etc/metadefs/compute-xenapi.json#n24 is actually a nice description | |
| 22:26:03 | dansmith | mriedem: dtantsur|afk: ironic job passes with that change, so I'll fix it up and hopefully we're good after that | |
| 22:30:00 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary https://review.openstack.org/473565 | |
| 22:30:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Make compute_node_statistics() work across cells https://review.openstack.org/473931 | |
| 22:31:54 | mriedem | dansmith: cool | |
| 22:32:06 | mriedem | i hope the commit message is super verbose and explanatory about all of the weirdness | |
| 22:32:15 | mriedem | dansmith: also, i think you'd agree with this, but need to check https://review.openstack.org/#/c/459593/26/nova/conductor/manager.py@532 | |
| 22:33:06 | dansmith | mriedem: yeah makes sense I think | |
| 22:33:10 | mriedem | awesome | |
| 22:43:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix misleading description in PUT /os-services/disable https://review.openstack.org/473997 | |
| 22:53:48 | mikal | . | |
| 22:57:02 | mriedem | .O. | |
| 22:59:54 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/472814 | |
| 23:06:38 | openstackgerrit | Merged openstack/nova master: libvirt: dump debug info when interface detach times out https://review.openstack.org/472377 | |
| 23:17:46 | mikal | mriedem: you peek over the wall more than me | |
| 23:52:10 | openstackgerrit | sean mooney proposed openstack/nova master: WIP set ethernet script to /bin/ture https://review.openstack.org/474006 | |
| #openstack-nova - 2017-06-14 | |||
| 00:35:03 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/472814 | |
| 01:31:21 | openstack | Launchpad bug 1697580 in OpenStack Compute (nova) "Cinder volumes not always attached to instance in order presented" [Undecided,New] | |
| 01:31:21 | mriedem | oh yay https://bugs.launchpad.net/nova/+bug/1697580 | |
| 01:31:34 | Kevin_Zheng | mriedem: do you have time to checkout my reply on https://review.openstack.org/#/c/459593/ , especially about [TagList] | |
| 01:32:56 | mriedem | Kevin_Zheng: block device mappings also have an instance_uuid field but we don't send those to conductor as a list | |
| 01:33:11 | mriedem | so something must be wrong in the api code that is requiring this to be different from how BDMs are handled | |
| 01:34:24 | Kevin_Zheng | hmm I will check | |
| 01:50:06 | Kevin_Zheng | mriedem: hmm, instance uuid for bdm was added in conductor to bdm obj, I will try if I can do it for TagObj | |
| 01:53:13 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: device tagging https://review.openstack.org/333781 | |
| 01:54:22 | mriedem | Kevin_Zheng: good lukc | |
| 01:54:24 | mriedem | *luck | |
| 01:54:41 | mriedem | i'm going to call it a day | |
| 01:54:55 | Kevin_Zheng | mriedem: have a nice sleep | |
| 01:55:20 | openstackgerrit | jimmygc proposed openstack/nova master: Remove Xen networking plugin https://review.openstack.org/474024 | |
| 02:04:29 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 02:09:06 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 02:43:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects https://review.openstack.org/466201 | |
| 02:48:43 | openstackgerrit | Hieu LE proposed openstack/nova master: remove mox from unit/virt/vmwareapi/test_driver_api.py https://review.openstack.org/452128 | |
| 03:28:08 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | |