Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-20
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.
22:08:50 mriedem bauzas: and there are other cores around next week too
22:09:06 bauzas mriedem: np, enjoy your time-off
22:09:24 bauzas mriedem: I'm just done with trying to fix all the placement issues
22:09:48 mriedem i'm not sure what that means
22:09:58 mriedem i thought you just needed to pass in the 1.4 microversion
22:10:46 stewie925_ thought I found something with the placement api error ... but I didnt :(
22:12:04 bauzas mriedem: I *could* do that easily now with the direct get call, but I prefer to update the ReportClient by having a min and max versions
22:13:22 bauzas I'd have actually by far preferred we hadn't gone that way with providing directly the header in the get method but meh.
22:15:08 melwitt mriedem: so for non-grenade, it stays where it is, for grenade it moves right after the run of stack.sh
22:16:03 melwitt or does this mean it moves out of d-g for grenade and into the grenade repo?
22:16:20 sdague melwitt: so... it could be either honestly
22:16:48 melwitt okay. just making sure I understand what to do if I have to do it
22:16:49 sdague it needs to happen after the post-stack.sh run
22:17:14 sdague that's where d-g puts the subnode orchestration commands
22:18:20 sdague https://github.com/openstack-dev/grenade/blob/489202bd707866de9abc25a46a3f31c10e939076/grenade.sh#L244
22:18:51 sdague either way, a NOTE should be added about this being a work around for ocata until there is some auto registration for the single node case
22:18:56 sdague sorry, single cell case
22:19:10 melwitt I had just now found that, thanks. I think I understand now
22:21:21 sdague cool, ok off to dinner, have a good weekend folks
22:22:18 mriedem bauzas: yeah version discovery is obviously better
22:22:20 mriedem things are just tight
22:22:32 mriedem bauzas: so maybe work with sdague and jaypipes on the best way forward there on monday
22:22:47 mriedem bauzas: we're pretty close though and we've already said you need placement for n-sch in ocata,
22:22:53 mriedem so it's going to have to be there,
22:23:18 mriedem so i think requesting 1.4 and handling the error if it's not at 1.4 (newton placement) with an ERROR and NoValidHost is expected behavior given the ML discussion
22:23:29 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
22:23:54 bauzas mriedem: fair point
22:24:20 bauzas mriedem: given the gate, maybe just directly calling 1.4 is the quickiest one
22:24:24 mriedem bauzas: which reminds me that we're going to need a docs update in addition to reno in your change, or immediately following it,
22:24:45 mriedem i.e. we need to update the upgrade docs devref to make a note that placement is required for n-sch in ocata, and should be upgraded first or at the same time
22:24:53 bauzas roger
22:24:54 mriedem bauzas: yes i think we do 1.4
22:34:20 melwitt mriedem: is your latest comment saying we might not need discover_hosts at all for multi-node grenade?
22:36:01 aunnam stewie925_, I looked at your patch, by looking at the logs I think the placement api is not the problem. Commented on your patch.
22:36:33 stewie925_ aunnam: thanks I'll check it out
22:37:46 mriedem melwitt: it says i'm not totally sure :)
22:37:54 mriedem we'll have to see
22:38:03 melwitt okay
22:39:38 mriedem alright i'm signing off, ttyl
22:39:55 melwitt k, have nice time off
22:46:04 bauzas any idea why the gate is consistently resetting ?

Earlier   Later