| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-19 | |||
| 09:13:06 | gibi | bauzas: thanks for the baggage heads up, I have to check on that | |
| 09:13:12 | amorin | http://logs.openstack.org/40/508640/6/check/legacy-tempest-dsvm-cells/5c83769/job-output.txt.gz | |
| 09:13:19 | amorin | ERROR: the main setup script run by this job failed - exit code: 137 | |
| 09:13:20 | gibi | bauzas: if you are super rich you can have private jet ;) | |
| 09:13:32 | bauzas | amorin: there are some gate timeout issues AFAICT | |
| 09:13:47 | bauzas | gibi: my company travel policy forbids it | |
| 09:13:48 | bauzas | :p | |
| 09:13:56 | amorin | bauzas: thanks | |
| 09:14:07 | amorin | should I wait or can I recheck now> | |
| 09:14:10 | amorin | ? | |
| 09:14:20 | alex_xu | jaypipes: gibi bauzas, hi guys, the API patch for expose traits https://review.openstack.org/#/c/535642/, appreciate if you can take a look at | |
| 09:14:35 | bauzas | gibi: but I'm living litterally 500 meters away from a very small air field | |
| 09:14:58 | bauzas | gibi: I have as a wish to pass my private pilot license :) | |
| 09:15:25 | bauzas | amorin: which change ? | |
| 09:15:42 | bauzas | alex_xu: sure, I'll mostly focus on priorities | |
| 09:15:56 | bauzas | as gibi +Wd some of my changes :p | |
| 09:16:01 | alex_xu | bauzas: thanks | |
| 09:16:43 | bauzas | oh fuuuuuuu*** I need to rebadse | |
| 09:17:16 | Spazmotic | Zuuuuuuuuuuuul! heh | |
| 09:17:41 | gibi | bauzas: having a pilot license, that sounds a cool plan | |
| 09:18:00 | ildikov | bauzas: morning, do you plan to get through the multi-attach patches today too? :) | |
| 09:18:04 | bauzas | gibi: that's just super expensive :p | |
| 09:18:17 | gibi | bauzas: I can imagine | |
| 09:18:41 | bauzas | ildikov: today is like a busy day for me, but as soon as I can tackle internal stuff (again) and rebase on my own changes, I'll focus on priority changes reviews | |
| 09:19:26 | amorin | bauzas: https://review.openstack.org/#/c/508640/ | |
| 09:22:26 | bauzas | amorin: I've seen that a couple of times | |
| 09:22:47 | bauzas | amorin: the fact that we stop running tempest if we are too much delayed | |
| 09:22:52 | bauzas | amorin: try a recheck | |
| 09:23:06 | amorin | ok thanks | |
| 09:24:12 | ildikov | bauzas: if you could put this on top of the pile that would be great as I travel on Monday and the gate will be crazy next week too, so I would like to be able to fix comments in case needed before that | |
| 09:24:22 | ildikov | bauzas: thanks in advance | |
| 09:26:05 | bauzas | ildikov: no promise, but I'll try | |
| 09:26:22 | Spazmotic | I would love to get my Xenapi piddly poop in the gate this week but I definately don't see it happening hehe. So will just learn Placements code and maybe fix this local delete bug | |
| 09:27:00 | ildikov | bauzas: thanks | |
| 09:27:28 | bauzas | Spazmotic: bugs are not impacted by the queens-3 deadline | |
| 09:27:45 | Spazmotic | Well I just mean because the core reviewers are so busy, sir :) | |
| 09:31:19 | bauzas | Spazmotic: I'm French, we did cut the head of all our lords in the past, so you don't need to call me "sir" | |
| 09:31:28 | bauzas | I'm neither too old nor too lordy | |
| 09:32:07 | Spazmotic | Hehe it's a respect thing, I'll do my best. | |
| 09:47:40 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | |
| 09:47:40 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | |
| 09:47:41 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Fix suspending guest with attached vGPUs https://review.openstack.org/535693 | |
| 09:47:41 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: pass the mdevs when rebooting the guest https://review.openstack.org/533818 | |
| 09:48:15 | Spazmotic | Ah yeah I meant to tell you those hit merge conflictr | |
| 09:48:17 | Spazmotic | Sorry I Forgot | |
| 09:53:11 | bauzas | gibi: need again your +W, I needed to rebase https://review.openstack.org/#/c/528832/ | |
| 09:53:17 | bauzas | (merge conflict) | |
| 09:54:01 | bauzas | gibi: also, I just rebased https://review.openstack.org/#/c/533642 but it lost your +2 due to a merge solve | |
| 09:57:00 | lyarwood | mdbooth: https://review.openstack.org/#/c/523958/11/nova/tests/unit/virt/libvirt/test_driver.py@6604 - remind me again what you mean by autospec here? | |
| 09:57:26 | mdbooth | lyarwood: IIRC I wasn't overly exercised by that as we don't do it consistently | |
| 09:57:41 | mdbooth | However, I was just wondering if you wanted to mock the class with an autospec | |
| 09:57:53 | mdbooth | Which you've done in a few other places, and is generally awesome | |
| 09:57:54 | vish_18 | frickler: hello | |
| 09:58:17 | vish_18 | frickler: https://bugs.launchpad.net/keystone/+bug/1714937. i am not able to reproduce this issue on Pike | |
| 09:58:18 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: support traits in allocation candidates API https://review.openstack.org/535642 | |
| 09:58:18 | openstack | Launchpad bug 1714937 in OpenStack Identity (keystone) "keystone returns 500 on password change" [Low,Confirmed] - Assigned to Vishakha Agarwal (vishakha.agarwal) | |
| 09:59:34 | gibi | bauzas: I will check those soon | |
| 09:59:38 | lyarwood | mdbooth: kk, can you actually do that with @mock.patch directly? | |
| 09:59:41 | vish_18 | frickler: kindly help me to reproduce | |
| 10:00:12 | mdbooth | lyarwood: I think so, but as I say it wasn't a top priority to me | |
| 10:00:21 | bauzas | gibi: cool thanks | |
| 10:00:37 | bauzas | gibi: oh, fuuuuu, did you run a nova meeting yesterday evening ? | |
| 10:00:45 | bauzas | totally forgot it | |
| 10:01:47 | mdbooth | Over my dead body > problem > would prefer > nit > suggestion | |
| 10:02:13 | lyarwood | mdbooth: haha :) | |
| 10:03:05 | lyarwood | mdbooth: cool, so I'm obviously sorting the tests out this morning, I've given up on the P to Q LM tests for now, I think we can add them to the legacy-grenade-dsvm-neutron-multinode-live-migration pretty easily, just can't get grenade to play nice with f26 at the moment | |
| 10:03:18 | gibi | bauzas: no I didn't but I think efried did | |
| 10:03:27 | bauzas | k | |
| 10:03:35 | bauzas | will look at the minutes then | |
| 10:03:44 | mdbooth | lyarwood: Yeah. I wanted to do the tests for you yesterday but got unexpectedly bogged down. Sorry about that. | |
| 10:03:52 | lyarwood | mdbooth: np | |
| 10:11:08 | openstackgerrit | Deepak Mourya proposed openstack/nova master: Handle TZ change in iso8601 >=1.12.0 https://review.openstack.org/535700 | |
| 10:27:24 | gmann | vish_18: better to ask on keystone channel. | |
| 11:41:05 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/535030 | |
| 11:41:20 | openstackgerrit | Merged openstack/nova master: conf: Remove 'vendordata_driver' opt https://review.openstack.org/397835 | |
| 11:47:35 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for encrypted volumes https://review.openstack.org/523958 | |
| 11:47:57 | lyarwood | mdbooth, stephenfin; ^ if you have time, should be almost ready to go now | |
| 11:48:21 | mdbooth | lyarwood: Looking now | |
| 11:49:44 | lyarwood | hmmm merge conflict, let me rebase the series | |
| 11:50:43 | lyarwood | oh nice, the multi-attach stuff landed overnight :) | |
| 11:52:34 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Collocate encryptor and volume driver calls https://review.openstack.org/460243 | |
| 11:52:35 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for encrypted volumes https://review.openstack.org/523958 | |
| 11:52:35 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Introduce disk encryption config classes https://review.openstack.org/464008 | |
| 11:52:44 | lyarwood | aaaaaaaaaand done. | |
| 11:57:08 | Spazmotic | There are so few ways to clean up allocations in the current Placement API to a point that i think i'm missing something here | |
| 11:58:57 | Spazmotic | How in the world are single allocations cleaned up.. | |
| 12:02:13 | Spazmotic | ew.. | |
| 12:09:08 | Spazmotic | Ew so it's just zeroing out the allocations | |
| 12:12:34 | Spazmotic | This can't be the openstack way to handle this.. | |
| 12:13:19 | Spazmotic | Allocations are not using the updated_at and deleted_at standards and in order to update an allocation you have to pass the entire list of their current allocations plus changes into the JSON because it purges the entire list for a "Clean slate"? | |
| 12:13:46 | Spazmotic | The only other place i've seen this is how neutron handles Static IP addresses in Ports which isn't so bad since it's an internal dictionary.. not database sets. | |
| 12:14:10 | Spazmotic | This must be ridiculously unwieldy for large consumers.. not to mention dangerous. | |
| 12:15:00 | Spazmotic | And I don't understand why ad esire for a clean slate every time a minor new allocation wants to be set for a consumer. | |
| 12:15:06 | Spazmotic | efried, can you shine any light on this? | |
| 12:18:46 | Spazmotic | This local delete issue.. the best way I can see to resolve this is to allow a new API call to empty a resource provider, but the way that allocations are being handled is incredibly gross and would make that process extremely top heavy. Each alloc per RP would have to pull the consumer, then pull their allocs and reconstruct the allocs without the chosen alloc and then submit the entire list. | |
| 12:24:47 | ttallgren | Hi all, I hit a bug in Nova 17.0.0.1b2 with automated Ansible testing (OPNFV XCI): http://p.ip.fi/a733 | |
| 12:26:05 | rabel | hi there. there seems to be a problem with building the docs in nova. running tox -e docs i get an error message: | |
| 12:26:10 | ttallgren | It is calling nova-manage cell_v2 discover_hosts and the final error is Duplicate entry 'compute01' for key 'uniq_host_mappings0host | |
| 12:26:18 | rabel | nova/nova/policies/config_drive.py", line 46, in <module> deprecated_since='17.0.0'),TypeError: __init__() got an unexpected keyword argument 'deprecated_since' | |
| 12:31:37 | frickler | vish_18: commented on the bug report. also lyarwood is right, this is a keystone issue | |