| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-20 | |||
| 20:09:19 | dims | mriedem :) | |
| 20:09:46 | dims | sdague : gate is 39 deep, something got busted? | |
| 20:10:02 | sdague | dims: as far as I can, it's just a ton of paper cuts | |
| 20:10:20 | sdague | nova notification tests are sometimes failing | |
| 20:10:23 | dims | ouch ok. at least we have weekend to catch up on | |
| 20:10:26 | sdague | I saw some db things failing on cinder | |
| 20:10:31 | dims | will watch a bit as well | |
| 20:10:42 | sdague | the keystone os-api-ref one, which I pushed a fix (and email on) | |
| 20:11:29 | dims | ack y i saw that. thanks sdague | |
| 20:14:50 | dims | ugh. just spotted this in the gate queue - http://logs.openstack.org/11/423111/2/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/60c1d57/logs/screen-n-cpu.txt.gz?level=ERROR#_2017-01-20_19_36_46_607 | |
| 20:15:00 | dims | libvirtError: internal error: client socket is closed | |
| 20:16:42 | mriedem | dims: that's known | |
| 20:16:50 | mriedem | libvirt crashes intermittently on xenial | |
| 20:17:10 | mriedem | http://status.openstack.org//elastic-recheck/index.html#1643911 | |
| 20:17:18 | mriedem | http://status.openstack.org//elastic-recheck/index.html#1646779 | |
| 20:17:25 | mriedem | http://status.openstack.org//elastic-recheck/index.html#1638982 | |
| 20:18:36 | dims | ack mriedem | |
| 20:21:54 | mriedem | jroll: vdrok: sambetts|afk: can i mark this as complete too? https://blueprints.launchpad.net/nova/+spec/ironic-plug-unplug-vifs-update | |
| 20:22:00 | mriedem | the vifs plug/unplug patch is merged, | |
| 20:22:05 | mriedem | the interface attach/detach thing isn't | |
| 20:22:21 | jroll | mriedem: yes, that doesn't mention the latter, that should be a separate blueprint imo | |
| 20:22:24 | mriedem | which is https://review.openstack.org/#/c/419975/ | |
| 20:22:26 | mriedem | ok | |
| 20:22:27 | mriedem | cool | |
| 20:23:01 | mriedem | nice both of those bps are merged then (portgroups and vifs update) | |
| 20:28:35 | bauzas | oh man, the gate... | |
| 20:37:26 | mriedem | ew we unconditionally run osprofiler on all unit test runs | |
| 20:37:30 | mriedem | even if i just want to test a specific path | |
| 20:50:40 | openstackgerrit | Julian proposed openstack/nova: Create cmd/script that prints JSON Schema of Notification Payload Classes https://review.openstack.org/411836 | |
| 20:52:30 | openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | |
| 20:52:55 | mriedem | sean-k-mooney: leakypipes: i'm happy with ^ now | |
| 20:53:53 | mriedem | actually, hmm | |
| 20:54:18 | leakypipes | mriedem: need to boot into Windows VM... biab. (yep, Webex calls) | |
| 21:00:58 | bauzas | mriedem: sdague: I'm currently writing a patch for adding the microversion header in the report client | |
| 21:01:12 | bauzas | mriedem: sdague: do you think we should default to the min version or the max one ? | |
| 21:01:18 | bauzas | I think the latter | |
| 21:01:39 | bauzas | but maybe lemme upload the change | |
| 21:01:49 | sdague | bauzas: I thought we always default to min unless specified | |
| 21:02:03 | bauzas | sdague: I first thought about that | |
| 21:02:46 | bauzas | sdague: okay, lemme try to do this | |
| 21:02:56 | bauzas | and upload it so we can disucss | |
| 21:08:26 | mriedem | bauzas: i put up something like that last night | |
| 21:09:03 | openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | |
| 21:09:09 | mriedem | bauzas: in here https://review.openstack.org/#/c/407309/ | |
| 21:09:17 | mriedem | that requests 1.1 specifically for the aggregates GET it needs | |
| 21:09:28 | mriedem | which may fail, but then it's just that aggregates GET that fails and not everything else | |
| 21:09:35 | bauzas | mriedem: :( | |
| 21:09:44 | mriedem | the scheduler report client is currently not doing version discovery | |
| 21:09:51 | bauzas | lemme upload it | |
| 21:11:50 | mriedem | sfinucan: +2 on https://review.openstack.org/#/c/385061/ | |
| 21:15:31 | openstackgerrit | Matt Riedemann proposed openstack/nova: Ironic: Add soft reboot support to ironic driver https://review.openstack.org/403745 | |
| 21:17:57 | bauzas | mriedem: KABOOOM | |
| 21:18:08 | bauzas | mriedem: my patch is trampled by the above ^ | |
| 21:18:17 | bauzas | mriedem: so I'll remove the get header | |
| 21:19:04 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | |
| 21:29:51 | mriedem | https://review.openstack.org/#/c/403745/ also looks good to go | |
| 21:30:30 | slaweq_ | oomichi: hello | |
| 21:30:54 | slaweq_ | oomichi: some time ago You reviewed my small patch https://review.openstack.org/#/c/420176/ | |
| 21:31:15 | slaweq_ | oomichi: I answered to Your comment there, can You maybe have a look on it? | |
| 21:31:19 | slaweq_ | oomichi: thx in advance | |
| 21:32:47 | mriedem | claudiub: you're going to have to sync up with markus_z or someone from s390 on https://review.openstack.org/#/c/420960/ else we'll just have to move that forward without the s390 features | |
| 21:37:05 | mriedem | melwitt: booya http://logs.openstack.org/05/423005/6/check/gate-grenade-dsvm-neutron-ubuntu-xenial/90ab2f2/logs/devstack-gate-discover-hosts.txt.gz | |
| 21:37:59 | mriedem | the stable jobs are still failing, looks like they are still trying to run discover_hosts, | |
| 21:38:05 | mriedem | http://logs.openstack.org/05/423005/6/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton-nv/39d0108/logs/devstack-gate-discover-hosts.txt.gz | |
| 21:38:21 | mriedem | i wonder if the "test -f discover_hosts.sh" is making it exit with 1 | |
| 21:39:00 | mriedem | user@xubuntu:~/git/devstack$ cd ~/git/devstack && test -f discover_hosts1.sh && sudo -H -u user stdbuf -oL -eL cat discover_hosts.sh | |
| 21:39:00 | mriedem | i think it is | |
| 21:39:01 | mriedem | 1 | |
| 21:39:01 | mriedem | user@xubuntu:~/git/devstack$ echo $? | |
| 21:40:41 | mriedem | which means i basically can't avoid branch logic in d-g for this | |
| 21:44:24 | mriedem | actually nvm | |
| 21:44:28 | mriedem | || true to the rescue | |
| 21:50:06 | bauzas | I'm soooo f*** up | |
| 21:50:16 | bauzas | the gate is horribly late | |
| 21:50:34 | _gryf | mriedem, you might be interested in closing this bug https://review.openstack.org/#/c/422780/ | |
| 21:50:46 | bauzas | and I can't upload new changes unless resetting my current ones in the gate | |
| 21:53:32 | mriedem | _gryf: commented | |
| 21:53:37 | mriedem | bauzas: -R | |
| 21:53:39 | mriedem | git review -R | |
| 21:53:49 | bauzas | mriedem: yup I know | |
| 21:54:02 | bauzas | mriedem: but I want to rebase based on the one you merged | |
| 21:54:17 | bauzas | mriedem: you know, the one having the microversion in get() | |
| 21:54:51 | mriedem | oh | |
| 21:55:09 | mriedem | well, you can always work the weekend :) unless your govt has outlawed it | |
| 21:55:37 | mriedem | just put on some loverboy | |
| 21:55:49 | mriedem | https://www.youtube.com/watch?v=zL8G5pBZ5CI | |
| 21:56:27 | _gryf | mriedem, oh shit. i'll cover that. | |
| 21:56:59 | bauzas | mriedem: well, I'm not trumped by my govt :p | |
| 21:57:10 | mriedem | i <3 trump | |
| 21:57:10 | _gryf | i mean "ow snap" or smth like this | |
| 21:57:17 | _gryf | ;P | |
| 21:57:58 | bauzas | mriedem: I mean, I'm working now because I could see a wall next week between you and me | |
| 21:59:50 | stewie925_ | hi all, I am fix a merge conflict for one of my nova notificaion commits. After rebasing I ran the functional test and it shows the following erro: No authentication information found for placement API. Placement is optional in Newton, but required in Ocata. Please enable the placement service before upgrading. | |
| 22:02:33 | stewie925_ | oops - think I may have found something... brb | |
| 22:05:39 | melwitt | mriedem: a-ha, coolness on the d-g change | |
| 22:08:21 | mriedem | melwitt: i'm told in -infra that discover_hosts needs to be moved into post-stack for grenade | |
| 22:08:30 | mriedem | but that can happen after we see if this d-g run is good | |
| 22:08:38 | melwitt | okay | |
| 22:08:39 | mriedem | and then either you or dansmith will have to do that | |
| 22:08:43 | mriedem | bauzas: i'll have email. | |