| *** diogogmt has joined #openstack-nova | 00:00 | |
| clarkb | dansmith: no, maybe they are both getting the requests and the other is cranky about it? | 00:00 |
|---|---|---|
| *** jamielennox|away is now known as jamielennox | 00:00 | |
| clarkb | also that change seems to reliably fail neutron grenade which is a bit wat since we don't set up overlays for the single node neutron grenade test | 00:01 |
| *** ducttape_ has joined #openstack-nova | 00:02 | |
| dansmith | clarkb: I guess.. do they definitely have different hostnames/ | 00:02 |
| *** takedakn has quit IRC | 00:02 | |
| dansmith | gdi I can't hit ? to save my life | 00:02 |
| *** slagle has joined #openstack-nova | 00:03 | |
| clarkb | dansmith: ya pretty sure they had differnet hostnames | 00:03 |
| clarkb | trying to double check that now with the logs | 00:04 |
| clarkb | dansmith: syslog reports different names | 00:05 |
| dansmith | clarkb: only matters what nova thinks, but okay | 00:05 |
| dansmith | not really sure what else would cause that | 00:05 |
| clarkb | I am not sure how to check what nova thinks :) but if syslog says they are different I am hoping nova agrees | 00:05 |
| dansmith | http://logs.openstack.org/42/370442/1/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial/13109f3/logs/screen-n-cpu.txt.gz?#_2016-09-14_21_50_07_880 | 00:06 |
| dansmith | clarkb: the host line there | 00:06 |
| clarkb | http://logs.openstack.org/42/370442/1/check/gate-tempest-dsvm-multinode-full-ubuntu-xenial/13109f3/logs/subnode-2/screen-n-cpu.txt.gz#_2016-09-14_21_57_33_168 is the other | 00:07 |
| clarkb | and they are different | 00:07 |
| dansmith | clarkb: oh | 00:09 |
| dansmith | I lied | 00:09 |
| dansmith | I read too quickly, I thought this was a complaint about the instance, but it's a complaint about the migration | 00:09 |
| dansmith | which can happen if a migration isn't completed, like an instance is deleted first or something | 00:09 |
| dansmith | we have an open bug for that | 00:09 |
| *** sai has quit IRC | 00:10 | |
| clarkb | so maybe don't worry about that too much¿ | 00:10 |
| dansmith | yeah | 00:10 |
| clarkb | also silly keyboard I wanted ? | 00:10 |
| dansmith | jej | 00:10 |
| dansmith | er, heh | 00:10 |
| clarkb | ok now just to figure out what is going on with that grenade job | 00:10 |
| *** lucas-dinner has quit IRC | 00:10 | |
| *** hieulq has joined #openstack-nova | 00:13 | |
| *** lucasagomes has joined #openstack-nova | 00:16 | |
| *** sai has joined #openstack-nova | 00:16 | |
| *** thorst has joined #openstack-nova | 00:17 | |
| *** Swami__ has joined #openstack-nova | 00:19 | |
| *** Swami_ has quit IRC | 00:22 | |
| *** thorst has quit IRC | 00:24 | |
| mriedem | swamireddy: can you explain why you think a release note is needed for this fix? https://review.openstack.org/#/c/335884/ | 00:25 |
| *** hoangcx has joined #openstack-nova | 00:26 | |
| *** ekhugen_alt has quit IRC | 00:29 | |
| *** kaisers has joined #openstack-nova | 00:31 | |
| *** ekhugen has joined #openstack-nova | 00:32 | |
| *** kaisers3 has joined #openstack-nova | 00:32 | |
| *** annegentle has joined #openstack-nova | 00:33 | |
| *** kaisers2 has quit IRC | 00:34 | |
| *** kaisers1 has quit IRC | 00:34 | |
| *** Swami__ has quit IRC | 00:36 | |
| *** Swami__ has joined #openstack-nova | 00:37 | |
| *** jlvillal has quit IRC | 00:41 | |
| *** vishwanathj has quit IRC | 00:43 | |
| mriedem | melwitt: do you know what's going on here https://review.openstack.org/#/c/335884/ ? | 00:45 |
| mriedem | i always feel dumb for asking | 00:45 |
| *** rkrum has joined #openstack-nova | 00:46 | |
| mriedem | mikal: if you're around, cdent has some placement api patches with a +2 that are ready to go i think https://review.openstack.org/#/q/owner:cdent%2540anticdent.org+status:open | 00:46 |
| *** jlvillal has joined #openstack-nova | 00:46 | |
| *** jlvillal has quit IRC | 00:47 | |
| *** jlvillal has joined #openstack-nova | 00:47 | |
| *** jlvillal has quit IRC | 00:47 | |
| *** jlvillal has joined #openstack-nova | 00:47 | |
| mikal | mriedem: I have to step out for an hour or so, will look when I get back | 00:48 |
| *** tblakes has quit IRC | 00:49 | |
| *** phuongnh has joined #openstack-nova | 00:49 | |
| *** JoseMello has joined #openstack-nova | 00:50 | |
| *** akshai has joined #openstack-nova | 00:51 | |
| *** akshai has quit IRC | 00:53 | |
| melwitt | mriedem: I haven't looked at that yet so I don't know yet :P | 00:54 |
| *** gongysh has joined #openstack-nova | 00:55 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix an error in archiving 'migrations' table https://review.openstack.org/323684 | 00:55 |
| melwitt | mriedem: I'm looking through it but it's not something I have familiarity about. I'll look at it more later and let you know what I find if you don't get to it first. the commit message is pretty sparse | 01:00 |
| melwitt | that is, it's saying the rescue config drive isn't being mapped to the instance but it's not clear to me how it used to be mapped to the instance before the change mentioned in the commit | 01:01 |
| melwitt | *commit message | 01:01 |
| *** rkrum has quit IRC | 01:02 | |
| *** hoangcx has quit IRC | 01:05 | |
| *** tuanla_ has joined #openstack-nova | 01:07 | |
| *** namnh has joined #openstack-nova | 01:08 | |
| *** tiendc has joined #openstack-nova | 01:08 | |
| hieulq | pumaranikar: ping | 01:08 |
| hieulq | pumaranikar: which part in libvirt config file you want to mention? help text or what? | 01:09 |
| *** vladikr has quit IRC | 01:10 | |
| mriedem | melwitt: yeah me neither, glad i'm not the only one | 01:11 |
| *** mmedvede has joined #openstack-nova | 01:13 | |
| *** ijw has quit IRC | 01:17 | |
| *** thorst has joined #openstack-nova | 01:21 | |
| *** cdelatte has quit IRC | 01:25 | |
| *** ijw has joined #openstack-nova | 01:25 | |
| *** tiendc has quit IRC | 01:27 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova: Handle TypeError when disabling host service https://review.openstack.org/290919 | 01:27 |
| *** EinstCrazy has joined #openstack-nova | 01:27 | |
| mriedem | melwitt: i'm +2 on https://review.openstack.org/#/c/323684/ if you wanted to get that in | 01:29 |
| *** thorst has quit IRC | 01:29 | |
| *** hoangcx has joined #openstack-nova | 01:30 | |
| *** EinstCrazy has quit IRC | 01:33 | |
| *** thorst has joined #openstack-nova | 01:35 | |
| *** Sukhdev has quit IRC | 01:37 | |
| *** ducttape_ has quit IRC | 01:37 | |
| *** baoli has joined #openstack-nova | 01:39 | |
| *** baoli has quit IRC | 01:41 | |
| *** salv-orlando has joined #openstack-nova | 01:41 | |
| *** baoli has joined #openstack-nova | 01:41 | |
| openstackgerrit | Nguyen Phuong An proposed openstack/nova: Update info of machine-processable description https://review.openstack.org/365426 | 01:42 |
| *** ssurana has quit IRC | 01:43 | |
| *** salv-orl_ has quit IRC | 01:43 | |
| *** thorst has quit IRC | 01:45 | |
| *** baoli has quit IRC | 01:46 | |
| *** Guest2984 has quit IRC | 01:47 | |
| *** gongysh has quit IRC | 01:47 | |
| *** woodster_ has quit IRC | 01:50 | |
| *** raunak has quit IRC | 01:51 | |
| *** ducttape_ has joined #openstack-nova | 01:55 | |
| *** rama_y has quit IRC | 01:55 | |
| *** ducttape_ has quit IRC | 01:56 | |
| *** ijw has quit IRC | 01:56 | |
| *** hoangcx has quit IRC | 02:00 | |
| *** Apoorva_ has joined #openstack-nova | 02:01 | |
| *** rmcall has quit IRC | 02:02 | |
| *** rmcall has joined #openstack-nova | 02:03 | |
| *** Apoorva has quit IRC | 02:04 | |
| *** ijw has joined #openstack-nova | 02:05 | |
| *** Apoorva_ has quit IRC | 02:06 | |
| *** unicell has quit IRC | 02:06 | |
| *** diogogmt has quit IRC | 02:09 | |
| *** 32NABYP9K has joined #openstack-nova | 02:14 | |
| *** TravT has quit IRC | 02:16 | |
| *** TravT has joined #openstack-nova | 02:16 | |
| *** bkopilov has quit IRC | 02:16 | |
| *** TravT has quit IRC | 02:20 | |
| *** namnh has quit IRC | 02:22 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Treat 'body_map' as a mapping of HTTP request bodies https://review.openstack.org/363457 | 02:25 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host https://review.openstack.org/357105 | 02:25 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix wrong parameters in os-volumes.inc https://review.openstack.org/347112 | 02:25 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix unnecessary string interpolation https://review.openstack.org/357577 | 02:25 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix 'id' (attachment_id) parameters https://review.openstack.org/349863 | 02:26 |
| *** michchap has quit IRC | 02:26 | |
| *** michchap has joined #openstack-nova | 02:26 | |
| *** ducttape_ has joined #openstack-nova | 02:30 | |
| *** shangxdy has joined #openstack-nova | 02:36 | |
| *** ssurana has joined #openstack-nova | 02:37 | |
| *** annegentle has quit IRC | 02:38 | |
| *** thorst has joined #openstack-nova | 02:47 | |
| *** ducttape_ has quit IRC | 02:47 | |
| *** tiendc has joined #openstack-nova | 02:48 | |
| *** mriedem has quit IRC | 02:49 | |
| *** gongysh has joined #openstack-nova | 02:50 | |
| *** hoangcx has joined #openstack-nova | 02:54 | |
| *** rrecio_ has quit IRC | 02:55 | |
| *** rrecio has joined #openstack-nova | 02:56 | |
| *** thorst has quit IRC | 02:56 | |
| *** rmcall has quit IRC | 02:59 | |
| *** rmcall has joined #openstack-nova | 02:59 | |
| *** shangxdy has quit IRC | 03:00 | |
| *** sdake has joined #openstack-nova | 03:01 | |
| melwitt | takashin: can you tell me why this is needed if we are soft-deleting migration records along with instances? is it just to clean up potentially old migration records? https://review.openstack.org/#/c/323684/28/nova/db/sqlalchemy/api.py@6323 | 03:04 |
| *** ijw has quit IRC | 03:04 | |
| *** ijw has joined #openstack-nova | 03:05 | |
| melwitt | takashin: nevermind, I think it's obvious now that's why | 03:07 |
| *** gjayavelu has quit IRC | 03:09 | |
| *** yamahata has quit IRC | 03:11 | |
| *** ijw has quit IRC | 03:13 | |
| *** code-R has joined #openstack-nova | 03:15 | |
| *** code-R_ has joined #openstack-nova | 03:18 | |
| *** gouthamr has quit IRC | 03:19 | |
| *** namnh has joined #openstack-nova | 03:19 | |
| *** code-R has quit IRC | 03:21 | |
| *** gongysh has quit IRC | 03:21 | |
| *** ssurana has quit IRC | 03:22 | |
| *** marst has joined #openstack-nova | 03:23 | |
| *** namnh has quit IRC | 03:24 | |
| *** ssurana has joined #openstack-nova | 03:24 | |
| *** gongysh has joined #openstack-nova | 03:25 | |
| *** namnh has joined #openstack-nova | 03:25 | |
| *** namnh has quit IRC | 03:25 | |
| *** namnh has joined #openstack-nova | 03:25 | |
| *** raunak has joined #openstack-nova | 03:25 | |
| *** namnh has quit IRC | 03:26 | |
| *** namnh has joined #openstack-nova | 03:27 | |
| *** namnh has quit IRC | 03:27 | |
| *** namnh has joined #openstack-nova | 03:27 | |
| *** hoangcx has quit IRC | 03:28 | |
| *** ssurana has quit IRC | 03:31 | |
| *** hoangcx has joined #openstack-nova | 03:31 | |
| *** bkopilov has joined #openstack-nova | 03:33 | |
| *** namnh has quit IRC | 03:39 | |
| *** namnh has joined #openstack-nova | 03:40 | |
| *** namnh has quit IRC | 03:40 | |
| *** namnh has joined #openstack-nova | 03:41 | |
| *** namnh has quit IRC | 03:41 | |
| *** namnh has joined #openstack-nova | 03:42 | |
| *** namnh has quit IRC | 03:43 | |
| *** namnh has joined #openstack-nova | 03:44 | |
| *** thorst has joined #openstack-nova | 03:44 | |
| *** thorst has quit IRC | 03:46 | |
| *** ducttape_ has joined #openstack-nova | 03:48 | |
| *** raunak has quit IRC | 03:48 | |
| *** jdurgin has joined #openstack-nova | 03:49 | |
| *** ijw_ has joined #openstack-nova | 03:49 | |
| *** ducttape_ has quit IRC | 03:53 | |
| *** hoangcx has quit IRC | 03:53 | |
| *** abhishekk has joined #openstack-nova | 03:53 | |
| *** ijw_ has quit IRC | 03:54 | |
| *** raunak has joined #openstack-nova | 04:02 | |
| *** mingyu has joined #openstack-nova | 04:05 | |
| *** hoangcx has joined #openstack-nova | 04:05 | |
| *** phschwartz has quit IRC | 04:07 | |
| *** mordred has quit IRC | 04:08 | |
| *** mordred has joined #openstack-nova | 04:10 | |
| *** phschwartz has joined #openstack-nova | 04:11 | |
| *** gjayavelu has joined #openstack-nova | 04:12 | |
| *** ratailor has joined #openstack-nova | 04:14 | |
| *** adisky has joined #openstack-nova | 04:15 | |
| *** ssurana has joined #openstack-nova | 04:17 | |
| *** psachin has joined #openstack-nova | 04:21 | |
| *** Swami__ has quit IRC | 04:21 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: WIP: Shared state host manager https://review.openstack.org/332463 | 04:22 |
| *** ociuhandu has joined #openstack-nova | 04:24 | |
| *** thorst has joined #openstack-nova | 04:29 | |
| *** fragatin_ has joined #openstack-nova | 04:29 | |
| *** fragatina has quit IRC | 04:32 | |
| *** fragatin_ has quit IRC | 04:33 | |
| *** coolsvap has joined #openstack-nova | 04:34 | |
| *** thorst has quit IRC | 04:39 | |
| abhishekk | hi, has anyone from India applied for Spain visa? | 04:42 |
| *** thorst has joined #openstack-nova | 04:45 | |
| *** jaosorior has joined #openstack-nova | 04:47 | |
| *** aswadr_ has joined #openstack-nova | 04:49 | |
| *** ducttape_ has joined #openstack-nova | 04:49 | |
| *** takedakn has joined #openstack-nova | 04:50 | |
| *** unicell has joined #openstack-nova | 04:51 | |
| *** ducttape_ has quit IRC | 04:55 | |
| *** sudipto_ has joined #openstack-nova | 04:55 | |
| *** sudipto has joined #openstack-nova | 04:55 | |
| *** unicell has quit IRC | 04:57 | |
| *** unicell has joined #openstack-nova | 04:57 | |
| *** anush__ has quit IRC | 04:58 | |
| *** thorst has quit IRC | 04:58 | |
| *** thorst has joined #openstack-nova | 04:58 | |
| *** anush__ has joined #openstack-nova | 04:59 | |
| *** Jeffrey4l__ has joined #openstack-nova | 05:00 | |
| *** lujinluo has joined #openstack-nova | 05:01 | |
| *** unicell1 has joined #openstack-nova | 05:01 | |
| *** jaosorior has quit IRC | 05:01 | |
| *** unicell has quit IRC | 05:02 | |
| *** jaosorior has joined #openstack-nova | 05:02 | |
| *** sdake has quit IRC | 05:03 | |
| *** rama_y has joined #openstack-nova | 05:03 | |
| *** mingyu has quit IRC | 05:03 | |
| *** Jeffrey4l_ has quit IRC | 05:04 | |
| *** thorst has quit IRC | 05:07 | |
| *** slaweq_ has joined #openstack-nova | 05:08 | |
| *** thorst has joined #openstack-nova | 05:09 | |
| *** thorst has quit IRC | 05:13 | |
| *** raunak has quit IRC | 05:14 | |
| *** thorst has joined #openstack-nova | 05:15 | |
| *** rcernin has joined #openstack-nova | 05:17 | |
| *** aunnam has joined #openstack-nova | 05:19 | |
| *** thorst has quit IRC | 05:19 | |
| *** takedakn has quit IRC | 05:21 | |
| *** sdake has joined #openstack-nova | 05:22 | |
| *** lpetrut has joined #openstack-nova | 05:28 | |
| *** raunak has joined #openstack-nova | 05:29 | |
| *** artom has quit IRC | 05:30 | |
| *** dimtruck is now known as zz_dimtruck | 05:31 | |
| *** gouthamr has joined #openstack-nova | 05:31 | |
| *** artom has joined #openstack-nova | 05:32 | |
| *** fragatina has joined #openstack-nova | 05:32 | |
| *** artom has quit IRC | 05:32 | |
| *** artom has joined #openstack-nova | 05:32 | |
| *** thorst has joined #openstack-nova | 05:34 | |
| *** fragatina has quit IRC | 05:37 | |
| *** markvoelker has quit IRC | 05:41 | |
| *** thorst has quit IRC | 05:42 | |
| *** thorst has joined #openstack-nova | 05:44 | |
| *** vishwanathj has joined #openstack-nova | 05:47 | |
| *** thorst has quit IRC | 05:49 | |
| *** raunak has quit IRC | 05:50 | |
| *** thorst has joined #openstack-nova | 05:50 | |
| *** zz_dimtruck is now known as dimtruck | 05:52 | |
| *** rkmrhj has joined #openstack-nova | 05:52 | |
| *** thorst has quit IRC | 05:55 | |
| *** oanson has joined #openstack-nova | 05:56 | |
| *** thorst has joined #openstack-nova | 05:56 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Removed unnecessary parantheses in return statements https://review.openstack.org/370549 | 06:00 |
| *** thorst has quit IRC | 06:01 | |
| *** mjura has joined #openstack-nova | 06:01 | |
| *** nherciu has joined #openstack-nova | 06:02 | |
| *** thorst has joined #openstack-nova | 06:02 | |
| *** dimtruck is now known as zz_dimtruck | 06:03 | |
| *** pcaruana has joined #openstack-nova | 06:04 | |
| *** paul-carlton has joined #openstack-nova | 06:05 | |
| *** karthiks has joined #openstack-nova | 06:06 | |
| *** thorst has quit IRC | 06:07 | |
| *** mjura has quit IRC | 06:08 | |
| *** thorst has joined #openstack-nova | 06:08 | |
| *** karthiks has quit IRC | 06:10 | |
| *** jdurgin has quit IRC | 06:13 | |
| *** pcaruana is now known as pcaruana|afk| | 06:13 | |
| *** claudiub has joined #openstack-nova | 06:15 | |
| *** ssurana has quit IRC | 06:16 | |
| *** thorst has quit IRC | 06:17 | |
| *** nkrinner_afk is now known as nkrinner | 06:17 | |
| *** artom has quit IRC | 06:17 | |
| *** thorst has joined #openstack-nova | 06:19 | |
| *** jdurgin has joined #openstack-nova | 06:19 | |
| *** artom has joined #openstack-nova | 06:19 | |
| *** mjura has joined #openstack-nova | 06:21 | |
| *** zz_dimtruck is now known as dimtruck | 06:22 | |
| *** jdurgin has quit IRC | 06:23 | |
| *** artom has quit IRC | 06:24 | |
| *** thorst has quit IRC | 06:24 | |
| *** thorst has joined #openstack-nova | 06:25 | |
| *** artom has joined #openstack-nova | 06:25 | |
| *** sahid has joined #openstack-nova | 06:28 | |
| *** raunak has joined #openstack-nova | 06:29 | |
| *** thorst has quit IRC | 06:30 | |
| *** andreas_s has joined #openstack-nova | 06:30 | |
| *** thorst has joined #openstack-nova | 06:32 | |
| *** dimtruck is now known as zz_dimtruck | 06:32 | |
| *** takedakn has joined #openstack-nova | 06:34 | |
| *** sdake has quit IRC | 06:36 | |
| *** thorst has quit IRC | 06:36 | |
| *** jdurgin has joined #openstack-nova | 06:40 | |
| *** pcaruana|afk| is now known as pcaruana | 06:40 | |
| *** gouthamr has quit IRC | 06:41 | |
| *** warp has joined #openstack-nova | 06:43 | |
| *** thorst has joined #openstack-nova | 06:44 | |
| *** jamielennox is now known as jamielennox|away | 06:45 | |
| *** jojden has joined #openstack-nova | 06:48 | |
| *** lpetrut has quit IRC | 06:49 | |
| jojden | hi | 06:49 |
| jojden | how to do host evacuation using nova clent | 06:50 |
| jojden | self.nova_conn.servers.evacuate(server='c9d901fd-2d83-4e2d-be0b-8304eb9eeba9') | 06:50 |
| jojden | I am getting | 06:50 |
| jojden | Instance c9d901fd-2d83-4e2d-be0b-8304eb9eeba9 could not be found. (HTTP 404) (Request-ID: req-caa8e530-fb2e-4c0e-a5a5-b204a9745767) | 06:50 |
| jojden | error | 06:50 |
| *** salv-orlando has quit IRC | 06:50 | |
| *** salv-orlando has joined #openstack-nova | 06:51 | |
| *** ducttape_ has joined #openstack-nova | 06:52 | |
| *** thorst has quit IRC | 06:52 | |
| *** raunak_ has joined #openstack-nova | 06:53 | |
| *** raunak has quit IRC | 06:53 | |
| *** raunak_ is now known as raunak | 06:53 | |
| *** markus_z has joined #openstack-nova | 06:53 | |
| *** sridharg has joined #openstack-nova | 06:54 | |
| *** thorst has joined #openstack-nova | 06:54 | |
| *** hoangcx has quit IRC | 06:55 | |
| *** ducttape_ has quit IRC | 06:56 | |
| *** hoangcx has joined #openstack-nova | 06:56 | |
| *** thorst has quit IRC | 06:59 | |
| *** thorst has joined #openstack-nova | 07:00 | |
| *** JoseMello has quit IRC | 07:01 | |
| *** liverpooler has joined #openstack-nova | 07:01 | |
| *** jdurgin1 has joined #openstack-nova | 07:01 | |
| *** jdurgin has quit IRC | 07:02 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/nova: Add members in InstanceGroup object members field https://review.openstack.org/370567 | 07:03 |
| *** thorst has quit IRC | 07:05 | |
| *** jdurgin1 has quit IRC | 07:06 | |
| *** jdurgin has joined #openstack-nova | 07:06 | |
| *** RuiChen has quit IRC | 07:08 | |
| *** code-R_ has quit IRC | 07:08 | |
| *** RuiChen has joined #openstack-nova | 07:10 | |
| *** zul has joined #openstack-nova | 07:10 | |
| *** thorst has joined #openstack-nova | 07:11 | |
| *** takedakn has quit IRC | 07:11 | |
| *** takedakn has joined #openstack-nova | 07:12 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/nova: Clean up libvirt guest code https://review.openstack.org/365756 | 07:13 |
| *** mingyu has joined #openstack-nova | 07:14 | |
| *** mingyu has quit IRC | 07:16 | |
| *** mingyu has joined #openstack-nova | 07:17 | |
| *** thorst has quit IRC | 07:17 | |
| *** ekuris has joined #openstack-nova | 07:17 | |
| *** takedakn has quit IRC | 07:17 | |
| *** artom has quit IRC | 07:17 | |
| *** thorst has joined #openstack-nova | 07:18 | |
| *** takedakn has joined #openstack-nova | 07:18 | |
| *** zenoway has joined #openstack-nova | 07:18 | |
| *** rama_y has quit IRC | 07:18 | |
| *** jed56 has joined #openstack-nova | 07:19 | |
| *** artom has joined #openstack-nova | 07:19 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/nova: Add members in InstanceGroup object members field https://review.openstack.org/370567 | 07:20 |
| *** thorst_ has joined #openstack-nova | 07:21 | |
| *** takedakn has quit IRC | 07:23 | |
| *** thorst has quit IRC | 07:23 | |
| *** matrohon has joined #openstack-nova | 07:24 | |
| *** takedakn has joined #openstack-nova | 07:25 | |
| *** thorst_ has quit IRC | 07:25 | |
| *** takedakn has quit IRC | 07:26 | |
| *** takedakn has joined #openstack-nova | 07:26 | |
| *** takedakn has quit IRC | 07:28 | |
| *** takedakn has joined #openstack-nova | 07:29 | |
| *** jlanoux has joined #openstack-nova | 07:29 | |
| *** takedakn has quit IRC | 07:31 | |
| *** takedakn has joined #openstack-nova | 07:34 | |
| *** artom has quit IRC | 07:35 | |
| *** takedakn has quit IRC | 07:35 | |
| *** raunak has quit IRC | 07:39 | |
| *** gongysh has quit IRC | 07:40 | |
| *** tobasco has quit IRC | 07:40 | |
| *** artom has joined #openstack-nova | 07:40 | |
| *** salv-orl_ has joined #openstack-nova | 07:40 | |
| *** markvoelker has joined #openstack-nova | 07:42 | |
| *** salv-orlando has quit IRC | 07:43 | |
| *** zenoway has quit IRC | 07:46 | |
| *** jlanoux_ has joined #openstack-nova | 07:46 | |
| *** jlanoux has quit IRC | 07:46 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/nova: Add members in InstanceGroup object members field https://review.openstack.org/370567 | 07:48 |
| *** markvoelker has quit IRC | 07:48 | |
| *** tobasco has joined #openstack-nova | 07:52 | |
| *** zenoway has joined #openstack-nova | 07:53 | |
| *** warp has quit IRC | 07:53 | |
| *** abhishekk has quit IRC | 07:55 | |
| *** hoangcx has quit IRC | 07:56 | |
| *** zul has quit IRC | 07:57 | |
| *** jamielennox|away is now known as jamielennox | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** bks has joined #openstack-nova | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** takashin has left #openstack-nova | 08:01 | |
| *** gjayavelu has quit IRC | 08:03 | |
| openstackgerrit | Krzysztof Świątek proposed openstack/nova: Fix issue with not removing rbd rescue disk https://review.openstack.org/314928 | 08:05 |
| *** paul-carlton has quit IRC | 08:05 | |
| zigo | bauzas: Hi there! | 08:11 |
| zigo | Around? | 08:11 |
| bauzas | zigo: hey :) | 08:12 |
| bauzas | zigo: yup | 08:12 |
| bauzas | wazzup ? | 08:12 |
| zigo | bauzas: Nova pretends (through its requirements.txt) that it is compatible with netaddr 0.7.12, though that's just not true. | 08:12 |
| zigo | bauzas: Look at the build log: http://logs.openstack.org/79/369879/2/check/gate-deb-nova-dpkg-debian-jessie/10d7c56/console.html | 08:12 |
| zigo | (yes! Build log in OpenStack infra!!!) | 08:12 |
| *** yamahata has joined #openstack-nova | 08:13 | |
| *** sree has joined #openstack-nova | 08:13 | |
| *** amotoki has joined #openstack-nova | 08:14 | |
| zigo | bauzas: Should I open a bug or what? | 08:14 |
| zigo | In 0.7.12, there's no such thing as modified_eui64() | 08:15 |
| zigo | I greped in a netaddr installed package. | 08:15 |
| zigo | 0.7.18 has it. | 08:15 |
| *** yassine has joined #openstack-nova | 08:15 | |
| * bauzas looking | 08:15 | |
| *** lpetrut has joined #openstack-nova | 08:16 | |
| *** yassine is now known as Guest67165 | 08:16 | |
| bauzas | zigo: so, Nova asks for an upper or the same version https://github.com/openstack/nova/blob/master/requirements.txt#L22 | 08:16 |
| bauzas | zigo: while https://github.com/openstack/requirements/blob/master/global-requirements.txt#L90 says we should just avoid 0.7.16 | 08:17 |
| zigo | bauzas: Yup, it says that netaddr 0.7.12 is valid, which is what Jessie has by default, and which doesn't work at all (ie: it doesn't have the modified_eui64() function that nova uses). | 08:17 |
| zigo | I don't know when modified_eui64() was introduced, we could investigate, but I just think it'd be easier to require 0.7.18 (which should work, and which is packaged in Sid/Stretch and Xenial). | 08:18 |
| bauzas | zigo: okay, lemme see what version uses the gate | 08:18 |
| zigo | http://packages.ubuntu.com/search?keywords=python-netaddr | 08:19 |
| zigo | The gate uses 0.7.18 | 08:19 |
| zigo | That's what is in upper-constraints.txt | 08:19 |
| *** lpetrut has quit IRC | 08:20 | |
| bauzas | yeah, that's what I can see | 08:20 |
| bauzas | zigo: okay, so the change was introduced recently | 08:22 |
| bauzas | zigo: given the gate was using a newer netaddr it never catched the problem | 08:23 |
| *** zz_dimtruck is now known as dimtruck | 08:23 | |
| zigo | Yeah ... | 08:23 |
| zigo | We don't check lower bounds! :) | 08:23 |
| zigo | It was introduced at least before b2 was released. | 08:23 |
| bauzas | zigo: https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e2 | 08:23 |
| zigo | Because b2 was failing, and I tried upgrading. | 08:23 |
| bauzas | that's in b2 | 08:24 |
| zigo | Yup. | 08:24 |
| zigo | It matches. | 08:24 |
| bauzas | zigo: the real problem I see is that Nova uses the global requirements for knowing the lower bound | 08:25 |
| zigo | Like everyone. | 08:26 |
| zigo | If global-requirements.txt is broken, then we should fix global-requirements.txt ! :) | 08:26 |
| bauzas | I tend to agree | 08:26 |
| zigo | Either that, or get rid of the modified_eui64() call. | 08:26 |
| bauzas | zactly | 08:26 |
| zigo | The later is IMO more dangerous. | 08:26 |
| bauzas | zigo: file a bug report | 08:26 |
| bauzas | tonyb: still around ? | 08:27 |
| *** code-R has joined #openstack-nova | 08:27 | |
| * bauzas invoking the shaman dance to awake some Aussie folk | 08:27 | |
| bauzas | tonyb: I need your Requirements PTL hat :p | 08:27 |
| *** GB21 has joined #openstack-nova | 08:28 | |
| *** code-R_ has joined #openstack-nova | 08:29 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Testing DROP SERVICE_HOST patch https://review.openstack.org/343569 | 08:31 |
| bauzas | zigo: file a bug report and I'll triage it so we could discuss about the proper way to fix it | 08:31 |
| bauzas | zigo: either way, it's an unittest, right? if so, that's not impacting our users, rather our deployers that run tests against our code | 08:32 |
| *** code-R has quit IRC | 08:32 | |
| bauzas | zigo: so, I get your point, I just think it needs to be fixed properly in time without really needing to be critical :) | 08:32 |
| *** dimtruck is now known as zz_dimtruck | 08:33 | |
| zigo | Right. | 08:33 |
| zigo | bauzas: FYI, launchpad is broken, and I can't file the bug... :( | 08:33 |
| zigo | Timeout error | 08:33 |
| zigo | Sorry, something just went wrong in Launchpad. | 08:33 |
| tonyb | bauzas: 'sup? | 08:34 |
| bauzas | tonyb: hola | 08:34 |
| bauzas | \o/ | 08:34 |
| *** hoangcx has joined #openstack-nova | 08:35 | |
| bauzas | tonyb: so, zigo pointed me that Nova honors the global requirements on netaddr which is cool, but unfortunatly we have an unittest that fails for the lower bound version | 08:35 |
| zigo | bauzas: tonyb: https://bugs.launchpad.net/nova/+bug/1623838 | 08:36 |
| openstack | Launchpad bug 1623838 in OpenStack Compute (nova) "Nova requires netaddr >= 0.7.12 which is not enough" [Undecided,New] | 08:36 |
| tonyb | zigo, bauzas: okay. Open a bug against nova an requirements but it wont get fixed in Newton. | 08:36 |
| bauzas | zigo: https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e2 introduces modified_eui64() which is not available for netaddr-0.7.12 | 08:36 |
| zigo | bauzas: query ... | 08:36 |
| tonyb | zigo, bauzas: we have it on the list of high priority things to fix in Ocata | 08:36 |
| bauzas | tonyb: my question is, what would be the right approach ? bump the lower version in the global requirements ? | 08:37 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 08:37 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: remove py26 compat code in "get_console_output" https://review.openstack.org/334480 | 08:37 |
| zigo | tonyb: As a package maintainer, it is my view that it should be fixed in global-requirements.txt, even after the freeze... | 08:37 |
| tonyb | bauzas: Yeah that's the correct fix | 08:37 |
| zigo | (or fixing nova if that's not too complicated / risky) | 08:37 |
| tonyb | bauzas: but we can't do that now because of where we're at in the cycle and once stable/newton is branched we can't fix it at all | 08:37 |
| bauzas | tonyb: yeah, we're branching tomorrow, I know :) | 08:38 |
| tonyb | zigo: You can certainly advocate for that but in the past wev'r said we wont do that thing | 08:38 |
| bauzas | tonyb: but say we fix the lower bound in the global requirements, can we allow some requirements upgrade in the stable branch that will be newton ? | 08:39 |
| tonyb | zigo: we've also stated that packagers shoudl whereever possible stick with the versions in u-c | 08:39 |
| zigo | tonyb: Yeah, and I've been very vocal that this was a really stupid way to handle things. | 08:39 |
| zigo | tonyb: That's like if we were saying: "Oh, we have a bug, but because we're in freeze, let's not address it..." | 08:39 |
| tonyb | zigo: sure I get that but the knock on effects are big | 08:40 |
| zigo | tonyb: I do agree with u-c, we try to do that as much as possible. I just discovered this because I forgot to backport netaddr within OpenStack infra (because I'm now building package there...). | 08:40 |
| tonyb | bauzas: we can only block things on stable branches not raise minimum requirements | 08:40 |
| *** roeyc has joined #openstack-nova | 08:41 | |
| bauzas | tonyb: okay, I see | 08:41 |
| tonyb | zigo: It seems that in the last week we;ve had many people test lower bounds (for $reasons) if ony it'd happend 2 weeks ago :/ | 08:41 |
| bauzas | tonyb: zigo: either way, the bug is in the tests, there is no big deal I guess | 08:42 |
| zigo | No big deal. | 08:42 |
| bauzas | we could do some hack in Nova and backport that if that's a mess | 08:42 |
| zigo | It just feels completely wrong that we can't fix a bug when we discover it. | 08:42 |
| zigo | Anyway, I got to go. | 08:43 |
| zigo | TTYL | 08:43 |
| bauzas | zigo: this is the price to pay for stability | 08:43 |
| tonyb | zigo: sure but some times we need to be pragmatic. | 08:43 |
| *** moshele has joined #openstack-nova | 08:44 | |
| *** derekh has joined #openstack-nova | 08:46 | |
| *** jfding has quit IRC | 08:48 | |
| bauzas | frickler: around ? | 08:48 |
| bauzas | frickler: I'm looking at https://bugs.launchpad.net/nova/+bug/1623838 | 08:50 |
| openstack | Launchpad bug 1623838 in OpenStack Compute (nova) "Nova requires netaddr >= 0.7.12 which is not enough" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | 08:50 |
| *** hoangcx has quit IRC | 08:50 | |
| frickler | bauzas: yep | 08:50 |
| bauzas | frickler: and https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e2 introduced some netaddr API call that isn't supported by the lower bound we honot | 08:50 |
| bauzas | honor* | 08:50 |
| frickler | bauzas: fyi, zigo's bug is useraffecting IMO, just commented on the bug | 08:50 |
| *** danpb has joined #openstack-nova | 08:51 | |
| bauzas | frickler: oh f* you're right, that's a lib | 08:51 |
| bauzas | frickler: so I just looked at https://pythonhosted.org/netaddr/_modules/netaddr/eui.html#EUI.modified_eui64 | 08:52 |
| bauzas | frickler: any reason why you used that method ? I could just copy the object and use the regular eui64() call, right? | 08:53 |
| *** tuanla has joined #openstack-nova | 08:54 | |
| frickler | bauzas: yes, one can revert that part. I think I saw it used similarly in neutron, let me do a quick search, maybe neutron code has the same issue | 08:54 |
| *** ihrachys has joined #openstack-nova | 08:55 | |
| bauzas | frickler: I'll handle the bugfix | 08:55 |
| *** hoangcx has joined #openstack-nova | 08:55 | |
| *** ratailor has quit IRC | 08:57 | |
| *** jdurgin has quit IRC | 08:57 | |
| *** tuanla_ has quit IRC | 08:57 | |
| jojden | any idea regarding nova evacuate ? | 08:58 |
| johnthetubaguy | bauzas: I just saw that pop up, what you are thinking we could do? | 08:58 |
| johnthetubaguy | jojden: does this help: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/ | 08:58 |
| bauzas | johnthetubaguy: I'm testing netaddr in my local venv to make sure I can fix that code section | 08:59 |
| jojden | Hi johnthetubaguy | 08:59 |
| jojden | i done the cli mode | 08:59 |
| jojden | but when i try with novaclient | 08:59 |
| jojden | i am getting error | 08:59 |
| bauzas | johnthetubaguy: then, I'll provide a bugfix against https://bugs.launchpad.net/nova/+bug/1623838 so we won't need to bump the lower version (for us) which is problematic close to the RC1 | 08:59 |
| openstack | Launchpad bug 1623838 in OpenStack Compute (nova) "Nova requires netaddr >= 0.7.12 which is not enough" [Critical,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | 08:59 |
| *** jdurgin has joined #openstack-nova | 08:59 | |
| jojden | self.nova_conn.servers.evacuate(server='6c3e0048-fd71-4f5a-bd70-43dd8b6cbf29') | 09:00 |
| jojden | this is the code I am using | 09:00 |
| jojden | and getting error as Instance 6c3e0048-fd71-4f5a-bd70-43dd8b6cbf29 could not be found. (HTTP 404) (Request-ID: req-562759cc-478c-4c15-8ca8-3f8427148d51) | 09:00 |
| johnthetubaguy | bauzas: OK, that probably is our only option at this point | 09:00 |
| jojden | but I have that instance with id 6c3e0048-fd71-4f5a-bd70-43dd8b6cbf29 | 09:00 |
| johnthetubaguy | jojden: I would check you are using credentials that have access | 09:01 |
| *** efoley has joined #openstack-nova | 09:02 | |
| jojden | yes , I have, otherwise we will be getting error upon connection right ? | 09:02 |
| johnthetubaguy | jojden: just see if you can get the instance show | 09:02 |
| johnthetubaguy | jojden: no, 404 could be an authorisation error, as its url related (it could also be loads of other things... sadly) | 09:03 |
| jojden | i am getting hypervisor list using the same connection | 09:04 |
| *** lpetrut has joined #openstack-nova | 09:07 | |
| kevinbenton | johnthetubaguy: may i have a brief moment of yours. I'm experiencing a crisis of sanity :) | 09:08 |
| johnthetubaguy | kevinbenton: not sure I am expert fixing those, but I could try :) | 09:08 |
| johnthetubaguy | jojden: hmm, thats confusing | 09:08 |
| jojden | ok | 09:08 |
| kevinbenton | johnthetubaguy: i take it at some point we have started using os-vif to plug interfaces? | 09:08 |
| jojden | i will do more debugging | 09:09 |
| johnthetubaguy | jojden: I haven't got much experience debugging the client I am afraid, probably worth trying to get the dubug logs getting out of that | 09:09 |
| johnthetubaguy | kevinbenton: yep | 09:09 |
| jojden | sure | 09:09 |
| johnthetubaguy | kevinbenton: well, for a subset of vifs, just ovs and linux bridge I think | 09:09 |
| jojden | thank you johnthetubaguy | 09:09 |
| *** tuanla_ has joined #openstack-nova | 09:09 | |
| johnthetubaguy | kevinbenton: danpb will be able to help with the details | 09:10 |
| kevinbenton | johnthetubaguy: from what i can see, those plugins do not respect the MTU in the neutron network model at all | 09:10 |
| kevinbenton | johnthetubaguy: e.g. https://github.com/openstack/os-vif/blob/stable/newton/vif_plug_ovs/ovs.py#L109-L110 | 09:10 |
| johnthetubaguy | kevinbenton: oh, that seems surprizing, did you have a dig in os vif about that already? | 09:10 |
| johnthetubaguy | looking | 09:10 |
| johnthetubaguy | oh, bummer | 09:10 |
| johnthetubaguy | kevinbenton: I bet that code for os-vif branched before we starting consume the neutron MTU | 09:11 |
| kevinbenton | johnthetubaguy: so this next question sucks, but how do we revert in the least damaging way? | 09:11 |
| kevinbenton | johnthetubaguy: that it did | 09:11 |
| kevinbenton | johnthetubaguy: we were fixing MTU after that | 09:11 |
| kevinbenton | johnthetubaguy: we got some stuff into os-vif, but not this apparently | 09:11 |
| johnthetubaguy | kevinbenton: thats a tricky question, as we are past the lib freeze of course | 09:11 |
| *** tuanla has quit IRC | 09:11 | |
| danpb | just submit patches to os-vif and we can cut a new bug fix release with it in | 09:11 |
| bauzas | but we can't bump the requirements in Newton | 09:12 |
| kevinbenton | danpb: from what i can see, it will require a model change in os-vif as well to actually store the MTU on the objects | 09:12 |
| kevinbenton | no MTU anywhere in here: http://git.openstack.org/cgit/openstack/os-vif/tree/os_vif/objects/vif.py | 09:12 |
| danpb | bauzas: IMHO the process shouldn't be so strict as to prevent us getting regression fixes released | 09:12 |
| danpb | bauzas: if needed we can do it on an os-vif branch so we don't pull in any other features from os-vif apart from the fix | 09:13 |
| danpb | kevinbenton: yep, seems to be missing | 09:13 |
| danpb | anyway, no matter what we do, getting it into os-vif is the first step | 09:14 |
| danpb | once we have that done, then we can look at how we get it included in nova for neutron | 09:14 |
| kevinbenton | danpb: you mean for Newton, right? | 09:14 |
| danpb | yes | 09:14 |
| bauzas | danpb: sure, I understand that is an important regression | 09:14 |
| danpb | damn, i'll be glad when we're off the newton release | 09:14 |
| * johnthetubaguy is going to be glad when the newton/neutron thing goes away | 09:15 | |
| danpb | snap! | 09:15 |
| johnthetubaguy | heh | 09:15 |
| kevinbenton | johnthetubaguy: definitely :) | 09:15 |
| danpb | new rule for release naming: must not be easily confused with a project name | 09:15 |
| kevinbenton | danpb: ok. i'll start some patches | 09:15 |
| bauzas | tonyb was around 30 mins earlier, we can hassle him for a bit again :p | 09:15 |
| johnthetubaguy | danpb: feels like we should do that proper fix even if we can't get it into newton, so that makes sense | 09:15 |
| tonyb | 'sup? | 09:15 |
| johnthetubaguy | new os-vif release? | 09:15 |
| bauzas | tonyb: yet another fun story | 09:15 |
| tonyb | bauzas: Oh joy ;P | 09:16 |
| bauzas | tonyb: tl;dr: os-vif doesn't take correct MTUs so there should be a new release | 09:16 |
| danpb | johnthetubaguy: yeah, we'll need it for sure for ocata | 09:16 |
| bauzas | AFAIU, newton is regressing on that | 09:16 |
| bauzas | tonyb: so the big question is, how can we ship that so close to our branching ? | 09:16 |
| bauzas | given the bugfix isn't yet implemented | 09:17 |
| *** mingyu has quit IRC | 09:17 | |
| johnthetubaguy | will be a raise the min-requirements thing | 09:17 |
| *** mingyu has joined #openstack-nova | 09:17 | |
| tonyb | bauzas: It'd have to be a high impact bug | 09:17 |
| tonyb | bauzas: unless we can blacklist the latest release of os-vif | 09:17 |
| johnthetubaguy | mtu broken for all new vms using neutron? | 09:18 |
| johnthetubaguy | since the beginning of os-vif | 09:18 |
| tonyb | johnthetubaguy: *sigh* that's bad | 09:19 |
| *** mvk has quit IRC | 09:20 | |
| tonyb | bauzas, danpb get the release made and we can probably do it as long as nova a neutron sign off on it | 09:20 |
| bauzas | tonyb: yeah I told you it was a fun story | 09:20 |
| tonyb | I think they're the only users of os-vif | 09:20 |
| kevinbenton | tonyb: neutron doesn't even use os-vif yet | 09:20 |
| kevinbenton | tonyb: so it's completely up to nova | 09:20 |
| danpb | johnthetubaguy: kevinbenton also this clearly illustrates a tempest testing gap | 09:20 |
| danpb | we should have something written to validate MTU | 09:21 |
| *** gabor_antal has joined #openstack-nova | 09:21 | |
| johnthetubaguy | danpb: +1 | 09:21 |
| bauzas | danpb: I had the same problem with ceph testing yesterday, and I tend to think we should have some in-tree devstack testing | 09:21 |
| kevinbenton | danpb: i think the issue is that the gate is too intelligent at setting that config value :) | 09:21 |
| ihrachys | danpb: + on that, just discussed that yesterday with amuller... | 09:21 |
| tonyb | danpb: Yeah add a tempest test also (but that may fail on liberty and mistaka I guess) :( | 09:22 |
| ihrachys | danpb: we need both tempest AND fullstack in neutron to validate the neutron connectivity sanity. | 09:22 |
| bauzas | mistaka ? :p | 09:22 |
| bauzas | like, the one who did a fault ? :p | 09:22 |
| *** hieulq has quit IRC | 09:22 | |
| tonyb | bauzas: ;p | 09:22 |
| *** tiendc has quit IRC | 09:22 | |
| kevinbenton | tonyb: we actually fixed in mitaka and back-ported to liberty | 09:22 |
| *** hieulq has joined #openstack-nova | 09:22 | |
| bauzas | tonyb: you're begged, it's 9pm for you, right? | 09:22 |
| kevinbenton | tonyb: it was switch to os-vif that messed up | 09:23 |
| bauzas | oops | 09:23 |
| *** tiendc has joined #openstack-nova | 09:23 | |
| bauzas | you're pardoned | 09:23 |
| kevinbenton | tonyb: so a tempest test should pass on all 3 | 09:23 |
| johnthetubaguy | kevinbenton: yeah, good point, that test should pass | 09:23 |
| tonyb | kevinbenton: Oh I thought we had os-vif in mitaka ... Happy to be wrong | 09:23 |
| kevinbenton | tonyb: for some stuff, not for the plugging though | 09:23 |
| ihrachys | kevinbenton: I was looking at introducing some mtu related tests in fullstack lately: https://review.openstack.org/#/c/369470/ now that we hit bug 1622017. but that would not catch the nova glitch. | 09:24 |
| openstack | bug 1622017 in neutron "OVS agent is not removing VLAN tags before tunnels when configured with native OF interface" [High,In progress] https://launchpad.net/bugs/1622017 - Assigned to IWAMOTO Toshihiro (iwamoto) | 09:24 |
| johnthetubaguy | it was more a twinkle in our eye in mitaka | 09:24 |
| kevinbenton | ok, time to work on a fix. can someone point me to a guide on how to submit a patch to openstack? | 09:24 |
| kevinbenton | :D | 09:24 |
| johnthetubaguy | ihrachys: kevinbenton: did you want to work out how to test / validate the MTU is correct? | 09:25 |
| ihrachys | in the meantime, I will search for a guide on review guidelines | 09:25 |
| ihrachys | johnthetubaguy: not just correct on devices (that we validate in functional tests for neutron), but that connectivity with MTU sized frames actually work (ping -M do -s <mtu>) | 09:25 |
| *** alex_xu has quit IRC | 09:26 | |
| johnthetubaguy | oh, that sounds even better | 09:26 |
| johnthetubaguy | do you think the test could be in place in the next day or so? | 09:26 |
| johnthetubaguy | of is that make believe | 09:26 |
| johnthetubaguy | or | 09:26 |
| ihrachys | johnthetubaguy: but we need tempest part too; fullstack is just to validate that neutron allows connectivity itself (for multiple config scenarios) | 09:26 |
| johnthetubaguy | ah, no nova in the fullstack thingy | 09:27 |
| kevinbenton | johnthetubaguy: we could conceivably request the gate set a smaller MTU for everything | 09:27 |
| ihrachys | timing wise, I think I can try to craft something this week for tempest; I was thinking to spend time on fullstack but seems like we have a more generic problem in N now than just 'mtu incorrect in some particular deployment scenario' | 09:27 |
| *** alex_xu has joined #openstack-nova | 09:27 | |
| kevinbenton | johnthetubaguy: that alone should break things (SSH hangs) | 09:28 |
| johnthetubaguy | oh, miss configure it, but neutron network has the correct MTU? | 09:28 |
| johnthetubaguy | I like the sound of that | 09:28 |
| kevinbenton | johnthetubaguy: configure it to 1400, so neutron is setting 1400 on everything | 09:28 |
| kevinbenton | johnthetubaguy: then if nova doesn't set the right MTU, instances wouldn't be reachable | 09:29 |
| kevinbenton | johnthetubaguy: or even something silly like 800 to really ensure it breaks | 09:29 |
| danpb | johnthetubaguy: kevinbenton tonyb: you know what, we don't neccessarily need an os-vif release to fix this in nova for newton | 09:29 |
| johnthetubaguy | kevinbenton: could you get a patch up for that, so it can depend on our fix? | 09:29 |
| johnthetubaguy | danpb: can we just set self.config.mtu? | 09:29 |
| danpb | we can just edit nova/virt/libvirt/vif.py in the _plug_os_vif() method to add some special case hack to set mtu directly after os-vif has done its work | 09:29 |
| johnthetubaguy | oh... good idea | 09:30 |
| ihrachys | makes sense | 09:30 |
| *** jlanoux_ has quit IRC | 09:30 | |
| ihrachys | you control the code, so you should be able to consider the attribute 'kinda public' for a cycle | 09:30 |
| *** mingyu has quit IRC | 09:30 | |
| danpb | i'd still want to see us do the proper fix in os-vif first though | 09:30 |
| *** jlanoux has joined #openstack-nova | 09:30 | |
| kevinbenton | forcing self.config.mtu won't work for mixed encapsulation deployments | 09:31 |
| *** mingyu has joined #openstack-nova | 09:31 | |
| ihrachys | kevinbenton: oh self.config is global for all networks? bummer. | 09:31 |
| kevinbenton | unless you're changing the driver before every plug call | 09:31 |
| ihrachys | a driver per network? :) | 09:31 |
| kevinbenton | driver is stored globally in os-vif | 09:32 |
| kevinbenton | from what i can see | 09:32 |
| danpb | what is this self.config thing you're referring to ? | 09:32 |
| johnthetubaguy | right, I was assuming the later, although I prefer danpb's suggestion I think | 09:32 |
| johnthetubaguy | I am thinking the OvsPlugin.config.mtu | 09:32 |
| danpb | oh, that's the global default MTU in os-vif | 09:32 |
| danpb | the mtu from the port binding should always override that, if set in the port binding | 09:33 |
| *** jdurgin has quit IRC | 09:33 | |
| *** avolkov_ has joined #openstack-nova | 09:33 | |
| johnthetubaguy | you mean on the network? I guess thats the same thing | 09:33 |
| johnthetubaguy | it was this bit that looks like it doesn't do that: https://github.com/openstack/os-vif/blob/stable/newton/vif_plug_ovs/ovs.py#L109-L110 | 09:34 |
| johnthetubaguy | we could be reading it all wrong | 09:34 |
| *** phuongnh has quit IRC | 09:34 | |
| *** phuongnh has joined #openstack-nova | 09:34 | |
| ihrachys | or, nova could leave os-vif do everything as-is, then after plug get back and fix the mtu | 09:34 |
| *** avolkov_ has left #openstack-nova | 09:35 | |
| danpb | ihrachys: that's exactly what i meant | 09:35 |
| danpb | nova can just fix mtu after the fat | 09:35 |
| danpb | s/fat/fact/ | 09:35 |
| ihrachys | ok, misread, sorry. | 09:35 |
| ihrachys | I thought more like patching the lib to use the right value. | 09:36 |
| danpb | in nova.virt.libvirt.vif.VIFDriver._plug_os_vif() just add a set mtu call after os_vif.plug() | 09:36 |
| *** cmd-t has joined #openstack-nova | 09:36 | |
| danpb | ihrachys: you can't patch the lib, because the data model passed into the lib doesn't record the mtu value | 09:36 |
| cmd-t | HI | 09:36 |
| cmd-t | I am using python-novaclient | 09:37 |
| *** thorst has joined #openstack-nova | 09:38 | |
| cmd-t | can anyone suggest how to use sever-group to launch a vm (for anti-affinity). I am not asking about cli commands, from cli it cn be done by, using --hints={server-group-id}... how to do it while coding | 09:39 |
| cmd-t | for what parameter {key:value} i need to pass, | 09:39 |
| *** sree has quit IRC | 09:41 | |
| johnthetubaguy | cmd-t: take a look at how the CLI is implemented, that might help you: https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/shell.py#L379 | 09:42 |
| cmd-t | i did, and i found this parameter.. scheduler_hints | 09:44 |
| *** markvoelker has joined #openstack-nova | 09:44 | |
| cmd-t | si i tried it like: nova.servers.create( name, image, flavor, nics=[{'net-id': network_id}], scheduler_hints={ 'name': 'test_server_group', 'policies': ['anti-affinity']} ) | 09:44 |
| cmd-t | nut still whenever i am launching new vm its launching in same compute node | 09:45 |
| cmd-t | nut==but | 09:45 |
| *** yamahata has quit IRC | 09:49 | |
| *** markvoelker has quit IRC | 09:49 | |
| *** mvk has joined #openstack-nova | 09:50 | |
| johnthetubaguy | cmd-t: is it not scheduler_hints={"group": "test_server_group"} | 09:50 |
| *** sree has joined #openstack-nova | 09:51 | |
| johnthetubaguy | danpb: ihrachys: kevinbenton: just checking, who is working on what bits? | 09:51 |
| kevinbenton | i'm working on os-vif | 09:51 |
| cmd-t | so i just need to pass name only.. | 09:53 |
| johnthetubaguy | cmd-t: I think thats all you need to pass on the CLI, but I could be remembering wrongly, what CLI cmd worked for you | 09:54 |
| *** mingyu has quit IRC | 09:54 | |
| *** sree has quit IRC | 09:56 | |
| cmd-t | i am not working in CLI, and in CLI first we need to create server-group and then we need to pass its id, as --hints={server-group-id} while launching vm | 09:56 |
| johnthetubaguy | cmd-t: right, it will be the same with the client | 09:56 |
| danpb | johnthetubaguy: i'm working on nothing :-) | 09:57 |
| danpb | but i'll review what's produced | 09:57 |
| *** mingyu has joined #openstack-nova | 09:57 | |
| *** lujinluo has quit IRC | 09:57 | |
| ihrachys | kevinbenton: tempest | 09:57 |
| johnthetubaguy | danpb: I can take a look at that mtu hack | 09:57 |
| ihrachys | johnthetubaguy: sorry, meant for you | 09:57 |
| *** phuongnh has quit IRC | 09:58 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue https://review.openstack.org/335884 | 09:58 |
| ihrachys | do we have a bug reported? | 09:58 |
| johnthetubaguy | ihrachys: cool, is that the configuration hack kevinbenton mentioned? | 09:58 |
| johnthetubaguy | ihrachys: oops, probably not, we should do that | 09:59 |
| *** phuongnh has joined #openstack-nova | 09:59 | |
| ihrachys | johnthetubaguy: no, an actual scenario that will try to ping between nodes using MTU sized frames | 09:59 |
| cmd-t | johnthetubaguy | 09:59 |
| ihrachys | johnthetubaguy: size as returned by neutron mtu api extension | 09:59 |
| johnthetubaguy | OK, just wondering what would test this bug quickly | 09:59 |
| cmd-t | johnthetubaguy: so how to use it from python program .... | 10:00 |
| johnthetubaguy | but if we can get the better think right of, thats even better | 10:00 |
| *** salv-orl_ has quit IRC | 10:01 | |
| johnthetubaguy | cmd-t: I mean that will have to create the security group and then boot the instance, if you look at shell.py in python-novaclient you should be able to work out what the python program would call. FWIW, this is what the docs say: http://docs.openstack.org/user-guide/sdk-overview.html#per-project-client-libraries | 10:02 |
| kevinbenton | johnthetubaguy: do you have a bug for this yet? | 10:03 |
| ihrachys | johnthetubaguy: hm, I am thinking, without config changes the test won't catch the failure unless neutron network is jumbo frame aware (meaning, custom configuration for devstack and infra); because otherwise the mtu on the tap device will mismatch what neutron may want but generally will be higher than that, so it's ok. | 10:03 |
| johnthetubaguy | kevinbenton: nope, if you could create one, that would be great, you have better context for the bug | 10:03 |
| johnthetubaguy | ihrachys: could we configure the MTU to be tiny (like doesn't work tiny), then make it bigger in the neutron network? | 10:04 |
| *** phuongnh has quit IRC | 10:04 | |
| *** phuongnh has joined #openstack-nova | 10:05 | |
| ihrachys | johnthetubaguy: yeah, another way is downgrading network_device_mtu in nova to something below neutron network MTU (1400?) | 10:05 |
| kevinbenton | johnthetubaguy: working on it | 10:05 |
| ihrachys | johnthetubaguy: but the thing is, without a dedicated job you won't hit it in gate. | 10:05 |
| kevinbenton | ihrachys: yeah, actually we could have the gate just set network_device_mtu to something tiny | 10:05 |
| johnthetubaguy | ihrachys: I was just thinking we do a depends-on chain of patches, to prove the test works | 10:05 |
| johnthetubaguy | I mean prove the patch works | 10:06 |
| ihrachys | johnthetubaguy: so there is still value in the tempest scenario; and it may fail in some setups; but it's not like gate will provide us coverage | 10:06 |
| *** moshele has quit IRC | 10:06 | |
| ihrachys | time for cross-project fullstack? :P | 10:06 |
| *** moshele has joined #openstack-nova | 10:06 | |
| ihrachys | johnthetubaguy: btw, didn't nova remove the mtu setting very lately? | 10:06 |
| ihrachys | johnthetubaguy: we could set throw-away scaffolding with devstack-gate patches and all | 10:07 |
| *** efoley has quit IRC | 10:07 | |
| ihrachys | johnthetubaguy: I meant https://review.openstack.org/#/c/319543/ | 10:07 |
| johnthetubaguy | ihrachys: oops, you are right, but os-vif still has it I think | 10:07 |
| *** mingyu has quit IRC | 10:08 | |
| kevinbenton | johnthetubaguy: https://bugs.launchpad.net/os-vif/+bug/1623876 | 10:09 |
| openstack | Launchpad bug 1623876 in os-vif "nova is not setting the MTU provided by Neutron" [Undecided,New] - Assigned to Kevin Benton (kevinbenton) | 10:09 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Remove use of modified_eui64() https://review.openstack.org/370665 | 10:09 |
| *** efoley has joined #openstack-nova | 10:09 | |
| *** hoangcx has quit IRC | 10:09 | |
| *** roeyc has quit IRC | 10:09 | |
| bauzas | frickler: johnthetubaguy: zigo: ^ | 10:09 |
| ihrachys | johnthetubaguy: I see. would it make sense to deprecate it too? | 10:10 |
| cmd-t | johnthetubaguy: still dont understand how to solve it | 10:10 |
| *** gongysh has joined #openstack-nova | 10:10 | |
| openstackgerrit | Kevin Benton proposed openstack/os-vif: Add MTU to Network model and use it in plugging https://review.openstack.org/370667 | 10:10 |
| cmd-t | johnthetubaguy: can u provide some more help | 10:11 |
| *** mingyu has joined #openstack-nova | 10:11 | |
| *** mingyu has quit IRC | 10:12 | |
| cmd-t | johnthetubaguy: one more thing is i just tried it as u said scheduler_hints={"group": "test_server_group"}, and it says Invalid input for field/attribute group. Value: 'test_server_group' is not a 'uuid' | 10:13 |
| *** mingyu has joined #openstack-nova | 10:14 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue https://review.openstack.org/335884 | 10:15 |
| johnthetubaguy | cmd-t: ah, good point, it should be the uuid of the server group | 10:15 |
| lyarwood | mdbooth: ^ I've rebased my newton-rc-potential bugfix now that your test rewrite has landed | 10:16 |
| lyarwood | mdbooth: any chance of a quick review this morning if you have time | 10:16 |
| cmd-t | well i tried that also | 10:17 |
| cmd-t | but what could be the key for uuid, if it is in dict format | 10:17 |
| *** mbirru has quit IRC | 10:19 | |
| johnthetubaguy | cmd-t: {"group": "<uuid goes here>"}, you get the uuid back from the api when you create the server group: http://developer.openstack.org/api-ref/compute/?expanded=create-server-group-detail | 10:19 |
| cmd-t | johnthetubaguy: it only takes **kwargs or from my understanding a dictionary only.... :( | 10:19 |
| cmd-t | johnthetubaguy: thanks again but i have check all of these things.... | 10:20 |
| cmd-t | and tried also | 10:20 |
| cmd-t | johnthetubaguy: and as u asked yes i can get uuid of server_group | 10:21 |
| *** mingyu has quit IRC | 10:21 | |
| johnthetubaguy | cmd-t: so the error you got says just supply a uuid instead of where you put the name | 10:21 |
| *** akrivoka has joined #openstack-nova | 10:21 | |
| *** zz_dimtruck is now known as dimtruck | 10:24 | |
| *** mingyu has joined #openstack-nova | 10:25 | |
| akrivoka | I'm failing to get novaclient to authenticate using keystone sessions, NotFound is raised when I try to list flavors in this example https://paste.fedoraproject.org/428299/47393422/ | 10:28 |
| cmd-t | johnthetubaguy: u mean to say {'group':'uuid'} | 10:28 |
| akrivoka | could anyone tell me what I'm doing wrong in ^ | 10:28 |
| *** gszasz has joined #openstack-nova | 10:28 | |
| mdbooth | lyarwood: On a call, but I'll get on it straight after. | 10:32 |
| openstackgerrit | Krzysztof Świątek proposed openstack/nova: Fix issue with not removing rbd rescue disk https://review.openstack.org/314928 | 10:32 |
| *** dimtruck is now known as zz_dimtruck | 10:33 | |
| openstackgerrit | Kevin Benton proposed openstack/nova: Pass MTU into os-vif Network object https://review.openstack.org/370679 | 10:35 |
| kevinbenton | johnthetubaguy, danpb: ^^ | 10:36 |
| cmd-t | johnthetubaguy: it tried this {'group':'uuid'} didn't work, should i pass some other key(instead of 'group'), what it would be | 10:36 |
| johnthetubaguy | cmd-t: what error did you get with group and its uuid | 10:37 |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Override MTU for os_vif attachments https://review.openstack.org/370681 | 10:39 |
| *** moshele has quit IRC | 10:39 | |
| lyarwood | mdbooth: cool, thanks | 10:41 |
| cmd-t | johnthetubaguy: error Instance group aea4f782-0bf1-4852-92a0-efef20f23ac3 could not be found. | 10:41 |
| johnthetubaguy | cmd-t: can you show that instance group in the API? looks like you are very close, it was expecting an instance group | 10:42 |
| *** moshele has joined #openstack-nova | 10:42 | |
| cmd-t | johnthetubaguy: u mean output of nova server-group-list | 10:44 |
| cmd-t | as i have only one group: | 10:44 |
| pkoniszewski | hi nova folks, we have one new 'feature' that is backward incompatible, i.e., it is possible to live migrate an instance with read-only config drive attached from mitaka to newton, but it cannot be live migrated back to mitaka, should we communicate it through a release note? | 10:44 |
| cmd-t | aea4f782-0bf1-4852-92a0-efef20f23ac3 | test_server_group | [u'anti-affinity'] | [] | {} | 10:45 |
| pkoniszewski | i'm talking about this change https://review.openstack.org/#/c/234659/ which does not have release note at all right now (but i think it deserves one) | 10:45 |
| johnthetubaguy | cmd-t: its possible the credentials are not quite right, but its hard to know | 10:45 |
| cmd-t | johnthetubaguy: which credentials, because i am doing all my requests from admin user | 10:46 |
| *** ratailor has joined #openstack-nova | 10:47 | |
| johnthetubaguy | pkoniszewski: yeah, probably worth adding a reno for that | 10:47 |
| cmd-t | johnthetubaguy: i think its just a format of passing values that i am not able to understand | 10:47 |
| johnthetubaguy | cmd-t: it seems to say it can't find the instance group with the uuid of the one you want, so it sounds like its a permissions error rather than a format error, but I could be wong | 10:48 |
| *** rtheis has joined #openstack-nova | 10:48 | |
| *** kylek3h has joined #openstack-nova | 10:48 | |
| *** kylek3h has quit IRC | 10:48 | |
| *** kylek3h has joined #openstack-nova | 10:48 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: Check hypervisor version while getting bus type https://review.openstack.org/282398 | 10:48 |
| cmd-t | johnthetubaguy: i more thing i like to share is, the call i am making is going to file ..... python2.7/site-packages/novaclient/v2/servers.py | 10:49 |
| cmd-t | in this file i m getting errors.. because after that its openstack REST request | 10:50 |
| *** sree has joined #openstack-nova | 10:50 | |
| *** hieulq has quit IRC | 10:51 | |
| *** hieulq has joined #openstack-nova | 10:51 | |
| johnthetubaguy | danpb: I am hitting NotImplementedError: Cannot load 'bridge' in the base class when I do vif.network.bridge in the unit tests, have you got any pointers before I go digging deeper? | 10:54 |
| cmd-t | johnthetubaguy: oh, sorry that was a different file, | 10:54 |
| cmd-t | johnthetubaguy: my error file is ..../python2.7/site-packages/novaclient/v2/server_groups.py | 10:54 |
| *** sree has quit IRC | 10:55 | |
| *** ftersin has joined #openstack-nova | 10:55 | |
| cmd-t | To be more specific, i am getting error in this code ::: | 10:57 |
| cmd-t | johnthetubaguy: | 10:57 |
| cmd-t | def create(self, **kwargs): """Create (allocate) a server group. :rtype: list of :class:`ServerGroup` """ body = {'server_group': kwargs} return self._create('/os-server-groups', body, 'server_group') | 10:57 |
| *** pilgrimstack has quit IRC | 10:58 | |
| johnthetubaguy | cmd-t: so you have having issues creating the server group? | 10:58 |
| cmd-t | sorry nope | 10:59 |
| johnthetubaguy | cmd-t: I think we should move this conversation to #openstack-sdks or #openstack, its not really a Nova dev conversation, you can ping me in those channels, but there may be other folks who can help you better over in those channels. | 11:00 |
| cmd-t | i am able to create group but i don't know how to launch vm with a id of this group | 11:00 |
| *** pilgrimstack has joined #openstack-nova | 11:00 | |
| *** sdague has joined #openstack-nova | 11:01 | |
| *** salv-orlando has joined #openstack-nova | 11:02 | |
| lyarwood | avolkov: hey, re https://review.openstack.org/#/c/290793/ - would you mind taking a look at my latest comments? I'd like to move the _create_volume_bdm call into the try block and add some cleanup code for cases where we fail to reply from the reserve_block_device_name RPC call to the compute. | 11:03 |
| lyarwood | avolkov: happy to do this myself if you don't have time to work on this btw | 11:04 |
| *** lucasagomes is now known as lucas-hungry | 11:06 | |
| *** dave-mccowan has joined #openstack-nova | 11:06 | |
| *** roeyc has joined #openstack-nova | 11:06 | |
| *** salv-orlando has quit IRC | 11:06 | |
| openstackgerrit | John Garbutt proposed openstack/nova: WIP: Override MTU for os_vif attachments https://review.openstack.org/370681 | 11:06 |
| *** bkopilov has quit IRC | 11:08 | |
| *** efoley_ has joined #openstack-nova | 11:10 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Override MTU for os_vif attachments https://review.openstack.org/370681 | 11:12 |
| *** efoley has quit IRC | 11:13 | |
| *** cmd-t has quit IRC | 11:14 | |
| *** jlanoux has quit IRC | 11:14 | |
| *** jlanoux has joined #openstack-nova | 11:14 | |
| *** jlanoux has quit IRC | 11:17 | |
| *** jlanoux has joined #openstack-nova | 11:17 | |
| *** sridharg has quit IRC | 11:19 | |
| *** sudipto_ has quit IRC | 11:20 | |
| *** sudipto has quit IRC | 11:20 | |
| *** bvanhav has joined #openstack-nova | 11:20 | |
| *** mingyu_ has joined #openstack-nova | 11:20 | |
| *** mingyu has quit IRC | 11:23 | |
| *** phuongnh has quit IRC | 11:23 | |
| *** jfding has joined #openstack-nova | 11:24 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Override MTU for os_vif attachments https://review.openstack.org/370681 | 11:24 |
| johnthetubaguy | kevinbenton: ihrachys: so this is the hack that might help us this side of the release, danpb had a quick look at that already | 11:25 |
| *** roeyc has quit IRC | 11:26 | |
| *** thorst has quit IRC | 11:27 | |
| *** salv-orlando has joined #openstack-nova | 11:28 | |
| mdbooth | lyarwood: Jenkins was grumpy. Haven't looked at why yet, but reviewing anyway. | 11:28 |
| mdbooth | lyarwood: unit test failures. | 11:28 |
| *** roeyc has joined #openstack-nova | 11:29 | |
| *** fragatina has joined #openstack-nova | 11:30 | |
| * johnthetubaguy goes for some lunch | 11:33 | |
| *** fragatina has quit IRC | 11:34 | |
| kevinbenton | danpb, ihrachys: so something i'm confused about. why doesn't nova have obj_to_primitive downgrade for this? https://github.com/openstack/nova/blob/master/nova/objects/instance_group.py#L122-L146 | 11:34 |
| kevinbenton | sorry, obj_make_compatible | 11:35 |
| danpb | kevinbenton: dunno - it is not unheard of for nova to make mistakes :-) | 11:36 |
| ihrachys | kevinbenton: as far as I understand the latest comment, it's because prev versions were not on the wire | 11:36 |
| danpb | my understanding was that we need obj_make_compatible for every time we add a new attribute | 11:36 |
| johnthetubaguy | yeah, looks like it got missed | 11:36 |
| ihrachys | so it was easier for folks to just bump without backport rules | 11:36 |
| ihrachys | otherwise a bug | 11:36 |
| kevinbenton | danpb: ok. so i noticed os-vif doesn't have oslo.utils in it's requirements | 11:37 |
| kevinbenton | danpb: should i add it? | 11:37 |
| *** sahid has quit IRC | 11:37 | |
| mdbooth | Could we extend the lifetimes of gerrit sessions? | 11:37 |
| mdbooth | Or at least if you submit a comment after your session has expired it saves the bloody comment after login. | 11:37 |
| danpb | kevinbenton: presumably because we've never needed oslo.utils apis ? | 11:38 |
| danpb | or are we using them and getting the oslo.utils dep indirectly via something else we depend on | 11:38 |
| openstackgerrit | Moshe Levi proposed openstack/nova: pci: remove pci device from claims and allocations when freeing it https://review.openstack.org/369180 | 11:38 |
| kevinbenton | danpb: ok. it's only used in the two drivers | 11:39 |
| kevinbenton | danpb: wasn't sure if that was why | 11:39 |
| kevinbenton | danpb: ok. i'll write the downgrade method | 11:39 |
| *** edmondsw has joined #openstack-nova | 11:42 | |
| *** markvoelker has joined #openstack-nova | 11:45 | |
| *** zenoway has quit IRC | 11:48 | |
| *** rfolco has joined #openstack-nova | 11:49 | |
| *** baoli has joined #openstack-nova | 11:49 | |
| *** markvoelker has quit IRC | 11:50 | |
| openstackgerrit | Kevin Benton proposed openstack/os-vif: Add MTU to Network model and use it in plugging https://review.openstack.org/370667 | 11:52 |
| openstackgerrit | Kevin Benton proposed openstack/nova: Pass MTU into os-vif Network object https://review.openstack.org/370679 | 11:53 |
| *** sahid has joined #openstack-nova | 11:53 | |
| kevinbenton | danpb: https://review.openstack.org/370667 | 11:53 |
| *** mingyu_ has quit IRC | 11:56 | |
| *** mingyu has joined #openstack-nova | 11:58 | |
| *** baoli_ has joined #openstack-nova | 11:58 | |
| *** sahid_ has joined #openstack-nova | 11:58 | |
| kevinbenton | johnthetubaguy: that _plug_os_vif is called for both linux bridge and OVS, right? | 11:58 |
| *** baoli has quit IRC | 12:01 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add hypervisor_links into hypervisor https://review.openstack.org/370732 | 12:01 |
| slaweq | hello, can someone take a look on https://bugs.launchpad.net/nova/+bug/1623809 and tell me if it is reasonable bug or behaviour described there is normal and it shouldn't be fixed/changed? | 12:02 |
| openstack | Launchpad bug 1623809 in OpenStack Compute (nova) "Quota exceeded when spawning instances in server group" [Undecided,New] - Assigned to Slawek Kaplonski (slaweq) | 12:02 |
| *** jkraj has joined #openstack-nova | 12:04 | |
| *** tiendc has quit IRC | 12:04 | |
| gabor_antal | Would someone take a look at this documentation bug? Am I true or missed something? https://bugs.launchpad.net/nova/+bug/1185902 | 12:04 |
| openstack | Launchpad bug 1185902 in OpenStack Compute (nova) ""nova-manage flavor create" produces the wrong help" [Low,Won't fix] | 12:04 |
| *** sahid has quit IRC | 12:05 | |
| gabor_antal | Sorry, wrong link: https://bugs.launchpad.net/nova/+bug/1623908 | 12:05 |
| openstack | Launchpad bug 1623908 in OpenStack Compute (nova) "Nova documentation on creating flavors has depreceated examples" [Undecided,New] - Assigned to Gábor Antal (gabor.antal) | 12:05 |
| bauzas | kevinbenton: johnthetubaguy: danpb: FWIW triaged https://bugs.launchpad.net/nova/+bug/1623876 as critical and Newton blocker, but we can discuss whether it's really important or not | 12:06 |
| openstack | Launchpad bug 1623876 in OpenStack Compute (nova) "nova is not setting the MTU provided by Neutron" [Critical,In progress] - Assigned to Kevin Benton (kevinbenton) | 12:06 |
| johnthetubaguy | bauzas: I think thats accurate, I meant to update that | 12:07 |
| johnthetubaguy | kevinbenton: yeah, your comment is spot on | 12:07 |
| kevinbenton | johnthetubaguy: which one? | 12:07 |
| johnthetubaguy | kevinbenton: I was meaning the one in the review, and _plug_os_vif being called for both cases | 12:08 |
| *** namnh has quit IRC | 12:09 | |
| *** jlanoux has quit IRC | 12:09 | |
| openstackgerrit | jichenjc proposed openstack/nova: Trivial fix remove not used var in parameters https://review.openstack.org/370740 | 12:09 |
| *** TravT has joined #openstack-nova | 12:09 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue https://review.openstack.org/335884 | 12:10 |
| *** sree has joined #openstack-nova | 12:10 | |
| mdbooth | lyarwood: Hope you don't mind, but I updated your patch. | 12:11 |
| *** gongysh has quit IRC | 12:12 | |
| *** xyang1 has joined #openstack-nova | 12:12 | |
| *** pradk has joined #openstack-nova | 12:13 | |
| gabor_antal | slaweq: your report seems reasonable for me. I think you should check the source, why it was happening this way. Maybe it is intentional. | 12:13 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova-specs: Enable to share PCI devices between numa nodes https://review.openstack.org/361140 | 12:13 |
| *** ducttape_ has joined #openstack-nova | 12:14 | |
| avolkov | lyarwood: hi, thanks for you comments, could you please clarify what you are expecting if volume reserving are failed? | 12:14 |
| *** sree has quit IRC | 12:14 | |
| frickler | gabor_antal: this looks like someone recycled a "nova flavor-create" command incorrectly, yes | 12:15 |
| slaweq | gabor_antal: I was checking it and it looks that when "num_instances" is calculated and quota reserved then server_group_members quota is not checked | 12:15 |
| slaweq | I will dig more into it | 12:15 |
| *** claudiub has quit IRC | 12:16 | |
| *** lucas-hungry is now known as lucasagomes | 12:17 | |
| gabor_antal | slaweq: I think it would makes sense if those quotas are checked in somewhere correctly and nova should make them consistent in a way, or just warn the user that somehing magical happens :) | 12:17 |
| gabor_antal | frickler: thanks. I think I fix this | 12:18 |
| avolkov | lyarwood: I mean if we are doing reverve in cinder and it fails, should we really call unreserve? I don't sure for this. | 12:19 |
| slaweq | gabor_antal: as I understand --min-count and --max-count options it wouldn't be "magical" if nova would spawn number of instances which is somewhere between those values provided by user | 12:21 |
| slaweq | for example user spawn vms with "--min-count=2 --max-count=10" but in server_group can be max 5 instances - then IMHO nova should spawn those 5 instances | 12:22 |
| slaweq | is my understanding correct? | 12:22 |
| lyarwood | mdbooth: ah thanks, I removed **self.test_instance without running the damn tests again | 12:23 |
| *** ratailor has quit IRC | 12:23 | |
| *** zz_dimtruck is now known as dimtruck | 12:24 | |
| lyarwood | avolkov: that's something I wanted to look into today if possible but it's seperate to the issue I was talking about in my review tbh | 12:25 |
| openstackgerrit | Krzysztof Świątek proposed openstack/nova: Fix issue with not removing rbd rescue disk https://review.openstack.org/314928 | 12:26 |
| *** maticue has joined #openstack-nova | 12:27 | |
| kashyap | mdbooth: Maybe you've already noticed, there's someone proposing "filesystem pools" similar to storage pools" upstream libvirt -- https://www.redhat.com/archives/libvir-list/2016-September/msg00463.html | 12:28 |
| openstackgerrit | jichenjc proposed openstack/nova: Add keypairs_links into resp https://review.openstack.org/370760 | 12:28 |
| mdbooth | paul-carlton2: What kashyap said ^^^ | 12:28 |
| *** mgoddard has joined #openstack-nova | 12:29 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33 https://review.openstack.org/370732 | 12:30 |
| *** mingyu has quit IRC | 12:30 | |
| avolkov | lyarwood: ook, maybe I didn't understand. your comment for https://review.openstack.org/#/c/290793/10/nova/compute/api.py@3321. Isn't it already in try .. except with https://review.openstack.org/#/c/290793/10/nova/compute/api.py@3303? | 12:32 |
| *** bkopilov has joined #openstack-nova | 12:32 | |
| *** dimtruck is now known as zz_dimtruck | 12:34 | |
| mdbooth | kashyap: There are 5 additions of more than a KLOC in that series! | 12:34 |
| mdbooth | That's a big 'ol patch series. | 12:34 |
| zigo | bauzas: U r0x! :) | 12:35 |
| kashyap | Yeah, I'd imagine someone will ask them to split the series | 12:35 |
| mdbooth | It's already in 8 parts, may be ok | 12:35 |
| mdbooth | I'm not entirely clear what it does, though | 12:35 |
| mdbooth | Why would libvirt want to manipulate individual files in a guest-managed filesystem? | 12:36 |
| lyarwood | avolkov: sorry wasn't watching irc, yeah it's in the that try block but a failure here isn't going to result in any attempt to clean up the bdm | 12:36 |
| kashyap | mdbooth: Maybe you can already review there :-) | 12:36 |
| lyarwood | avolkov: there's a failure case here that we've seen downstream where by the reply from the compute fails and we end up with an orphan bdm that we never clean up | 12:36 |
| mdbooth | That's more akin to what libguestfs does, rather than what libvirt does with storage pools. | 12:37 |
| * mdbooth has probably misunderstood | 12:37 | |
| openstackgerrit | jichenjc proposed openstack/nova: fix broken link in api-ref https://review.openstack.org/370770 | 12:37 |
| lyarwood | avolkov: if we can drop it into the try block below we can just add an additional elif to cover volume_bdm not being set, search for and then remove any orphaned bdm for the instance_uuid and volume_id | 12:37 |
| *** sdake has joined #openstack-nova | 12:38 | |
| *** markvoelker has joined #openstack-nova | 12:38 | |
| *** tuanla_ has quit IRC | 12:38 | |
| *** bks has quit IRC | 12:39 | |
| avolkov | lyarwood: yes, I see your point, will fix, sorry for delay | 12:40 |
| openstackgerrit | jichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33 https://review.openstack.org/370732 | 12:40 |
| *** ducttape_ has quit IRC | 12:42 | |
| *** sahid_ has quit IRC | 12:42 | |
| *** hemanthm|away is now known as hemanthm | 12:44 | |
| *** ljxiash has joined #openstack-nova | 12:44 | |
| *** mriedem has joined #openstack-nova | 12:44 | |
| *** sahid has joined #openstack-nova | 12:45 | |
| *** burgerk has joined #openstack-nova | 12:45 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add keypairs_links into resp https://review.openstack.org/370760 | 12:45 |
| openstackgerrit | jichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33 https://review.openstack.org/370732 | 12:46 |
| lyarwood | avolkov: np at all and thanks :) | 12:47 |
| sdague | alex_xu: https://review.openstack.org/#/c/338051/ missing response parameters | 12:47 |
| *** vladikr has joined #openstack-nova | 12:49 | |
| *** mgoddard_ has joined #openstack-nova | 12:52 | |
| *** mgoddard has quit IRC | 12:52 | |
| *** ljxiash has quit IRC | 12:53 | |
| *** annegentle has joined #openstack-nova | 12:53 | |
| gabor_antal | slaweq: Yes, you are right | 12:54 |
| *** ekuris has quit IRC | 12:55 | |
| slaweq | gabor_antal: thx for help :) | 12:55 |
| gabor_antal | not at all, i think you're right :) | 12:55 |
| *** burgerk has quit IRC | 12:56 | |
| markus_z | mriedem: JFYI, I'll be on vacation for the next 2 weeks. Will be back on 10/04 | 12:56 |
| mriedem | 10-4 that good buddy | 12:57 |
| *** mgoddard_ has quit IRC | 12:58 | |
| markus_z | TIL: "10-4" :D | 12:59 |
| *** hshiina has joined #openstack-nova | 13:00 | |
| *** ociuhandu has quit IRC | 13:01 | |
| *** akrivoka has left #openstack-nova | 13:01 | |
| *** jaosorior has quit IRC | 13:02 | |
| sdague | we should definitely switch from irc to cb radio for all nova communication :) | 13:02 |
| *** jaosorior has joined #openstack-nova | 13:02 | |
| *** burgerk has joined #openstack-nova | 13:03 | |
| mriedem | i get to be burt reynolds if we do | 13:03 |
| *** liverpooler has quit IRC | 13:03 | |
| gabor_antal | :) | 13:03 |
| markus_z | mighty mustache is mandatory | 13:03 |
| bauzas | mriedem: aloha | 13:04 |
| mriedem | mahalo | 13:04 |
| bauzas | mriedem: seen the 2 critical blocking bugs ? :p | 13:05 |
| mriedem | nope | 13:05 |
| bauzas | I love mornings like this | 13:05 |
| mriedem | i'm off in about 10 minutes for the bus, just fyi | 13:05 |
| bauzas | sure, no worries, Nova will burn in the meantime :p | 13:05 |
| mriedem | https://bugs.launchpad.net/nova/+bugs?search=Search&field.importance=Critical&field.status=New&field.status=Incomplete&field.status=Confirmed&field.status=Triaged&field.status=In+Progress&field.status=Fix+Committed | 13:05 |
| bauzas | yuuuup | 13:05 |
| *** oanson has quit IRC | 13:06 | |
| sdague | bauzas: catching up, can you explain what the issue is here - https://bugs.launchpad.net/nova/+bug/1623876 ? | 13:06 |
| openstack | Launchpad bug 1623876 in OpenStack Compute (nova) "nova is not setting the MTU provided by Neutron" [Critical,In progress] - Assigned to Kevin Benton (kevinbenton) | 13:06 |
| mriedem | it's not getting down through os-vif? | 13:06 |
| markus_z | bauzas: http://i0.kym-cdn.com/photos/images/newsfeed/000/962/640/658.png ;) | 13:06 |
| bauzas | one could be breaking people deploying Nova with Ipv6 and an old release | 13:06 |
| mriedem | clarkb: did i tip you off somehow with mtu things the other day? | 13:06 |
| bauzas | sdague: the other means that we don't use the real MTU | 13:06 |
| bauzas | sdague: which could be a problem for example if using bigger ones | 13:07 |
| bauzas | like jumbo packets | 13:07 |
| *** kylek3h has quit IRC | 13:07 | |
| bauzas | oops jumbo frames I meant | 13:07 |
| mriedem | danpb: you said in the bug above that we're working around this in nova? | 13:08 |
| mriedem | but i only see the changes from kevinbenton for os-vif and nova that depends on the os-vif change | 13:08 |
| bauzas | there are 2 changes from kevinbenton | 13:08 |
| bauzas | yeah | 13:08 |
| danpb | mriedem: johnthetubaguy submitted the nova hack | 13:09 |
| danpb | https://review.openstack.org/370681 | 13:09 |
| johnthetubaguy | yeah, just trying to straighten that out | 13:09 |
| openstackgerrit | John Garbutt proposed openstack/nova: Override MTU for os_vif attachments https://review.openstack.org/370681 | 13:10 |
| *** akshai has joined #openstack-nova | 13:10 | |
| johnthetubaguy | kevinbenton: I think that what you meant for the ovs hybrid case ^ | 13:10 |
| johnthetubaguy | ihrachys: do we have that config hack to see if we can prove my hack would fix things? | 13:11 |
| *** DaveJ__ has joined #openstack-nova | 13:11 | |
| sdague | bauzas: have we asking about just doing a req bump? | 13:11 |
| ihrachys | johnthetubaguy: hang on, pushing the tempest test, then I will start scaffolding dependencies | 13:11 |
| sdague | https://review.openstack.org/#/c/370665/1 seems kind of silly cruft | 13:11 |
| *** annegentle has quit IRC | 13:11 | |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] consolidate json handling in util module https://review.openstack.org/361422 | 13:11 |
| mriedem | sdague: the os-vif fix hasn't merged | 13:12 |
| mriedem | so not even close to released | 13:12 |
| ihrachys | kevinbenton: johnthetubaguy: the test at https://review.openstack.org/370804 | 13:12 |
| mriedem | oh nvm | 13:12 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: WIP: try a bigger TIMEOUT_SCALING_FACTOR value for migration tests https://review.openstack.org/370805 | 13:12 |
| mriedem | you were talking about the netaddr one | 13:12 |
| sdague | mriedem: yeh, the netaddr one | 13:12 |
| sdague | seems like lots of folks on the other issue | 13:12 |
| *** cdent has joined #openstack-nova | 13:13 | |
| bauzas | sdague: I discussed about that with tonyb | 13:13 |
| sdague | and? | 13:13 |
| bauzas | sdague: he's afraid of bumping the min ver so close to the newton branch | 13:13 |
| sdague | we haven't been testing with that minimum since forever | 13:14 |
| bauzas | sdague: so I finally went to that approach of fixing the module itself | 13:14 |
| *** sdake_ has joined #openstack-nova | 13:14 | |
| bauzas | sdague: I know | 13:14 |
| sdague | bauzas: but we're not really going to be testing any of that code | 13:14 |
| sdague | I really think we should just fix the fiction in requirements | 13:14 |
| DaveJ__ | Hi Guys, the nova test are failing on a fresh install of Fedora 24. Should it be supported ,or should I be using another OS? | 13:14 |
| sdague | we known minimums aren't tested right | 13:14 |
| DaveJ__ | https://www.irccloud.com/pastebin/uMnhHtKA/ | 13:14 |
| sdague | bauzas: I also don't see this as critical | 13:14 |
| *** salv-orlando has quit IRC | 13:15 | |
| *** DaveJ__ is now known as DaveJ | 13:15 | |
| mdbooth | DaveJ: Python3: nobody uses that :) | 13:15 |
| * mdbooth gives it a go anyway | 13:15 | |
| *** sdake has quit IRC | 13:16 | |
| bauzas | sdague: I'm fine with either way, bumping or fixing in code | 13:16 |
| mdbooth | DaveJ: Ah... F24 has python 3.5. That might be the issue | 13:16 |
| bauzas | sdague: I just want to fix that for Newton | 13:16 |
| openstackgerrit | Kevin Benton proposed openstack/os-vif: Add MTU to Network model and use it in plugging https://review.openstack.org/370667 | 13:16 |
| bauzas | sdague: because a nova list command can be impacted | 13:16 |
| DaveJ | mdbooth: Thanks - I was a bit confused about that. It says py34 fails, but path from the stacktrace seems to indicate its python2.7 /home/dave/Workspace/Openstack/nova/.tox/pip-missing-reqs/lib/python2.7 | 13:16 |
| johnthetubaguy | sdague: I was expecting the packagers to use the list of deps from the gate rather than worry about the minimums I guess | 13:17 |
| kevinbenton | johnthetubaguy, danpb: i tested the nova change and os-vif change locally and it fixes the issue | 13:17 |
| *** sudipto_ has joined #openstack-nova | 13:17 | |
| *** sudipto has joined #openstack-nova | 13:17 | |
| bauzas | johnthetubaguy: sure, but we say we accept that version, which isn't true :) | 13:17 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue https://review.openstack.org/335884 | 13:17 |
| johnthetubaguy | kevinbenton: cool, do you want to check the hack? to see if thats an option | 13:17 |
| DaveJ | mdbooth: I manually installed python3.4 but 3.5 is still installed - do you think it's interfering ? | 13:17 |
| ihrachys | johnthetubaguy: https://review.openstack.org/#/c/370681/ that one right? | 13:17 |
| lyarwood | ^ mdbooth, mriedem ; now with mdbooth's change for your reviewing pleasure | 13:17 |
| kevinbenton | johnthetubaguy: sure | 13:18 |
| sdague | johnthetubaguy: yes, we tell them that | 13:18 |
| bauzas | johnthetubaguy: sdagueokay, I'm writing a change against the global requirements to bump netaddr to at least 0.7.13 | 13:18 |
| mriedem | lyarwood: thanks | 13:18 |
| mdbooth | DaveJ: Just attempting to run py35 tests | 13:18 |
| *** krtaylor has joined #openstack-nova | 13:18 | |
| *** rlrossit has joined #openstack-nova | 13:18 | |
| johnthetubaguy | ihrachys: yeah | 13:18 |
| ihrachys | johnthetubaguy: https://review.openstack.org/#/q/topic:bug/1623876 see the DNM patches | 13:19 |
| johnthetubaguy | kevinbenton: awesome, thanks, I kinda hope we don't want my match mind | 13:19 |
| sdague | bauzas: let's assume we move forward on the min bump as the approach, I also downgraded this to medium. I definitely don't think this is a blocking issue | 13:19 |
| bauzas | sdague: okay, fair enough, I'll see whether we can merge the requirements change asap before cutting newton | 13:20 |
| sdague | bauzas: sounds good | 13:20 |
| mriedem | bauzas: sdague: i dropped the rc tag from https://bugs.launchpad.net/nova/+bug/1623838 | 13:21 |
| openstack | Launchpad bug 1623838 in OpenStack Compute (nova) "Nova requires netaddr >= 0.7.12 which is not enough" [Medium,In progress] - Assigned to Sylvain Bauza (sylvain-bauza) | 13:21 |
| *** ltomasbo has joined #openstack-nova | 13:21 | |
| kevinbenton | johnthetubaguy: yeah, your patch looks like it works | 13:21 |
| kevinbenton | johnthetubaguy: i see veth pairs and bridge set to 1450 MTU | 13:21 |
| kevinbenton | johnthetubaguy: (config is 1500) | 13:21 |
| johnthetubaguy | kevinbenton: cool, I wasn't overly confident about the veth stuff | 13:21 |
| *** mdrabe has joined #openstack-nova | 13:22 | |
| mdbooth | DaveJ: So my python3 comment was only slightly facetious. There are a ton of tests we skip for python 3 precisely because the code they run doesn't work in python 3. I suspect Nova isn't actually functional under python 3, but perhaps somebody here can correct me. | 13:22 |
| *** yamahata has joined #openstack-nova | 13:23 | |
| mdbooth | DaveJ: If you run the 27 tests explicitly it should work fine. | 13:23 |
| johnthetubaguy | mdbooth: right, we don't expect Nova to work on py3 at this point | 13:23 |
| *** brian_price has quit IRC | 13:23 | |
| DaveJ | Ah ok - so safe to ignore then. So assuming jenkins doesn't run py3 tests on commit ? | 13:23 |
| kevinbenton | johnthetubaguy: note that this is going back through nova rootwrap and not privsep, not sure if you've already restricted filters | 13:23 |
| mdbooth | DaveJ: It does, but only the ones which aren't skipped. | 13:23 |
| johnthetubaguy | DaveJ: we have a test white list the tox target uses | 13:23 |
| bauzas | mriedem: well, if we drop that tag, it means that we need to be clear that netaddr needs 0.7.13 in order to work with Nova | 13:24 |
| mdbooth | What johnthetubaguy said | 13:24 |
| johnthetubaguy | actually its probably a black list, but you know what I mean | 13:24 |
| bauzas | mriedem: which is something we don't say atm | 13:24 |
| DaveJ | Yep got it. | 13:24 |
| DaveJ | Cheers | 13:24 |
| mriedem | kevinbenton: we haven't started dropping filters | 13:24 |
| *** gouthamr has joined #openstack-nova | 13:24 | |
| mriedem | bauzas: you mean nova needs netaddr>=0.7.13? | 13:24 |
| sdague | mriedem: yes | 13:24 |
| mriedem | but this isn't a new issue is it? | 13:25 |
| sdague | mriedem: no | 13:25 |
| mriedem | is this a regression? | 13:25 |
| sdague | oh, no, it is a new issue | 13:25 |
| mriedem | hm | 13:25 |
| mriedem | so we're doing the g-r bump? | 13:25 |
| sdague | we started using a function that's only 0.7.13 | 13:25 |
| bauzas | sdague: no, it's a regression | 13:25 |
| mriedem | netaddr is used in a lot of projects | 13:25 |
| sdague | mriedem: yes | 13:25 |
| mriedem | so there is going to be a big sync | 13:25 |
| bauzas | sdague: introduced by https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e2 | 13:25 |
| bauzas | which was merged by newton-2 | 13:25 |
| sdague | mriedem: sure, but, it only needs to land in nova before nova rc | 13:26 |
| bauzas | we were totally okay before that | 13:26 |
| mriedem | alternatively we revert ^ | 13:26 |
| sdague | if the other projects have a different value it's fine | 13:26 |
| mriedem | it doesn't fix a bug, it's a cleanup patch right? | 13:26 |
| bauzas | frickler: ? | 13:26 |
| sdague | mriedem: honestly, that's way riskier | 13:26 |
| johnthetubaguy | right, we have all been testing in the gate with the version that works | 13:26 |
| sdague | we *know* that req minimums aren't validated | 13:26 |
| bauzas | mriedem: it doesn't seem only cosmetic | 13:26 |
| mriedem | ok | 13:27 |
| * mriedem cowers under the backlash | 13:27 | |
| sdague | a bad req minimum should be considered a medium level issue to get fixed when we can | 13:27 |
| * bauzas on it | 13:27 | |
| mriedem | ok, so 'fixed when we can' doesn't sound like an rc blocker to me | 13:27 |
| sdague | we tell everyone to use upper-constraints as the tested versions (which is 0.7.18) | 13:27 |
| sdague | mriedem: agreed | 13:27 |
| bauzas | sdague: mriedem: we can mitigate that by our releasenotes | 13:27 |
| mriedem | bauzas: was just going to say that | 13:28 |
| mriedem | with a known issue reno, something we rarely use :) | 13:28 |
| sdague | so... I'm even torn on that | 13:28 |
| *** edand has joined #openstack-nova | 13:28 | |
| bauzas | not a known issue | 13:28 |
| sdague | because by saying "we know this min req is wrong" | 13:28 |
| bauzas | just a general comment | 13:28 |
| frickler | bauzas: I still think it is a bug fix, nova list --ip6 should be broken with netaddr 0.7.12 | 13:28 |
| sdague | we kind of imply all the other min req are validated | 13:28 |
| sdague | which, they aren't | 13:28 |
| frickler | bauzas: if that is the question | 13:28 |
| mriedem | alright well i have to drop off and get my kid to school | 13:28 |
| mriedem | back in ~30 minutes | 13:29 |
| *** kylek3h has joined #openstack-nova | 13:29 | |
| bauzas | \o | 13:29 |
| *** mriedem has quit IRC | 13:29 | |
| *** pwnall has joined #openstack-nova | 13:29 | |
| dhellmann | sdague , bauzas : I agree that a release note is a reasonable way to address this, since we're so close to the final cut-off | 13:30 |
| *** pwnall is now known as Guest59228 | 13:30 | |
| bauzas | sdague: any way to look at all openstack repositories so I can see if that specific API call is made elsewhere out of nova ? | 13:31 |
| sdague | dhellmann: ok, I get a little concerned that a release note about a min being bad does imply all the other ones are good. It feels like we need a more general disclaimer about the role of min req and upper constraints somewhere for the release | 13:31 |
| bauzas | AFAIR, there is some way to do that | 13:31 |
| sdague | codesearch.openstack.org | 13:31 |
| bauzas | cool, thanks | 13:31 |
| bauzas | sdague: that is a reasonable concern | 13:31 |
| dhellmann | sdague : sure, you'll want to be careful with wording, I just meant trying to fix the requirements list right now is going to cascade a bunch of other changes | 13:32 |
| bauzas | dhellmann: sdagueokay, I'm providing the change with a super verbose comment saying "please don't merge that for Newton, I'll handle it for Nova thru a relnote" | 13:32 |
| bauzas | but I need to see whether other projects could be impacted as well | 13:33 |
| bauzas | because AFAIR, frickler who wrote the code thinks he saw that snippet in Neutron | 13:33 |
| bauzas | no brainer http://codesearch.openstack.org/?q=.modified_eui64%28%29&i=nope&files=&repos= | 13:34 |
| sdague | dhellmann: it will propose some min bumps, but it doesn't change any testing | 13:34 |
| *** tblakes has joined #openstack-nova | 13:34 | |
| sdague | and if only nova merges that, it's fine | 13:34 |
| bauzas | so, there is no other project impacted, let's move on and fill a relnote saying how bad it would be to deploy based on minimums | 13:34 |
| dims | rpodolyaka is a beast! :) nice work on that opportunistic tests debugging! | 13:34 |
| sdague | the rest can merge in on stable/newton | 13:35 |
| *** BjoernT has joined #openstack-nova | 13:36 | |
| *** eharney has joined #openstack-nova | 13:36 | |
| *** Guest59228 has quit IRC | 13:37 | |
| bauzas | sdague: dhellmann: FWIW https://review.openstack.org/370833 | 13:39 |
| bauzas | and I'm writing a reno file about how wrong it could be to run nova with minimums given we don't test it | 13:40 |
| *** ducttape_ has joined #openstack-nova | 13:42 | |
| sdague | bauzas: ok, well I think we should just merge the req bump, I tried to explain why with details in the review | 13:43 |
| bauzas | sdague: good point | 13:43 |
| *** akshai has quit IRC | 13:44 | |
| *** cleong has joined #openstack-nova | 13:44 | |
| *** TravT has quit IRC | 13:46 | |
| *** TravT has joined #openstack-nova | 13:47 | |
| *** ducttape_ has quit IRC | 13:47 | |
| *** TravT has quit IRC | 13:47 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: WIP config options: improve libvirt https://review.openstack.org/370843 | 13:47 |
| *** akshai has joined #openstack-nova | 13:48 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: WIP config options: improve libvirt https://review.openstack.org/370843 | 13:48 |
| *** EinstCrazy has joined #openstack-nova | 13:48 | |
| *** eharney_ has joined #openstack-nova | 13:49 | |
| *** bhagyashri has joined #openstack-nova | 13:49 | |
| *** eharney has quit IRC | 13:49 | |
| *** eharney_ is now known as eharney | 13:49 | |
| *** akshai has quit IRC | 13:50 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Update minimum requirement for netaddr https://review.openstack.org/370846 | 13:50 |
| sdague | bauzas: that is the depends on change that would address it if we can get aggreement on merging the g-r change | 13:50 |
| *** gongysh has joined #openstack-nova | 13:51 | |
| *** tonytan4ever has joined #openstack-nova | 13:51 | |
| bauzas | sdague: cool thank | 13:51 |
| *** baoli has joined #openstack-nova | 13:54 | |
| *** akshai has joined #openstack-nova | 13:55 | |
| *** gongysh has quit IRC | 13:55 | |
| *** tonytan_brb has joined #openstack-nova | 13:56 | |
| *** baoli__ has joined #openstack-nova | 13:56 | |
| *** akshai has quit IRC | 13:57 | |
| *** baoli_ has quit IRC | 13:57 | |
| rpodolyaka | dims: heh, thanks!) | 13:57 |
| *** akshai has joined #openstack-nova | 13:58 | |
| *** tonytan4ever has quit IRC | 13:58 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: WIP: try a bigger TIMEOUT_SCALING_FACTOR value for migration tests https://review.openstack.org/370805 | 13:58 |
| *** baoli has quit IRC | 13:59 | |
| *** akshai has quit IRC | 14:01 | |
| *** paul-carlton has joined #openstack-nova | 14:03 | |
| *** mriedem has joined #openstack-nova | 14:03 | |
| *** akshai has joined #openstack-nova | 14:03 | |
| mriedem | are all things fixed now? | 14:03 |
| sdague | rpodolyaka: can we just remove the WIP there | 14:04 |
| rpodolyaka | sdague: sure! | 14:05 |
| bauzas | mriedem: https://review.openstack.org/370846 and https://review.openstack.org/370833 for the netaddr problem | 14:05 |
| sdague | I'd rather just merge that change and see what the statistics look like once it's in | 14:05 |
| *** ducttape_ has joined #openstack-nova | 14:05 | |
| bauzas | mriedem: for os-vif, I haven't heard further progress | 14:05 |
| sdague | rpodolyaka: thank you for diving into that one | 14:05 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova: Set a bigger TIMEOUT_SCALING_FACTOR value for migration tests https://review.openstack.org/370805 | 14:06 |
| rpodolyaka | sdague: no problem! | 14:06 |
| mriedem | bauzas: sdague: is there a reqs ffe for the netaddr change? | 14:06 |
| sdague | mriedem: I am not fully sure the process there | 14:07 |
| johnthetubaguy | bauzas: mriedem: for os-vif, there is my patch uploaded, it was tested, but its not merged yet (may have some unit test gremlins still) | 14:07 |
| *** roeyc has quit IRC | 14:07 | |
| mriedem | sdague: bauzas: ok i'll post the ffe in the ML | 14:07 |
| sdague | it's a little weird, because it's litterally a zero risk change from our environment | 14:07 |
| *** gouthamr has quit IRC | 14:07 | |
| sdague | the effect in upstream ci is basically being a code comment change | 14:07 |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 14:08 |
| *** ducttape_ has quit IRC | 14:09 | |
| *** claudiub has joined #openstack-nova | 14:09 | |
| *** code-R has joined #openstack-nova | 14:09 | |
| mriedem | i don't make the rules | 14:09 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Add reno for live migraton with cd-rom config drive https://review.openstack.org/370864 | 14:10 |
| sdague | mriedem: that's fine | 14:11 |
| *** code-R__ has joined #openstack-nova | 14:12 | |
| *** omnipresent has joined #openstack-nova | 14:12 | |
| *** zz_dimtruck is now known as dimtruck | 14:12 | |
| *** mlavalle has joined #openstack-nova | 14:13 | |
| *** code-R_ has quit IRC | 14:13 | |
| *** burgerk has quit IRC | 14:14 | |
| sdague | mriedem: I put the closes-bug on the nova change | 14:14 |
| mriedem | i know | 14:15 |
| *** code-R has quit IRC | 14:15 | |
| mriedem | i posted it for reqs reviewers | 14:15 |
| mriedem | since they'll ask | 14:15 |
| sdague | because the bug isn't actually closed until that merges | 14:15 |
| sdague | ah, ok | 14:15 |
| mriedem | i know | 14:15 |
| mriedem | i'm pretty sure doug had an email reminder about doing this | 14:15 |
| sdague | sorry, I was reading your comment incorrectly | 14:15 |
| mriedem | so they don't have to ask | 14:15 |
| mriedem | johnthetubaguy: i think water will kill unit test gremlins | 14:17 |
| *** ducttape_ has joined #openstack-nova | 14:18 | |
| *** takedakn has joined #openstack-nova | 14:18 | |
| *** abalutoiu has joined #openstack-nova | 14:19 | |
| sdague | are there any final placement bits needed? | 14:19 |
| sdague | or are we finally at the cut line there | 14:19 |
| *** rlrossit has quit IRC | 14:19 | |
| mriedem | there is a bug fix | 14:19 |
| mriedem | just watching https://review.openstack.org/#/c/369147/ through the gate | 14:20 |
| markus_z | mriedem: There are a few others which are stuck: https://review.openstack.org/#/q/project:openstack/nova++label:workflow%253D%252B1+status:pending | 14:21 |
| *** moshele has quit IRC | 14:21 | |
| markus_z | by "stuck" I mean the 3 oldest in that list | 14:22 |
| *** ltomasbo has quit IRC | 14:22 | |
| dansmith | mriedem: right, when that lands I think we're good | 14:22 |
| *** ducttape_ has quit IRC | 14:23 | |
| *** marst has quit IRC | 14:26 | |
| bauzas | dansmith: https://review.openstack.org/#/c/369147/10 is the top patch we want to land before tagging RC1 for the placement stuff ? | 14:26 |
| *** annegentle has joined #openstack-nova | 14:26 | |
| bauzas | https://etherpad.openstack.org/p/placement-next seems to say yes | 14:26 |
| dansmith | bauzas: yep, that's what mriedem just said.. once that one is on (and the one it depends on) I'm good from the placement side | 14:26 |
| dansmith | s/on/in/ | 14:26 |
| *** jheroux has joined #openstack-nova | 14:27 | |
| *** danpb has quit IRC | 14:27 | |
| bauzas | kewl | 14:27 |
| *** danpb has joined #openstack-nova | 14:27 | |
| *** null0 has joined #openstack-nova | 14:28 | |
| *** yassine has joined #openstack-nova | 14:28 | |
| *** yassine is now known as Guest31676 | 14:28 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Improve consistency in serial_console https://review.openstack.org/348902 | 14:29 |
| *** pwnall has joined #openstack-nova | 14:29 | |
| *** pwnall is now known as Guest88340 | 14:29 | |
| *** paul-carlton has quit IRC | 14:29 | |
| *** Guest67165 has quit IRC | 14:29 | |
| *** null0 has quit IRC | 14:30 | |
| *** sree has joined #openstack-nova | 14:31 | |
| *** adisky has quit IRC | 14:32 | |
| *** edand has quit IRC | 14:33 | |
| *** sree has quit IRC | 14:35 | |
| *** marst has joined #openstack-nova | 14:36 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add is_cell0 helper method https://review.openstack.org/370894 | 14:36 |
| *** cdent has quit IRC | 14:37 | |
| mriedem | sdague: fyi, the netaddr bump is being discussed in #openstack-requirements | 14:40 |
| sdague | ok, joining, it got lost in my auto join list | 14:40 |
| mriedem | http://eavesdrop.openstack.org/irclogs/%23openstack-requirements/%23openstack-requirements.2016-09-15.log.html#t2016-09-15T14:16:34 | 14:40 |
| bauzas | mriedem: FWIW joining as well | 14:41 |
| *** pradk has quit IRC | 14:42 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova: Separate ovs command to create vif port https://review.openstack.org/318128 | 14:43 |
| *** rama_y has joined #openstack-nova | 14:43 | |
| *** rama_y has quit IRC | 14:44 | |
| *** rama_y has joined #openstack-nova | 14:44 | |
| *** EinstCrazy has quit IRC | 14:44 | |
| *** pradk has joined #openstack-nova | 14:45 | |
| *** jed56 has quit IRC | 14:45 | |
| *** ducttape_ has joined #openstack-nova | 14:47 | |
| zigo | bauzas: I still get a single unit test failure when building Nova b3: http://logs.openstack.org/79/369879/2/check/gate-deb-nova-dpkg-debian-jessie/de73da2/console.html#_2016-09-15_13_38_28_450354 | 14:48 |
| zigo | Any idea why? | 14:48 |
| zigo | (note: I prefer to discuss before badly filing bugs...) | 14:48 |
| *** liverpooler has joined #openstack-nova | 14:49 | |
| *** akshai has quit IRC | 14:49 | |
| *** burgerk has joined #openstack-nova | 14:51 | |
| *** akshai has joined #openstack-nova | 14:51 | |
| *** akshai has quit IRC | 14:51 | |
| *** TravT has joined #openstack-nova | 14:52 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Override MTU for os_vif attachments https://review.openstack.org/370681 | 14:53 |
| *** akshai has joined #openstack-nova | 14:53 | |
| bauzas | zigo: first time I see that error :/ | 14:54 |
| *** nkrinner is now known as nkrinner_afk | 14:54 | |
| *** baoli has joined #openstack-nova | 14:54 | |
| *** baoli__ has quit IRC | 14:55 | |
| *** ManishD has joined #openstack-nova | 14:55 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/os-vif: os-vif pollute test output https://review.openstack.org/370908 | 14:56 |
| *** rrecio_ has joined #openstack-nova | 14:56 | |
| *** tonytan_brb is now known as tonytan4ever | 14:57 | |
| ManishD | Dansmith: Hi, about the encryption volume with reboot, the case is applicable when VM is shutdown or halt. So we thought it is better to detach when reboot/shutdown, especially for shutdown | 14:58 |
| ManishD | Dansmith: about this: https://review.openstack.org/#/c/357131 | 14:58 |
| dansmith | ManishD: I have to run off in a few minutes, but.. I could maybe get behind detach for shutdown, although I'd prefer not | 14:59 |
| dansmith | ManishD: but detach for reboot is just asking for more trouble, IMHO | 14:59 |
| dansmith | and not for any good reason other than that it seems like minimal change to achieve the goal | 14:59 |
| *** rrecio has quit IRC | 14:59 | |
| *** mjura has quit IRC | 14:59 | |
| ManishD | dansmith: OK, Thanks, Please send your feedback whenever you get some time and I shall revert. | 15:01 |
| dansmith | ManishD: thought I already did, but okay | 15:03 |
| dansmith | biab | 15:03 |
| *** unicell1 has quit IRC | 15:03 | |
| *** diogogmt has joined #openstack-nova | 15:03 | |
| *** jojden has quit IRC | 15:04 | |
| ManishD | dansmith: yes got it but want to understand what the other way around to achive that. | 15:05 |
| *** tri2sing has joined #openstack-nova | 15:05 | |
| ihrachys | johnthetubaguy: you are bumping my Depends-On from gate :)) | 15:06 |
| *** yamahata has quit IRC | 15:06 | |
| ihrachys | johnthetubaguy: but the one that was expected to show the tempest test will fail with bad network_device_mtu actually failed: http://logs.openstack.org/10/370810/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/dec53e0/logs/testr_results.html.gz | 15:06 |
| *** siva_krishnan has joined #openstack-nova | 15:06 | |
| *** jaypipes has joined #openstack-nova | 15:06 | |
| ihrachys | johnthetubaguy: so now we just need to show that it passes with your patch in the queue | 15:06 |
| ihrachys | johnthetubaguy: I rechecked | 15:06 |
| *** unicell has joined #openstack-nova | 15:07 | |
| johnthetubaguy | ah, sweet | 15:07 |
| *** slaweq_ has quit IRC | 15:07 | |
| *** baoli_ has joined #openstack-nova | 15:07 | |
| *** slaweq_ has joined #openstack-nova | 15:08 | |
| johnthetubaguy | ihrachys: you going to depend-on my change as well, or something like that? | 15:08 |
| *** mtanino has joined #openstack-nova | 15:08 | |
| ihrachys | johnthetubaguy: https://review.openstack.org/#/c/370813/ depends on your fix | 15:08 |
| *** tblakes has quit IRC | 15:08 | |
| ihrachys | johnthetubaguy: it just gets bumped off gate every time you respin nova fix | 15:08 |
| ihrachys | johnthetubaguy: so I struggle to get a result :) | 15:08 |
| johnthetubaguy | ah, sorry, I get you now | 15:09 |
| johnthetubaguy | so, I have the unit tests passing problem, I believe | 15:09 |
| johnthetubaguy | I keep getting pulled away form my keyboard, but less of those problems right now | 15:09 |
| *** baoli has quit IRC | 15:10 | |
| mriedem | i'll throw https://review.openstack.org/#/c/370813/ into our etherpad to make sure we see it passing before approving the nova change | 15:10 |
| *** takedakn has quit IRC | 15:10 | |
| *** takedakn has joined #openstack-nova | 15:11 | |
| *** zul has joined #openstack-nova | 15:12 | |
| *** slaweq_ has quit IRC | 15:12 | |
| *** amrith has joined #openstack-nova | 15:12 | |
| johnthetubaguy | mriedem: thanks | 15:12 |
| mriedem | ihrachys: is there a specific job we need to check in https://review.openstack.org/#/c/370813/ ? | 15:12 |
| openstackgerrit | ianeta hutchinson proposed openstack/nova: Updates URL and removes trailing characters https://review.openstack.org/368185 | 15:13 |
| mriedem | ihrachys: or would it just be any neutron job? | 15:13 |
| *** hemnafk is now known as hemna | 15:13 | |
| siva_krishnan | artom: can you review my patch whenever you find time https://review.openstack.org/#/c/369176/ | 15:14 |
| johnthetubaguy | mriedem: I think we want this red to go green: http://logs.openstack.org/10/370810/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/dec53e0/logs/testr_results.html.gz | 15:14 |
| *** amrith has left #openstack-nova | 15:15 | |
| *** gouthamr has joined #openstack-nova | 15:15 | |
| mriedem | ah yeah | 15:15 |
| *** takedakn has quit IRC | 15:16 | |
| ihrachys | mriedem: the failed patch https://review.openstack.org/#/c/370810/ | 15:16 |
| ihrachys | mriedem: that one without the nova fix | 15:16 |
| ihrachys | so -full- | 15:16 |
| *** rhagarty has joined #openstack-nova | 15:17 | |
| *** gouthamr has quit IRC | 15:17 | |
| *** mtanin___ has joined #openstack-nova | 15:17 | |
| *** gouthamr has joined #openstack-nova | 15:17 | |
| mriedem | ok cool | 15:17 |
| mriedem | ihrachys: if you need to drop i can babysit those and recheck throughout my day | 15:18 |
| *** kfarr has joined #openstack-nova | 15:18 | |
| artom | siva_krishnan, heya, yeah, it's on my list :) | 15:18 |
| *** jaosorior has quit IRC | 15:18 | |
| ihrachys | mriedem: it's nice to have some eyes, but I will be here the next 3h or so | 15:18 |
| *** mtanino has quit IRC | 15:18 | |
| *** pcaruana has quit IRC | 15:19 | |
| artom | siva_krishnan, though if you've made sure the fix works in a test environment it's an easy +1 (for the code at least, haven't looked at the test) since it's a 1 line change :) | 15:19 |
| *** salv-orlando has joined #openstack-nova | 15:19 | |
| *** andreas_s has quit IRC | 15:19 | |
| *** diogogmt has quit IRC | 15:20 | |
| *** salv-orlando has quit IRC | 15:22 | |
| siva_krishnan | artom it passed all the tox tests. I was able to update the DB if the device was tagged. Waiting for comments! | 15:22 |
| *** slaweq_ has joined #openstack-nova | 15:22 | |
| *** zul has quit IRC | 15:22 | |
| artom | siva_krishnan, right, but have you made sure booting an instance with your fix with a tagged ephemeral disk results in a tag in the database? | 15:24 |
| *** ssurana has joined #openstack-nova | 15:26 | |
| *** ManishD has quit IRC | 15:26 | |
| *** Matti_ has joined #openstack-nova | 15:27 | |
| *** moshele has joined #openstack-nova | 15:27 | |
| *** Matti_ has quit IRC | 15:29 | |
| *** Matti_ has joined #openstack-nova | 15:29 | |
| *** sree has joined #openstack-nova | 15:29 | |
| siva_krishnan | artom yeah it works | 15:30 |
| *** takedakn has joined #openstack-nova | 15:30 | |
| artom | siva_krishnan, then you're good :) I'll look into the test in more detail and leave a comment soon-ish | 15:30 |
| siva_krishnan | artom thanks! | 15:31 |
| *** mdrabe has quit IRC | 15:32 | |
| *** gjayavelu has joined #openstack-nova | 15:32 | |
| *** sree has quit IRC | 15:34 | |
| sdague | mriedem: is there a known ceph / volume / snapshot error? | 15:34 |
| *** markus_z has quit IRC | 15:34 | |
| *** woodster_ has joined #openstack-nova | 15:36 | |
| *** mtanino has joined #openstack-nova | 15:36 | |
| sdague | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Snapshot%5C%22%20AND%20message%3A%5C%22failed%20to%20build%20and%20is%20in%20ERROR%20status%5C%22 | 15:36 |
| *** ducttape_ has quit IRC | 15:37 | |
| *** mtanin___ has quit IRC | 15:37 | |
| *** mdrabe has joined #openstack-nova | 15:38 | |
| *** tblakes has joined #openstack-nova | 15:39 | |
| mriedem | is that a backup test? | 15:39 |
| sdague | it's the bfv scenario pretty often | 15:40 |
| sdague | but that does a lot of hoops | 15:40 |
| bauzas | danpb: others, that looks like a feature request, right ? https://bugs.launchpad.net/nova/+bug/1623871 | 15:41 |
| openstack | Launchpad bug 1623871 in OpenStack Compute (nova) "Nova hugepage support does not include aarch64" [Undecided,In progress] - Assigned to Veena (mveenasl) | 15:41 |
| sdague | that scenario includes doing a volume snapshot, then booting from that | 15:41 |
| sdague | bauzas: if it's litterally just adding an arch name to a string list, it feels like you could call it a bug | 15:42 |
| sdague | if it's more complicated than that, it's probably a feature | 15:42 |
| bauzas | sdague: well, I'm concerned by the sentence "Both for NUMA and Hugepage support, AARCH64 needs to be added. " | 15:43 |
| bauzas | that looks more than a conditional | 15:43 |
| sdague | well, iirc the ppc64 numa add was litterally just extending the list of arches | 15:43 |
| bauzas | vladikr: around ? | 15:43 |
| vladikr | bauzas, sure | 15:43 |
| mriedem | sdague: my guess would be http://status.openstack.org//elastic-recheck/index.html#1475333 | 15:43 |
| bauzas | vladikr: was triaging and saw https://bugs.launchpad.net/nova/+bug/1623871 | 15:44 |
| openstack | Launchpad bug 1623871 in OpenStack Compute (nova) "Nova hugepage support does not include aarch64" [Undecided,In progress] - Assigned to Veena (mveenasl) | 15:44 |
| sdague | bauzas: I'd probably put it into the camp that it's not a high enough priority bug to land at this point in the cycle anyway | 15:44 |
| bauzas | vladikr: does it sound just a miss that we made when writing the features, or does that require more than that? | 15:44 |
| bauzas | sdague: I was just about to say that :p | 15:44 |
| danpb | bauzas: yep, its absolutely a feature reqeust | 15:44 |
| bauzas | like, it shouldn't take more than 5 mins of my time :p | 15:44 |
| vladikr | yea | 15:44 |
| bauzas | danpb: okay, I'll point out the right process scenario, thnaks | 15:45 |
| danpb | sdague: bauzas at the very least it will need someone to figure out min required versions of libvirt & qemu for huge pages | 15:45 |
| bauzas | danpb: oooh | 15:45 |
| danpb | sdague: pp64 actually caused bugs because it turned out to really need more than just extending the arch list | 15:45 |
| sdague | danpb: ah, ok | 15:45 |
| danpb | because PPC can have sparse numa topologies and we didn;'t handle that right | 15:45 |
| danpb | eg, node 0 and node 25, with no other nodes in between | 15:46 |
| *** gjayavelu has quit IRC | 15:46 | |
| danpb | i dunno if aarch64 will have similar surprises | 15:46 |
| *** cdent has joined #openstack-nova | 15:46 | |
| *** gjayavelu has joined #openstack-nova | 15:47 | |
| *** salv-orlando has joined #openstack-nova | 15:49 | |
| *** ducttape_ has joined #openstack-nova | 15:49 | |
| *** priteau has joined #openstack-nova | 15:49 | |
| mriedem | danpb: are you aware of https://github.com/openstack/nova/commit/abc24acfa1982a0ffccbe08a006ac7c7a9f4ecda ? | 15:49 |
| mriedem | that landed this week | 15:49 |
| danpb | mriedem: no, hadn't see it | 15:51 |
| *** jgriffith has quit IRC | 15:52 | |
| danpb | i would have expected the test suite to be expanded more than that - for example ppc supports *many* huge pages sizes, where as x86 only supports 1 or 2 sizes, so would have been nice to see a ppc specific example data added | 15:53 |
| *** paul-carlton has joined #openstack-nova | 15:53 | |
| *** moshele has quit IRC | 15:55 | |
| *** brian_price has joined #openstack-nova | 15:55 | |
| mriedem | lyarwood: question in https://review.openstack.org/#/c/335884/ | 15:57 |
| mriedem | lyarwood: that still looks to me like a latent bug fix, not a regression fix | 15:58 |
| *** gjayavelu has quit IRC | 15:58 | |
| *** jkraj has quit IRC | 15:58 | |
| lyarwood | mriedem: that's correct, I don't think I've called it out as a regression anywhere | 15:59 |
| mriedem | doh | 15:59 |
| *** ltomasbo has joined #openstack-nova | 16:00 | |
| mriedem | the bug was marked newton-rc-potential | 16:00 |
| *** thumpba has joined #openstack-nova | 16:00 | |
| lyarwood | ah right, and we only accept regressions at this point? | 16:00 |
| mriedem | and b/c of the ref to artom's change i thought it was supposed to be a regression fix | 16:00 |
| *** ltomasbo has quit IRC | 16:00 | |
| mriedem | well, we just scrutinize bugs more | 16:00 |
| *** ltomasbo has joined #openstack-nova | 16:00 | |
| *** cdent has quit IRC | 16:00 | |
| *** takedakn has quit IRC | 16:03 | |
| *** hshiina has quit IRC | 16:05 | |
| *** rlrossit has joined #openstack-nova | 16:07 | |
| *** rlrossit has quit IRC | 16:08 | |
| *** mdrabe has quit IRC | 16:08 | |
| *** mdrabe has joined #openstack-nova | 16:08 | |
| *** Apoorva has joined #openstack-nova | 16:09 | |
| *** jgriffith has joined #openstack-nova | 16:11 | |
| *** yamahata has joined #openstack-nova | 16:15 | |
| jaypipes | morning all | 16:17 |
| mriedem | jaypipes: o/ | 16:18 |
| mriedem | ihrachys: passed http://logs.openstack.org/13/370813/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ddf5f49/ | 16:19 |
| johnthetubaguy | mriedem: aunnam is looking at https://bugs.launchpad.net/nova/+bug/1463856 which will need us to make those cinder keystone auth config like we have for neutron. Just thought I would mention it in case you were looking at that sometime soon | 16:19 |
| openstack | Launchpad bug 1463856 in OpenStack Compute (nova) "Cinder volume isn't available after instance soft-deleted timer expired while volume is still attached" [Undecided,In progress] - Assigned to Anusha Unnam (anusha-unnam) | 16:19 |
| *** matrohon has quit IRC | 16:19 | |
| mriedem | johnthetubaguy: that bug is over a year old | 16:20 |
| *** lucasagomes is now known as lucas-dinner | 16:24 | |
| mriedem | bauzas: do it https://review.openstack.org/#/c/370846/1 | 16:24 |
| mriedem | the release team is moving forward with the bump | 16:24 |
| mriedem | the depends-on blocks it anyway | 16:24 |
| *** raunak has joined #openstack-nova | 16:24 | |
| *** ssurana has quit IRC | 16:25 | |
| *** nic has joined #openstack-nova | 16:25 | |
| *** rcernin has quit IRC | 16:26 | |
| *** bkopilov has quit IRC | 16:30 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 16:31 |
| *** oomichi_ has joined #openstack-nova | 16:32 | |
| *** TravT has quit IRC | 16:32 | |
| *** oomichi_ has quit IRC | 16:32 | |
| *** TravT has joined #openstack-nova | 16:32 | |
| *** ltomasbo has quit IRC | 16:32 | |
| *** oomichi_ has joined #openstack-nova | 16:32 | |
| *** ltomasbo has joined #openstack-nova | 16:32 | |
| *** baoli_ has quit IRC | 16:34 | |
| *** cdent has joined #openstack-nova | 16:35 | |
| *** moshele has joined #openstack-nova | 16:36 | |
| *** paul-carlton has quit IRC | 16:38 | |
| *** gouthamr has quit IRC | 16:40 | |
| *** tonytan4ever has quit IRC | 16:42 | |
| *** gouthamr has joined #openstack-nova | 16:42 | |
| *** priteau has quit IRC | 16:44 | |
| *** rlrossit has joined #openstack-nova | 16:46 | |
| *** thumpba has quit IRC | 16:48 | |
| *** thumpba has joined #openstack-nova | 16:49 | |
| *** derekh has quit IRC | 16:49 | |
| *** browne has joined #openstack-nova | 16:49 | |
| *** rlrossit has quit IRC | 16:49 | |
| *** lpetrut has quit IRC | 16:50 | |
| *** annegentle has quit IRC | 16:52 | |
| *** rlrossit has joined #openstack-nova | 16:53 | |
| *** thumpba has quit IRC | 16:54 | |
| *** aunnam_ has joined #openstack-nova | 16:54 | |
| *** sudipto has quit IRC | 16:55 | |
| *** sudipto_ has quit IRC | 16:55 | |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 16:55 |
| *** cdent has quit IRC | 16:55 | |
| *** TravT has quit IRC | 16:57 | |
| *** yamahata has quit IRC | 16:58 | |
| *** yamahata has joined #openstack-nova | 16:58 | |
| *** sree has joined #openstack-nova | 16:58 | |
| *** fragatina has joined #openstack-nova | 16:59 | |
| *** paul-carlton has joined #openstack-nova | 17:00 | |
| *** _ducttape_ has joined #openstack-nova | 17:00 | |
| *** fragatina has quit IRC | 17:00 | |
| *** moshele has quit IRC | 17:00 | |
| *** code-R__ has quit IRC | 17:01 | |
| *** fragatina has joined #openstack-nova | 17:01 | |
| *** TravT has joined #openstack-nova | 17:01 | |
| *** rlrossit has quit IRC | 17:01 | |
| *** unicell has quit IRC | 17:02 | |
| *** sree has quit IRC | 17:03 | |
| *** rfolco has quit IRC | 17:03 | |
| *** ducttape_ has quit IRC | 17:03 | |
| *** itlinux has joined #openstack-nova | 17:03 | |
| *** rfolco has joined #openstack-nova | 17:04 | |
| *** Fdaisuke has quit IRC | 17:04 | |
| johnthetubaguy | mriedem: sorry, more context, no where near release ciritical, just a heads up around the config difference we were bumping into a few weeks back | 17:04 |
| *** sahid has quit IRC | 17:04 | |
| *** efoley_ has quit IRC | 17:13 | |
| *** thumpba has joined #openstack-nova | 17:17 | |
| * dansmith stares at the gate helplessly | 17:17 | |
| *** amrith has joined #openstack-nova | 17:18 | |
| *** gszasz has quit IRC | 17:18 | |
| siva_krishnan | johnthetubaguy: this is regarding https://bugs.launchpad.net/nova/+bug/1456899. As you suggested, I was about invalidate this bug and the one related to this(floating ip). Just noticed that openstack client library on 'openstack limits show' gets info from compute on security groups and while doing 'openstack security group create' it creates it through neturon API. Isn't this a bug ? | 17:18 |
| openstack | Launchpad bug 1456899 in OpenStack Compute (nova) "nova absolute-limits Security groups count incorrect when using Neutron" [Low,Confirmed] - Assigned to Sivasathurappan Radhakrishnan (siva-radhakrishnan) | 17:18 |
| *** Matti_ has quit IRC | 17:18 | |
| *** tblakes_ has joined #openstack-nova | 17:22 | |
| *** hemanthm is now known as hemanthm|lunch | 17:22 | |
| *** _ducttape_ has quit IRC | 17:22 | |
| *** tblakes has quit IRC | 17:23 | |
| *** tblakes_ is now known as tblakes | 17:23 | |
| *** ducttape_ has joined #openstack-nova | 17:23 | |
| *** mbirru has joined #openstack-nova | 17:25 | |
| *** abalutoiu has quit IRC | 17:29 | |
| *** danpb has quit IRC | 17:31 | |
| *** mvk has quit IRC | 17:32 | |
| *** ducttape_ has quit IRC | 17:34 | |
| *** rlrossit has joined #openstack-nova | 17:34 | |
| *** hemna_ has joined #openstack-nova | 17:35 | |
| mriedem | johnthetubaguy: ack | 17:36 |
| *** lpetrut has joined #openstack-nova | 17:36 | |
| *** fragatina has quit IRC | 17:36 | |
| *** tonytan4ever has joined #openstack-nova | 17:37 | |
| mriedem | dansmith: we can look at this https://review.openstack.org/#/c/370681/ | 17:38 |
| *** ssurana has joined #openstack-nova | 17:38 | |
| *** gjayavelu has joined #openstack-nova | 17:39 | |
| *** smatzek has joined #openstack-nova | 17:39 | |
| *** GB21 has quit IRC | 17:41 | |
| *** lpetrut has quit IRC | 17:41 | |
| *** hemanthm|lunch is now known as hemanthm | 17:42 | |
| *** itlinux has quit IRC | 17:43 | |
| *** ihrachys has quit IRC | 17:43 | |
| *** ltomasbo has quit IRC | 17:45 | |
| *** omnipresent has quit IRC | 17:45 | |
| *** andrewbogott has quit IRC | 17:47 | |
| *** andrewbogott has joined #openstack-nova | 17:47 | |
| *** hemna_ has quit IRC | 17:49 | |
| *** unicell has joined #openstack-nova | 17:52 | |
| *** itlinux has joined #openstack-nova | 17:53 | |
| *** GB21 has joined #openstack-nova | 17:54 | |
| *** moshele has joined #openstack-nova | 17:54 | |
| *** aunnam_ has quit IRC | 17:55 | |
| *** ducttape_ has joined #openstack-nova | 17:55 | |
| *** sudipto has joined #openstack-nova | 18:00 | |
| *** sudipto_ has joined #openstack-nova | 18:00 | |
| *** tblakes has quit IRC | 18:04 | |
| *** fragatina has joined #openstack-nova | 18:04 | |
| *** TravT has quit IRC | 18:06 | |
| *** sudipto has quit IRC | 18:07 | |
| *** sudipto_ has quit IRC | 18:07 | |
| *** mvk has joined #openstack-nova | 18:12 | |
| *** tblakes has joined #openstack-nova | 18:12 | |
| mriedem | jaypipes: need you to look at this https://review.openstack.org/#/c/370681/ | 18:13 |
| mriedem | wish danpb would have | 18:13 |
| *** yamahata has quit IRC | 18:13 | |
| mriedem | johnthetubaguy: i've posted quetsions in ^ but i'm in over my head there | 18:13 |
| *** liverpooler has quit IRC | 18:14 | |
| *** lpetrut has joined #openstack-nova | 18:15 | |
| *** psachin has quit IRC | 18:20 | |
| *** rodrigods has quit IRC | 18:22 | |
| *** rodrigods has joined #openstack-nova | 18:22 | |
| *** pcaruana has joined #openstack-nova | 18:22 | |
| *** annegentle has joined #openstack-nova | 18:26 | |
| *** sdake_ is now known as sdake | 18:26 | |
| *** paul-carlton has quit IRC | 18:31 | |
| *** kfarr has quit IRC | 18:31 | |
| *** amrith has left #openstack-nova | 18:32 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova: Send vif-plugged to dest host if evacuating https://review.openstack.org/371048 | 18:36 |
| *** moshele has quit IRC | 18:38 | |
| *** tongli has joined #openstack-nova | 18:39 | |
| jaypipes | mriedem: on it. | 18:40 |
| mriedem | gah we need to update this to test linuxbridge on libvirt vif changes | 18:41 |
| mriedem | https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L2055 | 18:41 |
| *** GB21 has quit IRC | 18:41 | |
| *** fragatina has quit IRC | 18:43 | |
| *** fragatina has joined #openstack-nova | 18:43 | |
| *** yamahata has joined #openstack-nova | 18:45 | |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 18:46 |
| *** baoli has joined #openstack-nova | 18:52 | |
| *** code-R has joined #openstack-nova | 18:55 | |
| *** tgtanya has joined #openstack-nova | 18:56 | |
| *** code-R_ has joined #openstack-nova | 18:59 | |
| *** code-R has quit IRC | 19:02 | |
| *** aunnam_ has joined #openstack-nova | 19:02 | |
| *** ijw has joined #openstack-nova | 19:03 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Remove deprecated compute_available_monitors option https://review.openstack.org/371062 | 19:03 |
| *** kfarr has joined #openstack-nova | 19:04 | |
| *** tgtanya_ has joined #openstack-nova | 19:04 | |
| *** rhagarty has quit IRC | 19:05 | |
| *** tgtanya has quit IRC | 19:06 | |
| *** krtaylor has quit IRC | 19:07 | |
| *** matrohon has joined #openstack-nova | 19:07 | |
| *** salv-orlando has quit IRC | 19:07 | |
| *** akshai has quit IRC | 19:09 | |
| dansmith | oh hell yes | 19:09 |
| dansmith | mriedem: bauzas: those patches merged | 19:09 |
| mriedem | cool | 19:10 |
| mriedem | dansmith: can you check my regex here? https://review.openstack.org/371065 | 19:10 |
| *** akshai has joined #openstack-nova | 19:11 | |
| dansmith | mriedem: um, that doesn't look right to me | 19:11 |
| dansmith | or, maybe (?..) makes the whole group optional? | 19:12 |
| mriedem | i was just testing with samples on regexpal.com | 19:12 |
| mriedem | it should mean, skip that job if the file is not in those | 19:12 |
| mriedem | so nova/compute/manager.py would not be tested | 19:12 |
| mriedem | nova/network/os_vif_util.py or nova/virt/libvirt/vif.py would | 19:13 |
| dansmith | I dunno what (? means.. normally I would do like (..){0,1} or something | 19:14 |
| dansmith | to make a group optional | 19:14 |
| dansmith | oh, lookahead | 19:15 |
| dansmith | I bet some former perl refugee could validate that | 19:15 |
| mriedem | auggy: ! | 19:15 |
| auggy | :) | 19:16 |
| mriedem | skipUnless(nova/network/* OR nova/virt/libvirt/vif.py) | 19:16 |
| dansmith | mriedem: but yeah, testing seems to do the right thing | 19:16 |
| mriedem | skipIf(!(nova/network/* OR nova/virt/libvirt/vif.py)) | 19:16 |
| dansmith | mriedem: so I guess it's right :) | 19:16 |
| mriedem | \o/ | 19:16 |
| mriedem | auggy: need your regex skillz on https://review.openstack.org/#/c/371065/1/zuul/layout.yaml | 19:17 |
| *** fragatina has quit IRC | 19:17 | |
| auggy | lemme take a look | 19:17 |
| *** twinklechawla has joined #openstack-nova | 19:17 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add keypairs_links into resp https://review.openstack.org/370760 | 19:17 |
| *** fragatina has joined #openstack-nova | 19:18 | |
| auggy | dansmith: sorry trying to get the phone, this guy would not shut up | 19:18 |
| auggy | protip if your employer ever does 401k stuff through bancorp, just run away | 19:19 |
| *** fragatina has quit IRC | 19:19 | |
| auggy | catching up on the conversation | 19:19 |
| *** fragatina has joined #openstack-nova | 19:19 | |
| auggy | i usually use ?() to make a group that matches but that i don't want to capture | 19:20 |
| *** twinklechawla has left #openstack-nova | 19:21 | |
| dansmith | ?() isn't a thing right? | 19:21 |
| dansmith | it would be ()? | 19:21 |
| auggy | mriedem: do you want it to match nova/network/.* OR nova/virt/libvirt/vif.py ? | 19:21 |
| auggy | it's ?: i think | 19:22 |
| dansmith | ? matches the thing before it, so ?() would be an error I think | 19:22 |
| auggy | i usually do ?:() | 19:22 |
| auggy | i'm looking to double check hehe | 19:23 |
| dansmith | yeah, doesn't work for me | 19:23 |
| dansmith | anyway, unrelated to the thing at hand :) | 19:23 |
| auggy | haha | 19:24 |
| mriedem | anyway, passes for me on http://www.regexpal.com/ | 19:24 |
| mriedem | so i'm just going to leave it | 19:24 |
| auggy | (?: foobar) means match but don't capture that | 19:24 |
| dansmith | okay (?: ) != ?() :) | 19:24 |
| auggy | haha yeah | 19:24 |
| auggy | sorry | 19:25 |
| auggy | ok so if you want to match either of those two things, you could just do ((nova/network/.*)|(?!nova/virt/libvirt/vif.py)) | 19:25 |
| *** ametts has joined #openstack-nova | 19:26 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33 https://review.openstack.org/370732 | 19:26 |
| auggy | what would this match? nova/virt/libvirt/vif.py.*$ | 19:26 |
| melwitt | mriedem: doesn't 'all-files-match-any' probably mean it's already doing an OR of all that's there? | 19:28 |
| auggy | oh i see you want it to not match those files | 19:29 |
| * auggy is a little slow to the party | 19:29 | |
| *** aswadr_ has quit IRC | 19:32 | |
| *** fragatina has quit IRC | 19:32 | |
| *** pcaruana has quit IRC | 19:32 | |
| *** bvanhav has quit IRC | 19:32 | |
| *** davidlenwell has quit IRC | 19:34 | |
| *** rama_y has quit IRC | 19:34 | |
| auggy | i haven't done consecutive negative looks aheads like this so my only question is if the point where it starts the lookahead on the second one is going to still be the start of the string | 19:35 |
| *** tongli has quit IRC | 19:38 | |
| *** fragatina has joined #openstack-nova | 19:38 | |
| *** itlinux has quit IRC | 19:39 | |
| *** rama_y has joined #openstack-nova | 19:39 | |
| *** fragatin_ has joined #openstack-nova | 19:40 | |
| *** fragatina has quit IRC | 19:40 | |
| *** david-lyle has quit IRC | 19:40 | |
| *** david-lyle has joined #openstack-nova | 19:40 | |
| *** Apoorva has quit IRC | 19:40 | |
| *** aunnam_ has quit IRC | 19:42 | |
| *** fragatin_ has quit IRC | 19:44 | |
| *** code-R_ has quit IRC | 19:44 | |
| *** davidlenwell has joined #openstack-nova | 19:47 | |
| *** itlinux has joined #openstack-nova | 19:48 | |
| *** sdake has quit IRC | 19:49 | |
| *** sdake_ has joined #openstack-nova | 19:49 | |
| *** rcernin has joined #openstack-nova | 19:53 | |
| *** baoli has quit IRC | 19:56 | |
| *** baoli has joined #openstack-nova | 19:56 | |
| *** krtaylor has joined #openstack-nova | 19:56 | |
| *** ametts has quit IRC | 20:01 | |
| *** fragatina has joined #openstack-nova | 20:01 | |
| *** ducttape_ has quit IRC | 20:03 | |
| auggy | mriedem: ok that looks legit, i tinkered around with it too just to make sure the lookaheads worked consecutively and it looks like they do | 20:04 |
| *** jvgrant_ has joined #openstack-nova | 20:04 | |
| *** fragatina has quit IRC | 20:05 | |
| mriedem | cool | 20:05 |
| auggy | regexes for me are like spanish... i know it pretty well but when i don't practice it every day i forget things ahaha | 20:06 |
| *** Apoorva has joined #openstack-nova | 20:06 | |
| *** Apoorva_ has joined #openstack-nova | 20:07 | |
| *** jvgrant has quit IRC | 20:08 | |
| *** fragatina has joined #openstack-nova | 20:10 | |
| *** Apoorva has quit IRC | 20:10 | |
| *** ijw has quit IRC | 20:11 | |
| *** ijw has joined #openstack-nova | 20:11 | |
| *** itlinux has quit IRC | 20:13 | |
| *** ametts has joined #openstack-nova | 20:13 | |
| *** fragatina has quit IRC | 20:15 | |
| *** pwnall has joined #openstack-nova | 20:15 | |
| *** pwnall is now known as Guest13036 | 20:15 | |
| *** ijw has quit IRC | 20:16 | |
| *** Guest88340 has quit IRC | 20:17 | |
| mriedem | netaddr fix is merrged | 20:17 |
| *** hamzy has quit IRC | 20:20 | |
| *** rhagarty has joined #openstack-nova | 20:22 | |
| *** Guest13036 has quit IRC | 20:23 | |
| bauzas | \o/ | 20:25 |
| *** tbachman has quit IRC | 20:25 | |
| * bauzas likes coming back to the chan and reading that | 20:25 | |
| *** ducttape_ has joined #openstack-nova | 20:27 | |
| mriedem | jaypipes: did you have a chance to look at https://review.openstack.org/#/c/370681/? | 20:27 |
| *** fragatina has joined #openstack-nova | 20:29 | |
| mriedem | ^ is passing ovs and linuxbridge jobs with ihar's new test.. | 20:29 |
| mriedem | so maybe ok | 20:30 |
| sdague | mriedem: the req bump merged | 20:31 |
| bauzas | WAAAAT? we now have quotas for server groups ? | 20:31 |
| * bauzas facepalms | 20:31 | |
| melwitt | bauzas: we've had them for a long time. apparently they don't always work based on the bug you're probably looking at that I had been meaning to look at | 20:32 |
| *** kfarr has quit IRC | 20:33 | |
| *** ijw has joined #openstack-nova | 20:33 | |
| melwitt | it doesn't look like it's a regression, if I understood the reporter said they saw the problem in juno first and then repro'd on master | 20:33 |
| bauzas | melwitt: yeah I don't think so | 20:33 |
| mriedem | sdague: yup | 20:33 |
| mriedem | sdague: markus_z and i broke puppet ci though :) | 20:33 |
| mriedem | \o/ | 20:34 |
| mriedem | sdague: maybe you want to look at https://review.openstack.org/#/c/370681/ ? | 20:34 |
| mriedem | melwitt: bauzas: ^ | 20:34 |
| mriedem | really all cores | 20:34 |
| mriedem | that's the last thing for rc1 | 20:34 |
| bauzas | mriedem: I was about to ask you about rc1 | 20:34 |
| *** zzzeek has quit IRC | 20:35 | |
| mriedem | ^ is our last change | 20:35 |
| bauzas | cool | 20:36 |
| mriedem | it's passing ovs and LB jobs with that change | 20:36 |
| mriedem | i have several comments/questions in there, | 20:36 |
| mriedem | like, i think we might be passing by luck but i'm not sure | 20:36 |
| mriedem | and/or, we're passing b/c we only use os-vif for ovs/lbr but not other things like vhostuser yet | 20:36 |
| jaypipes | mriedem: done | 20:36 |
| *** hemanthm is now known as hemanthm|away | 20:37 | |
| *** Apoorva_ has quit IRC | 20:37 | |
| *** ijw has quit IRC | 20:38 | |
| bauzas | mriedem: yeah the main problem could be with other network drivers not being tested and not using os-vif yet,right? | 20:38 |
| *** Apoorva has joined #openstack-nova | 20:38 | |
| bauzas | I mean other vif types tbc | 20:38 |
| mriedem | bauzas: other vif types won't go down this route | 20:38 |
| mriedem | they will go down the legacy path before tihs | 20:38 |
| mriedem | *this | 20:38 |
| bauzas | okay | 20:38 |
| mriedem | jaypipes: thanks | 20:38 |
| mriedem | need to check intel ci reuslts | 20:39 |
| mriedem | *results | 20:39 |
| bauzas | mriedem: oh right,L815 | 20:39 |
| *** zzzeek has joined #openstack-nova | 20:39 | |
| openstackgerrit | Sean Dague proposed openstack/nova: WIP: try utf8 force https://review.openstack.org/371147 | 20:40 |
| mriedem | hmm, neutron did not start | 20:40 |
| mriedem | http://intel-openstack-ci-logs.ovh/81/370681/7/check/tempest-dsvm-intel-nfv-xenial/e20e1c4/logs/devstacklog.txt.gz | 20:40 |
| *** baoli has quit IRC | 20:41 | |
| *** baoli has joined #openstack-nova | 20:41 | |
| *** rkmrhj has joined #openstack-nova | 20:42 | |
| *** ametts has quit IRC | 20:42 | |
| mriedem | jaypipes: any idea why he had to update all of those vhostuser unit tests to pass None for mtu? | 20:45 |
| mriedem | i think i see why | 20:51 |
| mriedem | self.vif_vhostuser_ovs* uses network_bridge which he changed to have mtu=none | 20:51 |
| mriedem | so those no longer pass mtu for those vhostuser fake vifs | 20:51 |
| mriedem | hmm, should have probably used a different/new network_model for that | 20:51 |
| *** tgtanya_ has quit IRC | 20:52 | |
| *** baoli has quit IRC | 20:53 | |
| *** ijw has joined #openstack-nova | 20:53 | |
| *** akshai has quit IRC | 20:54 | |
| *** gouthamr has quit IRC | 20:54 | |
| *** baoli has joined #openstack-nova | 20:55 | |
| *** nherciu has quit IRC | 20:57 | |
| *** tgtanya has joined #openstack-nova | 20:57 | |
| *** takashin has joined #openstack-nova | 20:57 | |
| *** ijw has quit IRC | 20:58 | |
| *** thumpba has quit IRC | 20:59 | |
| *** thumpba has joined #openstack-nova | 20:59 | |
| mriedem | hells bells nova meeting | 21:00 |
| *** rfolco has quit IRC | 21:02 | |
| *** tgtanya has quit IRC | 21:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Override MTU for os_vif attachments https://review.openstack.org/370681 | 21:04 |
| mriedem | jaypipes: ^ updated to cleanup the tests a bit | 21:04 |
| mriedem | so we don't touch those vhostuser tests | 21:04 |
| *** eharney has quit IRC | 21:05 | |
| *** matrohon has quit IRC | 21:05 | |
| *** TravT has joined #openstack-nova | 21:08 | |
| *** gouthamr has joined #openstack-nova | 21:09 | |
| *** ijw has joined #openstack-nova | 21:09 | |
| *** thumpba has quit IRC | 21:10 | |
| *** akshai has joined #openstack-nova | 21:10 | |
| *** gszasz has joined #openstack-nova | 21:11 | |
| *** salv-orlando has joined #openstack-nova | 21:11 | |
| *** catintheroof has joined #openstack-nova | 21:15 | |
| *** tblakes has quit IRC | 21:15 | |
| *** burgerk has quit IRC | 21:15 | |
| *** TravT has quit IRC | 21:16 | |
| *** krtaylor has quit IRC | 21:17 | |
| *** salv-orlando has quit IRC | 21:18 | |
| *** salv-orlando has joined #openstack-nova | 21:22 | |
| *** ijw has quit IRC | 21:25 | |
| *** tblakes has joined #openstack-nova | 21:26 | |
| *** gyee has joined #openstack-nova | 21:27 | |
| auggy | i wonder if i can add "derailed a nova meeting with talk of donuts" to the success tag haha | 21:35 |
| jaypipes | mriedem: cool, re-reviewing. | 21:35 |
| woodster_ | auggy: :) | 21:35 |
| *** rkmrhj has quit IRC | 21:35 | |
| *** pwnall has joined #openstack-nova | 21:36 | |
| *** pwnall is now known as Guest42864 | 21:37 | |
| jaypipes | mriedem: wallaby'd. | 21:37 |
| mriedem | jaypipes: thanks | 21:37 |
| jaypipes | mriedem: for you? any time. | 21:37 |
| mriedem | jaypipes: you asked for it http://logs.openstack.org/92/371092/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/62ba450/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-09-15_20_33_01_570 | 21:39 |
| mriedem | dansmith: ^ | 21:39 |
| dansmith | hmm | 21:40 |
| *** lpetrut has quit IRC | 21:41 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1624119 | 21:42 |
| openstack | Launchpad bug 1624119 in OpenStack Compute (nova) "remove_deleted_instances in RT stacktraces with AttributeError" [Undecided,New] | 21:42 |
| bauzas | mriedem: so did someone checked we *actually* changed the mtu value in logs ? :D | 21:43 |
| dansmith | mriedem: RT is putting dicts into self.tracked_instances? | 21:43 |
| dansmith | oh, this is from paulmurray's dict-or-dict-or-something method I bet | 21:44 |
| mriedem | bauzas: ihar added a tempest test for that which we were testing with | 21:44 |
| mriedem | http://logs.openstack.org/13/370813/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ddf5f49/console.html#_2016-09-15_16_02_33_232273 | 21:44 |
| mriedem | http://logs.openstack.org/51/371051/1/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/06f5d12/console.html#_2016-09-15_19_33_14_976025 | 21:44 |
| mriedem | it failed before john's change | 21:44 |
| bauzas | mriedem: oh cool | 21:45 |
| *** scottda has quit IRC | 21:45 | |
| *** anush has quit IRC | 21:45 | |
| *** adam_g has quit IRC | 21:45 | |
| *** knikolla has quit IRC | 21:45 | |
| bauzas | mriedem: because I was about to read logs to check that value | 21:45 |
| *** wasmum has quit IRC | 21:46 | |
| *** clif_h has quit IRC | 21:46 | |
| *** Vek has quit IRC | 21:46 | |
| *** cargonza has quit IRC | 21:46 | |
| *** mrda has quit IRC | 21:46 | |
| *** efried has quit IRC | 21:46 | |
| *** CustosLimen has quit IRC | 21:46 | |
| *** JayF has quit IRC | 21:46 | |
| *** jamielennox has quit IRC | 21:46 | |
| *** jhesketh has quit IRC | 21:47 | |
| *** swamireddy has quit IRC | 21:47 | |
| *** gus has quit IRC | 21:47 | |
| *** d34dh0r53 has quit IRC | 21:47 | |
| *** mrhillsman has quit IRC | 21:47 | |
| *** pradk has quit IRC | 21:48 | |
| *** gus has joined #openstack-nova | 21:48 | |
| *** scottda has joined #openstack-nova | 21:49 | |
| *** jhesketh has joined #openstack-nova | 21:49 | |
| jaypipes | dansmith: shall I fix? | 21:49 |
| dansmith | jaypipes: sure | 21:49 |
| bauzas | mriedem: meh the RT bug you wrote is very sad | 21:50 |
| bauzas | I could look at that by tomorrow morning | 21:50 |
| bauzas | unless US folks want to eat their afternoon looking at that | 21:50 |
| mriedem | well, i'll be on all night | 21:51 |
| *** adam_g has joined #openstack-nova | 21:51 | |
| mriedem | so if we want this in rc1 i'm game | 21:51 |
| *** adam_g has quit IRC | 21:51 | |
| *** adam_g has joined #openstack-nova | 21:51 | |
| *** d34dh0r53 has joined #openstack-nova | 21:51 | |
| *** CustosLimen has joined #openstack-nova | 21:52 | |
| *** cargonza has joined #openstack-nova | 21:52 | |
| dansmith | so, | 21:52 |
| *** ijw has joined #openstack-nova | 21:52 | |
| dansmith | this will auto-heal when we apply | 21:52 |
| *** knikolla has joined #openstack-nova | 21:52 | |
| dansmith | so I'm fine without it in rc1 | 21:52 |
| dansmith | up to you | 21:52 |
| *** rlrossit has quit IRC | 21:52 | |
| mriedem | if someone can get a quick fix up i'll check it once i'm back home | 21:52 |
| dansmith | if we have an rc2 then we do that, and if not, backport.. not the end of the world | 21:52 |
| mriedem | i think we'll have rc2 for translations | 21:53 |
| dansmith | yeah, so.. no reason to rush it IMHO | 21:53 |
| *** wasmum has joined #openstack-nova | 21:54 | |
| mriedem | i'd hate to release with big fat stack traces even if you're not using hte placement api | 21:54 |
| mriedem | so i think rc2 at least | 21:54 |
| bauzas | yeah, I was targeting RC2 | 21:54 |
| bauzas | hence my offer for tomorrow | 21:54 |
| dansmith | sure | 21:54 |
| mriedem | i think dan is out tomorrow, and i'm out in the afternoon | 21:55 |
| dansmith | yeah | 21:55 |
| bauzas | mriedem: planning to do the release change today I guess | 21:55 |
| mriedem | yes i'll tag rc1 tonight | 21:55 |
| bauzas | okay | 21:55 |
| bauzas | so yeah, I'll look at that bug and we could see what would be the resolution path for RC2 | 21:55 |
| bauzas | anyway, folks \o | 21:55 |
| bauzas | let's call it a day | 21:56 |
| dansmith | jaypipes: you're on it? | 21:56 |
| *** jheroux has quit IRC | 21:57 | |
| *** swamireddy has joined #openstack-nova | 21:57 | |
| mriedem | i'm heading home, back online later | 21:58 |
| *** mriedem has quit IRC | 21:58 | |
| *** cmartin has quit IRC | 21:58 | |
| *** mrda has joined #openstack-nova | 21:59 | |
| *** clif_h has joined #openstack-nova | 21:59 | |
| dansmith | oh, no it's just obj_to_primitive()'d .. heh how silly | 21:59 |
| dansmith | easy fix | 21:59 |
| *** 32NABYP9K is now known as sabari | 21:59 | |
| *** JayF has joined #openstack-nova | 21:59 | |
| *** mrhillsman has joined #openstack-nova | 22:00 | |
| *** jamielennox has joined #openstack-nova | 22:01 | |
| *** efried has joined #openstack-nova | 22:01 | |
| *** akshai has quit IRC | 22:02 | |
| *** smatzek has quit IRC | 22:03 | |
| *** kylek3h has quit IRC | 22:03 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix object assumption in remove_deleted_instances() https://review.openstack.org/371179 | 22:03 |
| dansmith | jaypipes: ^ | 22:03 |
| *** rtheis has quit IRC | 22:03 | |
| *** krtaylor has joined #openstack-nova | 22:04 | |
| *** rhagarty_ has joined #openstack-nova | 22:04 | |
| dansmith | the test didn't catch it because it only tested an empty compute node | 22:04 |
| *** cleong has quit IRC | 22:05 | |
| *** rhagarty has quit IRC | 22:06 | |
| *** xyang1 has quit IRC | 22:07 | |
| *** liangy has quit IRC | 22:09 | |
| *** Apoorva has quit IRC | 22:09 | |
| *** artom has quit IRC | 22:10 | |
| *** mdrabe has quit IRC | 22:11 | |
| *** edmondsw has quit IRC | 22:14 | |
| *** mdrabe has joined #openstack-nova | 22:14 | |
| *** artom has joined #openstack-nova | 22:15 | |
| *** unicell has quit IRC | 22:16 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Remove deprecated compute_available_monitors option https://review.openstack.org/371062 | 22:18 |
| *** jkraj has joined #openstack-nova | 22:19 | |
| *** ijw has quit IRC | 22:19 | |
| *** Guest42864 has quit IRC | 22:20 | |
| *** ijw has joined #openstack-nova | 22:22 | |
| *** Apoorva has joined #openstack-nova | 22:22 | |
| *** marst has quit IRC | 22:22 | |
| *** marst has joined #openstack-nova | 22:23 | |
| *** mdrabe has quit IRC | 22:23 | |
| *** marst has quit IRC | 22:26 | |
| *** tonytan4ever has quit IRC | 22:27 | |
| *** salv-orlando has quit IRC | 22:32 | |
| *** TravT has joined #openstack-nova | 22:33 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Remove deprecated compute_available_monitors option https://review.openstack.org/371062 | 22:39 |
| *** thumpba has joined #openstack-nova | 22:41 | |
| *** marst has joined #openstack-nova | 22:41 | |
| *** ducttape_ has quit IRC | 22:42 | |
| *** unicell has joined #openstack-nova | 22:43 | |
| jaypipes | dansmith: wallaby'd. | 22:43 |
| dansmith | jaypipes: thanks | 22:44 |
| jaypipes | dansmith: I had done it locally but then had meeting and had to leave tests running :) oh well | 22:44 |
| *** mriedem has joined #openstack-nova | 22:44 | |
| dansmith | jaypipes: I figured your big deal ness got in the way, figured I'd work it up being a peon and all :D | 22:44 |
| jaypipes | dansmith: ha ha. | 22:44 |
| dansmith | :P | 22:45 |
| mriedem | oh snap | 22:45 |
| mriedem | https://review.openstack.org/#/c/371179/ | 22:45 |
| mriedem | rc2 indeed | 22:45 |
| *** salv-orlando has joined #openstack-nova | 22:46 | |
| *** anush has joined #openstack-nova | 22:46 | |
| *** Cristina_ has joined #openstack-nova | 22:48 | |
| *** baoli has quit IRC | 22:48 | |
| *** yamahata has quit IRC | 22:49 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Remove config option snapshot_name_template https://review.openstack.org/371203 | 22:50 |
| *** jaypipes has quit IRC | 22:50 | |
| *** ijw has quit IRC | 22:50 | |
| *** CristinaPauna has quit IRC | 22:51 | |
| *** sdague has quit IRC | 22:52 | |
| *** artom has quit IRC | 22:55 | |
| *** hemna is now known as hemnafk | 22:55 | |
| *** artom has joined #openstack-nova | 22:56 | |
| *** claudiub has quit IRC | 23:06 | |
| *** ijw has joined #openstack-nova | 23:11 | |
| *** ducttape_ has joined #openstack-nova | 23:12 | |
| *** jkraj has quit IRC | 23:13 | |
| *** ijw has quit IRC | 23:16 | |
| *** nic has quit IRC | 23:18 | |
| *** maticue has quit IRC | 23:21 | |
| *** catintheroof has quit IRC | 23:21 | |
| *** takedakn has joined #openstack-nova | 23:23 | |
| *** takashin has left #openstack-nova | 23:25 | |
| *** Apoorva has quit IRC | 23:25 | |
| *** ducttape_ has quit IRC | 23:29 | |
| *** takedakn has quit IRC | 23:30 | |
| *** thumpba has quit IRC | 23:30 | |
| *** tblakes has quit IRC | 23:32 | |
| *** TravT has quit IRC | 23:36 | |
| *** armax has quit IRC | 23:37 | |
| *** hoonetorg has quit IRC | 23:37 | |
| *** markvoelker has quit IRC | 23:39 | |
| *** TravT has joined #openstack-nova | 23:40 | |
| *** Jeffrey4l__ is now known as Jeffrey4l | 23:40 | |
| *** BjoernT has quit IRC | 23:44 | |
| *** gszasz has quit IRC | 23:45 | |
| mriedem | gdi wtf http://logs.openstack.org/81/370681/8/gate/gate-nova-python27-db-ubuntu-xenial/87b6e37/console.html#_2016-09-15_22_54_49_868786 | 23:46 |
| mriedem | global change here? https://review.openstack.org/#/c/370681/8/nova/tests/unit/virt/libvirt/test_vif.py@1614 | 23:47 |
| *** rmcall has quit IRC | 23:47 | |
| *** unicell has quit IRC | 23:50 | |
| *** ijw has joined #openstack-nova | 23:50 | |
| *** tri2sing has quit IRC | 23:53 | |
| *** Vek has joined #openstack-nova | 23:54 | |
| *** armax has joined #openstack-nova | 23:54 | |
| *** hoonetorg has joined #openstack-nova | 23:55 | |
| *** thumpba has joined #openstack-nova | 23:57 | |
| *** ijw has quit IRC | 23:58 | |
| *** rama_y has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
