Wednesday, 2015-09-09

*** achanda has quit IRC00:04
*** mriedem_away has quit IRC00:06
*** mriedem has joined #openstack-qa00:08
*** mriedem1 has joined #openstack-qa00:09
*** mriedem has quit IRC00:12
*** mwagner_lap has joined #openstack-qa00:17
*** zz_dimtruck is now known as dimtruck00:17
*** achanda has joined #openstack-qa00:26
oomichimtreinish: hi, are you still there?00:31
*** jhenner has quit IRC00:31
oomichimtreinish: I'd like to release a new tempest-lib now.00:31
*** yamamoto has joined #openstack-qa00:32
*** yamamoto has quit IRC00:37
*** ranger81 has joined #openstack-qa00:38
openstackgerritJamie Lennox proposed openstack/tempest: Give access to CredClient in tests  https://review.openstack.org/21498700:41
*** jasonsb has joined #openstack-qa00:41
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Make create_server use **kwargs  https://review.openstack.org/21746800:41
*** Sukhdev has quit IRC00:43
*** dims__ has quit IRC00:46
oomichimasayukig: I am glad if you can take time for https://review.openstack.org/#/c/221216/00:47
oomichimasayukig: and https://review.openstack.org/#/c/220391/00:47
*** dims__ has joined #openstack-qa00:47
*** swamireddy has quit IRC00:48
*** hockeynut_afk has quit IRC00:48
*** lbragstad has quit IRC00:48
*** BadCub has quit IRC00:48
*** eglute_s has quit IRC00:48
*** dimtruck has quit IRC00:48
*** wendar has quit IRC00:48
*** Nakato has quit IRC00:48
*** mattoliverau has quit IRC00:48
*** wendar has joined #openstack-qa00:49
*** vaidy_ril has quit IRC00:49
*** otherwiseguy has quit IRC00:49
*** jamielennox has quit IRC00:49
*** flwang has quit IRC00:49
*** jaypipes has quit IRC00:50
*** StevenK has quit IRC00:50
*** pvaneck has quit IRC00:50
*** comstud has quit IRC00:50
*** jroll has quit IRC00:50
*** ttx has quit IRC00:50
*** Nakato has joined #openstack-qa00:50
*** mgagne has quit IRC00:50
*** mattoliverau has joined #openstack-qa00:51
*** BadCub has joined #openstack-qa00:51
*** StevenK has joined #openstack-qa00:51
*** dtroyer has quit IRC00:51
openstackgerritTakashi NATSUME proposed openstack/tempest: Add container and disk format parameters  https://review.openstack.org/21106500:52
*** ttx has joined #openstack-qa00:53
*** dimtruck has joined #openstack-qa00:53
*** otherwiseguy has joined #openstack-qa00:53
*** ranger81 has quit IRC00:53
*** jroll has joined #openstack-qa00:53
*** eglute has joined #openstack-qa00:53
*** comstud has joined #openstack-qa00:53
*** hockeynut has joined #openstack-qa00:53
*** mgagne has joined #openstack-qa00:54
*** jamielennox has joined #openstack-qa00:54
*** lbragstad has joined #openstack-qa00:54
*** dtroyer has joined #openstack-qa00:54
*** swamireddy has joined #openstack-qa00:54
*** flwang has joined #openstack-qa00:56
*** zhangjn has joined #openstack-qa00:56
*** vaidy_ril has joined #openstack-qa00:56
*** jaypipes has joined #openstack-qa00:56
*** ChanServ sets mode: +o jaypipes00:56
*** zhangjn has quit IRC00:58
*** tpeoples has quit IRC00:59
*** cody-somerville has quit IRC01:00
*** zhangjn has joined #openstack-qa01:06
*** rossella_s has quit IRC01:06
*** rossella_s has joined #openstack-qa01:06
*** mlavalle has quit IRC01:09
*** zhangjn_ has joined #openstack-qa01:11
*** zhangjn has quit IRC01:14
*** mriedem1 is now known as mriedem_away01:15
openstackrecheckConsole logs not available after 13:22s for gate-tempest-dsvm-full-test-accounts 165233,22,ba1d95101:21
*** dimtruck is now known as zz_dimtruck01:24
*** salv-orlando has joined #openstack-qa01:29
*** cody-somerville has joined #openstack-qa01:33
*** cody-somerville has joined #openstack-qa01:33
*** salv-orlando has quit IRC01:36
*** salv-orlando has joined #openstack-qa01:43
*** salv-orlando has quit IRC01:47
*** armax has joined #openstack-qa01:59
mtreinishoomichi: ok, one sec I'll push it out right now02:01
*** sabeen has joined #openstack-qa02:05
oomichimtreinish: cool, thanks so much02:05
*** zz_dimtruck is now known as dimtruck02:05
*** sabeen has quit IRC02:06
mtreinishoomichi: ok, the tag is pushed it's working its way through the gate now02:06
mtreinishoomichi: fwiw, you can just push the tag yourself :)02:06
oomichimtreinish: yeah, I can but I am the second ;)02:07
oomichimtreinish: tempest-lib affects a lot of probject gates02:07
oomichimtreinish: can we check the effect of new tempest-lib release before releasing?02:08
oomichimtreinish: I'd like to check the effect on Tempest gate which contains a lot of gate jobs.02:09
oomichimtreinish: before releasing02:09
oomichimtreinish: if possible02:09
mtreinishoomichi: oh, well constraints will protect us from breaking the world now on master02:09
mtreinishand stable has tempest-lib capped (except for tempest which is venv isolated)02:10
mtreinishand we know tempest still works because we gate a neutron job on tempest master02:10
mtreinishs/tempest master/tempest-lib changes02:10
mtreinishI think we might have a gap if we forgot to flip the venv only switch on stable/kilo, but that's simple enough to fix02:11
mtreinishand mriedem_away will catch it when he's back :)02:11
mtreinishoomichi: I mean if you want to push an infra job to add experimental jobs to tempest-lib for a bunch of different gate jobs that use tempest-lib feel free02:12
mtreinishthat way we can test a patch that we're going to use as release before we push a tag02:12
mtreinishoomichi: https://pypi.python.org/pypi/tempest-lib/0.8.002:13
*** yamamoto has joined #openstack-qa02:15
oomichimtreinish: yeah, we can always test the latest tempest-lib on a neutron job. my concern is we forgot effects to the other jobs.02:16
oomichimtreinish: but maybe a neutron job can cover most cases.02:16
*** tpeoples has joined #openstack-qa02:16
mtreinishoomichi: sure, I mean I'm ok with that risk, but if we wanted it wouldn't be too hard to add experimental jobs to tempest-lib to run other job types02:16
oomichimtreinish: I wanted to know if you have some magic for knowing the impact before releasing ;)02:16
mtreinishoomichi: heh, I just push the tag and cross my fingers02:17
mtreinishI've only broken things once, maybe twice :)02:17
oomichimtreinish: ok, I got it. and I can release a new version after now02:17
oomichimtreinish: :)02:18
oomichimtreinish: so I will release it morning for recovering some bad release quickly02:18
gmannmtreinish: In free time, can you please have a look - https://review.openstack.org/#/c/219169/02:18
oomichimtreinish: thanks for a new version, anyway :)02:19
oomichimtreinish: it is nice idea to add experimental jobs to tempest-lib02:19
*** yamahata has quit IRC02:19
openstackgerritYAMAMOTO Takashi proposed openstack/tempest: Add a simple test for port security vs security group  https://review.openstack.org/22122002:20
mtreinishoomichi: fwiw, to a certain extent I'm not even sure how many jobs are using tempest-lib. That's part of the fun of having a library02:21
mtreinishpeople use it and you have no idea :)02:21
mtreinishgmann: so, I'm looking at: https://review.openstack.org/#/c/200486/10/tempest/clients.py and from what I can tell the old config opt didn't do anything02:21
mtreinish(that's the patch which split up the options)02:21
*** yamamoto has quit IRC02:22
gmannmtreinish: actually we had default value of old config option 'endpoint_type' as publicURL02:23
gmannmtreinish: and our new option were made as depreciated opt for that old one02:23
oomichimtreinish: yeah, the fun and terrifying part as a library02:24
mtreinishgmann: oh, I see what you're saying. So if someone had set that to something else assuming it was publicurl even though it didn't do anything when the run new tempest with old config it breaks02:24
mtreinishgmann: ok, I'm fine with the patch then02:24
gmannmtreinish: so if old users have uncommented the old option 'endpoint_type = publicURL' in old version of config then, that will fail with new config02:24
gmannmtreinish: yea :)02:25
openstackgerritReedip proposed openstack/tempest: Added Test Case for InterfacesClient  https://review.openstack.org/21813302:25
gmannmtreinish: Thanks02:25
mtreinishoomichi: just fyi, constraints will block the new tempest-lib release from showing up in the gate job, you need to bump it in g-r first02:29
oomichimtreinish: oops, got it. and forgot it when recheck02:30
mtreinishgmann: I'll trade you: https://review.openstack.org/221330 and https://review.openstack.org/21643802:32
gmannmtreinish: sure, ll do.02:33
openstackgerritMatthew Treinish proposed openstack/tempest: Only add roles for enabled services  https://review.openstack.org/21652202:36
openstackgerritMatthew Treinish proposed openstack/tempest: Fix test_port_list_filter_by_ip with test_accounts  https://review.openstack.org/21688702:36
openstackgerritMatthew Treinish proposed openstack/tempest: Deprecate credential config options  https://review.openstack.org/21160102:36
mtreinishgmann: cool, thanks they're both simple doc changes02:37
oomichimtreinish: g-r patch is https://review.openstack.org/#/c/22157602:37
mtreinishgmann: if you get more adventurous ^^^ those 3 are to start the deprecation timer for the credential config options02:38
gmannmtreinish: yup02:38
*** amotoki has joined #openstack-qa02:38
*** woodster_ has quit IRC02:39
mtreinishoomichi: also upper-constraints.txt will need to be bumped. That's actually what's holding it at the lower version02:39
mtreinishoomichi: I think there is a bot which updates that periodically, but I think you can do it manually too02:39
* mtreinish hasn't had to deal with it yet02:40
mtreinishoomichi: the other thing is I think we might be in requirements freeze now because we're post L-302:41
* mtreinish checks02:41
oomichimtreinish: thanks for quick review, done02:41
mtreinishoomichi: err, I don't know where to look for that anymore02:42
mtreinishheh, I'm sure the g-r reviewers will let you know if we're in freeze :)02:42
oomichimtreinish: I think tempest-lib should be an exception from requirements freeze because tempest wll help improving quality02:43
oomichimtreinish: during -rc time02:43
mtreinishoomichi: it wasn't last year. We were blocked doing using the lib at all until the juno rc period on adding tempest-lib to requirements02:44
mtreinishoomichi: but I agree02:44
*** zhangjn has joined #openstack-qa02:46
*** zhangjn_ has quit IRC02:49
oomichimtreinish: ok, I will write the NOTE on the commit message to try getting an exception ;-)02:49
openstackgerritTakashi NATSUME proposed openstack/tempest: Use container/disk format settings in glance v1 test  https://review.openstack.org/21877502:51
*** dims_ has joined #openstack-qa02:56
*** dims__ has quit IRC02:58
*** salv-orlando has joined #openstack-qa03:03
*** ranger81 has joined #openstack-qa03:05
*** armax has quit IRC03:05
*** dimtruck is now known as zz_dimtruck03:08
*** yamamoto has joined #openstack-qa03:08
*** zhangjn_ has joined #openstack-qa03:09
*** ranger81 has quit IRC03:09
*** salv-orlando has quit IRC03:10
*** mylu_ has quit IRC03:12
*** zhangjn has quit IRC03:13
*** mylu has joined #openstack-qa03:15
*** zhangjn has joined #openstack-qa03:16
*** zhangjn_ has quit IRC03:19
*** armax has joined #openstack-qa03:22
*** zz_dimtruck has quit IRC03:29
*** ranger81 has joined #openstack-qa03:31
piyush_i have added the functionality of namespaces api https://review.openstack.org/#/c/217555/ please have a look..03:31
*** armax has quit IRC03:33
*** zhangjn_ has joined #openstack-qa03:36
*** zz_dimtruck has joined #openstack-qa03:38
openstackrecheckopenstack/tempest change: https://review.openstack.org/219169 failed gate-tempest-python34, gate-tempest-python27 in the gate queue with an unrecognized error03:38
*** zz_dimtruck is now known as dimtruck03:38
*** zhangjn has quit IRC03:40
mtreinishoomichi, gmann: so I think the tempest-lib release broke the tempest unit tests: http://logs.openstack.org/69/219169/1/gate/gate-tempest-python27/ddc6955/console.html#_2015-09-09_02_30_44_70403:41
mtreinishI don't have time to debug it right now (I shouldn't really be online right now)03:42
oomichimtreinish: ok, will see it soon03:42
*** fedexo has joined #openstack-qa03:42
mtreinishbut if you guys could take a look, I'll take over anything in motion when I'm online in the morning03:42
mtreinishoomichi: cool, thanks03:42
oomichimtreinish: no problem, thanks for working hardly in night03:43
*** ajmiller has joined #openstack-qa03:43
gmannmtreinish: Sure, have gn03:44
oomichioops, that is gmann's patch03:44
*** rbak has joined #openstack-qa03:46
*** rbak has quit IRC03:48
*** rbak has joined #openstack-qa03:49
*** dave-mccowan has quit IRC03:51
*** achanda has quit IRC03:53
*** ajmiller has quit IRC03:54
*** fedexo has quit IRC03:56
*** gamado has joined #openstack-qa03:58
*** salv-orlando has joined #openstack-qa03:59
openstackrecheckopenstack/tempest change: https://review.openstack.org/220391 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1355573, gate-tempest-python27: unrecognized error, gate-tempest-python34: unrecognized error03:59
openstackLaunchpad bug 1355573 in grenade "grenade volume scenarios periodically fail /w SSHTimeout" [Undecided,New]03:59
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130003:59
openstackgerritYAMAMOTO Takashi proposed openstack/tempest: Add a simple test for port security vs security group  https://review.openstack.org/22122004:00
*** salv-orlando has quit IRC04:04
*** alex_xu has quit IRC04:04
*** alex_xu has joined #openstack-qa04:05
openstackgerritKen'ichi Ohmichi proposed openstack/tempest: Switch using token_client to the latest for gate  https://review.openstack.org/22158804:13
oomichimtreinish: gmann: https://review.openstack.org/221588 is for fixing them.04:13
*** dims_ has quit IRC04:13
oomichibut to be honest, I cannot understand why stubs doesn't work for inheritated classes04:14
oomichianyway, I'd like to wait for the gate job because I didn't run all unit test locally. just some tests which failed04:15
gmannoomichi: yea me too, stub should work for that too04:15
*** armax has joined #openstack-qa04:16
oomichigmann: yeah,04:17
gmannoomichi: checking on oslotests for stub04:17
oomichigmann: mtreinish: anyway, the gate job passed on https://review.openstack.org/#/c/221588/04:17
gmannoomichi: but if someone else try to stub old client class?04:18
oomichigmann: yeah, nice point. if someone else also is using tempest-lib on the same way, they face the same problem04:19
gmannoomichi: yea04:19
gmannoomichi: m +2 on that to unblock gate and meanwhile i will try to find why stub did not work04:21
oomichigmann: thanks:-)04:21
*** achanda has joined #openstack-qa04:28
*** Sukhdev has joined #openstack-qa04:28
*** SridharG has joined #openstack-qa04:29
openstackgerritAnusha Ramineni proposed openstack/tempest: Full response for missed IdentityV2Client methods  https://review.openstack.org/21610904:30
*** jswarren has quit IRC04:30
*** ranger81 has quit IRC04:32
oomichigmann: oslotest seems to use mox for stubs instead of mock internally04:33
gmannoomichi: yea, they use mox304:33
oomichigmann: if we use mock instead of stub on unit tests, can we avoid that?04:34
gmannoomichi: may be but i think that should work for mox3 also04:34
oomichigmann: I didn't try it yet04:34
gmannoomichi: i can try quickly04:34
oomichigmann: thanks again04:35
*** Poornima has joined #openstack-qa04:37
openstackgerritAnusha Ramineni proposed openstack/tempest: Full response for missed IdentityV2Client methods  https://review.openstack.org/21610904:38
*** ranger81 has joined #openstack-qa04:38
oomichimkoderer: hello04:39
*** yfried_ has quit IRC04:41
openstackgerritgaryk proposed openstack-dev/devstack: VMware: remove configuration integration_bridge from nova  https://review.openstack.org/22159204:41
*** mylu has quit IRC04:43
*** fedexo has joined #openstack-qa04:43
*** salv-orlando has joined #openstack-qa04:55
*** tpeoples has quit IRC04:57
*** salv-orlando has quit IRC04:58
*** viktor-tikkanen has joined #openstack-qa05:05
*** rossella_s has quit IRC05:06
*** rossella_s has joined #openstack-qa05:06
*** BharatK has joined #openstack-qa05:08
*** zhangjn_ has quit IRC05:11
gmannoomichi: mtreinish : got the issue05:15
gmannoomichi: actually there is no issue with inherit class stub (raw_request was anyways rest_client.py method)05:16
gmannoomichi: actually we changed the auth provider class to use new class name - https://review.openstack.org/#/c/198930/11/tempest_lib/auth.py05:16
gmannoomichi: and due to that request start going through TokenClient not TokenClientJSON and thats why stub did not work05:17
gmannstub was done on TokenClientJSON class05:17
gmannoomichi: i think we should keep using old class in auth.py and change to new once we remove the old class ?05:18
gmannwhat you say?05:18
*** gamado has quit IRC05:25
*** dimtruck is now known as zz_dimtruck05:26
*** ranger81 has quit IRC05:29
*** avico has quit IRC05:29
*** avico has joined #openstack-qa05:32
*** BharatK has quit IRC05:34
*** yfried_ has joined #openstack-qa05:35
*** ekarlso- has joined #openstack-qa05:37
*** salv-orlando has joined #openstack-qa05:38
openstackgerritHirofumi Ichihara proposed openstack-dev/devstack: Provide hook for neutron plugin config  https://review.openstack.org/22114605:38
*** yfried__ has joined #openstack-qa05:38
*** fedexo has quit IRC05:39
*** yfried_ has quit IRC05:39
oomichigmann: thanks for your investitaion!05:40
oomichigmann: that is great input for me05:40
oomichigmann: I feel current unit test way seems bad05:41
*** BharatK has joined #openstack-qa05:41
oomichigmann: because current unit test way depends on internal implementation of tempest-lib05:41
gmannoomichi: yea05:41
openstackgerritJamie Lennox proposed openstack/tempest: Give access to CredClient in tests  https://review.openstack.org/21498705:41
oomichigmann: we need to mock the surface of using library interface directly without depending on internal implementation05:42
gmannoomichi: and I agree to break users if anything doing this kind of stub. (as earlier mock release broke many of project :))05:42
gmannoomichi: +205:42
*** yfried__ is now known as yfried|afk05:42
*** ranger81 has joined #openstack-qa05:42
*** mylu has joined #openstack-qa05:44
oomichiandreaf: hello05:46
oomichimkoderer: andreaf: can you take a look at https://review.openstack.org/#/c/221588/ ?05:47
*** ramineni_ has joined #openstack-qa05:49
*** mylu has quit IRC05:49
*** ekarlso- has quit IRC05:50
*** yfried|afk is now known as yfried__05:51
*** itzikb has joined #openstack-qa05:53
*** vgridnev has joined #openstack-qa05:58
openstackgerritWei Jiangang proposed openstack-dev/devstack: Fix repeatedly enter the same string (fake-service)  https://review.openstack.org/22159806:00
mkodereroomichi: good morning06:06
oomichimkoderer: yeah, Guten Morgen :)06:07
mkodereroomichi: oh... sry have no idea how to say good morning in japanese :)06:08
oomichimkoderer: no problem :) "O Ha Yo" works for japanese. it is useful on Tokyo Summit also06:09
mkodereroomichi: cool.. unfortunately I can't make it to the summit this time06:09
oomichimkoderer: oops, that is bad news :(06:10
oomichimkoderer: maybe you are busy for many data centers, I guess06:11
mkodereroomichi: +A06:11
oomichimkoderer: Danke!06:12
mkodereroomichi: no acutally good news for me... my child will be born in the next weeks :)06:12
*** neeti has joined #openstack-qa06:12
oomichimkoderer: oh!! super congress in advance!06:12
oomichimkoderer: that will be great news :)06:12
mkodereroomichi: thanks :)06:13
oomichimkoderer: is she/he your first baby ?06:13
mkodereroomichi: yes :)06:14
oomichimkoderer: you will be very busy after the birthday and it is hard to get enough time to sleep and review06:14
oomichimkoderer: anyway, I am looking forward to your good news06:15
*** SridharG has left #openstack-qa06:16
mkodereroomichi: yeah I guess I won't be available much for reviews during that time06:16
oomichimkoderer: no problem, I also could not take enough time when 3rd baby born 5 months ago.06:17
mkodereroomichi: 3rd baby.. cool.. I will contact you if I need an advice :)06:18
oomichimkoderer: nice to take care for your family at the time :)06:18
oomichimkoderer: ^^^ is really important for us(husband)06:18
openstackgerritMerged openstack/tempest: Switch using token_client to the latest for gate  https://review.openstack.org/22158806:20
*** yfried__ is now known as yfried|afk06:21
*** yog_ has joined #openstack-qa06:22
mkodereroomichi: if you have a sec for a review: https://review.openstack.org/#/c/217675/06:26
mkodererI would like to have have some unit testing for the tempest plugins06:27
*** Sukhdev has quit IRC06:28
oomichimkoderer: sure, that is necessary. will see it06:28
*** mpaolino has joined #openstack-qa06:31
*** sshnaidm has joined #openstack-qa06:38
*** yfried|afk is now known as yfried__06:41
*** ttx has quit IRC06:48
*** ttx has joined #openstack-qa06:48
*** Poornima has quit IRC06:51
*** shausy has joined #openstack-qa06:55
*** shardy_ has joined #openstack-qa06:56
*** shardy has quit IRC06:57
*** b10n1k__ has quit IRC06:59
*** jaypipes has quit IRC07:00
*** shardy_ has quit IRC07:01
*** shardy has joined #openstack-qa07:02
*** armax has quit IRC07:02
*** Poornima has joined #openstack-qa07:09
openstackgerritMerged openstack/tempest: Add docs section on config options to reviewers guide  https://review.openstack.org/22133007:12
*** coolsvap is now known as coolsvap|brb07:14
*** armax has joined #openstack-qa07:18
*** gamado has joined #openstack-qa07:19
*** safchain has joined #openstack-qa07:20
openstackgerritGhanshyam Mann proposed openstack/tempest: Return complete resp from volumes_extensions_client  https://review.openstack.org/21616407:23
*** SridharG has joined #openstack-qa07:25
*** anshul has joined #openstack-qa07:29
openstackgerritWei Jiangang proposed openstack-dev/devstack: Fix repeatedly enter the same string (fake-service)  https://review.openstack.org/22159807:32
*** achanda has quit IRC07:32
*** SridharG has quit IRC07:36
*** achanda has joined #openstack-qa07:36
*** jlanoux has joined #openstack-qa07:40
*** yfried__ is now known as yfried|afk07:44
*** gszasz has joined #openstack-qa07:44
*** gfidente has joined #openstack-qa07:45
*** shardy_ has joined #openstack-qa07:47
*** shardy has quit IRC07:49
*** salv-orlando has quit IRC07:51
*** SridharG has joined #openstack-qa07:51
*** shardy_ has quit IRC07:53
*** tkammer|bot has joined #openstack-qa07:53
*** jordanP has joined #openstack-qa07:53
*** shardy has joined #openstack-qa07:53
*** achanda has quit IRC08:00
*** vgridnev has quit IRC08:01
*** k4n0 has joined #openstack-qa08:02
*** yfried|afk is now known as yfried__08:02
*** jordanP has quit IRC08:02
*** jordanP has joined #openstack-qa08:02
*** achanda has joined #openstack-qa08:02
*** cazino has joined #openstack-qa08:03
*** cazino has left #openstack-qa08:03
*** cdent has joined #openstack-qa08:05
*** achanda has quit IRC08:07
openstackgerritMD NADEEM proposed openstack/tempest: Add unit tests for floating_ips_bulk_client  https://review.openstack.org/21916608:09
*** tkammer|bot has quit IRC08:11
openstackgerritMerged openstack/tempest: Add unit tests for security_groups_client  https://review.openstack.org/21964508:15
openstackgerritMerged openstack/tempest: Add unit test for floating_ip_pools_client  https://review.openstack.org/21407308:15
openstackgerritMerged openstack/tempest: Added Test Case for BaremetalNodesClient  https://review.openstack.org/21503808:16
*** jordanP has quit IRC08:17
*** yfried__ is now known as yfried|afk08:29
*** hichihara has quit IRC08:32
openstackgerritChris Dent proposed openstack-dev/devstack: Update lib/ceilometer to reflect script renames  https://review.openstack.org/22163408:36
*** yfried|afk is now known as yfried__08:36
*** ranger81 has quit IRC08:42
*** jordanP has joined #openstack-qa08:43
*** mylu has joined #openstack-qa08:47
*** mylu has quit IRC08:52
*** e0ne has joined #openstack-qa08:54
*** salv-orlando has joined #openstack-qa08:56
openstackgerritChris Dent proposed openstack-dev/devstack: Update lib/ceilometer to reflect script renames  https://review.openstack.org/22163408:58
*** oomichi has quit IRC09:01
*** e0ne has quit IRC09:04
*** SridharG has quit IRC09:04
*** rossella_s has quit IRC09:06
*** rossella_s has joined #openstack-qa09:06
*** e0ne has joined #openstack-qa09:07
openstackgerritItzik Brown proposed openstack/tempest: qos basic scenario  https://review.openstack.org/21717709:12
openstackgerritAnusha Ramineni proposed openstack/tempest: Add unit tests for volumes_extensions_client  https://review.openstack.org/21661809:13
*** Kennan_Vacation2 has quit IRC09:17
openstackgerritGhanshyam Mann proposed openstack/tempest: Return Rules not sec grp from list_security_group_rules  https://review.openstack.org/22166009:22
*** Kennan_Vacation has joined #openstack-qa09:25
openstackgerritTakashi NATSUME proposed openstack/tempest: Fix typos in scenario tests, etc.  https://review.openstack.org/22113909:26
*** yfried__ is now known as yfried|afk09:29
cdentjordanP: have you got devstack approval powers?09:30
jordanPcdent, nope09:33
cdentno worries, just hoping :)09:33
jordanPyeah I know you cant to unblock ceilo :)09:33
cdentpeople will wake up soon enough09:33
jordanP*you want09:34
*** vgridnev has joined #openstack-qa09:34
jordanPchmouel, should be in Europe09:35
jordanPbut 11h35 is a bit early for him :)09:35
openstackgerritMerged openstack/tempest: Fix identity new endpoint_type options for old users  https://review.openstack.org/21916909:35
openstackgerritMerged openstack/tempest: Cleanup API version schema  https://review.openstack.org/22039109:35
*** dtantsur|afk is now known as dtantsur09:36
*** tosky has joined #openstack-qa09:41
*** coolsvap|brb is now known as coolsvap09:42
cdentjordanP: any time before noon is too early for me09:44
*** nadya has joined #openstack-qa09:44
*** amotoki has quit IRC09:46
jordanPcdent, haha :)09:48
*** yfried|afk is now known as yfried__09:49
*** jffische1 has joined #openstack-qa09:50
*** jffische has quit IRC09:51
*** SridharG has joined #openstack-qa09:53
*** SridharG has left #openstack-qa09:53
*** SridharG has joined #openstack-qa09:54
*** yamamoto has quit IRC09:55
*** yfried__ is now known as yfried|afk09:59
*** nadya has quit IRC10:02
*** dims__ has joined #openstack-qa10:02
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130010:06
*** apevec has joined #openstack-qa10:24
*** apevec has quit IRC10:24
*** apevec has joined #openstack-qa10:24
*** yfried|afk is now known as yfried__10:25
*** mylu has joined #openstack-qa10:32
*** mylu has quit IRC10:36
*** SridharG1 has joined #openstack-qa10:40
*** nadya has joined #openstack-qa10:41
*** SridharG1 has left #openstack-qa10:41
openstackgerritVladyslav Drok proposed openstack/tempest: [WIP] Skip unnecessary tests when using Ironic virt driver  https://review.openstack.org/22022310:41
*** pc_m has joined #openstack-qa10:42
*** SridharG has quit IRC10:42
*** SridharG has joined #openstack-qa10:43
*** mpaolino has quit IRC10:47
*** ranger81 has joined #openstack-qa10:53
*** asselin_ has quit IRC10:59
*** sayali has quit IRC11:00
*** ranger81 has quit IRC11:00
*** mwagner_lap has quit IRC11:02
*** sayali has joined #openstack-qa11:03
*** vgridnev has quit IRC11:04
*** aix has quit IRC11:06
*** vgridnev has joined #openstack-qa11:07
*** jswarren has joined #openstack-qa11:12
openstackgerritnayna-patel proposed openstack/tempest: Adds 'list-cells' and 'cell-capacities' tests  https://review.openstack.org/22093211:13
piyush_https://review.openstack.org/217555/ if possible please review the same11:14
*** e0ne has quit IRC11:16
*** yfried__ is now known as yfried|afk11:22
*** ramineni_ has left #openstack-qa11:24
*** vgridnev has quit IRC11:24
*** yfried|afk is now known as yfried__11:24
*** jhenner has joined #openstack-qa11:25
*** vgridnev has joined #openstack-qa11:25
*** jffische1 has quit IRC11:26
*** zz_dimtruck is now known as dimtruck11:28
*** nadya has quit IRC11:29
openstackgerritMerged openstack-dev/devstack: Update lib/ceilometer to reflect script renames  https://review.openstack.org/22163411:35
*** jhenner has quit IRC11:41
*** dave-mccowan has joined #openstack-qa11:44
*** jhenner has joined #openstack-qa11:45
*** dtantsur is now known as dtantsur|brb11:49
openstackgerritItzik Brown proposed openstack/tempest: qos basic scenario  https://review.openstack.org/21717711:50
*** nadya has joined #openstack-qa11:50
*** tochi has quit IRC11:51
*** viktor-tikkanen has quit IRC11:52
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130011:53
*** dkranz has joined #openstack-qa11:53
*** SridharG has left #openstack-qa11:57
*** e0ne has joined #openstack-qa12:00
*** aysyd has joined #openstack-qa12:03
*** woodster_ has joined #openstack-qa12:06
*** rfolco has joined #openstack-qa12:06
*** mpaolino has joined #openstack-qa12:10
*** dimtruck is now known as zz_dimtruck12:12
*** krtaylor has joined #openstack-qa12:14
*** e0ne has quit IRC12:15
openstackgerritMerged openstack-dev/devstack: Add aodh and gnocchi to the plugin registry  https://review.openstack.org/21727712:22
*** zz_dimtruck is now known as dimtruck12:23
*** dimtruck is now known as zz_dimtruck12:23
*** coolsvap has quit IRC12:27
*** armax has quit IRC12:29
*** e0ne has joined #openstack-qa12:30
*** coolsvap has joined #openstack-qa12:30
*** nadya has quit IRC12:31
*** dkranz has quit IRC12:32
*** shausy has quit IRC12:33
*** shausy has joined #openstack-qa12:33
*** edmondsw has joined #openstack-qa12:33
*** tpeoples has joined #openstack-qa12:35
*** mwagner_lap has joined #openstack-qa12:36
*** shuquan has joined #openstack-qa12:36
*** amotoki has joined #openstack-qa12:37
*** BharatK has quit IRC12:39
*** gamado has quit IRC12:44
*** regXboi has joined #openstack-qa12:47
*** shardy_ has joined #openstack-qa12:50
*** sabeen1 has joined #openstack-qa12:50
*** aix has joined #openstack-qa12:51
*** weshay_xchat has joined #openstack-qa12:51
*** shardy has quit IRC12:51
*** jffische has joined #openstack-qa12:52
*** weshay_xchat has quit IRC12:52
*** frayedknot has joined #openstack-qa12:52
*** dims__ is now known as dims12:52
*** Poornima has quit IRC12:55
*** shardy_ has quit IRC12:55
mtreinishgmann: ah ok, yeah that's a bad way to stub it. It's honestly probably a hold over from when things lived in tempest before the lib migration12:55
mtreinishgmann: you're we should fix that to only do it on the surface of the interface12:56
*** shardy has joined #openstack-qa12:56
*** ctrath has joined #openstack-qa12:57
*** shuquan has quit IRC12:58
*** bknudson has joined #openstack-qa12:59
*** VanL has joined #openstack-qa12:59
*** jswarren has quit IRC13:03
*** ctrath has quit IRC13:04
*** yfried__ has quit IRC13:04
*** dustins has joined #openstack-qa13:07
*** armax has joined #openstack-qa13:10
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add DB API methods to deal with test metadata  https://review.openstack.org/22147813:11
openstackgerritMatthew Treinish proposed openstack-infra/subunit2sql: Add option to specify global test attr prefix in subunit2sql  https://review.openstack.org/22147913:12
*** jecarey has joined #openstack-qa13:13
*** lexholden has joined #openstack-qa13:16
mtreinishandreaf: ^^^ that's my idea for dealing with the idempotent ids from tempest13:17
mtreinishhogepodge: ^^^ you might be interested in that too13:17
*** jswarren has joined #openstack-qa13:18
*** zz_dimtruck is now known as dimtruck13:19
*** tosky has quit IRC13:21
*** lexholden has quit IRC13:23
mkoderermtreinish: did you used "tox -ecover" latley? it seems it produces nothing useful...13:24
*** nadya has joined #openstack-qa13:24
mtreinishmkoderer: on tempest? it's been a while since I tried it there13:25
* mtreinish tries13:26
*** hop has joined #openstack-qa13:26
*** mriedem_away is now known as mriedem13:29
mtreinishmkoderer: it seems to generate a report fine13:31
mtreinishheh, but the unit test coverage reports for tempest were always of questionable value13:31
mtreinishsince most of tempest is self validating because it's test code13:31
mkoderermtreinish: yeah I mean I don't get any coverage for the clients although we have unit test for them13:35
mtreinishmkoderer: hmm, I see coverage reported for the service clients that have tests13:37
mkoderermtreinish: e.g. the coverage of agents_client is only 30% in my env...13:37
mtreinishoh, I actually just found a unit test bug because of that13:37
mkoderermtreinish: hehe :)13:37
hopmtreinish: hi, could you please review https://review.openstack.org/#/c/189687/ and https://review.openstack.org/#/c/189795/ ?13:38
openstackgerritMatthew Treinish proposed openstack/tempest: Use proper method for nova disable service unit test  https://review.openstack.org/22175713:39
mtreinishmkoderer: ^^^13:39
mtreinishmkoderer: oh, that's weird my coverage report shows 93% on tempest/services/compute/json/agents_client13:39
*** lexholden has joined #openstack-qa13:40
*** dtantsur|brb is now known as dtantsur13:40
mkoderermtreinish: mh seems it runs for you env13:40
mtreinishthe only line not executed in the report is L31 because we never run it with params13:40
mtreinishmkoderer: I had to wipe all the old coverage stuff13:40
mtreinishand ran tox with -r13:40
mtreinishI had weird results before I did that13:40
mkoderermtreinish: let me try this one13:41
*** ctrath has joined #openstack-qa13:43
openstackgerritJohn Warren proposed openstack/tempest: Add unit test for floating_ips_client  https://review.openstack.org/21728913:44
mkoderermtreinish: thanks for fixing that - I felt like a code monkey after fixing all that code duplication13:44
mtreinishmkoderer: heh, sure np13:52
mtreinishI'm more surprised that I got value from that coverage report and found the issue :)13:52
mtreinishmkoderer: oh actually even after my patch it's still not right13:54
mkoderermtreinish: yep, you need to change self.client.enable_service to self.client.disable_service  in _test_disable_service too13:55
openstackgerritMatthew Treinish proposed openstack/tempest: Use proper method for nova disable service unit test  https://review.openstack.org/22175713:55
mtreinishmkoderer: ^^^ yep13:55
*** dkranz has joined #openstack-qa13:56
mkoderermtreinish: ok seems coverage works for me now... "-r" fixed the issue.... really strange13:57
*** bknudson has quit IRC13:57
mtreinishmkoderer: stale venvs always cause issues :)13:58
*** lexholden has quit IRC14:03
*** lexholden has joined #openstack-qa14:07
*** salv-orlando has quit IRC14:07
*** mmedvede has quit IRC14:09
vgridnevhey folks! please review the following change: https://review.openstack.org/#/c/214194/ . I already have a bunch of +1's14:12
*** sabeen1 has quit IRC14:12
*** hop has quit IRC14:16
*** jaypipes has joined #openstack-qa14:16
*** ChanServ sets mode: +o jaypipes14:16
*** gszasz has quit IRC14:20
*** pglass has joined #openstack-qa14:21
*** neeti has quit IRC14:22
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130014:23
openstackgerritJordan Pittier proposed openstack-dev/devstack: Tempest: add a Cinder extend_with_snapshot feature flag  https://review.openstack.org/21363814:24
*** malini has joined #openstack-qa14:27
*** yamahata has joined #openstack-qa14:31
*** VanL_ has joined #openstack-qa14:31
*** VanL has quit IRC14:35
*** yamahata has quit IRC14:35
*** yamahata has joined #openstack-qa14:36
*** ajmiller has joined #openstack-qa14:36
*** bknudson has joined #openstack-qa14:39
*** mmedvede has joined #openstack-qa14:39
*** k4n0 has quit IRC14:40
*** dwalleck has joined #openstack-qa14:40
openstackgerritDaryl Walleck proposed openstack/tempest-lib: Provide alternative ssh exec_command for non-linux environments  https://review.openstack.org/21269114:46
*** malini1 has joined #openstack-qa14:48
*** malini has quit IRC14:52
hogepodgemtreinish: thanks14:52
*** jsuchome has joined #openstack-qa14:53
*** armax has quit IRC14:54
sdaguedtroyer: are you about? Because I'd really like to get the permissions on the grenade launchpad project sorted, it's in a weird state14:56
jordanPmhorban, hi14:56
jordanPmhorban, a moment for https://review.openstack.org/#/c/153005 ?14:57
dtroyersdague: yup, lemme pull it up14:57
*** dwalleck has quit IRC14:57
*** nadya has quit IRC14:58
*** dwalleck has joined #openstack-qa14:59
dtroyerlooks like the standard is a -drivers team?  we only have a grenade-core ATM, could just use that15:02
*** dave-mccowan has quit IRC15:02
*** tpeoples has quit IRC15:03
sdaguedtroyer: yeh, and I think you are supposed to make it owned by openstack-admins15:05
*** dwalleck has quit IRC15:05
sdagueso that dhellmann and ttx can fix things if they go wrong15:06
dtroyerI see OpenStack release team as members in other projects.15:06
dtroyerIf I can find where I'll create Grenade Drivers so it's consistent15:06
dtroyerok, seeded it with you me and mtreinish, do you want administrator in the drivers group?15:09
*** dwalleck has joined #openstack-qa15:10
dtroyersdague: ok, I think I've got it, take a look15:12
*** tpeoples has joined #openstack-qa15:12
jsuchomehi, I wanted to disable one test from running (scenario/test_network_basic_ops/test_subnet_details) by setting empty value for dhcp_client in /etc/tempest/tempest.conf), but when I do it15:12
jsuchometempest fails with "Non-zero exit code (2) from test listing"15:12
*** erlon has joined #openstack-qa15:13
jsuchomewhat's wrong? only difference is that value change in tempest.conf15:13
mtreinishjsuchome: looking at the code the dhcp_client config option has only 2 allowed values "udhcpc" or "dhclient" (its the dhcp client installed in the guest image)15:14
mtreinishby setting it to none that'll cause an exception to be raised since it's not an allowed value15:14
mtreinishyou should have a traceback somewhere showing that15:15
*** dave-mccowan has joined #openstack-qa15:16
jsuchomemtreinish: no I think the test should be skipped based on https://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_network_basic_ops.py#n51115:16
mtreinishjsuchome: that's not relevent because during import the config loading will raise an exception based on the option definition: https://github.com/openstack/tempest/blob/master/tempest/config.py#L103715:17
jsuchomeand the problem is that after that config file change, no test is actually executed, it ends early with http://pastebin.com/LWxXbGd515:18
mtreinishyou asked why things were failing during the test list, that's why15:18
mtreinishif you update your testtools, subunit, and testrepository python packages it'll show you a traceback as to why test discovery failed15:18
mtreinishand you'll see a traceback from oslo config saying that the option didn't have a valid choice set15:19
*** sshnaidm has quit IRC15:19
jsuchomemtreinish: but the smae config.py says "'If left empty, update operation will be skipped. '15:19
mtreinishjsuchome: it's wrong15:20
*** vgridnev has quit IRC15:20
mtreinishwhich is clear based on what you're seeing15:20
mtreinishi's a bug that needs to be fixed15:20
mtreinishjsuchome: file one to update that help message15:20
jsuchomeso updating value of 'choices' could help?15:20
*** salv-orlando has joined #openstack-qa15:21
jsuchomemtreinish: thanks, I'll file a bug (or prepare pull request), but is there currently a correct way to skip that test?15:22
mtreinishjsuchome: just use the test runner to not run it. With testr you have to use a negative lookahead regex (there are examples in the tempest tox.ini)15:23
mtreinishjsuchome: just curious what specifically is your issue with the test?15:24
mtreinishis it the guest env doesn't have a supported dhcp client?15:24
mtreinishor you just don't want to be running it15:24
*** ccneill has joined #openstack-qa15:25
*** gszasz has joined #openstack-qa15:25
jsuchomemtreinish: I have docker based instance and that doesn't seem to have supported dhcp client15:25
*** gszasz has quit IRC15:26
mtreinishjsuchome: yeah, that's why was opposed to that test when it was proposed15:27
*** tosky has joined #openstack-qa15:27
mtreinishit's very env specific which makes it less useful15:27
openstackgerritVladyslav Drok proposed openstack/tempest: [WIP] Skip unnecessary tests when using Ironic virt driver  https://review.openstack.org/22022315:27
*** gszasz has joined #openstack-qa15:27
*** hemnafk is now known as hemna15:28
*** vgridnev has joined #openstack-qa15:28
*** vgridnev has quit IRC15:30
*** armax has joined #openstack-qa15:33
*** dkranz has quit IRC15:33
*** vgridnev has joined #openstack-qa15:35
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130015:36
*** Swami has joined #openstack-qa15:36
*** salv-orl_ has joined #openstack-qa15:40
*** VanL_ has quit IRC15:41
*** vgridnev has quit IRC15:44
*** itzikb has quit IRC15:44
*** salv-orlando has quit IRC15:44
*** asselin_ has joined #openstack-qa15:44
*** VanL has joined #openstack-qa15:45
*** mlavalle has joined #openstack-qa15:45
*** dkranz has joined #openstack-qa15:46
*** dspano has joined #openstack-qa15:47
*** prnk28 has joined #openstack-qa15:48
*** maurosr has quit IRC15:50
*** ranger81 has joined #openstack-qa15:51
*** anshul has quit IRC15:52
*** Sukhdev has joined #openstack-qa15:55
*** salv-orl_ has quit IRC15:57
*** salv-orlando has joined #openstack-qa15:58
*** nadya has joined #openstack-qa15:58
*** yog_ has quit IRC16:00
*** yunpengli has joined #openstack-qa16:01
*** nadya has quit IRC16:03
*** dwalleck has quit IRC16:04
*** safchain has quit IRC16:06
*** jlanoux has quit IRC16:07
*** jsuchome has left #openstack-qa16:08
*** dims has quit IRC16:10
*** salv-orl_ has joined #openstack-qa16:10
*** dims has joined #openstack-qa16:10
*** salv-orlando has quit IRC16:14
jordanPmkoderer, ping ?16:20
*** nic has joined #openstack-qa16:21
*** nadya has joined #openstack-qa16:25
*** dtantsur is now known as dtantsur|afk16:25
*** salv-orl_ has quit IRC16:33
*** jordanP has quit IRC16:34
openstackgerritYaroslav Lobankov proposed openstack/tempest: Add possibility to skip test_subnet_details if DHCP client not available  https://review.openstack.org/22187416:36
*** malini1 has quit IRC16:37
*** vgridnev has joined #openstack-qa16:40
*** malini has joined #openstack-qa16:40
*** amotoki has quit IRC16:43
openstackgerritChris Dent proposed openstack-dev/devstack: Remove ceilometer in favor of plugin  https://review.openstack.org/19638316:44
*** VanL_ has joined #openstack-qa16:47
*** shausy has quit IRC16:49
*** VanL has quit IRC16:51
*** cdent has quit IRC16:53
*** achanda has joined #openstack-qa16:53
openstackgerritArkadiy Kraminsky proposed openstack/tempest-lib: Add in a more verbose exception message  https://review.openstack.org/22144916:56
openstackgerritArkadiy Kraminsky proposed openstack/tempest-lib: Add in a more verbose exception message  https://review.openstack.org/22144916:57
*** dwalleck has joined #openstack-qa16:59
*** malini has quit IRC17:00
*** dzamboni has joined #openstack-qa17:02
*** lexholden has quit IRC17:05
*** rossella_s has quit IRC17:06
*** rossella_s has joined #openstack-qa17:06
*** dkranz has quit IRC17:06
*** e0ne has quit IRC17:08
*** tosky has quit IRC17:08
mtreinishandreaf: qq, does the __str__ method on credentials object print the password?17:10
andreafmtreinish: yes, I think it will print it17:11
andreafmtreinish: I guess it could be made optional?17:11
andreafmtreinish: security concerns about pwd in logs?17:12
mtreinishandreaf: yeah17:12
mtreinishthat's something we shouldn't normally be printing17:12
mtreinishandreaf: I was thinking about it for this patch: https://review.openstack.org/#/c/221449/717:14
mtreinishwe're hitting that exception on some of the experimental/periodic jobs right now and I can't figure out why17:14
mtreinishI expect it's a configuration issue, but I'm not sure what17:15
*** yamahata has quit IRC17:16
*** dwalleck has quit IRC17:16
*** malini has joined #openstack-qa17:18
*** yunpengli has quit IRC17:19
*** salv-orlando has joined #openstack-qa17:19
*** salv-orl_ has joined #openstack-qa17:21
*** dwalleck has joined #openstack-qa17:21
*** dkranz has joined #openstack-qa17:22
*** VanL_ has quit IRC17:22
*** salv-orlando has quit IRC17:23
*** Sukhdev has quit IRC17:30
*** ranger81 has quit IRC17:31
*** ranger81 has joined #openstack-qa17:32
*** achanda has quit IRC17:34
*** salv-orlando has joined #openstack-qa17:35
*** ccneill has quit IRC17:38
*** salv-orl_ has quit IRC17:38
*** shardy_ has joined #openstack-qa17:38
*** achanda has joined #openstack-qa17:38
*** shardy has quit IRC17:40
*** dwalleck has quit IRC17:42
*** VanL has joined #openstack-qa17:43
*** shardy_ has quit IRC17:43
*** shardy has joined #openstack-qa17:44
*** VanL has quit IRC17:47
*** apevec has quit IRC17:49
*** ranger81 has quit IRC17:49
*** ranger81 has joined #openstack-qa17:50
*** pvaneck has joined #openstack-qa17:50
*** mpaolino has quit IRC17:51
*** tosky has joined #openstack-qa17:52
*** yfried__ has joined #openstack-qa17:53
*** armax has quit IRC17:58
*** mylu has joined #openstack-qa17:58
*** rbak has quit IRC17:59
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130018:00
*** apevec has joined #openstack-qa18:03
*** vgridnev has quit IRC18:03
*** dkranz has quit IRC18:04
openstackgerritRoxana Gherle proposed openstack/tempest-lib: Introduce scope in the auth API  https://review.openstack.org/20270518:06
*** dkranz has joined #openstack-qa18:07
*** krtaylor has quit IRC18:13
*** rbak has joined #openstack-qa18:15
andreafmtreinish: I think using print(str()) would be the right approach, we can change __str__ to not include the pwd in clear18:16
*** vgridnev has joined #openstack-qa18:22
*** e0ne has joined #openstack-qa18:23
*** nadya has quit IRC18:28
*** mylu has quit IRC18:29
*** aix has quit IRC18:29
*** mylu has joined #openstack-qa18:30
mtreinishandreaf: ok cool, can you leave a review comment about that on the review18:30
*** mylu has quit IRC18:31
*** mylu has joined #openstack-qa18:31
*** rbak has quit IRC18:32
*** rbak has joined #openstack-qa18:32
*** jhesketh has quit IRC18:32
*** rbak has quit IRC18:32
*** rbak has joined #openstack-qa18:34
*** jhesketh has joined #openstack-qa18:34
*** nadya has joined #openstack-qa18:35
*** hoangcx has joined #openstack-qa18:37
*** VanL has joined #openstack-qa18:37
*** DinaBelova has quit IRC18:40
*** VanL has quit IRC18:42
*** vponomaryov has quit IRC18:42
*** dwalleck has joined #openstack-qa18:43
*** DinaBelova has joined #openstack-qa18:43
*** vponomaryov has joined #openstack-qa18:43
openstackgerritMauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node  https://review.openstack.org/22190918:44
*** apevec has quit IRC18:44
*** nadya has quit IRC18:46
*** e0ne has quit IRC18:47
*** dwalleck has quit IRC18:47
*** krtaylor has joined #openstack-qa18:48
openstackgerritMauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node  https://review.openstack.org/22190918:48
*** maurosr has joined #openstack-qa18:48
*** ccneill has joined #openstack-qa18:48
*** tosky has quit IRC18:51
*** tosky has joined #openstack-qa18:51
openstackgerritMauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node  https://review.openstack.org/22191218:51
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130018:52
*** Sukhdev has joined #openstack-qa18:55
*** yamahata has joined #openstack-qa18:57
openstackgerritMauro Rodrigues proposed openstack-dev/devstack: Enables the user to set arch for the baremetal flavor and ironic node  https://review.openstack.org/22191518:58
*** krtaylor has quit IRC19:00
*** ccneill has quit IRC19:00
*** krtaylor has joined #openstack-qa19:00
*** pglbutt has joined #openstack-qa19:04
*** jasonsb has quit IRC19:04
*** jasonsb has joined #openstack-qa19:05
*** jasonsb has quit IRC19:05
*** jasonsb has joined #openstack-qa19:06
*** pglass has quit IRC19:07
*** e0ne has joined #openstack-qa19:07
*** nic has quit IRC19:08
*** malini has quit IRC19:08
*** boris-42 has quit IRC19:10
*** jasonsb has quit IRC19:10
*** nadya has joined #openstack-qa19:10
mtreinishmkoderer: just curious did we ever get anywhere with the tempest-plugin-cookiecutter stuff?19:11
*** achanda has quit IRC19:13
*** malini has joined #openstack-qa19:13
openstackgerritRoxana Gherle proposed openstack/tempest-lib: Introduce scope in the auth API  https://review.openstack.org/20270519:16
*** nic has joined #openstack-qa19:17
mtreinishdkranz, andreaf: if you get a sec: https://review.openstack.org/#/c/216522/ it'll be good to unblock the periodic test-accounts jobs19:17
mtreinishI really want to start the timer on the creds option deprecation before the release19:17
*** kiran-r has joined #openstack-qa19:17
dkranzmtreinish: ok, it passed pep819:24
*** edmondsw has quit IRC19:25
openstackgerritDavanum Srinivas (dims) proposed openstack-dev/devstack: [WIP] Set v3 as IDENTITY_API_VERSION by default  https://review.openstack.org/22130019:25
mtreinishdkranz: yeah, it was off by 1 space. It took me a sec to see it19:25
*** hoangcx has quit IRC19:27
*** apevec has joined #openstack-qa19:29
*** kiran-r has quit IRC19:29
*** grafuls_ has joined #openstack-qa19:31
*** e0ne has quit IRC19:32
*** jhenner has quit IRC19:33
*** aix has joined #openstack-qa19:33
*** jhenner has joined #openstack-qa19:33
*** grafuls has quit IRC19:35
*** dtantsur|afk has quit IRC19:35
*** jhenner has quit IRC19:38
*** jhenner has joined #openstack-qa19:39
*** grafuls_ has quit IRC19:39
*** dtantsur has joined #openstack-qa19:40
*** jhenner1 has joined #openstack-qa19:42
*** jhenner has quit IRC19:43
openstackgerritOpenStack Proposal Bot proposed openstack/tempest: Updated from global requirements  https://review.openstack.org/21722219:43
*** mylu has quit IRC19:45
*** mylu has joined #openstack-qa19:46
*** pglass has joined #openstack-qa19:46
*** pglass has quit IRC19:46
*** pglbutt has quit IRC19:49
*** mylu has quit IRC19:50
*** nadya has quit IRC19:50
*** grafuls_ has joined #openstack-qa19:50
*** malini has quit IRC19:50
*** tosky has quit IRC19:51
*** tosky has joined #openstack-qa19:51
*** jhenner has joined #openstack-qa19:51
*** jesusaurus has joined #openstack-qa19:53
*** jhenner1 has quit IRC19:53
*** gszasz has quit IRC19:55
akraminsky_mtreinish, I looked at your comments and went ahead and changed __str__. just want your opinion before I push it up, this is my change http://paste.openstack.org/show/453131/19:58
*** pglass has joined #openstack-qa19:59
akraminsky_mtreinish, oops switched that not around too19:59
*** boris-42 has joined #openstack-qa20:00
openstackgerritMerged openstack/tempest: Use proper method for nova disable service unit test  https://review.openstack.org/22175720:00
mtreinishakraminsky_: yep, that looks like the basic idea20:02
mtreinishakraminsky_: it probably make senses to add unit tests to that too, just to verify the logic20:03
mtreinishwe also should probably split the changes into 2 patches, so make the first do the str fix20:03
mtreinishand then on top of that change the exception msg fix20:03
*** jasonsb has joined #openstack-qa20:05
*** dkranz has quit IRC20:10
*** e0ne has joined #openstack-qa20:11
*** achanda has joined #openstack-qa20:12
*** b10n1k__ has joined #openstack-qa20:16
*** achanda has quit IRC20:18
*** b10n1k__ has quit IRC20:21
*** tosky has quit IRC20:21
*** tosky has joined #openstack-qa20:21
*** salv-orlando has quit IRC20:22
*** salv-orlando has joined #openstack-qa20:22
*** ccneill has joined #openstack-qa20:22
*** dkranz has joined #openstack-qa20:23
*** dkranz has quit IRC20:23
*** achanda has joined #openstack-qa20:28
*** b10n1k__ has joined #openstack-qa20:33
openstackgerritJohn F. Fischer proposed openstack/tempest: Add unit test for hypervisor_client  https://review.openstack.org/21792520:35
*** salv-orlando has quit IRC20:40
*** salv-orlando has joined #openstack-qa20:41
openstackgerritMerged openstack-infra/subunit2sql: Add option to subunit2sql for attr regex  https://review.openstack.org/22147720:41
*** salv-orl_ has joined #openstack-qa20:42
mtreinishsdague, gmann: didn't we remove the nova v21 job definition as part of that nova catalog refactor in devstack?20:42
*** salv-or__ has joined #openstack-qa20:44
*** salv-orl_ has quit IRC20:44
*** prnk28 has quit IRC20:44
*** salv-orlando has quit IRC20:46
sdaguemtreinish: it's in the stack of cleanups20:46
sdagueit may not be landed yet20:46
*** ianw has quit IRC20:47
mtreinishsdague: ok, I just had a patch bounce off the job and I was surprised it was still a thing20:47
*** ianw has joined #openstack-qa20:48
*** achanda has quit IRC20:49
openstackgerritVitaly Gridnev proposed openstack-dev/devstack: Support BlockDeviceDriver as as cinder backend  https://review.openstack.org/21419420:50
*** dave-mccowan has quit IRC20:51
*** jasonsb has quit IRC20:51
*** nikita_09 has quit IRC20:51
*** jasonsb has joined #openstack-qa20:52
mtreinishandreaf: so I think I the ci job for test accounts might have found a smallish issue: http://logs.openstack.org/22/216522/5/experimental/gate-tempest-dsvm-full-test-accounts/d800694//console.html#_2015-09-09_20_15_47_75720:53
mtreinishI think thats caused by test accounts pulling an account with a role that has additional permissions so the quota operation being performed there is allowed20:53
mtreinishlike it's getting swift operator because test accounts treats all roles except for admin as nothing extra (unless it's asked for) and it's allowed to modify quotas20:54
mtreinishbut the test isn't expecting that20:54
*** Sukhdev has quit IRC20:55
mtreinishandreaf: err, maybe not that test should be always be using the operator role20:57
mtreinishoh nm, I think it's because the users generated for the accounts.yaml always have both operator and reseller_admin assigned together20:57
mtreinishinstead of being mutually exclusive20:58
mtreinishand the test isn't expecting a reseller_admin user to run the quota op20:58
*** achanda has joined #openstack-qa20:59
*** frayedknot has quit IRC21:02
*** jhesketh has quit IRC21:03
*** bknudson has quit IRC21:04
*** krtaylor has quit IRC21:05
*** rossella_s has quit IRC21:06
*** rossella_s has joined #openstack-qa21:06
*** jhesketh has joined #openstack-qa21:07
*** dustins has quit IRC21:08
openstackgerritMatthew Treinish proposed openstack/tempest: Fix test_port_list_filter_by_ip with test_accounts  https://review.openstack.org/21688721:08
sdaguemtreinish: https://review.openstack.org/#/c/219553/ - should be fixed when that merges21:09
mtreinishsdague: ok, thanks21:09
*** edmondsw has joined #openstack-qa21:10
sdaguemtreinish: so we're running those v2.0 jobs on nova check queue non-voting21:10
sdaguedo you want those in tempest experimental, or tempest check21:10
mtreinishsdague: I'd probably prefer experimental/periodic unless you think the difference between v2.1 and v2.0 is significant enough to warrant running it on check21:12
sdaguemtreinish: well it shouldn't be21:12
sdaguethat's the point21:12
sdagueoh... before I forget, why are there now 4 emails a day on the qa list?21:13
*** sabeen1 has joined #openstack-qa21:14
mtreinishsdague: someone added 3 neutron pg jobs to the qa periodic job list, I'm not sure who21:14
mtreinishI haven't gotten around to removing them21:14
mtreinishbecause there isn't a dashboard or any other reporting mechanism there really isn't anywhere else to dump the periodic test results to21:14
*** salv-orlando has joined #openstack-qa21:15
mtreinishsdague: give me a min, I'll push up the patch to remove the jobs21:15
mtreinishthey've never worked from what I can tell21:15
lifelessmtreinish: polleyz put up a dashboard thingy21:15
lifelessmtreinish: poc at least21:16
*** salv-or__ has quit IRC21:17
mtreinishlifeless: I haven't heard about that, do you have a link?21:17
mtreinishlifeless: but if he has something he should come to the qa sprint in ft collins on mon. :)21:17
mtreinishbecause I want to get that off the todo list once and for all21:18
openstackgerritMatthew Treinish proposed openstack/tempest: Split the operator and reseller users in the account generator  https://review.openstack.org/22196621:19
mtreinishtchaypo: ^^^21:21
tchaypopolleyz :)21:22
tchaypoI think that would be https://review.openstack.org/#/c/192253/21:23
mtreinishheh, it took me a sec to figure that out21:23
tchaypono wait21:23
tchaypoit’d be the other thing21:23
*** pc_m has quit IRC21:23
mtreinishtchaypo: that's the spec about what I want to get done at the sprint next week21:23
tchaypohttp://bruce.jamezpolley.com/reports/21:23
tchaypowhich is just an implementation of a slight rework of a script that derek originally wrote for tripleo21:24
mtreinishtchaypo: ah ok, cool21:25
mtreinishtchaypo: fwiw, I put some of my initial thoughts up on: https://etherpad.openstack.org/p/PuDfHylBgI21:25
mtreinishof how I thought it would look21:26
mtreinishof course I can't draw at all so it's just words poorly describing the pictures in my head21:26
*** pvaneck_ has joined #openstack-qa21:28
*** pvaneck has quit IRC21:28
*** sabeen1 has quit IRC21:31
*** jasonsb has quit IRC21:31
*** dave-mccowan has joined #openstack-qa21:32
*** jasonsb has joined #openstack-qa21:32
*** dspano has quit IRC21:35
*** e0ne has quit IRC21:39
*** jffische has quit IRC21:40
*** bknudson has joined #openstack-qa21:40
*** mestery has quit IRC21:41
*** mestery has joined #openstack-qa21:42
*** dave-mccowan has quit IRC21:42
*** tsekiyama has joined #openstack-qa21:42
*** frayedknot has joined #openstack-qa21:46
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/221545 failed because of: gate-grenade-dsvm-ceilometer: https://bugs.launchpad.net/bugs/146363121:49
openstackLaunchpad bug 1463631 in grenade "60_nova/resources.sh:106:ping_check_public fails intermittently" [Undecided,Confirmed]21:49
tchaypoyou mentioned a spring next week21:51
*** b10n1k__ has quit IRC21:53
openstackgerritArkadiy Kraminsky proposed openstack/tempest-lib: Add in a more verbose exception message  https://review.openstack.org/22144921:56
*** Sukhdev has joined #openstack-qa21:57
*** krtaylor has joined #openstack-qa22:04
*** vgridnev has quit IRC22:07
akraminsky_mtreinish, would the unit test for the str method go in test_credentials.py or test_auth.py?22:08
*** b10n1k__ has joined #openstack-qa22:10
*** regXboi has quit IRC22:12
*** ctrath has quit IRC22:12
*** dzamboni has quit IRC22:12
andreafmtreinish, jamielennox: I think https://review.openstack.org/#/c/202705 looks ok now, if you have time to review - I added a WIP tempest patch (https://review.openstack.org/#/c/221020/) which depends on this - on the tempest-lib src job you can see this code actually used22:16
*** pglass has quit IRC22:19
jamielennoxandreaf: it looks good to me22:22
andreafjamielennox: thanks22:22
*** masayukig has quit IRC22:24
*** masayukig has joined #openstack-qa22:26
*** mwagner_lap has quit IRC22:26
ianwjamielennox: this is correct? -> https://review.openstack.org/#/c/220458/  i'm pretty sure22:27
jamielennoxianw: sigh, swift again22:27
jamielennoxyea, it needs it. i thought the env variables were still versioned but apparently not if this is failing22:28
*** jecarey has quit IRC22:36
*** aysyd has quit IRC22:50
openstackgerritMerged openstack-dev/devstack: Tempest: add a Cinder extend_with_snapshot feature flag  https://review.openstack.org/21363822:50
*** ccneill has quit IRC22:53
openstackrecheckopenstack/tempest change: https://review.openstack.org/200108 failed gate-tempest-dsvm-full, gate-tempest-dsvm-postgres-full, gate-tempest-dsvm-neutron-full in the gate queue with an unrecognized error22:55
*** hemna is now known as hemnafk22:55
*** rbak has quit IRC22:55
*** mlavalle has quit IRC22:56
mtreinishtchaypo: yeah: https://wiki.openstack.org/wiki/QA/CodeSprintLibertyFortCollins23:05
mtreinishandreaf: ok, I'll take a look in a bit23:05
mtreinishmore realistically tomorrow23:05
jamielennoxmtreinish: do you have any idea what i did to screw up: https://review.openstack.org/#/c/214987/923:05
*** bknudson has quit IRC23:06
mtreinishthe unit test failures are unrelated, we pushed a new tempest-lib release last night (well your morning) and it broke the tempest unit tests in an unexpected way23:06
mtreinishbut that's been fixed23:06
jamielennoxok, so i'll try a rechec23:06
jamielennoxk23:06
mtreinishand the postgres failure looks like a quota bug, but I haven't seen that one in a while23:07
*** achanda has quit IRC23:08
jamielennoxyea, i've had a few gate issues so far but the 27,34 test failures threw me23:08
*** erlon has quit IRC23:08
jamielennoxit happens locally as well but i couldn't see where i would have changed anything23:08
mtreinishjamielennox: yeah it was tempest-lib 0.8.0 we deprecated/renamed a class and made it just a subclass that logged a warning on init23:09
mtreinishbut doing that some how broke a mox stub causing those tests to actually try and go out on the network23:10
jamielennoxergh, mox23:10
jamielennoxoh, because you use httplib2 and not requests23:10
mtreinishyeah, we need to stop using mox there23:10
mtreinishalso it was stubbing the wrong class it was relying on internals of the tempest-lib rest client23:11
*** david-lyle has quit IRC23:11
jamielennoxi was going to pimp requests-mock mut it's not much good if you're not using requests23:11
mtreinishheh23:11
*** david-lyle has joined #openstack-qa23:12
*** achanda has joined #openstack-qa23:20
*** david-lyle has quit IRC23:22
*** Swami has quit IRC23:23
*** edmondsw has quit IRC23:23
*** david-lyle has joined #openstack-qa23:23
*** b10n1k__ has quit IRC23:24
gmannmtreinish: just merged - for v21 job cleanup - https://review.openstack.org/#/c/219553/23:29
mtreinishgmann: cool, thanks23:31
openstackgerritArkadiy Kraminsky proposed openstack/tempest: Add volume backed live migration test  https://review.openstack.org/16523323:31
gmannmtreinish: on devstack side cleanup - https://review.openstack.org/#/c/219555/23:31
gmannmtreinish: and please look into related one for v20 as experimental on Tempest - https://review.openstack.org/#/c/221608/23:32
*** tosky has quit IRC23:33
mtreinishgmann: ok, I'll take a look at those tomorrow23:35
gmannmtreinish: sure. Thanks.23:35
mtreinishgmann: I'll trade you for: https://review.openstack.org/221966 , https://review.openstack.org/216887 , and https://review.openstack.org/21160123:36
mtreinishwhich are to fix bugs with test-accounts and start the deprecation timer on the creds config options23:36
gmannmtreinish: sure i will look today23:37
*** bknudson has joined #openstack-qa23:37
gmannmtreinish: can you quickly +A on https://review.openstack.org/#/c/217222/23:37
gmannmtreinish: updating oslo.config version which needed for your patch23:37
*** apevec has quit IRC23:38
*** ranger81 has quit IRC23:38
mtreinishgmann: done23:38
gmannmtreinish: Thanks23:38
*** ranger81 has joined #openstack-qa23:38
*** tochi has joined #openstack-qa23:39
*** jasonsb has quit IRC23:40
mtreinishakraminsky_: probably test_credentials.py, since you'll be adding unit tests for __str__ on the credentials class23:42
*** mriedem1 has joined #openstack-qa23:44
*** Sukhdev has quit IRC23:45
*** Sukhdev has joined #openstack-qa23:45
openstackgerritMerged openstack-dev/devstack: Fix typos in devstack/stack.sh  https://review.openstack.org/22123223:46
*** mriedem has quit IRC23:47
*** dimtruck is now known as zz_dimtruck23:48
*** jasonsb has joined #openstack-qa23:48
*** jasonsb has quit IRC23:49
*** nic has quit IRC23:51
*** Swami has joined #openstack-qa23:51
openstackgerritMerged openstack/tempest: Only add roles for enabled services  https://review.openstack.org/21652223:53
*** mmedvede has quit IRC23:55
*** ajmiller has quit IRC23:56
*** bknudson has quit IRC23:56

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