| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-08 | |||
| 17:14:07 | dtantsur | wanna me propose a patch? | |
| 17:14:14 | efried | jaypipes (and another core, maybe stephenfin or bauzas?) would you please add https://review.openstack.org/#/c/518633/ to your list? The patch above that is already +W, and it'll be nice to have for your nrp test case (https://review.openstack.org/#/c/531443/) | |
| 17:14:50 | stephenfin | efried: Won't get to it myself today, but if it's still there tomorrow then I'll take a shot | |
| 17:14:57 | efried | stephenfin Thanks | |
| 17:15:15 | mriedem | dtantsur: sure | |
| 17:15:40 | mriedem | efried: i'll take a look | |
| 17:15:48 | efried | mriedem Thanks. | |
| 17:20:47 | ralonsoh | stephenfin: sorry for the delay. [2] is working now | |
| 17:20:50 | ralonsoh | stephenfin: openstack-tox-functional-with-sudoSUCCESS in 4m 00s | |
| 17:21:07 | stephenfin | ralonsoh: Oh, congrats! Nice work | |
| 17:21:10 | ralonsoh | stephenfin: Am I missing something? | |
| 17:21:15 | ralonsoh | stephenfin: oooook! | |
| 17:21:33 | ralonsoh | stephenfin: Sean's magic.... you know | |
| 17:21:41 | stephenfin | Oh, I know :) | |
| 17:21:55 | stephenfin | Cool. I'll review that tomorrow morning so | |
| 17:22:03 | ralonsoh | stephenfin: thanks! | |
| 17:22:08 | stephenfin | np | |
| 17:25:41 | jaypipes | efried: just finishing up a review on that, yes. | |
| 17:29:01 | jaypipes | efried: done | |
| 17:29:15 | EmilienM | mriedem: hey matt, I remember we had a thread about how to handles patches like https://review.openstack.org/531055 - I don't recall the outcome, do you have the link of the thread? | |
| 17:31:02 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211 | |
| 17:31:40 | mriedem | EmilienM: the eventual outcome was the first contact SIG | |
| 17:31:57 | EmilienM | ok | |
| 17:32:00 | mriedem | https://wiki.openstack.org/wiki/First_Contact_SIG | |
| 17:32:50 | mriedem | and http://superuser.openstack.org/articles/sig-first-contact-openstack/ | |
| 17:32:54 | mriedem | i guess point them at that? | |
| 17:33:05 | EmilienM | mriedem: ok, thx | |
| 17:33:10 | mriedem | should probably build something into the first-time contributor comment that people get on their patch too | |
| 17:33:15 | EmilienM | mwhahaha: ^ | |
| 17:33:49 | EmilienM | the thing is I'm not sure this is a new contributor: https://review.openstack.org/#/q/owner:shangxiaobj%2540inspur.com+status:merged | |
| 17:34:11 | EmilienM | the person is doing http to https a lot and it waste so much CI resources imho | |
| 17:34:18 | mriedem | yeah they aren't, | |
| 17:34:27 | mriedem | i'd probably ping huang zhipeng (howard) | |
| 17:34:38 | mriedem | he's in china and on the first contact sig group | |
| 17:35:12 | mriedem | i think a bunch of these contributors from chinese companies actually hang out in a wechat group for openstack, | |
| 17:35:28 | mriedem | so he sometimes proxies stuff to that group where people are actually listening (and speaking the same language) | |
| 17:35:34 | EmilienM | mriedem: ok thx for the advises | |
| 17:35:51 | mriedem | yw | |
| 17:42:52 | jaypipes | tox -efast8 would be fast if it didn't try to reinstall all the global requirements each time I run it. | |
| 17:43:20 | mriedem | it shouldn't try to re-install them | |
| 17:43:29 | mriedem | pip + wheels should have all of that cached | |
| 17:43:40 | mriedem | plus your existing venv would have them | |
| 17:45:56 | openstackgerrit | Merged openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386 | |
| 17:50:39 | mriedem | efried: so i got looking at https://review.openstack.org/#/c/509892/ again, | |
| 17:50:54 | mriedem | and didn't realize it until now but we never leverage the session client stuff in neutronclient | |
| 17:50:58 | mriedem | https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L251 | |
| 17:51:49 | mriedem | oh nvm i guess that checks for a session kwarg https://github.com/openstack/python-neutronclient/blob/master/neutronclient/client.py#L389 | |
| 17:51:54 | mriedem | confusing method name | |
| 17:52:06 | jaypipes | mriedem: still takes 10+ seconds to run the "develop-inst-nodeps" part of the job :( | |
| 17:53:23 | jaypipes | mriedem: which I presume is just comparing pip freeze outputs? | |
| 17:53:32 | mriedem | no idea | |
| 17:54:16 | dtantsur | jaypipes: also downloads upper-constraint, if it's a typical job | |
| 17:54:24 | jaypipes | :) | |
| 17:56:58 | mriedem | probably an easy +W on efried's neutronclient change to use the ksa adapter stuff https://review.openstack.org/#/c/509892/ | |
| 17:58:38 | dtantsur | mriedem: oh, you guys haven't switched to in-tree zuul stuff, have you? | |
| 17:59:01 | mriedem | dtantsur: started to | |
| 17:59:06 | mriedem | but not much there yet | |
| 17:59:18 | mriedem | actually nothing until https://review.openstack.org/#/c/514309/ | |
| 17:59:19 | dtantsur | I see. so any changes still go to project-config? | |
| 17:59:25 | mriedem | dtantsur: yeah | |
| 17:59:32 | mriedem | unless you want to build it on top of https://review.openstack.org/#/c/514309/ | |
| 17:59:59 | mriedem | melwitt: you want to get https://review.openstack.org/#/c/514309/ since you've looked at the zuulv3 in-repo stuff before? | |
| 18:00:42 | dtantsur | lemme see what is easier | |
| 18:01:26 | mriedem | stvnoyes: an update on multiattach testing - i added some more tests over the weekend to https://review.openstack.org/#/c/266605/ - the list multiple attachments one and boot from volume one are passing ( https://review.openstack.org/#/c/531386/ ) - the snapshot one is failing, looks like due to some response validation in tempest that doesn't handle 2.59 | |
| 18:02:56 | stvnoyes | mreidem- ok thanks for the update. I am running into a m-a attach failure when attaching to the second instance. debugging that now. | |
| 18:03:15 | mriedem | stvnoyes: with libvirt 3.10? | |
| 18:03:48 | stvnoyes | no, with 3.6 | |
| 18:03:56 | mriedem | what version of qemu? | |
| 18:04:40 | stvnoyes | 2.8 | |
| 18:04:51 | mriedem | hmm | |
| 18:05:07 | mriedem | i'm getting things to work with lvm as the backend, and qemu 2.5 and libvirt 3.6 | |
| 18:05:10 | stvnoyes | may just be that I am missing a patch. I am running it down now | |
| 18:05:36 | stvnoyes | this had been working for me before | |
| 18:08:10 | dtantsur | mriedem: https://review.openstack.org/#/c/531868/ should do it. do you remember if Depends-On project-config changes works? | |
| 18:08:23 | mriedem | you can't depends-on a project-config change, no | |
| 18:09:27 | mriedem | andreas is usually pretty quick to hit these though | |
| 18:10:01 | dtantsur | yep | |
| 18:10:14 | mriedem | dtantsur: does ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode also test the multitenant stuff? | |
| 18:10:24 | mriedem | *multitenant network | |
| 18:10:30 | dtantsur | mriedem: that's the correct name of that job, so yes | |
| 18:10:46 | mriedem | oh maybe it was renamed and we just never noticd | |
| 18:10:54 | dtantsur | I think so | |
| 18:14:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | |
| 18:15:42 | jaypipes | mriedem, jgriffith_: your thoughts on this? https://review.openstack.org/#/c/528740/ | |
| 18:16:05 | mriedem | jaypipes: we can't change that as a bug fix | |
| 18:16:06 | mriedem | imo | |
| 18:16:14 | mriedem | it's an api change | |
| 18:16:24 | jaypipes | mriedem: oh, I agree... I don't think it's actually something we should do *at all*. | |
| 18:16:26 | mriedem | https://blueprints.launchpad.net/nova/+spec/volume-backed-server-rebuild | |
| 18:17:05 | mriedem | i've -2ed for now just due to procedure | |
| 18:17:25 | mriedem | i'm sure people want rebuild of volume-backed instances, but i'm not interested in that rats nest atm | |
| 18:19:16 | cdent | efried, jaypipes I've stubbed in https://blueprints.launchpad.net/nova/+spec/placement-aggregate-generation . Since it is microversion, officially we need a spec. | |
| 18:19:42 | ameeda | Hi , when I try to execute this command "mistral-db-manage --config-file /etc/mistral/mistral.conf populate", I got many errors as this http://paste.openstack.org/show/640994/ | |
| 18:20:25 | jaypipes | cdent: you want to handle the spec? | |
| 18:20:33 | cdent | yeah, will do | |
| 18:20:47 | mriedem | that's rocky at this point right? | |
| 18:20:57 | jaypipes | ameeda: wrong channel... try #openstack-mistral | |
| 18:21:08 | ameeda | Thanks ! | |
| 18:21:14 | jaypipes | mriedem: which one? | |
| 18:21:22 | jaypipes | mriedem: the placement agg gen? | |