| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-20 | |||
| 20:08:59 | sdague | the permission changing is done for the stack user right before the stack.sh hand off. It starts as being under the jenkins user. The stack user can itself sudo up until this point - https://github.com/openstack-infra/devstack-gate/blob/5883a8269a741d77c1853150c56d6d90fe01b83e/devstack-vm-gate.sh#L720-L724 | |
| 20:09:16 | sdague | it's sudo privs are taken away before tempest runs, so that tempest doesn't cheat | |
| 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 | i think it is | |
| 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:01 | mriedem | user@xubuntu:~/git/devstack$ echo $? | |
| 21:39:01 | mriedem | 1 | |
| 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 | _gryf | i mean "ow snap" or smth like this | |
| 21:57:10 | mriedem | i <3 trump | |
| 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 | |