| *** mriedem has quit IRC | 00:01 | |
| *** hemna is now known as hemnafk | 00:02 | |
| *** HenryG_ has joined #openstack-qa | 00:04 | |
| *** HenryG__ has joined #openstack-qa | 00:08 | |
| *** HenryG__ has quit IRC | 00:08 | |
| *** HenryG has quit IRC | 00:08 | |
| *** HenryG_ has quit IRC | 00:12 | |
| *** jasondotstar has joined #openstack-qa | 00:12 | |
| *** sthaha has joined #openstack-qa | 00:24 | |
| *** weshay has quit IRC | 00:52 | |
| *** marun has quit IRC | 01:03 | |
| *** lisandro2 has joined #openstack-qa | 01:18 | |
| *** nosnos has joined #openstack-qa | 01:22 | |
| *** lisandro2 has left #openstack-qa | 01:23 | |
| *** fandikurnia01 has joined #openstack-qa | 01:53 | |
| *** HenryG has joined #openstack-qa | 01:56 | |
| *** HenryG has quit IRC | 02:53 | |
| *** nosnos has quit IRC | 02:57 | |
| *** anteaya__ is now known as anteaya | 02:57 | |
| *** nosnos has joined #openstack-qa | 02:58 | |
| *** jhenner has joined #openstack-qa | 03:03 | |
| openstackgerrit | minsel proposed a change to openstack/tempest: Handling network resources in tenant isolation https://review.openstack.org/45578 | 03:15 |
|---|---|---|
| *** jhenner has quit IRC | 03:18 | |
| *** anteaya has quit IRC | 03:33 | |
| *** shanks has joined #openstack-qa | 03:56 | |
| *** SergeyLukjanov has joined #openstack-qa | 04:13 | |
| *** SergeyLukjanov has quit IRC | 04:14 | |
| *** prekarat has joined #openstack-qa | 04:16 | |
| *** SergeyLukjanov has joined #openstack-qa | 04:20 | |
| *** prekarat has quit IRC | 04:20 | |
| openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: test_server_actions into v3 part2 https://review.openstack.org/39609 | 04:29 |
| *** SergeyLukjanov has quit IRC | 04:30 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Bulk Subnets and Ports creation https://review.openstack.org/44870 | 04:31 |
| raies | jaypipes: hi | 04:32 |
| *** SergeyLukjanov has joined #openstack-qa | 04:33 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: dding negative test for port https://review.openstack.org/45372 | 04:39 |
| *** prekarat has joined #openstack-qa | 04:40 | |
| openstackgerrit | Mh Raies proposed a change to openstack/tempest: Adding negative test for port https://review.openstack.org/45372 | 04:41 |
| *** SergeyLukjanov has quit IRC | 04:48 | |
| *** SergeyLukjanov has joined #openstack-qa | 04:57 | |
| *** coolsvap has joined #openstack-qa | 05:02 | |
| raies | Hi I need help on git rebase | 05:11 |
| *** yfried has joined #openstack-qa | 05:22 | |
| *** jasondotstar has quit IRC | 05:31 | |
| *** harlowja has quit IRC | 05:31 | |
| *** harlowja has joined #openstack-qa | 05:31 | |
| clarkb | raies: best to just ask | 05:35 |
| raies | clarkb: I have a parent and its an child branch | 05:36 |
| raies | parent is updated | 05:36 |
| *** harlowja_ has joined #openstack-qa | 05:36 | |
| raies | not I need to update child | 05:36 |
| raies | so that parent is also being updated in child | 05:36 |
| clarkb | git review -d $parent_change_number #note resulting branch the git review -d $child_change_number and finally git rebase -i branchnameforparent | 05:37 |
| raies | you men I shold run follwing commands | 05:38 |
| raies | a. git review -d $parent_change_number | 05:38 |
| *** harlowja has quit IRC | 05:38 | |
| raies | b. git review -d $child_change_number on same terminal | 05:38 |
| raies | then c. git rebase -i | 05:39 |
| raies | ?? | 05:39 |
| *** harlowja has joined #openstack-qa | 05:39 | |
| clarkb | yes but rebase -i needs an argument | 05:39 |
| raies | yes i will give that also | 05:39 |
| clarkb | it is the branch name of the branch created by the first command | 05:39 |
| raies | yes i.e. for parent right ? | 05:40 |
| clarkb | yes | 05:40 |
| raies | actually I follwed last para of https://wiki.openstack.org/wiki/Gerrit_Workflow | 05:40 |
| raies | but there is a wrong (as per me) step "git checkout $DEV_TOPIC_BRANCH" | 05:41 |
| openstackgerrit | nayna-patel proposed a change to openstack/tempest: Added test to check list/show extensions-neutron https://review.openstack.org/45030 | 05:41 |
| raies | instead as per you here is "git review -d $child_change_number" | 05:41 |
| *** harlowja_ has quit IRC | 05:42 | |
| raies | clarkb: thanks | 05:42 |
| *** SergeyLukjanov has joined #openstack-qa | 05:42 | |
| clarkb | either way works | 05:42 |
| raies | no https://wiki.openstack.org/wiki/Gerrit_Workflow does not worke for me | 05:43 |
| raies | there is error no such branch during child checkout | 05:43 |
| clarkb | $DEV_TOPIC_BRANCH is a variable, a standin for where your child is | 05:44 |
| raies | yes I know I replaced that with topic branch | 05:45 |
| raies | clarkb: "error: could not apply error is " is commig on follwing your way after rebase | 05:46 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: added an api test for security_groups https://review.openstack.org/44607 | 05:46 |
| clarkb | raies can you paste the entire output beginning with the first step? | 05:46 |
| raies | ok wait for 1 min | 05:46 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Remove wait_for_image_resp_code https://review.openstack.org/38995 | 05:50 |
| openstackgerrit | afazekas proposed a change to openstack/tempest: Remove wait_for_image_resp_code https://review.openstack.org/38995 | 05:52 |
| raies | clarkb: http://pastebin.com/m4YVeSyy | 05:53 |
| raies | check L14 | 05:53 |
| clarkb | raies: you need to resolve the conflicts | 05:55 |
| clarkb | raies: `git status` will show you which files have conflicts, edit them, the file contents will be tagged around the conflicts, edit the file so that you remove the tags and the file has the content that you want | 05:56 |
| clarkb | raies: git add the files when you are done resolving the confligs then `git rebase --continue` | 05:56 |
| raies | ok | 05:56 |
| raies | clarkb: thanks | 06:06 |
| *** coolsvap has quit IRC | 06:13 | |
| *** dkorn has joined #openstack-qa | 06:16 | |
| *** nosnos has quit IRC | 06:17 | |
| *** wramthun has quit IRC | 06:22 | |
| *** EspressoLuv has joined #openstack-qa | 06:22 | |
| *** nosnos has joined #openstack-qa | 06:24 | |
| *** SergeyLukjanov has quit IRC | 06:43 | |
| *** prekarat has quit IRC | 06:50 | |
| *** afazekas has joined #openstack-qa | 06:57 | |
| *** prekarat has joined #openstack-qa | 06:59 | |
| *** afazekas_ has joined #openstack-qa | 07:05 | |
| *** afazekas_ has quit IRC | 07:06 | |
| raies | afazekas: please check https://review.openstack.org/#/c/44870/ . I have incorporated jaypipes review comments | 07:09 |
| raies | afazekas: now one subnet is created with name and other is without name parameter | 07:09 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add admin client for scenario tests https://review.openstack.org/46205 | 07:17 |
| afazekas | raies: I think once I commented would be nice to verify the other attributes when reading back the list. Can you add it ? | 07:28 |
| raies | afazekas: I did not understand | 07:29 |
| afazekas | You just assering the id but not the cidr or name | 07:30 |
| raies | where ? | 07:31 |
| afazekas | https://review.openstack.org/#/c/44870/9/tempest/api/network/test_networks.py L288 | 07:31 |
| raies | I think it is not necessary | 07:33 |
| raies | till now I am not seeing this type of check any where | 07:33 |
| raies | still if you are suggesting I can check that also | 07:33 |
| raies | what you say ? how can I get +2 from you :p | 07:34 |
| raies | afazekas: are u there ? | 07:37 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: add server resizing tests https://review.openstack.org/44732 | 07:38 |
| afazekas | raies: in some test cases they are verified in other test cases they are not, but the better way is to do it. +2 but I noted it. | 07:38 |
| afazekas | Can you review this one https://review.openstack.org/#/c/46050/ ? | 07:40 |
| raies | ok I will review it | 07:42 |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack-dev/devstack: remove whitebox configuration in tempest.conf https://review.openstack.org/46016 | 07:49 |
| raies | afazekas: what is the need of "_process_floating_response" | 07:51 |
| raies | does _root_tag_fetcher_and_xml_to_json_parse not handles all the resuests ? | 07:52 |
| *** afazekas has quit IRC | 07:55 | |
| *** giulivo has joined #openstack-qa | 07:56 | |
| *** yassine has joined #openstack-qa | 08:02 | |
| *** SergeyLukjanov has joined #openstack-qa | 08:09 | |
| *** fbo_away is now known as fbo | 08:17 | |
| *** tkammer has joined #openstack-qa | 08:17 | |
| openstackgerrit | A change was merged to openstack/tempest: Fix race condition for test_flavors_extra_specs https://review.openstack.org/45586 | 08:19 |
| openstackgerrit | A change was merged to openstack/tempest: Add service tags to api.volume https://review.openstack.org/45782 | 08:20 |
| *** bashok has joined #openstack-qa | 08:51 | |
| *** zhikunliu has joined #openstack-qa | 08:52 | |
| *** prekarat has quit IRC | 08:58 | |
| *** madan has joined #openstack-qa | 09:03 | |
| *** jhenner has joined #openstack-qa | 09:04 | |
| *** zhikunliu has quit IRC | 09:06 | |
| *** jhenner has quit IRC | 09:06 | |
| *** zhikunliu has joined #openstack-qa | 09:19 | |
| *** prekarat has joined #openstack-qa | 09:27 | |
| openstackgerrit | sahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's Container Quota middleware https://review.openstack.org/46232 | 09:31 |
| *** rdekel has joined #openstack-qa | 09:37 | |
| *** mkollaro has joined #openstack-qa | 09:39 | |
| openstackgerrit | sahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's container quotas middleware https://review.openstack.org/46232 | 09:39 |
| *** madan has quit IRC | 09:58 | |
| *** rdekel has quit IRC | 10:10 | |
| *** jhenner has joined #openstack-qa | 10:21 | |
| bashok | Hi, while running this test tempest.api.compute.security_groups.test_security_groups.SecurityGroupsTestJSON.test_server_security_groups[gate] on H3 code I am getting this error http://paste.openstack.org/show/46826/ , can anyone shed some light on this | 10:26 |
| *** madan has joined #openstack-qa | 10:26 | |
| openstackgerrit | Sahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's container quotas middleware https://review.openstack.org/46232 | 10:31 |
| *** SergeyLukjanov has quit IRC | 10:36 | |
| *** SergeyLukjanov has joined #openstack-qa | 10:38 | |
| *** afazekas has joined #openstack-qa | 10:51 | |
| *** SergeyLukjanov has quit IRC | 10:54 | |
| *** zhikunliu has quit IRC | 10:58 | |
| *** rdekel has joined #openstack-qa | 10:59 | |
| *** weshay has joined #openstack-qa | 11:02 | |
| *** SergeyLukjanov has joined #openstack-qa | 11:07 | |
| sdague | afazekas, cyeoh: can you guys vote on the new core thread when you get a chance, you are the remaining folks that haven't, thanks! | 11:08 |
| cyeoh | sdague: done | 11:12 |
| * afazekas searching | 11:17 | |
| sdague | afazekas: it's under the [qa] tag on dev, I sent it yesterday | 11:18 |
| *** rdekel has quit IRC | 11:20 | |
| afazekas | sdague: done, thx | 11:20 |
| sdague | no prob | 11:20 |
| bashok | afazekas, hi | 11:24 |
| bashok | afazekas, I am reposting my question again, | 11:24 |
| bashok | while running this test tempest.api.compute.security_groups.test_security_groups.SecurityGroupsTestJSON.test_server_security_groups[gate] on H3 code I am getting this error http://paste.openstack.org/show/46826/ , can anyone shed some light on this | 11:25 |
| bashok | afazekas, could you please provide some input, similarly I have other failure tests where I am seeing this "no attribute error" | 11:25 |
| afazekas | psedlak, jhenner, ianw: ^^ | 11:27 |
| *** fandikurnia01 has quit IRC | 11:28 | |
| openstackgerrit | Tal Kammer proposed a change to openstack/tempest: Initial basic setup of openstack and tempest config file https://review.openstack.org/42920 | 11:28 |
| *** jhenner has quit IRC | 11:39 | |
| *** jhenner has joined #openstack-qa | 11:40 | |
| *** HenryG has joined #openstack-qa | 11:40 | |
| afazekas | bashok: testtools version?, which tempest version?, how did you started tempest ? | 11:41 |
| sdague | gah, mkoderer I just realized I misspelled your first name through the whole core nomination process.... sorry about that. :( | 11:42 |
| mkoderer | sdague: I am used to that np :P | 11:43 |
| bashok | afazekas, http://paste.openstack.org/show/46836/ I followed the mentioned instructions | 11:44 |
| bashok | afazekas, having around 140+ 'error' tests as well, in total around 1000+ tests ran | 11:44 |
| afazekas | bashok: I will ask several guy about do they seen this issue, nowadays I am using fedora for test dev | 11:49 |
| sdague | mkoderer: well welcome to core, I just added privs to you, so you have +2 and +A now | 11:49 |
| sdague | giulivo: same to you | 11:49 |
| raies | sdague: please check and approve https://review.openstack.org/#/c/44870/ | 11:51 |
| bashok | afazekas, ok, I am mostly seeing "ImportError: No module named" and "no attirubute error" in the failed as well as 'error' tests. do you suggest to use any other versions of tempest? | 11:52 |
| afazekas | It looks like nose or testtools version issue | 11:55 |
| sdague | raies: looks fine, though I'd like jaypipes to make sure his concerns were addressed before approving | 11:58 |
| raies | sdague: ok | 12:00 |
| bashok | afazekas, nose==1.3.0 | 12:02 |
| bashok | this version is fine? | 12:02 |
| * giulivo didn't deserve that so much, but enjoys being trusted at least for not messing up :) | 12:02 | |
| Anju | mkoderer , giulivo : many many congratulations to both of you | 12:03 |
| *** annegentle has quit IRC | 12:08 | |
| giulivo | Anju, after all the time you managed to get that thing in the only week I was on pto! | 12:11 |
| giulivo | that was sad I wanted to +1 that | 12:12 |
| Anju | giulivo: talking about the volume tests | 12:13 |
| giulivo | yeah I meant the xml interface | 12:14 |
| Anju | yess | 12:14 |
| Anju | but now onwards u will give + 2 :) | 12:14 |
| *** madan has quit IRC | 12:17 | |
| mkoderer | giulivo: congrats :) | 12:18 |
| giulivo | mkoderer, so we only miss to meet now | 12:19 |
| openstackgerrit | Ralf Haferkamp proposed a change to openstack-dev/devstack: Fix LDAP support for openSUSE https://review.openstack.org/46265 | 12:27 |
| *** afazekas_ has joined #openstack-qa | 12:39 | |
| *** fandikurnia01 has joined #openstack-qa | 12:40 | |
| *** dkorn has quit IRC | 12:42 | |
| *** dkorn has joined #openstack-qa | 12:43 | |
| *** adalbas has joined #openstack-qa | 12:48 | |
| *** fandikurnia01 has quit IRC | 12:49 | |
| *** afazekas_ has quit IRC | 12:49 | |
| *** annegentle has joined #openstack-qa | 12:57 | |
| *** anteaya has joined #openstack-qa | 12:58 | |
| *** yves has joined #openstack-qa | 12:59 | |
| yves | Hi all, can someone help me with how jenkins checks python code? pep8 and flake8 succeed localy on my branch, but fail on the jenkeins server with this message : http://logs.openstack.org/54/46254/2/check/gate-horizon-pep8/acc14a5/console.html | 13:01 |
| *** annegentle is now known as annegentle_away | 13:02 | |
| timello | yves: you should import datetime only and use: datetime.timedelta instead | 13:03 |
| yves | timello: Thanks. How can I get these errors localy? what are the flake8 and pep8 settings so that I can check localy before submitting a patch? | 13:06 |
| *** yassine has quit IRC | 13:08 | |
| timello | yves: well... I don't know the horizon tree... but for nova we use run_tests.sh -p to check pep8 errors | 13:08 |
| *** yassine has joined #openstack-qa | 13:09 | |
| *** yfried has quit IRC | 13:09 | |
| yves | timello: works also for horizon, thanks :-) | 13:11 |
| timello | np | 13:11 |
| yves | timello: I was running "./run_tests.sh -N --pep8" but this was not reporting any error | 13:12 |
| *** mriedem has joined #openstack-qa | 13:15 | |
| timello | yves: yeah, I think -p does some extra things | 13:15 |
| yves | timello: well thanks for the help. is there a reason why we should only import full modules? | 13:22 |
| *** sthaha has quit IRC | 13:24 | |
| timello | yves: code standards for Openstack... but I think it explicitly shows where the function come from... it's more readable IMO. | 13:24 |
| *** malini_afk is now known as malini | 13:26 | |
| *** nosnos has quit IRC | 13:26 | |
| sdague | yves: -N prevents the hacking module from running | 13:31 |
| yves | timello: When I have a look here : https://wiki.openstack.org/wiki/CodingStandards it point to http://www.python.org/dev/peps/pep-0008/ which is less restrictive. Do you have any other link to the code standards for Openstack? Thanks a lot for your guidance. | 13:31 |
| sdague | which means -N -p gives the wrong answer | 13:31 |
| yves | sdague: Oh, ok, Thanks, the comments inside my run_sh file say "-N, --no-virtual-env Don't use virtualenv. Run tests in local environment" instead... | 13:35 |
| sdague | yves: correct, but hacking gets pulled because of the virtual env | 13:36 |
| yves | sdague: Thanks | 13:36 |
| sdague | https://github.com/openstack-dev/hacking is the repo we are using for this, and has the master HACKING.rst doc | 13:37 |
| sdague | projects can turn off checks if they want, so there may be some local variation | 13:37 |
| *** dkranz has joined #openstack-qa | 13:39 | |
| yves | sdague: thanks a lot all my questions are answered here https://github.com/openstack-dev/hacking/blob/master/HACKING.rst :-) | 13:40 |
| *** burt has joined #openstack-qa | 13:53 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add services tags to scenario tests https://review.openstack.org/45783 | 14:07 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add hacking check for service tags in scenario https://review.openstack.org/45784 | 14:07 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add service tag section to HACKING.rst https://review.openstack.org/45926 | 14:07 |
| *** yfried has joined #openstack-qa | 14:10 | |
| mtreinish | afazekas: ping | 14:11 |
| afazekas | mtreinish: pong | 14:12 |
| mtreinish | afazekas: hey is there a reason we need to keep bin/tempest around? | 14:12 |
| mtreinish | it's just an out of date duplicate of run_tests | 14:12 |
| afazekas | mtreinish: it is out of sync, you can remove it | 14:13 |
| afazekas | This script supposed to be used in released tempest version as an installed starter script, | 14:13 |
| mtreinish | afazekas: ok that's what I thought. I just wanted to make sure there weren't any setup.py implications | 14:13 |
| afazekas | now even the setup.cfg does not have it | 14:14 |
| afazekas | it hes | 14:14 |
| afazekas | it is referenced by the setup.cfg | 14:14 |
| mtreinish | afazekas: yeah I'll pull that out when I remove it | 14:15 |
| afazekas | Can we add the run_tests.sh with an another name ? | 14:15 |
| mtreinish | afazekas: add it to setup.cfg? | 14:17 |
| afazekas | yes, but the run_tests.sh seams too generic name | 14:17 |
| afazekas | https://review.openstack.org/#/c/45783/4 IMHO all test which boots a VM utilizes the image service | 14:18 |
| afazekas | mtreinish: can you review it https://review.openstack.org/#/c/35165/ | 14:20 |
| * afazekas I will be back | 14:24 | |
| *** annegentle_away is now known as annegentle | 14:24 | |
| *** adalbas has quit IRC | 14:28 | |
| *** afazekas has quit IRC | 14:30 | |
| openstackgerrit | Sahid Orentino Ferdjaoui proposed a change to openstack/tempest: Adds tests covering Swift's container quotas middleware https://review.openstack.org/46232 | 14:40 |
| *** SergeyLukjanov has quit IRC | 14:42 | |
| *** adalbas has joined #openstack-qa | 14:44 | |
| *** prekarat has quit IRC | 14:51 | |
| *** afazekas has joined #openstack-qa | 14:51 | |
| *** yassine has quit IRC | 14:58 | |
| *** yassine has joined #openstack-qa | 14:58 | |
| *** mkollaro has quit IRC | 15:03 | |
| *** dkorn has quit IRC | 15:07 | |
| *** yves has quit IRC | 15:15 | |
| *** mlavalle has joined #openstack-qa | 15:21 | |
| *** SergeyLukjanov has joined #openstack-qa | 15:21 | |
| *** yves has joined #openstack-qa | 15:28 | |
| *** raies_ has joined #openstack-qa | 15:32 | |
| raies_ | afazekas: please check https://review.openstack.org/#/c/45372/ | 15:34 |
| *** bashok has quit IRC | 15:36 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Remove bin/tempest script https://review.openstack.org/46305 | 15:40 |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Remove Whitebox tests https://review.openstack.org/46116 | 15:40 |
| *** marun has joined #openstack-qa | 16:02 | |
| *** yassine has quit IRC | 16:04 | |
| mtreinish | afazekas: about the image service tag, thats not a guarantee. It'll only pull the image from glance if there isn't a local copy in the cache. So I don't think we should tag that. | 16:09 |
| mtreinish | also if it was the case that it made the glance call every time then we wouldn't tag it anyway because the tag wouldn't mean anything then. (We'd have to tag every test that did a server create) | 16:10 |
| openstackgerrit | Sean McCully proposed a change to openstack/tempest: Changes Test, port is not gauranteed unused https://review.openstack.org/45459 | 16:10 |
| afazekas | Do you need a glance for the first cached version ? | 16:10 |
| *** fbo is now known as fbo_away | 16:11 | |
| afazekas | Even is the image cached is nova ask for the additional metadata when booting a new vm ? | 16:12 |
| mtreinish | afazekas: I'm not sure on the details I haven't looked at the nova code on it in detail. But, either way read the description for tags I put in the hacking file. | 16:14 |
| mtreinish | we only tag the test with a service if it makes a direct api call to the service. (or through an extension like nova images or volumes) | 16:14 |
| mtreinish | I'll be back for the meeting time to eat | 16:14 |
| *** fbo_away is now known as fbo | 16:24 | |
| *** yves has quit IRC | 16:26 | |
| *** DennyZhang has joined #openstack-qa | 16:26 | |
| DennyZhang | I've filed my first bug for tempest project early yesterday. (https://bugs.launchpad.net/tempest/+bug/1224155) . But I got no response since then. Do you guys have any suggestions for how to make it keep going? Thanks. | 16:27 |
| *** fbo is now known as fbo_away | 16:36 | |
| mkoderer | Meeting...... | 16:59 |
| mkoderer | :) | 17:00 |
| dkranz | sdague: You running the meeting? | 17:01 |
| sdague | yes, sorry just got back from lunch | 17:02 |
| *** DennyZhang has quit IRC | 17:02 | |
| *** mkollaro has joined #openstack-qa | 17:09 | |
| openstackgerrit | A change was merged to openstack/tempest: Add services tags to scenario tests https://review.openstack.org/45783 | 17:15 |
| *** rockyg has joined #openstack-qa | 17:21 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-dev/devstack: Enable tenant isolation to tempest for neutron https://review.openstack.org/46314 | 17:33 |
| mtreinish | mlavalle: ^^^ just fyi | 17:33 |
| *** HenryG_ has joined #openstack-qa | 17:33 | |
| mlavalle | mtreinish: cool! | 17:34 |
| openstackgerrit | DennyZhang proposed a change to openstack/tempest: Add missing import of 'subunit' in requirements.txt https://review.openstack.org/46315 | 17:35 |
| *** HenryG has quit IRC | 17:36 | |
| *** SergeyLukjanov has quit IRC | 17:42 | |
| *** SergeyLukjanov has joined #openstack-qa | 17:44 | |
| *** thomasbiege has joined #openstack-qa | 17:51 | |
| openstackgerrit | DennyZhang proposed a change to openstack/tempest: Add missing import of 'subunit' in requirements.txt https://review.openstack.org/46315 | 17:52 |
| openstackgerrit | DennyZhang proposed a change to openstack/tempest: Add missing import of 'subunit' in requirements.txt https://review.openstack.org/46324 | 17:58 |
| afazekas | https://review.openstack.org/#/c/42920/ | 18:01 |
| mkoderer | sdague: you mention a "meetup" in the meeting | 18:02 |
| mkoderer | sdague: what's your plan? | 18:02 |
| *** rockyg has quit IRC | 18:03 | |
| *** tkammer has quit IRC | 18:11 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Do not merge test of neutron tenant isolation. https://review.openstack.org/46330 | 18:13 |
| *** DennyZhang has joined #openstack-qa | 18:14 | |
| afazekas | sdague, mtreinish: https://review.openstack.org/#/c/35165/ | 18:21 |
| *** afazekas is now known as afazekas_zz | 18:23 | |
| *** raies_ has quit IRC | 18:29 | |
| mtreinish | mlavalle: it doesn't seem like it will pass the neutron gate job: http://logs.openstack.org/30/46330/1/check/gate-tempest-devstack-vm-neutron/87ca515/ | 18:31 |
| *** mlavalle has quit IRC | 18:34 | |
| *** morganfainberg has joined #openstack-qa | 18:38 | |
| *** atiwari has joined #openstack-qa | 18:40 | |
| dkranz | mkoderer: I think Sean was talking about a possible gathering between summits. I doubt he has a concrete plan at this point. | 18:41 |
| atiwari | anyone, please tell me how to fix the tempest issue occurring in https://review.openstack.org/#/c/46123/ due to my fix? | 18:41 |
| atiwari | it seems circular dependency me | 18:42 |
| atiwari | https://github.com/openstack/tempest/blob/master/tempest/api/identity/admin/v3/test_tokens.py test need some modification | 18:42 |
| openstackgerrit | A change was merged to openstack/tempest: Remove bin/tempest script https://review.openstack.org/46305 | 18:44 |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/devstack: WIP: Disable ceilometer.compute.nova_notifier driver https://review.openstack.org/46336 | 18:44 |
| dkranz | atiwari: Just a minute... | 18:51 |
| atiwari | thanks | 18:51 |
| *** mkollaro has quit IRC | 18:53 | |
| dkranz | atiwari: So is the issue that you are changing a failure code? | 18:53 |
| dkranz | atiwari: In some keystone api? | 18:53 |
| atiwari | actually as part of my fix, I changed KEystone API which used to return 401 and now it is 404 | 18:54 |
| dkranz | atiwari: Was this api part of grizzly? | 18:54 |
| atiwari | I think so | 18:55 |
| dkranz | atiwari: In any event, you will need to change the tempest test to accept either the old or new failure code. | 18:55 |
| atiwari | but if I make change in tempest all the existing commit will fail | 18:56 |
| *** thomasbiege has quit IRC | 18:56 | |
| atiwari | which still have the old API | 18:56 |
| dkranz | atiwari: I'm not sure it is a good idea to change this in the v2 api. | 18:57 |
| atiwari | it is v3 | 18:57 |
| atiwari | tempest/api/identity/admin/v3/test_tokens.py | 18:57 |
| dkranz | atiwari: was that part of grizzly? | 18:58 |
| dkranz | atiwari: Perhaps I was wrong. | 18:58 |
| dkranz | atiwari: One of the points of tempest is to block api changes by default. | 18:58 |
| dkranz | atiwari: If you want to do this you have to temporarily change tempest so that it will pass with either the old or new version | 18:59 |
| dkranz | atiwari: This has been called the tempest-two-step | 18:59 |
| atiwari | is there and guide line ? | 19:00 |
| dkranz | atiwari: I have to go now. I'm not sure. | 19:00 |
| dkranz | atiwari: Perhaps sdague or mtreinish can answer that. | 19:01 |
| atiwari | ok, thanks | 19:01 |
| dkranz | atiwari: I'll be back in a bit. | 19:01 |
| openstackgerrit | DennyZhang proposed a change to openstack/tempest: Add missing import of 'subunit' in test-requirements.txt https://review.openstack.org/46324 | 19:04 |
| atiwari | sdague or mtreinish , can you guys please help me? | 19:05 |
| *** mlavalle has joined #openstack-qa | 19:05 | |
| mlavalle | mtreinish: I have a couple of meetings this afternoon. But I'll take a look later today and get back to you. ok? | 19:08 |
| openstackgerrit | Edward Hope-Morley proposed a change to openstack/tempest: Add cinder-backup functional tests https://review.openstack.org/45465 | 19:11 |
| mlavalle | mtreinish: after a quick look on the logs, though, it seems we are exhausting the ip addresses available in the public network | 19:12 |
| *** fbo_away is now known as fbo | 19:12 | |
| mlavalle | mtreinish: how many parallel tests are you running in this vm? | 19:13 |
| mlavalle | mtreinish: this is the command in the log that creates the public net we are attaching to: | 19:27 |
| mlavalle | neutron subnet-create --ip_version 4 --gateway 172.24.4.225 --name public-subnet 1f400dd4-820d-480f-9789-3a6de905d477 172.24.4.224/28 -- --enable_dhcp=False | 19:27 |
| mlavalle | mtreinish: as you can see, we are using a CIDR /28. I think we have too few public addresses. | 19:29 |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/tempest: Disabling a test so that change 46123 can be reviewed https://review.openstack.org/46341 | 19:34 |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/tempest: Disabling a test so that change 46123 can be reviewed https://review.openstack.org/46341 | 19:35 |
| atiwari | can someone please review https://review.openstack.org/#/c/46341? | 19:36 |
| *** malini is now known as malini_afk | 19:40 | |
| *** adalbas has quit IRC | 19:44 | |
| *** adalbas has joined #openstack-qa | 19:46 | |
| *** SergeyLukjanov has quit IRC | 19:54 | |
| *** DennyZhang has quit IRC | 19:55 | |
| mtreinish | mlavalle: sorry I was in a meeting | 19:57 |
| mtreinish | mlavalle: it was running serially | 19:57 |
| mlavalle | mtreinish: in that case, i'll run some of those tests that failed tonight in my devstack instance and see what's the deal. I'll ping you tomorrow morning | 19:58 |
| *** mriedem has quit IRC | 19:59 | |
| *** mriedem1 has joined #openstack-qa | 20:00 | |
| *** mriedem1 has quit IRC | 20:01 | |
| afazekas_zz | mtreinish, sdague: https://review.openstack.org/#/c/36627/ | 20:02 |
| *** mriedem has joined #openstack-qa | 20:03 | |
| *** HenryG_ has quit IRC | 20:03 | |
| afazekas_zz | Does it need to be a gate config, or can it be devstack default ? | 20:03 |
| yfried | has anyone ever tried running nose with --pdb flag? | 20:05 |
| * afazekas_zz not | 20:07 | |
| *** afazekas_zz has quit IRC | 20:07 | |
| *** weshay has quit IRC | 20:12 | |
| *** fbo is now known as fbo_away | 20:16 | |
| *** weshay has joined #openstack-qa | 20:21 | |
| openstackgerrit | Arvind Tiwari proposed a change to openstack/tempest: Disabling a test so that change 46123 can be reviewed https://review.openstack.org/46341 | 20:32 |
| atiwari | mtreinish, waiting for review on https://review.openstack.org/#/c/46341, it is holding one of my Keystone fix | 20:36 |
| *** mlavalle_ has joined #openstack-qa | 20:55 | |
| sdague | mtreinish: so the thing that the nova team does on do not merge changes is to -2 their own patch | 20:55 |
| sdague | then you can control the hold, even in pushing new versions | 20:56 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: change security-groups to neutron in scenario https://review.openstack.org/46353 | 20:56 |
| mtreinish | sdague: ok I'll do that | 20:56 |
| *** mlavalle has quit IRC | 20:56 | |
| *** mlavalle_ is now known as mlavalle | 20:56 | |
| mtreinish | although I don't think that there will ever need to a be new version for it | 20:57 |
| sdague | mtreinish: sure, then you can just WIP it | 20:57 |
| *** mriedem1 has joined #openstack-qa | 21:00 | |
| *** bashok has joined #openstack-qa | 21:01 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: change security-groups to neutron in scenario https://review.openstack.org/46353 | 21:01 |
| *** mriedem has quit IRC | 21:02 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: change security-groups to neutron in scenario https://review.openstack.org/46353 | 21:05 |
| *** bashok has quit IRC | 21:05 | |
| adalbas | mtreinish, about the whitebox tests, they really use the sqlachemy schemas and changed the status in the database | 21:09 |
| adalbas | mtreinish, however, nova unit tests stubs all that on fakes. | 21:09 |
| adalbas | so, we dont really care about getting the status in the db once we remove the whitebox tests. Are my assumptions right? | 21:11 |
| mtreinish | adalbas: that's not entirely true. There are a set of nova unit tests that test migrations against a real db. | 21:14 |
| mtreinish | you would have to use an interface similar to that | 21:15 |
| adalbas | mtreinish, can you point where those nova tests are? | 21:15 |
| mtreinish | adalbas: sure one sec I'll dig them up | 21:16 |
| adalbas | mtreinish, i found nova/tests/db | 21:17 |
| adalbas | it seems to be those | 21:17 |
| mtreinish | adalbas: https://github.com/openstack/nova/blob/master/nova/tests/db/test_migrations.py | 21:18 |
| adalbas | mtreinish, right! tks, this one seems to have many of the tests too: https://github.com/openstack/nova/blob/master/nova/tests/db/test_db_api.py | 21:19 |
| *** dkranz has quit IRC | 21:21 | |
| *** weshay has quit IRC | 21:24 | |
| *** mriedem1 has quit IRC | 21:36 | |
| openstackgerrit | A change was merged to openstack/tempest: Add hacking check for service tags in scenario https://review.openstack.org/45784 | 21:52 |
| jog0 | sdague: ping | 21:52 |
| openstackgerrit | A change was merged to openstack/tempest: Add service tag section to HACKING.rst https://review.openstack.org/45926 | 21:58 |
| openstackgerrit | A change was merged to openstack/tempest: Remove Whitebox tests https://review.openstack.org/46116 | 21:59 |
| giulivo | mtreinish, maybe it's time for me to finally | 22:04 |
| giulivo | put in place into the checks thing somehing which would check properly for those skip messages? | 22:04 |
| giulivo | who such a bad working | 22:09 |
| giulivo | I meant the checks for the skip messages | 22:09 |
| *** mkollaro has joined #openstack-qa | 22:12 | |
| *** lifeless_ has joined #openstack-qa | 22:13 | |
| *** lifeless has quit IRC | 22:15 | |
| sdague | jog0: what's up? | 22:15 |
| *** lifeless_ is now known as lifeless | 22:18 | |
| jog0 | so I was thinking how to address the issue of people running recheck no bug when they shouldn't be | 22:18 |
| jog0 | and part of the issue is its sometimes hard to figure out what bug to mark | 22:18 |
| jog0 | sdague: so what about telling people to ask this room for help instead of running recheck no bug | 22:18 |
| jog0 | that will help us get better recheck numbers | 22:18 |
| sdague | jog0: we could, but the reality is the people that are consciencious already ask | 22:25 |
| sdague | and I don't think telling people to ask will help | 22:25 |
| sdague | in reality I think the current recheck system isn't fixable per say, we need something more robust that does pattern matching or provides user editting of the content later | 22:26 |
| jog0 | sdague: I am not sure if many people know about this room even | 22:26 |
| sdague | but it's better than nothing | 22:26 |
| jog0 | sdague: yeah logstash queries will be very useful | 22:27 |
| sdague | so the people that most abuse recheck no bug are developers that are pretty disconnected | 22:27 |
| sdague | I ran stats at some ptoin | 22:27 |
| sdague | point | 22:27 |
| jog0 | Ifind even russellb and others do recheck no bug | 22:27 |
| sdague | so I'd rather solve this in code | 22:27 |
| jog0 | maybe not often but sometimes | 22:27 |
| sdague | pretty infrequently | 22:27 |
| sdague | most of the time russellb will ask around for the right bug or dig it up, as will all the responsible devs. The reality is the manual process is pretty hard at this point | 22:28 |
| sdague | and I think we can tool it to be better | 22:28 |
| jog0 | sdague: ture | 22:29 |
| sdague | so I'd rather just live with what we have, and work towards full replacement tooling that groks the logs more | 22:29 |
| jog0 | true, I have been able to find some good logstash queries so far | 22:29 |
| *** atiwari has quit IRC | 22:29 | |
| sdague | yeh, once I finally stop traveling (man I know what mordred must feel like) my first goal is the stack trace checker in success runs | 22:30 |
| sdague | then this comes next | 22:30 |
| jog0 | sdague: hmm this may be easier then we think ... | 22:31 |
| jog0 | on a job fail search logstash for a set of queries | 22:31 |
| jog0 | and see if any pop up | 22:31 |
| sdague | sure, it should go through the es api though, so we can do it programatically | 22:32 |
| sdague | honestly, I started down that path, and then global requirements ate all my time :) | 22:32 |
| jog0 | yeah do you mind if I take a whack at it today | 22:32 |
| jog0 | sdague: heh | 22:32 |
| jog0 | have any working code on it? | 22:32 |
| sdague | I had some test connect code | 22:33 |
| sdague | it took me a bit to find the right search lib | 22:33 |
| sdague | but dinner is coming off right now, so I need to run to it. I can get it out to you in the morning (or maybe later tonight) | 22:34 |
| sdague | honestly have to find that git tree | 22:34 |
| jog0 | that would be great thanks | 22:34 |
| sdague | actually, this might be the test code I had - https://github.com/sdague/gatemine | 22:35 |
| clarkb | jog0: there is also a logstash/elasticsearch command line client written in ruby | 22:36 |
| clarkb | jog0: which may be helful for mocking something up | 22:36 |
| clarkb | (doing raw es in python is probably better for the actual thing) | 22:36 |
| jog0 | clarkb: thanks | 22:36 |
| openstackgerrit | A change was merged to openstack-dev/devstack: Added Trove (Database as a Service). https://review.openstack.org/38169 | 22:42 |
| *** jhenner has quit IRC | 22:56 | |
| *** adalbas has quit IRC | 23:11 | |
| jog0 | clarkb sdague: thanks for the help got the basics working, can filter a search to last 10 hits | 23:14 |
| *** mlavalle has quit IRC | 23:19 | |
| clarkb | cool. I would be interested in seeing what you have when you are ready to share | 23:23 |
| clarkb | jog0 ^ | 23:23 |
| openstackgerrit | Nachi Ueno proposed a change to openstack-dev/devstack: Add Elastic Search backend for ceilometer https://review.openstack.org/46384 | 23:29 |
| jog0 | clarkb: yeah http://paste.openstack.org/show/46941 | 23:29 |
| jog0 | first draft | 23:29 |
| jog0 | just lists 10 most recent of that | 23:30 |
| jog0 | next step is: When a tempest job failure is detected, by monitoring gerrit (using gerritlib), a collection of logstash queries will be run on the failed job to detect what the bug was. | 23:30 |
| clarkb | I like it | 23:31 |
| clarkb | I have been thinking that tracking the query string in the bugs may help too | 23:32 |
| clarkb | jog0 also not that when you get that event is the same time logstash gets it. and processing isnt immediate and may backlog | 23:33 |
| *** jhenner has joined #openstack-qa | 23:39 | |
| giulivo | ping dosaboy | 23:42 |
| *** jhenner has quit IRC | 23:44 | |
| giulivo | dosaboy for https://review.openstack.org/#/c/45465 I think you'll have to init the backupclient with the admin credentials | 23:44 |
| jog0 | clarkb: I was afraid that processing would be a bit later | 23:46 |
| jog0 | clarkb: yeah I want to hook into lauchpadlib too | 23:46 |
| clarkb | jog0: we can emit events/gearman jobs/somethign from the log processors if that helps | 23:47 |
| clarkb | they don't do that today though | 23:47 |
| jog0 | clarkb: so they would report when they are ready? | 23:48 |
| jog0 | I was wondering if there are any stream processing tools for elasticsearch | 23:48 |
| clarkb | es can do reverse queries, but logstash doesn't support them directly so they end up not being super useful and yes I believe you can stream | 23:49 |
| clarkb | we can also stream out of the logstash workers to other endpoints if that helps | 23:49 |
| jog0 | all I want to do is run a query on all incoming data | 23:49 |
| clarkb | jog0: reverse queries are the preferred solution I think | 23:50 |
| clarkb | but we would need to add support for them into the thing doing log processing | 23:50 |
| clarkb | which is logstash today | 23:50 |
| jog0 | for now I will just do the ugly solution, poll logtash until the results I want are in | 23:52 |
| clarkb | I think it is a good place to start | 23:53 |
| clarkb | jog0: if you delay ~60 seconds before the first query I expect most of your queries will hit what they need the first time around | 23:53 |
| clarkb | jog0: you can see what the delay is like by comparing the received_at field to the timestamp field for the last log entry of files | 23:54 |
| jog0 | oh good idea | 23:55 |
| clarkb | jog0: http://www.elasticsearch.org/guide/reference/api/percolate/ for the reverse queries (again not terribly useful now, but potentially useful if we make logstash understand them) | 23:57 |
| jog0 | clarkb: timestamp and recieved are pretty close | 23:58 |
| *** mkollaro has quit IRC | 23:58 | |
| jog0 | clarkb: yeah I saw that | 23:58 |
| jog0 | going to focus on the quick and dirty for now | 23:58 |
| clarkb | jog0: timestamps will be close when the system isn't under load | 23:58 |
| clarkb | if things go south we can fall behind (but typically catch up again in lulls) | 23:58 |
| jog0 | https://github.com/jogo/elasticRecheck | 23:59 |
| clarkb | awesome, I have watched that repo | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!