| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-04-24 | |||
| 12:14:59 | openstackgerrit | jichenjc proposed openstack/nova master: Add description for /consoles https://review.openstack.org/459266 | |
| 12:25:45 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | |
| 12:43:26 | mriedem | lyarwood: i replied in https://review.openstack.org/#/c/456896/ - i'm not sure i get your concern though | |
| 12:45:08 | lyarwood | mriedem: morning, yeah I missed the optimisation, I just assumed we were doing this to allow non-local connectors to be passed in as a kwarg that we would default to None when we wanted to use the local connector. | |
| 12:49:14 | mriedem | lyarwood: when i looked at the change that were passing a connector in, none of them had a local connector | |
| 12:49:28 | mriedem | *was | |
| 12:52:12 | lyarwood | mriedem: into _terminate_volume_connections? Sorry I don't follow. | |
| 12:52:44 | mriedem | earlier version of the patch https://review.openstack.org/#/c/456896/3/nova/compute/manager.py@3615 | |
| 12:52:54 | mriedem | was going out of it's way to get the connector and then pass it in to justify the kwarg | |
| 12:52:56 | mriedem | i don't know why | |
| 12:53:17 | mriedem | so until we actually need a connector=None kwarg on _terminate_volume_connections, i've omitted that | |
| 12:53:47 | lyarwood | mriedem: yeah that didn't make sense but that's the local compute connector, thus my confusion above ;) | |
| 12:54:07 | lyarwood | mriedem: and yeah, until we have a stashed connector to pass in the kwarg doesn't make sense | |
| 12:54:32 | lyarwood | mriedem: I just missed the optimisation and assumed we wanted to add the kwarg early | |
| 12:55:04 | lyarwood | anyway, back to these LM failures on stable | |
| 12:55:26 | mriedem | lyarwood: so are you still -1 on https://review.openstack.org/#/c/456896/ ? | |
| 12:55:36 | lyarwood | mriedem: no, I'll clean that up now | |
| 12:56:05 | mriedem | thanks | |
| 12:58:14 | mriedem | jgriffith: when you're around, i had some questions marked with TODOs in the swap volume change for using the new cinder attachment APIs https://review.openstack.org/#/c/456971/4/nova/compute/manager.py | |
| 12:58:30 | mriedem | like, will deleting an attachment mess up migrate_volume_completion on the cinder side? | |
| 12:59:10 | mriedem | https://review.openstack.org/#/c/456971/4/nova/compute/manager.py@4993 and https://review.openstack.org/#/c/456971/4/nova/compute/manager.py@5017 specifically | |
| 13:00:29 | lyarwood | mdbooth: https://review.openstack.org/#/c/458807/ updated if you have anytime this afternoon | |
| 13:00:36 | lyarwood | kashyap: ^ also | |
| 13:00:55 | lyarwood | kashyap: could you also take a look at https://review.openstack.org/#/c/458854/1 | |
| 13:01:04 | kashyap | lyarwood: Struggling with something else at the moment :-( | |
| 13:01:13 | lyarwood | kashyap: np | |
| 13:01:13 | kashyap | lyarwood: But I didn't forget your ping here | |
| 13:02:10 | kashyap | lyarwood: Oh, the latter one I have the context in my head | |
| 13:02:13 | kashyap | Should be easy :-) | |
| 13:02:31 | kashyap | It's in 1.2.7; which is well above the min version we have. | |
| 13:02:34 | kashyap | Nice find! | |
| 13:06:34 | kashyap | lyarwood: Done for the second one; the 'rebase' failures are from an iSCSI-related bug? | |
| 13:07:16 | lyarwood | kashyap: yup, from the SELinux issue | |
| 13:07:18 | kashyap | Yeah, at least the linked bug says so | |
| 13:07:45 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | |
| 13:17:30 | lyarwood | http://logs.openstack.org/43/458843/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/145ed83/console.html#_2017-04-24_13_15_00_520308 - Can anyone comment on why we would want to skip block live migration tests here? | |
| 13:18:31 | lyarwood | ah nvm I think it's https://bugs.launchpad.net/nova/+bug/1398999 | |
| 13:18:31 | openstack | Launchpad bug 1398999 in nova (Ubuntu Trusty) "Block migrate with attached volumes copies volumes to themselves" [High,Triaged] | |
| 13:22:21 | jaypipes | cdent: thx much for the review on the get_inventory() for libvirt patch. I'll address your comments after a meeting in a few minutes. | |
| 13:23:08 | cdent | ✔ | |
| 13:23:33 | mriedem | lyarwood: we've never run those as far as i know | |
| 13:24:13 | mriedem | ah yeah because of that bug | |
| 13:24:19 | mriedem | in the config option help text | |
| 13:26:51 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for OVS plugin https://review.openstack.org/459258 | |
| 13:26:52 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: HW offload support for openvswitch https://review.openstack.org/398277 | |
| 13:26:53 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Add documentation for Linux Bridge plugin https://review.openstack.org/459291 | |
| 13:27:51 | mriedem | lyarwood: looks like that bug is fixed https://review.openstack.org/#/c/227278/ | |
| 13:27:56 | mriedem | if you're using new enough libvirt | |
| 13:28:09 | mriedem | so we could probably actually turn that on in master now since we're using UCA | |
| 13:28:34 | mriedem | we're using libvirt 2.5.0 now | |
| 13:29:40 | kashyap | lyarwood: Just looking at the first change related to 'rebase' failure & dangling devices; so the disconnect_volume() is coming from os_brick (os_brick/initiator/connectors/iscsi.py) | |
| 13:29:54 | lyarwood | mriedem: yeah just reading through the bug now and thinking the same | |
| 13:30:07 | mriedem | i'll post a change to the live migration job to enable that test and see what happens | |
| 13:30:20 | lyarwood | cool thanks | |
| 13:30:42 | lyarwood | kashyap: well it's calling down into os-brick to disconnect the new volume, yeah. | |
| 13:30:57 | Shashi | Hello, anyone run into nova issues with the error "AttributeError: 'module' object has no attribute 'VIR_MIGRATE_POSTCOPY'" | |
| 13:31:30 | Shashi | I could see n-cpu services are not running because of this error | |
| 13:31:31 | kashyap | lyarwood: Was just writing out loud for my own edification, please be a little gentle :-) | |
| 13:31:52 | kashyap | mriedem: About those QEMU assertion failures we were debugging on Friday night (for me at least), I brought it up on upstream QEMU list - http://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg03923.html | |
| 13:31:58 | Shashi | facing this issue on stable/ocata by the way | |
| 13:31:58 | jaypipes | Shashi: see /topic. best to ask usage questions on the openstack@ mailing list. But that sounds like you need a more recent version of libvirt. | |
| 13:32:36 | mriedem | Shashi: you have an old libvirt-python | |
| 13:32:36 | kashyap | mriedem: A Huwawei dev points out that they seem to have a downstream-specific patch that fixes for them | |
| 13:33:07 | kashyap | Anyway, I suggested if they want to post it as an upstream backport for 2.8 stable - http://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg04090.html | |
| 13:33:39 | Shashi | I understand if I remove exusting libvirt and install the new one should resolve this issue right? | |
| 13:35:09 | jaypipes | Shashi: the new python-libvirt package and a new libvirt package, yes. | |
| 13:35:38 | Shashi | Thanks jaypipes | |
| 13:35:47 | jaypipes | np | |
| 13:36:43 | rfolco | Shashi, and make sure you clean up your wheels (.whl) for libvirt-python, force a rebuild. | |
| 13:37:23 | Shashi | ok rfolco | |
| 13:45:29 | figleaf | Scheduler subteam meeting in 15 minutes in #openstack-meeting-alt | |
| 13:45:41 | figleaf | oh, wait, it's not Friday anymore | |
| 13:46:15 | Tahvok | Guys, how should I configure the apache2 placement vhost? | |
| 13:46:23 | Tahvok | Can someone give me a working example? | |
| 13:55:13 | efried | Tahvok I think you may want to ask that question in #openstack | |
| 13:57:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Enable test_iscsi_volume in live migration job https://review.openstack.org/459316 | |
| 13:57:22 | mriedem | tdurakov: lyarwood: ^ let's see what happens | |
| 14:00:57 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Enable test_iscsi_volume in live migration job https://review.openstack.org/459316 | |
| 14:01:21 | mriedem | kashyap: which version of qemu has native iscsi support? | |
| 14:01:39 | mriedem | would be interesting to see if we can test against https://bugs.launchpad.net/nova/+bug/1524898 again | |
| 14:01:39 | openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | |
| 14:01:40 | kashyap | mriedem: I don't know top off my head, let me look | |
| 14:02:06 | bhagyashris | cdent: Hi, actually I am willing or curious to track the nfs share as a resource provider and manage its inventory with my own tooling as per you suggest me so How can I proceed Can you please give me some guide line or steps to do this? | |
| 14:04:46 | cdent | bhagyashris: I can show you how it is described in the spec, but you'll need some tooling of your own, and the information won't be very useful for two reasons: | |
| 14:05:05 | cdent | nothing will be using the info for scheduling, and nothing will be automatically writing allocations when disk is used | |
| 14:05:33 | kashyap | mriedem: If you're talking about the built-in iSCSI driver, it's there since "eons" -- version 1.0 (as the first commit dates to 2011) -- http://git.qemu.org/?p=qemu.git;a=commitdiff;h=c589b24 | |
| 14:06:00 | cdent | bhagyashris: some of the info here may be useful: http://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/generic-resource-pools.html#scenario-1-shared-disk-storage-used-for-vm-disk-images | |
| 14:06:29 | bhagyashris | cdent:ok. | |
| 14:06:29 | mriedem | kashyap: it wasn't in the ubuntu package though | |
| 14:06:39 | mriedem | kashyap: https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/qemu-built-in-iscsi-initiator.html | |
| 14:06:59 | mriedem | looks like it should be there now though http://packages.ubuntu.com/xenial/qemu-block-extra | |
| 14:07:22 | bhagyashris | cdent: yeah I have checked the generic-resource-pool spec long back ago. | |
| 14:07:51 | kashyap | mriedem: What is this 'qemu-block-extra'? | |
| 14:08:01 | kashyap | It's a separate package for Ubuntu? | |
| 14:08:09 | mriedem | yeah | |
| 14:08:10 | mriedem | i guess | |
| 14:08:25 | bhagyashris | cdent: thanks for information | |
| 14:08:36 | mriedem | kashyap: see https://bugs.launchpad.net/nova/+bug/1524898/comments/17 | |
| 14:08:37 | openstack | Launchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed] | |
| 14:09:19 | mriedem | kashyap: long ago i think danpb said that might not fix that bug, but it might provide better details when it fails if we're using native iscsi | |