Tuesday, 2016-05-24

*** jhesketh_ is now known as jhesketh00:02
*** spzala has joined #openstack-qa00:07
*** mtanino has joined #openstack-qa00:08
*** yangyapeng has quit IRC00:08
*** ddieterly has joined #openstack-qa00:13
openstackrecheckopenstack/horizon change: https://review.openstack.org/317738 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error00:13
*** salv-orl_ has quit IRC00:13
*** spzala has quit IRC00:14
*** samueldmq has joined #openstack-qa00:19
*** patrickeast has joined #openstack-qa00:20
*** tongl_ has quit IRC00:23
*** r-mibu has quit IRC00:25
*** r-mibu has joined #openstack-qa00:26
*** dimtruck is now known as zz_dimtruck00:34
*** vhosakot has quit IRC00:35
*** pcm_ has quit IRC00:36
*** hoangcx has joined #openstack-qa00:42
*** pc_m has joined #openstack-qa00:43
*** zz_dimtruck is now known as dimtruck00:49
*** rfolco has quit IRC00:54
*** suyash has quit IRC00:55
*** rfolco has joined #openstack-qa00:56
*** wznoinsk has joined #openstack-qa00:59
*** mtanino has quit IRC01:05
*** yangyapeng has joined #openstack-qa01:15
*** yangyapeng has quit IRC01:15
*** yangyapeng has joined #openstack-qa01:16
*** cody-somerville has joined #openstack-qa01:17
*** cody-somerville has quit IRC01:17
*** cody-somerville has joined #openstack-qa01:17
*** anteaya has quit IRC01:18
*** spzala has joined #openstack-qa01:19
*** cody-somerville_ has joined #openstack-qa01:21
*** Gen has quit IRC01:22
*** Gen_ has joined #openstack-qa01:22
*** cody-somerville has quit IRC01:23
*** EinstCrazy has joined #openstack-qa01:24
openstackrecheckopenstack/glance change: https://review.openstack.org/316494 failed because of: gate-tempest-dsvm-full-devstack-plugin-ceph: https://bugs.launchpad.net/bugs/1298006 https://bugs.launchpad.net/bugs/1282876, gate-tempest-dsvm-neutron-full: https://bugs.launchpad.net/bugs/128287601:25
openstackLaunchpad bug 1298006 in OpenStack-Gate "network sometimes unreachable accessing our git server" [Medium,Triaged]01:25
openstackLaunchpad bug 1282876 in OpenStack-Gate "git clone fails with "fatal: Not a git repository", "git remote update failed."" [Critical,Fix released] - Assigned to Jeremy Stanley (fungi)01:25
*** mtanino has joined #openstack-qa01:30
*** asselin has joined #openstack-qa01:34
*** Apoorva has quit IRC01:34
*** mtanino has quit IRC01:35
*** cody-somerville_ has quit IRC01:35
*** asselin has quit IRC01:38
*** yamahata has quit IRC01:42
*** vhosakot has joined #openstack-qa01:42
*** cody-somerville_ has joined #openstack-qa01:46
*** Gen has joined #openstack-qa01:48
*** Gen_ has quit IRC01:51
*** spzala has quit IRC01:52
*** cody-somerville_ is now known as cody-somerville01:56
*** cody-somerville has joined #openstack-qa01:57
*** tqtran has quit IRC01:59
*** EinstCrazy has quit IRC02:00
*** EinstCrazy has joined #openstack-qa02:00
*** vhosakot has quit IRC02:04
*** vhosakot has joined #openstack-qa02:04
*** fnaval_ has quit IRC02:08
*** yamahata has joined #openstack-qa02:11
*** ddieterly is now known as ddieterly[away]02:15
*** artom has quit IRC02:16
*** artom has joined #openstack-qa02:17
*** fnaval has joined #openstack-qa02:17
*** ddieterly[away] is now known as ddieterly02:20
*** fnaval_ has joined #openstack-qa02:20
*** fnaval has quit IRC02:21
*** markvoelker has quit IRC02:21
*** ddieterly is now known as ddieterly[away]02:24
*** spzala has joined #openstack-qa02:26
*** spzala has quit IRC02:26
*** spzala has joined #openstack-qa02:27
*** spzala has quit IRC02:31
*** salv-orlando has joined #openstack-qa02:38
openstackgerritZhenyu Zheng proposed openstack/tempest: Modify --endpoint-type to --os-endpoint-type for nova  https://review.openstack.org/31859202:40
*** salv-orlando has quit IRC02:43
*** tmatsu has quit IRC02:44
*** tmatsu has joined #openstack-qa02:45
*** Gen_ has joined #openstack-qa02:53
*** tmatsu has quit IRC02:54
*** tqtran has joined #openstack-qa02:56
*** Gen has quit IRC02:56
*** tqtran has quit IRC03:00
*** spzala has joined #openstack-qa03:02
*** spzala has quit IRC03:08
*** erlon_ has joined #openstack-qa03:08
*** yamamot__ has joined #openstack-qa03:09
*** rfolco has quit IRC03:10
*** Gen_ has quit IRC03:12
*** cody-somerville has quit IRC03:16
*** gouthamr_ has quit IRC03:17
*** tmatsu has joined #openstack-qa03:17
*** vhosakot has quit IRC03:20
*** fnaval has joined #openstack-qa03:21
*** markvoelker has joined #openstack-qa03:22
*** ddieterly[away] has quit IRC03:24
*** fnaval_ has quit IRC03:24
*** tmatsu has quit IRC03:25
*** markvoelker has quit IRC03:27
*** tqtran has joined #openstack-qa03:32
*** tqtran has quit IRC03:33
*** dave-mcc_ has quit IRC03:38
*** _alastor__ has quit IRC03:38
*** tochi has quit IRC03:40
*** dimtruck is now known as zz_dimtruck03:52
*** yamahata has quit IRC03:53
*** _alastor__ has joined #openstack-qa03:55
*** david-lyle_ has joined #openstack-qa03:55
*** david-lyle has quit IRC03:57
*** ramishra has quit IRC03:58
*** tochi has joined #openstack-qa04:00
*** ramishra has joined #openstack-qa04:00
openstackrecheckopenstack/horizon change: https://review.openstack.org/317738 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error04:03
*** spzala has joined #openstack-qa04:04
*** fnaval_ has joined #openstack-qa04:07
*** salv-orlando has joined #openstack-qa04:08
*** fnaval has quit IRC04:10
*** spzala has quit IRC04:10
*** tmatsu has joined #openstack-qa04:13
*** salv-orlando has quit IRC04:17
*** tmatsu has quit IRC04:19
*** tmatsu has joined #openstack-qa04:20
*** yamamot__ has quit IRC04:21
*** markvoelker has joined #openstack-qa04:23
*** markvoelker has quit IRC04:28
*** Gen has joined #openstack-qa04:32
*** Gen_ has joined #openstack-qa04:36
*** Gen has quit IRC04:36
*** Poornima has joined #openstack-qa04:36
openstackrecheckConsole logs not available after 13:26s for keystone-coverage-db 320156,9,82a946804:37
*** armax has quit IRC04:41
*** sridharg has joined #openstack-qa04:43
*** asselin has joined #openstack-qa04:46
*** shausy has joined #openstack-qa04:48
*** pgadiya has joined #openstack-qa04:53
*** cody-somerville has joined #openstack-qa05:01
*** spzala has joined #openstack-qa05:02
*** bkopilov has joined #openstack-qa05:02
*** yamamoto has joined #openstack-qa05:04
*** spzala has quit IRC05:06
*** sabeen1 has quit IRC05:06
*** bkopilov has quit IRC05:10
*** bkopilov has joined #openstack-qa05:10
*** gamado_wfh has joined #openstack-qa05:10
*** Gen_ has quit IRC05:10
*** bkopilov has quit IRC05:11
*** bkopilov has joined #openstack-qa05:11
*** dmellado has quit IRC05:13
*** yamamoto has quit IRC05:14
*** yamamoto has joined #openstack-qa05:16
*** dansmith has quit IRC05:16
*** dmellado has joined #openstack-qa05:17
*** shausy has quit IRC05:17
*** shausy has joined #openstack-qa05:18
*** dansmith has joined #openstack-qa05:18
*** dansmith is now known as Guest7113205:18
openstackgerritMasayuki Igawa proposed openstack/openstack-health: Add feedback and feedback-fix css class  https://review.openstack.org/31913105:19
*** markvoelker has joined #openstack-qa05:24
*** markvoelker has quit IRC05:29
*** yfried has joined #openstack-qa05:29
*** tmatsu has quit IRC05:31
*** tmatsu has joined #openstack-qa05:32
*** binbincong has quit IRC05:32
*** cody-somerville has quit IRC05:36
*** binbincong has joined #openstack-qa05:51
*** erlon_ has quit IRC05:56
*** amotoki has joined #openstack-qa05:56
*** Gen has joined #openstack-qa06:00
*** shausy has quit IRC06:00
*** fnaval has joined #openstack-qa06:04
*** Gen has quit IRC06:05
*** fnaval_ has quit IRC06:06
*** rcernin has joined #openstack-qa06:08
*** nadya has joined #openstack-qa06:08
*** Lord_Phoenix has joined #openstack-qa06:11
*** sridharg has quit IRC06:13
*** scottda has quit IRC06:14
*** DuncanT has quit IRC06:15
*** scottda has joined #openstack-qa06:16
*** betherly has quit IRC06:16
*** larainema has quit IRC06:17
*** lekha-away has quit IRC06:18
*** DuncanT has joined #openstack-qa06:18
*** betherly has joined #openstack-qa06:19
*** lekha-away has joined #openstack-qa06:20
*** larainema has joined #openstack-qa06:20
*** isq has quit IRC06:20
*** isq has joined #openstack-qa06:21
*** woodster_ has quit IRC06:22
*** markvoelker has joined #openstack-qa06:25
*** _vsaienko has joined #openstack-qa06:25
*** nadya has quit IRC06:25
*** woodster_ has joined #openstack-qa06:26
*** samueldmq has quit IRC06:27
*** samueldmq has joined #openstack-qa06:27
*** nineseconds has joined #openstack-qa06:28
*** markvoelker has quit IRC06:29
*** grafuls has quit IRC06:32
*** _vsaienko has quit IRC06:32
*** grafuls has joined #openstack-qa06:33
*** Serlex has joined #openstack-qa06:37
*** salv-orlando has joined #openstack-qa06:37
*** itzikb has joined #openstack-qa06:39
*** sridharg has joined #openstack-qa06:46
*** Lord_Phoenix has quit IRC06:51
*** Deng has joined #openstack-qa07:02
*** spzala has joined #openstack-qa07:02
*** gamado_wfh has quit IRC07:03
*** Lord_Phoenix has joined #openstack-qa07:03
*** tesseract has joined #openstack-qa07:04
*** spzala has quit IRC07:07
*** tesseract has quit IRC07:08
*** yamahata has joined #openstack-qa07:10
*** tmatsu has quit IRC07:14
*** tmatsu has joined #openstack-qa07:15
*** rossella_s has joined #openstack-qa07:16
*** rwsu has joined #openstack-qa07:19
*** tmatsu has quit IRC07:20
*** tmatsu has joined #openstack-qa07:22
*** rwsu has quit IRC07:22
*** yfried has quit IRC07:22
*** tmatsu_ has joined #openstack-qa07:24
*** Ravikiran_K has joined #openstack-qa07:25
*** markvoelker has joined #openstack-qa07:26
*** tmatsu has quit IRC07:28
*** jlanoux has joined #openstack-qa07:29
*** markvoelker has quit IRC07:32
*** rossella_s has quit IRC07:32
*** rossella_s has joined #openstack-qa07:33
*** Lord_Phoenix has quit IRC07:34
*** ihrachys has joined #openstack-qa07:37
*** Lord_Phoenix has joined #openstack-qa07:37
*** yfried has joined #openstack-qa07:40
*** zeih has joined #openstack-qa07:41
*** anshul has joined #openstack-qa07:44
*** anshul is now known as Guest8268607:44
*** jpena|off is now known as jpena07:47
*** woodster_ has quit IRC07:48
*** jordanP has joined #openstack-qa07:51
*** dtantsur|afk is now known as dtantsur07:52
*** zeih has quit IRC07:54
afazekas_andreaf, done07:55
*** zeih has joined #openstack-qa07:56
*** beisner has quit IRC07:56
*** Lord_Phoenix has quit IRC07:58
*** Lord_Phoenix has joined #openstack-qa07:59
*** spzala has joined #openstack-qa08:04
*** beisner has joined #openstack-qa08:07
*** spzala has quit IRC08:08
*** zeih has quit IRC08:10
*** drptbl has joined #openstack-qa08:10
*** shardy has joined #openstack-qa08:12
*** lucasagomes_ is now known as lucasagomes08:12
*** apevec has joined #openstack-qa08:19
*** e0ne has joined #openstack-qa08:20
*** tosky has joined #openstack-qa08:21
*** zeih has joined #openstack-qa08:22
*** venkat has joined #openstack-qa08:23
*** tosky has quit IRC08:23
*** salv-orlando has quit IRC08:25
*** tosky has joined #openstack-qa08:26
*** afazekas_ is now known as afazekas08:27
*** markvoelker has joined #openstack-qa08:28
*** drptbl has quit IRC08:30
*** e0ne has quit IRC08:31
*** gamado has joined #openstack-qa08:31
*** hichihara has joined #openstack-qa08:32
*** e0ne has joined #openstack-qa08:32
*** markvoelker has quit IRC08:32
*** EinstCrazy has quit IRC08:37
*** zeih has quit IRC08:38
andreafafazekas: cool, thanks08:39
*** EinstCrazy has joined #openstack-qa08:40
*** ig0r_ has joined #openstack-qa08:40
openstackgerritBenny Kopilov proposed openstack/tempest: Cinder multibackend migration tests  https://review.openstack.org/31856908:49
*** EinstCrazy has quit IRC08:51
*** gszasz has joined #openstack-qa08:52
*** cdent has joined #openstack-qa08:55
*** EinstCrazy has joined #openstack-qa08:57
*** wznoinsk has quit IRC08:59
*** wznoinsk has joined #openstack-qa08:59
andreafafazekas: I updated https://review.openstack.org/#/c/319394/4 as well09:03
andreafylobankov: a question regarding https://review.openstack.org/#/c/307789/909:06
openstackgerritzhufl proposed openstack/tempest: Not to create security group when security_group ext is disabled  https://review.openstack.org/32030909:07
openstackgerritMerged openstack/tempest: Remove deprecated legacy credentials provider  https://review.openstack.org/26337609:08
*** gfidente has joined #openstack-qa09:09
ylobankovandreaf: what question?09:10
andreafylobankov: the the commit message you say that image status can only be obtained via the HEAD - but according to the API docs HEAD will get image details (including status), and GET will get image details plus the image itself09:12
andreafylobankov: so is there a bug in the docs or in the glance API?09:12
andreafylobankov: the reason for using HEAD is that unless the actual image is needed it's much quicker and more efficient just to get the image metadata and status09:13
*** tochi has quit IRC09:15
ylobankovandreaf: we can get status from HEAD, but something like this: image['x-meta-umage-status'] (not just image['status']) as far as I remember09:18
andreafylobankov: you mean from GET?09:20
andreafin case of GET the image status will be in a header as the body will be the image itself I imaging09:20
andreafimagine09:20
ylobankovyes, from GET. Sorry09:20
*** cdent has left #openstack-qa09:21
andreafylobankov: the way I see it, the main point for using HEAD is because it's lightweight - we don't want to download the image to just get it's metadata09:24
andreafylobankov: it's not because it's the only way to get image status, but because it's the most convenient09:24
andreafylobankov: this patch is the important one: https://review.openstack.org/#/c/297511/1009:26
*** nadya has joined #openstack-qa09:27
*** yfried has quit IRC09:27
andreafylobankov: in terms of naming we should try an be consistent - so if other clients use show for GET we should stick to show, and use a different name for HEAD. We can and should address this via documentation, and by fixing the waiter method to use the the HEAD method and not the GET one09:28
*** yfried has joined #openstack-qa09:28
andreafylobankov: e.g. add a note to the show method telling that it should only be used when the full image is needed else use the HEAD one09:28
*** zigo_ is now known as zigo09:28
andreafylobankov: well that's my view at least :)09:29
ylobankovandreaf: the thing is that wait_for_image_status expects only client.show_image method. So that is why I made https://review.openstack.org/#/c/307789/09:31
andreafsure, but wait_for_image_status can be changed :)09:31
*** jordanP has quit IRC09:32
ylobankovandreaf: Probably, I should abandon https://review.openstack.org/#/c/307789/ and make something in https://review.openstack.org/#/c/297511/ :)09:32
*** tmatsu has joined #openstack-qa09:34
*** gmann_ has joined #openstack-qa09:34
andreafylobankov: that's an option09:35
ylobankovandreaf: Let's rename get_image_meta into show_image_meta in https://review.openstack.org/#/c/307789/ and make the corresponding changes in https://review.openstack.org/#/c/297511/09:35
*** Poornima has quit IRC09:37
*** tmatsu_ has quit IRC09:37
andreafylobankov: apart from the object-storage client, with list_account_metadata, all other usages of HEAD have a check_ method09:38
*** Poornima has joined #openstack-qa09:38
andreafylobankov: but it's difficult to get a consistent naming because not all APIs use the methods for the same purpose09:38
andreafylobankov: so if we go for a fixed method -> name mapping, the name will not always be consistent with what the method actually do09:39
andreafylobankov: but I think it's still the less confusing option - what the method does can go in docstring09:39
andreafylobankov: perhaps it's a topic for the meeting on Thursday? I would like to get an agreement on this, and add it to the HACKING does so we have a common rule to follow when coding / reviewing, and we can avoid driving people crazy with conflicting reviews09:40
ylobankovandreaf: what is better for you check_image_meta or show_image_meta ? ok, let's discuss it on Thursday with folks09:42
openstackgerritWenzhi Yu proposed openstack/tempest: WIP: Add Magnum clients to tempest  https://review.openstack.org/32033009:43
*** jordanP has joined #openstack-qa09:45
openstackgerritAndrea Frittoli proposed openstack/tempest: Hash credentials on user, project/tenant and pwd  https://review.openstack.org/31827209:52
*** Poornima has quit IRC09:52
*** mvk has quit IRC09:53
*** fnaval has quit IRC09:54
openstackgerritAndrea Frittoli proposed openstack/tempest: Account generator for identity v3  https://review.openstack.org/31710509:55
*** sambetts|afk is now known as sambetts09:55
openstackgerritLee Yarwood proposed openstack/tempest: Add stable disk device instance rescue tests  https://review.openstack.org/31986309:56
andreafmtreinish: I rebased the https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bug/1494293 series after the legacy cred patch merged - if you could re-review. The auth scope series does not seem to have any conflict09:59
openstackrecheckopenstack/horizon change: https://review.openstack.org/316763 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error09:59
*** hoangcx_ has joined #openstack-qa10:02
*** hoangcx has quit IRC10:03
*** Poornima has joined #openstack-qa10:04
*** gszasz has quit IRC10:05
*** spzala has joined #openstack-qa10:05
*** apevec has quit IRC10:07
*** spzala has quit IRC10:10
*** yamahata has quit IRC10:13
*** e0ne has quit IRC10:13
*** e0ne has joined #openstack-qa10:14
openstackgerritYaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client  https://review.openstack.org/30778910:14
*** tmatsu has quit IRC10:14
*** zeih has joined #openstack-qa10:17
*** Lord_Phoenix has quit IRC10:21
*** zeih has quit IRC10:21
*** mvk has joined #openstack-qa10:22
*** early has quit IRC10:23
*** early has joined #openstack-qa10:23
openstackgerritYaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client  https://review.openstack.org/30778910:28
*** nadya has quit IRC10:28
*** markvoelker has joined #openstack-qa10:29
*** jlanoux has quit IRC10:30
*** yfried has quit IRC10:31
openstackgerritYaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client  https://review.openstack.org/30778910:31
*** jlanoux has joined #openstack-qa10:31
*** asselin has quit IRC10:31
ylobankovandreaf, afazekas: what do you think about it? https://review.openstack.org/#/c/307789/10:31
*** EinstCrazy has quit IRC10:32
*** EinstCrazy has joined #openstack-qa10:32
openstackgerritYaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client  https://review.openstack.org/30778910:32
*** yangyapeng has quit IRC10:34
*** markvoelker has quit IRC10:34
*** EinstCrazy has quit IRC10:37
*** e0ne_ has joined #openstack-qa10:41
*** hoangcx_ has quit IRC10:41
openstackrecheckopenstack/horizon change: https://review.openstack.org/316766 failed gate-horizon-python27, gate-horizon-dsvm-integration in the gate queue with an unrecognized error10:41
*** e0ne has quit IRC10:42
*** jlanoux has quit IRC10:42
*** jlanoux has joined #openstack-qa10:43
*** binbincong has quit IRC10:44
*** gszasz has joined #openstack-qa10:58
andreafylobankov: ok that looks good to me. In the follow-up patch you'll have to invoke now a different method depending on whether it's v1 or v2 client10:59
ylobankovandreaf: yep11:00
andreafcool11:02
*** gszasz has quit IRC11:04
andreafmkoderer, jordanP: this should be pretty straight-forward: https://review.openstack.org/#/c/319394/11:05
openstackgerritYaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client  https://review.openstack.org/30778911:06
*** spzala has joined #openstack-qa11:06
jordanPandreaf,11:08
jordanP            client.update_user_own_password(user_id, password=old_pass,11:08
jordanP                                            original_password=new_pass)11:08
*** Ravikiran_K has quit IRC11:08
jordanParf no nothing I got it now11:08
jordanPsorry11:08
andreafjordanP: yep, restoring the original password11:09
ylobankovandreaf: I have updated the commit message and your +2 has gone. Could you please voite one more time? thanks in advance.11:09
andreafylobankov: done11:09
ylobankovandreaf: cool11:10
*** spzala has quit IRC11:10
openstackrecheckopenstack/horizon change: https://review.openstack.org/318357 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error11:11
*** salv-orlando has joined #openstack-qa11:15
*** venkat has quit IRC11:15
*** lucasagomes is now known as lucas-hungry11:18
*** salv-orlando has quit IRC11:19
openstackgerritYaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere  https://review.openstack.org/29751111:21
*** nadya has joined #openstack-qa11:25
openstackgerritDale Smith proposed openstack/tempest: Add config option identity/member_role  https://review.openstack.org/30233211:25
*** sdague has joined #openstack-qa11:25
*** jpena is now known as jpena|lunch11:27
*** gmann_ has quit IRC11:30
*** markvoelker has joined #openstack-qa11:30
*** rodrigods has quit IRC11:31
*** rodrigods has joined #openstack-qa11:31
*** nineseconds has quit IRC11:32
*** gamado has quit IRC11:32
*** vgridnev has quit IRC11:33
*** ig0r_ has quit IRC11:34
*** markvoelker has quit IRC11:35
*** vgridnev has joined #openstack-qa11:35
*** vgridnev has quit IRC11:40
*** vgridnev has joined #openstack-qa11:40
*** ddieterly has joined #openstack-qa11:41
*** hichihara has quit IRC11:43
daleesandreaf: sorry, i was slow posting my reply :)11:43
daleesandreaf: and thanks for the feedback. happy to discuss here also if that's easier11:44
*** yfried has joined #openstack-qa11:45
andreafdalees: just reading your comment11:45
*** salv-orlando has joined #openstack-qa11:46
andreafdalees: actually tempest allows to configure the list of roles to be used. 'Member' is only used as a default only if no list is configured, because if not role is added on the project then the account won't work11:47
openstackgerritLee Yarwood proposed openstack/tempest: Add stable disk device instance rescue tests  https://review.openstack.org/31986311:47
andreafdalees: an option could be to use 'Member' as default value for the existing config option, I'd have to check through the code if this would work but I think so11:48
*** Lord_Phoenix has joined #openstack-qa11:48
andreafdalees: so what I was proposing if for both credential providers (dynamic and pre-prov) to honour the same config option11:48
daleesandreaf, but these are preprovided creds, in our case this user is created with a role of '_member_', so I thought it'd be good to add support for that too.11:49
andreafhttp://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n7211:49
andreafright, in one case users are created by tempest with a list of roles, in another case users are pre-provisioned with a list of roles (or just one) - my proposal was to just have a single 'list of roles' config option which is used for both cases11:50
daleesandreaf, but i'm not trying too hard to push the limitations of my deployment on tempest; just make it more configurable. I'm happy to make the change to hardcode or remove hardcoding from dynamic creds also11:51
daleesahh11:51
daleesinteresting, so with dynamic creds, tempest_roles is applied to all users that aren't admin. Seems very similar to the list that I'm trying to add, i see now.11:53
andreafright11:53
*** askb has joined #openstack-qa11:53
andreafhttp://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/dynamic_creds.py#n13811:53
openstackgerritYaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere  https://review.openstack.org/29751111:54
andreafwell actually the list of roles is assigned to all users, including admin11:54
daleesit would need some updating of the help text, but would work. when a test runs that doesn't specify any roles it would need to require all roles from that list instead of 'anything'11:54
*** rfolco has joined #openstack-qa11:54
andreafyep11:55
andreafyes the help text would need updating11:55
*** ddieterly is now known as ddieterly[away]11:55
andreafdalees: we already have many many config options and it's one of the pain points of people using tempest, so I try to avoid adding new ones when there's a reasonable alternative :)11:56
*** ig0r_ has joined #openstack-qa11:56
openstackrecheckConsole logs not available after 13:29s for gate-tempest-dsvm-neutron-full 318334,3,ec1b59211:57
*** nineseconds has joined #openstack-qa11:58
daleesandreaf: yes, as a user i understand!11:58
daleesandreaf, I'll make the change to use tempest_roles. I think that will work fine for this; thanks for the feedback12:00
andreafdalees: thank you12:01
*** markvoelker has joined #openstack-qa12:02
*** yamamoto has quit IRC12:04
mkodererjordanP: andreaf change LGTM - any comment or should I +A?12:04
*** itzkb_ has joined #openstack-qa12:05
*** itzikb has quit IRC12:06
*** spzala has joined #openstack-qa12:07
*** lucas-hungry is now known as lucasagomes12:10
*** spzala has quit IRC12:11
*** Ravikiran_K has joined #openstack-qa12:16
*** ddieterly[away] is now known as ddieterly12:18
andreafmkoderer: no comment from my side :)12:21
andreafmkoderer: I think it should be fine to go12:21
*** Lord_Phoenix has quit IRC12:25
*** tmatsu has joined #openstack-qa12:26
*** tmatsu has quit IRC12:31
*** beisner has quit IRC12:32
*** salv-orlando has quit IRC12:35
openstackgerritRodrigo Duarte proposed openstack-dev/devstack: Tempest: add a Keystone reseller feature flag  https://review.openstack.org/29349712:36
*** itzkb_ has quit IRC12:39
*** itzikb has joined #openstack-qa12:39
*** aysyd has joined #openstack-qa12:42
openstackgerritKashyap Chamarthy proposed openstack-dev/devstack: functions-common: Use `systemctl` when applicable  https://review.openstack.org/31920912:43
openstackgerritKashyap Chamarthy proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon  https://review.openstack.org/31981912:46
*** jpena|lunch is now known as jpena12:47
*** asselin has joined #openstack-qa12:50
openstackgerritLee Yarwood proposed openstack-dev/devstack: Enable stable disk device rescue tests for libvirt  https://review.openstack.org/32042812:51
*** gouthamr has joined #openstack-qa12:51
*** itzikb has quit IRC12:53
*** itzikb has joined #openstack-qa12:53
*** akshai has joined #openstack-qa12:53
*** beisner has joined #openstack-qa12:54
*** akshai_ has joined #openstack-qa12:54
*** Lord_Phoenix has joined #openstack-qa12:54
*** vponomaryov has quit IRC12:55
*** jordanP has quit IRC12:55
*** jordanP has joined #openstack-qa12:57
*** vponomaryov has joined #openstack-qa12:57
*** Deng has quit IRC12:57
*** pauloewerton has joined #openstack-qa12:57
*** david-lyle has joined #openstack-qa12:57
*** akshai has quit IRC12:58
*** lucasagomes is now known as lucas-afk13:01
*** david-lyle_ has quit IRC13:01
*** ddieterly has quit IRC13:02
*** Lord_Phoenix has quit IRC13:04
*** edmondsw has joined #openstack-qa13:05
*** Lord_Phoenix has joined #openstack-qa13:06
*** rlrossit has joined #openstack-qa13:07
*** alyson_ has joined #openstack-qa13:07
*** spzala has joined #openstack-qa13:07
*** Lord_Phoenix has quit IRC13:08
*** Lord_Phoenix has joined #openstack-qa13:09
*** elynn has joined #openstack-qa13:10
*** salv-orlando has joined #openstack-qa13:11
openstackgerritMerged openstack/tempest: Removed q-vpn from the list of dirty logs  https://review.openstack.org/31124313:11
openstackgerritMerged openstack/tempest: Modify projects_client to receive more attributes  https://review.openstack.org/31525313:12
*** spzala has quit IRC13:12
kashyapVariables declared in 'stackrc' can be used by any source file, right?  (E.g. the ones in lib/)13:13
* kashyap finds it out by trying...13:15
*** elynn has left #openstack-qa13:16
openstackgerritKashyap Chamarthy proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon  https://review.openstack.org/31981913:16
*** porrua has joined #openstack-qa13:17
openstackrecheckopenstack/horizon change: https://review.openstack.org/309482 failed gate-horizon-python27 in the gate queue with an unrecognized error13:18
*** pgadiya has quit IRC13:21
*** hichihara has joined #openstack-qa13:21
*** _vsaienko has joined #openstack-qa13:22
*** pgadiya has joined #openstack-qa13:22
*** dave-mccowan has joined #openstack-qa13:24
*** nineseconds has quit IRC13:24
*** nineseconds has joined #openstack-qa13:24
*** dustins has joined #openstack-qa13:24
*** _vsaienko has quit IRC13:25
*** zz_dimtruck is now known as dimtruck13:27
*** nineseconds has quit IRC13:28
*** nineseconds has joined #openstack-qa13:28
openstackgerritKashyap Chamarthy proposed openstack-dev/devstack: functions-common: Use `systemctl` when applicable  https://review.openstack.org/31920913:28
*** dustins has quit IRC13:29
*** ddieterly has joined #openstack-qa13:29
*** dustins has joined #openstack-qa13:29
*** regXboi has joined #openstack-qa13:29
openstackgerritKashyap Chamarthy proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon  https://review.openstack.org/31981913:31
openstackgerritRodrigo Duarte proposed openstack/tempest: Add parent_id to create_project  https://review.openstack.org/28554113:33
*** yfried has quit IRC13:34
*** Poornima has quit IRC13:36
*** pgadiya has quit IRC13:36
*** fesp has joined #openstack-qa13:36
*** nineseconds has quit IRC13:38
*** nineseconds has joined #openstack-qa13:38
openstackgerritMerged openstack/tempest: Fix update_password tests  https://review.openstack.org/31939413:39
*** fesp has quit IRC13:40
*** ddieterly is now known as ddieterly[away]13:41
kashyapAnyone else notice it too?  With current DevStack git -- "./stack.sh: line 494: generate-subunit: command not found" -- http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/09/319209/9/16214/logs/devstacklog.txt.gz13:46
*** _vsaienko has joined #openstack-qa13:46
*** _vsaienko has quit IRC13:47
*** ddieterly[away] is now known as ddieterly13:48
*** nineseconds has quit IRC13:48
*** nineseconds has joined #openstack-qa13:48
kashyapafazekas: ^ When you get a sec, maybe you have a comment whether if I screwed up something...13:48
kashyapafazekas: Duh, ignore me; missing space after 'True'...13:50
*** nwkarsten has joined #openstack-qa13:51
*** yamahata has joined #openstack-qa13:52
*** jbernard_ is now known as jbernard13:52
*** _vsaienko has joined #openstack-qa13:53
*** anteaya has joined #openstack-qa13:54
*** fnaval has joined #openstack-qa13:54
dmelladokashyap: 'spacing' out? :P13:55
kashyapdmellado: Yeah, awful bash mistake :-)13:56
*** rbak has joined #openstack-qa13:58
*** hichihar_ has joined #openstack-qa14:03
*** sabeen has joined #openstack-qa14:03
*** yamamoto has joined #openstack-qa14:05
afazekaskashyap, http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%20%5C%22generate-subunit%3A%20command%20not%20found%5C%2214:08
*** dustins has quit IRC14:09
*** dustins has joined #openstack-qa14:09
*** fnaval has quit IRC14:09
*** woodster_ has joined #openstack-qa14:09
openstackgerritAndrea Frittoli proposed openstack/tempest: Introduce scope in the auth API  https://review.openstack.org/31317114:11
*** Guest71132 is now known as dansmith14:11
*** pglass has joined #openstack-qa14:11
*** dansmith is now known as Guest2162114:12
*** lucas-afk is now known as lucasagomes14:12
kashyapafazekas: Thanks14:12
afazekaskashyap, https://bugs.launchpad.net/devstack/+bug/154737914:13
openstackLaunchpad bug 1547379 in devstack "./stack.sh: line 488: generate-subunit: command not found" [Undecided,Invalid]14:13
kashyapAh-ha14:13
*** hichihara has quit IRC14:13
*** Guest21621 is now known as dansmith14:14
*** spzala has joined #openstack-qa14:14
*** cody-somerville has joined #openstack-qa14:14
*** cody-somerville has quit IRC14:14
*** cody-somerville has joined #openstack-qa14:14
*** spzala has quit IRC14:14
mtreinishafazekas: that error is a false positive, it's happening because generate-subunit is called in the exit hook14:15
mtreinishif devstack fails before os-testr gets pip installed (which is very early in stack.sh, but not first) that error message will appear, but it's not the underlying cause14:16
mtreinishkashyap: ^^^14:16
*** mfisher_ora has joined #openstack-qa14:16
openstackgerritMatthew Treinish proposed openstack/tempest: Add tempest run command  https://review.openstack.org/28441114:17
*** ajmiller_ has joined #openstack-qa14:18
*** spzala has joined #openstack-qa14:18
*** sridharg has quit IRC14:18
afazekasmtreinish: I just read the trace, BTW where is the detailed info ?14:18
afazekasmtreinish: Do not have the debug info from the  $TOP_DIR/tools/install_pip.sh14:20
*** rossella_s has quit IRC14:24
*** rossella_s has joined #openstack-qa14:24
afazekaskashyap, install_pip.sh likely does not show detailed info about 'source $TOP_DIR/stackrc'  issues , so if you modified some `inc`  thing it can be an issue in your change..14:26
kashyapSorry, what is `inc thing`?14:27
kashyapmtreinish: Thanks for correcting14:27
mtreinishafazekas: I was just looking at the devstacklogs from the logstash link14:27
mtreinishafazekas: I don't know why install_pip.sh failed14:28
*** ajmiller has quit IRC14:31
jordanPwhy on earth people keep duplicating these 3 lines:14:31
jordanP    def _delete_volume(self, volume_id):14:31
jordanP        self.volumes_client.delete_volume(volume_id)14:31
jordanP        self.volumes_client.wait_for_resource_deletion(volume_id)14:31
afazekasmtreinish, non of the logstash failures are from a gate job14:31
*** sridharg has joined #openstack-qa14:32
*** pilgrimstack1 has joined #openstack-qa14:32
jordanPandreaf, afazekas mkoderer could one of you have a look at https://review.openstack.org/#/c/311112/ I'd like to move on and focus on other area where tempest could be refactored/lose some weight14:32
jordanPthanks14:33
*** ddieterly is now known as ddieterly[away]14:33
*** _vsaienko has quit IRC14:34
mtreinishjordanP: by dechunkify I did mean print the body or a summary of the body (like <1024 Bytes of binary data>)14:34
*** _vsaienko has joined #openstack-qa14:34
mtreinishbecause just saying it's a generator object is a bit weird14:34
mtreinishbut that can be a follow up change14:34
*** e0ne has joined #openstack-qa14:34
*** e0ne_ has quit IRC14:35
*** askb has quit IRC14:37
jordanPmtreinish, got you. But just to print "1024 Bytes of binary data" I would have to "consume" the generator and "rewind "it. Maybe I can do without a generator though14:40
openstackgerritSteve Heyman proposed openstack/tempest: Remove neutron OO wrappers  https://review.openstack.org/32049514:41
*** cody-somerville has quit IRC14:42
*** anteaya has quit IRC14:42
*** david-lyle has quit IRC14:42
*** itzikb has quit IRC14:42
*** beisner has quit IRC14:42
*** aysyd has quit IRC14:42
*** mvk has quit IRC14:42
*** wznoinsk has quit IRC14:42
*** vponomaryov has quit IRC14:42
*** pilgrimstack has quit IRC14:42
*** akshai_ has quit IRC14:42
*** ajmiller_ has quit IRC14:43
*** StevenK_ is now known as StevenK14:44
*** _vsaienko has quit IRC14:45
*** regXboi has quit IRC14:46
*** regXboi has joined #openstack-qa14:46
afazekasjordanP, chunkify is it really working with any iterable ?14:47
*** grafuls has quit IRC14:47
*** rcernin has quit IRC14:47
jordanPafazekas, no :)14:47
afazekas-114:48
afazekasare you planning to fix it ?14:48
jordanPafazekas, no14:48
afazekasare you planning to use less confusing variable name ?14:49
jordanPI am not coding a ferari when all I need is a toyota14:49
*** _vs has joined #openstack-qa14:49
jordanPnot all iterables support slices14:49
jordanPso I am not sure what do you want me to change ?14:49
*** hichihar_ has quit IRC14:49
afazekassliceable14:49
jordanPis that a thing ?14:50
afazekasfrom today it is :)14:50
jordanPlol, Okay :) I like the idea14:50
*** akshai has joined #openstack-qa14:50
* afazekas I try to figure out some good iterable way quickly14:51
*** yangyapeng has joined #openstack-qa14:52
*** ddieterly[away] is now known as ddieterly14:55
*** weshay has joined #openstack-qa14:55
*** ddieterly is now known as ddieterly[away]14:57
openstackgerritBenny Kopilov proposed openstack/tempest: cinder service enable / disable tests  https://review.openstack.org/31963414:58
*** _vs has quit IRC14:58
*** slowrie_ is now known as slowrie14:59
mtreinishandreaf: good call on needing release notes on 311112. We need to get better about that :)14:59
*** rcernin has joined #openstack-qa14:59
*** nineseconds has quit IRC15:00
*** _vsaienko has joined #openstack-qa15:02
*** e0ne has quit IRC15:06
*** dustins has quit IRC15:06
afazekasjordanP, What about the solution with 194 votes ? http://stackoverflow.com/questions/312443/how-do-you-split-a-list-into-evenly-sized-chunks-in-python/31246415:10
afazekasizip_longest15:10
*** yamamoto has quit IRC15:11
*** sridharg has quit IRC15:11
*** Kennan has quit IRC15:11
jordanPreturn izip_longest(*[iter(iterable)]*n, fillvalue=padvalue)15:12
jordanPlol15:12
*** nineseconds has joined #openstack-qa15:12
jordanPI don't want to scratch my head or open the python documentation for 15min each time I read that code15:12
jordanPthere's no way I will remember what izip_longest do15:13
*** vhosakot has joined #openstack-qa15:13
*** _vsaienko has quit IRC15:13
jordanPbut I am going to replace "iterable" by "sliceable" it's a good suggestion and more accurate15:13
*** Kennan has joined #openstack-qa15:14
afazekasYes, it is really unbelievable the  izip_longest seams to be the best answer (still checking)15:14
afazekasThere should be one-- and preferably only one --obvious way to do it.15:14
afazekasAlthough that way may not be obvious at first unless you're Dutch.15:14
afazekasimport this ;)15:14
sdaguesc68cal / kashyap - why did we introduce a global variable that's not guarunteed to be created in functions-common?15:15
sdaguehttps://review.openstack.org/#/c/319209/1115:15
sdaguethe -x check was fine15:15
openstackgerritLiron Kuchlani proposed openstack/tempest: Volume pagination with specific tenant  https://review.openstack.org/31804815:15
*** Junhongl_ has joined #openstack-qa15:16
*** yamamoto has joined #openstack-qa15:17
afazekasjordanP:  Would nice if at least our library function would be able handle larger stream than our memory ..15:17
*** e0ne has joined #openstack-qa15:18
jordanPafazekas, agreed. But given that the bigest stream we have to handle in tests are ~MB I am not sure it's worth it15:18
sdaguedtroyer_zz: why do you really want abs paths in devstack?15:19
*** Junhongl_ has left #openstack-qa15:20
*** _vs has joined #openstack-qa15:22
openstackgerritSean Dague proposed openstack-dev/devstack: functions-common: Use `systemctl` when applicable  https://review.openstack.org/31920915:22
openstackgerritSean Dague proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon  https://review.openstack.org/31981915:22
*** jlanoux has quit IRC15:22
*** yamamoto has quit IRC15:24
afazekasjordanP, '50' voted answer seams the less tricky15:24
*** nineseconds has quit IRC15:25
openstackrecheckopenstack/horizon change: https://review.openstack.org/316766 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error15:26
*** sridharg has joined #openstack-qa15:27
openstackgerritStephen Lowrie proposed openstack/tempest: Centralized Workspaces  https://review.openstack.org/28377015:28
*** Lord_Phoenix has quit IRC15:28
dtroyer_zzsdague: because it is a solid best-practice with sudo in scripts.15:29
* dtroyer_zz just looked at that review again15:30
dtroyer_zzgood grief15:30
dtroyer_zzhonestly, I just don't care any more15:31
dtroyer_zztime to rewrite devstack in python15:32
*** nadya has quit IRC15:33
*** _vs has quit IRC15:33
mtreinishslowrie: did you get the run_argv stuff working? I'm hitting some issues when it when running it: http://paste.openstack.org/show/498683/15:33
sdaguedtroyer_zz: I would agree with that if you are targetting one platform15:33
sdaguebut reinventing PATH inside of devstack seems really really weird15:33
mtreinishslowrie: that's definitely on the testr side, I'm not sure why it can't find it's own partitioned test lists15:33
sdagueif [ -x /usr/sbin/service ]; .... elif [ -x /sbin/service ]; ....15:34
*** salv-orlando has quit IRC15:34
slowriemtreinish: The code I had in my WIP was working to my knowledge15:34
sdaguethat should be a signal to just use path for that15:34
dtroyer_zzsdague: it isn't re-inventing PATH, it is removing opportunities for surprise, which in other situations is a large security problem15:34
sdaguedtroyer_zz: sure15:34
dtroyer_zzbut whatever15:34
sdaguebut... in something as big as devstack.... which modifies everything, it seems odd to care. Honestly, I'm fine circling around on that bit later. The more important thing is we *really* need systemctl in there15:35
mtreinishslowrie: hmm it works when I pass --subunit, it must be something with the subunit-trace stdin side messing things up15:35
mtreinishbut that's a weird way for it to show up15:35
dtroyer_zzso put it in.  interesting that the current PS removes path from service as it adds full path for systemctl15:36
*** Lord_Phoenix has joined #openstack-qa15:36
slowriemtreinish: I do remember there being something weird about when I tried to take the stdout and put it in as stdin to another cmd15:36
slowriethat is a very strange way that it's bubbling out though15:36
dtroyer_zzsdague: can I quote and reuse the "stop over-optimizing" comment?15:36
dtroyer_zzI'm thinking of a plane with a banner for Barcelona15:37
sc68calsdague: gotcha. Sorry - yeah it was a micro-optimization15:37
jordanPdtroyer_zz, I would stand behind that banner !15:38
*** _vs has joined #openstack-qa15:38
kashyapsdague: I'm on a moving bus, reading the one line I was addressed to --15:38
*** ihrachys has quit IRC15:39
kashyapsdague: sc68cal was insisting on optimizing the check for `systemctl` -- FWIW, I argued that I don't see a great benefit there15:39
slowriemtreinish: One difference from mine & yours (not sure if it'll fix it though) is that for stdout I used 'io.TextIOWrapper(io.BufferedWriter(io.BytesIO()))' instead of straight io.BytesIO15:39
kashyapI see that sc68cal has now reviewed that change though15:39
*** fnaval has joined #openstack-qa15:40
*** vponomaryov has joined #openstack-qa15:40
* sc68cal has the dent in his head from sdague's stick15:40
openstackgerritDale Smith proposed openstack/tempest: tempest_roles now used as default roles for preprov_creds.  https://review.openstack.org/30233215:40
*** Junhongl_ has joined #openstack-qa15:40
mtreinishslowrie: that fixed it, I copied what you had in the wip (more or less, there was a couple typos) and now it works15:41
sdaguedtroyer_zz: ok, so I'm fine adding those other full paths in once we get some test results here. Because libvirt still isn't working, and we don't really know why15:41
*** salv-orlando has joined #openstack-qa15:41
kashyapsdague: Alright, the function is what exactly I suggested before.  I'll do that, and hopefully that'll suffice15:41
mtreinishslowrie: heh, doesn't it make sense why I used subprocess in ostestr...15:42
*** _vs has quit IRC15:42
slowriemtreinish: It's a nasty problem to solve, but that's more because Pythons types are insane15:42
kashyapdtroyer_zz: No full-path because, not sure if you've seen my comment on PS8, from reading `sudoers` man page and some testing15:43
kashyapdtroyer_zz: `sudo` retains all the paths we need15:43
kashyap    $ sudo -V | grep "Value to override user'"''"s \$PATH with"15:43
kashyap    Value to override user's $PATH with: /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin15:43
slowriemtreinish: Can you help push me in the right direction with oslo_concurrency.lockutils? Trying to add it to the workspace pr and now it just seems to be locking forever when I run my unittests15:43
kashyapsdague: dtroyer_zz The above, when run as `root` will tell us what will override user's path (although that's configurable still)15:43
slowriemtreinish: current code is up @ https://review.openstack.org/#/c/28377015:43
*** Junhongl_ has quit IRC15:44
openstackgerritMatthew Treinish proposed openstack/tempest: Add tempest run command  https://review.openstack.org/28441115:44
jordanPafazekas, that solution doesn't work as is15:44
jordanPit fails the unit test "test_chunkify" that I just added15:44
sdaguekashyap: honestly, I'm fine with full paths, but I'm now actually trying to figure out if stop / start is still going to be required for libvirt15:44
jordanPtesttools.matchers._impl.MismatchError: 'aa' != ['a', 'a']15:44
sdagueso... lets let the experimental results get back, and figure this thing out, then land the changes we need15:44
*** Junhongl_ has joined #openstack-qa15:44
kashyapsdague: Yeah, that `systemctl` presence optimization was a useless waste of time15:45
kashyapI told much earlier about a function for that15:45
*** grafuls_ has joined #openstack-qa15:45
*** ajmiller_ has joined #openstack-qa15:45
*** anteaya has joined #openstack-qa15:45
*** david-lyle has joined #openstack-qa15:45
*** beisner has joined #openstack-qa15:45
*** itzikb has joined #openstack-qa15:45
*** aysyd has joined #openstack-qa15:45
*** mvk has joined #openstack-qa15:45
*** wznoinsk has joined #openstack-qa15:45
*** ajmiller_ has quit IRC15:45
*** ajmiller_ has joined #openstack-qa15:45
*** grafuls_ has quit IRC15:45
*** grafuls_ has joined #openstack-qa15:45
openstackgerritJordan Pittier proposed openstack/tempest: Swift object client: use urllib3 builtin support for chunked transfer  https://review.openstack.org/31111215:46
kashyapsdague: Full paths or not - doesn't matter.  Are you saying: (a) retain full paths; or (b) you agree with the current note that avoiding it doesn't bring in any trouble?15:46
kashyapBut I don't want to get hung up on that.  And agreed - about the experimental results15:46
*** beisner has quit IRC15:46
jordanPhumm I forgot about thet Reno note...15:47
*** Junhongl_ has quit IRC15:47
*** Junhongl has joined #openstack-qa15:48
*** yamamoto has joined #openstack-qa15:49
mtreinishslowrie: you need to use: https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/fixture/lockutils.py#L5515:50
*** beisner has joined #openstack-qa15:50
mtreinishadd that to the setup in your unit tests and it should work15:50
openstackgerritRodrigo Duarte proposed openstack-dev/devstack: Tempest: add a Keystone reseller feature flag  https://review.openstack.org/29349715:51
slowrieok, will do15:51
*** Junhongl has quit IRC15:51
*** sridharg has quit IRC15:54
*** Swami has joined #openstack-qa15:55
openstackgerritYaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere  https://review.openstack.org/29751115:55
*** yamamoto has quit IRC15:57
*** grafuls_ has quit IRC15:57
*** ajmiller_ has quit IRC15:57
*** anteaya has quit IRC15:57
*** david-lyle has quit IRC15:57
*** itzikb has quit IRC15:57
*** aysyd has quit IRC15:57
*** mvk has quit IRC15:57
*** wznoinsk has quit IRC15:57
openstackgerritJordan Pittier proposed openstack/tempest: Swift object client: use urllib3 builtin support for chunked transfer  https://review.openstack.org/31111215:57
*** armax has joined #openstack-qa15:57
*** cody-somerville has joined #openstack-qa15:57
*** ddieterly[away] has quit IRC15:57
*** weshay has quit IRC15:58
*** cody-somerville has quit IRC16:00
*** _vs has joined #openstack-qa16:00
*** _vs has quit IRC16:02
*** ddieterly has joined #openstack-qa16:02
*** _vsaienko has joined #openstack-qa16:05
oomichiandreaf: about https://review.openstack.org/#/c/313171/19/tempest/lib/auth.py16:06
*** jlanoux has joined #openstack-qa16:06
*** grafuls_ has joined #openstack-qa16:06
*** ajmiller_ has joined #openstack-qa16:06
*** david-lyle has joined #openstack-qa16:06
*** itzikb has joined #openstack-qa16:06
*** aysyd has joined #openstack-qa16:06
*** mvk has joined #openstack-qa16:06
*** wznoinsk has joined #openstack-qa16:06
oomichiandreaf: self.scope and self._scope seem a little difficult to read. can we rename self._scope to clear meaning name16:06
jordanPandreaf, I added a reno note, could you have a look if that matches your expectation?16:07
*** yamamoto has joined #openstack-qa16:08
*** lezbar has joined #openstack-qa16:09
*** dustins has joined #openstack-qa16:10
slowriemtreinish: ended up fixing the problem by changing the name of the lock from workspace to workspaces16:13
kashyapsdague: Ah, thanks for the fix-up, just noticed it.16:14
*** e0ne has quit IRC16:14
*** _vsaienko has quit IRC16:14
openstackgerritJordan Pittier proposed openstack/tempest: Remove the Glance HTTP client. Use the common Rest Client instead.  https://review.openstack.org/31117416:14
openstackgerritStephen Lowrie proposed openstack/tempest: Centralized Workspaces  https://review.openstack.org/28377016:15
slowrieandreaf: ^ added oslo_concurrency.lockutils16:15
openstackgerritStephen Lowrie proposed openstack/tempest: Centralized Workspaces  https://review.openstack.org/28377016:16
*** itzikb has quit IRC16:16
*** _vsaienko has joined #openstack-qa16:18
openstackgerritSean M. Collins proposed openstack-dev/devstack: Neutron: stop creating neutron debug probles  https://review.openstack.org/31874616:19
openstackgerritSean M. Collins proposed openstack-dev/devstack: Neutron: stop creating neutron debug probes  https://review.openstack.org/31874616:21
*** yamamoto has quit IRC16:21
*** yamamoto has joined #openstack-qa16:22
*** yamahata has quit IRC16:22
*** ig0r_ has quit IRC16:24
openstackgerritJordan Pittier proposed openstack/tempest: Heat: wait condition: allow insecure HTTPS url  https://review.openstack.org/29281316:26
openstackgerritYaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere  https://review.openstack.org/29751116:28
openstackrecheckConsole logs not available after 13:37s for keystone-coverage-db 320156,9,ab61a7116:33
*** alex_xu has quit IRC16:35
openstackrecheckopenstack/horizon change: https://review.openstack.org/301770 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error16:36
*** alex_xu has joined #openstack-qa16:38
openstackgerritBenny Kopilov proposed openstack/tempest: cinder service enable / disable tests  https://review.openstack.org/31963416:39
*** grafuls_ has quit IRC16:40
*** Guest82686 has quit IRC16:43
*** _vsaienko has quit IRC16:43
*** Swami has quit IRC16:45
*** mvk has quit IRC16:45
afazekasjordanP: use ''.join instead of list()16:48
*** pilgrimstack1 has quit IRC16:49
*** yamahata has joined #openstack-qa16:49
jordanPafazekas, where ?16:49
afazekasjordanP, Which version did you tried ? the 50 vote ? it has en explicit list() call16:51
jordanPyeah the 50 vote16:51
jordanPbut nevermind I am not micro optimizing this. My 2 lines method works, is readable and fast enough. It covers all of the current needs16:52
afazekaswe need some encoding neutral builder..16:52
jordanPit's not needed16:52
jordanPit would be nice16:52
*** ifarkas has quit IRC16:53
jordanPthe point of this patch is to remove some ~100 LoC not to micro optimize or have a general approach for everyone's problem16:53
afazekasjordanP, makes sense, I am obviously over thinning it :) , Have you read the abc comment,  we might require a sequence at the moment  not a sliceable ;)16:53
afazekasnow, I am ok with fixing it when needed ..16:54
*** Apoorva has joined #openstack-qa16:55
jordanPafazekas, ok, sequence you got it :)16:55
afazekasThe normal fix would be adding something to the standard py library, which does this via  native code in a normal way ..16:56
openstackgerritJordan Pittier proposed openstack/tempest: Swift object client: use urllib3 builtin support for chunked transfer  https://review.openstack.org/31111216:57
jordanPyeah, I find it weird that it's not in the stdlib, it's a popular question on stackoverflow16:57
jordanPI have to go16:58
*** jordanP has quit IRC16:58
*** ddieterly is now known as ddieterly[away]16:58
*** e0ne has joined #openstack-qa16:59
*** Lord_Phoenix has quit IRC16:59
*** nwkarsten has quit IRC17:00
afazekasjordanP:Maybe we created non-sense problem that's why we do not have on obvious way to solve it ;)17:00
*** rbak has quit IRC17:02
*** lucasagomes is now known as lucas-afk17:02
*** ddieterly[away] is now known as ddieterly17:02
afazekasMaybe the chunkify should be outside to the  put_object_with_chunk ..17:04
openstackrecheckConsole logs not available after 13:25s for gate-keystone-python27-db 317529,4,0c4c3a617:06
openstackgerritSteve McLellan proposed openstack-dev/devstack: Fix sha1 download for elasticsearch>2.0  https://review.openstack.org/32015217:09
afazekasjordanP: Can we reshape the interfaces in a way where we do not need to solve this chunkify thing ?17:11
*** sambetts is now known as sambetts|afk17:11
*** jpena is now known as jpena|off17:11
*** e0ne has quit IRC17:12
*** rbak has joined #openstack-qa17:15
*** rbak has quit IRC17:15
*** rbak has joined #openstack-qa17:17
*** jlanoux has quit IRC17:17
openstackgerritFranklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint.  https://review.openstack.org/28427317:18
openstackgerritFranklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint.  https://review.openstack.org/28427317:19
*** cody-somerville has joined #openstack-qa17:20
fnavalhi - would anyone know if this is the right way to make a release note: https://review.openstack.org/#/c/284273/12/releasenotes/notes/bug-1486834-7ebca15836ae27a9.yaml17:23
*** Suyi has joined #openstack-qa17:25
*** e0ne has joined #openstack-qa17:30
openstackgerritFranklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint.  https://review.openstack.org/28427317:32
*** dtantsur is now known as dtantsur|afk17:32
*** cody-somerville has quit IRC17:32
*** rbak has quit IRC17:34
*** Ravikiran_K has quit IRC17:34
*** rbak has joined #openstack-qa17:36
*** nadya has joined #openstack-qa17:39
*** ddieterly is now known as ddieterly[away]17:39
*** yfried has joined #openstack-qa17:45
*** Swami has joined #openstack-qa17:48
*** yamamoto has quit IRC17:52
*** yamamoto has joined #openstack-qa17:53
rodrigodsoomichi, ping.. re: https://review.openstack.org/#/c/285541/17:54
*** rlrossit has quit IRC17:54
rodrigodsthat patch had a depends-on17:54
rodrigodsbut ianw -2 the devstack one and said to make in the other way around17:54
rodrigodshere https://review.openstack.org/#/c/293497/17:55
*** pvaneck has joined #openstack-qa17:58
afazekasjordanP: Rethinked , slicing/splitting the data for chunks should not be responsibility of anything in the tempest/services. In real life you expected to deal binary or string `chunked` at read time, or when you deal with it is already in the memory.  So having the chunkify to really able to deal with iterable is not a goal, but you should be able to use different method in real life. chunkify is good for testing swift..17:59
openstackgerritCastulo J. Martinez proposed openstack/tempest: Add missing tests for the image v2 API  https://review.openstack.org/31708817:59
afazekasjordanP:  The `Which part should do what`  was not good even before your change18:00
*** rlrossit has joined #openstack-qa18:01
*** cody-somerville has joined #openstack-qa18:09
*** yamamoto_ has joined #openstack-qa18:12
*** david-lyle_ has joined #openstack-qa18:12
*** Apoorva_ has joined #openstack-qa18:13
*** ajmiller__ has joined #openstack-qa18:14
*** wznoinsk_ has joined #openstack-qa18:14
*** aysyd has quit IRC18:15
*** yamamoto has quit IRC18:16
*** Apoorva has quit IRC18:16
*** ajmiller_ has quit IRC18:16
*** david-lyle has quit IRC18:16
*** wznoinsk has quit IRC18:16
*** salv-orlando has quit IRC18:21
*** ig0r_ has joined #openstack-qa18:23
*** nadya has quit IRC18:23
*** rossella_s has quit IRC18:24
*** rossella_s has joined #openstack-qa18:24
*** ddieterly[away] is now known as ddieterly18:26
openstackgerritSean M. Collins proposed openstack-dev/devstack: [WIP] Add neutron-metering agent and configuration  https://review.openstack.org/32061618:28
openstackgerritMerged openstack/tempest: Add a test case for rebuild of instances with volumes  https://review.openstack.org/17594918:29
*** rcernin has quit IRC18:29
openstackgerritMerged openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client  https://review.openstack.org/30778918:29
*** dustins has quit IRC18:33
*** dustins has joined #openstack-qa18:33
openstackgerritMatthew Treinish proposed openstack/tempest: Add whitelist and blacklist file options to tempest run  https://review.openstack.org/32015518:43
*** ntata| has quit IRC18:45
*** dimtruck is now known as zz_dimtruck18:50
mtreinishtimothyb89, david-lyle_: if you get a sec can you take a look at: https://review.openstack.org/#/c/319309/ and https://review.openstack.org/31951618:50
*** ddieterly is now known as ddieterly[away]18:51
openstackrecheckopenstack/horizon change: https://review.openstack.org/309482 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error18:51
*** e0ne has quit IRC18:53
*** cody-somerville has quit IRC18:56
*** porrua has quit IRC18:57
*** markvoelker_ has joined #openstack-qa18:57
*** markvoelker has quit IRC18:59
*** porrua has joined #openstack-qa18:59
*** markvoelker has joined #openstack-qa19:00
*** markvoelker_ has quit IRC19:02
*** cody-somerville has joined #openstack-qa19:03
*** cody-somerville has quit IRC19:04
*** cody-somerville has joined #openstack-qa19:05
*** ddieterly[away] is now known as ddieterly19:05
openstackrecheckopenstack/horizon change: https://review.openstack.org/320176 failed gate-horizon-selenium-headless in the gate queue with an unrecognized error19:06
*** tqtran has joined #openstack-qa19:14
*** zz_dimtruck is now known as dimtruck19:15
openstackgerritMatthew Treinish proposed openstack/tempest: Add whitelist and blacklist file options to tempest run  https://review.openstack.org/32015519:18
mtreinishoomichi, dims: if you get a sec can you take a look at: https://review.openstack.org/28366719:20
mtreinishthat's needed for the tempest run command work19:20
*** _vsaienko has joined #openstack-qa19:21
openstackgerritMerged openstack/os-testr: Seperate regex builder logic into a seperate module  https://review.openstack.org/31274719:22
mtreinishdims: thanks, once that merges I'm thinking it's time for a release. Are there any open patches you think we should land before I push the tag?19:23
dimsmtreinish : don't think i have anything pending :)19:24
mtreinishheh, ok. Figured I'd ask at least19:25
openstackgerritMerged openstack/os-testr: Split functionality out of main  https://review.openstack.org/28366719:25
*** mvk has joined #openstack-qa19:27
*** _vsaienko has quit IRC19:27
*** _vsaienko has joined #openstack-qa19:29
mtreinishdims: actually there is 1 open patch: https://review.openstack.org/#/c/314775/ but I haven't figured out the answer to the open question there19:29
mtreinishactually nm I don't think there is actually a difference19:30
*** cody-somerville has quit IRC19:31
*** cody-somerville has joined #openstack-qa19:31
*** cody-somerville has joined #openstack-qa19:31
*** gfidente has quit IRC19:34
openstackgerritCastulo J. Martinez proposed openstack/tempest: Add missing tests for the identity v3 API  https://review.openstack.org/31618219:34
*** anteaya has joined #openstack-qa19:34
slowriemtreinish: If I move subunit-describe-calls into tempest/tools is there anywhere the documentation can live?19:36
slowrieand the unittests19:36
mtreinishslowrie: it would have to be in cmd19:36
mtreinishfor unittests19:36
mtreinishfor docs you can put that right in docs/source19:36
slowriealright, I'm going to go through your comments on workspaces and sort those out then I'll swing back around to that and work on getting it moved over19:37
*** e0ne has joined #openstack-qa19:38
mtreinishok, there's not a rush19:38
mtreinishand sry to flip flop on that, I didn't really look at the details until I reviewed the change19:38
*** ig0r_ has quit IRC19:39
*** dimtruck is now known as zz_dimtruck19:40
slowrieIt's fine, I understand19:40
*** salv-orlando has joined #openstack-qa19:42
*** zz_dimtruck is now known as dimtruck19:49
*** _vsaienko has quit IRC19:55
*** mfisher_ora has quit IRC19:56
*** porrua has quit IRC19:58
openstackgerritMatthew Treinish proposed openstack/os-testr: Add whitelist file to ostestr docs  https://review.openstack.org/31477519:58
mtreinishandreaf, jordanP: ^^^20:01
mtreinishI updated it to say they are mutually exclusive20:01
openstackgerritMerged openstack/tempest: Add cleanup in identity/test_users_negative.py  https://review.openstack.org/31413020:03
*** Apoorva_ has quit IRC20:04
*** shardy has quit IRC20:06
*** pglass has quit IRC20:06
*** ddieterly is now known as ddieterly[away]20:06
openstackgerritSteve Heyman proposed openstack/tempest: Remove neutron OO wrappers  https://review.openstack.org/32049520:07
andreafoomichi: hi - I though calling a property internal value with the _ was normal, i.e. property is scope and internal is _scope20:09
*** sabeen has quit IRC20:09
oomichirodrigods: thanks, ianw +2 on that already. so I am waiting for jenkins job on the devstack patch to check the test result.20:10
*** _vsaienko has joined #openstack-qa20:12
openstackgerritMerged openstack/tempest: test_port_security_macspoofing_port: Don't assume DHCP port  https://review.openstack.org/25142320:13
openstackgerritMerged openstack-dev/devstack: Use path-mounted keystone when running in httpd  https://review.openstack.org/19389420:13
openstackgerritMerged openstack/openstack-health: Update the deployment docs  https://review.openstack.org/31930920:13
openstackgerritAndrea Frittoli proposed openstack/tempest: Add admin role on domain for v3  https://review.openstack.org/22629720:14
*** _vsaienko has quit IRC20:15
*** sabeen has joined #openstack-qa20:19
andreafmtreinish: https://review.openstack.org/#/c/318272/ I had to rebase it, but no real change, if you don't mind re +220:19
*** tmatsu has joined #openstack-qa20:21
*** Apoorva has joined #openstack-qa20:23
*** _vsaienko has joined #openstack-qa20:23
*** tmatsu has quit IRC20:26
*** ddieterly[away] is now known as ddieterly20:29
oomichiandreaf: yeah, OK. +A20:29
andreafoomichi: thanks!20:30
andreafjordanP: yes, the reno looks good20:37
openstackgerritStephen Lowrie proposed openstack/tempest: Centralized Workspaces  https://review.openstack.org/28377020:43
slowriemtreinish: andreaf: ^20:43
*** e0ne has quit IRC20:49
*** _vsaienko has quit IRC20:50
*** e0ne has joined #openstack-qa20:50
*** csomerville has joined #openstack-qa20:52
*** cody-somerville has quit IRC20:52
*** dimtruck is now known as zz_dimtruck20:54
*** dustins has quit IRC20:57
openstackgerritStephen Lowrie proposed openstack/tempest: Add subunit-describe-calls  https://review.openstack.org/32066120:59
slowriemtreinish: ^ is the move of subunit-describe-calls21:00
slowrieoomichi: ^21:01
*** rcernin has joined #openstack-qa21:02
*** dustins has joined #openstack-qa21:03
*** rfolco has quit IRC21:03
*** pauloewerton has quit IRC21:06
oomichislowrie: thanks for doing it. will check it after the gate passing.21:09
*** ddieterly is now known as ddieterly[away]21:09
*** _vsaienko has joined #openstack-qa21:12
*** gouthamr has quit IRC21:19
*** ddieterly[away] is now known as ddieterly21:20
*** ddieterly has quit IRC21:21
*** zz_dimtruck is now known as dimtruck21:21
*** rlrossit has quit IRC21:23
*** yamahata has quit IRC21:24
*** regXboi has quit IRC21:26
*** _vsaienko has quit IRC21:27
*** nwkarsten has joined #openstack-qa21:28
*** dimtruck is now known as zz_dimtruck21:29
*** e0ne has quit IRC21:30
*** _vs has joined #openstack-qa21:33
*** csomerville has quit IRC21:33
*** yfried has quit IRC21:34
*** pglass has joined #openstack-qa21:37
*** rcernin has quit IRC21:43
*** stevebak` is now known as stevebaker21:46
*** Serlex has quit IRC21:46
*** akshai has quit IRC21:48
*** _vs has quit IRC21:55
*** nwkarsten has quit IRC21:58
*** dustins has quit IRC22:00
*** nwkarsten has joined #openstack-qa22:00
*** _vsaienko has joined #openstack-qa22:01
*** apevec has joined #openstack-qa22:06
*** _vsaienko has quit IRC22:09
*** cody-somerville has joined #openstack-qa22:09
*** cody-somerville has quit IRC22:09
*** cody-somerville has joined #openstack-qa22:09
*** gouthamr has joined #openstack-qa22:12
*** _vs has joined #openstack-qa22:14
*** nwkarsten has quit IRC22:16
*** nwkarsten has joined #openstack-qa22:21
openstackgerritCindy Lu proposed openstack/tempest: Add more swift container tests  https://review.openstack.org/30850822:21
*** spzala has quit IRC22:22
*** _vs has quit IRC22:22
*** spzala has joined #openstack-qa22:22
*** cody-somerville has quit IRC22:23
*** rossella_s has quit IRC22:24
*** rossella_s has joined #openstack-qa22:24
*** _vs has joined #openstack-qa22:24
*** spzala has quit IRC22:27
*** nwkarsten has quit IRC22:28
*** pglass has quit IRC22:30
*** _vs has quit IRC22:30
*** cody-somerville has joined #openstack-qa22:32
*** apevec has left #openstack-qa22:32
*** apevec has quit IRC22:32
*** gouthamr_ has joined #openstack-qa22:34
*** _vs has joined #openstack-qa22:36
*** gouthamr has quit IRC22:37
*** _vs has quit IRC22:39
*** rbak has quit IRC22:43
*** spzala has joined #openstack-qa22:43
*** _vs has joined #openstack-qa22:47
*** edmondsw has quit IRC22:50
*** _vs has quit IRC22:52
*** _vs has joined #openstack-qa22:54
*** yamahata has joined #openstack-qa22:58
*** zackf has quit IRC23:00
*** _vs has quit IRC23:01
*** openstackgerrit has quit IRC23:03
*** openstackgerrit has joined #openstack-qa23:03
*** zackf has joined #openstack-qa23:05
*** _vs has joined #openstack-qa23:05
*** sdague has quit IRC23:09
*** zackf has quit IRC23:09
*** vhosakot has quit IRC23:10
*** markvoelker has quit IRC23:11
*** david-lyle_ is now known as david-lyle23:12
*** _vs has quit IRC23:16
openstackgerritSean M. Collins proposed openstack-dev/grenade: Call refactored entrypoints for Neutron startup  https://review.openstack.org/31204623:17
*** cody-somerville has quit IRC23:17
openstackgerritSean M. Collins proposed openstack-dev/grenade: Call refactored entrypoints for Neutron startup  https://review.openstack.org/31204623:17
*** _vsaienko has joined #openstack-qa23:20
openstackgerritSean M. Collins proposed openstack-dev/devstack: Neutron: Remove verbose and always set debug to true  https://review.openstack.org/31565523:23
*** _vsaienko has quit IRC23:27
*** nwkarsten has joined #openstack-qa23:29
*** Swami has quit IRC23:31
*** _vs has joined #openstack-qa23:34
*** nwkarsten has quit IRC23:34
*** tosky has quit IRC23:34
*** _vs has quit IRC23:40
*** tochi has joined #openstack-qa23:41
*** _vs has joined #openstack-qa23:45
*** yamahata has quit IRC23:48
*** _vs has quit IRC23:51
*** _vsaienko has joined #openstack-qa23:55

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