*** DennyZhang has quit IRC | 00:02 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50384 failed tempest with an unrecognized error | 00:04 |
---|---|---|
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for bug 1235486 https://review.openstack.org/49846 | 00:11 |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Add security group to large_ops test https://review.openstack.org/50514 | 00:12 |
openstackrecheck | openstack/swift change: https://review.openstack.org/50377 failed tempest because of: https://bugs.launchpad.net/bugs/1218391 | 00:18 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some negative tests for force_delete/restore https://review.openstack.org/49488 | 00:20 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Remove query for closed bug 1223469 https://review.openstack.org/50414 | 00:23 |
*** dims has quit IRC | 00:24 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some tests for aggregate https://review.openstack.org/49157 | 00:24 |
*** matsuhashi has joined #openstack-qa | 00:30 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/50514 failed tempest with an unrecognized error | 00:43 |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some negative tests for flavor https://review.openstack.org/49588 | 00:48 |
*** sthaha has joined #openstack-qa | 01:02 | |
*** huangtianhua has joined #openstack-qa | 01:03 | |
*** DennyZhang has joined #openstack-qa | 01:16 | |
*** wanghao has joined #openstack-qa | 01:21 | |
*** liudong has joined #openstack-qa | 01:26 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Call a script to check for ERRORs in logs on tempest success https://review.openstack.org/50143 | 01:31 |
openstackgerrit | A change was merged to openstack/tempest: add some negative tests for volume updating: https://review.openstack.org/48841 | 01:40 |
*** adalbas has quit IRC | 01:44 | |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some negative tests for tenant updating https://review.openstack.org/48917 | 01:45 |
*** liudong has quit IRC | 01:51 | |
*** DennyZhang has quit IRC | 01:54 | |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some negative tests for tenant updating https://review.openstack.org/48917 | 01:54 |
*** liudong has joined #openstack-qa | 01:56 | |
openstackgerrit | minsel proposed a change to openstack/tempest: Fixes test_network_basic_ops https://review.openstack.org/49977 | 02:05 |
openstackgerrit | A change was merged to openstack/tempest: Test for the nova diagnostics API https://review.openstack.org/48249 | 02:06 |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some negative tests for tenant updating https://review.openstack.org/48917 | 02:06 |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some negative tests for tenant https://review.openstack.org/48917 | 02:14 |
*** sourabh has joined #openstack-qa | 02:22 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Move negative action tests to right place https://review.openstack.org/49982 | 02:24 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Reduce vm creations in negative tests https://review.openstack.org/50374 | 02:24 |
*** sourabh has quit IRC | 02:27 | |
openstackgerrit | Zhu Zhu proposed a change to openstack/tempest: Enhance tests for flavor_extra_spec API https://review.openstack.org/50523 | 02:28 |
*** DennyZhang has joined #openstack-qa | 02:30 | |
openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests https://review.openstack.org/47983 | 02:34 |
*** rfolco has joined #openstack-qa | 02:35 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add shelve/unshelve test of nova APIs https://review.openstack.org/49892 | 02:39 |
*** sourabh has joined #openstack-qa | 02:43 | |
openstackgerrit | gongysh proposed a change to openstack-dev/devstack: Add an environment variable to enable extensions in keystone https://review.openstack.org/44401 | 02:55 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/48173 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 02:59 |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Add meta-config via local.conf https://review.openstack.org/46768 | 03:02 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/44195 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 03:03 |
kong | Anju, ping | 03:03 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49977 failed tempest with an unrecognized error | 03:06 |
*** sourabh has quit IRC | 03:07 | |
*** rfolco has quit IRC | 03:07 | |
*** openstack has joined #openstack-qa | 03:23 | |
*** hobana.freenode.net sets mode: +ns | 03:23 | |
-hobana.freenode.net- [freenode-info] please register your nickname...don't forget to auto-identify! http://freenode.net/faq.shtml#nicksetup | 03:23 | |
*** hobana.freenode.net sets mode: -o openstack | 03:29 | |
-hobana.freenode.net- *** Notice -- TS for #openstack-qa changed from 1381289012 to 1357234505 | 03:29 | |
*** hobana.freenode.net sets mode: +c-s | 03:29 | |
*** fungi has joined #openstack-qa | 03:29 | |
*** bdperkin has joined #openstack-qa | 03:29 | |
*** DennyZhang has joined #openstack-qa | 03:29 | |
*** liudong has joined #openstack-qa | 03:29 | |
*** wanghao has joined #openstack-qa | 03:29 | |
*** huangtianhua has joined #openstack-qa | 03:29 | |
*** sthaha has joined #openstack-qa | 03:29 | |
*** matsuhashi has joined #openstack-qa | 03:29 | |
*** torandu has joined #openstack-qa | 03:29 | |
*** ianw has joined #openstack-qa | 03:29 | |
*** rockyg has joined #openstack-qa | 03:29 | |
*** harlowja has joined #openstack-qa | 03:29 | |
*** salv-orlando has joined #openstack-qa | 03:29 | |
*** yfried has joined #openstack-qa | 03:29 | |
*** kashyap has joined #openstack-qa | 03:29 | |
*** mkoderer has joined #openstack-qa | 03:29 | |
*** giulivo has joined #openstack-qa | 03:29 | |
*** kong has joined #openstack-qa | 03:29 | |
*** jaypipes has joined #openstack-qa | 03:29 | |
*** dosaboy has joined #openstack-qa | 03:29 | |
*** HenryG has joined #openstack-qa | 03:29 | |
*** Anju has joined #openstack-qa | 03:29 | |
*** openstackgerrit has joined #openstack-qa | 03:29 | |
*** sileht has joined #openstack-qa | 03:29 | |
*** notmyname has joined #openstack-qa | 03:29 | |
*** pleia2 has joined #openstack-qa | 03:29 | |
*** marun has joined #openstack-qa | 03:29 | |
*** hemnafk has joined #openstack-qa | 03:29 | |
*** klrmn has joined #openstack-qa | 03:29 | |
*** openstackrecheck has joined #openstack-qa | 03:29 | |
*** krtaylor has joined #openstack-qa | 03:29 | |
*** mtreinish has joined #openstack-qa | 03:29 | |
*** jeblair has joined #openstack-qa | 03:29 | |
*** dtroyer has joined #openstack-qa | 03:29 | |
*** EmilienM has joined #openstack-qa | 03:29 | |
*** adam_g has joined #openstack-qa | 03:29 | |
*** fbo_away has joined #openstack-qa | 03:29 | |
*** annegentle_ has joined #openstack-qa | 03:29 | |
*** morganfainberg has joined #openstack-qa | 03:29 | |
*** ekarlso has joined #openstack-qa | 03:29 | |
*** mikal has joined #openstack-qa | 03:29 | |
*** rahmu has joined #openstack-qa | 03:29 | |
*** zigo_ has joined #openstack-qa | 03:29 | |
*** stevebaker has joined #openstack-qa | 03:29 | |
*** jamielennox has joined #openstack-qa | 03:29 | |
*** shanks has joined #openstack-qa | 03:29 | |
*** hobana.freenode.net sets mode: +o jaypipes | 03:29 | |
*** cyeoh has joined #openstack-qa | 03:29 | |
*** lifeless has joined #openstack-qa | 03:29 | |
*** sfineberg has joined #openstack-qa | 03:29 | |
*** samalba has joined #openstack-qa | 03:29 | |
*** sdague has joined #openstack-qa | 03:29 | |
*** soren has joined #openstack-qa | 03:29 | |
*** mordred has joined #openstack-qa | 03:29 | |
*** jog0 has joined #openstack-qa | 03:29 | |
*** timello has joined #openstack-qa | 03:29 | |
*** russellb has joined #openstack-qa | 03:29 | |
*** plomakin_ has joined #openstack-qa | 03:29 | |
*** jgriffith has joined #openstack-qa | 03:29 | |
*** adarazs has joined #openstack-qa | 03:29 | |
*** clarkb has joined #openstack-qa | 03:29 | |
*** maurosr has joined #openstack-qa | 03:29 | |
*** psedlak has joined #openstack-qa | 03:29 | |
*** hobana.freenode.net changes topic to "Bug Triage Day: http://status.openstack.org/bugday/" | 03:29 | |
*** sourabh has joined #openstack-qa | 03:30 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49892 failed tempest with an unrecognized error | 03:35 |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some tests for user api https://review.openstack.org/50218 | 03:38 |
*** sourabh has quit IRC | 03:39 | |
*** afazekas has joined #openstack-qa | 03:41 | |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some tests for user api https://review.openstack.org/50218 | 03:45 |
openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding glance API DELETE /images/<imageid> tests https://review.openstack.org/47983 | 03:54 |
*** dguitarbite has joined #openstack-qa | 03:54 | |
*** sourabh has joined #openstack-qa | 03:55 | |
*** dguitarbite has left #openstack-qa | 04:00 | |
giulivo | ping wanghao | 04:01 |
giulivo | on https://review.openstack.org/#/c/50217/ I think it's worth a bug in cinder yes | 04:01 |
*** afazekas has quit IRC | 04:01 | |
giulivo | you want to report it yourself? | 04:01 |
openstackgerrit | QianLin proposed a change to openstack/tempest: add a test for test_images https://review.openstack.org/50526 | 04:02 |
wanghao | Yes, I will report it. | 04:02 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/50463 failed tempest with an unrecognized error | 04:02 |
giulivo | wanghao, to be honest I'd address the strings validation as unit test not in tempest | 04:04 |
giulivo | but there seem to be quite a few around attempting to validate strings | 04:04 |
giulivo | yet this could be a topic for the next qa meeting as dkranz also had something to say about the negative tests | 04:05 |
*** dguitarbite has joined #openstack-qa | 04:07 | |
Anju | giulivo : i think the exception is correct | 04:09 |
Anju | giulivo: https://review.openstack.org/#/c/50217/4/tempest/api/volume/test_volumes_snapshots_negative.py | 04:09 |
Anju | wanghao, giulivo : ping | 04:10 |
giulivo | hi Anju :) you mean is correct as in "that is what cinder returns"? | 04:11 |
Anju | yes | 04:11 |
Anju | giulivo | 04:11 |
*** liudong has quit IRC | 04:11 | |
Anju | i am pasting the api response | 04:11 |
giulivo | oh yeah the tests are okay we were debating about that not being the optimal response from cinder | 04:11 |
*** liudong has joined #openstack-qa | 04:11 | |
Anju | http://dpaste.com/1410456/ | 04:12 |
Anju | giulivo: I do not think that cinder should return a Bad Request | 04:12 |
kong | giulivo, when is the coming qa meeting? Could anyone participate it? | 04:12 |
Anju | kong: any body can participate every week in Thursday | 04:13 |
giulivo | kong, sure anyone can https://wiki.openstack.org/wiki/Meetings/QATeamMeeting | 04:13 |
giulivo | Anju, for which of the tests you think it shouldn? | 04:14 |
kong | what's the time exactlly? | 04:14 |
giulivo | Anju, I think it should for two of those: invalid_volume_id and without_passing_volume_id | 04:15 |
kong | giulivo, wanghao, anju, do u think we should create a new negative file for every test class? | 04:15 |
kong | giulivo, wanghao, anju, maybe no consensus now | 04:16 |
*** yfried has quit IRC | 04:16 | |
wanghao | Anju, I think the cinder api should validate the parameter which come from user, instead of directly query volume info. | 04:16 |
giulivo | kong, as far as I can tell, we should at least try to merge the negatives related to the same 'service' or 'functionality' (when talking about nova) | 04:16 |
giulivo | kong, I was more of the idea to merge negatives but we parallelize on classes | 04:17 |
giulivo | so this reorganization would actually speed up things | 04:17 |
giulivo | (I mean moving negatives out of the base class) | 04:17 |
kong | <giulivo>, thanks, I got it. | 04:18 |
openstackgerrit | Qing Xin Meng proposed a change to openstack/tempest: Add test for creating a server by using scheduler hints https://review.openstack.org/48899 | 04:20 |
wanghao | kong, adding new negative file wil make structure of tempest more clearly. Right? | 04:20 |
kong | <giulivo>, actually yes. | 04:20 |
kong | <wanghao>, actually yes | 04:21 |
giulivo | ?? | 04:21 |
giulivo | :) | 04:21 |
giulivo | sorry kong I didn't get what were you replying to? | 04:21 |
openstackgerrit | Steven Dake proposed a change to openstack/tempest: Increase heat's default max_template_size https://review.openstack.org/50528 | 04:21 |
giulivo | oh wanghao question one line upper, sorry | 04:21 |
kong | <giulivo>, sorry, said to wanghao. | 04:22 |
kong | :) | 04:22 |
giulivo | it was just me who missed that line because of the irc client wrapping all around | 04:22 |
kong | :) | 04:23 |
openstackgerrit | Steven Dake proposed a change to openstack-dev/devstack: Increase heat's defaut template size https://review.openstack.org/50529 | 04:24 |
Anju | giulivo: according to a api should raise a Bad request | 04:24 |
Anju | whwn we doing something wrong in a pi | 04:24 |
Anju | whwn we doing something wrong in api | 04:24 |
kong | guys, take lunch now | 04:24 |
giulivo | wanghao, see Anju's comment | 04:25 |
Anju | giulivo: :) | 04:25 |
giulivo | Anju, is there any clear definition of what is wrong? empty values? malformed xml? | 04:25 |
Anju | giulivo: wait | 04:26 |
*** sourabh has quit IRC | 04:28 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Move negative action tests to right place https://review.openstack.org/49982 | 04:30 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Reduce vm creations in negative tests https://review.openstack.org/50374 | 04:30 |
*** afazekas has joined #openstack-qa | 04:38 | |
Anju | wanghao, giulivo : I ran a cli/api of nova volume-show with invalid volume id and without any volume id | 04:38 |
Anju | the exception in NotFound | 04:39 |
*** masayukig has joined #openstack-qa | 04:42 | |
*** Madan has joined #openstack-qa | 04:42 | |
*** sourabh has joined #openstack-qa | 04:47 | |
openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/48831 failed tempest with an unrecognized error | 04:50 |
*** afazekas has quit IRC | 04:53 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48899 failed tempest with an unrecognized error | 04:54 |
Anju | wanghao: , giulivo : same for glance image-show | 04:55 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45676 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 04:57 |
*** wanghao has quit IRC | 04:59 | |
*** afazekas has joined #openstack-qa | 05:07 | |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/50529 failed tempest with an unrecognized error | 05:14 |
*** sourabh has quit IRC | 05:15 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add shelve/unshelve test of nova APIs https://review.openstack.org/49892 | 05:18 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Move negative action tests to right place https://review.openstack.org/49982 | 05:18 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Reduce vm creations in negative tests https://review.openstack.org/50374 | 05:18 |
*** huangtianhua has quit IRC | 05:22 | |
*** DennyZhang has quit IRC | 05:25 | |
*** SergeyLukjanov has joined #openstack-qa | 05:27 | |
openstackgerrit | A change was merged to openstack/tempest: Add some test_cases for compute-cli https://review.openstack.org/50251 | 05:33 |
mkoderer | Anju: this sounds correct to me | 05:33 |
Anju | mkoderer: which one | 05:33 |
Anju | :) | 05:33 |
Anju | mkoderer: ragrding the exception | 05:34 |
Anju | ? | 05:34 |
mkoderer | <Anju> the exception in NotFound | 05:34 |
Anju | mkoderer: what do you think it should raise Bad Request ? | 05:35 |
mkoderer | ok I see now | 05:36 |
mkoderer | Anju: what exactly is a "invalid" volume id? | 05:36 |
Anju | mkoderer : https://review.openstack.org/#/c/50217/ | 05:36 |
Anju | mkoderer: like you making snapshot without giving a id | 05:37 |
Anju | or making snapshot which is wrong | 05:37 |
mkoderer | Anju: ok, I am wondering if '' is really empty.. I would say it should by None | 05:39 |
Anju | mkoderer: http://dpaste.com/1410456/ | 05:39 |
mkoderer | Anju: ok I got your point.. it should be a BadRequest is such cases | 05:39 |
Anju | according to me | 05:39 |
Anju | its correct mkoderer | 05:40 |
Anju | glance image-show | 05:40 |
Anju | mkoderer: glance image-show will raise a NotFound exception too | 05:40 |
mkoderer | Anju: you mean the result is NotFound... but is this really correct? | 05:41 |
Anju | I cont say it is completely wrong | 05:41 |
Anju | mkoderer: nova volume-show | 05:41 |
mkoderer | Anju: I think this is a general problem since there is no check if a ID follows the correct structure | 05:41 |
Anju | throwing a NotFound | 05:42 |
mkoderer | afazekas: do you know if there is a syntax check of IDs in oslo? | 05:43 |
mkoderer | Anju: if you completely sktp the volume_id filed you get a BadRequest right? | 05:44 |
mkoderer | s/sktp/skip/ | 05:44 |
mkoderer | Anju: I will check the code if I have some time later the day | 05:45 |
Anju | mkoderer: ok | 05:45 |
Anju | mkoderer: no if a comlpetely remove the volume-id in a api | 05:46 |
Anju | then the exception is notfound | 05:46 |
mkoderer | Anju: that is also incorrect for me | 05:46 |
giulivo | yah I think cinder should return BadRequest for empty ids as well as invalid uuids | 05:46 |
mkoderer | giulivo: and if mandatory fields are missing completely | 05:47 |
giulivo | but unless there is some clear definition of what is a wrong request ... | 05:47 |
mkoderer | :) | 05:47 |
Anju | giulivon mkoderer: glance is also throwing NotFound | 05:48 |
giulivo | sure, but there isn't any test for that... the conversation started around a couple of negatives which are expecting NotFOund | 05:48 |
giulivo | enjoy guys, | 05:48 |
giulivo | see you later | 05:48 |
Anju | giulivo: a test in tempest for aggregates | 05:49 |
*** giulivo has quit IRC | 05:54 | |
*** SergeyLukjanov has quit IRC | 06:02 | |
*** wanghao has joined #openstack-qa | 06:03 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45676 failed tempest with an unrecognized error | 06:04 |
openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members https://review.openstack.org/48902 | 06:06 |
*** dguitarbite has quit IRC | 06:09 | |
*** tiantian has joined #openstack-qa | 06:11 | |
*** rockyg has quit IRC | 06:11 | |
*** rockyg has joined #openstack-qa | 06:12 | |
*** tiantian is now known as huangtianhua | 06:13 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49892 failed tempest with an unrecognized error | 06:14 |
*** shanks` has joined #openstack-qa | 06:15 | |
*** shanks has quit IRC | 06:17 | |
*** sourabh has joined #openstack-qa | 06:18 | |
wanghao | Anju, Yes, Api should raise a bad request, when we using like empty values, or invalid volume id . As I think it is something wrong we use the api, and it is better than raising NotFound,isn' t? | 06:22 |
*** sourabh has quit IRC | 06:24 | |
Anju | wanghao: that is ok | 06:25 |
*** SergeyLukjanov has joined #openstack-qa | 06:25 | |
Anju | but some tests you can see of aggregates also | 06:25 |
Anju | which are raising a Not Found | 06:25 |
*** rdekel has joined #openstack-qa | 06:27 | |
wanghao | It is sure. Maybe the issue we discuss should be 'what is more appropriate execption to openstack api when using empty values or malformed xml.' | 06:32 |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavors into v3 part2 https://review.openstack.org/50536 | 06:32 |
*** dkorn has joined #openstack-qa | 06:33 | |
Anju | wanghao: then we need toa mke changes in glance also | 06:33 |
*** sourabh has joined #openstack-qa | 06:36 | |
*** marun has quit IRC | 06:36 | |
*** SergeyLukjanov has quit IRC | 06:37 | |
wanghao | Anju, Yeah, We really need a conclusion of how to fix this issue, or just leave it. | 06:45 |
Anju | hmm wanghao | 06:45 |
*** SergeyLukjanov has joined #openstack-qa | 06:46 | |
wanghao | Anju, :) | 06:46 |
*** jhenner has joined #openstack-qa | 06:49 | |
*** SergeyLukjanov has quit IRC | 06:53 | |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: port test_flavors into v3 part2 https://review.openstack.org/50536 | 06:53 |
openstackgerrit | Xiang Hui proposed a change to openstack/tempest: Add security groups updating test for ports https://review.openstack.org/46894 | 06:56 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48902 failed tempest with an unrecognized error | 07:04 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: fixup_stuff python 2.6 https://review.openstack.org/50540 | 07:08 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 07:10 |
*** hemnafk has quit IRC | 07:11 | |
*** hemnafk has joined #openstack-qa | 07:12 | |
openstackgerrit | wanghao proposed a change to openstack/tempest: add negative test cases for create snapshots: https://review.openstack.org/50217 | 07:14 |
openstackrecheck | openstack/cinder change: https://review.openstack.org/50463 failed tempest because of: https://bugs.launchpad.net/bugs/1218391 | 07:16 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 07:17 |
*** jlibosva has joined #openstack-qa | 07:17 | |
*** gszasz has joined #openstack-qa | 07:19 | |
*** jhenner has quit IRC | 07:19 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/50184 failed tempest with an unrecognized error | 07:22 |
*** yeylon_ has joined #openstack-qa | 07:22 | |
*** jhenner has joined #openstack-qa | 07:23 | |
*** gszasz has quit IRC | 07:28 | |
*** gszasz has joined #openstack-qa | 07:29 | |
*** prekarat has joined #openstack-qa | 07:30 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/50531 failed tempest with an unrecognized error | 07:31 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the agent management extension API https://review.openstack.org/48421 | 07:33 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/46894 failed tempest with an unrecognized error | 07:35 |
*** liudong has quit IRC | 07:36 | |
*** jhenner has quit IRC | 07:44 | |
*** giulivo has joined #openstack-qa | 07:46 | |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add token tests https://review.openstack.org/50548 | 07:47 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: protected_hardlinks should be disabled https://review.openstack.org/50550 | 07:49 |
*** jhenner has joined #openstack-qa | 07:50 | |
*** yassine has joined #openstack-qa | 07:57 | |
*** kong has left #openstack-qa | 08:01 | |
*** jamielennox is now known as jamielennox|away | 08:02 | |
*** Lingxian has joined #openstack-qa | 08:03 | |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add token tests https://review.openstack.org/50548 | 08:08 |
*** jhenner has quit IRC | 08:10 | |
openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members https://review.openstack.org/48902 | 08:14 |
*** fbo_away is now known as fbo | 08:21 | |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 08:24 |
*** giulivo has quit IRC | 08:27 | |
*** dkorn has quit IRC | 08:28 | |
*** dkorn has joined #openstack-qa | 08:28 | |
*** masayukig has quit IRC | 08:31 | |
*** Dharmit has joined #openstack-qa | 08:31 | |
*** wanghao has quit IRC | 08:32 | |
*** giulivo has joined #openstack-qa | 08:36 | |
*** masayukig has joined #openstack-qa | 08:37 | |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the agent management extension API https://review.openstack.org/48421 | 08:39 |
*** dkorn has quit IRC | 08:40 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: delete the duplicated variable definition https://review.openstack.org/50557 | 08:40 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Test for the neutron api with provider extension https://review.openstack.org/50542 | 08:42 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48902 failed tempest with an unrecognized error | 08:44 |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add token tests https://review.openstack.org/50548 | 08:50 |
*** jhenner has joined #openstack-qa | 08:56 | |
*** mkollaro has joined #openstack-qa | 08:56 | |
*** kong has joined #openstack-qa | 09:06 | |
*** tiantian has joined #openstack-qa | 09:06 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/50184 failed tempest with an unrecognized error | 09:06 |
*** Lingxian has quit IRC | 09:06 | |
*** huangtianhua has quit IRC | 09:07 | |
*** dkorn has joined #openstack-qa | 09:09 | |
tiantian | Hi,Sean Dague,I have modified the tests fllowing your opinion alredy,would you please to review it.Thanks very much.https://review.openstack.org/#/c/48917/ | 09:09 |
openstackgerrit | Yang Yu proposed a change to openstack/tempest: Adding active tests for updating/replacing image members https://review.openstack.org/48902 | 09:19 |
*** gszasz has quit IRC | 09:19 | |
*** wanghao has joined #openstack-qa | 09:24 | |
*** qianlin has joined #openstack-qa | 09:27 | |
qianlin | tiantian,hi | 09:28 |
qianlin | hi all .would you please review the codes: https://review.openstack.org/#/c/50526/ | 09:32 |
tiantian | Hi,Anju,I have modified the tests following your opinion already,would you please to review it.Thanks very much.https://review.openstack.org/#/c/50548/ | 09:33 |
Anju | tiantian: :) | 09:34 |
openstackgerrit | Salvatore Orlando proposed a change to openstack-dev/devstack: Nicira plugin: do not die if NVP gateway IP is missing https://review.openstack.org/47414 | 09:36 |
tiantian | Anju,Thanks :) | 09:37 |
*** SergeyLukjanov has joined #openstack-qa | 09:41 | |
*** ken1ohmichi has joined #openstack-qa | 09:44 | |
openstackgerrit | Lingxian Kong proposed a change to openstack/tempest: add some negative tests for force_delete/restore https://review.openstack.org/49488 | 09:50 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50292 failed tempest because of: https://bugs.launchpad.net/bugs/1230407 and https://bugs.launchpad.net/bugs/1235486 and | 09:53 |
openstackgerrit | Hoisaleshwara Madan V S proposed a change to openstack/tempest: Port test_keypairs to v3 part_1 https://review.openstack.org/50571 | 09:53 |
openstackgerrit | Arata Notsu proposed a change to openstack-dev/devstack: Correct logging_context_format_string for Heat https://review.openstack.org/50572 | 09:54 |
ken1ohmichi | afazekas: hi | 09:54 |
*** masayukig has quit IRC | 09:55 | |
afazekas | ken1ohmichi: hi | 09:56 |
*** gszasz has joined #openstack-qa | 09:57 | |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: fixup_stuff python 2.6 and prettytable fix https://review.openstack.org/50540 | 10:01 |
ken1ohmichi | afazekas: could you take a look at https://review.openstack.org/#/c/50374/ ? I think you know this mechanism well. | 10:01 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48902 failed tempest because of: https://bugs.launchpad.net/bugs/1217734 | 10:01 |
rahmu | hi. I want to write a tempest test to cover a middleware that is not activated by default in devstack (swift crossdomain). Are there any steps I should take to activate it? | 10:01 |
afazekas | https://review.openstack.org/#/c/47494/ | 10:04 |
afazekas | ken1ohmichi: a previous similar patch got -1 because this ind of logic should be supported by a common utility | 10:05 |
afazekas | s/ind/kind/ | 10:05 |
giulivo | ping salv-orlando | 10:08 |
ken1ohmichi | afazekas: thanks for the info, I am going to read it carefully. | 10:09 |
afazekas | rahmu: try submit patch against devstack | 10:09 |
afazekas | ken1ohmichi: the linked patch is related to critical bug, so we should have common logic soon | 10:10 |
ken1ohmichi | afazekas: should we have common function in Tempest? | 10:11 |
afazekas | yes | 10:12 |
ken1ohmichi | afazekas: are somebody else trying it? can I do it now on this patchset? | 10:12 |
afazekas | yes can. AFAIK All other related pacth abandoned at the moment | 10:13 |
*** jhenner has quit IRC | 10:13 | |
ken1ohmichi | afazekas: thanks:-) I will try it soon. | 10:14 |
rahmu | afazekas: will do. I'll let you know if I run into an issue. Thanks | 10:14 |
*** jhenner has joined #openstack-qa | 10:15 | |
Anju | afazekas: a question | 10:15 |
Anju | create_image_from_rescued_server is raising an exception of Duplicate | 10:15 |
Anju | is this correct? | 10:16 |
*** fbo is now known as fbo_away | 10:16 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/45400 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 10:17 |
openstackgerrit | Xiang Hui proposed a change to openstack/tempest: Add security groups updating test for ports https://review.openstack.org/46894 | 10:18 |
*** mkollaro has quit IRC | 10:18 | |
afazekas | Anju: If it is not an allowed operation, the Conflict alias Duplicate is a good error code | 10:20 |
afazekas | Anju: If you think it should be an allowed operation you can file a bug | 10:21 |
Anju | yes afazekas : i caught it from here https://review.openstack.org/#/c/50526/1/tempest/api/compute/images/test_images.py | 10:22 |
Anju | afazekas: i think this wrong in tempest | 10:24 |
Anju | beacuse 409 status is for conflicting request | 10:24 |
Anju | afazekas, giulivo | 10:25 |
afazekas | I am Ok with renaming the Duplicate exception to Conflict and renaming the ComputeFault exception to ServerFault exception | 10:26 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50381 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 10:26 |
Anju | afazekas: yes it should be | 10:27 |
Anju | beacuse nova is throwing the correct exception | 10:27 |
* afazekas is hunting for food | 10:29 | |
*** afazekas has quit IRC | 10:30 | |
salv-orlando | giulivo: pong | 10:31 |
openstackgerrit | huangtianhua proposed a change to openstack/tempest: add some negative tests for tenant https://review.openstack.org/48917 | 10:37 |
*** wanghao has quit IRC | 10:38 | |
*** prekarat has left #openstack-qa | 10:50 | |
*** adalbas has joined #openstack-qa | 10:55 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/50577 failed tempest with an unrecognized error | 10:58 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added some test fo image tags https://review.openstack.org/50592 | 11:09 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added some test fo image tags https://review.openstack.org/50592 | 11:14 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/48917 failed tempest with an unrecognized error | 11:17 |
*** weshay has joined #openstack-qa | 11:18 | |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added some test fo image tags https://review.openstack.org/50592 | 11:22 |
*** afazekas has joined #openstack-qa | 11:25 | |
*** gszasz has quit IRC | 11:27 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Move negative action tests to right place https://review.openstack.org/49982 | 11:32 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Reduce vm creations in negative tests https://review.openstack.org/50374 | 11:32 |
giulivo | salv-orlando, hey there | 11:34 |
openstackgerrit | afazekas proposed a change to openstack/tempest: glance v2 image sharing tests https://review.openstack.org/50122 | 11:34 |
*** fbo_away is now known as fbo | 11:36 | |
*** Madan has quit IRC | 11:40 | |
*** tiantian has quit IRC | 11:41 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: glance image v1 member test cleaup https://review.openstack.org/50266 | 11:42 |
openstackgerrit | Denis Egorenko proposed a change to openstack-dev/devstack: Added Savanna Project https://review.openstack.org/50601 | 11:45 |
openstackrecheck | openstack/nova change: https://review.openstack.org/50373 failed tempest with an unrecognized error | 11:45 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50391 failed tempest because of: https://bugs.launchpad.net/bugs/1235437 | 11:46 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/tempest: Add a create_router utility function https://review.openstack.org/47553 | 11:47 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/tempest: API tests for neutron router gateway https://review.openstack.org/47554 | 11:47 |
*** tiantian has joined #openstack-qa | 11:49 | |
*** salv-orlando has quit IRC | 11:51 | |
*** rfolco has joined #openstack-qa | 12:07 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/44711 failed tempest because of: https://bugs.launchpad.net/bugs/1218391 | 12:10 |
*** gszasz has joined #openstack-qa | 12:19 | |
*** masayukig has joined #openstack-qa | 12:21 | |
*** dkorn has left #openstack-qa | 12:32 | |
*** raies has joined #openstack-qa | 12:46 | |
*** dkranz has joined #openstack-qa | 12:49 | |
*** fbo has quit IRC | 12:51 | |
*** fbo has joined #openstack-qa | 12:51 | |
*** ken1ohmichi has quit IRC | 12:53 | |
*** weshay has quit IRC | 12:59 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50608 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 13:02 |
*** DennyZhang has joined #openstack-qa | 13:02 | |
*** anteaya has joined #openstack-qa | 13:02 | |
*** qianlin has quit IRC | 13:09 | |
*** mriedem has joined #openstack-qa | 13:13 | |
*** tiantian has quit IRC | 13:13 | |
*** qianlin has joined #openstack-qa | 13:14 | |
*** weshay has joined #openstack-qa | 13:25 | |
*** yassine has quit IRC | 13:28 | |
*** matsuhashi has quit IRC | 13:30 | |
*** matsuhashi has joined #openstack-qa | 13:30 | |
sdague | afazekas: any thoughts on the interesting tearDown issue here - https://jenkins02.openstack.org/job/gate-tempest-devstack-vm-neutron-pg-isolated/136/console ? | 13:31 |
*** krtaylor has quit IRC | 13:32 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50389 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 13:32 |
*** yassine has joined #openstack-qa | 13:34 | |
*** matsuhashi has quit IRC | 13:34 | |
afazekas | (tempest.api.volume.test_volumes_actions.VolumesActionsTest was the first issue, the root tearDownClass is not called so the at exit raises a runtime error RuntimeError | 13:38 |
afazekas | sdague: I can added a forced super call decorator to all tearDownClass | 13:39 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: use join to list multiple bugs https://review.openstack.org/50633 | 13:41 |
sdague | afazekas: so I guess the question I really had, was is that in any way related? | 13:41 |
sdague | to the actual fail | 13:41 |
afazekas | 'Error in atexit._run_exitfuncs' usually appears if any tearDownClass raises an exception (without a super call). The exception raise by the the 'tearDownClass' would lead to the test suite fail anyway | 13:44 |
sdague | ok, is there a way to at least annotate that print out so that people realize it's not the cause of their failure | 13:45 |
openstackgerrit | Zhu Zhu proposed a change to openstack/tempest: Enhance tests for flavor_extra_spec API https://review.openstack.org/50523 | 13:45 |
sdague | just to reduce us needing to explain it to folks :) | 13:46 |
openstackgerrit | Joe Hakim Rahme proposed a change to openstack/tempest: Adds tests to cover Swift's crossdomain middleware https://review.openstack.org/50637 | 13:48 |
afazekas | A tearDownClass decorator would not be too difficult for a forced super call.. | 13:49 |
afazekas | BTW: http://logs.openstack.org/91/50391/2/gate/gate-tempest-devstack-vm-neutron-pg-isolated/41a11a9/logs/screen-q-svc.txt.gz#_2013-10-09_13_15_09_255 this was the issue | 13:49 |
afazekas | The times are matching | 13:50 |
sdague | afazekas: probably is | 13:50 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50391 failed tempest because of: https://bugs.launchpad.net/bugs/1235486 | 13:54 |
*** gszasz has quit IRC | 13:54 | |
*** raies has quit IRC | 13:56 | |
*** raies has joined #openstack-qa | 13:56 | |
sdague | afazekas: yep, that's what er thinks as well :) | 13:57 |
*** jhenner has quit IRC | 13:59 | |
*** dims has joined #openstack-qa | 14:00 | |
*** yassine has quit IRC | 14:00 | |
*** yassine has joined #openstack-qa | 14:01 | |
*** jhenner has joined #openstack-qa | 14:02 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Missing tearDownClass exception more verbose https://review.openstack.org/50643 | 14:04 |
*** prekarat has joined #openstack-qa | 14:05 | |
afazekas | sdague: https://review.openstack.org/50643 | 14:06 |
*** dkranz has quit IRC | 14:08 | |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added some test fo image tags https://review.openstack.org/50592 | 14:12 |
*** gszasz has joined #openstack-qa | 14:12 | |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Added some test fo image tags https://review.openstack.org/50592 | 14:13 |
raies | afazekas: do you have any idea about keystone single sign on ? | 14:15 |
afazekas | psedlak: ^^ | 14:16 |
afazekas | raies: what is your question.? | 14:17 |
raies | afazekas: my question is that if there is any possiblity for single sign on for authentication using keystone | 14:19 |
raies | mean user login for first time then prompted for authentication | 14:20 |
raies | but for further there will no such requirement | 14:20 |
afazekas | raies: usually the token provided by keystone is valid for 24h for all services | 14:23 |
openstackgerrit | Anju Tiwari proposed a change to openstack/tempest: Changed the exception name https://review.openstack.org/50648 | 14:23 |
afazekas | the token also contains the endpoint url's | 14:23 |
raies | afazekas: hmm | 14:24 |
*** dkranz has joined #openstack-qa | 14:24 | |
raies | I searched for single sign on but did not find any justifying keystone. How I can explain single sign on wrt keystone ? | 14:26 |
raies | afazekas: https://wiki.openstack.org/wiki/Keystone/Federation/Blueprint | 14:28 |
*** SergeyLukjanov has quit IRC | 14:30 | |
psedlak | raies: what afazekas meant is that you can use the token you obtained for longer then just one request | 14:30 |
psedlak | raies: but that's probably not what you are asking for | 14:31 |
*** yfried has joined #openstack-qa | 14:31 | |
*** salv-orlando has joined #openstack-qa | 14:32 | |
psedlak | raies: imho there is no way how to integrate keystone to other sso things (down on the level that keystone would consume it) yet | 14:32 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/50637 failed tempest with an unrecognized error | 14:32 |
psedlak | raies: there are ongoing efforts like https://github.com/openstack/identity-api/blob/master/openstack-identity-api/v3/src/markdown/identity-api-v3-os-oauth1-ext.md, 'bind' attr of https://github.com/openstack/identity-api/blob/master/openstack-identity-api/v3/src/markdown/identity-api-v3.md#tokens | 14:33 |
afazekas | http://docs.openstack.org/developer/keystone/external-auth.html | 14:33 |
raies | psedlak: then what is correct meaning of single sign on wrt keystone ? | 14:33 |
*** salv-orlando has quit IRC | 14:33 | |
raies | as for as I mean here is get token once (first step) and use this token for access to all the services. am I correct at this ? | 14:34 |
psedlak | raies: yeah, that's exactly what afazekas pointed out first | 14:35 |
psedlak | (in the expiration period of the token /like 1day ...) | 14:35 |
raies | psedlak, afazekas: thanks a lot :) | 14:36 |
*** DennyZha` has joined #openstack-qa | 14:36 | |
Anju | psedlak: but what is single sign | 14:36 |
psedlak | Anju: what it 'should' be? | 14:37 |
Anju | this is a different or i can say new for me in keystone | 14:37 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/tempest: Add a create_router utility function https://review.openstack.org/47553 | 14:37 |
openstackgerrit | Salvatore Orlando proposed a change to openstack/tempest: API tests for neutron router gateway https://review.openstack.org/47554 | 14:37 |
Anju | i mean is single sign something different | 14:37 |
Anju | ? | 14:37 |
Anju | because i never heard this before | 14:38 |
*** jhenner has quit IRC | 14:38 | |
*** jhenner has joined #openstack-qa | 14:38 | |
*** dkorn has joined #openstack-qa | 14:38 | |
*** yeylon_ has quit IRC | 14:42 | |
*** dkranz has quit IRC | 14:43 | |
openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/48833 failed tempest with an unrecognized error | 14:48 |
*** dkranz has joined #openstack-qa | 14:56 | |
*** jhenner has quit IRC | 14:58 | |
*** jhenner has joined #openstack-qa | 15:00 | |
*** qianlin has quit IRC | 15:01 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/49309 failed tempest because of: https://bugs.launchpad.net/bugs/1230407 | 15:02 |
psedlak | Anju: i don't know, like the possible integration with/delegation to other sso solutions would make a difference but nothing special about this now i think | 15:03 |
Anju | psedlak: hmm, thanks | 15:04 |
*** SergeyLukjanov has joined #openstack-qa | 15:06 | |
raies | afazekas: is there any option on keystone cli to specify token expiration duration ? | 15:10 |
openstackrecheck | openstack/glance change: https://review.openstack.org/34801 failed tempest with an unrecognized error | 15:12 |
*** krtaylor has joined #openstack-qa | 15:12 | |
raies | afazekas: I have checked that it can be configured in configuration file. But is there any way to specify it from CLI ? | 15:12 |
afazekas | raies: AFAIK no | 15:12 |
raies | if this feature is implemented in API? | 15:13 |
raies | afazekas: I also did not find | 15:13 |
*** sthaha has quit IRC | 15:13 | |
raies | may be it would be implemented in APIs. Do you have any idea ? | 15:13 |
afazekas | http://docs.openstack.org/trunk/openstack-compute/install/yum/content/keystone-conf-file.html | 15:15 |
afazekas | expiration | 15:15 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/50648 failed tempest with an unrecognized error | 15:16 |
raies | afazekas: no no I had seen it in advance before asking. | 15:16 |
raies | afazekas: My question is that there is option to set it in conf file | 15:17 |
*** afazekas has quit IRC | 15:17 | |
raies | but is this feature is supported for API | 15:17 |
raies | mean is there any API which can accomplish this task ? | 15:17 |
*** afazekas has joined #openstack-qa | 15:19 | |
*** HenryG has quit IRC | 15:19 | |
raies | afazekas | 15:20 |
raies | ? | 15:20 |
raies | is there any API which can accomplish this task ? | 15:20 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/50486 failed tempest with an unrecognized error | 15:24 |
*** rdekel has quit IRC | 15:30 | |
*** dkorn has quit IRC | 15:31 | |
*** SergeyLukjanov has quit IRC | 15:34 | |
openstackgerrit | Joe Hakim Rahme proposed a change to openstack/tempest: Adds tests to cover Swift's crossdomain middleware https://review.openstack.org/50637 | 15:40 |
*** yassine has quit IRC | 15:45 | |
*** HenryG has joined #openstack-qa | 15:46 | |
*** DennyZhang has quit IRC | 15:47 | |
*** DennyZha` has quit IRC | 15:47 | |
*** jlibosva has quit IRC | 15:51 | |
*** yfried has quit IRC | 15:55 | |
openstackrecheck | openstack/python-glanceclient change: https://review.openstack.org/48850 failed tempest with an unrecognized error | 15:55 |
openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: Include plugins before nova lib https://review.openstack.org/50684 | 15:56 |
*** yfried has joined #openstack-qa | 15:57 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50665 failed tempest because of: https://bugs.launchpad.net/bugs/1224001 | 16:01 |
*** anteaya has quit IRC | 16:07 | |
*** mlavalle has joined #openstack-qa | 16:08 | |
openstackgerrit | Mate Lakat proposed a change to openstack-dev/devstack: xenapi: Set networking defaults https://review.openstack.org/50684 | 16:09 |
*** bdperkin has quit IRC | 16:09 | |
*** bdperkin has joined #openstack-qa | 16:09 | |
*** mkollaro has joined #openstack-qa | 16:10 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/42240 failed tempest because of: https://bugs.launchpad.net/bugs/1225664 | 16:20 |
*** fbo is now known as fbo_away | 16:22 | |
openstackrecheck | openstack/heat change: https://review.openstack.org/50682 failed tempest because of: https://bugs.launchpad.net/bugs/1218391 | 16:33 |
*** mlavalle has quit IRC | 16:36 | |
*** yfried has quit IRC | 16:37 | |
*** prekarat has quit IRC | 16:40 | |
*** prekarat has joined #openstack-qa | 16:44 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/50661 failed tempest with an unrecognized error | 16:48 |
openstackgerrit | minsel proposed a change to openstack/tempest: Fixes test_network_basic_ops https://review.openstack.org/49977 | 16:49 |
*** SergeyLukjanov has joined #openstack-qa | 16:54 | |
*** giulivo has quit IRC | 16:55 | |
*** DennyZhang has joined #openstack-qa | 16:56 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: use join to list multiple bugs https://review.openstack.org/50633 | 17:07 |
*** jhenner has quit IRC | 17:16 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49977 failed tempest with an unrecognized error | 17:19 |
*** Dharmit has quit IRC | 17:20 | |
*** DennyZhang has quit IRC | 17:30 | |
*** marun has joined #openstack-qa | 17:44 | |
*** marun has quit IRC | 17:47 | |
*** marun has joined #openstack-qa | 17:48 | |
*** fbo_away is now known as fbo | 18:01 | |
*** yfried has joined #openstack-qa | 18:02 | |
*** gszasz has quit IRC | 18:10 | |
*** prekarat has quit IRC | 18:10 | |
*** rockyg has quit IRC | 18:25 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Fix test_files_at_url_pass https://review.openstack.org/50706 | 18:30 |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Reorganize tests into unit and functional tests https://review.openstack.org/49391 | 18:30 |
*** mlavalle has joined #openstack-qa | 18:32 | |
*** yeylon_ has joined #openstack-qa | 18:34 | |
*** rockyg has joined #openstack-qa | 18:36 | |
*** fbo is now known as fbo_away | 18:38 | |
*** jhenner has joined #openstack-qa | 18:49 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/50698 failed tempest with an unrecognized error | 18:59 |
openstackgerrit | David Kranz proposed a change to openstack-infra/devstack-gate: Don't check logs for ERRORs with neutron https://review.openstack.org/50718 | 19:19 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/49309 failed tempest because of: https://bugs.launchpad.net/bugs/1218391 | 19:19 |
*** DennyZhang has joined #openstack-qa | 19:22 | |
*** yfried has quit IRC | 19:22 | |
*** mlavalle has quit IRC | 19:47 | |
openstackgerrit | Min Li proposed a change to openstack-dev/devstack: Fix bugs for docker, bug #1237581. https://review.openstack.org/50731 | 19:51 |
*** mlavalle has joined #openstack-qa | 19:51 | |
openstackgerrit | Matthew Treinish proposed a change to openstack-infra/elastic-recheck: Convert test_classifier to be a unit test https://review.openstack.org/50733 | 20:00 |
openstackrecheck | openstack/heat change: https://review.openstack.org/50716 failed tempest with an unrecognized error | 20:05 |
*** DennyZhang has quit IRC | 20:13 | |
*** SergeyLukjanov has quit IRC | 20:21 | |
mtreinish | jog0: I've got a few e-r unit test patches in progress. Can you take a look at them, then I'll push a patch to turn on unit tests for e-r | 20:22 |
mtreinish | there's only one test file left to convert (besides test_queries which I'm leaving alone_ | 20:23 |
*** SergeyLukjanov has joined #openstack-qa | 20:23 | |
*** SergeyLukjanov has quit IRC | 20:25 | |
dkranz | sdague: Is https://review.openstack.org/#/c/48792/ good to go? The link in the comment of afazekas now points to a blank line but I want to make sure. | 20:28 |
mtreinish | dkranz: what about the 'WIP - remove this before merge' comment? | 20:30 |
openstackgerrit | Cyril Roelandt proposed a change to openstack/tempest: Add missing CLI Neutron tests https://review.openstack.org/50739 | 20:30 |
dkranz | mtreinish: I had not looked at that code yet. | 20:32 |
sdague | dkranz: not yet | 20:32 |
dkranz | sdague: OK, can you mark it WIP? | 20:32 |
sdague | actually, I'm cleaning up a thing in it | 20:32 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: Clean up sudo calls for better debugability https://review.openstack.org/48792 | 20:32 |
sdague | yeh I'll do that now | 20:33 |
dkranz | sdague: Do you have a minute to discuss the whitelist error thing? | 20:33 |
sdague | sure, though my brain power today is a bit sapped. | 20:33 |
dkranz | sdague: SOrry to hear that. | 20:33 |
dkranz | sdague: So I had the code ready to go but decided to run it against some of the latest builds. | 20:34 |
dkranz | sdague: Each one showed new bogus errors that were not there in previous runs. | 20:34 |
dkranz | sdague: I am concerned that chasing this white list is just not going to work. | 20:34 |
openstackrecheck | openstack/nova change: https://review.openstack.org/49617 failed tempest with an unrecognized error | 20:34 |
dkranz | sdague: I can check in what I have but unless there is some action to fix the bugs it is not going to buy us much. | 20:36 |
dkranz | sdague: I can't really be an error log nanny constantly checking to see if new things should be added to the whitelist as they show up. | 20:36 |
sdague | dkranz: so it's going to be a race to get us to a lockdown | 20:36 |
dkranz | sdague: And we can't make this hard fail until no errors are consistent | 20:37 |
sdague | can you build something that automatically grabs the error list out of elastic search? | 20:37 |
sdague | that would at least let us seed something | 20:37 |
sdague | that we might be able to land | 20:37 |
dkranz | sdague: Not sure what you mean | 20:37 |
sdague | so the issue is manually building the whitelist is time consuming, right? | 20:38 |
sdague | and by the time we get it, new things showed up | 20:38 |
dkranz | sdague: No, the problem is that the bogosity is non-deterministic | 20:38 |
dkranz | sdague: I don't think new bugs are being introduced too fast | 20:38 |
sdague | dkranz: well with logstash we've got the last 2 weeks of logs, and you can search for traces | 20:39 |
sdague | so you could start with everything it returned | 20:39 |
sdague | some might be fixed | 20:40 |
sdague | and if we didn't get a trace in 2 weeks we'll call it's introduction a bug | 20:40 |
dkranz | sdague: We still need a whitelist | 20:40 |
sdague | dkranz: right, so the whitelist starts with that | 20:41 |
sdague | which is basically current status quo | 20:41 |
dkranz | sdague: That's kind of what I've been doing but not with logstash | 20:41 |
sdague | then it can't get worse | 20:41 |
dkranz | sdague: I can proceed I guess | 20:41 |
sdague | so I think logstash tooling would help you automate knowing how big the whitelist really is | 20:41 |
dkranz | sdague: Except that I have never used it programatically | 20:42 |
sdague | honestly, it's another angle on the same problem elastic-recheck is solving for us, the errors are statistical in nature | 20:42 |
sdague | dkranz: I'd just dive into the elastic recheck code, it's not too bad (and it's not too much at this point) | 20:43 |
dkranz | sdague: ok, I'll take a loog | 20:43 |
sdague | I think we're going to basically end up creating a ton of statistical tooling based on logstash this cycle that let us go after these stastically infrequent bugs | 20:44 |
clarkb | thats the goal (once etherpad is sorted, I plan on shifting to logstash related things) | 20:45 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Update to latest tox https://review.openstack.org/49559 | 20:46 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Split out unit tests as separate tox jobs https://review.openstack.org/49562 | 20:46 |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Convert remaining hypervisors to plugin model https://review.openstack.org/50748 | 20:51 |
*** dkranz has quit IRC | 21:12 | |
jog0 | mtreinish: ack, will look it in a few | 21:13 |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/50748 failed tempest with an unrecognized error | 21:15 |
openstackgerrit | Aaron Rosen proposed a change to openstack-dev/devstack: OVS still working? https://review.openstack.org/50756 | 21:24 |
mtreinish | jog0: ok thanks | 21:24 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Update to latest tox https://review.openstack.org/49559 | 21:27 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Split out unit tests as separate tox jobs https://review.openstack.org/49562 | 21:27 |
openstackrecheck | openstack/keystone change: https://review.openstack.org/50744 failed tempest with an unrecognized error | 21:33 |
*** mriedem has quit IRC | 21:41 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Convert remaining hypervisors to plugin model https://review.openstack.org/50748 | 21:41 |
*** adalbas has quit IRC | 21:46 | |
*** jhenner has quit IRC | 21:49 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/50184 failed tempest with an unrecognized error | 21:52 |
*** mkollaro has quit IRC | 21:54 | |
*** dosaboy has quit IRC | 22:01 | |
*** mkollaro has joined #openstack-qa | 22:04 | |
*** weshay has quit IRC | 22:06 | |
*** wramthun has joined #openstack-qa | 22:13 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49559 failed tempest with an unrecognized error | 22:17 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/49562 failed tempest with an unrecognized error | 22:17 |
openstackrecheck | openstack-dev/devstack change: https://review.openstack.org/50748 failed tempest with an unrecognized error | 22:17 |
*** rfolco has quit IRC | 22:20 | |
*** dosaboy has joined #openstack-qa | 22:21 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/49627 failed tempest with an unrecognized error | 22:23 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Update lockutils from oslo-incubator https://review.openstack.org/50768 | 22:23 |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Use built in cleanup for servers in test_large_ops https://review.openstack.org/50769 | 22:26 |
openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Add security group to large_ops test https://review.openstack.org/50514 | 22:26 |
openstackrecheck | openstack/horizon change: https://review.openstack.org/42240 failed tempest because of: https://bugs.launchpad.net/bugs/1225664 | 22:28 |
*** masayukig has quit IRC | 22:45 | |
*** jamielennox|away is now known as jamielennox | 22:51 | |
*** mlavalle has quit IRC | 23:03 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Update lockutils from oslo-incubator https://review.openstack.org/50768 | 23:20 |
*** mriedem has joined #openstack-qa | 23:20 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!