Wednesday, 2014-09-03

jogodtroyer: ping https://bugs.launchpad.net/bugs/133127400:05
jogosdague: said you had an idea about that one00:05
jogosomething about services not being killed properly00:08
*** melwitt has left #openstack-qa00:09
*** melwitt has joined #openstack-qa00:09
melwittjogo: hi, on https://bugs.launchpad.net/grenade/+bug/1349617 I was thinking if it would be reasonable to make a new log file for paramiko debug logs so we can look at details of ssh connections in tests?00:11
cyeohif there are any cores around could they have a look at https://review.openstack.org/#/c/112445/ (it was waiting for the nova change to be +2'd - which it has. And it has one +2 itself already00:18
*** sballe_ has quit IRC00:22
mtreinishcyeoh: +A00:23
cyeohmtreinish: thanks!00:23
*** dmorita has joined #openstack-qa00:25
dtroyerjogo: that's what I just proposed (again).  First thing is to make services start/stop reliably without screen, then we see what is left for that bug00:31
*** sballe has joined #openstack-qa00:31
*** yamahata has joined #openstack-qa00:34
*** yamahata has quit IRC00:36
*** mrodden has quit IRC00:36
*** yamahata has joined #openstack-qa00:36
*** mrodden has joined #openstack-qa00:37
jogomelwitt: yeah sounds like a good idea00:43
jogomelwitt: thanks for digging into that bug00:43
jogodtroyer: cool, link?00:43
dtroyerhttps://review.openstack.org/#/c/117339/00:44
melwittjogo: :) I think I'm not helping much but I'm trying. I'm checking into how to add the paramiko logging so I can put up a patch for it00:44
jogomelwitt: awesome00:47
jogodtroyer: cool00:49
jogodtroyer: btw I wonder if https://review.openstack.org/#/c/116484/ is related00:49
jogorather the bug underneat00:49
jogoh00:49
dtroyerjogo: there's a good chance it is00:49
jogoso that would nead a nova fix?00:50
jogocan you update the bug with your comments00:50
dtroyerI doubt it…if that bug happens outside devstack then it isn't related00:50
dtroyerwhat I'm changing is how we launch services inside devstack, and how we get the pid of those to kill them later00:51
dtroyer(only when USE_SCREEN=False)00:51
*** gmann has joined #openstack-qa00:52
jogobut we use screen in grenade00:52
jogowhere this bug is happening00:52
dtroyeronly because of not being able to stop services reliably00:52
jogoahh, so with this we can drop screen in grenade00:52
dtroyerthat's the idea00:52
jogoahh cool00:53
*** suneelb has quit IRC01:11
openstackgerritHirofumi Ichihara proposed a change to openstack-dev/devstack: Fix keystone auth parameter to neutron conf  https://review.openstack.org/11851201:11
*** dims has joined #openstack-qa01:11
*** mwagner_lap has joined #openstack-qa01:14
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Migrate baremetal_basic_ops to tempest clients  https://review.openstack.org/11767301:16
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Add baremetal tests for get node by instance/port  https://review.openstack.org/11851301:16
openstackgerritAdam Gandelman proposed a change to openstack-dev/grenade: Set required DSG variables (DO NOT MERGE)  https://review.openstack.org/11851501:20
*** catohornet has quit IRC01:23
*** dims has quit IRC01:23
*** dims has joined #openstack-qa01:23
*** dims_ has joined #openstack-qa01:26
*** zz_dimtruck is now known as dimtruck01:26
*** dims has quit IRC01:28
*** mestery has quit IRC01:31
*** mestery has joined #openstack-qa01:31
*** sballe has quit IRC01:32
*** mestery_ has joined #openstack-qa01:35
*** mestery has quit IRC01:37
*** yjiang5 has quit IRC01:42
gmanncoolsvap: ping01:47
*** melwitt has quit IRC01:48
gmanncoolsvap: one update needed in https://review.openstack.org/#/c/90306/ . please update that.01:48
gmanncoolsvap: Sorry i forgot to notice that earlier.  My bad....01:49
*** Sukhdev has joined #openstack-qa01:50
*** pcrews has quit IRC01:50
*** Sukhdev has quit IRC01:51
*** pcrews has joined #openstack-qa01:52
*** nosnos has joined #openstack-qa01:52
*** dims_ has quit IRC02:03
*** dims has joined #openstack-qa02:03
*** adalbas has quit IRC02:07
*** dims has quit IRC02:08
*** mestery_ is now known as mestery02:09
*** salv-orlando has quit IRC02:10
*** dims has joined #openstack-qa02:13
*** nosnos has quit IRC02:16
*** pcrews has quit IRC02:28
*** bkopilov has quit IRC02:35
*** mriedem has quit IRC02:42
*** harlowja is now known as harlowja_away02:52
*** jyuso has joined #openstack-qa02:53
openstackgerritMatthew Treinish proposed a change to openstack-infra/subunit2sql: Add subunit2sql-db-manage utility  https://review.openstack.org/11841802:54
*** gmann has quit IRC02:55
*** dims has quit IRC02:56
*** dims has joined #openstack-qa02:57
*** Madan has joined #openstack-qa03:00
*** yjiang5 has joined #openstack-qa03:01
*** dims has quit IRC03:01
*** jecarey_ has joined #openstack-qa03:04
openstackgerritYAMAMOTO Takashi proposed a change to openstack-dev/devstack: ofagent: Switch to IptablesFirewallDriver  https://review.openstack.org/10891503:16
openstackgerritAdam Gandelman proposed a change to openstack-dev/grenade: Set required DSG variables (DO NOT MERGE)  https://review.openstack.org/11851503:18
*** cmyster has quit IRC03:21
*** dimtruck is now known as zz_dimtruck03:26
*** Poornima has joined #openstack-qa03:42
*** gmann_ has quit IRC03:51
*** Sukhdev has joined #openstack-qa03:52
*** gmann has joined #openstack-qa03:53
*** bkopilov has joined #openstack-qa04:00
*** melwitt has joined #openstack-qa04:06
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Migrate test_server_cfn_init to tempest clients  https://review.openstack.org/11771404:09
*** melwitt1 has joined #openstack-qa04:13
*** melwitt has quit IRC04:16
*** SridharG has joined #openstack-qa04:24
*** SridharG has quit IRC04:28
*** Ch00k has joined #openstack-qa04:30
*** zz_dimtruck is now known as dimtruck04:32
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Fix unstable assertion in test_cinder_endpoints  https://review.openstack.org/11774204:34
*** ianw has quit IRC04:37
*** ianw has joined #openstack-qa04:37
*** jecarey_ has quit IRC04:41
*** nosnos has joined #openstack-qa04:50
*** amotoki has joined #openstack-qa04:56
*** yfried_ has quit IRC05:00
*** dimtruck is now known as zz_dimtruck05:00
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Remove resolved fingerprints  https://review.openstack.org/11791605:05
*** NuclearW0rm has quit IRC05:07
openstackgerritAdam Gandelman proposed a change to openstack-dev/grenade: Set required DSG variables (DO NOT MERGE)  https://review.openstack.org/11851505:09
*** chandankumar has joined #openstack-qa05:13
openstackgerritSwapnil Kulkarni proposed a change to openstack/tempest: Adds the clients and tests for CINDER QoS V1 & V2 APIs  https://review.openstack.org/9030605:15
*** neeti has joined #openstack-qa05:17
*** enikanorov__ has quit IRC05:23
*** Sukhdev has quit IRC05:52
*** SridharG has joined #openstack-qa05:52
*** rdekel has joined #openstack-qa05:53
openstackgerritvarun kumar yadav proposed a change to openstack/tempest: Test to update port with CIDR value  https://review.openstack.org/11386105:56
*** Ch00k has quit IRC06:00
*** yjiang5 has quit IRC06:08
*** raies has joined #openstack-qa06:13
openstackgerritA change was merged to openstack/tempest: Fix "mutable" object as default value  https://review.openstack.org/10013206:17
*** k4n0 has joined #openstack-qa06:18
*** afazekas has joined #openstack-qa06:18
raiescyeoh: hi06:21
cyeohraies: hi06:21
raiescyeoh: I am installing openstack multi compute node06:22
raiesnode 1 - controller, node 2 - compute 1 , node 3 - compute 206:22
raiescyeoh: controller install is successful06:22
raiescompute install is also successful on both compute nodes06:23
raiesbut06:23
raiesduring nova network installation (legacy mode)06:23
raieswhen I tried to install on both compute nodes06:23
raiesthen besides it was successfull, but during network create Networklist InstanceList not supported resulted06:24
raiesagain I remove nova network from compute 206:24
raiesand only install there nova-compute06:24
raiesand tried to install compute + nova-network on compute 106:25
raiesthen following error is encountered in the logs -06:25
raiescyeoh: http://pastebin.com/zM9bxDFV06:25
raiescyeoh: looking ??06:26
cyeohI don't know - looks like you have a set of packages for your install that is not self consistent06:27
cyeohbut I can't say I know what is really going on with your setup06:28
raiesokks06:29
raiesnp06:29
*** flaper87|afk is now known as flaper8706:30
*** yfried_ has joined #openstack-qa06:34
*** tkammer has joined #openstack-qa06:36
*** cmyster has joined #openstack-qa06:40
*** luqas has joined #openstack-qa06:41
*** evgenyf has joined #openstack-qa06:43
*** aimon has quit IRC06:54
*** jhenner has quit IRC06:57
*** ildikov has joined #openstack-qa06:58
*** chandankumar has quit IRC07:07
raiescyeoh: hi07:11
raiesanyone -- how to make nova version same across all nodes?07:13
*** chandan_kumar has joined #openstack-qa07:13
*** nmagnezi has joined #openstack-qa07:14
*** markus_z has joined #openstack-qa07:14
*** jhenner has joined #openstack-qa07:19
*** e0ne has joined #openstack-qa07:19
*** qba73 has joined #openstack-qa07:20
raiesanyone ^^07:20
*** flaper87 is now known as flaper87|afk07:26
openstackgerritA change was merged to openstack-infra/devstack-gate: Add oslo.utils and oslo.concurrency  https://review.openstack.org/11730707:28
*** shardy_afk is now known as shardy07:29
*** lyxus has quit IRC07:29
*** lyxus has joined #openstack-qa07:31
raiesgmann: hi07:31
raies have installed  openstack multinode07:32
raiesgmann: but on controller nova service-list does not show compute service info07:32
*** yfried_ has quit IRC07:32
raieswhile compute service is running on compute node07:32
raiesgmann: any idea07:33
*** chandan_kumar has quit IRC07:33
raiesgmann: It was issue related to amqp, when connected there is error like IncompatibleObjectVersion: Version 1.8 of InstanceList is not supported07:35
*** yfried has joined #openstack-qa07:35
*** wendar_ is now known as wendar07:36
*** jhenner has quit IRC07:37
*** e0ne has quit IRC07:38
*** e0ne has joined #openstack-qa07:38
*** flaper87|afk is now known as flaper8707:39
*** marun_ has quit IRC07:41
openstackgerritOpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements  https://review.openstack.org/11011007:42
gmannraies: hi07:42
*** jlibosva has joined #openstack-qa07:43
raiesgmann: sorry problem resolved :)07:43
gmannraies: :). what was the issue?07:43
raiesactually issue was during multinode installation we were unable to acces compute from controller07:44
raiesgmann: and somethis InstanceList version support isseue was there07:44
raies:)07:44
*** jhenner has joined #openstack-qa07:45
gmannraies: ohk. So how you fixed that07:45
*** tkammer has quit IRC07:46
raiesyes actually there was a conflict of with devstack packages07:46
*** jlanoux has joined #openstack-qa07:46
raiesI removed them and it is working nw07:46
raies:)07:46
*** chandan_kumar has joined #openstack-qa07:47
openstackgerritA change was merged to openstack-dev/devstack: Add myself to the MAINTAINERS file for Swift  https://review.openstack.org/11796907:52
*** safchain has joined #openstack-qa07:53
*** jhenner has quit IRC07:58
*** gszasz has joined #openstack-qa07:59
openstackgerritOpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements  https://review.openstack.org/11011008:01
*** Ch00k has joined #openstack-qa08:01
*** tkammer has joined #openstack-qa08:02
gmannraies: cool08:07
*** catohornet has joined #openstack-qa08:16
*** ZZelle has quit IRC08:18
*** ZZelle has joined #openstack-qa08:19
*** gszasz has quit IRC08:22
*** melwitt1 has quit IRC08:24
*** oomichi has quit IRC08:25
*** luqas has quit IRC08:28
*** chandan_kumar has quit IRC08:29
*** zz_johnthetubagu is now known as johnthetubaguy08:30
masayukigShould we stop to approve patches of Tempest except critical bug fixes for now? Because we should preserve the gate queue as ttx said and Tempest doesn't have 'Feature Freeze'.08:32
*** tkammer has quit IRC08:34
*** salv-orlando has joined #openstack-qa08:36
*** Longgeek has joined #openstack-qa08:39
*** ildikov_ has joined #openstack-qa08:44
*** tkammer has joined #openstack-qa08:46
*** ildikov has quit IRC08:47
*** mpaolino has joined #openstack-qa08:51
*** amotoki has quit IRC08:52
*** Longgeek has quit IRC08:53
*** Longgeek has joined #openstack-qa08:53
yfriedmasayukig: +108:53
*** andreaf has joined #openstack-qa08:53
yfriedmasayukig: seems like the gate is stuck as it is. it's now almost 24hr for a single merge. no reason adding new stuf08:54
yfriedstuff08:54
yfriedgmann: remember our talk yesterday about large_ops and connectivity checks?08:55
gmannyfried: yes08:55
*** gszasz has joined #openstack-qa08:56
gmannmasayukig: Agree. +108:56
yfriedgmann: so I'm testing it with 10 VMs. it's 66sec for no connectivity and 120sec with connectivity. though I think that it's not a 1:2 issue, but rather X+60sec issue since it's ~60 sec for a VM to achieve connectivity. and all VMs are doing this in parallel, so they should all be ready by the time the first few are ready08:58
*** masayukig has quit IRC09:00
*** masayukig has joined #openstack-qa09:00
yfriedgmann: I wonder how does it run with 100 VMs at the gate. quota is 10 and you need tenant id to change it. how do you change quota for all tenants09:01
gmannyfried: I see. +~60 sec should not be issue i think.09:02
*** luqas has joined #openstack-qa09:04
yfriedgmann: I can't check it on large scale without changing quotas for all tenants. how do I do that?09:04
yfriedgmann: nm - figured it out. gonna run some numbers now :)09:09
gmannyfried: thats need to be changed in nova. if you want to tests with 100 that can be done by changing - https://github.com/openstack/nova/blob/master/nova/quota.py09:09
gmannyfried: how ? by quota change in nova?09:10
yfriedgmann: nova.conf has default quotas09:10
yfriedthat's the link you just sune09:10
yfriedsent09:10
gmannyfried: just give try. but with 100 VM quota it might need more ram and other resources in your dev environment09:12
yfriedgmann: my setup is a baremetal machine and not a VM09:13
gmannyfried: ohk09:13
*** aix has joined #openstack-qa09:13
gmannyfried: i think if that work with large scale VM number. its a good to enhance that tests with connectivity check.09:14
gmannyfried: have to go. ll catch u later. best of luck :)09:15
*** cdent has joined #openstack-qa09:15
*** neeti has quit IRC09:18
*** neeti has joined #openstack-qa09:21
*** jhenner has joined #openstack-qa09:21
*** mkollaro has joined #openstack-qa09:21
openstackgerritSantosh Kumar proposed a change to openstack/tempest: Check port's status also in show port tests.  https://review.openstack.org/11677109:21
*** Longgeek has quit IRC09:24
*** evgenyf has quit IRC09:27
*** Longgeek has joined #openstack-qa09:27
*** evgenyf has joined #openstack-qa09:28
*** e0ne has quit IRC09:35
*** jhenner has quit IRC09:36
*** dmorita has quit IRC09:37
*** marun_ has joined #openstack-qa09:39
sergeyshhi09:39
sergeyshI'll appreciate your review in very short patches, please: https://review.openstack.org/#/c/118424/, https://review.openstack.org/#/c/117258/09:39
*** neeti has quit IRC09:40
*** e0ne has joined #openstack-qa09:41
*** Santosh_ has joined #openstack-qa09:49
*** neeti has joined #openstack-qa09:51
Santosh_hi there...09:51
*** tosky has joined #openstack-qa09:52
Santosh_gmann: hi09:55
nmagnezimasayukig, thanks for the review! (https://review.openstack.org/#/c/117454/) --> will make sure to follow the commit msg guidelines in my next patches10:02
*** jhenner has joined #openstack-qa10:05
*** jhenner has quit IRC10:10
*** yamahata has quit IRC10:11
*** jhenner has joined #openstack-qa10:16
coolsvapcyeoh, hi10:19
cyeohcoolsvap: hi10:20
coolsvapcyeoh, can you please have a look at https://review.openstack.org/90306 and https://review.openstack.org/11587110:20
cyeohcoolsvap: I'll put them on the list....10:21
coolsvapcyeoh, thanks10:21
cyeohI'd already commented on 115871 btw10:22
*** e0ne has quit IRC10:24
*** Longgeek has quit IRC10:31
*** Longgeek has joined #openstack-qa10:32
*** jhenner has quit IRC10:33
openstackgerritA change was merged to openstack/tempest: Migrate test_network_basic_ops to tempest clients  https://review.openstack.org/11171910:33
masayukignmagnezi: you're welcome :)10:34
openstackgerritA change was merged to openstack/tempest: Add a check_telemetry method to javelin  https://review.openstack.org/10235410:34
*** jlanoux has quit IRC10:35
*** Longgeek_ has joined #openstack-qa10:36
*** masayukig has quit IRC10:39
*** Longgeek has quit IRC10:39
*** e0ne has joined #openstack-qa10:39
*** Poornima has quit IRC10:54
*** jhenner has joined #openstack-qa10:56
*** Poornima has joined #openstack-qa10:57
openstackgerritNir Magnezi proposed a change to openstack/tempest: Adds status check for router Port in scenarios  https://review.openstack.org/11745410:59
*** fungi has quit IRC11:03
*** comstud has quit IRC11:03
*** dims has joined #openstack-qa11:06
*** jyuso has quit IRC11:08
*** comstud has joined #openstack-qa11:08
*** marun_ has quit IRC11:10
*** marun has joined #openstack-qa11:10
*** dims has quit IRC11:12
*** dims has joined #openstack-qa11:13
*** Poornima has quit IRC11:14
*** dims_ has joined #openstack-qa11:14
*** fungi has joined #openstack-qa11:17
*** dims has quit IRC11:17
openstackgerritSantosh Kumar proposed a change to openstack/tempest: Check port's status also in show port tests  https://review.openstack.org/11677111:20
*** mpaolino has quit IRC11:21
*** Poornima has joined #openstack-qa11:23
*** cdent has quit IRC11:26
openstackgerritFlavio Percoco proposed a change to openstack-dev/devstack: Set configs for glance_store  https://review.openstack.org/11860911:28
*** dims_ has quit IRC11:29
*** dims has joined #openstack-qa11:29
*** masayukig has joined #openstack-qa11:31
*** dims has quit IRC11:34
*** dims has joined #openstack-qa11:37
*** jlanoux has joined #openstack-qa11:39
*** masayukig_mobi has joined #openstack-qa11:40
*** mmedvede has joined #openstack-qa11:40
*** masayukig_mobi has quit IRC11:42
*** yamahata has joined #openstack-qa11:45
*** masayukig_mobi has joined #openstack-qa11:46
*** masayukig has quit IRC11:48
*** masayukig has joined #openstack-qa11:48
*** dims has quit IRC11:50
*** dims has joined #openstack-qa11:50
*** dims_ has joined #openstack-qa11:51
*** Longgeek_ has quit IRC11:53
*** masayukig_mobi has quit IRC11:53
*** Longgeek has joined #openstack-qa11:54
*** dims has quit IRC11:54
*** Longgeek_ has joined #openstack-qa11:54
*** marun_ has joined #openstack-qa11:56
*** yamahata has quit IRC11:58
*** Longgeek has quit IRC11:58
*** marun has quit IRC11:59
*** yamahata has joined #openstack-qa12:00
*** jlanoux_ has joined #openstack-qa12:00
*** jlanoux has quit IRC12:01
*** markmcclain has joined #openstack-qa12:02
*** markmcclain has quit IRC12:02
*** HenryG is now known as HenryG_afk12:04
*** markmcclain has joined #openstack-qa12:04
*** tkammer has quit IRC12:04
*** aysyd has joined #openstack-qa12:16
*** nosnos has quit IRC12:16
*** nosnos has joined #openstack-qa12:17
openstackgerritRoman Bogorodskiy proposed a change to openstack/tempest: Add coraid_repository_key config variable  https://review.openstack.org/11861912:17
*** Madan has quit IRC12:18
*** jlanoux has joined #openstack-qa12:18
*** Longgeek_ has quit IRC12:19
*** tkammer has joined #openstack-qa12:20
*** jlanoux_ has quit IRC12:21
*** nosnos has quit IRC12:21
*** adalbas has joined #openstack-qa12:21
*** dkranz has joined #openstack-qa12:21
*** dims_ has quit IRC12:29
*** dims has joined #openstack-qa12:30
*** Longgeek has joined #openstack-qa12:30
*** cnesa10 has quit IRC12:30
*** e0ne has quit IRC12:31
*** dims_ has joined #openstack-qa12:32
*** evgenyf has quit IRC12:34
*** dims has quit IRC12:34
*** e0ne has joined #openstack-qa12:34
*** cdent has joined #openstack-qa12:36
*** tkammer has quit IRC12:37
ekhugenandreaf, I saw your comments on https://review.openstack.org/#/c/118469/  I was putting that in because I picked up the bug.  I'm fine with it waiting until the test-accounts bp goes in, if the bug will be closed with that?  Thanks.12:40
andreafekhugen: ok -  I can the bug id to my patch so it's tracked there12:44
ekhugenandreaf, okay, I'll abandon the patch then, that's what you'd prefer?12:44
*** freyes has quit IRC12:45
*** dims_ has quit IRC12:45
andreafekhugen: just one question - the LP bug, is that an actual bug or just a nice improvement to have? Does it cause any actual failure?12:45
*** freyes has joined #openstack-qa12:46
andreafekhugen: if it does we should fix that before the test-accounts bp lands because that may take still a bit of time12:46
*** dims has joined #openstack-qa12:46
ekhugenandreaf, no the lp bug was just to streamline the tenant isolation code, no actual bug12:46
andreafekhugen: ok than I'm fine with you abandoning your patch12:47
andreafekhugen: thanks for working on that anyways12:48
ekhugenandreaf, you don't need any help with the bp, do you?  I was looking for something new to work on12:48
andreafekhugen: the bp is basically done, only it uncovered other shortcomings in tempest code, so it can't be merged until such issues are solved or bypassed somehow12:49
mkollarois there some way to disable s3 tests? they get run even if swift is disabled12:49
andreafekhugen: I'm working on a patch for the bypass approach12:49
*** tkammer has joined #openstack-qa12:49
andreafekhugen, and the actual solution is a brand new bp12:49
andreafekhugen, which will need volunteers once the spec is reviewed / approved12:50
andreafhttps://blueprints.launchpad.net/tempest/+spec/resource-cleanup12:50
ekhugenandreaf, okay, I'll keep an eye on that bp, thanks! and look for something else in the meantime12:51
andreafekhugen: thank you!12:51
andreafmkollaro: that sounds like a bug12:54
*** cdent has quit IRC12:56
*** vponomaryov has quit IRC12:57
*** aix has quit IRC12:58
mkollaroandreaf: but maybe it is fixed already...I'm forced to use a 5 month old version of tempest12:59
*** dkranz has quit IRC12:59
mkollaroin any case, does anybody have an idea how to debug the "S3ResponseError: S3ResponseError: 412 Precondition Failed" error in create_bucket? because I have no idea where to go from there13:00
*** enikanorov has joined #openstack-qa13:01
*** aix has joined #openstack-qa13:01
mkollarothe boto client class has create_bucket in ALLOWED_METHODS and that's it...I don't know where the implementation is13:01
*** timello_ has joined #openstack-qa13:02
*** bkopilov has quit IRC13:04
*** zz_dimtruck is now known as dimtruck13:07
*** timello_ has quit IRC13:09
*** chandankumar has joined #openstack-qa13:12
*** toure has joined #openstack-qa13:15
*** dkranz has joined #openstack-qa13:15
*** enikanorov__ has joined #openstack-qa13:17
*** timello_ has joined #openstack-qa13:18
*** enikanorov has quit IRC13:19
*** mriedem has joined #openstack-qa13:21
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Add Credentials Provider factory  https://review.openstack.org/10768513:29
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Force use of Accounts for test purposes  https://review.openstack.org/11457813:29
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Add a NotLockingAccounts credentials provider  https://review.openstack.org/11863513:29
andreafmtreinish: so here's my alternative to go ahead and merge the test-accounts work - preserving the functionality of the existing serial runs13:32
andreafmtreinish: let me know what you think. The only other problem left to solve is the migration from configured accounts to YAML file13:33
*** dpaterson has joined #openstack-qa13:34
andreafmtreinish: see https://review.openstack.org/118635 and https://review.openstack.org/114578 - and I've dropped the safe_setup patch from the list for now13:34
mtreinishandreaf: hmm, well we could handle that in code and just mark the old cred options as deprecated somehow13:35
mtreinishlike if there isn't a yaml file option specified pull it from config the old way13:36
mtreinishbut still use the new mechanism for grabbing the creds13:36
andreafmtreinish: yes that sounds like a good option13:36
andreafmtreinish: else apart from the periodic job tests will stop working for people not using isolated creds13:37
mtreinishandreaf: yeah we can't really break that without advanced notice because I know it's used a lot13:38
andreafmtreinish: so in this way only part of the new code is actually in use but we can close the bp and work on the resource cleanup one, and avoid endless rebases on test accounts13:38
*** Ch00k has quit IRC13:41
mtreinishandreaf: I'm thinking we should probably keep it open until we actually get it working, just retarget it and mark it as dependent on resource cleanup bp13:41
*** Ch00k has joined #openstack-qa13:42
*** malini has left #openstack-qa13:42
*** e0ne has quit IRC13:43
openstackgerritRoman Bogorodskiy proposed a change to openstack/tempest: Add coraid_repository_key config variable  https://review.openstack.org/11861913:44
*** e0ne has joined #openstack-qa13:50
openstackgerritRoman Bogorodskiy proposed a change to openstack/tempest: Add coraid_repository_key config variable  https://review.openstack.org/11861913:52
*** dimtruck is now known as zz_dimtruck13:53
andreafmtreinish: I think the advantages of merging it now would be: 1) start making people aware of the YAML file for credentials and allow for migration 2) merge the code that cleans up all the checks about tenant isolation in the code 3) avoid rebase nightmares on this13:56
*** e0ne has quit IRC13:56
*** cdent has joined #openstack-qa13:56
*** dhellmann has quit IRC13:57
*** dhellmann has joined #openstack-qa13:58
*** e0ne has joined #openstack-qa13:58
mtreinishandreaf: I was just talking about the bp13:59
mtreinishI agree merging things now makes sense13:59
*** nsaje_ has joined #openstack-qa14:00
*** rdekel has quit IRC14:00
*** nsaje_ is now known as nsaje14:00
andreafmtreinish: ah ok - yes it makes sense to keep the bp open14:02
andreafmtreinish: ok let me know what you think about the NotLockingAccounts - can you work on the fallback to credentials in conf, or shall I do that?14:03
*** pcrews has joined #openstack-qa14:04
mtreinishandreaf: if you could tackle it that'd probably be quicker right now14:05
mtreinishI might be able to get to it this afternoon14:06
andreafmtreinish: I'll try later tonight else tomorrow14:07
*** chandankumar has quit IRC14:07
*** yfried has quit IRC14:09
*** rbak has joined #openstack-qa14:09
*** zz_jgrimm is now known as jgrimm14:09
*** zz_dimtruck is now known as dimtruck14:10
andreafmtreinish: about the scenario migration bp - there are a few scenario tests which are not run in the gate right now - I would vote for migrating them anyways, so we can cleanup all the code related to OfficialClients and close the bp14:14
mtreinishandreaf: yeah I think we should migrate them too, but we need to show they work14:15
mtreinishprobably we should add WIP test patch on top of those which drop the skips so we can verify they run correctly14:15
mtreinishIIRC most of the skips were added because of unreliability in the gate at scale but a one off run should be reliable enough (assuming things didn't bitrot)14:16
*** chandankumar has joined #openstack-qa14:16
andreafmtreinish: ok14:17
andreafmtreinish: I don't see the skips in the code :S14:19
mtreinishandreaf: heh, which scenario tests are we skipping in the gate right now?14:20
andreafaccording to https://etherpad.openstack.org/p/tempest-client-scenarios it's test_stamp_pattern.py   , the baremetal one and the autoscaling one14:21
andreafso I guess baremetal scenario only runs in jobs with ironic virt driver enabled14:22
mtreinishandreaf: yep, and the autoscaling one only get's run on the heat slow job14:23
openstackgerritvigneshvar proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy  https://review.openstack.org/11798514:23
mtreinishand test_stamp_pattern has a skip because of bug 1205344: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_stamp_pattern.py#n15314:23
andreafok so that's the only one actually skipped14:24
andreafmtreinish, i fixed the etherpad14:26
*** andreaf has quit IRC14:28
openstackgerritMatthew Treinish proposed a change to openstack-infra/subunit2sql: Update pep8 ignore list  https://review.openstack.org/11865014:33
*** jecarey has joined #openstack-qa14:33
openstackgerritFlavio Percoco proposed a change to openstack-dev/devstack: Set configs for glance_store  https://review.openstack.org/11860914:33
sdaguemtreinish: why _hash in - https://review.openstack.org/#/c/114636/1214:35
*** HenryG_afk is now known as HenryG14:36
sdagueflaper87: there are still some hard tabs in your patch14:36
*** ildikov has joined #openstack-qa14:36
sdagueso it's going to fail again14:36
flaper87sdague: mmh, damn. let me fix that14:36
flaper87sdague: thanks for the heads up14:36
mtreinishsdague: hmm, I didn't push it that way? andreaf did you change that?14:36
mtreinishoh, he's offline now...14:36
openstackgerritFlavio Percoco proposed a change to openstack-dev/devstack: Set configs for glance_store  https://review.openstack.org/11860914:37
*** ildikov_ has quit IRC14:40
*** ildikov has quit IRC14:41
sdagueflaper87: cool, +214:42
flaper87sdague: thanks :)14:42
flaper87one more +2 to go (and wait for jenkins)14:42
sdagueare all the rest of the glance_store conversion patches for devstack / devstack-gate sorted?14:43
sdagueor are there things I can look at there to help review14:43
*** yfried has joined #openstack-qa14:44
*** Santosh has joined #openstack-qa14:46
openstackgerritAlexei Vinogradov proposed a change to openstack-dev/devstack: Add support of hbase database  https://review.openstack.org/8668314:46
*** atiwari has joined #openstack-qa14:47
flaper87sdague: that's the only one I have, thanks14:48
sdagueok, cool.14:48
openstackgerritvigneshvar proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy  https://review.openstack.org/11798514:50
*** mriedem has quit IRC15:00
*** mriedem has joined #openstack-qa15:01
*** neeti has quit IRC15:01
*** neeti has joined #openstack-qa15:02
*** Santosh has quit IRC15:04
*** markus_z has quit IRC15:11
*** SridharG has left #openstack-qa15:24
flaper87ianw: hey, when you get a chance, could you take a look at? https://review.openstack.org/#/c/118609/15:24
*** Sukhdev has joined #openstack-qa15:26
*** aix has quit IRC15:29
*** qba73 has quit IRC15:31
openstackgerritSean Dague proposed a change to openstack-dev/devstack: don't install boto from packages  https://review.openstack.org/11866115:32
*** amotoki has joined #openstack-qa15:34
*** Ch00k has quit IRC15:37
*** yamahata has quit IRC15:39
*** andreaf has joined #openstack-qa15:39
*** johnthetubaguy is now known as zz_johnthetubagu15:40
*** zz_johnthetubagu is now known as johnthetubaguy15:41
*** tkammer has quit IRC15:42
*** andreaf_ has joined #openstack-qa15:42
*** andreaf has quit IRC15:45
*** neeti has quit IRC15:46
openstackgerritA change was merged to openstack-infra/subunit2sql: Update pep8 ignore list  https://review.openstack.org/11865015:47
*** aix has joined #openstack-qa15:48
*** Poornima has quit IRC15:49
*** aimon has joined #openstack-qa15:53
*** mlavalle has joined #openstack-qa15:53
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: Add query for critical keystone regression  https://review.openstack.org/11866815:54
*** mlavalle has quit IRC15:54
*** mlavalle has joined #openstack-qa15:55
*** chandankumar has quit IRC15:55
*** gszasz has quit IRC15:59
*** enikanorov has joined #openstack-qa16:00
*** mdenny has joined #openstack-qa16:01
*** Longgeek has quit IRC16:02
*** sweston_ has joined #openstack-qa16:03
*** cbkyeoh has joined #openstack-qa16:03
*** eglynn__ has joined #openstack-qa16:04
ZZellemtreinish, hi16:05
*** tristanC_ has joined #openstack-qa16:06
*** afazekas has quit IRC16:07
*** masayukig_ has joined #openstack-qa16:07
*** dosaboy_ has joined #openstack-qa16:07
*** k4n0_ has joined #openstack-qa16:08
*** sweston has quit IRC16:08
*** cyeoh has quit IRC16:08
*** swartulv has quit IRC16:08
*** k4n0 has quit IRC16:08
*** sileht has quit IRC16:08
*** rossella_s has quit IRC16:08
*** eglynn has quit IRC16:08
*** tristanC has quit IRC16:08
*** danfinch has quit IRC16:08
*** enikanorov_ has quit IRC16:08
*** freyes has quit IRC16:08
*** StevenK has quit IRC16:08
*** luqas has quit IRC16:08
*** maurosr has quit IRC16:08
*** dosaboy has quit IRC16:08
*** masayukig has quit IRC16:08
*** krtaylor has quit IRC16:08
*** johnthetubaguy has quit IRC16:08
*** cbkyeoh is now known as cyeoh16:08
*** sweston_ is now known as sweston16:08
*** tristanC_ is now known as tristanC16:08
*** luqas has joined #openstack-qa16:09
*** rossella_s has joined #openstack-qa16:09
*** johnthetubaguy has joined #openstack-qa16:09
*** maurosr has joined #openstack-qa16:09
*** danfinch has joined #openstack-qa16:09
*** krtaylor has joined #openstack-qa16:10
*** freyes has joined #openstack-qa16:10
*** sileht has joined #openstack-qa16:11
*** krtaylor has quit IRC16:11
*** swartulv has joined #openstack-qa16:12
*** StevenK has joined #openstack-qa16:13
*** andreaf_ is now known as andreaf16:17
mtreinishZZelle: hi16:18
*** chandankumar has joined #openstack-qa16:19
ZZellemtreinish, about https://review.openstack.org/10349816:19
ZZellemtreinish, , because specific attributes (depending) on the deployment) are required to create flat/vlan/gre/vxlan networks16:20
ZZellemtreinish, i can not enable their testing by default16:20
mtreinishZZelle: that makes sense, but at the same time we need verification that the code works now and will continue to work moving forward16:21
ZZellemtreinish, for local networks, vmware does not support it and is not always available (depends on the deployment config/plugin)16:21
mtreinishand if it's not run as part of a ci job we don't have that16:21
ZZellemtreinish, the must i can do is to try create a local network and if it succeed, test local networks otherwise skip tests16:22
*** mwagner_lap has quit IRC16:22
ZZellemtreinish, that's why every test inherits from the same16:22
mtreinishZZelle: I'd also touch base with sc68cal, because at one point we had discussed having an extra devstack configs for testing provider networks.16:26
mtreinishI don't remember all the details of that discussion though16:26
*** safchain has quit IRC16:28
ZZelleIf local network support is enabled, we can check "must" of the code except the wrapper for flat/vlan/gre/vxlan tests which seems removable using testscenario16:31
ZZellemtreinish, and parse_raw_spec will be parsely covered (but i can add a unit test to check its behavior)16:32
mtreinishZZelle: yes, that's one of the advantages of using testscenarios :)16:32
*** jlanoux has quit IRC16:33
*** tkammer has joined #openstack-qa16:33
mtreinishZZelle: yeah that should provide decent enough coverage to move forward on that patch16:33
ZZellethe same trouble will append with its son change as it need multiple distinct network segment specs (which is not possible with only local networks) to test everything16:35
*** krtaylor has joined #openstack-qa16:37
*** luqas has quit IRC16:37
*** jlibosva has quit IRC16:38
*** mriedem1 has joined #openstack-qa16:40
openstackgerritvigneshvar proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy  https://review.openstack.org/11798516:42
*** dkranz has quit IRC16:43
*** mriedem has quit IRC16:44
*** arnaud has joined #openstack-qa16:45
*** tkammer has quit IRC16:48
*** chandankumar has quit IRC16:48
*** suneelb has joined #openstack-qa16:51
*** johnthetubaguy is now known as zz_johnthetubagu16:58
*** bkopilov has joined #openstack-qa16:59
*** marun_ has quit IRC17:00
*** yjiang5 has joined #openstack-qa17:02
*** gema has quit IRC17:02
*** tosky has quit IRC17:04
*** dkranz has joined #openstack-qa17:09
*** amotoki has quit IRC17:10
*** marun_ has joined #openstack-qa17:14
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for critical keystone regression  https://review.openstack.org/11866817:14
*** mwagner_lap has joined #openstack-qa17:16
*** yjiang5 has quit IRC17:18
*** harlowja_away is now known as harlowja17:20
*** yjiang5 has joined #openstack-qa17:23
*** gema has joined #openstack-qa17:25
openstackgerritAdam Gandelman proposed a change to openstack-infra/devstack-gate: Set required devstack variables for Ironic+grenade  https://review.openstack.org/11676117:26
*** Sukhdev has quit IRC17:26
*** yjiang5 has quit IRC17:28
e0nehi all! could you review https://review.openstack.org/#/c/116290/, please?17:30
*** chandankumar has joined #openstack-qa17:30
adam_gmtreinish, goodbye DEVSTACK_GATE_TEMPEST_REGEX https://review.openstack.org/11676117:31
*** yjiang5 has joined #openstack-qa17:33
mtreinishadam_g: did you mean: https://review.openstack.org/#/c/118700/1 :)17:34
mtreinishbut, it's good to finally get rid of using that17:34
adam_gmtreinish, oh yea :)17:34
*** e0ne has quit IRC17:35
openstackgerritAdam Gandelman proposed a change to openstack-dev/grenade: Set required DSG variables for Ironic DO NOT MERGE  https://review.openstack.org/11851517:35
*** tkammer has joined #openstack-qa17:38
*** chandankumar has quit IRC17:46
openstackgerritA change was merged to openstack/tempest: Removes test_force_delete_server_invalid_state negative test  https://review.openstack.org/11244517:46
*** pcrews has quit IRC17:47
*** morganfainberg is now known as morganfainberg_Z17:53
*** arnaud has quit IRC17:53
*** mkollaro has quit IRC17:54
clarkbthe ERRORs in log check is being tripped in places in the gate https://jenkins02.openstack.org/job/gate-grenade-dsvm/12009/console how do we typically treat those problems? is that open a bug against the services that logged the ERRORs?17:58
*** chandankumar has joined #openstack-qa17:59
*** aix has quit IRC18:00
mtreinishclarkb: yeah that's how I've always handled it18:01
mtreinishalthough I think I've seen that n-cond bug error in the logs before18:01
mtreinishmriedem1: ^^^ did you open a bug for it?18:01
*** weshay has joined #openstack-qa18:03
mriedem1mtreinish: clarkb: was talking with anteaya about that last week, it's a known bug 135313118:03
mriedem1the query we have is on the n-cond log, not console log18:04
mtreinishmriedem1: yeah, that's probably the right place for the query18:04
clarkbmriedem1: great, I just wasn't sure how we were dealing with those and agreed it should be in the n-cond log not console log18:05
mriedem1having said that, we frequently fail on the log scrub check18:05
mriedem1but e-r doesn't hit it18:05
clarkbmriedem1: is that because the scrub check is failign on additional things?18:06
mriedem1http://logs.openstack.org/77/118477/1/gate/gate-grenade-dsvm/dc695eb/logs/new/screen-n-cond.txt.gz#_2014-09-03_17_10_01_10018:06
mriedem1the e-r query should be hitting on that18:06
mriedem1this is the query: message:"Failed to commit reservations" AND tags:"screen-n-cond.txt" AND loglevel:"ERROR"18:06
*** mriedem1 is now known as mriedem18:07
clarkbhrm there are INFO lines for that file but not the ERROR18:09
mriedemhttp://logs.openstack.org/77/118477/1/gate/gate-grenade-dsvm/dc695eb/logs/new/screen-n-cond.txt.gz?level=ERROR#_2014-09-03_17_10_01_10018:09
mriedem?18:09
clarkbsorry in logstash18:09
mriedemah18:09
*** marun_ has quit IRC18:09
clarkbwe may have a parsing problem18:10
*** e0ne has joined #openstack-qa18:10
clarkb(it does show up in the console log which has much looser parsing)18:10
mriedemoh, and for some reason if you query on the failure in the console log, we don't get 100% failure in logstash, i think that's why i've left it out18:11
mriedemare there some jobs that don't have the log scrub check?18:11
mtreinishmriedem: hmm, I thought it would be on all the dsvm jobs18:11
mriedem.2% success18:12
mtreinishmriedem: hmm, which jobs does it succeed on? Sounds like a gap that needs to plugged18:13
mriedemgate-rally-dsvm-rally18:13
mtreinishoh, that's not gating so it doesn't matter18:13
mriedemand e-r doesn't filter out voting/non-voting jobs from what i remember18:13
mtreinishnope, not unless you do it in the query18:14
mriedemright18:14
mtreinishwe bake that into the e-r queries18:14
mriedemanyway, i rechecked that change18:15
mriedemand i think the oslo.db change will be approved this week18:15
mtreinishoh, cool18:15
clarkbneutron doesn't fail on the scrubber18:16
clarkbso all of the neutron tempest jobs18:16
clarkbanyways it looks like I need to do logstash debugging :/ hopefully I can figure out why we aren't getting that ERROR but moeny is on multiline blowing up18:17
*** mspreitz has joined #openstack-qa18:17
*** masayukig_ has quit IRC18:18
clarkbmriedem: mtreinish: you could do something like AND build_status:FAILURE as a hack to only get the failed jobs when checking the scrubbed list18:19
clarkbsince not all jobs fail on it, but those that do fail almost certainly fail at least partially due to the scrubber18:19
*** adalbas has quit IRC18:22
mtreinishclarkb: wouldn't that just mask cases where the bug occured? Isn't the issue that the scrubber isn't failing where it does everywhere else18:23
clarkbmtreinish: hrm good point so I guess tracking it on the success side is fine18:23
*** tkammer has quit IRC18:23
mtreinishclarkb: hmm, looking at the jjb configs only one dsvm job, the periodic stress job, turns off the scrubber faliures18:24
mtreinishso if it hits an error in a designated clean log it should fail everywhere18:25
*** masayukig has joined #openstack-qa18:25
clarkboh we only got the old side logs not the new side logs. ok starting to make progress on this18:25
clarkbmtreinish: neutron does it18:25
clarkbmtreinish: its done in devstack gate18:25
*** chandankumar has quit IRC18:26
clarkboh nevermind all that code was rewritten at some point18:26
clarkbI swear I have seen neutron jobs pass with non whitelisted errors though18:27
mtreinishclarkb: only some of the log files are designated to fail18:27
clarkbwow ok18:27
*** arnaud has joined #openstack-qa18:27
clarkbso we have multiple levels of whitelisting?18:28
mtreinisha bunch are too dirty so even if there are un-whitelisted errors things still pass18:28
*** pcrews has joined #openstack-qa18:28
mtreinishclarkb: yeah, it's an artifact of having too many things slip through when it was an explicit log message whitelist18:28
mtreinishso we switched to a per log file based whitelist18:28
clarkbI see. Would it be inappropriate to suggest we stop with the massive spam in logs then?18:28
clarkbit is extremely confusing18:29
mtreinishwe probably should remove the non-whitelisted errors18:29
mtreinishyeah I agree18:29
mtreinishit definitely is, I only realized it the other day when I was looking at one of these failures18:29
clarkbbut it also looks like logstash is derping pretty good. I picked a random tempest job and dont have n-cond logs in logstash for that one either18:30
clarkbso I need to look into that18:31
*** adalbas has joined #openstack-qa18:34
clarkbbut first I need food and caffeine. will dig into logstash stuff this afternoon18:39
mtreinishclarkb: I'm working on cleaning up the output from the log tool, should have a fix up in a few min18:42
clarkbmtreinish: awesome and thank you. That will probably help people when debugging (I know I probably attributed dirty logs to some failures inappropriately)18:43
*** e0ne has quit IRC18:47
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Move apache config creation to start_keystone()  https://review.openstack.org/11874018:48
mtreinishheh, testing the script out on coffee shop wifi probably wasn't the best idea...18:52
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Move apache config creation to start_keystone()  https://review.openstack.org/11874018:53
openstackgerritAdam Gandelman proposed a change to openstack-dev/devstack: Move apache config creation to start_keystone()  https://review.openstack.org/11874018:54
openstackgerritAdam Gandelman proposed a change to openstack-dev/grenade: Set FILES in upgrade-keystone  https://review.openstack.org/11874218:55
adam_gclarkb, re mod wsgi ^18:55
*** mkollaro has joined #openstack-qa18:59
adam_gdtroyer, any thoughts there? i think we need to merge the grenade change before the devstack one (assuming its approved)18:59
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Make output from check_logs less verbose  https://review.openstack.org/11874419:01
mtreinishclarkb: ^^^19:01
*** e0ne has joined #openstack-qa19:05
mtreinishdkranz: ^^^ you probably would be interested in that change too19:08
*** dmellado has quit IRC19:08
mtreinishclarkb: sample output: http://paste.openstack.org/show/105382/19:10
mtreinishI'm wondering if I should change the wording a bit19:11
*** dmellado has joined #openstack-qa19:11
mtreinishor maybe group it into 2 sets, those which have errors but didn't cause a failure and those that caused it to fail19:12
clarkbmtreinish: or even 'n-cpu log file has errors\nfoo-log log file has errors\nother-log log file has errors and is not allowed to have them"19:12
clarkbor and is not whitelisted or similar19:12
clarkbso that the listing is compact and direct19:12
*** e0ne has quit IRC19:16
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Migrate baremetal_basic_ops to tempest clients  https://review.openstack.org/11767319:16
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Add baremetal tests for get node by instance/port  https://review.openstack.org/11851319:16
*** e0ne has joined #openstack-qa19:19
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Remove PYTHONHASHSEED=0 from tox.ini  https://review.openstack.org/11674419:20
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Make unit tests not depend on random hash seed  https://review.openstack.org/11674119:20
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Update Heat CLI test to pass with random hash  https://review.openstack.org/11678619:20
mtreinishclarkb: yeah I like that, is this good: http://paste.openstack.org/show/105384/19:20
clarkbyup19:21
*** Anju has joined #openstack-qa19:21
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Make output from check_logs less verbose  https://review.openstack.org/11874419:22
mtreinishclarkb: ^^^19:22
mtreinishadam_g: did you need to rebase the random hashseed stack again?19:24
*** marun_ has joined #openstack-qa19:24
clarkbmtreinish: lgtm. you might also want to check the grenade switch I am not convinced it is working19:24
adam_gmtreinish, looked that way19:24
clarkbmtreinish: ya, we updated the DEVSTACK_GATE_GRENADE values19:24
adam_gconfig sample changes19:24
clarkbmtreinish: the check should be is_grenade = os.environ.get('DEVSTACK_GATE_GRENADE', None) != None (though you can drop the != None if you don't need True False values19:25
mtreinishadam_g: did we change config values in that tree? It shouldn't have caused a conflict19:26
mtreinishclarkb: yeah I was just looking at that, do you want to push out a patch for that?19:26
clarkbmtreinish: sure19:26
clarkbmtreinish: do you have a preference for truthiness values?19:26
mtreinishclarkb: heh, not really19:27
openstackgerritMatt Riedemann proposed a change to openstack-infra/devstack-gate: Deprecate API_RATE_LIMIT setting for Tempest runs  https://review.openstack.org/11600119:27
salv-orlandomtreinish: I am battling with bug 1349617, but I have a ridicoulous proposal for improving gate conditions19:28
mtreinishuh oh19:28
salv-orlandomtreinish: basically the failure happens always in two tests which are part of the advanced network scenario19:28
* mtreinish is on the edge of his seat19:28
salv-orlandostop_start and resize19:28
*** sabeen has joined #openstack-qa19:28
salv-orlandocan we skip them until the bug if fixed19:28
salv-orlandothe downside is that those test are also the only way to repro them19:29
mtreinishsalv-orlando: that's not a guaranteed fix, it could just move the failure19:29
mtreinishbut we can give it a try if you think those tests are passing reliably enough for right now19:29
salv-orlandomtreinish: which is also true. But I can try to push a patch for it, check it a reasonable number of times, and then make a call19:29
mtreinishs/are/aren't19:30
salv-orlandomtreinish: ok. also, if we see the failure moving somewhere else it might give us some more clues.19:30
adam_gmtreinish, hm my top patch was failing pep8 check_uptodate.sh19:30
salv-orlandomtreinish: thanks anyway for the advice.19:30
mtreinishsalv-orlando: yeah it's worth giving a shot at least19:31
openstackgerritClark Boylan proposed a change to openstack/tempest: Properly detect grenade in check_logs script  https://review.openstack.org/11875319:31
clarkbmtreinish: ^19:31
mtreinishclarkb: LGTM19:32
mtreinishadam_g: hmm whatever, if it makes it work that's all that counts19:33
mtreinishadam_g: wait, unless do you think a random hash was causing the check to fail?19:34
mtreinishby generating a different order for the config options in the temp file it creates?19:34
*** ZZelle_ has joined #openstack-qa19:34
ZZelle_mtreinish, one question19:34
mtreinishbecause the check script just does a diff between the file in etc and one generated when it's run19:35
adam_gmtreinish, oh man that woudl be awesome19:35
adam_gmtreinish, let me run the pep8 check in a loop19:35
mtreinishadam_g: that would explain why only the top patch was failing19:35
mtreinishZZelle_: sure, shoot19:36
adam_gmtreinish, good call. 2/5 checks failed locally :|19:36
ZZelle_mtreinish, if i use testscenarios for provider network tests, should i put them in tempest.scenario package? I would say because it seems not to feat tempest.scenario README.rst description19:37
adam_gmtreinish, i suppose running the only pep8 check with PYTHONHASHSEED=0 is better than running them all like that19:37
mtreinishdhellmann: ^^^ any thoughts? I know that the config generator tool is being replaced in newer oslo.config versions19:37
*** torandu has quit IRC19:37
mtreinishadam_g: yeah I think that's what we'll have to do moving forward19:37
mtreinishand we can get a fix in oslo long term19:38
adam_gmtreinish, ill revise that top patch19:38
mtreinishZZelle_: well if you don't think it's a scenario test then you shouldn't put it in tempest.scenario19:38
mtreinishtestscenarios is just a library you can use it anywhere19:38
mtreinishtake a look at the autogenerated negative api tests, they use testscenarios to generate tests in tempest.api19:39
ZZelle_mtreinish, ok, i am discovering testscenarios, so i am lost ... for the moment19:39
mtreinishZZelle_: yeah, it's a bit confusing because scenario is an overloaded term here19:39
mtreinishZZelle_: let me pull up a couple of examples in the repo for you19:39
mtreinishZZelle_: so http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/flavors/test_flavors_negative.py with http://git.openstack.org/cgit/openstack/tempest/tree/tempest/test.py#n402 is an example of where it's used in the api tests19:41
mtreinishalthough that's more complex than how you'll have to do it19:42
openstackgerritAdam Gandelman proposed a change to openstack/tempest: Remove PYTHONHASHSEED=0 from tox.ini  https://review.openstack.org/11674419:42
*** marun_ has quit IRC19:42
mtreinishZZelle_: a bit simpler of an example from the scenario tests is: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/utils.py and http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_server_basic_ops.py19:43
*** weshay has quit IRC19:46
*** sballe has joined #openstack-qa19:47
ZZelle_mtreinish, thanks, i think i get the mysterious (for me) ... scenario loading19:50
mtreinishZZelle_: nah, it's mysterious for pretty much everyone at first. It's kinda like black magic unless you've written code that uses it before.19:52
*** timello_ has quit IRC19:53
*** mkollaro has quit IRC19:54
*** marun_ has joined #openstack-qa19:55
lifelesso/19:57
*** e0ne has quit IRC20:06
mtreinishlifeless: \o20:08
*** Anju has quit IRC20:13
openstackgerritvigneshvar proposed a change to openstack-dev/devstack: Reset no_proxy when the node is installed behind a proxy  https://review.openstack.org/11798520:14
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1363103  https://review.openstack.org/11792320:19
*** sballe has quit IRC20:28
openstackgerritJeremy Stanley proposed a change to openstack-infra/elastic-recheck: Work toward Python 3.4 support and testing  https://review.openstack.org/11876720:28
*** cdent has quit IRC20:32
*** mwagner_lap has quit IRC20:34
*** nmagnezi has quit IRC20:37
*** cnesa10 has joined #openstack-qa20:37
*** adalbas has quit IRC20:43
*** sballe has joined #openstack-qa20:47
*** yfried has quit IRC20:50
*** markmcclain1 has joined #openstack-qa20:53
*** markmcclain has quit IRC20:54
*** toure has quit IRC20:57
openstackgerritMikhail S Medvedev proposed a change to openstack/tempest: Add neutron service tag  https://review.openstack.org/11888620:57
*** Sukhdev has joined #openstack-qa21:01
*** markmcclain1 has quit IRC21:01
*** markmcclain has joined #openstack-qa21:01
*** aysyd has quit IRC21:09
openstackgerritDavid Kranz proposed a change to openstack/tempest: Move success response checking to the token client  https://review.openstack.org/11762121:10
*** zz_johnthetubagu has quit IRC21:10
*** zz_johnthetubagu has joined #openstack-qa21:11
*** zz_johnthetubagu is now known as johnthetubaguy21:11
*** dpaterson has quit IRC21:16
*** dencaval has quit IRC21:16
*** melwitt has joined #openstack-qa21:17
*** melwitt has quit IRC21:22
*** melwitt has joined #openstack-qa21:23
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: Some scenario tests do not delete network resources  https://review.openstack.org/11889121:23
*** dhellmann is now known as dhellmann_21:28
mtreinishsalv-orlando: ^^^ I don't think that'll work, addCleanup doesn't work in setupClass...21:30
mtreinishoh, missed your comment, nm21:30
salv-orlandoyeah I know… I pushed it from the wrong machine21:31
salv-orlandoI initially did that, then moved the patch to another machine and fixed it, but then pushed from the 1st machine21:31
salv-orlandomtreinish: anyway, does it make sense? I don’t think it’s related to any gate issue, but still21:31
*** krtaylor has quit IRC21:32
mtreinishsalv-orlando: yeah, it's actually a bigger issue right now. It doesn't just happen in the scenario tests21:33
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: Some scenario tests do not delete network resources  https://review.openstack.org/11889121:33
salv-orlandomtreinish: from what I gather is a collateral of moving away from project client libraries to tempest clients?21:34
mtreinishsalv-orlando: it's the rationale behind the safe_setup (or whatever it's called) decorator21:34
salv-orlandomtreinish: oh. So I think in the medium term we should just stop creating network resources in isolated_creds?21:34
mtreinishsalv-orlando: well it's actually more of an issue caused by using unittest to run the tests. When something explodes it just kinda of stays there21:35
mtreinishlong term we need to fix it: https://review.openstack.org/#/c/118352/21:35
mtreinishsalv-orlando: can we get away with that? I thought we needed to allocate networks if we created a new tenant21:36
*** nsaje has quit IRC21:37
salv-orlandomtreinish: yes we do - it’s just a bit strange that a class called isolated_creds defines networks for tenants. However that’s fine for the time being. I thought this was somehting for safe_setup. But I see safe_setup is pretty much the addCleanup equivalent for setupClass21:38
mtreinishsalv-orlando: oh, I see what you're saying about the artifact of the client migration now after looking at the second rev of your patch21:40
salv-orlandomtreinish: some other scenario “manager” classes are either not using anymore net resources created by isolated creds or cleaning them up21:41
salv-orlandoanyway, I think the proposed blueprint makes sense as a medium term solution.21:41
mtreinishsalv-orlando: heh, so it's just the name, or did you want to split isolated_networks off into a separate class?21:42
mtreinishsalv-orlando: yeah hopefully we'll be able to make good progress on that soon21:42
salv-orlandoI think NeutronScenarioTest is already not using anymore the net resources in IsolatedCreds. If that’s correct and is the general direction then we should not worry about it anymore.21:43
mtreinishsalv-orlando: well there is only one test class using NeutronScenarioTest right now and it call set_network_resources() without any args21:45
mtreinishbut I thought that was the norm for neutron tests, because if they needed networking it was explicitly part of the test to create it21:45
mtreinishI feel like I'm missing some detail here...21:46
salv-orlandomtreinish: I seem to recall we did set_network_resources in a way that without params creates no resource21:46
*** mriedem has quit IRC21:46
mtreinishsalv-orlando: yep, that's how it works21:46
*** andreaf has quit IRC21:51
*** jecarey has quit IRC22:03
clarkbOSError: [Errno 24] Too many open files may be the root of our logstash trouble22:07
*** shardy is now known as shardy_z22:07
*** openstack has joined #openstack-qa22:11
mtreinishclarkb: heh, yeah that might cause a problem22:11
*** dkranz has quit IRC22:19
*** dkranz has joined #openstack-qa22:20
*** markmcclain has quit IRC22:20
*** dimtruck is now known as zz_dimtruck22:26
*** jgrimm is now known as zz_jgrimm22:29
*** krtaylor has joined #openstack-qa22:39
*** tkammer has joined #openstack-qa22:41
*** masayukig has quit IRC22:44
*** masayukig has joined #openstack-qa22:50
*** arnaud has quit IRC22:51
*** dims has quit IRC22:52
*** dims has joined #openstack-qa22:52
*** marun_ has quit IRC22:54
*** tkammer has quit IRC22:56
*** dims has quit IRC22:57
*** rbak has quit IRC22:58
*** Sukhdev has quit IRC23:00
*** masayukig has quit IRC23:07
*** masayukig has joined #openstack-qa23:09
*** masayukig_mobi has joined #openstack-qa23:10
*** masayukig_mobi has quit IRC23:13
*** mlavalle has quit IRC23:15
*** openstackstatus has quit IRC23:19
*** openstackstatus has joined #openstack-qa23:20
*** ChanServ sets mode: +v openstackstatus23:20
*** mlavalle_ has joined #openstack-qa23:21
openstackgerritAdam Gandelman proposed a change to openstack-dev/grenade: Set required DSG variables for Ironic DO NOT MERGE  https://review.openstack.org/11851523:22
openstackgerritSalvatore Orlando proposed a change to openstack/tempest: Test for helping root cause bug 1349617  https://review.openstack.org/11892523:22
*** ianw has quit IRC23:25
*** ianw has joined #openstack-qa23:25
*** coolsvap has quit IRC23:29
*** atiwari has quit IRC23:34
*** hemna is now known as hemna__23:36
*** coolsvap has joined #openstack-qa23:40
gmannSantosh_: hi23:43
*** mwagner_lap has joined #openstack-qa23:45
*** torandu has joined #openstack-qa23:52
*** mlavalle_ has quit IRC23:54

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