Monday, 2014-02-17

*** bhuvan_ has quit IRC00:00
*** bill_az_ has quit IRC00:03
*** bhuvan_ has joined #openstack-qa00:08
*** bhuvan has quit IRC00:08
*** banix has joined #openstack-qa00:20
*** matsuhashi has joined #openstack-qa00:23
*** KurtMartin has joined #openstack-qa00:26
cyeohif there's any tempest cores around would they mind looking at: https://review.openstack.org/#/c/72267/ https://review.openstack.org/#/c/72269/ as they need to be merged before the corresponding nova change can merge (and already have one +2)00:31
*** matsuhashi has quit IRC00:33
*** matsuhashi has joined #openstack-qa00:34
openstackgerritminsel proposed a change to openstack/tempest: Neutron LBaaS Agent Scheduler API test  https://review.openstack.org/6512000:34
*** Sukhdev has quit IRC00:35
*** matsuhashi has quit IRC00:38
*** cnesa has joined #openstack-qa00:39
*** bhuvan_ has quit IRC00:55
openstackgerritDaniel Salinas proposed a change to openstack-dev/devstack: Fixed missing backslash which was breaking the installation of trove with devstack  https://review.openstack.org/7389001:01
*** weshay has quit IRC01:03
*** matsuhashi has joined #openstack-qa01:04
*** nosnos has joined #openstack-qa01:05
*** dmorita has joined #openstack-qa01:11
openstackgerritminsel proposed a change to openstack/tempest: Neutron Allowed Address Pair API test  https://review.openstack.org/6625901:22
*** morganfainberg_Z is now known as morganfainberg01:28
openstackgerritminsel proposed a change to openstack/tempest: Neutron Extra DHCP Options API test  https://review.openstack.org/6774101:32
*** yamahata__ has quit IRC01:49
openstackgerritA change was merged to openstack-dev/devstack: Use database connection for keystone  https://review.openstack.org/7346301:52
*** yamahata_ has joined #openstack-qa01:54
*** matsuhashi has quit IRC02:18
*** matsuhashi has joined #openstack-qa02:26
openstackgerritKyle Mestery proposed a change to openstack-dev/devstack: Add support for the OpenDaylight Neutron MechanismDriver  https://review.openstack.org/6977402:41
*** KurtMartin has quit IRC02:54
*** ianw has quit IRC02:55
*** KurtMartin has joined #openstack-qa02:55
*** mriedem has quit IRC02:56
*** malini is now known as malini_afk02:57
*** ianw has joined #openstack-qa03:02
openstackgerritYongli He proposed a change to openstack/tempest: Use six.StringIO/BytesIO instead of StringIO.StringIO  https://review.openstack.org/7390303:10
*** samalba has quit IRC03:11
*** samalba has joined #openstack-qa03:11
openstackgerritChenZheng proposed a change to openstack/tempest: Add cold-migration on server test  https://review.openstack.org/6803303:11
*** otherwiseguy has quit IRC03:17
*** ivanzhu has joined #openstack-qa03:20
*** ivanzhu has quit IRC03:22
*** ivanzhu has joined #openstack-qa03:22
*** ivanzhu has quit IRC03:25
*** ivanzhu has joined #openstack-qa03:26
*** ivanzhu has quit IRC03:28
*** otherwiseguy has joined #openstack-qa03:29
*** ivanzhu1 has joined #openstack-qa03:30
*** otherwiseguy has quit IRC03:30
mikalAnyone here understand testing live migration in temptest?03:34
mikaltempest even03:34
mikalThe tests read to me like I need two compute hosts? Which I guess mean these tests never run in the gate?03:34
openstackgerritChenZheng proposed a change to openstack/tempest: Add extra list servers by host test  https://review.openstack.org/6523403:35
clarkbmikal correct03:47
openstackgerritChenZheng proposed a change to openstack/tempest: Add cold-migration on server test  https://review.openstack.org/6803303:47
*** matsuhashi has quit IRC03:48
*** ken1ohmichi has joined #openstack-qa03:58
* mikal goes off to learn how multi-node devstack works03:59
*** banix has quit IRC04:09
*** morganfainberg is now known as morganfainberg_Z04:13
openstackgerritYuiko Takada proposed a change to openstack/tempest: Separate negative tests for test_projects  https://review.openstack.org/7291204:25
*** matsuhashi has joined #openstack-qa04:32
*** amotoki has joined #openstack-qa04:39
openstackgerritA change was merged to openstack/tempest: Adds test_show_XX_fields cases to show some fields  https://review.openstack.org/6562204:43
openstackgerritChenZheng proposed a change to openstack/tempest: Add cold-migration on server test  https://review.openstack.org/6803304:44
*** yfried has quit IRC04:55
openstackgerritRushi Agrawal proposed a change to openstack/tempest: Add tests for filtering volumes  https://review.openstack.org/6338805:06
*** ivanzhu1 has quit IRC05:20
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391205:26
*** ken1ohmichi has quit IRC05:30
*** ivanzhu has joined #openstack-qa05:36
*** ken1ohmichi has joined #openstack-qa05:38
openstackgerritIvan-Zhu proposed a change to openstack/tempest: port nova v2 images related tests into nova v3 part2  https://review.openstack.org/6436105:38
*** yfried has joined #openstack-qa05:40
*** Madan1 has joined #openstack-qa05:54
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391206:01
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/compute)  https://review.openstack.org/6521706:08
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/{identity,orchestration,volume,utils})  https://review.openstack.org/7392106:09
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/object_storage)  https://review.openstack.org/7392306:12
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (common,services,stress)  https://review.openstack.org/7392406:12
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (scenario)  https://review.openstack.org/7392506:13
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (thirdparty)  https://review.openstack.org/7392606:13
*** saju_m has joined #openstack-qa06:15
openstackgerritnayna-patel proposed a change to openstack/tempest: Adds L3 agent test case to test_l3_agent_scheduler.py  https://review.openstack.org/6862606:18
openstackgerritChenZheng proposed a change to openstack/tempest: Add cold-migration on server test  https://review.openstack.org/6803306:36
openstackgerritA change was merged to openstack/tempest: Fix volume transfer tests with tenant isolation disabled  https://review.openstack.org/7203406:38
*** jaypipes has quit IRC06:42
*** jaypipes has joined #openstack-qa06:42
*** ChanServ sets mode: +o jaypipes06:42
*** afazekas_ has quit IRC06:42
*** nosnos has quit IRC06:42
openstackgerritChenZheng proposed a change to openstack/tempest: Add extra list servers by host test  https://review.openstack.org/6523406:46
openstackgerritIsaku Yamahata proposed a change to openstack-dev/devstack: neutron/service-agent: support consolidated service agent  https://review.openstack.org/7295206:46
*** nosnos has joined #openstack-qa06:52
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391207:03
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391207:06
*** nmagnezi has joined #openstack-qa07:09
*** KurtMartin has quit IRC07:16
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391207:25
*** afazekas_ has joined #openstack-qa07:27
openstackgerritshihanzhang proposed a change to openstack/tempest: Remove unused method _cleanup_ports  https://review.openstack.org/7395407:36
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391207:37
*** boris-42 has quit IRC07:40
*** saurabh_ has quit IRC07:44
*** ivanzhu has quit IRC07:51
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API  https://review.openstack.org/7391207:53
openstackgerritChenZheng proposed a change to openstack/tempest: Add cold-migration on server test  https://review.openstack.org/6803307:54
*** saurabh_ has joined #openstack-qa08:00
*** saurabh_ has quit IRC08:00
*** saurabh_ has joined #openstack-qa08:00
*** Anju has quit IRC08:01
openstackgerritValeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part1)  https://review.openstack.org/7386908:03
*** jhenner has quit IRC08:06
openstackgerritxu-haiwei proposed a change to openstack/tempest: Seperate negative tests for test_server_rescue  https://review.openstack.org/7145708:09
*** jlibosva has joined #openstack-qa08:10
*** ivanzhu has joined #openstack-qa08:12
openstackgerritafazekas proposed a change to openstack/tempest: Negative does need to be separated files  https://review.openstack.org/7396308:18
openstackgerritSylvain Baubeau proposed a change to openstack/tempest: Add Cinder tests for quota sets  https://review.openstack.org/7360608:20
*** saurabh_ has quit IRC08:20
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 (identity,volume,etc.)  https://review.openstack.org/7392108:21
*** jamielennox is now known as jamielennox|away08:22
*** saurabh_ has joined #openstack-qa08:24
*** Anju has joined #openstack-qa08:24
yfriedafazekas_: ping08:26
afazekas_yfried: pong08:27
*** afazekas_ is now known as afazekas08:27
openstackgerritafazekas proposed a change to openstack/tempest: Negative tests does need to be in a separated files  https://review.openstack.org/7396308:28
yfriedafazekas: I have a minor issue with patch that got merged while I was on vacation. it's mostly a variable-naming issue. should I fix it as part of a patch I'm submitting anyway, or should I submit it separately. should I open a bug for it?08:29
*** saurabh_ has quit IRC08:31
yfriedsalv-orlando: ^08:31
*** saju_m has quit IRC08:31
*** jroovers has joined #openstack-qa08:32
afazekasbug for variable naming ???08:32
afazekasno08:33
*** jroovers|afk has joined #openstack-qa08:33
*** saurabh_ has joined #openstack-qa08:34
yfriedafazekas: Had I seen it in review I would have commented, but since I haven't - I would like to change it now08:34
*** salv-orlando_ has joined #openstack-qa08:35
*** salv-orlando has quit IRC08:35
*** salv-orlando_ is now known as salv-orlando08:35
*** jroovers has quit IRC08:37
yfriedafazekas: is there a way to filter gerrit for changes to specific files?08:37
afazekasThe naming is irrelevant :)08:38
yfriedafazekas: or directories?08:38
yfriedafazekas: it really bugs me :) it's completely counter-intuitive08:39
afazekasUsually rename patches are not liked, because otherwise we would rename everything on every week..08:40
afazekasyfried: you can try to submit a patch, and we will see ..08:41
afazekasThere are always exceptions, when the rename is liked08:43
*** ivanzhu has quit IRC08:50
*** saju_m has joined #openstack-qa08:58
openstackgerritYair Fried proposed a change to openstack/tempest: Minor changes to scenario manager  https://review.openstack.org/7397308:58
*** jordanP has joined #openstack-qa08:59
yfriedafazekas: what do you think? https://review.openstack.org/#/c/73973/09:00
*** jhenner has joined #openstack-qa09:00
yfriedafazekas the variable is the last change on the manager09:00
*** jhenner has quit IRC09:03
openstackgerritshihanzhang proposed a change to openstack/tempest: Modify error decription in cleanup_ports  https://review.openstack.org/7395409:03
*** jhenner has joined #openstack-qa09:04
afazekasyfried: LGTM09:06
* afazekas waiting for jenkins09:06
yfriedafazekas: any chance it's small enough for you to approve immediately after jenkins?09:10
*** fbo_away is now known as fbo09:17
*** yassine has joined #openstack-qa09:17
afazekasyfried: I need a +2 from non Red Hat reviewer. before I can approve it. Normally a change needs 2 core reviewer and the submitter and reviewer needs originated form two different organization.  If a change unbreaks the gate, these guidelines does not needs to be strictly followed.09:26
jordanPyfried, could you elaborate your commit message plz ?09:27
yfriedjordanP: I really see how, apart from copying the diff results. any suggestions?09:29
yfriedjordanP: s/realy/"really don't"09:29
jordanP"use native neutron client to retrieve tenant's ressources instead of filtering in tempest"09:31
jordanPor something like this09:31
openstackgerritA change was merged to openstack/tempest: enable volume list tests for cinder v2 - part2  https://review.openstack.org/5973309:32
openstackgerritMarc Koderer proposed a change to openstack/tempest: Add multiple negative test generator support  https://review.openstack.org/7398209:35
*** andreaf has joined #openstack-qa09:47
*** rossella-s has joined #openstack-qa09:51
*** boris-42_ has joined #openstack-qa09:52
*** matsuhashi has quit IRC09:54
*** rossella-s has quit IRC09:54
*** rossella-s has joined #openstack-qa10:00
*** matsuhashi has joined #openstack-qa10:02
*** gszasz has joined #openstack-qa10:03
*** tkammer has joined #openstack-qa10:08
yfriedjordanP: very well. can you please -1?10:09
jordanPyfried, done10:11
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/compute)  https://review.openstack.org/6521710:12
*** matsuhashi has quit IRC10:17
*** matsuhashi has joined #openstack-qa10:17
*** matsuhashi has quit IRC10:22
*** mkollaro has joined #openstack-qa10:28
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: WIP: Add v3 os-migration API tests  https://review.openstack.org/7399510:28
openstackrecheckopenstack/horizon change: https://review.openstack.org/73748 failed check-tempest-dsvm-postgres-full because of: https://bugs.launchpad.net/bugs/126215310:34
openstackgerritA change was merged to openstack/tempest: Remove tests for Nova V3 API os-simple-tenant-usage  https://review.openstack.org/7226910:38
openstackgerritA change was merged to openstack/tempest: Remove using of deprecated self.headers (part1)  https://review.openstack.org/7386910:38
openstackgerritA change was merged to openstack/tempest: Add test case for snapshot-create with in-use volume  https://review.openstack.org/6628210:40
openstackgerritA change was merged to openstack-dev/devstack: Fix hook script for Ironic  https://review.openstack.org/7289010:41
openstackgerritA change was merged to openstack/tempest: Remove tests for Nova V3 API os-instance-usage-audit-log  https://review.openstack.org/7226710:41
*** vkozhukalov has joined #openstack-qa10:46
*** matsuhashi has joined #openstack-qa10:47
*** ociuhandu has quit IRC10:48
*** tosky has joined #openstack-qa10:50
*** jroovers|afk is now known as jroovers10:58
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: WIP: Add v3 os-migration API tests  https://review.openstack.org/7399511:01
*** ociuhandu has joined #openstack-qa11:06
*** yassine has quit IRC11:15
*** yassine has joined #openstack-qa11:15
*** akamyshnikova has quit IRC11:21
*** akamyshnikova has joined #openstack-qa11:24
*** matsuhashi has quit IRC11:37
openstackrecheckConsole logs not available after 13:23s for check-tempest-dsvm-cells-full 73924,1,7b93ca811:56
*** dmorita has quit IRC11:58
*** yfried has quit IRC12:10
openstackgerritYaroslav Lobankov proposed a change to openstack/tempest: Savanna: add changes to API client and new tests  https://review.openstack.org/7159212:15
openstackgerritA change was merged to openstack/tempest: Test updating port's admin_state_up  https://review.openstack.org/7282912:16
*** rfolco has joined #openstack-qa12:17
openstackgerritA change was merged to openstack/tempest: Ensure that bug number is actually a number for skip_because  https://review.openstack.org/7275012:21
*** dkranz has quit IRC12:24
*** yfried has joined #openstack-qa12:26
*** yfried has quit IRC12:26
*** vponomaryov has joined #openstack-qa12:29
*** yfried1 has joined #openstack-qa12:30
openstackgerritMarc Koderer proposed a change to openstack/tempest: Add multiple negative test generator support  https://review.openstack.org/7398212:34
*** weshay has joined #openstack-qa12:45
*** mkollaro1 has joined #openstack-qa12:52
*** mkollaro has quit IRC12:52
openstackgerritSergey Nikitin proposed a change to openstack/tempest: Improved tests in tempest.api.image.v2.*  https://review.openstack.org/6809612:55
openstackgerritYair Fried proposed a change to openstack/tempest: Minor changes to scenario manager  https://review.openstack.org/7397312:56
openstackgerritYair Fried proposed a change to openstack/tempest: Refactor cross_tenant to security_groups_basic_ops  https://review.openstack.org/6956712:56
*** pafuent has joined #openstack-qa12:57
openstackgerritElena Ezhova proposed a change to openstack/tempest: Add api tests for load balancer's VIPs and health monitors  https://review.openstack.org/6679613:00
openstackgerritElena Ezhova proposed a change to openstack/tempest: Refactor create_ and update_ methods for VIPs and health_monitors  https://review.openstack.org/6651613:00
openstackgerritYair Fried proposed a change to openstack/tempest: Preventing overlapping subnets in network scenarios  https://review.openstack.org/5299413:01
openstackgerritYair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops  https://review.openstack.org/6936113:01
openstackgerritYair Fried proposed a change to openstack/tempest: Splits network_basic_ops to fully isolated test cases  https://review.openstack.org/6687913:01
openstackgerritYair Fried proposed a change to openstack/tempest: Preventing overlapping subnets in network scenarios  https://review.openstack.org/5299413:05
openstackgerritYair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops  https://review.openstack.org/6936113:05
openstackgerritYair Fried proposed a change to openstack/tempest: Splits network_basic_ops to fully isolated test cases  https://review.openstack.org/6687913:05
openstackgerritYair Fried proposed a change to openstack/tempest: Minor changes to scenario manager  https://review.openstack.org/7397313:05
openstackgerritYair Fried proposed a change to openstack/tempest: Refactor cross_tenant to security_groups_basic_ops  https://review.openstack.org/6956713:05
*** nosnos has quit IRC13:08
*** yfried1 has quit IRC13:09
*** nmagnezi has quit IRC13:12
andreafmaurosr: ping13:15
openstackgerritA change was merged to openstack/tempest: port nova v2 images related tests into nova v3 part2  https://review.openstack.org/6436113:15
*** mkollaro has joined #openstack-qa13:17
*** nmagnezi has joined #openstack-qa13:18
*** ken1ohmichi has quit IRC13:18
openstackgerritGiulio Fidente proposed a change to openstack/tempest: Adds cinder backup functional tests  https://review.openstack.org/6761613:19
*** mkollaro1 has quit IRC13:21
*** malini_afk is now known as malini13:22
*** yfried has joined #openstack-qa13:25
*** malini is now known as malini_afk13:36
*** salv-orlando has quit IRC13:43
*** Madan1 has quit IRC13:46
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object  https://review.openstack.org/7370413:49
mtreinishHenryG: pong13:51
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object  https://review.openstack.org/7370413:57
openstackgerritYaroslav Lobankov proposed a change to openstack/tempest: Savanna: add changes to API client and new tests  https://review.openstack.org/7159214:02
openstackgerritGavin Brebner proposed a change to openstack-dev/devstack: Create glance-swift for swift or s-proxy  https://review.openstack.org/7404714:05
*** mkollaro has quit IRC14:05
*** tkammer has quit IRC14:06
HenryGmtreinish: I have a question about some different behavior between invoking a unit test case via tox, vs invoking the same test case via nosetests in the venv.14:13
HenryGmtreinish: the project is neutron, but I hope the answer is generic14:14
*** saju_m has quit IRC14:15
*** banix has joined #openstack-qa14:16
mtreinishHenryG: sure, it should be the same between projects. (although we use somethings in tempest that prevent nose from working)14:16
mtreinishHenryG: so what's the question?14:18
HenryGmtreinish: 1 sec, trying to get back to where I was yesterday when I hit the problem ...14:19
*** mriedem has joined #openstack-qa14:22
openstackgerritA change was merged to openstack/tempest: network: add metering api operations  https://review.openstack.org/6614314:23
HenryGmtreinish: so here is the test case:  neutron.tests.unit.ml2.test_ml2_plugin.TestMl2PortBinding.test_create_port_binding_profile_with_empty_dict14:24
HenryGmtreinish: runs fine with 'testr run ...'14:25
HenryGmtreinish: If I activate the venv and run it with 'nosetests ...' (using colon to separate module from class) it gives a traceback.14:26
HenryGmtreinish: the reason for the traceback is very obvious: it is an setup order problem in one of the setUp() methods for that test case.14:27
HenryGmtreinish: if I fix that so that nosetests works, then tox (testr) fails on other test cases.14:28
HenryGmtreinish: so it looks to me like tox/testr is initializing things very differently from nosetests.14:29
HenryGmtreinish: I guess that is the question. What is the difference?14:30
mtreinishHenryG: so you have to remember that tox unit test job for neutron runs testr in parallel. My guess is that the change is interacting with other tests somehow14:31
mtreinishI'm not sure what is causing the initial failure though if you can pastebin the tracebacks I'd get a better idea14:31
HenryGmtreinish: even when I specifically run just one single test case?14:32
mtreinishwell you said when you applied the fix other test cases were failing14:32
HenryGYes, but after the 'fix' even running one single other test case fails.14:33
HenryGLet me start pastebinning ...14:34
HenryGnosetests failure with no fix:  http://paste.openstack.org/show/66486/14:36
HenryGThe fix:  http://paste.openstack.org/show/66487/14:37
*** bill_az_ has joined #openstack-qa14:37
HenryGtestr failure after the fix:   http://paste.openstack.org/show/66488/14:39
openstackgerritSergey Lukjanov proposed a change to openstack-dev/devstack: Improve savanna-dashboard installation  https://review.openstack.org/7405414:39
*** afazekas has quit IRC14:46
mtreinishHenryG: can I see a pastebin of the success with testr before the change?14:47
mtreinishbecause that's clearly a bug in setUp() there14:47
mtreinishwithout the super call() the config object shouldn't be initialized14:47
mtreinishso there isn't anything to call set default on14:47
HenryGmtreinish: here:  http://paste.openstack.org/show/66493/14:49
*** shwetaap has quit IRC14:55
openstackgerritMikhail Durnosvistov proposed a change to openstack/tempest: Expand baremetal nodes coverage  https://review.openstack.org/7406514:58
openstackgerritJulien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH  https://review.openstack.org/7406714:58
*** gszasz has quit IRC15:00
*** gszasz has joined #openstack-qa15:01
*** YorikSar has joined #openstack-qa15:03
*** otherwiseguy has joined #openstack-qa15:04
*** yfried has quit IRC15:07
*** rustlebee is now known as russellb15:08
*** rwsu has joined #openstack-qa15:09
*** shwetaap has joined #openstack-qa15:11
*** malini_afk is now known as malini15:16
*** nmagnezi has quit IRC15:19
*** yfried has joined #openstack-qa15:19
*** yfried has quit IRC15:22
*** giulivo has joined #openstack-qa15:22
*** yfried has joined #openstack-qa15:23
*** dkranz has joined #openstack-qa15:25
*** coolsvap has joined #openstack-qa15:26
*** protux has joined #openstack-qa15:27
*** enikanorov has joined #openstack-qa15:31
mkodererdkranz: Hi, here's a new patch https://review.openstack.org/#/c/73982/15:33
mkodererdkranz: I rewrote the json generator and tried to make it more modular15:33
*** beekneemech is now known as bnemec15:34
*** mrodden has joined #openstack-qa15:37
dkranzmkoderer: ok, great. I'll take a look later.15:41
*** salv-orlando has joined #openstack-qa16:00
*** boris-42_ has quit IRC16:01
*** amotoki has quit IRC16:03
openstackgerritJulien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH  https://review.openstack.org/7406716:03
*** protux has quit IRC16:13
*** fbo is now known as fbo_away16:15
openstackgerritA change was merged to openstack-dev/devstack: Add cliff, pycadf, stevedore, & taskflow from oslo  https://review.openstack.org/7243716:17
*** jgrimm has joined #openstack-qa16:20
openstackgerritYair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops  https://review.openstack.org/6936116:21
openstackgerritYair Fried proposed a change to openstack/tempest: Splits network_basic_ops to fully isolated test cases  https://review.openstack.org/6687916:21
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object  https://review.openstack.org/7370416:23
*** fbo_away is now known as fbo16:24
HenryGmtreinish: Kyle posted a review for new tests in the ml2 plugin, so I took the opportunity to comment ... https://review.openstack.org/#/c/74079/1/neutron/tests/unit/ml2/test_ml2_plugin.py16:27
*** yfried has quit IRC16:33
openstackgerritMalini Kamalambal proposed a change to openstack-dev/devstack: Disable journaling for mongodb  https://review.openstack.org/7408816:37
*** andreaf has quit IRC16:40
openstackgerritValeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part2)  https://review.openstack.org/7409016:40
*** jhenner has quit IRC16:52
*** vkozhukalov has quit IRC17:00
*** jlibosva has quit IRC17:06
*** jaypipes has quit IRC17:07
*** jvrbanac has quit IRC17:07
*** jvrbanac has joined #openstack-qa17:08
*** tosky has quit IRC17:13
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Fix shocco errors  https://review.openstack.org/7410417:19
openstackgerritDaniel Salinas proposed a change to openstack-dev/devstack: Fixed missing backslash in lib/trove This is breaking the installation of trove with devstack  https://review.openstack.org/7389017:19
*** dkranz has quit IRC17:21
*** dkranz has joined #openstack-qa17:22
*** ianw has quit IRC17:23
*** ianw has joined #openstack-qa17:24
*** jordanP has quit IRC17:39
openstackgerritDean Troyer proposed a change to openstack-dev/grenade: Sync split functions files with DevStack  https://review.openstack.org/7054217:39
*** boris-42_ has joined #openstack-qa17:40
*** marun has joined #openstack-qa17:41
*** rossella-s has quit IRC17:46
openstackgerritValeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part2)  https://review.openstack.org/7409017:48
*** jaypipes has joined #openstack-qa17:53
*** ChanServ sets mode: +o jaypipes17:53
openstackgerritValeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part2)  https://review.openstack.org/7409017:55
*** gszasz has quit IRC17:56
*** mlavalle has joined #openstack-qa17:59
openstackrecheckopenstack-dev/pbr change: https://review.openstack.org/73946 failed check-tempest-dsvm-full because of: https://bugs.launchpad.net/bugs/125489018:03
*** fbo is now known as fbo_away18:03
openstackgerritMate Lakat proposed a change to openstack-infra/devstack-gate: Support XenServer in the gate  https://review.openstack.org/7411218:04
openstackgerritMate Lakat proposed a change to openstack-infra/devstack-gate: Support XenServer in the gate  https://review.openstack.org/7411218:12
openstackgerritMate Lakat proposed a change to openstack-infra/devstack-gate: Fix kill parameter  https://review.openstack.org/7411318:20
openstackrecheckConsole logs not available after 13:29s for check-tempest-dsvm-full 71295,10,5141e9c18:30
*** markmcclain has joined #openstack-qa18:33
*** markmcclain has quit IRC18:33
openstackgerritizikpenso proposed a change to openstack/tempest: Fix test by waiting to lbaas entity delete  https://review.openstack.org/7412118:39
*** markmcclain has joined #openstack-qa18:40
*** morganfainberg_Z is now known as morganfainberg18:42
*** vkozhukalov has joined #openstack-qa18:44
openstackrecheckopenstack/tempest change: https://review.openstack.org/66143 failed gate-tempest-dsvm-postgres-full because of: https://bugs.launchpad.net/bugs/1275500and https://bugs.launchpad.net/bugs/1254890and https://bugs.launchpad.net/bugs/127525618:45
*** mlavalle has quit IRC18:53
*** mlavalle has joined #openstack-qa18:53
*** openstackgerrit has quit IRC18:53
*** openstackgerrit has joined #openstack-qa18:54
*** giulivo has quit IRC18:57
openstackgerritDean Troyer proposed a change to openstack-dev/devstack: Move setup_develop() to common  https://review.openstack.org/7412419:00
openstackgerritDean Troyer proposed a change to openstack-dev/grenade: Sync split functions files with DevStack  https://review.openstack.org/7054219:02
*** otherwiseguy has quit IRC19:03
*** bdperkin has joined #openstack-qa19:09
mtreinishsalv-orlando: https://review.openstack.org/74131 sorry it took me so long, I forgot about it19:18
*** mrodden has quit IRC19:20
*** mrodden has joined #openstack-qa19:24
mtreinishHenryG: sorry I had some things come up and then I had to step away for a while19:24
mtreinishsomething weird is going on in that test file I need to dig into it some more to figure out what19:24
*** weshay has quit IRC19:36
*** yfried has joined #openstack-qa19:40
openstackgerritJorge Chai proposed a change to openstack/tempest: Add basic Delete Queue Marconi test  https://review.openstack.org/7413719:42
*** otherwiseguy has joined #openstack-qa19:44
salv-orlandomtreinish: thanks np19:53
HenryGmtreinish: np, any help is appreciated. mestery is probably interested also interested in hearing your findings20:09
* mestery listens in20:10
*** weshay has joined #openstack-qa20:13
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests  https://review.openstack.org/6815120:21
*** malini is now known as malini_afk20:27
openstackgerritYair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops  https://review.openstack.org/6936120:31
otherwiseguySo, on https://review.openstack.org/#/c/72576/ which just adds an empty file in neutron as a test of stable/havana, the check still fails. In one log (http://logs.openstack.org/76/72576/1/check/check-grenade-dsvm-neutron/bf9977e/console.html), I see a swift error cannot import name HTTPException that looks like the error here: https://review.openstack.org/#/c/48129/ which appears to be fixed in the stable/havana branch20:32
otherwiseguy. Is that test just using an old swift version or something?20:32
*** jroovers|afk has joined #openstack-qa20:33
*** jroovers has quit IRC20:35
mtreinishotherwiseguy: yeah but is it in grizzly?20:38
otherwiseguypretty sure it isn't.20:38
* otherwiseguy double checks20:39
mtreinishotherwiseguy: yeah it doesn't look like it is20:39
mtreinishI didn't see a backport and it was one of the last patches for havana20:39
mtreinishotherwiseguy: that's probably your problem there, because grenade on a havana branch will bring up a grizzly cloud run the devstack exercises and then upgrade20:39
otherwiseguyaha20:39
mtreinishbut you hit the failure on the exercises20:40
otherwiseguySo should I just submit a backport for grizzly too to fix it?20:40
mtreinishotherwiseguy: yeah that should hopefully fix that swift failure20:41
otherwiseguythen on to the next failure. :p20:41
*** jlibosva has joined #openstack-qa20:42
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests  https://review.openstack.org/6815120:49
SergeyLukjanovmtreinish, thx for reviewing savanna cli tests, are you ok the rest patch content?20:50
otherwiseguymtreinish: thanks. fixed in https://review.openstack.org/#/c/74150/ (and 74151 to make .gitreview have the right defaultbranch for grizzly)20:51
mtreinishSergeyLukjanov: yeah they looked fine to me20:52
mtreinishcli tests are normally pretty simple20:52
mtreinishoh actually found another nit :) tagging the negative test negative...20:52
SergeyLukjanovmtreinish, thx, just like to clarify that I'm in the right direction :)20:53
SergeyLukjanovmtreinish, I don't saw it other cli tests :(20:53
SergeyLukjanovform where I've copy-pasted some stuff ;)20:53
openstackgerritDean Troyer proposed a change to openstack-dev/grenade: Use DevStack functions in upgrade scripts  https://review.openstack.org/7054120:53
mtreinishSergeyLukjanov: oh, well it can't hurt either way. If it's a negative test mine as well tag it as such. I'm not sure why the other ones don't do that20:56
*** fbo_away is now known as fbo20:56
SergeyLukjanovmtreinish, just @test.attr(type='negative') ?20:56
mtreinishyeah that looks right20:57
SergeyLukjanovmtreinish, thx20:58
mtreinishotherwiseguy: actually looking again at the grenade failure the exception is coming from swiftclient not swift20:59
mtreinishso I don't think that backport will fix it20:59
otherwiseguymtreinish: The failure is coming from importing from swiftclient instead of httplib, isn't it?20:59
* otherwiseguy looks again20:59
mtreinishyeah but inside swiftclient http://logs.openstack.org/76/72576/1/check/check-grenade-dsvm-neutron/bf9977e/console.html#_2014-02-17_18_36_40_28921:00
mtreinishit runs a swiftclient command and raises an exception doing that import21:00
otherwiseguyah, yeah. /usr/local/bin/swift...21:00
*** amotoki has joined #openstack-qa21:00
otherwiseguyalthough it seems like I ran the test and it failed, then made the change and it didn't.21:01
* otherwiseguy tries again21:01
*** protux has joined #openstack-qa21:02
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests  https://review.openstack.org/6815121:02
*** bill_az_ has quit IRC21:02
otherwiseguymtreinish: tox -e py27 test.functionalnosetests.swift_testing fails in stable/grizzly, passes with the patch21:03
*** nati_ueno has joined #openstack-qa21:03
otherwiseguynot sure about grenade test specifically, but does at least fix it in the branch.21:04
*** nati_ueno has quit IRC21:06
*** nati_ueno has joined #openstack-qa21:06
otherwiseguyand HTTPException doesn't appear anywhere in swiftclient21:06
mtreinishotherwiseguy: the grenade failure is on running the devstack client-args exercise21:06
mtreinishotherwiseguy: http://git.openstack.org/cgit/openstack-dev/devstack/tree/exercises/client-args.sh?h=stable/grizzly#n13621:07
*** pafuent has left #openstack-qa21:09
otherwiseguyso basically we are using a swiftclient version that is not compatible with the version of swift we are testing.21:10
*** ianw has quit IRC21:14
*** ianw has joined #openstack-qa21:15
*** afazekas has joined #openstack-qa21:21
mtreinishotherwiseguy: no I don't think that's the issue, because it's failing on an import in the bin script21:24
mtreinishso it's not actually getting to the point were it will do anything21:24
*** salv-orlando has quit IRC21:25
*** salv-orlando has joined #openstack-qa21:25
openstackgerritJorge Chai proposed a change to openstack/tempest: Add basic Delete Queue Marconi test  https://review.openstack.org/7413721:25
*** shwetaap has quit IRC21:26
otherwiseguymtreinish: but doesn't devstack always use swiftclient master branch? And swiftclient master doesn't ever reference HTTPException. And the grizzly swift does try to import HTTPException from swiftclient.21:28
* otherwiseguy is trying to get his bearings21:29
mtreinishotherwiseguy: yeah but we don't ever run those swift tests that call swiftclient during a grenade run21:30
mtreinishotherwiseguy: the tests in a grenade run are the devstack exercises and tempest21:31
mtreinishthe issue here is that for whatever the master python-swiftclient had a reference to HTTPexception but that was pulled out of swiftclient with: https://review.openstack.org/#/c/69187/21:32
otherwiseguyI can see in version 1.9.0 of swiftclient the line 35 from swiftclient import Connection, HTTPException in bin/swift. Just not sure how we're getting a version that has that line in it.21:32
mtreinishotherwiseguy: it could be a python install issue or a repo sync issue21:32
mtreinishthe last one is unlikely though21:33
otherwiseguyI'll go ahead and abandon the backport. Not exactly where to go from there, though.21:34
otherwiseguylol, got the same error in the jenkins run with the backport. :p so, yeah... :)21:35
mtreinishotherwiseguy: I think I might have found it: http://git.openstack.org/cgit/openstack/requirements/tree/tools/pip-requires?h=stable/grizzly#n4821:40
*** dims has joined #openstack-qa21:41
otherwiseguymtreinish: aha.21:41
mtreinishI bet that's conflicting with the install from master21:42
mtreinishmordred: ^^^21:42
mtreinishas one of the few who understands requirements and python installing well :021:43
mordredaroo?21:43
*** cnesa has quit IRC21:43
mordredwhat's up?21:43
mtreinishthe stable havana grenade jobs are failing when using swiftclient21:43
mtreinishthere is an import for an exception class that doesn't exist in the master branch anymore21:44
mtreinishwhich is what is installed during grenade21:44
*** coolsvap has quit IRC21:44
mtreinishbut the g-r file has a cap on the version which master is greater than21:44
mtreinishso I'm wondering can that cause some kind of weirdness where the script in usr/local/bin doesn't match what's expected from the repo we clone21:45
mtreinishmordred: the fail logs: http://logs.openstack.org/76/72576/1/check/check-grenade-dsvm-neutron/bf9977e/console.html#_2014-02-17_18_36_40_28921:46
*** ociuhandu has quit IRC21:47
*** nati_ueno has quit IRC21:49
*** nati_ueno has joined #openstack-qa21:49
*** cnesa has joined #openstack-qa21:50
mtreinishwell I guess I'll just propose the uncap and see if that fixes it21:51
mtreinishinstead of just guessing21:51
*** vkozhukalov has quit IRC21:53
mtreinishotherwiseguy: https://review.openstack.org/#/c/74176/21:53
otherwiseguymtreinish: wouldn't uncapping it potentially break grizzly as swift in grizzly will import try to import HTTPException from swiftclient?21:53
mtreinishotherwiseguy: yeah I guess but only for the swift tests that use swiftclient (which would only be on proposed swift backports)21:54
mtreinishI don't think swift has a runtime dep on swiftclient21:54
mtreinishotherwiseguy: we can cross that bridge when/if someone tries to backport a swift change to stable/grizzly21:55
*** shwetaap has joined #openstack-qa21:56
*** mrodden has quit IRC21:59
*** mrodden has joined #openstack-qa22:00
*** nati_ueno has quit IRC22:04
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests  https://review.openstack.org/6815122:06
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object  https://review.openstack.org/7370422:07
*** rfolco has quit IRC22:11
*** shwetaap has quit IRC22:12
*** fbo is now known as fbo_away22:13
otherwiseguymtreinish: looks like the test is failing with the change, same error. https://jenkins03.openstack.org/job/check-devstack-dsvm-cells/1005/console22:13
mtreinishotherwiseguy: oh, I wonder if grizzly is before we forced g-r in the gate22:15
*** shwetaap has joined #openstack-qa22:16
*** markmcclain has quit IRC22:16
mtreinishbecause the underlying cause seems to be the same: http://logs.openstack.org/80/74180/1/check/check-devstack-dsvm-cells/a653eed/console.html#_2014-02-17_22_01_42_85122:16
*** jlibosva has quit IRC22:19
mordredmtreinish: hey - sorry, I stepped out for a second - looking22:20
mordredmtreinish: yes. that can cause that sort of weirdness22:21
*** banix has quit IRC22:28
*** protux has quit IRC22:33
*** protux has joined #openstack-qa22:34
*** shwetaap has quit IRC22:38
*** mriedem has quit IRC22:38
*** dkranz has quit IRC22:39
*** shwetaap has joined #openstack-qa22:40
*** jamielennox|away is now known as jamielennox22:45
*** amotoki has quit IRC23:12
openstackgerritYair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops  https://review.openstack.org/6936123:14
*** bill_az_ has joined #openstack-qa23:15
*** protux has quit IRC23:21
*** otherwiseguy has quit IRC23:22
*** mrodden has quit IRC23:22
*** jgrimm has quit IRC23:26
*** yassine has quit IRC23:28
*** mrodden has joined #openstack-qa23:40
*** mwagner_lap has quit IRC23:51
*** mlavalle has quit IRC23:55

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