*** jhesketh_ is now known as jhesketh | 00:02 | |
*** spzala has joined #openstack-qa | 00:07 | |
*** mtanino has joined #openstack-qa | 00:08 | |
*** yangyapeng has quit IRC | 00:08 | |
*** ddieterly has joined #openstack-qa | 00:13 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/317738 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error | 00:13 |
---|---|---|
*** salv-orl_ has quit IRC | 00:13 | |
*** spzala has quit IRC | 00:14 | |
*** samueldmq has joined #openstack-qa | 00:19 | |
*** patrickeast has joined #openstack-qa | 00:20 | |
*** tongl_ has quit IRC | 00:23 | |
*** r-mibu has quit IRC | 00:25 | |
*** r-mibu has joined #openstack-qa | 00:26 | |
*** dimtruck is now known as zz_dimtruck | 00:34 | |
*** vhosakot has quit IRC | 00:35 | |
*** pcm_ has quit IRC | 00:36 | |
*** hoangcx has joined #openstack-qa | 00:42 | |
*** pc_m has joined #openstack-qa | 00:43 | |
*** zz_dimtruck is now known as dimtruck | 00:49 | |
*** rfolco has quit IRC | 00:54 | |
*** suyash has quit IRC | 00:55 | |
*** rfolco has joined #openstack-qa | 00:56 | |
*** wznoinsk has joined #openstack-qa | 00:59 | |
*** mtanino has quit IRC | 01:05 | |
*** yangyapeng has joined #openstack-qa | 01:15 | |
*** yangyapeng has quit IRC | 01:15 | |
*** yangyapeng has joined #openstack-qa | 01:16 | |
*** cody-somerville has joined #openstack-qa | 01:17 | |
*** cody-somerville has quit IRC | 01:17 | |
*** cody-somerville has joined #openstack-qa | 01:17 | |
*** anteaya has quit IRC | 01:18 | |
*** spzala has joined #openstack-qa | 01:19 | |
*** cody-somerville_ has joined #openstack-qa | 01:21 | |
*** Gen has quit IRC | 01:22 | |
*** Gen_ has joined #openstack-qa | 01:22 | |
*** cody-somerville has quit IRC | 01:23 | |
*** EinstCrazy has joined #openstack-qa | 01:24 | |
openstackrecheck | openstack/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/1282876 | 01:25 |
openstack | Launchpad bug 1298006 in OpenStack-Gate "network sometimes unreachable accessing our git server" [Medium,Triaged] | 01:25 |
openstack | Launchpad 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-qa | 01:30 | |
*** asselin has joined #openstack-qa | 01:34 | |
*** Apoorva has quit IRC | 01:34 | |
*** mtanino has quit IRC | 01:35 | |
*** cody-somerville_ has quit IRC | 01:35 | |
*** asselin has quit IRC | 01:38 | |
*** yamahata has quit IRC | 01:42 | |
*** vhosakot has joined #openstack-qa | 01:42 | |
*** cody-somerville_ has joined #openstack-qa | 01:46 | |
*** Gen has joined #openstack-qa | 01:48 | |
*** Gen_ has quit IRC | 01:51 | |
*** spzala has quit IRC | 01:52 | |
*** cody-somerville_ is now known as cody-somerville | 01:56 | |
*** cody-somerville has joined #openstack-qa | 01:57 | |
*** tqtran has quit IRC | 01:59 | |
*** EinstCrazy has quit IRC | 02:00 | |
*** EinstCrazy has joined #openstack-qa | 02:00 | |
*** vhosakot has quit IRC | 02:04 | |
*** vhosakot has joined #openstack-qa | 02:04 | |
*** fnaval_ has quit IRC | 02:08 | |
*** yamahata has joined #openstack-qa | 02:11 | |
*** ddieterly is now known as ddieterly[away] | 02:15 | |
*** artom has quit IRC | 02:16 | |
*** artom has joined #openstack-qa | 02:17 | |
*** fnaval has joined #openstack-qa | 02:17 | |
*** ddieterly[away] is now known as ddieterly | 02:20 | |
*** fnaval_ has joined #openstack-qa | 02:20 | |
*** fnaval has quit IRC | 02:21 | |
*** markvoelker has quit IRC | 02:21 | |
*** ddieterly is now known as ddieterly[away] | 02:24 | |
*** spzala has joined #openstack-qa | 02:26 | |
*** spzala has quit IRC | 02:26 | |
*** spzala has joined #openstack-qa | 02:27 | |
*** spzala has quit IRC | 02:31 | |
*** salv-orlando has joined #openstack-qa | 02:38 | |
openstackgerrit | Zhenyu Zheng proposed openstack/tempest: Modify --endpoint-type to --os-endpoint-type for nova https://review.openstack.org/318592 | 02:40 |
*** salv-orlando has quit IRC | 02:43 | |
*** tmatsu has quit IRC | 02:44 | |
*** tmatsu has joined #openstack-qa | 02:45 | |
*** Gen_ has joined #openstack-qa | 02:53 | |
*** tmatsu has quit IRC | 02:54 | |
*** tqtran has joined #openstack-qa | 02:56 | |
*** Gen has quit IRC | 02:56 | |
*** tqtran has quit IRC | 03:00 | |
*** spzala has joined #openstack-qa | 03:02 | |
*** spzala has quit IRC | 03:08 | |
*** erlon_ has joined #openstack-qa | 03:08 | |
*** yamamot__ has joined #openstack-qa | 03:09 | |
*** rfolco has quit IRC | 03:10 | |
*** Gen_ has quit IRC | 03:12 | |
*** cody-somerville has quit IRC | 03:16 | |
*** gouthamr_ has quit IRC | 03:17 | |
*** tmatsu has joined #openstack-qa | 03:17 | |
*** vhosakot has quit IRC | 03:20 | |
*** fnaval has joined #openstack-qa | 03:21 | |
*** markvoelker has joined #openstack-qa | 03:22 | |
*** ddieterly[away] has quit IRC | 03:24 | |
*** fnaval_ has quit IRC | 03:24 | |
*** tmatsu has quit IRC | 03:25 | |
*** markvoelker has quit IRC | 03:27 | |
*** tqtran has joined #openstack-qa | 03:32 | |
*** tqtran has quit IRC | 03:33 | |
*** dave-mcc_ has quit IRC | 03:38 | |
*** _alastor__ has quit IRC | 03:38 | |
*** tochi has quit IRC | 03:40 | |
*** dimtruck is now known as zz_dimtruck | 03:52 | |
*** yamahata has quit IRC | 03:53 | |
*** _alastor__ has joined #openstack-qa | 03:55 | |
*** david-lyle_ has joined #openstack-qa | 03:55 | |
*** david-lyle has quit IRC | 03:57 | |
*** ramishra has quit IRC | 03:58 | |
*** tochi has joined #openstack-qa | 04:00 | |
*** ramishra has joined #openstack-qa | 04:00 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/317738 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error | 04:03 |
*** spzala has joined #openstack-qa | 04:04 | |
*** fnaval_ has joined #openstack-qa | 04:07 | |
*** salv-orlando has joined #openstack-qa | 04:08 | |
*** fnaval has quit IRC | 04:10 | |
*** spzala has quit IRC | 04:10 | |
*** tmatsu has joined #openstack-qa | 04:13 | |
*** salv-orlando has quit IRC | 04:17 | |
*** tmatsu has quit IRC | 04:19 | |
*** tmatsu has joined #openstack-qa | 04:20 | |
*** yamamot__ has quit IRC | 04:21 | |
*** markvoelker has joined #openstack-qa | 04:23 | |
*** markvoelker has quit IRC | 04:28 | |
*** Gen has joined #openstack-qa | 04:32 | |
*** Gen_ has joined #openstack-qa | 04:36 | |
*** Gen has quit IRC | 04:36 | |
*** Poornima has joined #openstack-qa | 04:36 | |
openstackrecheck | Console logs not available after 13:26s for keystone-coverage-db 320156,9,82a9468 | 04:37 |
*** armax has quit IRC | 04:41 | |
*** sridharg has joined #openstack-qa | 04:43 | |
*** asselin has joined #openstack-qa | 04:46 | |
*** shausy has joined #openstack-qa | 04:48 | |
*** pgadiya has joined #openstack-qa | 04:53 | |
*** cody-somerville has joined #openstack-qa | 05:01 | |
*** spzala has joined #openstack-qa | 05:02 | |
*** bkopilov has joined #openstack-qa | 05:02 | |
*** yamamoto has joined #openstack-qa | 05:04 | |
*** spzala has quit IRC | 05:06 | |
*** sabeen1 has quit IRC | 05:06 | |
*** bkopilov has quit IRC | 05:10 | |
*** bkopilov has joined #openstack-qa | 05:10 | |
*** gamado_wfh has joined #openstack-qa | 05:10 | |
*** Gen_ has quit IRC | 05:10 | |
*** bkopilov has quit IRC | 05:11 | |
*** bkopilov has joined #openstack-qa | 05:11 | |
*** dmellado has quit IRC | 05:13 | |
*** yamamoto has quit IRC | 05:14 | |
*** yamamoto has joined #openstack-qa | 05:16 | |
*** dansmith has quit IRC | 05:16 | |
*** dmellado has joined #openstack-qa | 05:17 | |
*** shausy has quit IRC | 05:17 | |
*** shausy has joined #openstack-qa | 05:18 | |
*** dansmith has joined #openstack-qa | 05:18 | |
*** dansmith is now known as Guest71132 | 05:18 | |
openstackgerrit | Masayuki Igawa proposed openstack/openstack-health: Add feedback and feedback-fix css class https://review.openstack.org/319131 | 05:19 |
*** markvoelker has joined #openstack-qa | 05:24 | |
*** markvoelker has quit IRC | 05:29 | |
*** yfried has joined #openstack-qa | 05:29 | |
*** tmatsu has quit IRC | 05:31 | |
*** tmatsu has joined #openstack-qa | 05:32 | |
*** binbincong has quit IRC | 05:32 | |
*** cody-somerville has quit IRC | 05:36 | |
*** binbincong has joined #openstack-qa | 05:51 | |
*** erlon_ has quit IRC | 05:56 | |
*** amotoki has joined #openstack-qa | 05:56 | |
*** Gen has joined #openstack-qa | 06:00 | |
*** shausy has quit IRC | 06:00 | |
*** fnaval has joined #openstack-qa | 06:04 | |
*** Gen has quit IRC | 06:05 | |
*** fnaval_ has quit IRC | 06:06 | |
*** rcernin has joined #openstack-qa | 06:08 | |
*** nadya has joined #openstack-qa | 06:08 | |
*** Lord_Phoenix has joined #openstack-qa | 06:11 | |
*** sridharg has quit IRC | 06:13 | |
*** scottda has quit IRC | 06:14 | |
*** DuncanT has quit IRC | 06:15 | |
*** scottda has joined #openstack-qa | 06:16 | |
*** betherly has quit IRC | 06:16 | |
*** larainema has quit IRC | 06:17 | |
*** lekha-away has quit IRC | 06:18 | |
*** DuncanT has joined #openstack-qa | 06:18 | |
*** betherly has joined #openstack-qa | 06:19 | |
*** lekha-away has joined #openstack-qa | 06:20 | |
*** larainema has joined #openstack-qa | 06:20 | |
*** isq has quit IRC | 06:20 | |
*** isq has joined #openstack-qa | 06:21 | |
*** woodster_ has quit IRC | 06:22 | |
*** markvoelker has joined #openstack-qa | 06:25 | |
*** _vsaienko has joined #openstack-qa | 06:25 | |
*** nadya has quit IRC | 06:25 | |
*** woodster_ has joined #openstack-qa | 06:26 | |
*** samueldmq has quit IRC | 06:27 | |
*** samueldmq has joined #openstack-qa | 06:27 | |
*** nineseconds has joined #openstack-qa | 06:28 | |
*** markvoelker has quit IRC | 06:29 | |
*** grafuls has quit IRC | 06:32 | |
*** _vsaienko has quit IRC | 06:32 | |
*** grafuls has joined #openstack-qa | 06:33 | |
*** Serlex has joined #openstack-qa | 06:37 | |
*** salv-orlando has joined #openstack-qa | 06:37 | |
*** itzikb has joined #openstack-qa | 06:39 | |
*** sridharg has joined #openstack-qa | 06:46 | |
*** Lord_Phoenix has quit IRC | 06:51 | |
*** Deng has joined #openstack-qa | 07:02 | |
*** spzala has joined #openstack-qa | 07:02 | |
*** gamado_wfh has quit IRC | 07:03 | |
*** Lord_Phoenix has joined #openstack-qa | 07:03 | |
*** tesseract has joined #openstack-qa | 07:04 | |
*** spzala has quit IRC | 07:07 | |
*** tesseract has quit IRC | 07:08 | |
*** yamahata has joined #openstack-qa | 07:10 | |
*** tmatsu has quit IRC | 07:14 | |
*** tmatsu has joined #openstack-qa | 07:15 | |
*** rossella_s has joined #openstack-qa | 07:16 | |
*** rwsu has joined #openstack-qa | 07:19 | |
*** tmatsu has quit IRC | 07:20 | |
*** tmatsu has joined #openstack-qa | 07:22 | |
*** rwsu has quit IRC | 07:22 | |
*** yfried has quit IRC | 07:22 | |
*** tmatsu_ has joined #openstack-qa | 07:24 | |
*** Ravikiran_K has joined #openstack-qa | 07:25 | |
*** markvoelker has joined #openstack-qa | 07:26 | |
*** tmatsu has quit IRC | 07:28 | |
*** jlanoux has joined #openstack-qa | 07:29 | |
*** markvoelker has quit IRC | 07:32 | |
*** rossella_s has quit IRC | 07:32 | |
*** rossella_s has joined #openstack-qa | 07:33 | |
*** Lord_Phoenix has quit IRC | 07:34 | |
*** ihrachys has joined #openstack-qa | 07:37 | |
*** Lord_Phoenix has joined #openstack-qa | 07:37 | |
*** yfried has joined #openstack-qa | 07:40 | |
*** zeih has joined #openstack-qa | 07:41 | |
*** anshul has joined #openstack-qa | 07:44 | |
*** anshul is now known as Guest82686 | 07:44 | |
*** jpena|off is now known as jpena | 07:47 | |
*** woodster_ has quit IRC | 07:48 | |
*** jordanP has joined #openstack-qa | 07:51 | |
*** dtantsur|afk is now known as dtantsur | 07:52 | |
*** zeih has quit IRC | 07:54 | |
afazekas_ | andreaf, done | 07:55 |
*** zeih has joined #openstack-qa | 07:56 | |
*** beisner has quit IRC | 07:56 | |
*** Lord_Phoenix has quit IRC | 07:58 | |
*** Lord_Phoenix has joined #openstack-qa | 07:59 | |
*** spzala has joined #openstack-qa | 08:04 | |
*** beisner has joined #openstack-qa | 08:07 | |
*** spzala has quit IRC | 08:08 | |
*** zeih has quit IRC | 08:10 | |
*** drptbl has joined #openstack-qa | 08:10 | |
*** shardy has joined #openstack-qa | 08:12 | |
*** lucasagomes_ is now known as lucasagomes | 08:12 | |
*** apevec has joined #openstack-qa | 08:19 | |
*** e0ne has joined #openstack-qa | 08:20 | |
*** tosky has joined #openstack-qa | 08:21 | |
*** zeih has joined #openstack-qa | 08:22 | |
*** venkat has joined #openstack-qa | 08:23 | |
*** tosky has quit IRC | 08:23 | |
*** salv-orlando has quit IRC | 08:25 | |
*** tosky has joined #openstack-qa | 08:26 | |
*** afazekas_ is now known as afazekas | 08:27 | |
*** markvoelker has joined #openstack-qa | 08:28 | |
*** drptbl has quit IRC | 08:30 | |
*** e0ne has quit IRC | 08:31 | |
*** gamado has joined #openstack-qa | 08:31 | |
*** hichihara has joined #openstack-qa | 08:32 | |
*** e0ne has joined #openstack-qa | 08:32 | |
*** markvoelker has quit IRC | 08:32 | |
*** EinstCrazy has quit IRC | 08:37 | |
*** zeih has quit IRC | 08:38 | |
andreaf | afazekas: cool, thanks | 08:39 |
*** EinstCrazy has joined #openstack-qa | 08:40 | |
*** ig0r_ has joined #openstack-qa | 08:40 | |
openstackgerrit | Benny Kopilov proposed openstack/tempest: Cinder multibackend migration tests https://review.openstack.org/318569 | 08:49 |
*** EinstCrazy has quit IRC | 08:51 | |
*** gszasz has joined #openstack-qa | 08:52 | |
*** cdent has joined #openstack-qa | 08:55 | |
*** EinstCrazy has joined #openstack-qa | 08:57 | |
*** wznoinsk has quit IRC | 08:59 | |
*** wznoinsk has joined #openstack-qa | 08:59 | |
andreaf | afazekas: I updated https://review.openstack.org/#/c/319394/4 as well | 09:03 |
andreaf | ylobankov: a question regarding https://review.openstack.org/#/c/307789/9 | 09:06 |
openstackgerrit | zhufl proposed openstack/tempest: Not to create security group when security_group ext is disabled https://review.openstack.org/320309 | 09:07 |
openstackgerrit | Merged openstack/tempest: Remove deprecated legacy credentials provider https://review.openstack.org/263376 | 09:08 |
*** gfidente has joined #openstack-qa | 09:09 | |
ylobankov | andreaf: what question? | 09:10 |
andreaf | ylobankov: 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 itself | 09:12 |
andreaf | ylobankov: so is there a bug in the docs or in the glance API? | 09:12 |
andreaf | ylobankov: 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 status | 09:13 |
*** tochi has quit IRC | 09:15 | |
ylobankov | andreaf: we can get status from HEAD, but something like this: image['x-meta-umage-status'] (not just image['status']) as far as I remember | 09:18 |
andreaf | ylobankov: you mean from GET? | 09:20 |
andreaf | in case of GET the image status will be in a header as the body will be the image itself I imaging | 09:20 |
andreaf | imagine | 09:20 |
ylobankov | yes, from GET. Sorry | 09:20 |
*** cdent has left #openstack-qa | 09:21 | |
andreaf | ylobankov: 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 metadata | 09:24 |
andreaf | ylobankov: it's not because it's the only way to get image status, but because it's the most convenient | 09:24 |
andreaf | ylobankov: this patch is the important one: https://review.openstack.org/#/c/297511/10 | 09:26 |
*** nadya has joined #openstack-qa | 09:27 | |
*** yfried has quit IRC | 09:27 | |
andreaf | ylobankov: 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 one | 09:28 |
*** yfried has joined #openstack-qa | 09:28 | |
andreaf | ylobankov: 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 one | 09:28 |
*** zigo_ is now known as zigo | 09:28 | |
andreaf | ylobankov: well that's my view at least :) | 09:29 |
ylobankov | andreaf: 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 |
andreaf | sure, but wait_for_image_status can be changed :) | 09:31 |
*** jordanP has quit IRC | 09:32 | |
ylobankov | andreaf: 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-qa | 09:34 | |
*** gmann_ has joined #openstack-qa | 09:34 | |
andreaf | ylobankov: that's an option | 09:35 |
ylobankov | andreaf: 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 IRC | 09:37 | |
*** tmatsu_ has quit IRC | 09:37 | |
andreaf | ylobankov: apart from the object-storage client, with list_account_metadata, all other usages of HEAD have a check_ method | 09:38 |
*** Poornima has joined #openstack-qa | 09:38 | |
andreaf | ylobankov: but it's difficult to get a consistent naming because not all APIs use the methods for the same purpose | 09:38 |
andreaf | ylobankov: so if we go for a fixed method -> name mapping, the name will not always be consistent with what the method actually do | 09:39 |
andreaf | ylobankov: but I think it's still the less confusing option - what the method does can go in docstring | 09:39 |
andreaf | ylobankov: 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 reviews | 09:40 |
ylobankov | andreaf: what is better for you check_image_meta or show_image_meta ? ok, let's discuss it on Thursday with folks | 09:42 |
openstackgerrit | Wenzhi Yu proposed openstack/tempest: WIP: Add Magnum clients to tempest https://review.openstack.org/320330 | 09:43 |
*** jordanP has joined #openstack-qa | 09:45 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Hash credentials on user, project/tenant and pwd https://review.openstack.org/318272 | 09:52 |
*** Poornima has quit IRC | 09:52 | |
*** mvk has quit IRC | 09:53 | |
*** fnaval has quit IRC | 09:54 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Account generator for identity v3 https://review.openstack.org/317105 | 09:55 |
*** sambetts|afk is now known as sambetts | 09:55 | |
openstackgerrit | Lee Yarwood proposed openstack/tempest: Add stable disk device instance rescue tests https://review.openstack.org/319863 | 09:56 |
andreaf | mtreinish: 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 conflict | 09:59 |
openstackrecheck | openstack/horizon change: https://review.openstack.org/316763 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error | 09:59 |
*** hoangcx_ has joined #openstack-qa | 10:02 | |
*** hoangcx has quit IRC | 10:03 | |
*** Poornima has joined #openstack-qa | 10:04 | |
*** gszasz has quit IRC | 10:05 | |
*** spzala has joined #openstack-qa | 10:05 | |
*** apevec has quit IRC | 10:07 | |
*** spzala has quit IRC | 10:10 | |
*** yamahata has quit IRC | 10:13 | |
*** e0ne has quit IRC | 10:13 | |
*** e0ne has joined #openstack-qa | 10:14 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client https://review.openstack.org/307789 | 10:14 |
*** tmatsu has quit IRC | 10:14 | |
*** zeih has joined #openstack-qa | 10:17 | |
*** Lord_Phoenix has quit IRC | 10:21 | |
*** zeih has quit IRC | 10:21 | |
*** mvk has joined #openstack-qa | 10:22 | |
*** early has quit IRC | 10:23 | |
*** early has joined #openstack-qa | 10:23 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client https://review.openstack.org/307789 | 10:28 |
*** nadya has quit IRC | 10:28 | |
*** markvoelker has joined #openstack-qa | 10:29 | |
*** jlanoux has quit IRC | 10:30 | |
*** yfried has quit IRC | 10:31 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client https://review.openstack.org/307789 | 10:31 |
*** jlanoux has joined #openstack-qa | 10:31 | |
*** asselin has quit IRC | 10:31 | |
ylobankov | andreaf, afazekas: what do you think about it? https://review.openstack.org/#/c/307789/ | 10:31 |
*** EinstCrazy has quit IRC | 10:32 | |
*** EinstCrazy has joined #openstack-qa | 10:32 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client https://review.openstack.org/307789 | 10:32 |
*** yangyapeng has quit IRC | 10:34 | |
*** markvoelker has quit IRC | 10:34 | |
*** EinstCrazy has quit IRC | 10:37 | |
*** e0ne_ has joined #openstack-qa | 10:41 | |
*** hoangcx_ has quit IRC | 10:41 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/316766 failed gate-horizon-python27, gate-horizon-dsvm-integration in the gate queue with an unrecognized error | 10:41 |
*** e0ne has quit IRC | 10:42 | |
*** jlanoux has quit IRC | 10:42 | |
*** jlanoux has joined #openstack-qa | 10:43 | |
*** binbincong has quit IRC | 10:44 | |
*** gszasz has joined #openstack-qa | 10:58 | |
andreaf | ylobankov: 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 client | 10:59 |
ylobankov | andreaf: yep | 11:00 |
andreaf | cool | 11:02 |
*** gszasz has quit IRC | 11:04 | |
andreaf | mkoderer, jordanP: this should be pretty straight-forward: https://review.openstack.org/#/c/319394/ | 11:05 |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client https://review.openstack.org/307789 | 11:06 |
*** spzala has joined #openstack-qa | 11:06 | |
jordanP | andreaf, | 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 IRC | 11:08 | |
jordanP | arf no nothing I got it now | 11:08 |
jordanP | sorry | 11:08 |
andreaf | jordanP: yep, restoring the original password | 11:09 |
ylobankov | andreaf: I have updated the commit message and your +2 has gone. Could you please voite one more time? thanks in advance. | 11:09 |
andreaf | ylobankov: done | 11:09 |
ylobankov | andreaf: cool | 11:10 |
*** spzala has quit IRC | 11:10 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/318357 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error | 11:11 |
*** salv-orlando has joined #openstack-qa | 11:15 | |
*** venkat has quit IRC | 11:15 | |
*** lucasagomes is now known as lucas-hungry | 11:18 | |
*** salv-orlando has quit IRC | 11:19 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere https://review.openstack.org/297511 | 11:21 |
*** nadya has joined #openstack-qa | 11:25 | |
openstackgerrit | Dale Smith proposed openstack/tempest: Add config option identity/member_role https://review.openstack.org/302332 | 11:25 |
*** sdague has joined #openstack-qa | 11:25 | |
*** jpena is now known as jpena|lunch | 11:27 | |
*** gmann_ has quit IRC | 11:30 | |
*** markvoelker has joined #openstack-qa | 11:30 | |
*** rodrigods has quit IRC | 11:31 | |
*** rodrigods has joined #openstack-qa | 11:31 | |
*** nineseconds has quit IRC | 11:32 | |
*** gamado has quit IRC | 11:32 | |
*** vgridnev has quit IRC | 11:33 | |
*** ig0r_ has quit IRC | 11:34 | |
*** markvoelker has quit IRC | 11:35 | |
*** vgridnev has joined #openstack-qa | 11:35 | |
*** vgridnev has quit IRC | 11:40 | |
*** vgridnev has joined #openstack-qa | 11:40 | |
*** ddieterly has joined #openstack-qa | 11:41 | |
*** hichihara has quit IRC | 11:43 | |
dalees | andreaf: sorry, i was slow posting my reply :) | 11:43 |
dalees | andreaf: and thanks for the feedback. happy to discuss here also if that's easier | 11:44 |
*** yfried has joined #openstack-qa | 11:45 | |
andreaf | dalees: just reading your comment | 11:45 |
*** salv-orlando has joined #openstack-qa | 11:46 | |
andreaf | dalees: 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 work | 11:47 |
openstackgerrit | Lee Yarwood proposed openstack/tempest: Add stable disk device instance rescue tests https://review.openstack.org/319863 | 11:47 |
andreaf | dalees: 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 so | 11:48 |
*** Lord_Phoenix has joined #openstack-qa | 11:48 | |
andreaf | dalees: so what I was proposing if for both credential providers (dynamic and pre-prov) to honour the same config option | 11:48 |
dalees | andreaf, 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 |
andreaf | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py#n72 | 11:49 |
andreaf | right, 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 cases | 11:50 |
dalees | andreaf, 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 also | 11:51 |
dalees | ahh | 11:51 |
dalees | interesting, 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 |
andreaf | right | 11:53 |
*** askb has joined #openstack-qa | 11:53 | |
andreaf | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/common/dynamic_creds.py#n138 | 11:53 |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere https://review.openstack.org/297511 | 11:54 |
andreaf | well actually the list of roles is assigned to all users, including admin | 11:54 |
dalees | it 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-qa | 11:54 | |
andreaf | yep | 11:55 |
andreaf | yes the help text would need updating | 11:55 |
*** ddieterly is now known as ddieterly[away] | 11:55 | |
andreaf | dalees: 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-qa | 11:56 | |
openstackrecheck | Console logs not available after 13:29s for gate-tempest-dsvm-neutron-full 318334,3,ec1b592 | 11:57 |
*** nineseconds has joined #openstack-qa | 11:58 | |
dalees | andreaf: yes, as a user i understand! | 11:58 |
dalees | andreaf, I'll make the change to use tempest_roles. I think that will work fine for this; thanks for the feedback | 12:00 |
andreaf | dalees: thank you | 12:01 |
*** markvoelker has joined #openstack-qa | 12:02 | |
*** yamamoto has quit IRC | 12:04 | |
mkoderer | jordanP: andreaf change LGTM - any comment or should I +A? | 12:04 |
*** itzkb_ has joined #openstack-qa | 12:05 | |
*** itzikb has quit IRC | 12:06 | |
*** spzala has joined #openstack-qa | 12:07 | |
*** lucas-hungry is now known as lucasagomes | 12:10 | |
*** spzala has quit IRC | 12:11 | |
*** Ravikiran_K has joined #openstack-qa | 12:16 | |
*** ddieterly[away] is now known as ddieterly | 12:18 | |
andreaf | mkoderer: no comment from my side :) | 12:21 |
andreaf | mkoderer: I think it should be fine to go | 12:21 |
*** Lord_Phoenix has quit IRC | 12:25 | |
*** tmatsu has joined #openstack-qa | 12:26 | |
*** tmatsu has quit IRC | 12:31 | |
*** beisner has quit IRC | 12:32 | |
*** salv-orlando has quit IRC | 12:35 | |
openstackgerrit | Rodrigo Duarte proposed openstack-dev/devstack: Tempest: add a Keystone reseller feature flag https://review.openstack.org/293497 | 12:36 |
*** itzkb_ has quit IRC | 12:39 | |
*** itzikb has joined #openstack-qa | 12:39 | |
*** aysyd has joined #openstack-qa | 12:42 | |
openstackgerrit | Kashyap Chamarthy proposed openstack-dev/devstack: functions-common: Use `systemctl` when applicable https://review.openstack.org/319209 | 12:43 |
openstackgerrit | Kashyap Chamarthy proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon https://review.openstack.org/319819 | 12:46 |
*** jpena|lunch is now known as jpena | 12:47 | |
*** asselin has joined #openstack-qa | 12:50 | |
openstackgerrit | Lee Yarwood proposed openstack-dev/devstack: Enable stable disk device rescue tests for libvirt https://review.openstack.org/320428 | 12:51 |
*** gouthamr has joined #openstack-qa | 12:51 | |
*** itzikb has quit IRC | 12:53 | |
*** itzikb has joined #openstack-qa | 12:53 | |
*** akshai has joined #openstack-qa | 12:53 | |
*** beisner has joined #openstack-qa | 12:54 | |
*** akshai_ has joined #openstack-qa | 12:54 | |
*** Lord_Phoenix has joined #openstack-qa | 12:54 | |
*** vponomaryov has quit IRC | 12:55 | |
*** jordanP has quit IRC | 12:55 | |
*** jordanP has joined #openstack-qa | 12:57 | |
*** vponomaryov has joined #openstack-qa | 12:57 | |
*** Deng has quit IRC | 12:57 | |
*** pauloewerton has joined #openstack-qa | 12:57 | |
*** david-lyle has joined #openstack-qa | 12:57 | |
*** akshai has quit IRC | 12:58 | |
*** lucasagomes is now known as lucas-afk | 13:01 | |
*** david-lyle_ has quit IRC | 13:01 | |
*** ddieterly has quit IRC | 13:02 | |
*** Lord_Phoenix has quit IRC | 13:04 | |
*** edmondsw has joined #openstack-qa | 13:05 | |
*** Lord_Phoenix has joined #openstack-qa | 13:06 | |
*** rlrossit has joined #openstack-qa | 13:07 | |
*** alyson_ has joined #openstack-qa | 13:07 | |
*** spzala has joined #openstack-qa | 13:07 | |
*** Lord_Phoenix has quit IRC | 13:08 | |
*** Lord_Phoenix has joined #openstack-qa | 13:09 | |
*** elynn has joined #openstack-qa | 13:10 | |
*** salv-orlando has joined #openstack-qa | 13:11 | |
openstackgerrit | Merged openstack/tempest: Removed q-vpn from the list of dirty logs https://review.openstack.org/311243 | 13:11 |
openstackgerrit | Merged openstack/tempest: Modify projects_client to receive more attributes https://review.openstack.org/315253 | 13:12 |
*** spzala has quit IRC | 13:12 | |
kashyap | Variables 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-qa | 13:16 | |
openstackgerrit | Kashyap Chamarthy proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon https://review.openstack.org/319819 | 13:16 |
*** porrua has joined #openstack-qa | 13:17 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/309482 failed gate-horizon-python27 in the gate queue with an unrecognized error | 13:18 |
*** pgadiya has quit IRC | 13:21 | |
*** hichihara has joined #openstack-qa | 13:21 | |
*** _vsaienko has joined #openstack-qa | 13:22 | |
*** pgadiya has joined #openstack-qa | 13:22 | |
*** dave-mccowan has joined #openstack-qa | 13:24 | |
*** nineseconds has quit IRC | 13:24 | |
*** nineseconds has joined #openstack-qa | 13:24 | |
*** dustins has joined #openstack-qa | 13:24 | |
*** _vsaienko has quit IRC | 13:25 | |
*** zz_dimtruck is now known as dimtruck | 13:27 | |
*** nineseconds has quit IRC | 13:28 | |
*** nineseconds has joined #openstack-qa | 13:28 | |
openstackgerrit | Kashyap Chamarthy proposed openstack-dev/devstack: functions-common: Use `systemctl` when applicable https://review.openstack.org/319209 | 13:28 |
*** dustins has quit IRC | 13:29 | |
*** ddieterly has joined #openstack-qa | 13:29 | |
*** dustins has joined #openstack-qa | 13:29 | |
*** regXboi has joined #openstack-qa | 13:29 | |
openstackgerrit | Kashyap Chamarthy proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon https://review.openstack.org/319819 | 13:31 |
openstackgerrit | Rodrigo Duarte proposed openstack/tempest: Add parent_id to create_project https://review.openstack.org/285541 | 13:33 |
*** yfried has quit IRC | 13:34 | |
*** Poornima has quit IRC | 13:36 | |
*** pgadiya has quit IRC | 13:36 | |
*** fesp has joined #openstack-qa | 13:36 | |
*** nineseconds has quit IRC | 13:38 | |
*** nineseconds has joined #openstack-qa | 13:38 | |
openstackgerrit | Merged openstack/tempest: Fix update_password tests https://review.openstack.org/319394 | 13:39 |
*** fesp has quit IRC | 13:40 | |
*** ddieterly is now known as ddieterly[away] | 13:41 | |
kashyap | Anyone 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.gz | 13:46 |
*** _vsaienko has joined #openstack-qa | 13:46 | |
*** _vsaienko has quit IRC | 13:47 | |
*** ddieterly[away] is now known as ddieterly | 13:48 | |
*** nineseconds has quit IRC | 13:48 | |
*** nineseconds has joined #openstack-qa | 13:48 | |
kashyap | afazekas: ^ When you get a sec, maybe you have a comment whether if I screwed up something... | 13:48 |
kashyap | afazekas: Duh, ignore me; missing space after 'True'... | 13:50 |
*** nwkarsten has joined #openstack-qa | 13:51 | |
*** yamahata has joined #openstack-qa | 13:52 | |
*** jbernard_ is now known as jbernard | 13:52 | |
*** _vsaienko has joined #openstack-qa | 13:53 | |
*** anteaya has joined #openstack-qa | 13:54 | |
*** fnaval has joined #openstack-qa | 13:54 | |
dmellado | kashyap: 'spacing' out? :P | 13:55 |
kashyap | dmellado: Yeah, awful bash mistake :-) | 13:56 |
*** rbak has joined #openstack-qa | 13:58 | |
*** hichihar_ has joined #openstack-qa | 14:03 | |
*** sabeen has joined #openstack-qa | 14:03 | |
*** yamamoto has joined #openstack-qa | 14:05 | |
afazekas | kashyap, http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%20%5C%22generate-subunit%3A%20command%20not%20found%5C%22 | 14:08 |
*** dustins has quit IRC | 14:09 | |
*** dustins has joined #openstack-qa | 14:09 | |
*** fnaval has quit IRC | 14:09 | |
*** woodster_ has joined #openstack-qa | 14:09 | |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Introduce scope in the auth API https://review.openstack.org/313171 | 14:11 |
*** Guest71132 is now known as dansmith | 14:11 | |
*** pglass has joined #openstack-qa | 14:11 | |
*** dansmith is now known as Guest21621 | 14:12 | |
*** lucas-afk is now known as lucasagomes | 14:12 | |
kashyap | afazekas: Thanks | 14:12 |
afazekas | kashyap, https://bugs.launchpad.net/devstack/+bug/1547379 | 14:13 |
openstack | Launchpad bug 1547379 in devstack "./stack.sh: line 488: generate-subunit: command not found" [Undecided,Invalid] | 14:13 |
kashyap | Ah-ha | 14:13 |
*** hichihara has quit IRC | 14:13 | |
*** Guest21621 is now known as dansmith | 14:14 | |
*** spzala has joined #openstack-qa | 14:14 | |
*** cody-somerville has joined #openstack-qa | 14:14 | |
*** cody-somerville has quit IRC | 14:14 | |
*** cody-somerville has joined #openstack-qa | 14:14 | |
*** spzala has quit IRC | 14:14 | |
mtreinish | afazekas: that error is a false positive, it's happening because generate-subunit is called in the exit hook | 14:15 |
mtreinish | if 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 cause | 14:16 |
mtreinish | kashyap: ^^^ | 14:16 |
*** mfisher_ora has joined #openstack-qa | 14:16 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add tempest run command https://review.openstack.org/284411 | 14:17 |
*** ajmiller_ has joined #openstack-qa | 14:18 | |
*** spzala has joined #openstack-qa | 14:18 | |
*** sridharg has quit IRC | 14:18 | |
afazekas | mtreinish: I just read the trace, BTW where is the detailed info ? | 14:18 |
afazekas | mtreinish: Do not have the debug info from the $TOP_DIR/tools/install_pip.sh | 14:20 |
*** rossella_s has quit IRC | 14:24 | |
*** rossella_s has joined #openstack-qa | 14:24 | |
afazekas | kashyap, 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 |
kashyap | Sorry, what is `inc thing`? | 14:27 |
kashyap | mtreinish: Thanks for correcting | 14:27 |
mtreinish | afazekas: I was just looking at the devstacklogs from the logstash link | 14:27 |
mtreinish | afazekas: I don't know why install_pip.sh failed | 14:28 |
*** ajmiller has quit IRC | 14:31 | |
jordanP | why 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 |
afazekas | mtreinish, non of the logstash failures are from a gate job | 14:31 |
*** sridharg has joined #openstack-qa | 14:32 | |
*** pilgrimstack1 has joined #openstack-qa | 14:32 | |
jordanP | andreaf, 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 weight | 14:32 |
jordanP | thanks | 14:33 |
*** ddieterly is now known as ddieterly[away] | 14:33 | |
*** _vsaienko has quit IRC | 14:34 | |
mtreinish | jordanP: 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-qa | 14:34 | |
mtreinish | because just saying it's a generator object is a bit weird | 14:34 |
mtreinish | but that can be a follow up change | 14:34 |
*** e0ne has joined #openstack-qa | 14:34 | |
*** e0ne_ has quit IRC | 14:35 | |
*** askb has quit IRC | 14:37 | |
jordanP | mtreinish, 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 though | 14:40 |
openstackgerrit | Steve Heyman proposed openstack/tempest: Remove neutron OO wrappers https://review.openstack.org/320495 | 14:41 |
*** cody-somerville has quit IRC | 14:42 | |
*** anteaya has quit IRC | 14:42 | |
*** david-lyle has quit IRC | 14:42 | |
*** itzikb has quit IRC | 14:42 | |
*** beisner has quit IRC | 14:42 | |
*** aysyd has quit IRC | 14:42 | |
*** mvk has quit IRC | 14:42 | |
*** wznoinsk has quit IRC | 14:42 | |
*** vponomaryov has quit IRC | 14:42 | |
*** pilgrimstack has quit IRC | 14:42 | |
*** akshai_ has quit IRC | 14:42 | |
*** ajmiller_ has quit IRC | 14:43 | |
*** StevenK_ is now known as StevenK | 14:44 | |
*** _vsaienko has quit IRC | 14:45 | |
*** regXboi has quit IRC | 14:46 | |
*** regXboi has joined #openstack-qa | 14:46 | |
afazekas | jordanP, chunkify is it really working with any iterable ? | 14:47 |
*** grafuls has quit IRC | 14:47 | |
*** rcernin has quit IRC | 14:47 | |
jordanP | afazekas, no :) | 14:47 |
afazekas | -1 | 14:48 |
afazekas | are you planning to fix it ? | 14:48 |
jordanP | afazekas, no | 14:48 |
afazekas | are you planning to use less confusing variable name ? | 14:49 |
jordanP | I am not coding a ferari when all I need is a toyota | 14:49 |
*** _vs has joined #openstack-qa | 14:49 | |
jordanP | not all iterables support slices | 14:49 |
jordanP | so I am not sure what do you want me to change ? | 14:49 |
*** hichihar_ has quit IRC | 14:49 | |
afazekas | sliceable | 14:49 |
jordanP | is that a thing ? | 14:50 |
afazekas | from today it is :) | 14:50 |
jordanP | lol, Okay :) I like the idea | 14:50 |
*** akshai has joined #openstack-qa | 14:50 | |
* afazekas I try to figure out some good iterable way quickly | 14:51 | |
*** yangyapeng has joined #openstack-qa | 14:52 | |
*** ddieterly[away] is now known as ddieterly | 14:55 | |
*** weshay has joined #openstack-qa | 14:55 | |
*** ddieterly is now known as ddieterly[away] | 14:57 | |
openstackgerrit | Benny Kopilov proposed openstack/tempest: cinder service enable / disable tests https://review.openstack.org/319634 | 14:58 |
*** _vs has quit IRC | 14:58 | |
*** slowrie_ is now known as slowrie | 14:59 | |
mtreinish | andreaf: good call on needing release notes on 311112. We need to get better about that :) | 14:59 |
*** rcernin has joined #openstack-qa | 14:59 | |
*** nineseconds has quit IRC | 15:00 | |
*** _vsaienko has joined #openstack-qa | 15:02 | |
*** e0ne has quit IRC | 15:06 | |
*** dustins has quit IRC | 15:06 | |
afazekas | jordanP, What about the solution with 194 votes ? http://stackoverflow.com/questions/312443/how-do-you-split-a-list-into-evenly-sized-chunks-in-python/312464 | 15:10 |
afazekas | izip_longest | 15:10 |
*** yamamoto has quit IRC | 15:11 | |
*** sridharg has quit IRC | 15:11 | |
*** Kennan has quit IRC | 15:11 | |
jordanP | return izip_longest(*[iter(iterable)]*n, fillvalue=padvalue) | 15:12 |
jordanP | lol | 15:12 |
*** nineseconds has joined #openstack-qa | 15:12 | |
jordanP | I don't want to scratch my head or open the python documentation for 15min each time I read that code | 15:12 |
jordanP | there's no way I will remember what izip_longest do | 15:13 |
*** vhosakot has joined #openstack-qa | 15:13 | |
*** _vsaienko has quit IRC | 15:13 | |
jordanP | but I am going to replace "iterable" by "sliceable" it's a good suggestion and more accurate | 15:13 |
*** Kennan has joined #openstack-qa | 15:14 | |
afazekas | Yes, it is really unbelievable the izip_longest seams to be the best answer (still checking) | 15:14 |
afazekas | There should be one-- and preferably only one --obvious way to do it. | 15:14 |
afazekas | Although that way may not be obvious at first unless you're Dutch. | 15:14 |
afazekas | import this ;) | 15:14 |
sdague | sc68cal / kashyap - why did we introduce a global variable that's not guarunteed to be created in functions-common? | 15:15 |
sdague | https://review.openstack.org/#/c/319209/11 | 15:15 |
sdague | the -x check was fine | 15:15 |
openstackgerrit | Liron Kuchlani proposed openstack/tempest: Volume pagination with specific tenant https://review.openstack.org/318048 | 15:15 |
*** Junhongl_ has joined #openstack-qa | 15:16 | |
*** yamamoto has joined #openstack-qa | 15:17 | |
afazekas | jordanP: Would nice if at least our library function would be able handle larger stream than our memory .. | 15:17 |
*** e0ne has joined #openstack-qa | 15:18 | |
jordanP | afazekas, agreed. But given that the bigest stream we have to handle in tests are ~MB I am not sure it's worth it | 15:18 |
sdague | dtroyer_zz: why do you really want abs paths in devstack? | 15:19 |
*** Junhongl_ has left #openstack-qa | 15:20 | |
*** _vs has joined #openstack-qa | 15:22 | |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: functions-common: Use `systemctl` when applicable https://review.openstack.org/319209 | 15:22 |
openstackgerrit | Sean Dague proposed openstack-dev/devstack: functions-libvirt: Issue an explicit 'restart' of libvirt daemon https://review.openstack.org/319819 | 15:22 |
*** jlanoux has quit IRC | 15:22 | |
*** yamamoto has quit IRC | 15:24 | |
afazekas | jordanP, '50' voted answer seams the less tricky | 15:24 |
*** nineseconds has quit IRC | 15:25 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/316766 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error | 15:26 |
*** sridharg has joined #openstack-qa | 15:27 | |
openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 15:28 |
*** Lord_Phoenix has quit IRC | 15:28 | |
dtroyer_zz | sdague: because it is a solid best-practice with sudo in scripts. | 15:29 |
* dtroyer_zz just looked at that review again | 15:30 | |
dtroyer_zz | good grief | 15:30 |
dtroyer_zz | honestly, I just don't care any more | 15:31 |
dtroyer_zz | time to rewrite devstack in python | 15:32 |
*** nadya has quit IRC | 15:33 | |
*** _vs has quit IRC | 15:33 | |
mtreinish | slowrie: 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 |
sdague | dtroyer_zz: I would agree with that if you are targetting one platform | 15:33 |
sdague | but reinventing PATH inside of devstack seems really really weird | 15:33 |
mtreinish | slowrie: that's definitely on the testr side, I'm not sure why it can't find it's own partitioned test lists | 15:33 |
sdague | if [ -x /usr/sbin/service ]; .... elif [ -x /sbin/service ]; .... | 15:34 |
*** salv-orlando has quit IRC | 15:34 | |
slowrie | mtreinish: The code I had in my WIP was working to my knowledge | 15:34 |
sdague | that should be a signal to just use path for that | 15:34 |
dtroyer_zz | sdague: it isn't re-inventing PATH, it is removing opportunities for surprise, which in other situations is a large security problem | 15:34 |
sdague | dtroyer_zz: sure | 15:34 |
dtroyer_zz | but whatever | 15:34 |
sdague | but... 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 there | 15:35 |
mtreinish | slowrie: hmm it works when I pass --subunit, it must be something with the subunit-trace stdin side messing things up | 15:35 |
mtreinish | but that's a weird way for it to show up | 15:35 |
dtroyer_zz | so put it in. interesting that the current PS removes path from service as it adds full path for systemctl | 15:36 |
*** Lord_Phoenix has joined #openstack-qa | 15:36 | |
slowrie | mtreinish: I do remember there being something weird about when I tried to take the stdout and put it in as stdin to another cmd | 15:36 |
slowrie | that is a very strange way that it's bubbling out though | 15:36 |
dtroyer_zz | sdague: can I quote and reuse the "stop over-optimizing" comment? | 15:36 |
dtroyer_zz | I'm thinking of a plane with a banner for Barcelona | 15:37 |
sc68cal | sdague: gotcha. Sorry - yeah it was a micro-optimization | 15:37 |
jordanP | dtroyer_zz, I would stand behind that banner ! | 15:38 |
*** _vs has joined #openstack-qa | 15:38 | |
kashyap | sdague: I'm on a moving bus, reading the one line I was addressed to -- | 15:38 |
*** ihrachys has quit IRC | 15:39 | |
kashyap | sdague: sc68cal was insisting on optimizing the check for `systemctl` -- FWIW, I argued that I don't see a great benefit there | 15:39 |
slowrie | mtreinish: 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.BytesIO | 15:39 |
kashyap | I see that sc68cal has now reviewed that change though | 15:39 |
*** fnaval has joined #openstack-qa | 15:40 | |
*** vponomaryov has joined #openstack-qa | 15:40 | |
* sc68cal has the dent in his head from sdague's stick | 15:40 | |
openstackgerrit | Dale Smith proposed openstack/tempest: tempest_roles now used as default roles for preprov_creds. https://review.openstack.org/302332 | 15:40 |
*** Junhongl_ has joined #openstack-qa | 15:40 | |
mtreinish | slowrie: that fixed it, I copied what you had in the wip (more or less, there was a couple typos) and now it works | 15:41 |
sdague | dtroyer_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 why | 15:41 |
*** salv-orlando has joined #openstack-qa | 15:41 | |
kashyap | sdague: Alright, the function is what exactly I suggested before. I'll do that, and hopefully that'll suffice | 15:41 |
mtreinish | slowrie: heh, doesn't it make sense why I used subprocess in ostestr... | 15:42 |
*** _vs has quit IRC | 15:42 | |
slowrie | mtreinish: It's a nasty problem to solve, but that's more because Pythons types are insane | 15:42 |
kashyap | dtroyer_zz: No full-path because, not sure if you've seen my comment on PS8, from reading `sudoers` man page and some testing | 15:43 |
kashyap | dtroyer_zz: `sudo` retains all the paths we need | 15: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:/bin | 15:43 |
slowrie | mtreinish: 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 unittests | 15:43 |
kashyap | sdague: dtroyer_zz The above, when run as `root` will tell us what will override user's path (although that's configurable still) | 15:43 |
slowrie | mtreinish: current code is up @ https://review.openstack.org/#/c/283770 | 15:43 |
*** Junhongl_ has quit IRC | 15:44 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add tempest run command https://review.openstack.org/284411 | 15:44 |
jordanP | afazekas, that solution doesn't work as is | 15:44 |
jordanP | it fails the unit test "test_chunkify" that I just added | 15:44 |
sdague | kashyap: 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 libvirt | 15:44 |
jordanP | testtools.matchers._impl.MismatchError: 'aa' != ['a', 'a'] | 15:44 |
sdague | so... lets let the experimental results get back, and figure this thing out, then land the changes we need | 15:44 |
*** Junhongl_ has joined #openstack-qa | 15:44 | |
kashyap | sdague: Yeah, that `systemctl` presence optimization was a useless waste of time | 15:45 |
kashyap | I told much earlier about a function for that | 15:45 |
*** grafuls_ has joined #openstack-qa | 15:45 | |
*** ajmiller_ has joined #openstack-qa | 15:45 | |
*** anteaya has joined #openstack-qa | 15:45 | |
*** david-lyle has joined #openstack-qa | 15:45 | |
*** beisner has joined #openstack-qa | 15:45 | |
*** itzikb has joined #openstack-qa | 15:45 | |
*** aysyd has joined #openstack-qa | 15:45 | |
*** mvk has joined #openstack-qa | 15:45 | |
*** wznoinsk has joined #openstack-qa | 15:45 | |
*** ajmiller_ has quit IRC | 15:45 | |
*** ajmiller_ has joined #openstack-qa | 15:45 | |
*** grafuls_ has quit IRC | 15:45 | |
*** grafuls_ has joined #openstack-qa | 15:45 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Swift object client: use urllib3 builtin support for chunked transfer https://review.openstack.org/311112 | 15:46 |
kashyap | sdague: 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 |
kashyap | But I don't want to get hung up on that. And agreed - about the experimental results | 15:46 |
*** beisner has quit IRC | 15:46 | |
jordanP | humm I forgot about thet Reno note... | 15:47 |
*** Junhongl_ has quit IRC | 15:47 | |
*** Junhongl has joined #openstack-qa | 15:48 | |
*** yamamoto has joined #openstack-qa | 15:49 | |
mtreinish | slowrie: you need to use: https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/fixture/lockutils.py#L55 | 15:50 |
*** beisner has joined #openstack-qa | 15:50 | |
mtreinish | add that to the setup in your unit tests and it should work | 15:50 |
openstackgerrit | Rodrigo Duarte proposed openstack-dev/devstack: Tempest: add a Keystone reseller feature flag https://review.openstack.org/293497 | 15:51 |
slowrie | ok, will do | 15:51 |
*** Junhongl has quit IRC | 15:51 | |
*** sridharg has quit IRC | 15:54 | |
*** Swami has joined #openstack-qa | 15:55 | |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere https://review.openstack.org/297511 | 15:55 |
*** yamamoto has quit IRC | 15:57 | |
*** grafuls_ has quit IRC | 15:57 | |
*** ajmiller_ has quit IRC | 15:57 | |
*** anteaya has quit IRC | 15:57 | |
*** david-lyle has quit IRC | 15:57 | |
*** itzikb has quit IRC | 15:57 | |
*** aysyd has quit IRC | 15:57 | |
*** mvk has quit IRC | 15:57 | |
*** wznoinsk has quit IRC | 15:57 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Swift object client: use urllib3 builtin support for chunked transfer https://review.openstack.org/311112 | 15:57 |
*** armax has joined #openstack-qa | 15:57 | |
*** cody-somerville has joined #openstack-qa | 15:57 | |
*** ddieterly[away] has quit IRC | 15:57 | |
*** weshay has quit IRC | 15:58 | |
*** cody-somerville has quit IRC | 16:00 | |
*** _vs has joined #openstack-qa | 16:00 | |
*** _vs has quit IRC | 16:02 | |
*** ddieterly has joined #openstack-qa | 16:02 | |
*** _vsaienko has joined #openstack-qa | 16:05 | |
oomichi | andreaf: about https://review.openstack.org/#/c/313171/19/tempest/lib/auth.py | 16:06 |
*** jlanoux has joined #openstack-qa | 16:06 | |
*** grafuls_ has joined #openstack-qa | 16:06 | |
*** ajmiller_ has joined #openstack-qa | 16:06 | |
*** david-lyle has joined #openstack-qa | 16:06 | |
*** itzikb has joined #openstack-qa | 16:06 | |
*** aysyd has joined #openstack-qa | 16:06 | |
*** mvk has joined #openstack-qa | 16:06 | |
*** wznoinsk has joined #openstack-qa | 16:06 | |
oomichi | andreaf: self.scope and self._scope seem a little difficult to read. can we rename self._scope to clear meaning name | 16:06 |
jordanP | andreaf, I added a reno note, could you have a look if that matches your expectation? | 16:07 |
*** yamamoto has joined #openstack-qa | 16:08 | |
*** lezbar has joined #openstack-qa | 16:09 | |
*** dustins has joined #openstack-qa | 16:10 | |
slowrie | mtreinish: ended up fixing the problem by changing the name of the lock from workspace to workspaces | 16:13 |
kashyap | sdague: Ah, thanks for the fix-up, just noticed it. | 16:14 |
*** e0ne has quit IRC | 16:14 | |
*** _vsaienko has quit IRC | 16:14 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Remove the Glance HTTP client. Use the common Rest Client instead. https://review.openstack.org/311174 | 16:14 |
openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 16:15 |
slowrie | andreaf: ^ added oslo_concurrency.lockutils | 16:15 |
openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 16:16 |
*** itzikb has quit IRC | 16:16 | |
*** _vsaienko has joined #openstack-qa | 16:18 | |
openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Neutron: stop creating neutron debug probles https://review.openstack.org/318746 | 16:19 |
openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Neutron: stop creating neutron debug probes https://review.openstack.org/318746 | 16:21 |
*** yamamoto has quit IRC | 16:21 | |
*** yamamoto has joined #openstack-qa | 16:22 | |
*** yamahata has quit IRC | 16:22 | |
*** ig0r_ has quit IRC | 16:24 | |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Heat: wait condition: allow insecure HTTPS url https://review.openstack.org/292813 | 16:26 |
openstackgerrit | Yaroslav Lobankov proposed openstack/tempest: Use common "waiters.wait_for_image_status" function everywhere https://review.openstack.org/297511 | 16:28 |
openstackrecheck | Console logs not available after 13:37s for keystone-coverage-db 320156,9,ab61a71 | 16:33 |
*** alex_xu has quit IRC | 16:35 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/301770 failed gate-horizon-dsvm-integration in the gate queue with an unrecognized error | 16:36 |
*** alex_xu has joined #openstack-qa | 16:38 | |
openstackgerrit | Benny Kopilov proposed openstack/tempest: cinder service enable / disable tests https://review.openstack.org/319634 | 16:39 |
*** grafuls_ has quit IRC | 16:40 | |
*** Guest82686 has quit IRC | 16:43 | |
*** _vsaienko has quit IRC | 16:43 | |
*** Swami has quit IRC | 16:45 | |
*** mvk has quit IRC | 16:45 | |
afazekas | jordanP: use ''.join instead of list() | 16:48 |
*** pilgrimstack1 has quit IRC | 16:49 | |
*** yamahata has joined #openstack-qa | 16:49 | |
jordanP | afazekas, where ? | 16:49 |
afazekas | jordanP, Which version did you tried ? the 50 vote ? it has en explicit list() call | 16:51 |
jordanP | yeah the 50 vote | 16:51 |
jordanP | but nevermind I am not micro optimizing this. My 2 lines method works, is readable and fast enough. It covers all of the current needs | 16:52 |
afazekas | we need some encoding neutral builder.. | 16:52 |
jordanP | it's not needed | 16:52 |
jordanP | it would be nice | 16:52 |
*** ifarkas has quit IRC | 16:53 | |
jordanP | the point of this patch is to remove some ~100 LoC not to micro optimize or have a general approach for everyone's problem | 16:53 |
afazekas | jordanP, 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 |
afazekas | now, I am ok with fixing it when needed .. | 16:54 |
*** Apoorva has joined #openstack-qa | 16:55 | |
jordanP | afazekas, ok, sequence you got it :) | 16:55 |
afazekas | The normal fix would be adding something to the standard py library, which does this via native code in a normal way .. | 16:56 |
openstackgerrit | Jordan Pittier proposed openstack/tempest: Swift object client: use urllib3 builtin support for chunked transfer https://review.openstack.org/311112 | 16:57 |
jordanP | yeah, I find it weird that it's not in the stdlib, it's a popular question on stackoverflow | 16:57 |
jordanP | I have to go | 16:58 |
*** jordanP has quit IRC | 16:58 | |
*** ddieterly is now known as ddieterly[away] | 16:58 | |
*** e0ne has joined #openstack-qa | 16:59 | |
*** Lord_Phoenix has quit IRC | 16:59 | |
*** nwkarsten has quit IRC | 17:00 | |
afazekas | jordanP:Maybe we created non-sense problem that's why we do not have on obvious way to solve it ;) | 17:00 |
*** rbak has quit IRC | 17:02 | |
*** lucasagomes is now known as lucas-afk | 17:02 | |
*** ddieterly[away] is now known as ddieterly | 17:02 | |
afazekas | Maybe the chunkify should be outside to the put_object_with_chunk .. | 17:04 |
openstackrecheck | Console logs not available after 13:25s for gate-keystone-python27-db 317529,4,0c4c3a6 | 17:06 |
openstackgerrit | Steve McLellan proposed openstack-dev/devstack: Fix sha1 download for elasticsearch>2.0 https://review.openstack.org/320152 | 17:09 |
afazekas | jordanP: 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|afk | 17:11 | |
*** jpena is now known as jpena|off | 17:11 | |
*** e0ne has quit IRC | 17:12 | |
*** rbak has joined #openstack-qa | 17:15 | |
*** rbak has quit IRC | 17:15 | |
*** rbak has joined #openstack-qa | 17:17 | |
*** jlanoux has quit IRC | 17:17 | |
openstackgerrit | Franklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint. https://review.openstack.org/284273 | 17:18 |
openstackgerrit | Franklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint. https://review.openstack.org/284273 | 17:19 |
*** cody-somerville has joined #openstack-qa | 17:20 | |
fnaval | hi - 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.yaml | 17:23 |
*** Suyi has joined #openstack-qa | 17:25 | |
*** e0ne has joined #openstack-qa | 17:30 | |
openstackgerrit | Franklin Naval proposed openstack/tempest: Allows specifying a name for a particular endpoint. https://review.openstack.org/284273 | 17:32 |
*** dtantsur is now known as dtantsur|afk | 17:32 | |
*** cody-somerville has quit IRC | 17:32 | |
*** rbak has quit IRC | 17:34 | |
*** Ravikiran_K has quit IRC | 17:34 | |
*** rbak has joined #openstack-qa | 17:36 | |
*** nadya has joined #openstack-qa | 17:39 | |
*** ddieterly is now known as ddieterly[away] | 17:39 | |
*** yfried has joined #openstack-qa | 17:45 | |
*** Swami has joined #openstack-qa | 17:48 | |
*** yamamoto has quit IRC | 17:52 | |
*** yamamoto has joined #openstack-qa | 17:53 | |
rodrigods | oomichi, ping.. re: https://review.openstack.org/#/c/285541/ | 17:54 |
*** rlrossit has quit IRC | 17:54 | |
rodrigods | that patch had a depends-on | 17:54 |
rodrigods | but ianw -2 the devstack one and said to make in the other way around | 17:54 |
rodrigods | here https://review.openstack.org/#/c/293497/ | 17:55 |
*** pvaneck has joined #openstack-qa | 17:58 | |
afazekas | jordanP: 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 |
openstackgerrit | Castulo J. Martinez proposed openstack/tempest: Add missing tests for the image v2 API https://review.openstack.org/317088 | 17:59 |
afazekas | jordanP: The `Which part should do what` was not good even before your change | 18:00 |
*** rlrossit has joined #openstack-qa | 18:01 | |
*** cody-somerville has joined #openstack-qa | 18:09 | |
*** yamamoto_ has joined #openstack-qa | 18:12 | |
*** david-lyle_ has joined #openstack-qa | 18:12 | |
*** Apoorva_ has joined #openstack-qa | 18:13 | |
*** ajmiller__ has joined #openstack-qa | 18:14 | |
*** wznoinsk_ has joined #openstack-qa | 18:14 | |
*** aysyd has quit IRC | 18:15 | |
*** yamamoto has quit IRC | 18:16 | |
*** Apoorva has quit IRC | 18:16 | |
*** ajmiller_ has quit IRC | 18:16 | |
*** david-lyle has quit IRC | 18:16 | |
*** wznoinsk has quit IRC | 18:16 | |
*** salv-orlando has quit IRC | 18:21 | |
*** ig0r_ has joined #openstack-qa | 18:23 | |
*** nadya has quit IRC | 18:23 | |
*** rossella_s has quit IRC | 18:24 | |
*** rossella_s has joined #openstack-qa | 18:24 | |
*** ddieterly[away] is now known as ddieterly | 18:26 | |
openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: [WIP] Add neutron-metering agent and configuration https://review.openstack.org/320616 | 18:28 |
openstackgerrit | Merged openstack/tempest: Add a test case for rebuild of instances with volumes https://review.openstack.org/175949 | 18:29 |
*** rcernin has quit IRC | 18:29 | |
openstackgerrit | Merged openstack/tempest: Rename get_image_meta method into check_image for Glance V1 client https://review.openstack.org/307789 | 18:29 |
*** dustins has quit IRC | 18:33 | |
*** dustins has joined #openstack-qa | 18:33 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add whitelist and blacklist file options to tempest run https://review.openstack.org/320155 | 18:43 |
*** ntata| has quit IRC | 18:45 | |
*** dimtruck is now known as zz_dimtruck | 18:50 | |
mtreinish | timothyb89, david-lyle_: if you get a sec can you take a look at: https://review.openstack.org/#/c/319309/ and https://review.openstack.org/319516 | 18:50 |
*** ddieterly is now known as ddieterly[away] | 18:51 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/309482 failed gate-horizon-npm-run-test in the gate queue with an unrecognized error | 18:51 |
*** e0ne has quit IRC | 18:53 | |
*** cody-somerville has quit IRC | 18:56 | |
*** porrua has quit IRC | 18:57 | |
*** markvoelker_ has joined #openstack-qa | 18:57 | |
*** markvoelker has quit IRC | 18:59 | |
*** porrua has joined #openstack-qa | 18:59 | |
*** markvoelker has joined #openstack-qa | 19:00 | |
*** markvoelker_ has quit IRC | 19:02 | |
*** cody-somerville has joined #openstack-qa | 19:03 | |
*** cody-somerville has quit IRC | 19:04 | |
*** cody-somerville has joined #openstack-qa | 19:05 | |
*** ddieterly[away] is now known as ddieterly | 19:05 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/320176 failed gate-horizon-selenium-headless in the gate queue with an unrecognized error | 19:06 |
*** tqtran has joined #openstack-qa | 19:14 | |
*** zz_dimtruck is now known as dimtruck | 19:15 | |
openstackgerrit | Matthew Treinish proposed openstack/tempest: Add whitelist and blacklist file options to tempest run https://review.openstack.org/320155 | 19:18 |
mtreinish | oomichi, dims: if you get a sec can you take a look at: https://review.openstack.org/283667 | 19:20 |
mtreinish | that's needed for the tempest run command work | 19:20 |
*** _vsaienko has joined #openstack-qa | 19:21 | |
openstackgerrit | Merged openstack/os-testr: Seperate regex builder logic into a seperate module https://review.openstack.org/312747 | 19:22 |
mtreinish | dims: 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 |
dims | mtreinish : don't think i have anything pending :) | 19:24 |
mtreinish | heh, ok. Figured I'd ask at least | 19:25 |
openstackgerrit | Merged openstack/os-testr: Split functionality out of main https://review.openstack.org/283667 | 19:25 |
*** mvk has joined #openstack-qa | 19:27 | |
*** _vsaienko has quit IRC | 19:27 | |
*** _vsaienko has joined #openstack-qa | 19:29 | |
mtreinish | dims: actually there is 1 open patch: https://review.openstack.org/#/c/314775/ but I haven't figured out the answer to the open question there | 19:29 |
mtreinish | actually nm I don't think there is actually a difference | 19:30 |
*** cody-somerville has quit IRC | 19:31 | |
*** cody-somerville has joined #openstack-qa | 19:31 | |
*** cody-somerville has joined #openstack-qa | 19:31 | |
*** gfidente has quit IRC | 19:34 | |
openstackgerrit | Castulo J. Martinez proposed openstack/tempest: Add missing tests for the identity v3 API https://review.openstack.org/316182 | 19:34 |
*** anteaya has joined #openstack-qa | 19:34 | |
slowrie | mtreinish: If I move subunit-describe-calls into tempest/tools is there anywhere the documentation can live? | 19:36 |
slowrie | and the unittests | 19:36 |
mtreinish | slowrie: it would have to be in cmd | 19:36 |
mtreinish | for unittests | 19:36 |
mtreinish | for docs you can put that right in docs/source | 19:36 |
slowrie | alright, 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 over | 19:37 |
*** e0ne has joined #openstack-qa | 19:38 | |
mtreinish | ok, there's not a rush | 19:38 |
mtreinish | and sry to flip flop on that, I didn't really look at the details until I reviewed the change | 19:38 |
*** ig0r_ has quit IRC | 19:39 | |
*** dimtruck is now known as zz_dimtruck | 19:40 | |
slowrie | It's fine, I understand | 19:40 |
*** salv-orlando has joined #openstack-qa | 19:42 | |
*** zz_dimtruck is now known as dimtruck | 19:49 | |
*** _vsaienko has quit IRC | 19:55 | |
*** mfisher_ora has quit IRC | 19:56 | |
*** porrua has quit IRC | 19:58 | |
openstackgerrit | Matthew Treinish proposed openstack/os-testr: Add whitelist file to ostestr docs https://review.openstack.org/314775 | 19:58 |
mtreinish | andreaf, jordanP: ^^^ | 20:01 |
mtreinish | I updated it to say they are mutually exclusive | 20:01 |
openstackgerrit | Merged openstack/tempest: Add cleanup in identity/test_users_negative.py https://review.openstack.org/314130 | 20:03 |
*** Apoorva_ has quit IRC | 20:04 | |
*** shardy has quit IRC | 20:06 | |
*** pglass has quit IRC | 20:06 | |
*** ddieterly is now known as ddieterly[away] | 20:06 | |
openstackgerrit | Steve Heyman proposed openstack/tempest: Remove neutron OO wrappers https://review.openstack.org/320495 | 20:07 |
andreaf | oomichi: hi - I though calling a property internal value with the _ was normal, i.e. property is scope and internal is _scope | 20:09 |
*** sabeen has quit IRC | 20:09 | |
oomichi | rodrigods: 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-qa | 20:12 | |
openstackgerrit | Merged openstack/tempest: test_port_security_macspoofing_port: Don't assume DHCP port https://review.openstack.org/251423 | 20:13 |
openstackgerrit | Merged openstack-dev/devstack: Use path-mounted keystone when running in httpd https://review.openstack.org/193894 | 20:13 |
openstackgerrit | Merged openstack/openstack-health: Update the deployment docs https://review.openstack.org/319309 | 20:13 |
openstackgerrit | Andrea Frittoli proposed openstack/tempest: Add admin role on domain for v3 https://review.openstack.org/226297 | 20:14 |
*** _vsaienko has quit IRC | 20:15 | |
*** sabeen has joined #openstack-qa | 20:19 | |
andreaf | mtreinish: https://review.openstack.org/#/c/318272/ I had to rebase it, but no real change, if you don't mind re +2 | 20:19 |
*** tmatsu has joined #openstack-qa | 20:21 | |
*** Apoorva has joined #openstack-qa | 20:23 | |
*** _vsaienko has joined #openstack-qa | 20:23 | |
*** tmatsu has quit IRC | 20:26 | |
*** ddieterly[away] is now known as ddieterly | 20:29 | |
oomichi | andreaf: yeah, OK. +A | 20:29 |
andreaf | oomichi: thanks! | 20:30 |
andreaf | jordanP: yes, the reno looks good | 20:37 |
openstackgerrit | Stephen Lowrie proposed openstack/tempest: Centralized Workspaces https://review.openstack.org/283770 | 20:43 |
slowrie | mtreinish: andreaf: ^ | 20:43 |
*** e0ne has quit IRC | 20:49 | |
*** _vsaienko has quit IRC | 20:50 | |
*** e0ne has joined #openstack-qa | 20:50 | |
*** csomerville has joined #openstack-qa | 20:52 | |
*** cody-somerville has quit IRC | 20:52 | |
*** dimtruck is now known as zz_dimtruck | 20:54 | |
*** dustins has quit IRC | 20:57 | |
openstackgerrit | Stephen Lowrie proposed openstack/tempest: Add subunit-describe-calls https://review.openstack.org/320661 | 20:59 |
slowrie | mtreinish: ^ is the move of subunit-describe-calls | 21:00 |
slowrie | oomichi: ^ | 21:01 |
*** rcernin has joined #openstack-qa | 21:02 | |
*** dustins has joined #openstack-qa | 21:03 | |
*** rfolco has quit IRC | 21:03 | |
*** pauloewerton has quit IRC | 21:06 | |
oomichi | slowrie: 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-qa | 21:12 | |
*** gouthamr has quit IRC | 21:19 | |
*** ddieterly[away] is now known as ddieterly | 21:20 | |
*** ddieterly has quit IRC | 21:21 | |
*** zz_dimtruck is now known as dimtruck | 21:21 | |
*** rlrossit has quit IRC | 21:23 | |
*** yamahata has quit IRC | 21:24 | |
*** regXboi has quit IRC | 21:26 | |
*** _vsaienko has quit IRC | 21:27 | |
*** nwkarsten has joined #openstack-qa | 21:28 | |
*** dimtruck is now known as zz_dimtruck | 21:29 | |
*** e0ne has quit IRC | 21:30 | |
*** _vs has joined #openstack-qa | 21:33 | |
*** csomerville has quit IRC | 21:33 | |
*** yfried has quit IRC | 21:34 | |
*** pglass has joined #openstack-qa | 21:37 | |
*** rcernin has quit IRC | 21:43 | |
*** stevebak` is now known as stevebaker | 21:46 | |
*** Serlex has quit IRC | 21:46 | |
*** akshai has quit IRC | 21:48 | |
*** _vs has quit IRC | 21:55 | |
*** nwkarsten has quit IRC | 21:58 | |
*** dustins has quit IRC | 22:00 | |
*** nwkarsten has joined #openstack-qa | 22:00 | |
*** _vsaienko has joined #openstack-qa | 22:01 | |
*** apevec has joined #openstack-qa | 22:06 | |
*** _vsaienko has quit IRC | 22:09 | |
*** cody-somerville has joined #openstack-qa | 22:09 | |
*** cody-somerville has quit IRC | 22:09 | |
*** cody-somerville has joined #openstack-qa | 22:09 | |
*** gouthamr has joined #openstack-qa | 22:12 | |
*** _vs has joined #openstack-qa | 22:14 | |
*** nwkarsten has quit IRC | 22:16 | |
*** nwkarsten has joined #openstack-qa | 22:21 | |
openstackgerrit | Cindy Lu proposed openstack/tempest: Add more swift container tests https://review.openstack.org/308508 | 22:21 |
*** spzala has quit IRC | 22:22 | |
*** _vs has quit IRC | 22:22 | |
*** spzala has joined #openstack-qa | 22:22 | |
*** cody-somerville has quit IRC | 22:23 | |
*** rossella_s has quit IRC | 22:24 | |
*** rossella_s has joined #openstack-qa | 22:24 | |
*** _vs has joined #openstack-qa | 22:24 | |
*** spzala has quit IRC | 22:27 | |
*** nwkarsten has quit IRC | 22:28 | |
*** pglass has quit IRC | 22:30 | |
*** _vs has quit IRC | 22:30 | |
*** cody-somerville has joined #openstack-qa | 22:32 | |
*** apevec has left #openstack-qa | 22:32 | |
*** apevec has quit IRC | 22:32 | |
*** gouthamr_ has joined #openstack-qa | 22:34 | |
*** _vs has joined #openstack-qa | 22:36 | |
*** gouthamr has quit IRC | 22:37 | |
*** _vs has quit IRC | 22:39 | |
*** rbak has quit IRC | 22:43 | |
*** spzala has joined #openstack-qa | 22:43 | |
*** _vs has joined #openstack-qa | 22:47 | |
*** edmondsw has quit IRC | 22:50 | |
*** _vs has quit IRC | 22:52 | |
*** _vs has joined #openstack-qa | 22:54 | |
*** yamahata has joined #openstack-qa | 22:58 | |
*** zackf has quit IRC | 23:00 | |
*** _vs has quit IRC | 23:01 | |
*** openstackgerrit has quit IRC | 23:03 | |
*** openstackgerrit has joined #openstack-qa | 23:03 | |
*** zackf has joined #openstack-qa | 23:05 | |
*** _vs has joined #openstack-qa | 23:05 | |
*** sdague has quit IRC | 23:09 | |
*** zackf has quit IRC | 23:09 | |
*** vhosakot has quit IRC | 23:10 | |
*** markvoelker has quit IRC | 23:11 | |
*** david-lyle_ is now known as david-lyle | 23:12 | |
*** _vs has quit IRC | 23:16 | |
openstackgerrit | Sean M. Collins proposed openstack-dev/grenade: Call refactored entrypoints for Neutron startup https://review.openstack.org/312046 | 23:17 |
*** cody-somerville has quit IRC | 23:17 | |
openstackgerrit | Sean M. Collins proposed openstack-dev/grenade: Call refactored entrypoints for Neutron startup https://review.openstack.org/312046 | 23:17 |
*** _vsaienko has joined #openstack-qa | 23:20 | |
openstackgerrit | Sean M. Collins proposed openstack-dev/devstack: Neutron: Remove verbose and always set debug to true https://review.openstack.org/315655 | 23:23 |
*** _vsaienko has quit IRC | 23:27 | |
*** nwkarsten has joined #openstack-qa | 23:29 | |
*** Swami has quit IRC | 23:31 | |
*** _vs has joined #openstack-qa | 23:34 | |
*** nwkarsten has quit IRC | 23:34 | |
*** tosky has quit IRC | 23:34 | |
*** _vs has quit IRC | 23:40 | |
*** tochi has joined #openstack-qa | 23:41 | |
*** _vs has joined #openstack-qa | 23:45 | |
*** yamahata has quit IRC | 23:48 | |
*** _vs has quit IRC | 23:51 | |
*** _vsaienko has joined #openstack-qa | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!