Thursday, 2016-09-15

*** diogogmt has joined #openstack-nova00:00
clarkbdansmith: no, maybe they are both getting the requests and the other is cranky about it?00:00
*** jamielennox|away is now known as jamielennox00:00
clarkbalso 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 test00:01
*** ducttape_ has joined #openstack-nova00:02
dansmithclarkb: I guess.. do they definitely have different hostnames/00:02
*** takedakn has quit IRC00:02
dansmithgdi I can't hit ? to save my life00:02
*** slagle has joined #openstack-nova00:03
clarkbdansmith: ya pretty sure they had differnet hostnames00:03
clarkbtrying to double check that now with the logs00:04
clarkbdansmith: syslog reports different names00:05
dansmithclarkb: only matters what nova thinks, but okay00:05
dansmithnot really sure what else would cause that00:05
clarkbI am not sure how to check what nova thinks :) but if syslog says they are different I am hoping nova agrees00:05
dansmithhttp://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_88000:06
dansmithclarkb: the host line there00:06
clarkbhttp://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 other00:07
clarkband they are different00:07
dansmithclarkb: oh00:09
dansmithI lied00:09
dansmithI read too quickly, I thought this was a complaint about the instance, but it's a complaint about the migration00:09
dansmithwhich can happen if a migration isn't completed, like an instance is deleted first or something00:09
dansmithwe have an open bug for that00:09
*** sai has quit IRC00:10
clarkbso maybe don't worry about that too much¿00:10
dansmithyeah00:10
clarkbalso silly keyboard I wanted ?00:10
dansmithjej00:10
dansmither, heh00:10
clarkbok now just to figure out what is going on with that grenade job00:10
*** lucas-dinner has quit IRC00:10
*** hieulq has joined #openstack-nova00:13
*** lucasagomes has joined #openstack-nova00:16
*** sai has joined #openstack-nova00:16
*** thorst has joined #openstack-nova00:17
*** Swami__ has joined #openstack-nova00:19
*** Swami_ has quit IRC00:22
*** thorst has quit IRC00:24
mriedemswamireddy: 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-nova00:26
*** ekhugen_alt has quit IRC00:29
*** kaisers has joined #openstack-nova00:31
*** ekhugen has joined #openstack-nova00:32
*** kaisers3 has joined #openstack-nova00:32
*** annegentle has joined #openstack-nova00:33
*** kaisers2 has quit IRC00:34
*** kaisers1 has quit IRC00:34
*** Swami__ has quit IRC00:36
*** Swami__ has joined #openstack-nova00:37
*** jlvillal has quit IRC00:41
*** vishwanathj has quit IRC00:43
mriedemmelwitt: do you know what's going on here https://review.openstack.org/#/c/335884/ ?00:45
mriedemi always feel dumb for asking00:45
*** rkrum has joined #openstack-nova00:46
mriedemmikal: 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:open00:46
*** jlvillal has joined #openstack-nova00:46
*** jlvillal has quit IRC00:47
*** jlvillal has joined #openstack-nova00:47
*** jlvillal has quit IRC00:47
*** jlvillal has joined #openstack-nova00:47
mikalmriedem: I have to step out for an hour or so, will look when I get back00:48
*** tblakes has quit IRC00:49
*** phuongnh has joined #openstack-nova00:49
*** JoseMello has joined #openstack-nova00:50
*** akshai has joined #openstack-nova00:51
*** akshai has quit IRC00:53
melwittmriedem: I haven't looked at that yet so I don't know yet :P00:54
*** gongysh has joined #openstack-nova00:55
openstackgerritTakashi NATSUME proposed openstack/nova: Fix an error in archiving 'migrations' table  https://review.openstack.org/32368400:55
melwittmriedem: 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 sparse01:00
melwittthat 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 commit01:01
melwitt*commit message01:01
*** rkrum has quit IRC01:02
*** hoangcx has quit IRC01:05
*** tuanla_ has joined #openstack-nova01:07
*** namnh has joined #openstack-nova01:08
*** tiendc has joined #openstack-nova01:08
hieulqpumaranikar: ping01:08
hieulqpumaranikar: which part in libvirt config file you want to mention? help text or what?01:09
*** vladikr has quit IRC01:10
mriedemmelwitt: yeah me neither, glad i'm not the only one01:11
*** mmedvede has joined #openstack-nova01:13
*** ijw has quit IRC01:17
*** thorst has joined #openstack-nova01:21
*** cdelatte has quit IRC01:25
*** ijw has joined #openstack-nova01:25
*** tiendc has quit IRC01:27
openstackgerritGiridhar Jayavelu proposed openstack/nova: Handle TypeError when disabling host service  https://review.openstack.org/29091901:27
*** EinstCrazy has joined #openstack-nova01:27
mriedemmelwitt: i'm +2 on https://review.openstack.org/#/c/323684/ if you wanted to get that in01:29
*** thorst has quit IRC01:29
*** hoangcx has joined #openstack-nova01:30
*** EinstCrazy has quit IRC01:33
*** thorst has joined #openstack-nova01:35
*** Sukhdev has quit IRC01:37
*** ducttape_ has quit IRC01:37
*** baoli has joined #openstack-nova01:39
*** baoli has quit IRC01:41
*** salv-orlando has joined #openstack-nova01:41
*** baoli has joined #openstack-nova01:41
openstackgerritNguyen Phuong An proposed openstack/nova: Update info of machine-processable description  https://review.openstack.org/36542601:42
*** ssurana has quit IRC01:43
*** salv-orl_ has quit IRC01:43
*** thorst has quit IRC01:45
*** baoli has quit IRC01:46
*** Guest2984 has quit IRC01:47
*** gongysh has quit IRC01:47
*** woodster_ has quit IRC01:50
*** raunak has quit IRC01:51
*** ducttape_ has joined #openstack-nova01:55
*** rama_y has quit IRC01:55
*** ducttape_ has quit IRC01:56
*** ijw has quit IRC01:56
*** hoangcx has quit IRC02:00
*** Apoorva_ has joined #openstack-nova02:01
*** rmcall has quit IRC02:02
*** rmcall has joined #openstack-nova02:03
*** Apoorva has quit IRC02:04
*** ijw has joined #openstack-nova02:05
*** Apoorva_ has quit IRC02:06
*** unicell has quit IRC02:06
*** diogogmt has quit IRC02:09
*** 32NABYP9K has joined #openstack-nova02:14
*** TravT has quit IRC02:16
*** TravT has joined #openstack-nova02:16
*** bkopilov has quit IRC02:16
*** TravT has quit IRC02:20
*** namnh has quit IRC02:22
openstackgerritTakashi NATSUME proposed openstack/nova: Treat 'body_map' as a mapping of HTTP request bodies  https://review.openstack.org/36345702:25
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host  https://review.openstack.org/35710502:25
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix wrong parameters in os-volumes.inc  https://review.openstack.org/34711202:25
openstackgerritTakashi NATSUME proposed openstack/nova: Fix unnecessary string interpolation  https://review.openstack.org/35757702:25
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix 'id' (attachment_id) parameters  https://review.openstack.org/34986302:26
*** michchap has quit IRC02:26
*** michchap has joined #openstack-nova02:26
*** ducttape_ has joined #openstack-nova02:30
*** shangxdy has joined #openstack-nova02:36
*** ssurana has joined #openstack-nova02:37
*** annegentle has quit IRC02:38
*** thorst has joined #openstack-nova02:47
*** ducttape_ has quit IRC02:47
*** tiendc has joined #openstack-nova02:48
*** mriedem has quit IRC02:49
*** gongysh has joined #openstack-nova02:50
*** hoangcx has joined #openstack-nova02:54
*** rrecio_ has quit IRC02:55
*** rrecio has joined #openstack-nova02:56
*** thorst has quit IRC02:56
*** rmcall has quit IRC02:59
*** rmcall has joined #openstack-nova02:59
*** shangxdy has quit IRC03:00
*** sdake has joined #openstack-nova03:01
melwitttakashin: 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@632303:04
*** ijw has quit IRC03:04
*** ijw has joined #openstack-nova03:05
melwitttakashin: nevermind, I think it's obvious now that's why03:07
*** gjayavelu has quit IRC03:09
*** yamahata has quit IRC03:11
*** ijw has quit IRC03:13
*** code-R has joined #openstack-nova03:15
*** code-R_ has joined #openstack-nova03:18
*** gouthamr has quit IRC03:19
*** namnh has joined #openstack-nova03:19
*** code-R has quit IRC03:21
*** gongysh has quit IRC03:21
*** ssurana has quit IRC03:22
*** marst has joined #openstack-nova03:23
*** namnh has quit IRC03:24
*** ssurana has joined #openstack-nova03:24
*** gongysh has joined #openstack-nova03:25
*** namnh has joined #openstack-nova03:25
*** namnh has quit IRC03:25
*** namnh has joined #openstack-nova03:25
*** raunak has joined #openstack-nova03:25
*** namnh has quit IRC03:26
*** namnh has joined #openstack-nova03:27
*** namnh has quit IRC03:27
*** namnh has joined #openstack-nova03:27
*** hoangcx has quit IRC03:28
*** ssurana has quit IRC03:31
*** hoangcx has joined #openstack-nova03:31
*** bkopilov has joined #openstack-nova03:33
*** namnh has quit IRC03:39
*** namnh has joined #openstack-nova03:40
*** namnh has quit IRC03:40
*** namnh has joined #openstack-nova03:41
*** namnh has quit IRC03:41
*** namnh has joined #openstack-nova03:42
*** namnh has quit IRC03:43
*** namnh has joined #openstack-nova03:44
*** thorst has joined #openstack-nova03:44
*** thorst has quit IRC03:46
*** ducttape_ has joined #openstack-nova03:48
*** raunak has quit IRC03:48
*** jdurgin has joined #openstack-nova03:49
*** ijw_ has joined #openstack-nova03:49
*** ducttape_ has quit IRC03:53
*** hoangcx has quit IRC03:53
*** abhishekk has joined #openstack-nova03:53
*** ijw_ has quit IRC03:54
*** raunak has joined #openstack-nova04:02
*** mingyu has joined #openstack-nova04:05
*** hoangcx has joined #openstack-nova04:05
*** phschwartz has quit IRC04:07
*** mordred has quit IRC04:08
*** mordred has joined #openstack-nova04:10
*** phschwartz has joined #openstack-nova04:11
*** gjayavelu has joined #openstack-nova04:12
*** ratailor has joined #openstack-nova04:14
*** adisky has joined #openstack-nova04:15
*** ssurana has joined #openstack-nova04:17
*** psachin has joined #openstack-nova04:21
*** Swami__ has quit IRC04:21
openstackgerritRyan Rossiter proposed openstack/nova: WIP: Shared state host manager  https://review.openstack.org/33246304:22
*** ociuhandu has joined #openstack-nova04:24
*** thorst has joined #openstack-nova04:29
*** fragatin_ has joined #openstack-nova04:29
*** fragatina has quit IRC04:32
*** fragatin_ has quit IRC04:33
*** coolsvap has joined #openstack-nova04:34
*** thorst has quit IRC04:39
abhishekkhi, has anyone from India applied for Spain visa?04:42
*** thorst has joined #openstack-nova04:45
*** jaosorior has joined #openstack-nova04:47
*** aswadr_ has joined #openstack-nova04:49
*** ducttape_ has joined #openstack-nova04:49
*** takedakn has joined #openstack-nova04:50
*** unicell has joined #openstack-nova04:51
*** ducttape_ has quit IRC04:55
*** sudipto_ has joined #openstack-nova04:55
*** sudipto has joined #openstack-nova04:55
*** unicell has quit IRC04:57
*** unicell has joined #openstack-nova04:57
*** anush__ has quit IRC04:58
*** thorst has quit IRC04:58
*** thorst has joined #openstack-nova04:58
*** anush__ has joined #openstack-nova04:59
*** Jeffrey4l__ has joined #openstack-nova05:00
*** lujinluo has joined #openstack-nova05:01
*** unicell1 has joined #openstack-nova05:01
*** jaosorior has quit IRC05:01
*** unicell has quit IRC05:02
*** jaosorior has joined #openstack-nova05:02
*** sdake has quit IRC05:03
*** rama_y has joined #openstack-nova05:03
*** mingyu has quit IRC05:03
*** Jeffrey4l_ has quit IRC05:04
*** thorst has quit IRC05:07
*** slaweq_ has joined #openstack-nova05:08
*** thorst has joined #openstack-nova05:09
*** thorst has quit IRC05:13
*** raunak has quit IRC05:14
*** thorst has joined #openstack-nova05:15
*** rcernin has joined #openstack-nova05:17
*** aunnam has joined #openstack-nova05:19
*** thorst has quit IRC05:19
*** takedakn has quit IRC05:21
*** sdake has joined #openstack-nova05:22
*** lpetrut has joined #openstack-nova05:28
*** raunak has joined #openstack-nova05:29
*** artom has quit IRC05:30
*** dimtruck is now known as zz_dimtruck05:31
*** gouthamr has joined #openstack-nova05:31
*** artom has joined #openstack-nova05:32
*** fragatina has joined #openstack-nova05:32
*** artom has quit IRC05:32
*** artom has joined #openstack-nova05:32
*** thorst has joined #openstack-nova05:34
*** fragatina has quit IRC05:37
*** markvoelker has quit IRC05:41
*** thorst has quit IRC05:42
*** thorst has joined #openstack-nova05:44
*** vishwanathj has joined #openstack-nova05:47
*** thorst has quit IRC05:49
*** raunak has quit IRC05:50
*** thorst has joined #openstack-nova05:50
*** zz_dimtruck is now known as dimtruck05:52
*** rkmrhj has joined #openstack-nova05:52
*** thorst has quit IRC05:55
*** oanson has joined #openstack-nova05:56
*** thorst has joined #openstack-nova05:56
openstackgerritGábor Antal proposed openstack/nova: Removed unnecessary parantheses in return statements  https://review.openstack.org/37054906:00
*** thorst has quit IRC06:01
*** mjura has joined #openstack-nova06:01
*** nherciu has joined #openstack-nova06:02
*** thorst has joined #openstack-nova06:02
*** dimtruck is now known as zz_dimtruck06:03
*** pcaruana has joined #openstack-nova06:04
*** paul-carlton has joined #openstack-nova06:05
*** karthiks has joined #openstack-nova06:06
*** thorst has quit IRC06:07
*** mjura has quit IRC06:08
*** thorst has joined #openstack-nova06:08
*** karthiks has quit IRC06:10
*** jdurgin has quit IRC06:13
*** pcaruana is now known as pcaruana|afk|06:13
*** claudiub has joined #openstack-nova06:15
*** ssurana has quit IRC06:16
*** thorst has quit IRC06:17
*** nkrinner_afk is now known as nkrinner06:17
*** artom has quit IRC06:17
*** thorst has joined #openstack-nova06:19
*** jdurgin has joined #openstack-nova06:19
*** artom has joined #openstack-nova06:19
*** mjura has joined #openstack-nova06:21
*** zz_dimtruck is now known as dimtruck06:22
*** jdurgin has quit IRC06:23
*** artom has quit IRC06:24
*** thorst has quit IRC06:24
*** thorst has joined #openstack-nova06:25
*** artom has joined #openstack-nova06:25
*** sahid has joined #openstack-nova06:28
*** raunak has joined #openstack-nova06:29
*** thorst has quit IRC06:30
*** andreas_s has joined #openstack-nova06:30
*** thorst has joined #openstack-nova06:32
*** dimtruck is now known as zz_dimtruck06:32
*** takedakn has joined #openstack-nova06:34
*** sdake has quit IRC06:36
*** thorst has quit IRC06:36
*** jdurgin has joined #openstack-nova06:40
*** pcaruana|afk| is now known as pcaruana06:40
*** gouthamr has quit IRC06:41
*** warp has joined #openstack-nova06:43
*** thorst has joined #openstack-nova06:44
*** jamielennox is now known as jamielennox|away06:45
*** jojden has joined #openstack-nova06:48
*** lpetrut has quit IRC06:49
jojdenhi06:49
jojdenhow to do host evacuation using nova clent06:50
jojdenself.nova_conn.servers.evacuate(server='c9d901fd-2d83-4e2d-be0b-8304eb9eeba9')06:50
jojdenI am getting06:50
jojdenInstance c9d901fd-2d83-4e2d-be0b-8304eb9eeba9 could not be found. (HTTP 404) (Request-ID: req-caa8e530-fb2e-4c0e-a5a5-b204a9745767)06:50
jojdenerror06:50
*** salv-orlando has quit IRC06:50
*** salv-orlando has joined #openstack-nova06:51
*** ducttape_ has joined #openstack-nova06:52
*** thorst has quit IRC06:52
*** raunak_ has joined #openstack-nova06:53
*** raunak has quit IRC06:53
*** raunak_ is now known as raunak06:53
*** markus_z has joined #openstack-nova06:53
*** sridharg has joined #openstack-nova06:54
*** thorst has joined #openstack-nova06:54
*** hoangcx has quit IRC06:55
*** ducttape_ has quit IRC06:56
*** hoangcx has joined #openstack-nova06:56
*** thorst has quit IRC06:59
*** thorst has joined #openstack-nova07:00
*** JoseMello has quit IRC07:01
*** liverpooler has joined #openstack-nova07:01
*** jdurgin1 has joined #openstack-nova07:01
*** jdurgin has quit IRC07:02
openstackgerritSlawek Kaplonski proposed openstack/nova: Add members in InstanceGroup object members field  https://review.openstack.org/37056707:03
*** thorst has quit IRC07:05
*** jdurgin1 has quit IRC07:06
*** jdurgin has joined #openstack-nova07:06
*** RuiChen has quit IRC07:08
*** code-R_ has quit IRC07:08
*** RuiChen has joined #openstack-nova07:10
*** zul has joined #openstack-nova07:10
*** thorst has joined #openstack-nova07:11
*** takedakn has quit IRC07:11
*** takedakn has joined #openstack-nova07:12
openstackgerritSlawek Kaplonski proposed openstack/nova: Clean up libvirt guest code  https://review.openstack.org/36575607:13
*** mingyu has joined #openstack-nova07:14
*** mingyu has quit IRC07:16
*** mingyu has joined #openstack-nova07:17
*** thorst has quit IRC07:17
*** ekuris has joined #openstack-nova07:17
*** takedakn has quit IRC07:17
*** artom has quit IRC07:17
*** thorst has joined #openstack-nova07:18
*** takedakn has joined #openstack-nova07:18
*** zenoway has joined #openstack-nova07:18
*** rama_y has quit IRC07:18
*** jed56 has joined #openstack-nova07:19
*** artom has joined #openstack-nova07:19
openstackgerritSlawek Kaplonski proposed openstack/nova: Add members in InstanceGroup object members field  https://review.openstack.org/37056707:20
*** thorst_ has joined #openstack-nova07:21
*** takedakn has quit IRC07:23
*** thorst has quit IRC07:23
*** matrohon has joined #openstack-nova07:24
*** takedakn has joined #openstack-nova07:25
*** thorst_ has quit IRC07:25
*** takedakn has quit IRC07:26
*** takedakn has joined #openstack-nova07:26
*** takedakn has quit IRC07:28
*** takedakn has joined #openstack-nova07:29
*** jlanoux has joined #openstack-nova07:29
*** takedakn has quit IRC07:31
*** takedakn has joined #openstack-nova07:34
*** artom has quit IRC07:35
*** takedakn has quit IRC07:35
*** raunak has quit IRC07:39
*** gongysh has quit IRC07:40
*** tobasco has quit IRC07:40
*** artom has joined #openstack-nova07:40
*** salv-orl_ has joined #openstack-nova07:40
*** markvoelker has joined #openstack-nova07:42
*** salv-orlando has quit IRC07:43
*** zenoway has quit IRC07:46
*** jlanoux_ has joined #openstack-nova07:46
*** jlanoux has quit IRC07:46
openstackgerritSlawek Kaplonski proposed openstack/nova: Add members in InstanceGroup object members field  https://review.openstack.org/37056707:48
*** markvoelker has quit IRC07:48
*** tobasco has joined #openstack-nova07:52
*** zenoway has joined #openstack-nova07:53
*** warp has quit IRC07:53
*** abhishekk has quit IRC07:55
*** hoangcx has quit IRC07:56
*** zul has quit IRC07:57
*** jamielennox|away is now known as jamielennox07:57
*** zzzeek has quit IRC08:00
*** bks has joined #openstack-nova08:00
*** zzzeek has joined #openstack-nova08:00
*** takashin has left #openstack-nova08:01
*** gjayavelu has quit IRC08:03
openstackgerritKrzysztof Świątek proposed openstack/nova: Fix issue with not removing rbd rescue disk  https://review.openstack.org/31492808:05
*** paul-carlton has quit IRC08:05
zigobauzas: Hi there!08:11
zigoAround?08:11
bauzaszigo: hey :)08:12
bauzaszigo: yup08:12
bauzaswazzup ?08:12
zigobauzas: Nova pretends (through its requirements.txt) that it is compatible with netaddr 0.7.12, though that's just not true.08:12
zigobauzas: Look at the build log: http://logs.openstack.org/79/369879/2/check/gate-deb-nova-dpkg-debian-jessie/10d7c56/console.html08:12
zigo(yes! Build log in OpenStack infra!!!)08:12
*** yamahata has joined #openstack-nova08:13
*** sree has joined #openstack-nova08:13
*** amotoki has joined #openstack-nova08:14
zigobauzas: Should I open a bug or what?08:14
zigoIn 0.7.12, there's no such thing as modified_eui64()08:15
zigoI greped in a netaddr installed package.08:15
zigo0.7.18 has it.08:15
*** yassine has joined #openstack-nova08:15
* bauzas looking08:15
*** lpetrut has joined #openstack-nova08:16
*** yassine is now known as Guest6716508:16
bauzaszigo: so, Nova asks for an upper or the same version https://github.com/openstack/nova/blob/master/requirements.txt#L2208:16
bauzaszigo: while https://github.com/openstack/requirements/blob/master/global-requirements.txt#L90 says we should just avoid 0.7.1608:17
zigobauzas: 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
zigoI 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
bauzaszigo: okay, lemme see what version uses the gate08:18
zigohttp://packages.ubuntu.com/search?keywords=python-netaddr08:19
zigoThe gate uses 0.7.1808:19
zigoThat's what is in upper-constraints.txt08:19
*** lpetrut has quit IRC08:20
bauzasyeah, that's what I can see08:20
bauzaszigo: okay, so the change was introduced recently08:22
bauzaszigo: given the gate was using a newer netaddr it never catched the problem08:23
*** zz_dimtruck is now known as dimtruck08:23
zigoYeah ...08:23
zigoWe don't check lower bounds! :)08:23
zigoIt was introduced at least before b2 was released.08:23
bauzaszigo: https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e208:23
zigoBecause b2 was failing, and I tried upgrading.08:23
bauzasthat's in b208:24
zigoYup.08:24
zigoIt matches.08:24
bauzaszigo: the real problem I see is that Nova uses the global requirements for knowing the lower bound08:25
zigoLike everyone.08:26
zigoIf global-requirements.txt is broken, then we should fix global-requirements.txt ! :)08:26
bauzasI tend to agree08:26
zigoEither that, or get rid of the modified_eui64() call.08:26
bauzaszactly08:26
zigoThe later is IMO more dangerous.08:26
bauzaszigo: file a bug report08:26
bauzastonyb: still around ?08:27
*** code-R has joined #openstack-nova08:27
* bauzas invoking the shaman dance to awake some Aussie folk08:27
bauzastonyb: I need your Requirements PTL hat :p08:27
*** GB21 has joined #openstack-nova08:28
*** code-R_ has joined #openstack-nova08:29
openstackgerritVasyl Saienko proposed openstack/nova: Testing DROP SERVICE_HOST patch  https://review.openstack.org/34356908:31
bauzaszigo: file a bug report and I'll triage it so we could discuss about the proper way to fix it08:31
bauzaszigo: either way, it's an unittest, right? if so, that's not impacting our users, rather our deployers that run tests against our code08:32
*** code-R has quit IRC08:32
bauzaszigo: 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_dimtruck08:33
zigoRight.08:33
zigobauzas: FYI, launchpad is broken, and I can't file the bug... :(08:33
zigoTimeout error08:33
zigoSorry, something just went wrong in Launchpad.08:33
tonybbauzas: 'sup?08:34
bauzastonyb: hola08:34
bauzas\o/08:34
*** hoangcx has joined #openstack-nova08:35
bauzastonyb: 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 version08:35
zigobauzas: tonyb: https://bugs.launchpad.net/nova/+bug/162383808:36
openstackLaunchpad bug 1623838 in OpenStack Compute (nova) "Nova requires netaddr >= 0.7.12 which is not enough" [Undecided,New]08:36
tonybzigo, bauzas: okay.  Open a bug against nova an requirements but it wont get fixed in Newton.08:36
bauzaszigo: https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e2 introduces modified_eui64() which is not available for netaddr-0.7.1208:36
zigobauzas: query ...08:36
tonybzigo, bauzas: we have it on the list of high priority things to fix in Ocata08:36
bauzastonyb: my question is, what would be the right approach ? bump the lower version in the global requirements ?08:37
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices  https://review.openstack.org/32376508:37
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: remove py26 compat code in "get_console_output"  https://review.openstack.org/33448008:37
zigotonyb: As a package maintainer, it is my view that it should be fixed in global-requirements.txt, even after the freeze...08:37
tonybbauzas: Yeah that's the correct fix08:37
zigo(or fixing nova if that's not too complicated / risky)08:37
tonybbauzas: 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 all08:37
bauzastonyb: yeah, we're branching tomorrow, I know :)08:38
tonybzigo: You can certainly advocate for that but in the past wev'r said we wont do that thing08:38
bauzastonyb: 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
tonybzigo: we've also stated that packagers shoudl whereever possible stick with the versions in u-c08:39
zigotonyb: Yeah, and I've been very vocal that this was a really stupid way to handle things.08:39
zigotonyb: 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
tonybzigo: sure I get that but the knock on effects are big08:40
zigotonyb: 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
tonybbauzas: we can only block things on stable branches not raise minimum requirements08:40
*** roeyc has joined #openstack-nova08:41
bauzastonyb: okay, I see08:41
tonybzigo: 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
bauzastonyb: zigo: either way, the bug is in the tests, there is no big deal I guess08:42
zigoNo big deal.08:42
bauzaswe could do some hack in Nova and backport that if that's a mess08:42
zigoIt just feels completely wrong that we can't fix a bug when we discover it.08:42
zigoAnyway, I got to go.08:43
zigoTTYL08:43
bauzaszigo: this is the price to pay for stability08:43
tonybzigo: sure but some times we need to be pragmatic.08:43
*** moshele has joined #openstack-nova08:44
*** derekh has joined #openstack-nova08:46
*** jfding has quit IRC08:48
bauzasfrickler: around ?08:48
bauzasfrickler: I'm looking at https://bugs.launchpad.net/nova/+bug/162383808:50
openstackLaunchpad 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 IRC08:50
fricklerbauzas: yep08:50
bauzasfrickler: and https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e2 introduced some netaddr API call that isn't supported by the lower bound we honot08:50
bauzashonor*08:50
fricklerbauzas: fyi, zigo's bug is useraffecting IMO, just commented on the bug08:50
*** danpb has joined #openstack-nova08:51
bauzasfrickler: oh f* you're right, that's a lib08:51
bauzasfrickler: so I just looked at https://pythonhosted.org/netaddr/_modules/netaddr/eui.html#EUI.modified_eui6408:52
bauzasfrickler: 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-nova08:54
fricklerbauzas: 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 issue08:54
*** ihrachys has joined #openstack-nova08:55
bauzasfrickler: I'll handle the bugfix08:55
*** hoangcx has joined #openstack-nova08:55
*** ratailor has quit IRC08:57
*** jdurgin has quit IRC08:57
*** tuanla_ has quit IRC08:57
jojdenany idea regarding nova evacuate ?08:58
johnthetubaguybauzas: I just saw that pop up, what you are thinking we could do?08:58
johnthetubaguyjojden: does this help: http://www.danplanet.com/blog/2016/03/03/evacuate-in-nova-one-command-to-confuse-us-all/08:58
bauzasjohnthetubaguy: I'm testing netaddr in my local venv to make sure I can fix that code section08:59
jojdenHi johnthetubaguy08:59
jojdeni done the cli mode08:59
jojdenbut when i try with novaclient08:59
jojdeni am getting error08:59
bauzasjohnthetubaguy: 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 RC108:59
openstackLaunchpad 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-nova08:59
jojdenself.nova_conn.servers.evacuate(server='6c3e0048-fd71-4f5a-bd70-43dd8b6cbf29')09:00
jojdenthis is the code I am using09:00
jojdenand 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
johnthetubaguybauzas: OK, that probably is our only option at this point09:00
jojdenbut I have that instance with id 6c3e0048-fd71-4f5a-bd70-43dd8b6cbf2909:00
johnthetubaguyjojden: I would check you are using credentials that have access09:01
*** efoley has joined #openstack-nova09:02
jojdenyes , I have, otherwise we will be getting error upon connection  right ?09:02
johnthetubaguyjojden: just see if you can get the instance show09:02
johnthetubaguyjojden: no, 404 could be an authorisation error, as its url related (it could also be loads of other things... sadly)09:03
jojdeni am getting hypervisor list using the same connection09:04
*** lpetrut has joined #openstack-nova09:07
kevinbentonjohnthetubaguy: may i have a brief moment of yours. I'm experiencing a crisis of sanity :)09:08
johnthetubaguykevinbenton: not sure I am expert fixing those, but I could try :)09:08
johnthetubaguyjojden: hmm, thats confusing09:08
jojdenok09:08
kevinbentonjohnthetubaguy: i take it at some point we have started using os-vif to plug interfaces?09:08
jojdeni will do more debugging09:09
johnthetubaguyjojden: I haven't got much experience debugging the client I am afraid, probably worth trying to get the dubug logs getting out of that09:09
johnthetubaguykevinbenton: yep09:09
jojdensure09:09
johnthetubaguykevinbenton: well, for a subset of vifs, just ovs and linux bridge I think09:09
jojdenthank you  johnthetubaguy09:09
*** tuanla_ has joined #openstack-nova09:09
johnthetubaguykevinbenton: danpb will be able to help with the details09:10
kevinbentonjohnthetubaguy: from what i can see, those plugins do not respect the MTU in the neutron network model at all09:10
kevinbentonjohnthetubaguy: e.g. https://github.com/openstack/os-vif/blob/stable/newton/vif_plug_ovs/ovs.py#L109-L11009:10
johnthetubaguykevinbenton: oh, that seems surprizing, did you have a dig in os vif about that already?09:10
johnthetubaguylooking09:10
johnthetubaguyoh, bummer09:10
johnthetubaguykevinbenton: I bet that code for os-vif branched before we starting consume the neutron MTU09:11
kevinbentonjohnthetubaguy: so this next question sucks, but how do we revert in the least damaging way?09:11
kevinbentonjohnthetubaguy: that it did09:11
kevinbentonjohnthetubaguy: we were fixing MTU after that09:11
kevinbentonjohnthetubaguy: we got some stuff into os-vif, but not this apparently09:11
johnthetubaguykevinbenton: thats a tricky question, as we are past the lib freeze of course09:11
*** tuanla has quit IRC09:11
danpbjust submit patches to os-vif and we can cut a new bug fix release with it in09:11
bauzasbut we can't bump the requirements in Newton09:12
kevinbentondanpb: from what i can see, it will require a model change in os-vif as well to actually store the MTU on the objects09:12
kevinbentonno MTU anywhere in here: http://git.openstack.org/cgit/openstack/os-vif/tree/os_vif/objects/vif.py09:12
danpbbauzas: IMHO the process shouldn't be so strict as to prevent us getting regression fixes released09:12
danpbbauzas: 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 fix09:13
danpbkevinbenton: yep, seems to be missing09:13
danpbanyway, no matter what we do, getting it into os-vif is the first step09:14
danpbonce we have that done, then we can look at how we get it included in nova for neutron09:14
kevinbentondanpb: you mean for Newton, right?09:14
danpbyes09:14
bauzasdanpb: sure, I understand that is an important regression09:14
danpbdamn, i'll be glad when we're off the newton release09:14
* johnthetubaguy is going to be glad when the newton/neutron thing goes away09:15
danpbsnap!09:15
johnthetubaguyheh09:15
kevinbentonjohnthetubaguy: definitely :)09:15
danpbnew rule for release naming: must not be easily confused with a project name09:15
kevinbentondanpb: ok. i'll start some patches09:15
bauzastonyb was around 30 mins earlier, we can hassle him for a bit again :p09:15
johnthetubaguydanpb: feels like we should do that proper fix even if we can't get it into newton, so that makes sense09:15
tonyb'sup?09:15
johnthetubaguynew os-vif release?09:15
bauzastonyb: yet another fun story09:15
tonybbauzas: Oh joy ;P09:16
bauzastonyb: tl;dr: os-vif doesn't take correct MTUs so there should be a new release09:16
danpbjohnthetubaguy: yeah, we'll need it for sure for ocata09:16
bauzasAFAIU, newton is regressing on that09:16
bauzastonyb: so the big question is, how can we ship that so close to our branching ?09:16
bauzasgiven the bugfix isn't yet implemented09:17
*** mingyu has quit IRC09:17
johnthetubaguywill be a raise the min-requirements thing09:17
*** mingyu has joined #openstack-nova09:17
tonybbauzas: It'd have to be a high impact bug09:17
tonybbauzas: unless we can blacklist the latest release of os-vif09:17
johnthetubaguymtu broken for all new vms using neutron?09:18
johnthetubaguysince the beginning of os-vif09:18
tonybjohnthetubaguy: *sigh* that's bad09:19
*** mvk has quit IRC09:20
tonybbauzas, danpb get the release made and we can probably do it as long as nova a neutron sign off on it09:20
bauzastonyb: yeah I told you it was a fun story09:20
tonybI think they're the only users of os-vif09:20
kevinbentontonyb: neutron doesn't even use os-vif yet09:20
kevinbentontonyb: so it's completely up to nova09:20
danpbjohnthetubaguy: kevinbenton  also this clearly illustrates a tempest testing gap09:20
danpbwe should have something written to validate MTU09:21
*** gabor_antal has joined #openstack-nova09:21
johnthetubaguydanpb: +109:21
bauzasdanpb: I had the same problem with ceph testing yesterday, and I tend to think we should have some in-tree devstack testing09:21
kevinbentondanpb: i think the issue is that the gate is too intelligent at setting that config value :)09:21
ihrachysdanpb: + on that, just discussed that yesterday with amuller...09:21
tonybdanpb: Yeah add a tempest test also (but that may fail on liberty and mistaka I guess) :(09:22
ihrachysdanpb: we need both tempest AND fullstack in neutron to validate the neutron connectivity sanity.09:22
bauzasmistaka ? :p09:22
bauzaslike, the one who did a fault ? :p09:22
*** hieulq has quit IRC09:22
tonybbauzas: ;p09:22
*** tiendc has quit IRC09:22
kevinbentontonyb: we actually fixed in mitaka and back-ported to liberty09:22
*** hieulq has joined #openstack-nova09:22
bauzastonyb: you're begged, it's 9pm for you, right?09:22
kevinbentontonyb: it was switch to os-vif that messed up09:23
bauzasoops09:23
*** tiendc has joined #openstack-nova09:23
bauzasyou're pardoned09:23
kevinbentontonyb: so a tempest test should pass on all 309:23
johnthetubaguykevinbenton: yeah, good point, that test should pass09:23
tonybkevinbenton: Oh I thought we had os-vif in mitaka ... Happy to be wrong09:23
kevinbentontonyb: for some stuff, not for the plugging though09:23
ihrachyskevinbenton: 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
openstackbug 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
johnthetubaguyit was more a twinkle in our eye in mitaka09:24
kevinbentonok, time to work on a fix. can someone point me to a guide on how to submit a patch to openstack?09:24
kevinbenton:D09:24
johnthetubaguyihrachys: kevinbenton: did you want to work out how to test / validate the MTU is correct?09:25
ihrachysin the meantime, I will search for a guide on review guidelines09:25
ihrachysjohnthetubaguy: 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 IRC09:26
johnthetubaguyoh, that sounds even better09:26
johnthetubaguydo you think the test could be in place in the next day or so?09:26
johnthetubaguyof is that make believe09:26
johnthetubaguyor09:26
ihrachysjohnthetubaguy: but we need tempest part too; fullstack is just to validate that neutron allows connectivity itself (for multiple config scenarios)09:26
johnthetubaguyah, no nova in the fullstack thingy09:27
kevinbentonjohnthetubaguy: we could conceivably request the gate set a smaller MTU for everything09:27
ihrachystiming 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-nova09:27
kevinbentonjohnthetubaguy: that alone should break things (SSH hangs)09:28
johnthetubaguyoh, miss configure it, but neutron network has the correct MTU?09:28
johnthetubaguyI like the sound of that09:28
kevinbentonjohnthetubaguy: configure it to 1400, so neutron is setting 1400 on everything09:28
kevinbentonjohnthetubaguy: then if nova doesn't set the right MTU, instances wouldn't be reachable09:29
kevinbentonjohnthetubaguy: or even something silly like 800 to really ensure it breaks09:29
danpbjohnthetubaguy: kevinbenton tonyb: you know what, we don't neccessarily need an os-vif release to fix this in nova for newton09:29
johnthetubaguykevinbenton: could you get a patch up for that, so it can depend on our fix?09:29
johnthetubaguydanpb: can we just set self.config.mtu?09:29
danpbwe 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 work09:29
johnthetubaguyoh... good idea09:30
ihrachysmakes sense09:30
*** jlanoux_ has quit IRC09:30
ihrachysyou control the code, so you should be able to consider the attribute 'kinda public' for a cycle09:30
*** mingyu has quit IRC09:30
danpbi'd still want to see us do the proper fix in os-vif first though09:30
*** jlanoux has joined #openstack-nova09:30
kevinbentonforcing self.config.mtu won't work for mixed encapsulation deployments09:31
*** mingyu has joined #openstack-nova09:31
ihrachyskevinbenton: oh self.config is global for all networks? bummer.09:31
kevinbentonunless you're changing the driver before every plug call09:31
ihrachysa driver per network? :)09:31
kevinbentondriver is stored globally in os-vif09:32
kevinbentonfrom what i can see09:32
danpbwhat is this self.config thing you're referring to ?09:32
johnthetubaguyright, I was assuming the later, although I prefer danpb's suggestion I think09:32
johnthetubaguyI am thinking the OvsPlugin.config.mtu09:32
danpboh, that's the global default MTU  in os-vif09:32
danpbthe mtu from the port binding should always override that, if set in the port binding09:33
*** jdurgin has quit IRC09:33
*** avolkov_ has joined #openstack-nova09:33
johnthetubaguyyou mean on the network? I guess thats the same thing09:33
johnthetubaguyit 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-L11009:34
johnthetubaguywe could be reading it all wrong09:34
*** phuongnh has quit IRC09:34
*** phuongnh has joined #openstack-nova09:34
ihrachysor, nova could leave os-vif do everything as-is, then after plug get back and fix the mtu09:34
*** avolkov_ has left #openstack-nova09:35
danpbihrachys: that's exactly what i meant09:35
danpbnova can just fix mtu after the fat09:35
danpbs/fat/fact/09:35
ihrachysok, misread, sorry.09:35
ihrachysI thought more like patching the lib to use the right value.09:36
danpbin 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-nova09:36
danpbihrachys: you can't patch the lib, because the data model passed into the lib doesn't record the mtu value09:36
cmd-tHI09:36
cmd-tI am using python-novaclient09:37
*** thorst has joined #openstack-nova09:38
cmd-tcan 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 coding09:39
cmd-tfor what parameter {key:value} i need to pass,09:39
*** sree has quit IRC09:41
johnthetubaguycmd-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#L37909:42
cmd-ti did, and i found this parameter.. scheduler_hints09:44
*** markvoelker has joined #openstack-nova09:44
cmd-tsi 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-tnut still whenever i am launching new vm its launching in same compute node09:45
cmd-tnut==but09:45
*** yamahata has quit IRC09:49
*** markvoelker has quit IRC09:49
*** mvk has joined #openstack-nova09:50
johnthetubaguycmd-t: is it not scheduler_hints={"group": "test_server_group"}09:50
*** sree has joined #openstack-nova09:51
johnthetubaguydanpb: ihrachys: kevinbenton: just checking, who is working on what bits?09:51
kevinbentoni'm working on os-vif09:51
cmd-tso i just need to pass name only..09:53
johnthetubaguycmd-t: I think thats all you need to pass on the CLI, but I could be remembering wrongly, what CLI cmd worked for you09:54
*** mingyu has quit IRC09:54
*** sree has quit IRC09:56
cmd-ti 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 vm09:56
johnthetubaguycmd-t: right, it will be the same with the client09:56
danpbjohnthetubaguy: i'm working on nothing :-)09:57
danpbbut i'll review what's produced09:57
*** mingyu has joined #openstack-nova09:57
*** lujinluo has quit IRC09:57
ihrachyskevinbenton: tempest09:57
johnthetubaguydanpb: I can take a look at that mtu hack09:57
ihrachysjohnthetubaguy: sorry, meant for you09:57
*** phuongnh has quit IRC09:58
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue  https://review.openstack.org/33588409:58
ihrachysdo we have a bug reported?09:58
johnthetubaguyihrachys: cool, is that the configuration hack kevinbenton mentioned?09:58
johnthetubaguyihrachys: oops, probably not, we should do that09:59
*** phuongnh has joined #openstack-nova09:59
ihrachysjohnthetubaguy: no, an actual scenario that will try to ping between nodes using MTU sized frames09:59
cmd-tjohnthetubaguy09:59
ihrachysjohnthetubaguy: size as returned by neutron mtu api extension09:59
johnthetubaguyOK, just wondering what would test this bug quickly09:59
cmd-tjohnthetubaguy: so how to use it from python program ....10:00
johnthetubaguybut if we can get the better think right of, thats even better10:00
*** salv-orl_ has quit IRC10:01
johnthetubaguycmd-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-libraries10:02
kevinbentonjohnthetubaguy: do you have a bug for this yet?10:03
ihrachysjohnthetubaguy: 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
johnthetubaguykevinbenton: nope, if you could create one, that would be great, you have better context for the bug10:03
johnthetubaguyihrachys: 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 IRC10:04
*** phuongnh has joined #openstack-nova10:05
ihrachysjohnthetubaguy: yeah, another way is downgrading network_device_mtu in nova to something below neutron network MTU (1400?)10:05
kevinbentonjohnthetubaguy: working on it10:05
ihrachysjohnthetubaguy: but the thing is, without a dedicated job you won't hit it in gate.10:05
kevinbentonihrachys: yeah, actually we could have the gate just set network_device_mtu to something tiny10:05
johnthetubaguyihrachys: I was just thinking we do a depends-on chain of patches, to prove the test works10:05
johnthetubaguyI mean prove the patch works10:06
ihrachysjohnthetubaguy: 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 coverage10:06
*** moshele has quit IRC10:06
ihrachystime for cross-project fullstack? :P10:06
*** moshele has joined #openstack-nova10:06
ihrachysjohnthetubaguy: btw, didn't nova remove the mtu setting very lately?10:06
ihrachysjohnthetubaguy: we could set throw-away scaffolding with devstack-gate patches and all10:07
*** efoley has quit IRC10:07
ihrachysjohnthetubaguy: I meant https://review.openstack.org/#/c/319543/10:07
johnthetubaguyihrachys: oops, you are right, but os-vif still has it I think10:07
*** mingyu has quit IRC10:08
kevinbentonjohnthetubaguy: https://bugs.launchpad.net/os-vif/+bug/162387610:09
openstackLaunchpad bug 1623876 in os-vif "nova is not setting the MTU provided by Neutron" [Undecided,New] - Assigned to Kevin Benton (kevinbenton)10:09
openstackgerritSylvain Bauza proposed openstack/nova: Remove use of modified_eui64()  https://review.openstack.org/37066510:09
*** efoley has joined #openstack-nova10:09
*** hoangcx has quit IRC10:09
*** roeyc has quit IRC10:09
bauzasfrickler: johnthetubaguy: zigo: ^10:09
ihrachysjohnthetubaguy: I see. would it make sense to deprecate it too?10:10
cmd-tjohnthetubaguy: still dont understand how to solve it10:10
*** gongysh has joined #openstack-nova10:10
openstackgerritKevin Benton proposed openstack/os-vif: Add MTU to Network model and use it in plugging  https://review.openstack.org/37066710:10
cmd-tjohnthetubaguy: can u provide some more help10:11
*** mingyu has joined #openstack-nova10:11
*** mingyu has quit IRC10:12
cmd-tjohnthetubaguy: 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-nova10:14
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue  https://review.openstack.org/33588410:15
johnthetubaguycmd-t: ah, good point, it should be the uuid of the server group10:15
lyarwoodmdbooth: ^ I've rebased my newton-rc-potential bugfix now that your test rewrite has landed10:16
lyarwoodmdbooth: any chance of a quick review this morning if you have time10:16
cmd-twell i tried that also10:17
cmd-tbut what could be the key for uuid, if it is in dict format10:17
*** mbirru has quit IRC10:19
johnthetubaguycmd-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-detail10:19
cmd-tjohnthetubaguy: it only takes **kwargs or from my understanding a dictionary only.... :(10:19
cmd-tjohnthetubaguy: thanks again but i have check all of these things....10:20
cmd-tand tried also10:20
cmd-tjohnthetubaguy: and as u asked yes i can get uuid of server_group10:21
*** mingyu has quit IRC10:21
johnthetubaguycmd-t: so the error you got says just supply a uuid instead of where you put the name10:21
*** akrivoka has joined #openstack-nova10:21
*** zz_dimtruck is now known as dimtruck10:24
*** mingyu has joined #openstack-nova10:25
akrivokaI'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-tjohnthetubaguy: u mean to say {'group':'uuid'}10:28
akrivokacould anyone tell me what I'm doing wrong in ^10:28
*** gszasz has joined #openstack-nova10:28
mdboothlyarwood: On a call, but I'll get on it straight after.10:32
openstackgerritKrzysztof Świątek proposed openstack/nova: Fix issue with not removing rbd rescue disk  https://review.openstack.org/31492810:32
*** dimtruck is now known as zz_dimtruck10:33
openstackgerritKevin Benton proposed openstack/nova: Pass MTU into os-vif Network object  https://review.openstack.org/37067910:35
kevinbentonjohnthetubaguy, danpb: ^^10:36
cmd-tjohnthetubaguy: it tried this  {'group':'uuid'} didn't work, should i pass some other key(instead of 'group'), what it would be10:36
johnthetubaguycmd-t: what error did you get with group and its uuid10:37
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Override MTU for os_vif attachments  https://review.openstack.org/37068110:39
*** moshele has quit IRC10:39
lyarwoodmdbooth: cool, thanks10:41
cmd-tjohnthetubaguy:  error Instance group aea4f782-0bf1-4852-92a0-efef20f23ac3 could not be found.10:41
johnthetubaguycmd-t: can you show that instance group in the API? looks like you are very close, it was expecting an instance group10:42
*** moshele has joined #openstack-nova10:42
cmd-tjohnthetubaguy: u mean output of nova server-group-list10:44
cmd-tas i have only one group:10:44
pkoniszewskihi 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
pkoniszewskii'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
johnthetubaguycmd-t: its possible the credentials are not quite right, but its hard to know10:45
cmd-tjohnthetubaguy: which credentials, because i am doing all my requests from admin user10:46
*** ratailor has joined #openstack-nova10:47
johnthetubaguypkoniszewski: yeah, probably worth adding a reno for that10:47
cmd-tjohnthetubaguy: i think its just a format of passing values that i am not able to understand10:47
johnthetubaguycmd-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 wong10:48
*** rtheis has joined #openstack-nova10:48
*** kylek3h has joined #openstack-nova10:48
*** kylek3h has quit IRC10:48
*** kylek3h has joined #openstack-nova10:48
openstackgerritMikhail Feoktistov proposed openstack/nova: Check hypervisor version while getting bus type  https://review.openstack.org/28239810: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.py10:49
cmd-tin this file i m getting errors.. because after that its openstack REST request10:50
*** sree has joined #openstack-nova10:50
*** hieulq has quit IRC10:51
*** hieulq has joined #openstack-nova10:51
johnthetubaguydanpb: 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-tjohnthetubaguy: my error file is ..../python2.7/site-packages/novaclient/v2/server_groups.py10:54
*** sree has quit IRC10:55
*** ftersin has joined #openstack-nova10:55
cmd-tTo be more specific, i am getting error in this code :::10:57
cmd-tjohnthetubaguy:10:57
cmd-tdef 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 IRC10:58
johnthetubaguycmd-t: so you have having issues creating the server group?10:58
cmd-tsorry nope10:59
johnthetubaguycmd-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-ti am able to create group but i don't know how to launch vm with a id of this group11:00
*** pilgrimstack has joined #openstack-nova11:00
*** sdague has joined #openstack-nova11:01
*** salv-orlando has joined #openstack-nova11:02
lyarwoodavolkov: 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
lyarwoodavolkov: happy to do this myself if you don't have time to work on this btw11:04
*** lucasagomes is now known as lucas-hungry11:06
*** dave-mccowan has joined #openstack-nova11:06
*** roeyc has joined #openstack-nova11:06
*** salv-orlando has quit IRC11:06
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Override MTU for os_vif attachments  https://review.openstack.org/37068111:06
*** bkopilov has quit IRC11:08
*** efoley_ has joined #openstack-nova11:10
openstackgerritJohn Garbutt proposed openstack/nova: Override MTU for os_vif attachments  https://review.openstack.org/37068111:12
*** efoley has quit IRC11:13
*** cmd-t has quit IRC11:14
*** jlanoux has quit IRC11:14
*** jlanoux has joined #openstack-nova11:14
*** jlanoux has quit IRC11:17
*** jlanoux has joined #openstack-nova11:17
*** sridharg has quit IRC11:19
*** sudipto_ has quit IRC11:20
*** sudipto has quit IRC11:20
*** bvanhav has joined #openstack-nova11:20
*** mingyu_ has joined #openstack-nova11:20
*** mingyu has quit IRC11:23
*** phuongnh has quit IRC11:23
*** jfding has joined #openstack-nova11:24
openstackgerritJohn Garbutt proposed openstack/nova: Override MTU for os_vif attachments  https://review.openstack.org/37068111:24
johnthetubaguykevinbenton: ihrachys: so this is the hack that might help us this side of the release, danpb had a quick look at that already11:25
*** roeyc has quit IRC11:26
*** thorst has quit IRC11:27
*** salv-orlando has joined #openstack-nova11:28
mdboothlyarwood: Jenkins was grumpy. Haven't looked at why yet, but reviewing anyway.11:28
mdboothlyarwood: unit test failures.11:28
*** roeyc has joined #openstack-nova11:29
*** fragatina has joined #openstack-nova11:30
* johnthetubaguy goes for some lunch11:33
*** fragatina has quit IRC11:34
kevinbentondanpb, 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-L14611:34
kevinbentonsorry, obj_make_compatible11:35
danpbkevinbenton: dunno - it is not unheard of for nova to make mistakes :-)11:36
ihrachyskevinbenton: as far as I understand the latest comment, it's because prev versions were not on the wire11:36
danpbmy understanding was that we need obj_make_compatible for every time we add a new attribute11:36
johnthetubaguyyeah, looks like it got missed11:36
ihrachysso it was easier for folks to just bump without backport rules11:36
ihrachysotherwise a bug11:36
kevinbentondanpb: ok. so i noticed os-vif doesn't have oslo.utils in it's requirements11:37
kevinbentondanpb: should i add it?11:37
*** sahid has quit IRC11:37
mdboothCould we extend the lifetimes of gerrit sessions?11:37
mdboothOr at least if you submit a comment after your session has expired it saves the bloody comment after login.11:37
danpbkevinbenton: presumably because we've never needed oslo.utils apis ?11:38
danpbor are we using them and getting the oslo.utils dep indirectly via something else we depend on11:38
openstackgerritMoshe Levi proposed openstack/nova: pci: remove pci device from claims and allocations when freeing it  https://review.openstack.org/36918011:38
kevinbentondanpb: ok. it's only used in the two drivers11:39
kevinbentondanpb: wasn't sure if that was why11:39
kevinbentondanpb: ok. i'll write the downgrade method11:39
*** edmondsw has joined #openstack-nova11:42
*** markvoelker has joined #openstack-nova11:45
*** zenoway has quit IRC11:48
*** rfolco has joined #openstack-nova11:49
*** baoli has joined #openstack-nova11:49
*** markvoelker has quit IRC11:50
openstackgerritKevin Benton proposed openstack/os-vif: Add MTU to Network model and use it in plugging  https://review.openstack.org/37066711:52
openstackgerritKevin Benton proposed openstack/nova: Pass MTU into os-vif Network object  https://review.openstack.org/37067911:53
*** sahid has joined #openstack-nova11:53
kevinbentondanpb: https://review.openstack.org/37066711:53
*** mingyu_ has quit IRC11:56
*** mingyu has joined #openstack-nova11:58
*** baoli_ has joined #openstack-nova11:58
*** sahid_ has joined #openstack-nova11:58
kevinbentonjohnthetubaguy: that _plug_os_vif is called for both linux bridge and OVS, right?11:58
*** baoli has quit IRC12:01
openstackgerritjichenjc proposed openstack/nova: Add hypervisor_links into hypervisor  https://review.openstack.org/37073212: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
openstackLaunchpad 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-nova12:04
*** tiendc has quit IRC12:04
gabor_antalWould someone take a look at this documentation bug? Am I true or missed something? https://bugs.launchpad.net/nova/+bug/118590212:04
openstackLaunchpad bug 1185902 in OpenStack Compute (nova) ""nova-manage flavor create" produces the wrong help" [Low,Won't fix]12:04
*** sahid has quit IRC12:05
gabor_antalSorry, wrong link: https://bugs.launchpad.net/nova/+bug/162390812:05
openstackLaunchpad 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
bauzaskevinbenton: 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 not12:06
openstackLaunchpad 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
johnthetubaguybauzas: I think thats accurate, I meant to update that12:07
johnthetubaguykevinbenton: yeah, your comment is spot on12:07
kevinbentonjohnthetubaguy: which one?12:07
johnthetubaguykevinbenton: I was meaning the one in the review, and _plug_os_vif being called for both cases12:08
*** namnh has quit IRC12:09
*** jlanoux has quit IRC12:09
openstackgerritjichenjc proposed openstack/nova: Trivial fix remove not used var in parameters  https://review.openstack.org/37074012:09
*** TravT has joined #openstack-nova12:09
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue  https://review.openstack.org/33588412:10
*** sree has joined #openstack-nova12:10
mdboothlyarwood: Hope you don't mind, but I updated your patch.12:11
*** gongysh has quit IRC12:12
*** xyang1 has joined #openstack-nova12:12
*** pradk has joined #openstack-nova12:13
gabor_antalslaweq: 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
openstackgerritSergey Nikitin proposed openstack/nova-specs: Enable to share PCI devices between numa nodes  https://review.openstack.org/36114012:13
*** ducttape_ has joined #openstack-nova12:14
avolkovlyarwood: hi, thanks for you comments, could you please clarify what you are expecting if volume reserving are failed?12:14
*** sree has quit IRC12:14
fricklergabor_antal: this looks like someone recycled a "nova flavor-create" command incorrectly, yes12:15
slaweqgabor_antal: I was checking it and it looks that when "num_instances" is calculated and quota reserved then server_group_members quota is not checked12:15
slaweqI will dig more into it12:15
*** claudiub has quit IRC12:16
*** lucas-hungry is now known as lucasagomes12:17
gabor_antalslaweq: 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_antalfrickler: thanks. I think I fix this12:18
avolkovlyarwood: 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
slaweqgabor_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 user12:21
slaweqfor 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 instances12:22
slaweqis my understanding correct?12:22
lyarwoodmdbooth: ah thanks, I removed **self.test_instance without running the damn tests again12:23
*** ratailor has quit IRC12:23
*** zz_dimtruck is now known as dimtruck12:24
lyarwoodavolkov: 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 tbh12:25
openstackgerritKrzysztof Świątek proposed openstack/nova: Fix issue with not removing rbd rescue disk  https://review.openstack.org/31492812:26
*** maticue has joined #openstack-nova12:27
kashyapmdbooth: 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.html12:28
openstackgerritjichenjc proposed openstack/nova: Add keypairs_links into resp  https://review.openstack.org/37076012:28
mdboothpaul-carlton2: What kashyap said ^^^12:28
*** mgoddard has joined #openstack-nova12:29
openstackgerritjichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33  https://review.openstack.org/37073212:30
*** mingyu has quit IRC12:30
avolkovlyarwood: 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-nova12:32
*** dimtruck is now known as zz_dimtruck12:34
mdboothkashyap: There are 5 additions of more than a KLOC in that series!12:34
mdboothThat's a big 'ol patch series.12:34
zigobauzas: U r0x! :)12:35
kashyapYeah, I'd imagine someone will ask them to split the series12:35
mdboothIt's already in 8 parts, may be ok12:35
mdboothI'm not entirely clear what it does, though12:35
mdboothWhy would libvirt want to manipulate individual files in a guest-managed filesystem?12:36
lyarwoodavolkov: 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 bdm12:36
kashyapmdbooth: Maybe you can already review there :-)12:36
lyarwoodavolkov: 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 up12:36
mdboothThat's more akin to what libguestfs does, rather than what libvirt does with storage pools.12:37
* mdbooth has probably misunderstood12:37
openstackgerritjichenjc proposed openstack/nova: fix broken link in api-ref  https://review.openstack.org/37077012:37
lyarwoodavolkov: 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_id12:37
*** sdake has joined #openstack-nova12:38
*** markvoelker has joined #openstack-nova12:38
*** tuanla_ has quit IRC12:38
*** bks has quit IRC12:39
avolkovlyarwood: yes, I see your point, will fix, sorry for delay12:40
openstackgerritjichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33  https://review.openstack.org/37073212:40
*** ducttape_ has quit IRC12:42
*** sahid_ has quit IRC12:42
*** hemanthm|away is now known as hemanthm12:44
*** ljxiash has joined #openstack-nova12:44
*** mriedem has joined #openstack-nova12:44
*** sahid has joined #openstack-nova12:45
*** burgerk has joined #openstack-nova12:45
openstackgerritjichenjc proposed openstack/nova: Add keypairs_links into resp  https://review.openstack.org/37076012:45
openstackgerritjichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33  https://review.openstack.org/37073212:46
lyarwoodavolkov: np at all and thanks :)12:47
sdaguealex_xu: https://review.openstack.org/#/c/338051/ missing response parameters12:47
*** vladikr has joined #openstack-nova12:49
*** mgoddard_ has joined #openstack-nova12:52
*** mgoddard has quit IRC12:52
*** ljxiash has quit IRC12:53
*** annegentle has joined #openstack-nova12:53
gabor_antalslaweq: Yes, you are right12:54
*** ekuris has quit IRC12:55
slaweqgabor_antal: thx for help :)12:55
gabor_antalnot at all, i think you're right :)12:55
*** burgerk has quit IRC12:56
markus_zmriedem: JFYI, I'll be on vacation for the next 2 weeks. Will be back on 10/0412:56
mriedem10-4 that good buddy12:57
*** mgoddard_ has quit IRC12:58
markus_zTIL: "10-4" :D12:59
*** hshiina has joined #openstack-nova13:00
*** ociuhandu has quit IRC13:01
*** akrivoka has left #openstack-nova13:01
*** jaosorior has quit IRC13:02
sdaguewe should definitely switch from irc to cb radio for all nova communication :)13:02
*** jaosorior has joined #openstack-nova13:02
*** burgerk has joined #openstack-nova13:03
mriedemi get to be burt reynolds if we do13:03
*** liverpooler has quit IRC13:03
gabor_antal:)13:03
markus_zmighty mustache is mandatory13:03
bauzasmriedem: aloha13:04
mriedemmahalo13:04
bauzasmriedem: seen the 2 critical blocking bugs ? :p13:05
mriedemnope13:05
bauzasI love mornings like this13:05
mriedemi'm off in about 10 minutes for the bus, just fyi13:05
bauzassure, no worries, Nova will burn in the meantime :p13:05
mriedemhttps://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+Committed13:05
bauzasyuuuup13:05
*** oanson has quit IRC13:06
sdaguebauzas: catching up, can you explain what the issue is here - https://bugs.launchpad.net/nova/+bug/1623876 ?13:06
openstackLaunchpad 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
mriedemit's not getting down through os-vif?13:06
markus_zbauzas: http://i0.kym-cdn.com/photos/images/newsfeed/000/962/640/658.png ;)13:06
bauzasone could be breaking people deploying Nova with Ipv6 and an old release13:06
mriedemclarkb: did i tip you off somehow with mtu things the other day?13:06
bauzassdague: the other means that we don't use the real MTU13:06
bauzassdague: which could be a problem for example if using bigger ones13:07
bauzaslike jumbo packets13:07
*** kylek3h has quit IRC13:07
bauzasoops jumbo frames I meant13:07
mriedemdanpb: you said in the bug above that we're working around this in nova?13:08
mriedembut i only see the changes from kevinbenton for os-vif and nova that depends on the os-vif change13:08
bauzasthere are 2 changes from kevinbenton13:08
bauzasyeah13:08
danpbmriedem: johnthetubaguy  submitted the nova hack13:09
danpbhttps://review.openstack.org/37068113:09
johnthetubaguyyeah, just trying to straighten that out13:09
openstackgerritJohn Garbutt proposed openstack/nova: Override MTU for os_vif attachments  https://review.openstack.org/37068113:10
*** akshai has joined #openstack-nova13:10
johnthetubaguykevinbenton: I think that what you meant for the ovs hybrid case ^13:10
johnthetubaguyihrachys: do we have that config hack to see if we can prove my hack would fix things?13:11
*** DaveJ__ has joined #openstack-nova13:11
sdaguebauzas: have we asking about just doing a req bump?13:11
ihrachysjohnthetubaguy: hang on, pushing the tempest test, then I will start scaffolding dependencies13:11
sdaguehttps://review.openstack.org/#/c/370665/1 seems kind of silly cruft13:11
*** annegentle has quit IRC13:11
openstackgerritChris Dent proposed openstack/nova: [placement] consolidate json handling in util module  https://review.openstack.org/36142213:11
mriedemsdague: the os-vif fix hasn't merged13:12
mriedemso not even close to released13:12
ihrachyskevinbenton: johnthetubaguy: the test at https://review.openstack.org/37080413:12
mriedemoh nvm13:12
openstackgerritRoman Podoliaka proposed openstack/nova: WIP: try a bigger TIMEOUT_SCALING_FACTOR value for migration tests  https://review.openstack.org/37080513:12
mriedemyou were talking about the netaddr one13:12
sdaguemriedem: yeh, the netaddr one13:12
sdagueseems like lots of folks on the other issue13:12
*** cdent has joined #openstack-nova13:13
bauzassdague: I discussed about that with tonyb13:13
sdagueand?13:13
bauzassdague: he's afraid of bumping the min ver so close to the newton branch13:13
sdaguewe haven't been testing with that minimum since forever13:14
bauzassdague: so I finally went to that approach of fixing the module itself13:14
*** sdake_ has joined #openstack-nova13:14
bauzassdague: I know13:14
sdaguebauzas: but we're not really going to be testing any of that code13:14
sdagueI really think we should just fix the fiction in requirements13: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
sdaguewe known minimums aren't tested right13:14
DaveJ__https://www.irccloud.com/pastebin/uMnhHtKA/13:14
sdaguebauzas: I also don't see this as critical13:14
*** salv-orlando has quit IRC13:15
*** DaveJ__ is now known as DaveJ13:15
mdboothDaveJ: Python3: nobody uses that :)13:15
* mdbooth gives it a go anyway13:15
*** sdake has quit IRC13:16
bauzassdague: I'm fine with either way, bumping or fixing in code13:16
mdboothDaveJ: Ah... F24 has python 3.5. That might be the issue13:16
bauzassdague: I just want to fix that for Newton13:16
openstackgerritKevin Benton proposed openstack/os-vif: Add MTU to Network model and use it in plugging  https://review.openstack.org/37066713:16
bauzassdague: because a nova list command can be impacted13:16
DaveJmdbooth: 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.713:16
johnthetubaguysdague: I was expecting the packagers to use the list of deps from the gate rather than worry about the minimums I guess13:17
kevinbentonjohnthetubaguy, danpb: i tested the nova change and os-vif change locally and it fixes the issue13:17
*** sudipto_ has joined #openstack-nova13:17
*** sudipto has joined #openstack-nova13:17
bauzasjohnthetubaguy: sure, but we say we accept that version, which isn't true :)13:17
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Use the recreated disk.config.rescue during a rescue  https://review.openstack.org/33588413:17
johnthetubaguykevinbenton: cool, do you want to check the hack? to see if thats an option13:17
DaveJmdbooth: I manually installed python3.4 but 3.5 is still installed - do you think it's interfering ?13:17
ihrachysjohnthetubaguy: https://review.openstack.org/#/c/370681/ that one right?13:17
lyarwood^ mdbooth, mriedem ; now with mdbooth's change for your reviewing pleasure13:17
kevinbentonjohnthetubaguy: sure13:18
sdaguejohnthetubaguy: yes, we tell them that13:18
bauzasjohnthetubaguy: sdagueokay, I'm writing a change against the global requirements to bump netaddr to at least 0.7.1313:18
mriedemlyarwood: thanks13:18
mdboothDaveJ: Just attempting to run py35 tests13:18
*** krtaylor has joined #openstack-nova13:18
*** rlrossit has joined #openstack-nova13:18
johnthetubaguyihrachys: yeah13:18
ihrachysjohnthetubaguy: https://review.openstack.org/#/q/topic:bug/1623876 see the DNM patches13:19
johnthetubaguykevinbenton: awesome, thanks, I kinda hope we don't want my match mind13:19
sdaguebauzas: 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 issue13:19
bauzassdague: okay, fair enough, I'll see whether we can merge the requirements change asap before cutting newton13:20
sdaguebauzas: sounds good13:20
mriedembauzas: sdague: i dropped the rc tag from https://bugs.launchpad.net/nova/+bug/162383813:21
openstackLaunchpad 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-nova13:21
kevinbentonjohnthetubaguy: yeah, your patch looks like it works13:21
kevinbentonjohnthetubaguy: i see veth pairs and bridge set to 1450 MTU13:21
kevinbentonjohnthetubaguy: (config is 1500)13:21
johnthetubaguykevinbenton: cool, I wasn't overly confident about the veth stuff13:21
*** mdrabe has joined #openstack-nova13:22
mdboothDaveJ: 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-nova13:23
mdboothDaveJ: If you run the 27 tests explicitly it should work fine.13:23
johnthetubaguymdbooth: right, we don't expect Nova to work on py3 at this point13:23
*** brian_price has quit IRC13:23
DaveJAh ok - so safe to ignore then.  So assuming jenkins doesn't run py3 tests on commit ?13:23
kevinbentonjohnthetubaguy: note that this is going back through nova rootwrap and not privsep, not sure if you've already restricted filters13:23
mdboothDaveJ: It does, but only the ones which aren't skipped.13:23
johnthetubaguyDaveJ: we have a test white list the tox target uses13:23
bauzasmriedem: 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 Nova13:24
mdboothWhat johnthetubaguy said13:24
johnthetubaguyactually its probably a black list, but you know what I mean13:24
bauzasmriedem: which is something we don't say atm13:24
DaveJYep got it.13:24
DaveJCheers13:24
mriedemkevinbenton: we haven't started dropping filters13:24
*** gouthamr has joined #openstack-nova13:24
mriedembauzas: you mean nova needs netaddr>=0.7.13?13:24
sdaguemriedem: yes13:24
mriedembut this isn't a new issue is it?13:25
sdaguemriedem: no13:25
mriedemis this a regression?13:25
sdagueoh, no, it is a new issue13:25
mriedemhm13:25
mriedemso we're doing the g-r bump?13:25
sdaguewe started using a function that's only 0.7.1313:25
bauzassdague: no, it's a regression13:25
mriedemnetaddr is used in a lot of projects13:25
sdaguemriedem: yes13:25
mriedemso there is going to be a big sync13:25
bauzassdague: introduced by https://github.com/openstack/nova/commit/4647f418afb9ced223c089f9d49cd686eccae9e213:25
bauzaswhich was merged by newton-213:25
sdaguemriedem: sure, but, it only needs to land in nova before nova rc13:26
bauzaswe were totally okay before that13:26
mriedemalternatively we revert ^13:26
sdagueif the other projects have a different value it's fine13:26
mriedemit doesn't fix a bug, it's a cleanup patch right?13:26
bauzasfrickler: ?13:26
sdaguemriedem: honestly, that's way riskier13:26
johnthetubaguyright, we have all been testing in the gate with the version that works13:26
sdaguewe *know* that req minimums aren't validated13:26
bauzasmriedem: it doesn't seem only cosmetic13:26
mriedemok13:27
* mriedem cowers under the backlash13:27
sdaguea bad req minimum should be considered a medium level issue to get fixed when we can13:27
* bauzas on it13:27
mriedemok, so 'fixed when we can' doesn't sound like an rc blocker to me13:27
sdaguewe tell everyone to use upper-constraints as the tested versions (which is 0.7.18)13:27
sdaguemriedem: agreed13:27
bauzassdague: mriedem: we can mitigate that by our releasenotes13:27
mriedembauzas: was just going to say that13:28
mriedemwith a known issue reno, something we rarely use :)13:28
sdagueso... I'm even torn on that13:28
*** edand has joined #openstack-nova13:28
bauzasnot a known issue13:28
sdaguebecause by saying "we know this min req is wrong"13:28
bauzasjust a general comment13:28
fricklerbauzas: I still think it is a bug fix, nova list --ip6 should be broken with netaddr 0.7.1213:28
sdaguewe kind of imply all the other min req are validated13:28
sdaguewhich, they aren't13:28
fricklerbauzas: if that is the question13:28
mriedemalright well i have to drop off and get my kid to school13:28
mriedemback in ~30 minutes13:29
*** kylek3h has joined #openstack-nova13:29
bauzas\o13:29
*** mriedem has quit IRC13:29
*** pwnall has joined #openstack-nova13:29
dhellmannsdague , bauzas : I agree that a release note is a reasonable way to address this, since we're so close to the final cut-off13:30
*** pwnall is now known as Guest5922813:30
bauzassdague: 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
sdaguedhellmann: 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 release13:31
bauzasAFAIR, there is some way to do that13:31
sdaguecodesearch.openstack.org13:31
bauzascool, thanks13:31
bauzassdague: that is a reasonable concern13:31
dhellmannsdague : 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 changes13:32
bauzasdhellmann: 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
bauzasbut I need to see whether other projects could be impacted as well13:33
bauzasbecause AFAIR, frickler who wrote the code thinks he saw that snippet in Neutron13:33
bauzasno brainer http://codesearch.openstack.org/?q=.modified_eui64%28%29&i=nope&files=&repos=13:34
sdaguedhellmann: it will propose some min bumps, but it doesn't change any testing13:34
*** tblakes has joined #openstack-nova13:34
sdagueand if only nova merges that, it's fine13:34
bauzasso, there is no other project impacted, let's move on and fill a relnote saying how bad it would be to deploy based on minimums13:34
dimsrpodolyaka is a beast! :) nice work on that opportunistic tests debugging!13:34
sdaguethe rest can merge in on stable/newton13:35
*** BjoernT has joined #openstack-nova13:36
*** eharney has joined #openstack-nova13:36
*** Guest59228 has quit IRC13:37
bauzassdague: dhellmann: FWIW https://review.openstack.org/37083313:39
bauzasand I'm writing a reno file about how wrong it could be to run nova with minimums given we don't test it13:40
*** ducttape_ has joined #openstack-nova13:42
sdaguebauzas: ok, well I think we should just merge the req bump, I tried to explain why with details in the review13:43
bauzassdague: good point13:43
*** akshai has quit IRC13:44
*** cleong has joined #openstack-nova13:44
*** TravT has quit IRC13:46
*** TravT has joined #openstack-nova13:47
*** ducttape_ has quit IRC13:47
*** TravT has quit IRC13:47
openstackgerritMaciej Szankin proposed openstack/nova: WIP config options: improve libvirt  https://review.openstack.org/37084313:47
*** akshai has joined #openstack-nova13:48
openstackgerritMaciej Szankin proposed openstack/nova: WIP config options: improve libvirt  https://review.openstack.org/37084313:48
*** EinstCrazy has joined #openstack-nova13:48
*** eharney_ has joined #openstack-nova13:49
*** bhagyashri has joined #openstack-nova13:49
*** eharney has quit IRC13:49
*** eharney_ is now known as eharney13:49
*** akshai has quit IRC13:50
openstackgerritSean Dague proposed openstack/nova: Update minimum requirement for netaddr  https://review.openstack.org/37084613:50
sdaguebauzas: that is the depends on change that would address it if we can get aggreement on merging the g-r change13:50
*** gongysh has joined #openstack-nova13:51
*** tonytan4ever has joined #openstack-nova13:51
bauzassdague: cool thank13:51
*** baoli has joined #openstack-nova13:54
*** akshai has joined #openstack-nova13:55
*** gongysh has quit IRC13:55
*** tonytan_brb has joined #openstack-nova13:56
*** baoli__ has joined #openstack-nova13:56
*** akshai has quit IRC13:57
*** baoli_ has quit IRC13:57
rpodolyakadims: heh, thanks!)13:57
*** akshai has joined #openstack-nova13:58
*** tonytan4ever has quit IRC13:58
openstackgerritRoman Podoliaka proposed openstack/nova: WIP: try a bigger TIMEOUT_SCALING_FACTOR value for migration tests  https://review.openstack.org/37080513:58
*** baoli has quit IRC13:59
*** akshai has quit IRC14:01
*** paul-carlton has joined #openstack-nova14:03
*** mriedem has joined #openstack-nova14:03
*** akshai has joined #openstack-nova14:03
mriedemare all things fixed now?14:03
sdaguerpodolyaka: can we just remove the WIP there14:04
rpodolyakasdague: sure!14:05
bauzasmriedem: https://review.openstack.org/370846 and https://review.openstack.org/370833 for the netaddr problem14:05
sdagueI'd rather just merge that change and see what the statistics look like once it's in14:05
*** ducttape_ has joined #openstack-nova14:05
bauzasmriedem: for os-vif, I haven't heard further progress14:05
sdaguerpodolyaka: thank you for diving into that one14:05
openstackgerritRoman Podoliaka proposed openstack/nova: Set a bigger TIMEOUT_SCALING_FACTOR value for migration tests  https://review.openstack.org/37080514:06
rpodolyakasdague: no problem!14:06
mriedembauzas: sdague: is there a reqs ffe for the netaddr change?14:06
sdaguemriedem: I am not fully sure the process there14:07
johnthetubaguybauzas: 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 IRC14:07
mriedemsdague: bauzas: ok i'll post the ffe in the ML14:07
sdagueit's a little weird, because it's litterally a zero risk change from our environment14:07
*** gouthamr has quit IRC14:07
sdaguethe effect in upstream ci is basically being a code comment change14:07
openstackgerritChristopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback  https://review.openstack.org/37086114:08
*** ducttape_ has quit IRC14:09
*** claudiub has joined #openstack-nova14:09
*** code-R has joined #openstack-nova14:09
mriedemi don't make the rules14:09
openstackgerritPawel Koniszewski proposed openstack/nova: Add reno for live migraton with cd-rom config drive  https://review.openstack.org/37086414:10
sdaguemriedem: that's fine14:11
*** code-R__ has joined #openstack-nova14:12
*** omnipresent has joined #openstack-nova14:12
*** zz_dimtruck is now known as dimtruck14:12
*** mlavalle has joined #openstack-nova14:13
*** code-R_ has quit IRC14:13
*** burgerk has quit IRC14:14
sdaguemriedem: I put the closes-bug on the nova change14:14
mriedemi know14:15
*** code-R has quit IRC14:15
mriedemi posted it for reqs reviewers14:15
mriedemsince they'll ask14:15
sdaguebecause the bug isn't actually closed until that merges14:15
sdagueah, ok14:15
mriedemi know14:15
mriedemi'm pretty sure doug had an email reminder about doing this14:15
sdaguesorry, I was reading your comment incorrectly14:15
mriedemso they don't have to ask14:15
mriedemjohnthetubaguy: i think water will kill unit test gremlins14:17
*** ducttape_ has joined #openstack-nova14:18
*** takedakn has joined #openstack-nova14:18
*** abalutoiu has joined #openstack-nova14:19
sdagueare there any final placement bits needed?14:19
sdagueor are we finally at the cut line there14:19
*** rlrossit has quit IRC14:19
mriedemthere is a bug fix14:19
mriedemjust watching https://review.openstack.org/#/c/369147/ through the gate14:20
markus_zmriedem: There are a few others which are stuck: https://review.openstack.org/#/q/project:openstack/nova++label:workflow%253D%252B1+status:pending14:21
*** moshele has quit IRC14:21
markus_zby "stuck" I mean the 3 oldest in that list14:22
*** ltomasbo has quit IRC14:22
dansmithmriedem: right, when that lands I think we're good14:22
*** ducttape_ has quit IRC14:23
*** marst has quit IRC14:26
bauzasdansmith: 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-nova14:26
bauzashttps://etherpad.openstack.org/p/placement-next seems to say yes14:26
dansmithbauzas: yep, that's what mriedem just said.. once that one is on (and the one it depends on) I'm good from the placement side14:26
dansmiths/on/in/14:26
*** jheroux has joined #openstack-nova14:27
*** danpb has quit IRC14:27
bauzaskewl14:27
*** danpb has joined #openstack-nova14:27
*** null0 has joined #openstack-nova14:28
*** yassine has joined #openstack-nova14:28
*** yassine is now known as Guest3167614:28
openstackgerritMaciej Szankin proposed openstack/nova: Improve consistency in serial_console  https://review.openstack.org/34890214:29
*** pwnall has joined #openstack-nova14:29
*** pwnall is now known as Guest8834014:29
*** paul-carlton has quit IRC14:29
*** Guest67165 has quit IRC14:29
*** null0 has quit IRC14:30
*** sree has joined #openstack-nova14:31
*** adisky has quit IRC14:32
*** edand has quit IRC14:33
*** sree has quit IRC14:35
*** marst has joined #openstack-nova14:36
openstackgerritAndrew Laski proposed openstack/nova: Add is_cell0 helper method  https://review.openstack.org/37089414:36
*** cdent has quit IRC14:37
mriedemsdague: fyi, the netaddr bump is being discussed in #openstack-requirements14:40
sdagueok, joining, it got lost in my auto join list14:40
mriedemhttp://eavesdrop.openstack.org/irclogs/%23openstack-requirements/%23openstack-requirements.2016-09-15.log.html#t2016-09-15T14:16:3414:40
bauzasmriedem: FWIW joining as well14:41
*** pradk has quit IRC14:42
openstackgerritHironori Shiina proposed openstack/nova: Separate ovs command to create vif port  https://review.openstack.org/31812814:43
*** rama_y has joined #openstack-nova14:43
*** rama_y has quit IRC14:44
*** rama_y has joined #openstack-nova14:44
*** EinstCrazy has quit IRC14:44
*** pradk has joined #openstack-nova14:45
*** jed56 has quit IRC14:45
*** ducttape_ has joined #openstack-nova14:47
zigobauzas: 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_45035414:48
zigoAny idea why?14:48
zigo(note: I prefer to discuss before badly filing bugs...)14:48
*** liverpooler has joined #openstack-nova14:49
*** akshai has quit IRC14:49
*** burgerk has joined #openstack-nova14:51
*** akshai has joined #openstack-nova14:51
*** akshai has quit IRC14:51
*** TravT has joined #openstack-nova14:52
openstackgerritJohn Garbutt proposed openstack/nova: Override MTU for os_vif attachments  https://review.openstack.org/37068114:53
*** akshai has joined #openstack-nova14:53
bauzaszigo: first time I see that error :/14:54
*** nkrinner is now known as nkrinner_afk14:54
*** baoli has joined #openstack-nova14:54
*** baoli__ has quit IRC14:55
*** ManishD has joined #openstack-nova14:55
openstackgerritPushkar Umaranikar proposed openstack/os-vif: os-vif pollute test output  https://review.openstack.org/37090814:56
*** rrecio_ has joined #openstack-nova14:56
*** tonytan_brb is now known as tonytan4ever14:57
ManishDDansmith: 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 shutdown14:58
ManishDDansmith: about this: https://review.openstack.org/#/c/35713114:58
dansmithManishD: I have to run off in a few minutes, but.. I could maybe get behind detach for shutdown, although I'd prefer not14:59
dansmithManishD: but detach for reboot is just asking for more trouble, IMHO14:59
dansmithand not for any good reason other than that it seems like minimal change to achieve the goal14:59
*** rrecio has quit IRC14:59
*** mjura has quit IRC14:59
ManishDdansmith: OK, Thanks, Please send your feedback whenever you get some time and I shall revert.15:01
dansmithManishD: thought I already did, but okay15:03
dansmithbiab15:03
*** unicell1 has quit IRC15:03
*** diogogmt has joined #openstack-nova15:03
*** jojden has quit IRC15:04
ManishDdansmith: yes got it but want to understand what the other way around to achive that.15:05
*** tri2sing has joined #openstack-nova15:05
ihrachysjohnthetubaguy: you are bumping my Depends-On from gate :))15:06
*** yamahata has quit IRC15:06
ihrachysjohnthetubaguy: 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.gz15:06
*** siva_krishnan has joined #openstack-nova15:06
*** jaypipes has joined #openstack-nova15:06
ihrachysjohnthetubaguy: so now we just need to show that it passes with your patch in the queue15:06
ihrachysjohnthetubaguy: I rechecked15:06
*** unicell has joined #openstack-nova15:07
johnthetubaguyah, sweet15:07
*** slaweq_ has quit IRC15:07
*** baoli_ has joined #openstack-nova15:07
*** slaweq_ has joined #openstack-nova15:08
johnthetubaguyihrachys: you going to depend-on my change as well, or something like that?15:08
*** mtanino has joined #openstack-nova15:08
ihrachysjohnthetubaguy: https://review.openstack.org/#/c/370813/ depends on your fix15:08
*** tblakes has quit IRC15:08
ihrachysjohnthetubaguy: it just gets bumped off gate every time you respin nova fix15:08
ihrachysjohnthetubaguy: so I struggle to get a result :)15:08
johnthetubaguyah, sorry, I get you now15:09
johnthetubaguyso, I have the unit tests passing problem, I believe15:09
johnthetubaguyI keep getting pulled away form my keyboard, but less of those problems right now15:09
*** baoli has quit IRC15:10
mriedemi'll throw https://review.openstack.org/#/c/370813/ into our etherpad to make sure we see it passing before approving the nova change15:10
*** takedakn has quit IRC15:10
*** takedakn has joined #openstack-nova15:11
*** zul has joined #openstack-nova15:12
*** slaweq_ has quit IRC15:12
*** amrith has joined #openstack-nova15:12
johnthetubaguymriedem: thanks15:12
mriedemihrachys: is there a specific job we need to check in https://review.openstack.org/#/c/370813/ ?15:12
openstackgerritianeta hutchinson proposed openstack/nova: Updates URL and removes trailing characters  https://review.openstack.org/36818515:13
mriedemihrachys: or would it just be any neutron job?15:13
*** hemnafk is now known as hemna15:13
siva_krishnanartom: can you review my patch whenever you find time https://review.openstack.org/#/c/369176/15:14
johnthetubaguymriedem: 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.gz15:14
*** amrith has left #openstack-nova15:15
*** gouthamr has joined #openstack-nova15:15
mriedemah yeah15:15
*** takedakn has quit IRC15:16
ihrachysmriedem: the failed patch https://review.openstack.org/#/c/370810/15:16
ihrachysmriedem: that one without the nova fix15:16
ihrachysso -full-15:16
*** rhagarty has joined #openstack-nova15:17
*** gouthamr has quit IRC15:17
*** mtanin___ has joined #openstack-nova15:17
*** gouthamr has joined #openstack-nova15:17
mriedemok cool15:17
mriedemihrachys: if you need to drop i can babysit those and recheck throughout my day15:18
*** kfarr has joined #openstack-nova15:18
artomsiva_krishnan, heya, yeah, it's on my list :)15:18
*** jaosorior has quit IRC15:18
ihrachysmriedem: it's nice to have some eyes, but I will be here the next 3h or so15:18
*** mtanino has quit IRC15:18
*** pcaruana has quit IRC15:19
artomsiva_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-nova15:19
*** andreas_s has quit IRC15:19
*** diogogmt has quit IRC15:20
*** salv-orlando has quit IRC15:22
siva_krishnanartom 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-nova15:22
*** zul has quit IRC15:22
artomsiva_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-nova15:26
*** ManishD has quit IRC15:26
*** Matti_ has joined #openstack-nova15:27
*** moshele has joined #openstack-nova15:27
*** Matti_ has quit IRC15:29
*** Matti_ has joined #openstack-nova15:29
*** sree has joined #openstack-nova15:29
siva_krishnanartom yeah it works15:30
*** takedakn has joined #openstack-nova15:30
artomsiva_krishnan, then you're good :) I'll look into the test in more detail and leave a comment soon-ish15:30
siva_krishnanartom thanks!15:31
*** mdrabe has quit IRC15:32
*** gjayavelu has joined #openstack-nova15:32
*** sree has quit IRC15:34
sdaguemriedem: is there a known ceph / volume / snapshot error?15:34
*** markus_z has quit IRC15:34
*** woodster_ has joined #openstack-nova15:36
*** mtanino has joined #openstack-nova15:36
sdaguehttp://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%2215:36
*** ducttape_ has quit IRC15:37
*** mtanin___ has quit IRC15:37
*** mdrabe has joined #openstack-nova15:38
*** tblakes has joined #openstack-nova15:39
mriedemis that a backup test?15:39
sdagueit's the bfv scenario pretty often15:40
sdaguebut that does a lot of hoops15:40
bauzasdanpb: others, that looks like a feature request, right ? https://bugs.launchpad.net/nova/+bug/162387115:41
openstackLaunchpad bug 1623871 in OpenStack Compute (nova) "Nova hugepage support does not include aarch64" [Undecided,In progress] - Assigned to Veena (mveenasl)15:41
sdaguethat scenario includes doing a volume snapshot, then booting from that15:41
sdaguebauzas: if it's litterally just adding an arch name to a string list, it feels like you could call it a bug15:42
sdagueif it's more complicated than that, it's probably a feature15:42
bauzassdague: well, I'm concerned by the sentence "Both for NUMA and Hugepage support, AARCH64 needs to be added. "15:43
bauzasthat looks more than a conditional15:43
sdaguewell, iirc the ppc64 numa add was litterally just extending the list of arches15:43
bauzasvladikr: around ?15:43
vladikrbauzas, sure15:43
mriedemsdague: my guess would be http://status.openstack.org//elastic-recheck/index.html#147533315:43
bauzasvladikr: was triaging and saw https://bugs.launchpad.net/nova/+bug/162387115:44
openstackLaunchpad bug 1623871 in OpenStack Compute (nova) "Nova hugepage support does not include aarch64" [Undecided,In progress] - Assigned to Veena (mveenasl)15:44
sdaguebauzas: 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 anyway15:44
bauzasvladikr: does it sound just a miss that we made when writing the features, or does that require more than that?15:44
bauzassdague: I was just about to say that :p15:44
danpbbauzas: yep, its absolutely a feature reqeust15:44
bauzaslike, it shouldn't take more than 5 mins of my time :p15:44
vladikryea15:44
bauzasdanpb: okay, I'll point out the right process scenario, thnaks15:45
danpbsdague: bauzas at the very least it will need someone to figure out min required versions of libvirt & qemu for huge pages15:45
bauzasdanpb: oooh15:45
danpbsdague: pp64 actually caused bugs because it turned out to really need more than just extending the arch list15:45
sdaguedanpb: ah, ok15:45
danpbbecause PPC can have sparse numa topologies and we didn;'t handle that right15:45
danpbeg, node 0 and node 25, with no other nodes in between15:46
*** gjayavelu has quit IRC15:46
danpbi dunno if aarch64 will have similar surprises15:46
*** cdent has joined #openstack-nova15:46
*** gjayavelu has joined #openstack-nova15:47
*** salv-orlando has joined #openstack-nova15:49
*** ducttape_ has joined #openstack-nova15:49
*** priteau has joined #openstack-nova15:49
mriedemdanpb: are you aware of https://github.com/openstack/nova/commit/abc24acfa1982a0ffccbe08a006ac7c7a9f4ecda ?15:49
mriedemthat landed this week15:49
danpbmriedem: no, hadn't see it15:51
*** jgriffith has quit IRC15:52
danpbi 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 added15:53
*** paul-carlton has joined #openstack-nova15:53
*** moshele has quit IRC15:55
*** brian_price has joined #openstack-nova15:55
mriedemlyarwood: question in https://review.openstack.org/#/c/335884/15:57
mriedemlyarwood: that still looks to me like a latent bug fix, not a regression fix15:58
*** gjayavelu has quit IRC15:58
*** jkraj has quit IRC15:58
lyarwoodmriedem: that's correct, I don't think I've called it out as a regression anywhere15:59
mriedemdoh15:59
*** ltomasbo has joined #openstack-nova16:00
mriedemthe bug was marked newton-rc-potential16:00
*** thumpba has joined #openstack-nova16:00
lyarwoodah right, and we only accept regressions at this point?16:00
mriedemand b/c of the ref to artom's change i thought it was supposed to be a regression fix16:00
*** ltomasbo has quit IRC16:00
mriedemwell, we just scrutinize bugs more16:00
*** ltomasbo has joined #openstack-nova16:00
*** cdent has quit IRC16:00
*** takedakn has quit IRC16:03
*** hshiina has quit IRC16:05
*** rlrossit has joined #openstack-nova16:07
*** rlrossit has quit IRC16:08
*** mdrabe has quit IRC16:08
*** mdrabe has joined #openstack-nova16:08
*** Apoorva has joined #openstack-nova16:09
*** jgriffith has joined #openstack-nova16:11
*** yamahata has joined #openstack-nova16:15
jaypipesmorning all16:17
mriedemjaypipes: o/16:18
mriedemihrachys: passed http://logs.openstack.org/13/370813/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ddf5f49/16:19
johnthetubaguymriedem: 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 soon16:19
openstackLaunchpad 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 IRC16:19
mriedemjohnthetubaguy: that bug is over a year old16:20
*** lucasagomes is now known as lucas-dinner16:24
mriedembauzas: do it https://review.openstack.org/#/c/370846/116:24
mriedemthe release team is moving forward with the bump16:24
mriedemthe depends-on blocks it anyway16:24
*** raunak has joined #openstack-nova16:24
*** ssurana has quit IRC16:25
*** nic has joined #openstack-nova16:25
*** rcernin has quit IRC16:26
*** bkopilov has quit IRC16:30
openstackgerritAndrey Volkov proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937916:31
*** oomichi_ has joined #openstack-nova16:32
*** TravT has quit IRC16:32
*** oomichi_ has quit IRC16:32
*** TravT has joined #openstack-nova16:32
*** ltomasbo has quit IRC16:32
*** oomichi_ has joined #openstack-nova16:32
*** ltomasbo has joined #openstack-nova16:32
*** baoli_ has quit IRC16:34
*** cdent has joined #openstack-nova16:35
*** moshele has joined #openstack-nova16:36
*** paul-carlton has quit IRC16:38
*** gouthamr has quit IRC16:40
*** tonytan4ever has quit IRC16:42
*** gouthamr has joined #openstack-nova16:42
*** priteau has quit IRC16:44
*** rlrossit has joined #openstack-nova16:46
*** thumpba has quit IRC16:48
*** thumpba has joined #openstack-nova16:49
*** derekh has quit IRC16:49
*** browne has joined #openstack-nova16:49
*** rlrossit has quit IRC16:49
*** lpetrut has quit IRC16:50
*** annegentle has quit IRC16:52
*** rlrossit has joined #openstack-nova16:53
*** thumpba has quit IRC16:54
*** aunnam_ has joined #openstack-nova16:54
*** sudipto has quit IRC16:55
*** sudipto_ has quit IRC16:55
openstackgerritChristopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback  https://review.openstack.org/37086116:55
*** cdent has quit IRC16:55
*** TravT has quit IRC16:57
*** yamahata has quit IRC16:58
*** yamahata has joined #openstack-nova16:58
*** sree has joined #openstack-nova16:58
*** fragatina has joined #openstack-nova16:59
*** paul-carlton has joined #openstack-nova17:00
*** _ducttape_ has joined #openstack-nova17:00
*** fragatina has quit IRC17:00
*** moshele has quit IRC17:00
*** code-R__ has quit IRC17:01
*** fragatina has joined #openstack-nova17:01
*** TravT has joined #openstack-nova17:01
*** rlrossit has quit IRC17:01
*** unicell has quit IRC17:02
*** sree has quit IRC17:03
*** rfolco has quit IRC17:03
*** ducttape_ has quit IRC17:03
*** itlinux has joined #openstack-nova17:03
*** rfolco has joined #openstack-nova17:04
*** Fdaisuke has quit IRC17:04
johnthetubaguymriedem: sorry, more context, no where near release ciritical, just a heads up around the config difference we were bumping into a few weeks back17:04
*** sahid has quit IRC17:04
*** efoley_ has quit IRC17:13
*** thumpba has joined #openstack-nova17:17
* dansmith stares at the gate helplessly17:17
*** amrith has joined #openstack-nova17:18
*** gszasz has quit IRC17:18
siva_krishnanjohnthetubaguy:  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
openstackLaunchpad 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 IRC17:18
*** tblakes_ has joined #openstack-nova17:22
*** hemanthm is now known as hemanthm|lunch17:22
*** _ducttape_ has quit IRC17:22
*** tblakes has quit IRC17:23
*** tblakes_ is now known as tblakes17:23
*** ducttape_ has joined #openstack-nova17:23
*** mbirru has joined #openstack-nova17:25
*** abalutoiu has quit IRC17:29
*** danpb has quit IRC17:31
*** mvk has quit IRC17:32
*** ducttape_ has quit IRC17:34
*** rlrossit has joined #openstack-nova17:34
*** hemna_ has joined #openstack-nova17:35
mriedemjohnthetubaguy: ack17:36
*** lpetrut has joined #openstack-nova17:36
*** fragatina has quit IRC17:36
*** tonytan4ever has joined #openstack-nova17:37
mriedemdansmith: we can look at this https://review.openstack.org/#/c/370681/17:38
*** ssurana has joined #openstack-nova17:38
*** gjayavelu has joined #openstack-nova17:39
*** smatzek has joined #openstack-nova17:39
*** GB21 has quit IRC17:41
*** lpetrut has quit IRC17:41
*** hemanthm|lunch is now known as hemanthm17:42
*** itlinux has quit IRC17:43
*** ihrachys has quit IRC17:43
*** ltomasbo has quit IRC17:45
*** omnipresent has quit IRC17:45
*** andrewbogott has quit IRC17:47
*** andrewbogott has joined #openstack-nova17:47
*** hemna_ has quit IRC17:49
*** unicell has joined #openstack-nova17:52
*** itlinux has joined #openstack-nova17:53
*** GB21 has joined #openstack-nova17:54
*** moshele has joined #openstack-nova17:54
*** aunnam_ has quit IRC17:55
*** ducttape_ has joined #openstack-nova17:55
*** sudipto has joined #openstack-nova18:00
*** sudipto_ has joined #openstack-nova18:00
*** tblakes has quit IRC18:04
*** fragatina has joined #openstack-nova18:04
*** TravT has quit IRC18:06
*** sudipto has quit IRC18:07
*** sudipto_ has quit IRC18:07
*** mvk has joined #openstack-nova18:12
*** tblakes has joined #openstack-nova18:12
mriedemjaypipes: need you to look at this https://review.openstack.org/#/c/370681/18:13
mriedemwish danpb would have18:13
*** yamahata has quit IRC18:13
mriedemjohnthetubaguy: i've posted quetsions in ^ but i'm in over my head there18:13
*** liverpooler has quit IRC18:14
*** lpetrut has joined #openstack-nova18:15
*** psachin has quit IRC18:20
*** rodrigods has quit IRC18:22
*** rodrigods has joined #openstack-nova18:22
*** pcaruana has joined #openstack-nova18:22
*** annegentle has joined #openstack-nova18:26
*** sdake_ is now known as sdake18:26
*** paul-carlton has quit IRC18:31
*** kfarr has quit IRC18:31
*** amrith has left #openstack-nova18:32
openstackgerritArtom Lifshitz proposed openstack/nova: Send vif-plugged to dest host if evacuating  https://review.openstack.org/37104818:36
*** moshele has quit IRC18:38
*** tongli has joined #openstack-nova18:39
jaypipesmriedem: on it.18:40
mriedemgah we need to update this to test linuxbridge on libvirt vif changes18:41
mriedemhttps://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L205518:41
*** GB21 has quit IRC18:41
*** fragatina has quit IRC18:43
*** fragatina has joined #openstack-nova18:43
*** yamahata has joined #openstack-nova18:45
openstackgerritChristopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback  https://review.openstack.org/37086118:46
*** baoli has joined #openstack-nova18:52
*** code-R has joined #openstack-nova18:55
*** tgtanya has joined #openstack-nova18:56
*** code-R_ has joined #openstack-nova18:59
*** code-R has quit IRC19:02
*** aunnam_ has joined #openstack-nova19:02
*** ijw has joined #openstack-nova19:03
openstackgerritAnusha Unnam proposed openstack/nova: Remove deprecated compute_available_monitors option  https://review.openstack.org/37106219:03
*** kfarr has joined #openstack-nova19:04
*** tgtanya_ has joined #openstack-nova19:04
*** rhagarty has quit IRC19:05
*** tgtanya has quit IRC19:06
*** krtaylor has quit IRC19:07
*** matrohon has joined #openstack-nova19:07
*** salv-orlando has quit IRC19:07
*** akshai has quit IRC19:09
dansmithoh hell yes19:09
dansmithmriedem: bauzas: those patches merged19:09
mriedemcool19:10
mriedemdansmith: can you check my regex here? https://review.openstack.org/37106519:10
*** akshai has joined #openstack-nova19:11
dansmithmriedem: um, that doesn't look right to me19:11
dansmithor, maybe (?..) makes the whole group optional?19:12
mriedemi was just testing with samples on regexpal.com19:12
mriedemit should mean, skip that job if the file is not in those19:12
mriedemso nova/compute/manager.py would not be tested19:12
mriedemnova/network/os_vif_util.py or nova/virt/libvirt/vif.py would19:13
dansmithI dunno what (? means.. normally I would do like (..){0,1} or something19:14
dansmithto make a group optional19:14
dansmithoh, lookahead19:15
dansmithI bet some former perl refugee could validate that19:15
mriedemauggy: !19:15
auggy:)19:16
mriedemskipUnless(nova/network/* OR nova/virt/libvirt/vif.py)19:16
dansmithmriedem: but yeah, testing seems to do the right thing19:16
mriedemskipIf(!(nova/network/* OR nova/virt/libvirt/vif.py))19:16
dansmithmriedem: so I guess it's right :)19:16
mriedem\o/19:16
mriedemauggy: need your regex skillz on https://review.openstack.org/#/c/371065/1/zuul/layout.yaml19:17
*** fragatina has quit IRC19:17
auggylemme take a look19:17
*** twinklechawla has joined #openstack-nova19:17
openstackgerritjichenjc proposed openstack/nova: Add keypairs_links into resp  https://review.openstack.org/37076019:17
*** fragatina has joined #openstack-nova19:18
auggydansmith: sorry trying to get the phone, this guy would not shut up19:18
auggyprotip if your employer ever does 401k stuff through bancorp, just run away19:19
*** fragatina has quit IRC19:19
auggycatching up on the conversation19:19
*** fragatina has joined #openstack-nova19:19
auggyi usually use ?() to make a group that matches but that i don't want to capture19:20
*** twinklechawla has left #openstack-nova19:21
dansmith?() isn't a thing right?19:21
dansmithit would be ()?19:21
auggymriedem: do you want it to match nova/network/.* OR nova/virt/libvirt/vif.py ?19:21
auggyit's ?: i think19:22
dansmith? matches the thing before it, so ?() would be an error I think19:22
auggyi usually do ?:()19:22
auggyi'm looking to double check hehe19:23
dansmithyeah, doesn't work for me19:23
dansmithanyway, unrelated to the thing at hand :)19:23
auggyhaha19:24
mriedemanyway, passes for me on http://www.regexpal.com/19:24
mriedemso i'm just going to leave it19:24
auggy(?: foobar) means match but don't capture that19:24
dansmithokay (?: ) != ?() :)19:24
auggyhaha yeah19:24
auggysorry19:25
auggyok 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-nova19:26
openstackgerritjichenjc proposed openstack/nova: Add hypervisor_links into hypervisor v2.33  https://review.openstack.org/37073219:26
auggywhat would this match? nova/virt/libvirt/vif.py.*$19:26
melwittmriedem: doesn't 'all-files-match-any' probably mean it's already doing an OR of all that's there?19:28
auggyoh i see you want it to not match those files19:29
* auggy is a little slow to the party19:29
*** aswadr_ has quit IRC19:32
*** fragatina has quit IRC19:32
*** pcaruana has quit IRC19:32
*** bvanhav has quit IRC19:32
*** davidlenwell has quit IRC19:34
*** rama_y has quit IRC19:34
auggyi 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 string19:35
*** tongli has quit IRC19:38
*** fragatina has joined #openstack-nova19:38
*** itlinux has quit IRC19:39
*** rama_y has joined #openstack-nova19:39
*** fragatin_ has joined #openstack-nova19:40
*** fragatina has quit IRC19:40
*** david-lyle has quit IRC19:40
*** david-lyle has joined #openstack-nova19:40
*** Apoorva has quit IRC19:40
*** aunnam_ has quit IRC19:42
*** fragatin_ has quit IRC19:44
*** code-R_ has quit IRC19:44
*** davidlenwell has joined #openstack-nova19:47
*** itlinux has joined #openstack-nova19:48
*** sdake has quit IRC19:49
*** sdake_ has joined #openstack-nova19:49
*** rcernin has joined #openstack-nova19:53
*** baoli has quit IRC19:56
*** baoli has joined #openstack-nova19:56
*** krtaylor has joined #openstack-nova19:56
*** ametts has quit IRC20:01
*** fragatina has joined #openstack-nova20:01
*** ducttape_ has quit IRC20:03
auggymriedem: ok that looks legit, i tinkered around with it too just to make sure the lookaheads worked consecutively and it looks like they do20:04
*** jvgrant_ has joined #openstack-nova20:04
*** fragatina has quit IRC20:05
mriedemcool20:05
auggyregexes for me are like spanish... i know it pretty well but when i don't practice it every day i forget things ahaha20:06
*** Apoorva has joined #openstack-nova20:06
*** Apoorva_ has joined #openstack-nova20:07
*** jvgrant has quit IRC20:08
*** fragatina has joined #openstack-nova20:10
*** Apoorva has quit IRC20:10
*** ijw has quit IRC20:11
*** ijw has joined #openstack-nova20:11
*** itlinux has quit IRC20:13
*** ametts has joined #openstack-nova20:13
*** fragatina has quit IRC20:15
*** pwnall has joined #openstack-nova20:15
*** pwnall is now known as Guest1303620:15
*** ijw has quit IRC20:16
*** Guest88340 has quit IRC20:17
mriedemnetaddr fix is merrged20:17
*** hamzy has quit IRC20:20
*** rhagarty has joined #openstack-nova20:22
*** Guest13036 has quit IRC20:23
bauzas\o/20:25
*** tbachman has quit IRC20:25
* bauzas likes coming back to the chan and reading that20:25
*** ducttape_ has joined #openstack-nova20:27
mriedemjaypipes: did you have a chance to look at https://review.openstack.org/#/c/370681/?20:27
*** fragatina has joined #openstack-nova20:29
mriedem^ is passing ovs and linuxbridge jobs with ihar's new test..20:29
mriedemso maybe ok20:30
sdaguemriedem: the req bump merged20:31
bauzasWAAAAT? we now have quotas for server groups ?20:31
* bauzas facepalms20:31
melwittbauzas: 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 at20:32
*** kfarr has quit IRC20:33
*** ijw has joined #openstack-nova20:33
melwittit 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 master20:33
bauzasmelwitt: yeah I don't think so20:33
mriedemsdague: yup20:33
mriedemsdague: markus_z and i broke puppet ci though :)20:33
mriedem\o/20:34
mriedemsdague: maybe you want to look at https://review.openstack.org/#/c/370681/ ?20:34
mriedemmelwitt: bauzas: ^20:34
mriedemreally all cores20:34
mriedemthat's the last thing for rc120:34
bauzasmriedem: I was about to ask you about rc120:34
*** zzzeek has quit IRC20:35
mriedem^ is our last change20:35
bauzascool20:36
mriedemit's passing ovs and LB jobs with that change20:36
mriedemi have several comments/questions in there,20:36
mriedemlike, i think we might be passing by luck but i'm not sure20:36
mriedemand/or, we're passing b/c we only use os-vif for ovs/lbr but not other things like vhostuser yet20:36
jaypipesmriedem: done20:36
*** hemanthm is now known as hemanthm|away20:37
*** Apoorva_ has quit IRC20:37
*** ijw has quit IRC20:38
bauzasmriedem: 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-nova20:38
bauzasI mean other vif types tbc20:38
mriedembauzas: other vif types won't go down this route20:38
mriedemthey will go down the legacy path before tihs20:38
mriedem*this20:38
bauzasokay20:38
mriedemjaypipes: thanks20:38
mriedemneed to check intel ci reuslts20:39
mriedem*results20:39
bauzasmriedem: oh right,L81520:39
*** zzzeek has joined #openstack-nova20:39
openstackgerritSean Dague proposed openstack/nova: WIP: try utf8 force  https://review.openstack.org/37114720:40
mriedemhmm, neutron did not start20:40
mriedemhttp://intel-openstack-ci-logs.ovh/81/370681/7/check/tempest-dsvm-intel-nfv-xenial/e20e1c4/logs/devstacklog.txt.gz20:40
*** baoli has quit IRC20:41
*** baoli has joined #openstack-nova20:41
*** rkmrhj has joined #openstack-nova20:42
*** ametts has quit IRC20:42
mriedemjaypipes: any idea why he had to update all of those vhostuser unit tests to pass None for mtu?20:45
mriedemi think i see why20:51
mriedemself.vif_vhostuser_ovs* uses network_bridge which he changed to have mtu=none20:51
mriedemso those no longer pass mtu for those vhostuser fake vifs20:51
mriedemhmm, should have probably used a different/new network_model for that20:51
*** tgtanya_ has quit IRC20:52
*** baoli has quit IRC20:53
*** ijw has joined #openstack-nova20:53
*** akshai has quit IRC20:54
*** gouthamr has quit IRC20:54
*** baoli has joined #openstack-nova20:55
*** nherciu has quit IRC20:57
*** tgtanya has joined #openstack-nova20:57
*** takashin has joined #openstack-nova20:57
*** ijw has quit IRC20:58
*** thumpba has quit IRC20:59
*** thumpba has joined #openstack-nova20:59
mriedemhells bells nova meeting21:00
*** rfolco has quit IRC21:02
*** tgtanya has quit IRC21:02
openstackgerritMatt Riedemann proposed openstack/nova: Override MTU for os_vif attachments  https://review.openstack.org/37068121:04
mriedemjaypipes: ^ updated to cleanup the tests a bit21:04
mriedemso we don't touch those vhostuser tests21:04
*** eharney has quit IRC21:05
*** matrohon has quit IRC21:05
*** TravT has joined #openstack-nova21:08
*** gouthamr has joined #openstack-nova21:09
*** ijw has joined #openstack-nova21:09
*** thumpba has quit IRC21:10
*** akshai has joined #openstack-nova21:10
*** gszasz has joined #openstack-nova21:11
*** salv-orlando has joined #openstack-nova21:11
*** catintheroof has joined #openstack-nova21:15
*** tblakes has quit IRC21:15
*** burgerk has quit IRC21:15
*** TravT has quit IRC21:16
*** krtaylor has quit IRC21:17
*** salv-orlando has quit IRC21:18
*** salv-orlando has joined #openstack-nova21:22
*** ijw has quit IRC21:25
*** tblakes has joined #openstack-nova21:26
*** gyee has joined #openstack-nova21:27
auggyi wonder if i can add "derailed a nova meeting with talk of donuts" to the success tag haha21:35
jaypipesmriedem: cool, re-reviewing.21:35
woodster_auggy: :)21:35
*** rkmrhj has quit IRC21:35
*** pwnall has joined #openstack-nova21:36
*** pwnall is now known as Guest4286421:37
jaypipesmriedem: wallaby'd.21:37
mriedemjaypipes: thanks21:37
jaypipesmriedem: for you? any time.21:37
mriedemjaypipes: 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_57021:39
mriedemdansmith: ^21:39
dansmithhmm21:40
*** lpetrut has quit IRC21:41
mriedemhttps://bugs.launchpad.net/nova/+bug/162411921:42
openstackLaunchpad bug 1624119 in OpenStack Compute (nova) "remove_deleted_instances in RT stacktraces with AttributeError" [Undecided,New]21:42
bauzasmriedem: so did someone checked we *actually* changed the mtu value in logs ? :D21:43
dansmithmriedem: RT is putting dicts into self.tracked_instances?21:43
dansmithoh, this is from paulmurray's dict-or-dict-or-something method I bet21:44
mriedembauzas: ihar added a tempest test for that which we were testing with21:44
mriedemhttp://logs.openstack.org/13/370813/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/ddf5f49/console.html#_2016-09-15_16_02_33_23227321:44
mriedemhttp://logs.openstack.org/51/371051/1/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/06f5d12/console.html#_2016-09-15_19_33_14_97602521:44
mriedemit failed before john's change21:44
bauzasmriedem: oh cool21:45
*** scottda has quit IRC21:45
*** anush has quit IRC21:45
*** adam_g has quit IRC21:45
*** knikolla has quit IRC21:45
bauzasmriedem: because I was about to read logs to check that value21:45
*** wasmum has quit IRC21:46
*** clif_h has quit IRC21:46
*** Vek has quit IRC21:46
*** cargonza has quit IRC21:46
*** mrda has quit IRC21:46
*** efried has quit IRC21:46
*** CustosLimen has quit IRC21:46
*** JayF has quit IRC21:46
*** jamielennox has quit IRC21:46
*** jhesketh has quit IRC21:47
*** swamireddy has quit IRC21:47
*** gus has quit IRC21:47
*** d34dh0r53 has quit IRC21:47
*** mrhillsman has quit IRC21:47
*** pradk has quit IRC21:48
*** gus has joined #openstack-nova21:48
*** scottda has joined #openstack-nova21:49
*** jhesketh has joined #openstack-nova21:49
jaypipesdansmith: shall I fix?21:49
dansmithjaypipes: sure21:49
bauzasmriedem: meh the RT bug you wrote is very sad21:50
bauzasI could look at that by tomorrow morning21:50
bauzasunless US folks want to eat their afternoon looking at that21:50
mriedemwell, i'll be on all night21:51
*** adam_g has joined #openstack-nova21:51
mriedemso if we want this in rc1 i'm game21:51
*** adam_g has quit IRC21:51
*** adam_g has joined #openstack-nova21:51
*** d34dh0r53 has joined #openstack-nova21:51
*** CustosLimen has joined #openstack-nova21:52
*** cargonza has joined #openstack-nova21:52
dansmithso,21:52
*** ijw has joined #openstack-nova21:52
dansmiththis will auto-heal when we apply21:52
*** knikolla has joined #openstack-nova21:52
dansmithso I'm fine without it in rc121:52
dansmithup to you21:52
*** rlrossit has quit IRC21:52
mriedemif someone can get a quick fix up i'll check it once i'm back home21:52
dansmithif we have an rc2 then we do that, and if not, backport.. not the end of the world21:52
mriedemi think we'll have rc2 for translations21:53
dansmithyeah, so.. no reason to rush it IMHO21:53
*** wasmum has joined #openstack-nova21:54
mriedemi'd hate to release with big fat stack traces even if you're not using hte placement api21:54
mriedemso i think rc2 at least21:54
bauzasyeah, I was targeting RC221:54
bauzashence my offer for tomorrow21:54
dansmithsure21:54
mriedemi think dan is out tomorrow, and i'm out in the afternoon21:55
dansmithyeah21:55
bauzasmriedem: planning to do the release change today I guess21:55
mriedemyes i'll tag rc1 tonight21:55
bauzasokay21:55
bauzasso yeah, I'll look at that bug and we could see what would be the resolution path for RC221:55
bauzasanyway, folks \o21:55
bauzaslet's call it a day21:56
dansmithjaypipes: you're on it?21:56
*** jheroux has quit IRC21:57
*** swamireddy has joined #openstack-nova21:57
mriedemi'm heading home, back online later21:58
*** mriedem has quit IRC21:58
*** cmartin has quit IRC21:58
*** mrda has joined #openstack-nova21:59
*** clif_h has joined #openstack-nova21:59
dansmithoh, no it's just obj_to_primitive()'d  .. heh how silly21:59
dansmitheasy fix21:59
*** 32NABYP9K is now known as sabari21:59
*** JayF has joined #openstack-nova21:59
*** mrhillsman has joined #openstack-nova22:00
*** jamielennox has joined #openstack-nova22:01
*** efried has joined #openstack-nova22:01
*** akshai has quit IRC22:02
*** smatzek has quit IRC22:03
*** kylek3h has quit IRC22:03
openstackgerritDan Smith proposed openstack/nova: Fix object assumption in remove_deleted_instances()  https://review.openstack.org/37117922:03
dansmithjaypipes: ^22:03
*** rtheis has quit IRC22:03
*** krtaylor has joined #openstack-nova22:04
*** rhagarty_ has joined #openstack-nova22:04
dansmiththe test didn't catch it because it only tested an empty compute node22:04
*** cleong has quit IRC22:05
*** rhagarty has quit IRC22:06
*** xyang1 has quit IRC22:07
*** liangy has quit IRC22:09
*** Apoorva has quit IRC22:09
*** artom has quit IRC22:10
*** mdrabe has quit IRC22:11
*** edmondsw has quit IRC22:14
*** mdrabe has joined #openstack-nova22:14
*** artom has joined #openstack-nova22:15
*** unicell has quit IRC22:16
openstackgerritAnusha Unnam proposed openstack/nova: Remove deprecated compute_available_monitors option  https://review.openstack.org/37106222:18
*** jkraj has joined #openstack-nova22:19
*** ijw has quit IRC22:19
*** Guest42864 has quit IRC22:20
*** ijw has joined #openstack-nova22:22
*** Apoorva has joined #openstack-nova22:22
*** marst has quit IRC22:22
*** marst has joined #openstack-nova22:23
*** mdrabe has quit IRC22:23
*** marst has quit IRC22:26
*** tonytan4ever has quit IRC22:27
*** salv-orlando has quit IRC22:32
*** TravT has joined #openstack-nova22:33
openstackgerritAnusha Unnam proposed openstack/nova: Remove deprecated compute_available_monitors option  https://review.openstack.org/37106222:39
*** thumpba has joined #openstack-nova22:41
*** marst has joined #openstack-nova22:41
*** ducttape_ has quit IRC22:42
*** unicell has joined #openstack-nova22:43
jaypipesdansmith: wallaby'd.22:43
dansmithjaypipes: thanks22:44
jaypipesdansmith: I had done it locally but then had meeting and had to leave tests running :) oh well22:44
*** mriedem has joined #openstack-nova22:44
dansmithjaypipes: I figured your big deal ness got in the way, figured I'd work it up being a peon and all :D22:44
jaypipesdansmith: ha ha.22:44
dansmith:P22:45
mriedemoh snap22:45
mriedemhttps://review.openstack.org/#/c/371179/22:45
mriedemrc2 indeed22:45
*** salv-orlando has joined #openstack-nova22:46
*** anush has joined #openstack-nova22:46
*** Cristina_ has joined #openstack-nova22:48
*** baoli has quit IRC22:48
*** yamahata has quit IRC22:49
openstackgerritAnusha Unnam proposed openstack/nova: Remove config option snapshot_name_template  https://review.openstack.org/37120322:50
*** jaypipes has quit IRC22:50
*** ijw has quit IRC22:50
*** CristinaPauna has quit IRC22:51
*** sdague has quit IRC22:52
*** artom has quit IRC22:55
*** hemna is now known as hemnafk22:55
*** artom has joined #openstack-nova22:56
*** claudiub has quit IRC23:06
*** ijw has joined #openstack-nova23:11
*** ducttape_ has joined #openstack-nova23:12
*** jkraj has quit IRC23:13
*** ijw has quit IRC23:16
*** nic has quit IRC23:18
*** maticue has quit IRC23:21
*** catintheroof has quit IRC23:21
*** takedakn has joined #openstack-nova23:23
*** takashin has left #openstack-nova23:25
*** Apoorva has quit IRC23:25
*** ducttape_ has quit IRC23:29
*** takedakn has quit IRC23:30
*** thumpba has quit IRC23:30
*** tblakes has quit IRC23:32
*** TravT has quit IRC23:36
*** armax has quit IRC23:37
*** hoonetorg has quit IRC23:37
*** markvoelker has quit IRC23:39
*** TravT has joined #openstack-nova23:40
*** Jeffrey4l__ is now known as Jeffrey4l23:40
*** BjoernT has quit IRC23:44
*** gszasz has quit IRC23:45
mriedemgdi wtf http://logs.openstack.org/81/370681/8/gate/gate-nova-python27-db-ubuntu-xenial/87b6e37/console.html#_2016-09-15_22_54_49_86878623:46
mriedemglobal change here? https://review.openstack.org/#/c/370681/8/nova/tests/unit/virt/libvirt/test_vif.py@161423:47
*** rmcall has quit IRC23:47
*** unicell has quit IRC23:50
*** ijw has joined #openstack-nova23:50
*** tri2sing has quit IRC23:53
*** Vek has joined #openstack-nova23:54
*** armax has joined #openstack-nova23:54
*** hoonetorg has joined #openstack-nova23:55
*** thumpba has joined #openstack-nova23:57
*** ijw has quit IRC23:58
*** rama_y has quit IRC23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!