| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-19 | |||
| 17:29:57 | bauzas | mriedem: jaypipes: FWIW, I'll need to disappear a bit tonight because of a meetup so I could miss the nova meeting | |
| 17:30:49 | bauzas | mriedem: my two concerns are #1 the critical bug, but the success path is found | |
| 17:31:10 | bauzas | mriedem: #2 the placement upcall, but the ML thread is open with my reply, so I'd love thoughts on that | |
| 17:31:20 | jaypipes | bauzas: just finishing up response to the ML thread. | |
| 17:32:06 | bauzas | jaypipes: I really need to disappear in like 2 mins, but I'll check my emails | |
| 17:34:29 | jaypipes | bauzas: response sent (to mwhahaha not your reply) | |
| 17:34:47 | mwhahaha | jaypipes: i shall read and flame appropriately :) | |
| 17:34:59 | jaypipes | mwhahaha: I would expect nothing less :) | |
| 17:39:16 | bauzas | jaypipes: roger | |
| 17:39:30 | bauzas | jaypipes: I made two proposals, I'd love to see your thoughts | |
| 17:40:24 | bauzas | jaypipes: nevermind, saw your reply | |
| 17:40:26 | jaypipes | bauzas: k. if you could look at https://review.openstack.org/#/c/398473/ later, would be great. | |
| 17:40:55 | bauzas | jaypipes: oh that one is absolutely gorgeous | |
| 17:41:06 | bauzas | I'll try to look at it when I have time | |
| 17:41:06 | jaypipes | lol | |
| 17:41:39 | bauzas | jaypipes: I'm a serious lover of your RT cleanup series | |
| 17:41:55 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | |
| 17:42:02 | jaypipes | bauzas: cool, thx :) | |
| 17:42:02 | openstackgerrit | Roman Dobosz proposed openstack/nova: Prevent compute crash on discovery failure https://review.openstack.org/422780 | |
| 17:42:25 | bauzas | I just need to teleport now, see you folks | |
| 17:57:13 | dims | sean-k-mooney : jaypipes : gah! nova/nova/network/linux_net.py has diverged from os-vif/vif_plug_linux_bridge/iptables.py the os-vif one does not have any tests and is outdated :( | |
| 17:58:18 | jaypipes | dims: yeah, an unfortunate side effect of not having an oslo.netutils library :( | |
| 17:58:35 | sean-k-mooney | dims: is there a bug for that against os-vif | |
| 17:58:36 | dims | :) | |
| 17:59:00 | sean-k-mooney | also alot of the nova/virt/libvirt/vif.py is not tested ... | |
| 17:59:01 | dims | i just spotted it when i tried to use linuxbridge in a py35 job sean-k-mooney (http://logs.openstack.org/09/421209/16/experimental/gate-tempest-dsvm-nova-py35-ubuntu-xenial/15a4fc7/logs/screen-n-cpu.txt.gz?level=ERROR) | |
| 18:00:09 | dims | so back to devstack defaults now | |
| 18:00:53 | sean-k-mooney | dims: so is linux bridge not deployable currently? | |
| 18:01:04 | dims | not on py35 | |
| 18:01:06 | sean-k-mooney | dims: with python 3.5 | |
| 18:01:11 | sean-k-mooney | ok | |
| 18:01:13 | dims | correct | |
| 18:01:25 | sean-k-mooney | well not ok but we should adress it | |
| 18:02:52 | dims | sean-k-mooney : got this far with a number of depends-on. so yea, there's a bunch of work. i am trying to find a golden path that works (https://review.openstack.org/#/c/421209/) | |
| 18:02:56 | jaypipes | sean-k-mooney: you want to push a fix or should I? | |
| 18:03:01 | sean-k-mooney | jaypipes: the experimenatl job for os-vif seams to work fine so i would like to move it to the check pipepline. perhaps we should also add a linuxbridge varient once we fix it | |
| 18:03:11 | jaypipes | sean-k-mooney: ++ | |
| 18:04:18 | sean-k-mooney | jaypipes: do you have a linux bridge enviroment that you can test with? or a local.conf you can share. | |
| 18:04:57 | sean-k-mooney | if you can fix quickly go ahead other wise i can try and set up a dev environment but like wont get to it till next week. | |
| 18:05:29 | jaypipes | sean-k-mooney: I can fix the filter() thing quickly. don't have a LB env to use at the moment no. | |
| 18:07:32 | sean-k-mooney | dims: that is a faily deep dependcy tree for your experimental job | |
| 18:08:36 | sean-k-mooney | dims: are they all required? | |
| 18:09:16 | sean-k-mooney | jaypipes: will the filter change need to be made to nova or os-vif | |
| 18:09:16 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 18:09:17 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 18:09:18 | openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | |
| 18:09:18 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | |
| 18:09:19 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | |
| 18:09:19 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | |
| 18:09:20 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 18:09:20 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | |
| 18:09:40 | jaypipes | sean-k-mooney: os-vif. which means becaus eof the non-client freeze we need to do it today. | |
| 18:09:58 | jaypipes | sean-k-mooney: shouldn't be a hard fix. will work on it shortly. | |
| 18:10:02 | jaypipes | it's just a py35 thing. | |
| 18:10:21 | dims | sean-k-mooney : y, nothing too big in there, all small stuff. very close 1 failure left (http://logs.openstack.org/09/421209/15/experimental/gate-tempest-dsvm-nova-py35-ubuntu-xenial/b47de57/logs/testr_results.html.gz) | |
| 18:10:23 | sean-k-mooney | yes but it means we need to do a 1.4.1 release also correct | |
| 18:11:57 | sean-k-mooney | dims: no reason other then if i wanted to replicate locally its a little trickyer to recreate | |
| 18:13:48 | openstackgerrit | Vladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata https://review.openstack.org/405951 | |
| 18:13:49 | openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: expose virtual interfaces with vlans to metadata https://review.openstack.org/405954 | |
| 18:19:19 | jaypipes | dims: so you're already working on a fix for that os-vif thing? or should I do that? | |
| 18:20:04 | dims | jaypipes : please go ahead. | |
| 18:20:50 | jaypipes | dims: you got it :) | |
| 18:21:10 | dims | thanks much jaypipes ! | |
| 18:24:09 | openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (console) https://review.openstack.org/409402 | |
| 18:26:40 | openstackgerrit | Eric Fried proposed openstack/nova: WIP(4): Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | |
| 18:50:03 | jaypipes | dims: did you have a bug report about this os-vif thing? | |
| 18:50:14 | jaypipes | dims: if not, I'll add one. | |
| 18:50:38 | dims | jaypipes : no i have not added one | |
| 18:51:19 | jaypipes | dims: k, no problemo. | |
| 18:51:35 | dims | starting hard at logs :) | |
| 19:21:14 | mriedem | what os-vif issue do we have? | |
| 19:28:50 | jaypipes | mriedem: it's a py35 thing... | |
| 19:28:56 | jaypipes | http://logs.openstack.org/09/421209/16/experimental/gate-tempest-dsvm-nova-py35-ubuntu-xenial/15a4fc7/logs/screen-n-cpu.txt.gz?level=ERROR | |
| 19:29:13 | mriedem | oh that's known | |
| 19:29:20 | mriedem | oh wait no it's not | |
| 19:29:51 | mriedem | but, that's also the experimental queue so not blocking us right now | |
| 19:29:59 | mriedem | and therefore i will put my head in the sand | |
| 19:30:54 | openstackgerrit | Bob Ball proposed openstack/nova: Use physical utilisation for cached images https://review.openstack.org/353862 | |
| 19:30:56 | jaypipes | mriedem: :) | |
| 19:32:01 | mriedem | and for whatever reason https://review.openstack.org/#/c/422709/ is taking forever to land | |
| 19:36:33 | sean-k-mooney | mriedem: well looking at zull it is currently in the intergrated gate queue | |
| 19:37:33 | sean-k-mooney | mriedem: its being cogated with quite a few other patches but should merge soonish | |
| 19:38:52 | sdague | mriedem: we found a zuul bug | |
| 19:38:56 | sdague | or... limitation | |
| 19:39:05 | sdague | it's more fair to call it a limitation | |
| 19:39:54 | sean-k-mooney | sdague: mind shareing? whats the limitation? | |
| 19:40:48 | sdague | sean-k-mooney: in order to match jgit's merge behavior, zuul is using a git merge strategy that falls over after 6 way merge | |
| 19:41:12 | openstackgerrit | Jay Pipes proposed openstack/os-vif: Correct py27 assumption that filter() returns list https://review.openstack.org/422838 | |
| 19:41:14 | jaypipes | dims: ^^ | |
| 19:41:26 | sdague | which means when you get a whole stream of unchained commits eventually they merge fail | |
| 19:41:46 | sdague | requirements upper-constraints auto generated items fall into that category | |
| 19:42:11 | sean-k-mooney | sdague: ah ok | |
| 19:42:41 | sean-k-mooney | and im guessing if we iterativly merged the patches instade of 6+ way merge it would break compaiblity with jgit | |
| 19:43:20 | sdague | sean-k-mooney: yeh, it would instead look like it was going to pass, get to top of queue, and gerrit would fail it | |
| 19:43:29 | sdague | which used to happen | |
| 19:44:01 | sdague | this fails things in a few more situations though, so it's kind of a toss up as to when you get kicked in the pants | |
| 19:44:46 | sean-k-mooney | sdague: actully looking at the zull gate queue specifcally for the patch mriedem was interested in https://review.openstack.org/#/c/422709 | |
| 19:45:14 | sean-k-mooney | there are 2 other requirements changes in front of it that will cause the rest of the queue to restart when they merge | |
| 19:45:22 | sdague | no | |
| 19:45:40 | sdague | the symptom was a merge fail when it entered the gate queue | |