Monday, 2015-08-10

*** funzo has joined #refstack01:05
*** funzo has quit IRC01:10
*** funzo has joined #refstack02:36
*** funzo has quit IRC02:41
sslypushenkohogepodge catherineD Hi! Please take part in discusson around this path https://review.openstack.org/#/c/192709. mtreinish and andreaf have already clarified their point view.09:19
sslypushenkoSo we need to come to some concerted solution here.09:20
openstackgerritMerged stackforge/refstack: Fix the link to the wiki page in about.html.  https://review.openstack.org/21059909:38
*** funzo has joined #refstack12:22
*** funzo has quit IRC12:27
*** funzo has joined #refstack13:38
*** funzo has quit IRC13:42
*** geoffarnold has joined #refstack14:02
*** esker has joined #refstack14:56
*** geoffarnold is now known as geoffarnoldX15:02
*** funzo has joined #refstack15:09
*** geoffarnoldX is now known as geoffarnold15:09
*** funzo has quit IRC15:13
*** geoffarnold has quit IRC15:15
*** resker has joined #refstack15:31
*** esker has quit IRC15:32
*** resker has quit IRC15:49
*** esker has joined #refstack15:50
*** geoffarnold has joined #refstack15:53
*** funzo has joined #refstack16:09
*** funzo has quit IRC16:14
*** catherineD|2 has joined #refstack16:25
*** catherineD has quit IRC16:27
*** rockyg has joined #refstack17:09
*** pvaneck has joined #refstack17:57
*** rockyg has quit IRC18:09
*** funzo has joined #refstack18:10
*** cody-somerville has joined #refstack18:15
*** funzo has quit IRC18:15
*** funzo has joined #refstack19:11
*** funzo has quit IRC19:16
catherineD|2sslypushenko: about https://review.openstack.org/#/c/19270920:00
sslypushenkoyeap20:00
sslypushenkoI think some additional opinions can help us20:00
catherineD|2what do we really tested in this test case?  We want to test the non-admin user can execute list_tenant command?20:00
sslypushenkoBasicaly - yes20:01
catherineD|2I am thinking in most of the case ... the result will be only one tenant ... because the credential is depending on tenant ...20:01
sslypushenkoAll admin test will failed for non-admin user20:01
sslypushenkoSo we need cover us mush as possible20:02
catherineD|2in tempest config we have 2 users20:02
catherineD|2user and alt_user20:02
catherineD|2technically these 2 users belong to 2 tenants ...20:02
sslypushenkoWe have) Bus user of public cloud - does not)20:02
catherineD|2but currently to run the tests we requires 2 users for tempest config20:03
sslypushenkoWe really need to verify that this endpoint return one tenant20:03
catherineD|2so public cloud will need to provide 2 users ..20:03
sslypushenkoand user can login whit it20:03
catherineD|2but according to Matt ... he thinks that is already a given condition ...20:04
sslypushenkoMatt thinks from his own point of view20:04
sslypushenkoQA team don't care about all usecases20:05
sslypushenkoAnd they are not suppouse to do this, actually20:05
sslypushenkoI think that if we clarify our use case they will agreed with our suggestion20:06
catherineD|2what if we check for assertion of tenant name of alt_user ... (compare what returned with a known tenant that the user does not belong to )20:06
sslypushenkoWe can, but there is no gurantees about alt uset tenant20:07
sslypushenkoit can be the same and it can ve differnt20:07
sslypushenkoAnd it is really out of scope for this test20:08
sslypushenkoYeap, this test is very simple, but it is because its logic are very simple20:09
catherineD|2sslypushenko: we kind of  be cause we ask the the tenant of user and alt_user have to be different ... see page 6 of https://docs.google.com/presentation/d/1yNPjJZwu34DfEWWHQlpUPq8CtmkpGwOLJWTpqVmntlc/edit#slide=id.g9f226f371_0_5720:09
catherineD|2sslypushenko: our goal is to have this test merged .. if we need to adjust some how  ... we are open to it right?20:10
catherineD|2so if we test to compare with alt_user .... it is kind of a negative test ...20:10
catherineD|2as Matt and Andreas had suggested20:10
sslypushenkoHmm....20:11
catherineD|2sslypushenko: I will think about it some more ... that is just a suggestion for now ..20:11
sslypushenkoIf we stick to idea with at least 2 users it will work20:11
catherineD|2definitely we do not want to create or update tenants for test set up ...20:11
catherineD|2sslypushenko: we can stick to 2 users because that is our current configuration requirement ... I know that there is suggestion for one user ..20:12
sslypushenkoI thought that refstack should work with 1 user creds20:12
catherineD|2but there are many isolation test cases that really need 2 users ...20:12
sslypushenkoOk. Thanks for clarification) and slides)20:13
catherineD|2sslypushenko: let's check with Chris on the user requirements ...20:13
sslypushenkoI hope test with 2 users will work for qa-team20:14
catherineD|2I think we will need 2 users .... unless we don't care about isolation test cases ....20:14
catherineD|2sslypushenko: I will think about this some more .... maybe we will have better solution ... give this one some thoughts ...20:15
sslypushenkoRequirment of 2 non-admin users looks reasonable and realistic20:15
catherineD|2sslypushenko: thx .. running to an other meeting ... let;s discuss some more if either of us come up with somethign new20:16
sslypushenkoGreate thx for discussion)20:16
*** funzo has joined #refstack20:42
*** funzo has quit IRC20:46
*** funzo has joined #refstack21:57
*** catherineD has joined #refstack21:59
*** funzo has quit IRC22:02
*** catherineD|2 has quit IRC22:02
*** funzo has joined #refstack22:58
*** funzo has quit IRC23:03
*** pvaneck has quit IRC23:57

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