Friday, 2014-09-26

*** bknudson has quit IRC00:01
openstackgerritIan Wienand proposed a change to openstack-dev/devstack: Single quote iniset argument in merge_config_file  https://review.openstack.org/12422700:01
*** oomichi has joined #openstack-qa00:03
*** boden has joined #openstack-qa00:10
*** boden has quit IRC00:15
*** dims has quit IRC00:20
*** dims has joined #openstack-qa00:21
*** dims has quit IRC00:25
*** andreaf has quit IRC00:26
*** andreaf has joined #openstack-qa00:26
*** dmorita has joined #openstack-qa00:29
*** suneelb has quit IRC00:32
*** yamahata_ has joined #openstack-qa00:41
*** mmedvede has quit IRC00:43
*** zz_dimtruck is now known as dimtruck00:53
*** yamahata_ has quit IRC00:59
*** aimon has joined #openstack-qa01:00
*** catohornet has quit IRC01:02
*** jyuso has joined #openstack-qa01:07
*** dims has joined #openstack-qa01:07
*** dims_ has joined #openstack-qa01:09
*** boden has joined #openstack-qa01:11
*** dims has quit IRC01:13
*** boden has quit IRC01:15
*** arnaud__ has quit IRC01:23
*** arnaud has quit IRC01:23
*** Sukhdev has quit IRC01:23
openstackgerritDonald Stufft proposed a change to openstack-dev/devstack: WIP - DO NOT MERGE - Test pre-release setuptools  https://review.openstack.org/11948601:31
openstackgerritDavid Paterson proposed a change to openstack/tempest: Add tempest post-run cleanup script  https://review.openstack.org/11258101:31
openstackgerritDavid Paterson proposed a change to openstack/tempest: Add tempest post-run cleanup script  https://review.openstack.org/11258101:36
*** weshay has quit IRC01:40
openstackgerritDavid Paterson proposed a change to openstack/tempest: Add tempest post-run cleanup script  https://review.openstack.org/11258101:43
*** nosnos has joined #openstack-qa01:45
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Use shared values for test_security_group_rules  https://review.openstack.org/12400201:45
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Separate security group rule test for each args  https://review.openstack.org/12400301:45
*** salv-orlando has quit IRC01:45
*** JayJ_ has joined #openstack-qa01:46
*** JayJ_ has quit IRC01:46
*** JayJ_ has joined #openstack-qa01:47
*** HenryG has joined #openstack-qa01:49
*** mlavalle has quit IRC01:52
*** andreaf has quit IRC01:52
*** JayJ_ has quit IRC02:10
*** JayJ__ has joined #openstack-qa02:14
*** Poornima has joined #openstack-qa02:19
*** Sukhdev has joined #openstack-qa02:33
*** harlowja is now known as harlowja_away02:38
*** JayJ__ has quit IRC02:39
*** yamahata_ has joined #openstack-qa02:40
*** nosnos has quit IRC02:55
*** nosnos_ has joined #openstack-qa02:56
*** nosnos_ has quit IRC02:57
*** nosnos has joined #openstack-qa02:57
*** nosnos has quit IRC03:01
*** jyuso has quit IRC03:02
*** dims_ has quit IRC03:11
*** pcrews has quit IRC03:13
*** arnaud has joined #openstack-qa03:15
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Cleanup leftover dependencies to official clients  https://review.openstack.org/12401603:26
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop OfficialClientManager from tenant isolation  https://review.openstack.org/12401703:26
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop OfficialClientManager and references to it  https://review.openstack.org/12401803:26
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop client_type for auth module  https://review.openstack.org/12401903:27
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop OfficialClientTest and NetworkTest  https://review.openstack.org/12401303:27
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Migrate scenario utils to tempest client  https://review.openstack.org/12401503:27
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Rename NeutronScenarioTest to NetworkScenarioTest  https://review.openstack.org/12401403:27
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop python-*client from requirements  https://review.openstack.org/12425303:27
*** dimtruck is now known as zz_dimtruck03:31
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop python-*client from requirements  https://review.openstack.org/12425303:31
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Drop python-*client from requirements  https://review.openstack.org/12425303:36
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a server console-log test with unlimited length  https://review.openstack.org/12235103:40
*** pcrews has joined #openstack-qa03:42
*** zhiyan has quit IRC03:54
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Use shared values for test_security_group_rules  https://review.openstack.org/12400203:55
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Separate security group rule test for each args  https://review.openstack.org/12400303:55
*** zhiyan has joined #openstack-qa03:57
*** adalbas has quit IRC03:59
openstackgerritOpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements  https://review.openstack.org/11011004:00
*** arnaud has quit IRC04:01
*** Poornima has quit IRC04:04
*** yamahata_ has quit IRC04:08
*** yamahata_ has joined #openstack-qa04:08
*** dims has joined #openstack-qa04:12
*** om has joined #openstack-qa04:13
*** boden has joined #openstack-qa04:13
*** dims has quit IRC04:17
*** boden has quit IRC04:17
*** gmann_ has joined #openstack-qa04:24
*** nosnos has joined #openstack-qa04:28
*** Sukhdev has quit IRC04:33
*** Poornima has joined #openstack-qa04:42
*** Poornima has quit IRC04:43
*** edwa6643 has quit IRC04:56
*** Poornima has joined #openstack-qa04:59
*** boden has joined #openstack-qa05:14
*** Madan has joined #openstack-qa05:15
*** boden has quit IRC05:18
*** edwa6643 has joined #openstack-qa05:22
*** pcrews has quit IRC05:23
openstackgerritMasayuki Igawa proposed a change to openstack/tempest-lib: Remove unused method and variable in base.py  https://review.openstack.org/12427505:26
*** pcrews has joined #openstack-qa05:30
*** rushiagr_away is now known as rushiagr05:41
*** yamahata_ has quit IRC05:43
*** yamahata_ has joined #openstack-qa05:43
*** afazekas has joined #openstack-qa05:48
openstackgerritMasayuki Igawa proposed a change to openstack/tempest-lib: Remove a duplicate ignore rule of pep8  https://review.openstack.org/12032105:51
*** swamireddy has joined #openstack-qa05:53
*** SridharG has joined #openstack-qa05:58
openstackgerritMarc Koderer proposed a change to openstack/tempest: Add test to validate negative testing schema's  https://review.openstack.org/12003306:00
*** naggappan has joined #openstack-qa06:09
*** om has quit IRC06:12
*** boden has joined #openstack-qa06:14
openstackgerritShalini khandelwal proposed a change to openstack-dev/devstack: Corrected the position to get token in uploading image  https://review.openstack.org/12429806:16
swamireddyafazekas: Jenkins verfied with +1 for the patch https://review.openstack.org/#/c/115944/06:16
*** boden has quit IRC06:19
*** melwitt has quit IRC06:26
*** DinaBelova has quit IRC06:27
*** devananda has quit IRC06:29
*** swartulv has quit IRC06:29
*** SergeyLukjanov has quit IRC06:29
*** devananda has joined #openstack-qa06:31
*** swartulv has joined #openstack-qa06:31
*** om has joined #openstack-qa06:32
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add expected response checks of secgroup rules  https://review.openstack.org/12400506:33
*** SergeyLukjanov has joined #openstack-qa06:35
*** e0ne has joined #openstack-qa06:36
openstackgerritYuiko Takada proposed a change to openstack/tempest: Move baremetal API tests to valid file  https://review.openstack.org/12430506:36
*** DinaBelova has joined #openstack-qa06:37
*** k4n0 has joined #openstack-qa06:37
*** e0ne has quit IRC06:41
*** pcrews has quit IRC06:44
swamireddyafazekas:  Thanks for review with +206:46
*** Longgeek has joined #openstack-qa06:49
*** coolsvap|afk is now known as coolsvap06:55
openstackgerritYuiko Takada proposed a change to openstack/tempest: Add Ironic API tests  https://review.openstack.org/12430907:02
*** dims has joined #openstack-qa07:02
*** amotoki has joined #openstack-qa07:03
*** flaper87|afk is now known as flaper8707:06
*** dims has quit IRC07:07
*** Albert_13 has joined #openstack-qa07:12
*** boris-42 has quit IRC07:15
*** swartulv has quit IRC07:15
*** boden has joined #openstack-qa07:15
*** swartulv has joined #openstack-qa07:18
*** markus_z has joined #openstack-qa07:19
*** boden has quit IRC07:19
*** jordanP has joined #openstack-qa07:24
*** jordanP has quit IRC07:29
openstackgerritYuiko Takada proposed a change to openstack/tempest: Add Ironic driver API tests  https://review.openstack.org/12431307:30
*** jordanP has joined #openstack-qa07:31
*** jordanP has quit IRC07:36
*** jlanoux has joined #openstack-qa07:37
*** gfidente has joined #openstack-qa07:43
*** qba73 has joined #openstack-qa07:47
*** jordanP has joined #openstack-qa07:48
*** zz_johnthetubagu is now known as johnthetubaguy07:56
*** boden has joined #openstack-qa07:56
openstackgerritChristopher Yeoh proposed a change to openstack/tempest: Fix and re-enable test_disassociate_not_associated_floating_ip  https://review.openstack.org/12432308:00
*** boden has quit IRC08:00
*** oomichi has quit IRC08:11
*** jpich has quit IRC08:12
*** safchain has joined #openstack-qa08:13
*** gmann_ has quit IRC08:15
*** gszasz has joined #openstack-qa08:15
*** jlibosva has joined #openstack-qa08:17
openstackgerritA change was merged to openstack/tempest: Restoring dashboard tests  https://review.openstack.org/12328108:17
mkodererandreaf_: masayukig: hi folks, whats the matter with https://review.openstack.org/#/c/124014 .. can I +A?08:24
mkodererah it's missing a dependend patch I see08:27
*** Longgeek has quit IRC08:31
*** Longgeek has joined #openstack-qa08:35
*** aix has joined #openstack-qa08:49
*** boden has joined #openstack-qa08:51
*** naggappan has quit IRC08:56
*** e0ne has joined #openstack-qa09:02
*** Dafna has joined #openstack-qa09:07
*** yamahata_ has quit IRC09:23
openstackgerritClaudiu Belu proposed a change to openstack/tempest: Adds TestSecurityGroupsBasicOps created VM's security_groups check  https://review.openstack.org/11816709:25
*** masayukig_mob has joined #openstack-qa09:27
*** andreaf has joined #openstack-qa09:28
masayukig_mobandreaf: I just rebasesd for my patch. So I think there is no problem to +A.09:33
*** dmorita has quit IRC09:34
andreafmasayukig_mob, hi thanks for your patch I missed that :)09:36
andreafmasayukig_mob, I'll make sure all patches are merge-able and passing check09:37
masayukig_mobandreaf: np :)09:38
*** Longgeek has quit IRC09:38
openstackgerritA change was merged to openstack/tempest: Properly preserve trace on error during wait  https://review.openstack.org/12286109:41
*** Longgeek has joined #openstack-qa09:42
*** masayukig_mob has quit IRC09:43
*** jhenner has joined #openstack-qa09:48
*** vigneshvar has joined #openstack-qa09:54
*** jyuso497 has joined #openstack-qa10:05
*** nosnos has quit IRC10:09
*** nosnos has joined #openstack-qa10:09
*** Longgeek has quit IRC10:11
*** tkammer has joined #openstack-qa10:11
gmannandreaf: for admin cred  -  https://review.openstack.org/#/c/107685/40/tempest/common/accounts.py10:17
*** Longgeek has joined #openstack-qa10:17
gmannandreaf: I m not able to open BP link :). I agree with your point on it will be wait for admin tests in such cases10:18
sdaguelifeless: man... that bugs old. Honestly, I can't remember what I was thinking there. Let me ponder more.10:20
*** amotoki has quit IRC10:20
andreafgmann, yep I'm not sure there's a bp yet, only a spec: https://review.openstack.org/#/c/86967/10:24
*** amotoki has joined #openstack-qa10:24
*** vigneshvar has quit IRC10:26
*** boden has quit IRC10:32
*** vigneshvar has joined #openstack-qa10:32
*** jyuso497 has quit IRC10:33
lifelesssdague: ack10:34
*** gmann has quit IRC10:35
*** gmann has joined #openstack-qa10:36
gmannandreaf: I see. that is nice spec to handle admin tests if cred missing.10:36
andreafgmann: unfortunately the concept of admin is somewhat vague in openstack, as it really depends on what your policy files look like10:37
*** dims has joined #openstack-qa10:37
gmannandreaf: I think it will be fine to handle the invalid conf error and skip the admin tests which is actually part of that spec not account one :)10:37
andreafgmann: ok - and thanks for the review10:38
andreafgmann, hierarchical multi-tenancy will help making things a bit more tidy in terms of admin I think10:38
openstackgerritMarc Koderer proposed a change to openstack/tempest: Reduce complexity during import phase  https://review.openstack.org/11527310:39
gmannandreaf: yaa.10:39
openstackgerritA change was merged to openstack-dev/grenade: Allow BASE_RUN_SMOKE to be overridden by localrc  https://review.openstack.org/12353810:40
openstackgerritA change was merged to openstack-dev/grenade: javelin resources: add device for volumes  https://review.openstack.org/12274710:40
gmannandreaf:  https://review.openstack.org/#/c/107685/ looks fine for me now. just need rebase there.10:41
gmannandreaf: Thanks for clarification.10:41
andreafgmann, yes I have a few pending rebases to do - I'll take care of them later today10:42
*** e0ne has quit IRC10:57
openstackgerritIlya Shakhat proposed a change to openstack/tempest: Fix resource cleanup in server rescue tests  https://review.openstack.org/12435810:57
*** e0ne has joined #openstack-qa10:58
*** jhenner has quit IRC11:03
*** jhenner has joined #openstack-qa11:03
*** Longgeek has quit IRC11:15
sdaguelifeless: man... you still awake :)11:17
sdaguei didn't expect a response until tomorrow11:17
lifelesssdague: I am11:18
lifelesssdague: noodling on WSGI211:18
sdaguecool, what kind of noodling?11:18
lifelessttps://github.com/python-web-sig/wsgi-ng/issues/4#issuecomment-5693433611:19
openstackgerritIlya Shakhat proposed a change to openstack/tempest: Fix traceback in resource_cleanup of server rescue tests  https://review.openstack.org/12436411:20
sdaguecool11:21
lifelessand reading up on things like https://tools.ietf.org/html/draft-thomson-webpush-http2-0011:22
lifelesswhich really should be informed in tings like zaqar11:22
lifelessbasically, we can't use any of the modern cool shit in OpenStack until a bunch of upstream plumbing is fixed11:22
lifelessand I don't want us doing-it-badly :)11:23
lifeless(e.g. doing HTTP/1.x long poll brain deadness like comet rather than websockets or HTTP/2)11:23
*** Madan has quit IRC11:25
*** mkoderer_cloud has joined #openstack-qa11:26
lifelesssdague: however, tis bedtime now :)11:27
*** Longgeek has joined #openstack-qa11:29
sdaguehave a good night11:29
*** Poornima has quit IRC11:32
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Cleanup leftover dependencies to official clients  https://review.openstack.org/12401611:35
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Drop OfficialClientManager from tenant isolation  https://review.openstack.org/12401711:35
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Drop OfficialClientManager and references to it  https://review.openstack.org/12401811:35
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Drop client_type for auth module  https://review.openstack.org/12401911:35
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Drop OfficialClientTest and NetworkTest  https://review.openstack.org/12401311:35
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate scenario utils to tempest client  https://review.openstack.org/12401511:35
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Rename NeutronScenarioTest to NetworkScenarioTest  https://review.openstack.org/12401411:35
andreafsdague: I know the gate won't be happy about this ^^^ unfortunately it would not merge anymore I had to rebase11:37
andreafmtreinish, masayuki: I took the opportunity to change order of the patches I put the cleanup one first11:37
openstackgerritIlya Shakhat proposed a change to openstack/tempest: Fix resource cleanup in server rescue tests  https://review.openstack.org/12435811:41
*** tosky has joined #openstack-qa11:43
openstackgerritIlya Shakhat proposed a change to openstack/tempest: Fix traceback in resource_cleanup of server rescue tests  https://review.openstack.org/12436411:43
*** k4n0 has quit IRC11:46
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: remove old queries with no hits that reference closed bugs.  https://review.openstack.org/12437911:46
openstackgerritA change was merged to openstack-infra/elastic-recheck: Remove query for bug 1357677  https://review.openstack.org/12092511:49
*** dkranz has joined #openstack-qa11:54
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Add Credentials Provider factory  https://review.openstack.org/10768511:54
*** adalbas has joined #openstack-qa11:55
jordanPhi. I am having a hard time understanding https://review.openstack.org/#/c/124298/1 I've made several comments but I am starting to think that I am saying non-sense. Can someone give a quick look at the 2 lines patch plz ?11:56
*** cnesa14 has joined #openstack-qa11:57
*** cnesa13 has quit IRC11:57
*** mmedvede has joined #openstack-qa11:59
openstackgerritEmilien Macchi proposed a change to openstack/tempest: Javelin: enable volume resources  https://review.openstack.org/12164812:05
*** boden has joined #openstack-qa12:06
*** nosnos has quit IRC12:24
*** nosnos has joined #openstack-qa12:24
openstackgerritChris Dent proposed a change to openstack-dev/devstack: Allow ceilometer-api to run under mod_wsgi  https://review.openstack.org/12185012:28
*** nosnos has quit IRC12:29
*** JayJ__ has joined #openstack-qa12:31
*** coolsvap is now known as coolsvap|afk12:44
*** tkammer has quit IRC12:44
*** aysyanne has joined #openstack-qa12:45
*** vigneshvar has quit IRC12:56
*** dims has quit IRC12:58
*** Obulpathi has joined #openstack-qa12:59
*** dims has joined #openstack-qa12:59
*** tkammer has joined #openstack-qa12:59
*** gordc has joined #openstack-qa13:00
*** Obulpathi has quit IRC13:10
*** edwa66431 has joined #openstack-qa13:10
*** edwa6643 has quit IRC13:13
*** dhellmann_ is now known as dhellmann13:13
*** dustins has joined #openstack-qa13:14
*** yamahata_ has joined #openstack-qa13:19
openstackgerritA change was merged to openstack-infra/elastic-recheck: remove old queries with no hits that reference closed bugs.  https://review.openstack.org/12437913:20
*** mriedem has joined #openstack-qa13:21
openstackgerritMatt Riedemann proposed a change to openstack/tempest: POC: Add list-networks test for os-networks API  https://review.openstack.org/11294413:40
*** mestery has quit IRC13:40
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Add list-networks test for os-networks API  https://review.openstack.org/11294413:40
*** mkoderer_cloud has quit IRC13:43
*** rushiagr is now known as rushiagr_away13:43
*** mpaolino has joined #openstack-qa13:54
*** swamireddy has quit IRC13:54
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for encrypted volume delete race bug 1374458  https://review.openstack.org/12440713:57
mriedemsdague: ^13:57
*** mpaolino has quit IRC13:58
sdaguemriedem: did you look for the really long running blockdev call in there?14:00
mriedemdigging through the cinder logs now14:00
sdagueit's actually in the n-cpu log14:00
mriedemyeah, making my way there14:01
sdagueso, honestly, I suspect a bunch these are really all related to really long held locks14:01
mriedemthere are periodic tasks in the cinder-volume logs for that teardown/detach14:01
mriedemlots of locky locky going on14:01
sdagueyeh14:02
sdagueso for those, I found it was best to pull the log and grep out the right parts14:02
mriedemhttp://logs.openstack.org/98/124198/3/check/check-grenade-dsvm-icehouse/c89f18f/logs/new/screen-n-cpu.txt.gz?level=TRACE#_2014-09-26_03_32_17_40314:02
sdagueso you could see the sequencing14:02
mriedemit fails to detach the volume which is why it can't delete it and we timeout14:02
sdagueso shouldn't it retry that op?14:03
*** e0ne has quit IRC14:03
sdagueis that just optimistic coding14:03
mriedemhaven't gotten to the code yet14:04
mriedemsdague: i'm not even 1/2 through my first cup of coffee, c'mon! :)14:04
sdagueheh14:04
sdaguesorry, I'm on cup #3 :)14:04
*** e0ne has joined #openstack-qa14:05
mriedemi swear i've reported this bug before14:06
mriedemi might be thinking of https://bugs.launchpad.net/bugs/134820414:06
*** rbak has joined #openstack-qa14:06
mriedemah https://bugs.launchpad.net/cinder/+bug/1348204 was closed but it still shows up, so i will re-open14:07
*** e0ne has quit IRC14:09
*** e0ne has joined #openstack-qa14:10
*** yamahata_ has quit IRC14:10
*** andreaf has quit IRC14:16
*** andreaf has joined #openstack-qa14:17
sdaguemriedem: I did a big er query purge this morning14:18
sdague0 hit bugs with closed bug artifacts14:18
*** yamahata_ has joined #openstack-qa14:18
sdaguethere are still some 0 hit bugs up14:18
mriedemsdague: i saw that, thanks. i meant to do that on monday but forgot.14:18
*** dims has quit IRC14:19
mriedemsdague: i'll be posting a fix for the luks detach issue soon14:19
*** SridharG has left #openstack-qa14:19
sdagueso it needs another purge, but that one requires more investigation14:19
sdaguemriedem: nice14:19
mriedemlooks like we can key off the error code 5, it means the device is busy14:19
sdagueyep14:19
mriedemso i'll retry on that until timeout or something14:19
mriedembut should resolve the race14:19
sdagueyeh, that would be cool14:19
mriedemthe code doesn't have any retry loop14:19
sdagueit might14:19
sdagueit might be actually stuck forever, however a retry seems appropriate14:19
mriedemi think processutils has some built in kwarg to at least retry once14:20
mriedemwill check it out14:20
*** dims has joined #openstack-qa14:21
*** zz_jgrimm is now known as jgrimm14:23
*** dims has quit IRC14:25
*** jecarey has joined #openstack-qa14:26
*** andreaf has quit IRC14:26
*** andreaf has joined #openstack-qa14:27
*** jaypipes has joined #openstack-qa14:31
*** ChanServ sets mode: +o jaypipes14:31
*** dims has joined #openstack-qa14:32
*** dims has quit IRC14:32
*** dims has joined #openstack-qa14:33
*** dims has quit IRC14:34
*** dims has joined #openstack-qa14:35
*** dims has quit IRC14:35
*** jlibosva has quit IRC14:35
*** nsaje has joined #openstack-qa14:36
*** salv-orlando has joined #openstack-qa14:40
*** dims has joined #openstack-qa14:42
*** dims has quit IRC14:42
*** dimsum_ has joined #openstack-qa14:43
*** dimsum_ has quit IRC14:43
*** dimsum_ has joined #openstack-qa14:45
*** nikhil_k|afk is now known as nikhil_k14:52
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for scheduling race bug 1372093  https://review.openstack.org/12442814:54
mriedemsdague: ^ i've got a tempest patch up for that, but didn't have a query14:54
mriedembut keep hitting it14:54
*** nikhil_k is now known as nikhil_k|afk14:54
mtreinishmriedem: +A14:55
mriedemmtreinish: good morning L)14:55
*** mestery has joined #openstack-qa14:55
mriedemsdague: i'm going to add 1355929 back to e-r, just hit it on master14:57
mriedemand the bug wasn't closed14:57
mtreinishmriedem: heh, I've been online for 2 hrs... Just nothing to say until now :)14:59
*** marun has joined #openstack-qa15:01
*** hemnass is now known as hemna15:01
*** nikhil_k|afk is now known as nikhil_k15:02
*** Longgeek has quit IRC15:07
*** pcrews has joined #openstack-qa15:07
*** jaypipes is now known as leakypipes15:08
*** qba73 has quit IRC15:09
*** markus_z has quit IRC15:14
*** markmcclain has joined #openstack-qa15:15
*** jogo is now known as flashgordon15:17
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for encrypted volume delete race bug 1374458  https://review.openstack.org/12440715:19
*** afazekas has quit IRC15:19
*** anteaya has quit IRC15:23
*** Sukhdev has joined #openstack-qa15:23
*** mlavalle has joined #openstack-qa15:40
*** melwitt has joined #openstack-qa15:40
*** aimon has quit IRC15:41
*** andreaf has quit IRC15:41
*** andreaf has joined #openstack-qa15:42
*** JayJ__ has quit IRC15:43
*** JayJ__ has joined #openstack-qa15:44
andreafdkranz, sdague, cyeoh, mkoderer: the resource-cleanup series could help some more reviews: https://review.openstack.org/#/q/topic:+bp/resource-cleanup+status:+open,n,z15:46
andreafdkranz, sdague, cyeoh, mkoderer: also the post scenario migration cleanup patches are available for review: https://review.openstack.org/#/q/topic:+bp/tempest-client-scenarios+status:+open,n,z15:47
andreafboth series should be pretty straight-forward to review15:47
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add query for scheduling race bug 1372093  https://review.openstack.org/12442815:48
*** suneelb has joined #openstack-qa15:50
*** jordanP has quit IRC15:54
openstackgerritA change was merged to openstack-dev/devstack: Configure endpoints to use SSL natively or via proxy  https://review.openstack.org/9885415:58
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Force use of Accounts for test purposes  https://review.openstack.org/11457815:58
*** aimon has joined #openstack-qa15:59
openstackgerritA change was merged to openstack/tempest: Updates Tempest to match Zaqar's API name change  https://review.openstack.org/12325215:59
*** bnemec is now known as beekneemech16:00
andreafmtreinish: I rebased the patch to force the use of Accounts (the locking one) as well, so we can retest that as well. I that's ok, and the tenant isolation off seems to be ok, we should be ready to merge the patch that enabled the test-accounts code16:00
andreafmtreinish, the "force use of accounts" is https://review.openstack.org/#/c/114578/2116:01
mtreinishandreaf: ok, I was going to give it a +2 after the non-isolated jobs came back successful16:04
mtreinishbut I'm curious to see how far we get with the test patch16:04
*** aimon has quit IRC16:06
*** jlanoux has quit IRC16:07
jgriffithsdague: mriedem here :)16:08
jgriffithsdague: mriedem re lvm lockup16:08
jgriffithsdague: mriedem couple things come to mind:16:08
*** arnaud has joined #openstack-qa16:09
jgriffithsdague: mriedem first hunch was around IO scheduler16:09
jgriffithsdague: mriedem that's truly just a hunch, but I'm curious about the idea of setting IO scheduler to noop16:09
jgriffithsdague: mriedem as I think this might actually be a side effect of some other load issues on the systems16:09
jgriffithBUT16:09
jgriffithI came across something last night on some of our machines here16:10
*** zz_dimtruck is now known as dimtruck16:10
jgriffithsdague: mriedem https://bugs.launchpad.net/ubuntu/+source/linux/+bug/135423416:10
mriedemjgriffith: i've already got 2 patches up for attach/detach of encrypted volumes in nova16:10
sdaguehttp://logs.openstack.org/98/124198/3/check/check-grenade-dsvm-icehouse/c89f18f/logs/new/screen-n-cpu.txt.gz?level=TRACE#_2014-09-26_03_32_17_403 are we talking about this error?16:10
jgriffithmriedem: ahh... that's a diff issue :(16:10
mriedemjgriffith: detach https://review.openstack.org/#/c/124418/16:10
mriedemjgriffith: attach part 1 https://review.openstack.org/#/c/124448/16:10
jgriffithsdague: mriedem no, sorry16:11
jgriffithsdague: mriedem I'm referring to the syslog hang on lvs16:11
jgriffithcinder-vol just locks up16:11
sdagueoh, ok16:11
sdaguejgriffith: ok, so what did you discover16:11
jgriffithsdague: need me to find the bug or you remember it?16:11
sdaguejgriffith: no, I remember now16:12
mriedemdanpb also put notes in that bug16:12
jgriffithsdague: so... we have some systems here that started exhibiting similar behavior, certain kernel modules just "lock up"16:12
*** mestery has quit IRC16:12
jgriffithsdague: now TBF they're not LVM related at all, but I think the underlying kernel issue might be the root cause in both cases16:12
*** om has quit IRC16:12
jgriffiththe behavior in terms of the lock up is almost identical16:13
jgriffithsdague: the thing I can't find is exactly what kernel version the Instances in our gate use?16:13
sdagueis it dumped in one of the devstack-gate logs?16:14
sdagueif not, we can add it16:14
jgriffithsdague: I've been poking around a bit this mornign without luck16:14
sdaguethere are 2 kernel packages installed, so we can't infer from that16:14
*** suneelb has quit IRC16:14
jgriffithsdague: was hoping you might say.. "Yeah it's right here" :)16:14
sdagueyeh, we might not have a clean uname -a16:14
jgriffithsdague: might be worth adding16:15
jgriffithsdague: even if it's not an issue in this case, could be useful in the future IMO16:15
jgriffithsdague: I can try and emulate more closely what's in the gate locally16:15
*** Albert_13 has quit IRC16:15
jgriffithsdague: so far I haven't been able to trigger it16:16
openstackgerritMichele Paolino proposed a change to openstack-dev/devstack: Support for libvirt/QEMU tar releases  https://review.openstack.org/10871416:16
sdaguehttp://logs.openstack.org/01/123801/1/check/check-tempest-dsvm-full/b5f8b37/logs/devstack-gate-setup-host.txt.gz is where it would be if we have it16:16
jgriffithsdague: just running local CI against LVM16:16
sdagueif not, we should add that to the sanity check there16:16
mtreinishjgriffith: according to the dpkg list 3.13.0.35.42 is installed16:16
jgriffithmtreinish: ahh16:16
mtreinishand looking at the devstacklog I don't see it being installed so that's probably what's running16:16
openstackgerritboden proposed a change to openstack-infra/elastic-recheck: query for teardown bug 1317298  https://review.openstack.org/12445616:16
sdaguemtreinish: I thought there were usually 2 kernels installed though16:16
mtreinishsdague: oh, you're right. There are a few versions listed: http://logs.openstack.org/90/124190/1/check/check-tempest-dsvm-full/0c0fa01/logs/dpkg-l.txt.gz16:17
mtreinishso I guess that's not very helpful16:17
jgriffithso 3.13.0-36 is the "hopeful" kern16:18
jgriffithsigh... that complicates things a big16:18
jgriffithbit16:18
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: expose what kernel is running  https://review.openstack.org/12445816:18
jgriffithmtreinish: how often do we update images?16:18
mtreinishjgriffith: yeah at least on that job they were all 3.13.0-35.* or 3.13.0-30.*16:18
*** bkopilov has quit IRC16:19
mtreinishjgriffith: I belive nodepool creates a new image every night16:19
clarkbideally once a day. that slips occasionally16:19
clarkbbut I have dib working ish now so should get better16:19
jgriffithmtreinish: well yeah, but does it actually pull in a new cloud-image from Ubuntu?16:19
sdaguehmmmm16:19
sdagueWed, 03 Sep 2014 12:40:49 -040016:19
sdagueis the 3.13.0.36.43 release16:20
clarkbno it upgrades the images our providers give us16:20
clarkbdib does the same but with official images16:20
sdagueso that would make me think we've not updated in a long time?16:20
jgriffithclarkb: sdague I believe the kernel update was pretty recent16:21
jgriffithlike just the other day... but not sure16:21
sdaguethat was the changelog on it16:21
sdaguemaybe it was sitting in a testing queue for a bit16:21
jgriffithOk, looks like it just went out on the 22'nd16:22
*** eglynn is now known as eglynn-officeafk16:23
jgriffithmight just be a slight lag16:23
jgriffithsdague: clarkb so maybe adding an explicit uname -a somewhere in logging would be good16:23
mtreinishjgriffith: https://review.openstack.org/#/c/124458/16:23
clarkbI think sdague just pushed that change16:23
jgriffithand see if/when we start picking it up and if infact it is the culprit16:23
*** suneelb has joined #openstack-qa16:23
*** safchain has quit IRC16:24
jgriffithmtreinish: Rock On!!16:24
*** aimon has joined #openstack-qa16:24
jgriffithok, I'll remember to monitor this a bit16:24
jgriffithsee what's up, then go down the scheduler path if necessary16:24
jgriffithmtreinish: clarkb sdague thanks for the collab16:24
mtreinishjgriffith: btw, I forgot to ask if you saw my ping last week about: https://review.openstack.org/#/c/118619/16:25
sdaguejgriffith: no prob16:26
jgriffithmtreinish: I did not16:26
jgriffithummmm16:26
*** mriedem is now known as mriedem_burgers16:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate thirdparty API tests to resource_* fixtures  https://review.openstack.org/12155216:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate cli tests to resource_* fixtures  https://review.openstack.org/12155316:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Hacking rule to forbid resource unsafe fixtures  https://review.openstack.org/12155416:27
jgriffithsdague: no16:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Drop unused safe_setup decorator  https://review.openstack.org/12253116:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate telemetry API tests to resource_* fixtures  https://review.openstack.org/12154916:27
jgriffitherr...16:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate queuing API tests to resource_* fixtures  https://review.openstack.org/12154816:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate scenario API tests to resource_* fixtures  https://review.openstack.org/12155116:27
jgriffithmtreinish: no16:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate volume API tests to resource_* fixtures  https://review.openstack.org/12155016:27
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Migrate orchestration API tests to resource_* fixtures  https://review.openstack.org/12154716:27
andreafmtreinish, sdague: the zaquar rename forced me to rebase the resource-cleanup patch series, so running checks again now16:28
mtreinishjgriffith: basically I'm thinking that it's uneccesary, because it looks like it's just something broken with the coraid driver16:28
jgriffithmtreinish: for the most part yes16:28
jgriffithmtreinish: they shouldn't have to rely on that16:28
jgriffithmtreinish: I'll vote on it and try and catch up with them more16:28
jgriffithmtreinish: it's also a good candidate for lib_cinder/plugins IMO16:29
mtreinishjgriffith: ok thanks, I was going to -2 it but I wanted to get my facts straight first :)16:29
jgriffithwith a little trickery maybe16:29
jgriffithmtreinish: yeah, I think you're right on target16:29
jgriffithmtreinish: thanks for the heads up16:29
*** anteaya has joined #openstack-qa16:33
flashgordoncinder V2 API16:42
flashgordonnova isn't set up to use it in the gate16:43
flashgordonmtreinish jgriffith: is that intentional?16:43
jgriffithflashgordon: nope16:43
jgriffithflashgordon: suspected oversight16:43
flashgordon2014-09-25 01:13:53.121 WARNING nova.volume.cinder [req-4ee7615d-a6bb-4447-a972-c9c0aa3d4d99 EC2VolumesTest-1099875960 EC2VolumesTest-2025087518] Cinder V1 API is deprecated as of the Juno release, and Nova is still configured to use it. Enable the V2 API in Cinder and set cinder_catalog_info in nova.conf to use it.16:43
mtreinishflashgordon: can't you just change the default in nova?16:44
flashgordonmtreinish: good question16:45
flashgordonthis late in the cycle I don't think so16:45
mtreinishflashgordon: yeah that makes sense, I think we probably should just switch the default in nova when kilo opens. That isn't the kinda thing I feel we should change in devstack16:46
*** mkollaro has quit IRC16:47
flashgordonmtreinish: agreed, although its unavoidable16:48
flashgordonthe cinder_endpoint_template is overwritten16:48
flashgordonbecause this code is ugly16:48
flashgordon(nova's code)16:48
flashgordonoh that is only if ssl is enabled16:49
*** mestery has joined #openstack-qa16:51
*** mestery has quit IRC16:51
*** mestery has joined #openstack-qa16:52
openstackgerritJoe Gordon proposed a change to openstack-dev/devstack: Cleanup nova-cinder nova.conf section  https://review.openstack.org/12446416:52
*** Dafna has quit IRC16:53
flashgordontiny cleanup ^16:53
flashgordongoing to psuh a nova patch to set cinder v2 to test it out16:53
mtreinishflashgordon: heh, I like that: don't override default values with default values16:54
flashgordonmtreinish: yup :)16:54
*** andreaf has quit IRC16:55
*** andreaf has joined #openstack-qa16:56
flashgordongot a few more to fix http://paste.openstack.org/show/11565316:56
*** dencaval has joined #openstack-qa16:56
*** yamahata_ has quit IRC16:57
dencavalHey guys, Is there unicode test cases for swift (container and object names) in tempest.api?16:59
openstackgerritJoe Gordon proposed a change to openstack-dev/devstack: Stop setting nova.conf settings that mean nothing  https://review.openstack.org/12446517:01
mtreinishdencaval: for metadata I don't think there are any tempest tests which explicitly use unicode for swift.17:01
*** e0ne has quit IRC17:02
mtreinishflashgordon: ^^^ how were we setting the workers then?17:02
notmynamedencaval: I don't know about inside of tempest (if not there should be), but we do test that in the in-tree swift tests17:03
jgriffithflashgordon: sorry.... dropped off17:04
jgriffithflashgordon: so the update to Nova took a "long" time (like a year) to go in17:05
jgriffithand then went in just a month or so ago17:05
jgriffithI suspect the ball was dropped on the config piece17:05
flashgordonmtreinish: I don't think we were17:06
dencavalnotmyname maybe to add unicode test cases, tempest should handle unicode instead of ASCII, I made a small test, but got ASCII error...17:06
*** dimtruck is now known as zz_dimtruck17:07
mtreinishdencaval: there are unicode tests in tempest for other projects, so it should work fine17:07
openstackgerritFergal Mc Carthy proposed a change to openstack-dev/devstack: local.conf processing doesn't handle '=' in values  https://review.openstack.org/12446717:12
dencavalmtreinish I have changed test_upload_object to upload a object name with unicode character and got this error:17:13
dencavalmtreinish "SyntaxError: Non-ASCII character '\xc3' in file tempest/api/object_storage/test_object_services.py on line 111, but no encoding declared"17:13
flashgordonjgriffith: yeah its easy for these things to get lost17:13
flashgordonmtreinish jgriffith: https://review.openstack.org/124468 is the test17:13
dencavalmtreinish so I added this to the file: # -*- coding: utf-8 -*-17:13
flashgordonjgriffith: if that fails, we have issues17:14
dencavalmtreinish  And I got this error at object_client.py: UnicodeEncodeError: 'ascii' codec can't encode character u'\xe9' in position 94: ordinal not in range(128)17:14
*** gszasz has quit IRC17:17
openstackgerritJoe Gordon proposed a change to openstack-dev/devstack: don't set nova.conf auth_strategy  https://review.openstack.org/12446917:18
*** dhellmann is now known as dhellmann_17:18
sdagueflashgordon / mtreinish / mriedem_burgers17:20
*** om has joined #openstack-qa17:20
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: extract logging setup to separate file  https://review.openstack.org/12447017:20
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add verbose mode to graph cmd  https://review.openstack.org/12447117:20
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: update web pages to support additional date info  https://review.openstack.org/12447217:21
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add support for computing relevant dates  https://review.openstack.org/12447317:21
sdaguethat provides ES delay - http://dl.dropbox.com/u/6514884/screenshot_197.png17:21
*** imkarrer has joined #openstack-qa17:21
sdaguealso clarkb17:22
flashgordonjgriffith: while I have your attention https://bugs.launchpad.net/cinder/+bug/131729817:22
flashgordonsdague: nice ^17:23
*** tkammer has quit IRC17:23
*** Sukhdev has quit IRC17:23
sdagueI'm going to step away for a bit ... I think getting that in would reduce confusion a lot17:24
sdagueI've had to explain the delay issue .... a lot recently17:25
*** arnaud has quit IRC17:25
dimsum_sdague: wow, that's quite a bit17:26
sdagueyep17:26
imkarrerGood afternoon.  I have some questions about some of the Tempest tests for Swift.  Specifically the tests contained in test_account_quotas.py, test_account_quotas_negative.py, test_container_acl.py, and test_container_acl_negative.py.  During the SetUp phase of these tests new users are created.  What auth system is expected when creating these new users?  It appears it is not Keystone.17:28
*** Sukhdev has joined #openstack-qa17:30
*** dhellmann_ is now known as dhellmann17:36
*** johnthetubaguy is now known as zz_johnthetubagu17:45
*** jecarey has quit IRC17:45
*** nsaje has quit IRC17:47
*** arnaud has joined #openstack-qa17:53
*** arnaud__ has joined #openstack-qa17:55
mtreinishdencaval: here's an example: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/images/test_images_oneserver.py#n10817:57
*** sballe_ has joined #openstack-qa17:59
*** harlowja_away is now known as harlowja17:59
mtreinishimkarrer: it only uses keystone18:01
mtreinishandreaf: were there any issues with: https://review.openstack.org/118418 or was it just the weird gerrit formmating?18:12
*** tosky has quit IRC18:13
openstackgerritMatthew Treinish proposed a change to openstack-infra/subunit2sql: Add subunit2sql-db-manage utility  https://review.openstack.org/11841818:14
*** e0ne has joined #openstack-qa18:15
*** e0ne has quit IRC18:16
*** e0ne has joined #openstack-qa18:20
*** e0ne has quit IRC18:24
*** Sukhdev has quit IRC18:27
openstackgerritA change was merged to openstack-infra/elastic-recheck: query for teardown bug 1317298  https://review.openstack.org/12445618:27
*** e0ne has joined #openstack-qa18:28
*** e0ne has quit IRC18:30
*** mriedem_burgers is now known as mriedem18:32
sdaguegah... import error18:35
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: extract logging setup to separate file  https://review.openstack.org/12447018:35
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add verbose mode to graph cmd  https://review.openstack.org/12447118:35
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: update web pages to support additional date info  https://review.openstack.org/12447218:35
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add support for computing relevant dates  https://review.openstack.org/12447318:35
*** andreaf has quit IRC18:36
*** andreaf has joined #openstack-qa18:37
*** zz_dimtruck is now known as dimtruck18:39
*** e0ne has joined #openstack-qa18:41
*** e0ne has quit IRC18:49
*** e0ne has joined #openstack-qa18:50
*** amotoki has quit IRC18:52
*** Sukhdev has joined #openstack-qa18:52
*** marun has quit IRC18:55
*** dhellmann is now known as dhellmann_18:57
*** e0ne has quit IRC19:01
*** markmcclain has quit IRC19:01
*** sabeen3 has quit IRC19:02
*** e0ne has joined #openstack-qa19:05
*** sabeen1 has joined #openstack-qa19:07
*** e0ne has quit IRC19:11
*** rushiagr_away is now known as rushiagr19:11
andreafmtreinish: it looks good. perhaps a sample config file would be nice, but it can be added later19:12
*** e0ne has joined #openstack-qa19:15
*** e0ne has quit IRC19:18
*** gordc has quit IRC19:19
*** andreaf has quit IRC19:21
*** e0ne has joined #openstack-qa19:22
*** andreaf has joined #openstack-qa19:22
*** e0ne has quit IRC19:26
*** boris-42 has joined #openstack-qa19:27
*** rushiagr is now known as rushiagr_away19:29
mtreinishandreaf: ok thanks, I'll merge it then. I'll push out a patch to do the sample config soon. I wonder if how the generator will work with 3 different sets of cli options19:30
*** e0ne has joined #openstack-qa19:31
*** e0ne has quit IRC19:35
*** sabeen3 has joined #openstack-qa19:36
*** sabeen1 has quit IRC19:38
*** yjiang5 has joined #openstack-qa19:43
*** andreaf has quit IRC19:44
*** andreaf has joined #openstack-qa19:45
openstackgerritA change was merged to openstack-infra/subunit2sql: Add subunit2sql-db-manage utility  https://review.openstack.org/11841819:49
openstackgerritBaodong (Robert) Li proposed a change to openstack-dev/devstack: Preserve quotes and allow multi-line config items in meta-section of local.conf  https://review.openstack.org/12450219:52
*** dhellmann_ is now known as dhellmann19:55
*** gordc has joined #openstack-qa19:55
*** mmedvede has quit IRC19:56
*** om has quit IRC20:06
*** boden has quit IRC20:10
*** boden has joined #openstack-qa20:10
*** boden has quit IRC20:15
openstackgerritBaodong (Robert) Li proposed a change to openstack-dev/devstack: Preserve quotes and allow multi-line config items in meta-section of local.conf  https://review.openstack.org/12450220:16
flashgordonmtreinish: I think https://review.openstack.org/#/c/119894 can be abandoned20:17
mtreinishflashgordon: ok, done20:18
flashgordonmtreinish: noticed it when going over the open gate bug fixes20:19
*** aix has quit IRC20:21
*** gordc has left #openstack-qa20:24
*** markmcclain has joined #openstack-qa20:26
*** jhenner has quit IRC20:28
*** JayJ__ has quit IRC20:30
*** rwsu has quit IRC20:31
*** JayJ__ has joined #openstack-qa20:31
*** mmedvede has joined #openstack-qa20:31
*** morganfainberg is now known as morgan20:36
*** morgan is now known as morganfainberg20:37
*** dustins has quit IRC20:42
*** Sukhdev has quit IRC20:52
*** jgrimm is now known as zz_jgrimm20:52
sdagueflashgordon: hey, would you want to do a future bootstrapping hour on hacking and writing project specific hacking rules?20:55
sdaguethat feels like one of those arcane things that about 12 people know how to do, and others might find useful20:55
*** dimsum_ has quit IRC20:56
*** dimsum_ has joined #openstack-qa20:56
*** markmcclain1 has joined #openstack-qa20:57
mtreinishsdague: you can spend an hour on that?20:57
*** markmcclain has quit IRC20:58
flashgordonsdague: sure20:58
sdagueif you showed a number of examples, what to do, what not to do, for sure20:58
flashgordonsdague: where is this weeks episode btw20:58
sdaguewe're really shooting for 30 minutes content + questions20:59
sdagueflashgordon: we're doing bi-weekly20:59
sdaguethe next topic will be announced soon20:59
flashgordonahh20:59
flashgordonsdague: yeah I would be happy to help with a hacking one21:00
flashgordongetting more people who know that code would be useful21:00
sdaguemostly, it's enough organizing effort, we decided every other week would keep us from getting burned out with it :)21:00
*** dimsum_ has quit IRC21:00
*** sabeen3 has quit IRC21:04
flashgordonmakes sense yeah every week would be a lot21:09
*** dimtruck is now known as zz_dimtruck21:12
*** mestery has quit IRC21:14
*** mestery has joined #openstack-qa21:15
*** Sukhdev has joined #openstack-qa21:17
*** dimsum_ has joined #openstack-qa21:21
lifelessflashgordon: please spend an hour on making hacing not be needed :)21:25
flashgordonlifeless: heh autopep8 isn't bad actually I still want to add a plug into to that21:25
*** markmcclain1 has quit IRC21:27
sdaguelifeless: it's still pretty handy for project specific rules21:28
sdagueeven if the base rules all went away21:29
*** mriedem has quit IRC21:31
*** andreaf has quit IRC21:31
*** andreaf has joined #openstack-qa21:32
lifelesssdague: my objection is when we make humans second guess an algorithm21:35
lifelesssdague: we should just have the algorithm output the right content21:35
lifelesssdague: I don't care /where/ the implementation lives ;)21:35
sdaguethe algorithm might not be formatting21:35
sdaguein nova we use hacking rules to block certain code from getting called from certain modules21:36
sdaguefor instance21:36
*** jhenner has joined #openstack-qa21:40
*** boden has joined #openstack-qa21:41
*** Sukhdev_ has joined #openstack-qa21:44
*** marun has joined #openstack-qa21:45
*** boden has quit IRC21:45
*** Sukhdev has quit IRC21:46
*** zz_dimtruck has quit IRC21:48
*** leakypipes has quit IRC21:49
*** nikhil_k is now known as nikhil_k|afk21:53
*** boden has joined #openstack-qa21:57
*** aysyanne has quit IRC21:58
*** boden has quit IRC22:01
*** imkarrer has quit IRC22:02
*** imkarrer has joined #openstack-qa22:02
EmilienMflashgordon: my javelin/cinder patch is ready for review I think: https://review.openstack.org/#/c/121648/22:12
*** arnaud__ has quit IRC22:14
*** arnaud has quit IRC22:14
*** arnaud has joined #openstack-qa22:14
*** arnaud__ has joined #openstack-qa22:14
*** andreaf has quit IRC22:19
*** andreaf has joined #openstack-qa22:19
*** Sukhdev_ has quit IRC22:23
*** Sukhdev has joined #openstack-qa22:31
*** jhenner has quit IRC22:31
*** dhellmann is now known as dhellmann_22:42
*** dimsum_ has quit IRC22:43
*** dimsum_ has joined #openstack-qa22:44
*** rwsu has joined #openstack-qa22:47
*** dimsum_ has quit IRC22:48
*** pcrews has quit IRC22:54
*** boden has joined #openstack-qa22:58
*** boden has quit IRC23:02
flashgordonEmilienM: reviewed23:05
EmilienMflashgordon: super cool, have a nice week-end !23:06
flashgordonEmilienM: you too23:07
*** dimsum_ has joined #openstack-qa23:07
*** yamahata_ has joined #openstack-qa23:09
*** JayJ__ has quit IRC23:20
*** arnaud__ has quit IRC23:23
*** arnaud has quit IRC23:23
*** hemna is now known as hemnass23:26
*** rwsu has quit IRC23:30
openstackgerritA change was merged to openstack/tempest: Add response schema for V2 sec grp default rule  https://review.openstack.org/11998623:35
*** gfidente has quit IRC23:39
*** pcrews has joined #openstack-qa23:43
*** rbak has quit IRC23:49
*** aimon has quit IRC23:50
*** marun has quit IRC23:50
flashgordonjgriffith: looks like cinderv2 works with nova :) https://review.openstack.org/#/c/124468/23:54
*** jeblair is now known as corvus23:58
*** boden has joined #openstack-qa23:59

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