| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-16 | |||
| 07:11:05 | ratailor | alex_xu, it gets stuck in merge conflict every now and than. | |
| 08:35:36 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: reno for notification-transformation-queens https://review.openstack.org/518018 | |
| 08:48:07 | gibi | cores, I think this is a fix for a problematic master stability issue, could you check it? https://review.openstack.org/#/c/533210/ | |
| 08:55:28 | gmann | gibi: +1, if you want you can optimism it more. | |
| 08:56:17 | ratailor | Can anybody provide steps to create live-migration setup ? | |
| 08:56:43 | ratailor | I have created with one all-in-one controller and 1 compute node, and setup ssh keys as mentioned here. https://docs.openstack.org/devstack/latest/guides/multinode-lab.html#live-migration | |
| 08:57:09 | ratailor | But I couldn't live-migrate vm from one node to other. | |
| 08:58:12 | openstackgerrit | Merged openstack/nova master: Transform rescue/unrescue instance notifications https://review.openstack.org/385644 | |
| 09:00:20 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 09:03:52 | gibi | gmann: thanks, I replied inline | |
| 09:08:41 | gmann | gibi: is it? irrelevant-files override should work in inherited jobs. | |
| 09:09:17 | gibi | gmann: it doesn't work. Let me dig out the infra log | |
| 09:09:31 | gmann | i am trying to push irrelevant-files on base jobs like legacy dvsm etc which can be issue then | |
| 09:10:06 | gibi | gmann: discussion starts here http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2018-01-12.log.html#t2018-01-12T14:50:01 | |
| 09:10:07 | gmann | i have not tired the overridden scenario but with discussion with clark long back it should. | |
| 09:13:41 | ratailor | I am getting below error in n-cpu logs: | |
| 09:13:42 | ratailor | Jan 16 07:42:16 devstack-compute nova-compute: libvirtError: internal error: unable to execute QEMU command 'drive-mirror': Failed to connect socket: Connection timed out | |
| 09:13:42 | ratailor | Jan 16 07:42:16 devstack-compute nova-compute: if ret == -1: raise libvirtError ('virDomainMigrateToURI3() failed', dom=self) | |
| 09:13:42 | ratailor | Jan 16 07:42:16 devstack-compute nova-compute: File "/usr/lib64/python2.7/site-packages/libvirt.py", line 1702, in migrateToURI3 | |
| 09:14:05 | kashyap | ratailor: Can you successfully repeat it? | |
| 09:14:10 | kashyap | Is it with today's Git master? | |
| 09:14:48 | kashyap | ratailor: That error means, QEMU has quit. | |
| 09:16:53 | gmann | gibi: i think that issue was different where auto converted job definition caused that bug. irrelevant-files is defined as regex text in zuulv3 and it override among job inheritance. | |
| 09:18:07 | openstackgerrit | Michał Dulko proposed openstack/os-vif master: Revert "Move 'ips' field from Subnet object to VIF object" https://review.openstack.org/534206 | |
| 09:18:09 | kashyap | ratailor: You can see what happens in the libvirtd.log | |
| 09:18:44 | gibi | gmann: OK, I will try what you suggest to see if in the in tree definition the override works | |
| 09:20:45 | gmann | gibi: thanks, i am trying to fill base job with basic irrelevant-files so that they can be overridden if needed like this- https://review.openstack.org/#/c/531194/ | |
| 09:25:03 | ratailor | kashyap, let me try it again. | |
| 09:32:04 | ildikov | johnthetubaguy: hi | |
| 09:32:43 | ildikov | johnthetubaguy: if you could go through the multi-attach series rather the first half of this week that would be great so I can fix things before the gate gets impossible | |
| 09:33:11 | ildikov | johnthetubaguy: series starts here: https://review.openstack.org/#/c/267587/ | |
| 09:33:33 | ildikov | johnthetubaguy: experimental job: https://review.openstack.org/#/c/532689/ | |
| 09:33:45 | ildikov | johnthetubaguy: the multi-attach tests ran fine | |
| 09:37:11 | ildikov | gibi: same pretty please goes to you for this one: https://review.openstack.org/#/c/271047/ :) | |
| 09:38:55 | kashyap | mdbooth: I don't suppose this ever happened, did it? http://lists.openstack.org/pipermail/openstack-operators/2017-January/012540.html | |
| 09:39:07 | kashyap | mdbooth: It is your proposal to remove "support for sparse LVM volumes from Nova" | |
| 09:39:47 | mdbooth | kashyap: I think that was in the context of work which didn't land, so no | |
| 09:39:52 | kashyap | I see. | |
| 09:51:24 | openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | |
| 09:51:27 | openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | |
| 09:51:28 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 10:05:33 | bhujay | Hello Everybody , any idea how to configure nova libvirt section for a FC shared storage instead of rbd | |
| 10:13:45 | openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | |
| 10:14:51 | openstackgerrit | Jacek Tomasiak proposed openstack/python-novaclient master: Fix listing of instances above API max_limit https://review.openstack.org/534222 | |
| 10:17:30 | openstackgerrit | Chris Dent proposed openstack/nova master: Enable py36 unit tests in tox https://review.openstack.org/534223 | |
| 10:20:33 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | |
| 10:20:33 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | |
| 10:27:44 | cdent | gibi: if you're around and have a moment, I could do with your input on https://review.openstack.org/#/c/533797/ It's passing all functional tests except for 8 in 'notification_sample_tests' and the cause is not immediately obvious (the zuul runs are out of date) | |
| 10:32:34 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | |
| 10:37:58 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | |
| 10:38:15 | sean-k-mooney | stephenfin: you about? | |
| 11:05:01 | ameeda | Hi, Regarding this gerrit "https://review.openstack.org/#/c/528385/", when I run the functional test i got this kind of failurs "http://paste.openstack.org/show/645697/" , can you please guess me what I can do with ? | |
| 11:05:50 | stephenfin | sean-k-mooney: I am | |
| 11:07:17 | sean-k-mooney | stephenfin: i pushed a patch for the nova unit tests but i looks like pyrote2 may leak on python2 based on matthew thode's comment on https://review.openstack.org/#/c/532987/ | |
| 11:07:31 | sean-k-mooney | https://github.com/svinota/pyroute2/pull/443 | |
| 11:08:39 | openstackgerrit | nafeiyang proposed openstack/nova master: z/VM driver: add snapshot function https://review.openstack.org/534240 | |
| 11:08:44 | stephenfin | sean-k-mooney: So that explains our "too many open files issue", I guess | |
| 11:09:09 | sean-k-mooney | since rodolfo like to test thing when he wrote the pyroute 2 implementation he almost wrote a ip tools impl in the functional tests https://github.com/openstack/os-vif/blob/master/os_vif/tests/functional/internal/command/ip/test_impl_pyroute2.py#L28-L94 | |
| 11:09:38 | stephenfin | sean-k-mooney: So I guess we can revert/disable the pyroute2 path until a new version is released | |
| 11:09:43 | sean-k-mooney | stephenfin: what i was thing was i could make that an actully conforming implementation to the api and we could merge and make it the defalut | |
| 11:09:59 | stephenfin | We can't rely on that being released before the deadline | |
| 11:10:58 | sean-k-mooney | well i was going to do it now. if i dont have it finished and up for review by this evening ill submit a reviert for rodolfos patch | |
| 11:11:11 | stephenfin | sean-k-mooney: But why not keep what we had, in that case? The reason to switch to pyroute2 was that it used the netlink protocol in Python, rather than wrapping existing command-line tools | |
| 11:11:30 | stephenfin | Before integrating pyroute2, we _were_ using subprocess | |
| 11:11:55 | stephenfin | Well, processutils | |
| 11:12:05 | openstackgerrit | Merged openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556 | |
| 11:12:25 | sean-k-mooney | yes though really the api is 3 fucntion add device delete device and set properties on device so its really small | |
| 11:13:48 | stephenfin | sean-k-mooney: Cool. If you get it finished then let me know and I'll review | |
| 11:14:19 | stephenfin | We can mark the processutils variant as immediately deprecated and switch to pyroute2 once a new version is released | |
| 11:14:38 | sean-k-mooney | give me an hour or so and i should have a second driver working and a config option that default to iptools instead of pyroute 2 done. ill ping you the review when its up | |
| 11:14:58 | gibi | cdent: I will check it later today | |
| 11:15:01 | sean-k-mooney | stephenfin: ya that was my reasoning too | |
| 11:15:06 | cdent | thanks gibi | |
| 11:15:25 | cdent | If I have it figured out before then, I'll let you know | |
| 11:20:02 | openstackgerrit | Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | |
| 11:23:23 | cdent | stephenfin: see my second set of comments on https://review.openstack.org/#/c/534223/ I don't think your suggestion works (at least not for me) | |
| 11:25:32 | stephenfin | Oh, maybe not | |
| 11:27:45 | stephenfin | cdent: Right you are. I retract my -1 | |
| 11:28:06 | cdent | stephenfin: do you still think I should change the default list? | |
| 11:28:30 | stephenfin | cdent: I don't think so. We haven't switched to py36 in CIs yet (I think?), so it doesn't need to run by default | |
| 11:34:28 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Refactor WSGI apps and utils to limit imports https://review.openstack.org/533797 | |
| 11:34:28 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] use simple FaultWrapper https://review.openstack.org/533752 | |
| 11:36:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 11:38:42 | mdbooth | stephenfin: Please could you kick this back into the gate for me? https://review.openstack.org/#/c/529028/ Previously failed, then rebased. | |
| 11:38:56 | stephenfin | mdbooth: Done | |
| 11:39:04 | mdbooth | stephenfin: Thanks | |
| 11:40:05 | mdbooth | Hmm, that patch series is starting to descend into merge conflict hell. | |
| 12:13:51 | mdbooth | How do I hate thee? Let me count the ways. | |
| 12:14:17 | mdbooth | Actually, I'm out of fingers :/ | |
| 12:15:08 | lyarwood | mdbooth: it did | |
| 12:15:11 | lyarwood | <https://matrix.devlab.redhat.com/_matrix/media/v1/download/redhat.com/ZOkiHajvlKvFxQUcknOXyfhJ> | |
| 12:15:11 | lyarwood | * mdbooth sent a long message: mdbooth_2018-01-16_12:13:39.txt | |
| 12:15:23 | mdbooth | lyarwood: Hehe, yeah. | |
| 12:17:03 | mdbooth | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7519 | |
| 12:17:09 | mdbooth | There down to... | |
| 12:17:12 | mdbooth | except Exception: pass | |
| 12:17:17 | mdbooth | Yes, really | |
| 12:17:56 | mdbooth | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7519-L7530 | |
| 12:21:38 | openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | |
| 12:36:05 | cdent | Does anyone recall if nova/objects/ec2.py is used any more. It seems only in tests | |