*** bhuvan_ has quit IRC | 00:00 | |
*** bill_az_ has quit IRC | 00:03 | |
*** bhuvan_ has joined #openstack-qa | 00:08 | |
*** bhuvan has quit IRC | 00:08 | |
*** banix has joined #openstack-qa | 00:20 | |
*** matsuhashi has joined #openstack-qa | 00:23 | |
*** KurtMartin has joined #openstack-qa | 00:26 | |
cyeoh | if 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 IRC | 00:33 | |
*** matsuhashi has joined #openstack-qa | 00:34 | |
openstackgerrit | minsel proposed a change to openstack/tempest: Neutron LBaaS Agent Scheduler API test https://review.openstack.org/65120 | 00:34 |
*** Sukhdev has quit IRC | 00:35 | |
*** matsuhashi has quit IRC | 00:38 | |
*** cnesa has joined #openstack-qa | 00:39 | |
*** bhuvan_ has quit IRC | 00:55 | |
openstackgerrit | Daniel Salinas proposed a change to openstack-dev/devstack: Fixed missing backslash which was breaking the installation of trove with devstack https://review.openstack.org/73890 | 01:01 |
*** weshay has quit IRC | 01:03 | |
*** matsuhashi has joined #openstack-qa | 01:04 | |
*** nosnos has joined #openstack-qa | 01:05 | |
*** dmorita has joined #openstack-qa | 01:11 | |
openstackgerrit | minsel proposed a change to openstack/tempest: Neutron Allowed Address Pair API test https://review.openstack.org/66259 | 01:22 |
*** morganfainberg_Z is now known as morganfainberg | 01:28 | |
openstackgerrit | minsel proposed a change to openstack/tempest: Neutron Extra DHCP Options API test https://review.openstack.org/67741 | 01:32 |
*** yamahata__ has quit IRC | 01:49 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Use database connection for keystone https://review.openstack.org/73463 | 01:52 |
*** yamahata_ has joined #openstack-qa | 01:54 | |
*** matsuhashi has quit IRC | 02:18 | |
*** matsuhashi has joined #openstack-qa | 02:26 | |
openstackgerrit | Kyle Mestery proposed a change to openstack-dev/devstack: Add support for the OpenDaylight Neutron MechanismDriver https://review.openstack.org/69774 | 02:41 |
*** KurtMartin has quit IRC | 02:54 | |
*** ianw has quit IRC | 02:55 | |
*** KurtMartin has joined #openstack-qa | 02:55 | |
*** mriedem has quit IRC | 02:56 | |
*** malini is now known as malini_afk | 02:57 | |
*** ianw has joined #openstack-qa | 03:02 | |
openstackgerrit | Yongli He proposed a change to openstack/tempest: Use six.StringIO/BytesIO instead of StringIO.StringIO https://review.openstack.org/73903 | 03:10 |
*** samalba has quit IRC | 03:11 | |
*** samalba has joined #openstack-qa | 03:11 | |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add cold-migration on server test https://review.openstack.org/68033 | 03:11 |
*** otherwiseguy has quit IRC | 03:17 | |
*** ivanzhu has joined #openstack-qa | 03:20 | |
*** ivanzhu has quit IRC | 03:22 | |
*** ivanzhu has joined #openstack-qa | 03:22 | |
*** ivanzhu has quit IRC | 03:25 | |
*** ivanzhu has joined #openstack-qa | 03:26 | |
*** ivanzhu has quit IRC | 03:28 | |
*** otherwiseguy has joined #openstack-qa | 03:29 | |
*** ivanzhu1 has joined #openstack-qa | 03:30 | |
*** otherwiseguy has quit IRC | 03:30 | |
mikal | Anyone here understand testing live migration in temptest? | 03:34 |
mikal | tempest even | 03:34 |
mikal | The tests read to me like I need two compute hosts? Which I guess mean these tests never run in the gate? | 03:34 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add extra list servers by host test https://review.openstack.org/65234 | 03:35 |
clarkb | mikal correct | 03:47 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add cold-migration on server test https://review.openstack.org/68033 | 03:47 |
*** matsuhashi has quit IRC | 03:48 | |
*** ken1ohmichi has joined #openstack-qa | 03:58 | |
* mikal goes off to learn how multi-node devstack works | 03:59 | |
*** banix has quit IRC | 04:09 | |
*** morganfainberg is now known as morganfainberg_Z | 04:13 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Separate negative tests for test_projects https://review.openstack.org/72912 | 04:25 |
*** matsuhashi has joined #openstack-qa | 04:32 | |
*** amotoki has joined #openstack-qa | 04:39 | |
openstackgerrit | A change was merged to openstack/tempest: Adds test_show_XX_fields cases to show some fields https://review.openstack.org/65622 | 04:43 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add cold-migration on server test https://review.openstack.org/68033 | 04:44 |
*** yfried has quit IRC | 04:55 | |
openstackgerrit | Rushi Agrawal proposed a change to openstack/tempest: Add tests for filtering volumes https://review.openstack.org/63388 | 05:06 |
*** ivanzhu1 has quit IRC | 05:20 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 05:26 |
*** ken1ohmichi has quit IRC | 05:30 | |
*** ivanzhu has joined #openstack-qa | 05:36 | |
*** ken1ohmichi has joined #openstack-qa | 05:38 | |
openstackgerrit | Ivan-Zhu proposed a change to openstack/tempest: port nova v2 images related tests into nova v3 part2 https://review.openstack.org/64361 | 05:38 |
*** yfried has joined #openstack-qa | 05:40 | |
*** Madan1 has joined #openstack-qa | 05:54 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 06:01 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/compute) https://review.openstack.org/65217 | 06:08 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/{identity,orchestration,volume,utils}) https://review.openstack.org/73921 | 06:09 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/object_storage) https://review.openstack.org/73923 | 06:12 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (common,services,stress) https://review.openstack.org/73924 | 06:12 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (scenario) https://review.openstack.org/73925 | 06:13 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (thirdparty) https://review.openstack.org/73926 | 06:13 |
*** saju_m has joined #openstack-qa | 06:15 | |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds L3 agent test case to test_l3_agent_scheduler.py https://review.openstack.org/68626 | 06:18 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add cold-migration on server test https://review.openstack.org/68033 | 06:36 |
openstackgerrit | A change was merged to openstack/tempest: Fix volume transfer tests with tenant isolation disabled https://review.openstack.org/72034 | 06:38 |
*** jaypipes has quit IRC | 06:42 | |
*** jaypipes has joined #openstack-qa | 06:42 | |
*** ChanServ sets mode: +o jaypipes | 06:42 | |
*** afazekas_ has quit IRC | 06:42 | |
*** nosnos has quit IRC | 06:42 | |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add extra list servers by host test https://review.openstack.org/65234 | 06:46 |
openstackgerrit | Isaku Yamahata proposed a change to openstack-dev/devstack: neutron/service-agent: support consolidated service agent https://review.openstack.org/72952 | 06:46 |
*** nosnos has joined #openstack-qa | 06:52 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 07:03 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 07:06 |
*** nmagnezi has joined #openstack-qa | 07:09 | |
*** KurtMartin has quit IRC | 07:16 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 07:25 |
*** afazekas_ has joined #openstack-qa | 07:27 | |
openstackgerrit | shihanzhang proposed a change to openstack/tempest: Remove unused method _cleanup_ports https://review.openstack.org/73954 | 07:36 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 07:37 |
*** boris-42 has quit IRC | 07:40 | |
*** saurabh_ has quit IRC | 07:44 | |
*** ivanzhu has quit IRC | 07:51 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add a version API test for Nova v2 API https://review.openstack.org/73912 | 07:53 |
openstackgerrit | ChenZheng proposed a change to openstack/tempest: Add cold-migration on server test https://review.openstack.org/68033 | 07:54 |
*** saurabh_ has joined #openstack-qa | 08:00 | |
*** saurabh_ has quit IRC | 08:00 | |
*** saurabh_ has joined #openstack-qa | 08:00 | |
*** Anju has quit IRC | 08:01 | |
openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part1) https://review.openstack.org/73869 | 08:03 |
*** jhenner has quit IRC | 08:06 | |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Seperate negative tests for test_server_rescue https://review.openstack.org/71457 | 08:09 |
*** jlibosva has joined #openstack-qa | 08:10 | |
*** ivanzhu has joined #openstack-qa | 08:12 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Negative does need to be separated files https://review.openstack.org/73963 | 08:18 |
openstackgerrit | Sylvain Baubeau proposed a change to openstack/tempest: Add Cinder tests for quota sets https://review.openstack.org/73606 | 08:20 |
*** saurabh_ has quit IRC | 08:20 | |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 (identity,volume,etc.) https://review.openstack.org/73921 | 08:21 |
*** jamielennox is now known as jamielennox|away | 08:22 | |
*** saurabh_ has joined #openstack-qa | 08:24 | |
*** Anju has joined #openstack-qa | 08:24 | |
yfried | afazekas_: ping | 08:26 |
afazekas_ | yfried: pong | 08:27 |
*** afazekas_ is now known as afazekas | 08:27 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Negative tests does need to be in a separated files https://review.openstack.org/73963 | 08:28 |
yfried | afazekas: 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 IRC | 08:31 | |
yfried | salv-orlando: ^ | 08:31 |
*** saju_m has quit IRC | 08:31 | |
*** jroovers has joined #openstack-qa | 08:32 | |
afazekas | bug for variable naming ??? | 08:32 |
afazekas | no | 08:33 |
*** jroovers|afk has joined #openstack-qa | 08:33 | |
*** saurabh_ has joined #openstack-qa | 08:34 | |
yfried | afazekas: Had I seen it in review I would have commented, but since I haven't - I would like to change it now | 08:34 |
*** salv-orlando_ has joined #openstack-qa | 08:35 | |
*** salv-orlando has quit IRC | 08:35 | |
*** salv-orlando_ is now known as salv-orlando | 08:35 | |
*** jroovers has quit IRC | 08:37 | |
yfried | afazekas: is there a way to filter gerrit for changes to specific files? | 08:37 |
afazekas | The naming is irrelevant :) | 08:38 |
yfried | afazekas: or directories? | 08:38 |
yfried | afazekas: it really bugs me :) it's completely counter-intuitive | 08:39 |
afazekas | Usually rename patches are not liked, because otherwise we would rename everything on every week.. | 08:40 |
afazekas | yfried: you can try to submit a patch, and we will see .. | 08:41 |
afazekas | There are always exceptions, when the rename is liked | 08:43 |
*** ivanzhu has quit IRC | 08:50 | |
*** saju_m has joined #openstack-qa | 08:58 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Minor changes to scenario manager https://review.openstack.org/73973 | 08:58 |
*** jordanP has joined #openstack-qa | 08:59 | |
yfried | afazekas: what do you think? https://review.openstack.org/#/c/73973/ | 09:00 |
*** jhenner has joined #openstack-qa | 09:00 | |
yfried | afazekas the variable is the last change on the manager | 09:00 |
*** jhenner has quit IRC | 09:03 | |
openstackgerrit | shihanzhang proposed a change to openstack/tempest: Modify error decription in cleanup_ports https://review.openstack.org/73954 | 09:03 |
*** jhenner has joined #openstack-qa | 09:04 | |
afazekas | yfried: LGTM | 09:06 |
* afazekas waiting for jenkins | 09:06 | |
yfried | afazekas: any chance it's small enough for you to approve immediately after jenkins? | 09:10 |
*** fbo_away is now known as fbo | 09:17 | |
*** yassine has joined #openstack-qa | 09:17 | |
afazekas | yfried: 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 |
jordanP | yfried, could you elaborate your commit message plz ? | 09:27 |
yfried | jordanP: I really see how, apart from copying the diff results. any suggestions? | 09:29 |
yfried | jordanP: s/realy/"really don't" | 09:29 |
jordanP | "use native neutron client to retrieve tenant's ressources instead of filtering in tempest" | 09:31 |
jordanP | or something like this | 09:31 |
openstackgerrit | A change was merged to openstack/tempest: enable volume list tests for cinder v2 - part2 https://review.openstack.org/59733 | 09:32 |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add multiple negative test generator support https://review.openstack.org/73982 | 09:35 |
*** andreaf has joined #openstack-qa | 09:47 | |
*** rossella-s has joined #openstack-qa | 09:51 | |
*** boris-42_ has joined #openstack-qa | 09:52 | |
*** matsuhashi has quit IRC | 09:54 | |
*** rossella-s has quit IRC | 09:54 | |
*** rossella-s has joined #openstack-qa | 10:00 | |
*** matsuhashi has joined #openstack-qa | 10:02 | |
*** gszasz has joined #openstack-qa | 10:03 | |
*** tkammer has joined #openstack-qa | 10:08 | |
yfried | jordanP: very well. can you please -1? | 10:09 |
jordanP | yfried, done | 10:11 |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Prepare for enabling H302 rule (api/compute) https://review.openstack.org/65217 | 10:12 |
*** matsuhashi has quit IRC | 10:17 | |
*** matsuhashi has joined #openstack-qa | 10:17 | |
*** matsuhashi has quit IRC | 10:22 | |
*** mkollaro has joined #openstack-qa | 10:28 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: WIP: Add v3 os-migration API tests https://review.openstack.org/73995 | 10:28 |
openstackrecheck | openstack/horizon change: https://review.openstack.org/73748 failed check-tempest-dsvm-postgres-full because of: https://bugs.launchpad.net/bugs/1262153 | 10:34 |
openstackgerrit | A change was merged to openstack/tempest: Remove tests for Nova V3 API os-simple-tenant-usage https://review.openstack.org/72269 | 10:38 |
openstackgerrit | A change was merged to openstack/tempest: Remove using of deprecated self.headers (part1) https://review.openstack.org/73869 | 10:38 |
openstackgerrit | A change was merged to openstack/tempest: Add test case for snapshot-create with in-use volume https://review.openstack.org/66282 | 10:40 |
openstackgerrit | A change was merged to openstack-dev/devstack: Fix hook script for Ironic https://review.openstack.org/72890 | 10:41 |
openstackgerrit | A change was merged to openstack/tempest: Remove tests for Nova V3 API os-instance-usage-audit-log https://review.openstack.org/72267 | 10:41 |
*** vkozhukalov has joined #openstack-qa | 10:46 | |
*** matsuhashi has joined #openstack-qa | 10:47 | |
*** ociuhandu has quit IRC | 10:48 | |
*** tosky has joined #openstack-qa | 10:50 | |
*** jroovers|afk is now known as jroovers | 10:58 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: WIP: Add v3 os-migration API tests https://review.openstack.org/73995 | 11:01 |
*** ociuhandu has joined #openstack-qa | 11:06 | |
*** yassine has quit IRC | 11:15 | |
*** yassine has joined #openstack-qa | 11:15 | |
*** akamyshnikova has quit IRC | 11:21 | |
*** akamyshnikova has joined #openstack-qa | 11:24 | |
*** matsuhashi has quit IRC | 11:37 | |
openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-cells-full 73924,1,7b93ca8 | 11:56 |
*** dmorita has quit IRC | 11:58 | |
*** yfried has quit IRC | 12:10 | |
openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Savanna: add changes to API client and new tests https://review.openstack.org/71592 | 12:15 |
openstackgerrit | A change was merged to openstack/tempest: Test updating port's admin_state_up https://review.openstack.org/72829 | 12:16 |
*** rfolco has joined #openstack-qa | 12:17 | |
openstackgerrit | A change was merged to openstack/tempest: Ensure that bug number is actually a number for skip_because https://review.openstack.org/72750 | 12:21 |
*** dkranz has quit IRC | 12:24 | |
*** yfried has joined #openstack-qa | 12:26 | |
*** yfried has quit IRC | 12:26 | |
*** vponomaryov has joined #openstack-qa | 12:29 | |
*** yfried1 has joined #openstack-qa | 12:30 | |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add multiple negative test generator support https://review.openstack.org/73982 | 12:34 |
*** weshay has joined #openstack-qa | 12:45 | |
*** mkollaro1 has joined #openstack-qa | 12:52 | |
*** mkollaro has quit IRC | 12:52 | |
openstackgerrit | Sergey Nikitin proposed a change to openstack/tempest: Improved tests in tempest.api.image.v2.* https://review.openstack.org/68096 | 12:55 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Minor changes to scenario manager https://review.openstack.org/73973 | 12:56 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Refactor cross_tenant to security_groups_basic_ops https://review.openstack.org/69567 | 12:56 |
*** pafuent has joined #openstack-qa | 12:57 | |
openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Add api tests for load balancer's VIPs and health monitors https://review.openstack.org/66796 | 13:00 |
openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Refactor create_ and update_ methods for VIPs and health_monitors https://review.openstack.org/66516 | 13:00 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Preventing overlapping subnets in network scenarios https://review.openstack.org/52994 | 13:01 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops https://review.openstack.org/69361 | 13:01 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Splits network_basic_ops to fully isolated test cases https://review.openstack.org/66879 | 13:01 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Preventing overlapping subnets in network scenarios https://review.openstack.org/52994 | 13:05 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops https://review.openstack.org/69361 | 13:05 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Splits network_basic_ops to fully isolated test cases https://review.openstack.org/66879 | 13:05 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Minor changes to scenario manager https://review.openstack.org/73973 | 13:05 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Refactor cross_tenant to security_groups_basic_ops https://review.openstack.org/69567 | 13:05 |
*** nosnos has quit IRC | 13:08 | |
*** yfried1 has quit IRC | 13:09 | |
*** nmagnezi has quit IRC | 13:12 | |
andreaf | maurosr: ping | 13:15 |
openstackgerrit | A change was merged to openstack/tempest: port nova v2 images related tests into nova v3 part2 https://review.openstack.org/64361 | 13:15 |
*** mkollaro has joined #openstack-qa | 13:17 | |
*** nmagnezi has joined #openstack-qa | 13:18 | |
*** ken1ohmichi has quit IRC | 13:18 | |
openstackgerrit | Giulio Fidente proposed a change to openstack/tempest: Adds cinder backup functional tests https://review.openstack.org/67616 | 13:19 |
*** mkollaro1 has quit IRC | 13:21 | |
*** malini_afk is now known as malini | 13:22 | |
*** yfried has joined #openstack-qa | 13:25 | |
*** malini is now known as malini_afk | 13:36 | |
*** salv-orlando has quit IRC | 13:43 | |
*** Madan1 has quit IRC | 13:46 | |
openstackgerrit | andrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object https://review.openstack.org/73704 | 13:49 |
mtreinish | HenryG: pong | 13:51 |
openstackgerrit | andrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object https://review.openstack.org/73704 | 13:57 |
openstackgerrit | Yaroslav Lobankov proposed a change to openstack/tempest: Savanna: add changes to API client and new tests https://review.openstack.org/71592 | 14:02 |
openstackgerrit | Gavin Brebner proposed a change to openstack-dev/devstack: Create glance-swift for swift or s-proxy https://review.openstack.org/74047 | 14:05 |
*** mkollaro has quit IRC | 14:05 | |
*** tkammer has quit IRC | 14:06 | |
HenryG | mtreinish: 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 |
HenryG | mtreinish: the project is neutron, but I hope the answer is generic | 14:14 |
*** saju_m has quit IRC | 14:15 | |
*** banix has joined #openstack-qa | 14:16 | |
mtreinish | HenryG: sure, it should be the same between projects. (although we use somethings in tempest that prevent nose from working) | 14:16 |
mtreinish | HenryG: so what's the question? | 14:18 |
HenryG | mtreinish: 1 sec, trying to get back to where I was yesterday when I hit the problem ... | 14:19 |
*** mriedem has joined #openstack-qa | 14:22 | |
openstackgerrit | A change was merged to openstack/tempest: network: add metering api operations https://review.openstack.org/66143 | 14:23 |
HenryG | mtreinish: so here is the test case: neutron.tests.unit.ml2.test_ml2_plugin.TestMl2PortBinding.test_create_port_binding_profile_with_empty_dict | 14:24 |
HenryG | mtreinish: runs fine with 'testr run ...' | 14:25 |
HenryG | mtreinish: If I activate the venv and run it with 'nosetests ...' (using colon to separate module from class) it gives a traceback. | 14:26 |
HenryG | mtreinish: 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 |
HenryG | mtreinish: if I fix that so that nosetests works, then tox (testr) fails on other test cases. | 14:28 |
HenryG | mtreinish: so it looks to me like tox/testr is initializing things very differently from nosetests. | 14:29 |
HenryG | mtreinish: I guess that is the question. What is the difference? | 14:30 |
mtreinish | HenryG: 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 somehow | 14:31 |
mtreinish | I'm not sure what is causing the initial failure though if you can pastebin the tracebacks I'd get a better idea | 14:31 |
HenryG | mtreinish: even when I specifically run just one single test case? | 14:32 |
mtreinish | well you said when you applied the fix other test cases were failing | 14:32 |
HenryG | Yes, but after the 'fix' even running one single other test case fails. | 14:33 |
HenryG | Let me start pastebinning ... | 14:34 |
HenryG | nosetests failure with no fix: http://paste.openstack.org/show/66486/ | 14:36 |
HenryG | The fix: http://paste.openstack.org/show/66487/ | 14:37 |
*** bill_az_ has joined #openstack-qa | 14:37 | |
HenryG | testr failure after the fix: http://paste.openstack.org/show/66488/ | 14:39 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-dev/devstack: Improve savanna-dashboard installation https://review.openstack.org/74054 | 14:39 |
*** afazekas has quit IRC | 14:46 | |
mtreinish | HenryG: can I see a pastebin of the success with testr before the change? | 14:47 |
mtreinish | because that's clearly a bug in setUp() there | 14:47 |
mtreinish | without the super call() the config object shouldn't be initialized | 14:47 |
mtreinish | so there isn't anything to call set default on | 14:47 |
HenryG | mtreinish: here: http://paste.openstack.org/show/66493/ | 14:49 |
*** shwetaap has quit IRC | 14:55 | |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/tempest: Expand baremetal nodes coverage https://review.openstack.org/74065 | 14:58 |
openstackgerrit | Julien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH https://review.openstack.org/74067 | 14:58 |
*** gszasz has quit IRC | 15:00 | |
*** gszasz has joined #openstack-qa | 15:01 | |
*** YorikSar has joined #openstack-qa | 15:03 | |
*** otherwiseguy has joined #openstack-qa | 15:04 | |
*** yfried has quit IRC | 15:07 | |
*** rustlebee is now known as russellb | 15:08 | |
*** rwsu has joined #openstack-qa | 15:09 | |
*** shwetaap has joined #openstack-qa | 15:11 | |
*** malini_afk is now known as malini | 15:16 | |
*** nmagnezi has quit IRC | 15:19 | |
*** yfried has joined #openstack-qa | 15:19 | |
*** yfried has quit IRC | 15:22 | |
*** giulivo has joined #openstack-qa | 15:22 | |
*** yfried has joined #openstack-qa | 15:23 | |
*** dkranz has joined #openstack-qa | 15:25 | |
*** coolsvap has joined #openstack-qa | 15:26 | |
*** protux has joined #openstack-qa | 15:27 | |
*** enikanorov has joined #openstack-qa | 15:31 | |
mkoderer | dkranz: Hi, here's a new patch https://review.openstack.org/#/c/73982/ | 15:33 |
mkoderer | dkranz: I rewrote the json generator and tried to make it more modular | 15:33 |
*** beekneemech is now known as bnemec | 15:34 | |
*** mrodden has joined #openstack-qa | 15:37 | |
dkranz | mkoderer: ok, great. I'll take a look later. | 15:41 |
*** salv-orlando has joined #openstack-qa | 16:00 | |
*** boris-42_ has quit IRC | 16:01 | |
*** amotoki has quit IRC | 16:03 | |
openstackgerrit | Julien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH https://review.openstack.org/74067 | 16:03 |
*** protux has quit IRC | 16:13 | |
*** fbo is now known as fbo_away | 16:15 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Add cliff, pycadf, stevedore, & taskflow from oslo https://review.openstack.org/72437 | 16:17 |
*** jgrimm has joined #openstack-qa | 16:20 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops https://review.openstack.org/69361 | 16:21 |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Splits network_basic_ops to fully isolated test cases https://review.openstack.org/66879 | 16:21 |
openstackgerrit | andrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object https://review.openstack.org/73704 | 16:23 |
*** fbo_away is now known as fbo | 16:24 | |
HenryG | mtreinish: 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.py | 16:27 |
*** yfried has quit IRC | 16:33 | |
openstackgerrit | Malini Kamalambal proposed a change to openstack-dev/devstack: Disable journaling for mongodb https://review.openstack.org/74088 | 16:37 |
*** andreaf has quit IRC | 16:40 | |
openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part2) https://review.openstack.org/74090 | 16:40 |
*** jhenner has quit IRC | 16:52 | |
*** vkozhukalov has quit IRC | 17:00 | |
*** jlibosva has quit IRC | 17:06 | |
*** jaypipes has quit IRC | 17:07 | |
*** jvrbanac has quit IRC | 17:07 | |
*** jvrbanac has joined #openstack-qa | 17:08 | |
*** tosky has quit IRC | 17:13 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Fix shocco errors https://review.openstack.org/74104 | 17:19 |
openstackgerrit | Daniel 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/73890 | 17:19 |
*** dkranz has quit IRC | 17:21 | |
*** dkranz has joined #openstack-qa | 17:22 | |
*** ianw has quit IRC | 17:23 | |
*** ianw has joined #openstack-qa | 17:24 | |
*** jordanP has quit IRC | 17:39 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/grenade: Sync split functions files with DevStack https://review.openstack.org/70542 | 17:39 |
*** boris-42_ has joined #openstack-qa | 17:40 | |
*** marun has joined #openstack-qa | 17:41 | |
*** rossella-s has quit IRC | 17:46 | |
openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part2) https://review.openstack.org/74090 | 17:48 |
*** jaypipes has joined #openstack-qa | 17:53 | |
*** ChanServ sets mode: +o jaypipes | 17:53 | |
openstackgerrit | Valeriy Ponomaryov proposed a change to openstack/tempest: Remove using of deprecated self.headers (part2) https://review.openstack.org/74090 | 17:55 |
*** gszasz has quit IRC | 17:56 | |
*** mlavalle has joined #openstack-qa | 17:59 | |
openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/73946 failed check-tempest-dsvm-full because of: https://bugs.launchpad.net/bugs/1254890 | 18:03 |
*** fbo is now known as fbo_away | 18:03 | |
openstackgerrit | Mate Lakat proposed a change to openstack-infra/devstack-gate: Support XenServer in the gate https://review.openstack.org/74112 | 18:04 |
openstackgerrit | Mate Lakat proposed a change to openstack-infra/devstack-gate: Support XenServer in the gate https://review.openstack.org/74112 | 18:12 |
openstackgerrit | Mate Lakat proposed a change to openstack-infra/devstack-gate: Fix kill parameter https://review.openstack.org/74113 | 18:20 |
openstackrecheck | Console logs not available after 13:29s for check-tempest-dsvm-full 71295,10,5141e9c | 18:30 |
*** markmcclain has joined #openstack-qa | 18:33 | |
*** markmcclain has quit IRC | 18:33 | |
openstackgerrit | izikpenso proposed a change to openstack/tempest: Fix test by waiting to lbaas entity delete https://review.openstack.org/74121 | 18:39 |
*** markmcclain has joined #openstack-qa | 18:40 | |
*** morganfainberg_Z is now known as morganfainberg | 18:42 | |
*** vkozhukalov has joined #openstack-qa | 18:44 | |
openstackrecheck | openstack/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/1275256 | 18:45 |
*** mlavalle has quit IRC | 18:53 | |
*** mlavalle has joined #openstack-qa | 18:53 | |
*** openstackgerrit has quit IRC | 18:53 | |
*** openstackgerrit has joined #openstack-qa | 18:54 | |
*** giulivo has quit IRC | 18:57 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Move setup_develop() to common https://review.openstack.org/74124 | 19:00 |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/grenade: Sync split functions files with DevStack https://review.openstack.org/70542 | 19:02 |
*** otherwiseguy has quit IRC | 19:03 | |
*** bdperkin has joined #openstack-qa | 19:09 | |
mtreinish | salv-orlando: https://review.openstack.org/74131 sorry it took me so long, I forgot about it | 19:18 |
*** mrodden has quit IRC | 19:20 | |
*** mrodden has joined #openstack-qa | 19:24 | |
mtreinish | HenryG: sorry I had some things come up and then I had to step away for a while | 19:24 |
mtreinish | something weird is going on in that test file I need to dig into it some more to figure out what | 19:24 |
*** weshay has quit IRC | 19:36 | |
*** yfried has joined #openstack-qa | 19:40 | |
openstackgerrit | Jorge Chai proposed a change to openstack/tempest: Add basic Delete Queue Marconi test https://review.openstack.org/74137 | 19:42 |
*** otherwiseguy has joined #openstack-qa | 19:44 | |
salv-orlando | mtreinish: thanks np | 19:53 |
HenryG | mtreinish: np, any help is appreciated. mestery is probably interested also interested in hearing your findings | 20:09 |
* mestery listens in | 20:10 | |
*** weshay has joined #openstack-qa | 20:13 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests https://review.openstack.org/68151 | 20:21 |
*** malini is now known as malini_afk | 20:27 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops https://review.openstack.org/69361 | 20:31 |
otherwiseguy | So, 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 branch | 20:32 |
otherwiseguy | . Is that test just using an old swift version or something? | 20:32 |
*** jroovers|afk has joined #openstack-qa | 20:33 | |
*** jroovers has quit IRC | 20:35 | |
mtreinish | otherwiseguy: yeah but is it in grizzly? | 20:38 |
otherwiseguy | pretty sure it isn't. | 20:38 |
* otherwiseguy double checks | 20:39 | |
mtreinish | otherwiseguy: yeah it doesn't look like it is | 20:39 |
mtreinish | I didn't see a backport and it was one of the last patches for havana | 20:39 |
mtreinish | otherwiseguy: that's probably your problem there, because grenade on a havana branch will bring up a grizzly cloud run the devstack exercises and then upgrade | 20:39 |
otherwiseguy | aha | 20:39 |
mtreinish | but you hit the failure on the exercises | 20:40 |
otherwiseguy | So should I just submit a backport for grizzly too to fix it? | 20:40 |
mtreinish | otherwiseguy: yeah that should hopefully fix that swift failure | 20:41 |
otherwiseguy | then on to the next failure. :p | 20:41 |
*** jlibosva has joined #openstack-qa | 20:42 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests https://review.openstack.org/68151 | 20:49 |
SergeyLukjanov | mtreinish, thx for reviewing savanna cli tests, are you ok the rest patch content? | 20:50 |
otherwiseguy | mtreinish: thanks. fixed in https://review.openstack.org/#/c/74150/ (and 74151 to make .gitreview have the right defaultbranch for grizzly) | 20:51 |
mtreinish | SergeyLukjanov: yeah they looked fine to me | 20:52 |
mtreinish | cli tests are normally pretty simple | 20:52 |
mtreinish | oh actually found another nit :) tagging the negative test negative... | 20:52 |
SergeyLukjanov | mtreinish, thx, just like to clarify that I'm in the right direction :) | 20:53 |
SergeyLukjanov | mtreinish, I don't saw it other cli tests :( | 20:53 |
SergeyLukjanov | form where I've copy-pasted some stuff ;) | 20:53 |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/grenade: Use DevStack functions in upgrade scripts https://review.openstack.org/70541 | 20:53 |
mtreinish | SergeyLukjanov: 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 that | 20:56 |
*** fbo_away is now known as fbo | 20:56 | |
SergeyLukjanov | mtreinish, just @test.attr(type='negative') ? | 20:56 |
mtreinish | yeah that looks right | 20:57 |
SergeyLukjanov | mtreinish, thx | 20:58 |
mtreinish | otherwiseguy: actually looking again at the grenade failure the exception is coming from swiftclient not swift | 20:59 |
mtreinish | so I don't think that backport will fix it | 20:59 |
otherwiseguy | mtreinish: The failure is coming from importing from swiftclient instead of httplib, isn't it? | 20:59 |
* otherwiseguy looks again | 20:59 | |
mtreinish | yeah but inside swiftclient http://logs.openstack.org/76/72576/1/check/check-grenade-dsvm-neutron/bf9977e/console.html#_2014-02-17_18_36_40_289 | 21:00 |
mtreinish | it runs a swiftclient command and raises an exception doing that import | 21:00 |
otherwiseguy | ah, yeah. /usr/local/bin/swift... | 21:00 |
*** amotoki has joined #openstack-qa | 21:00 | |
otherwiseguy | although it seems like I ran the test and it failed, then made the change and it didn't. | 21:01 |
* otherwiseguy tries again | 21:01 | |
*** protux has joined #openstack-qa | 21:02 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests https://review.openstack.org/68151 | 21:02 |
*** bill_az_ has quit IRC | 21:02 | |
otherwiseguy | mtreinish: tox -e py27 test.functionalnosetests.swift_testing fails in stable/grizzly, passes with the patch | 21:03 |
*** nati_ueno has joined #openstack-qa | 21:03 | |
otherwiseguy | not sure about grenade test specifically, but does at least fix it in the branch. | 21:04 |
*** nati_ueno has quit IRC | 21:06 | |
*** nati_ueno has joined #openstack-qa | 21:06 | |
otherwiseguy | and HTTPException doesn't appear anywhere in swiftclient | 21:06 |
mtreinish | otherwiseguy: the grenade failure is on running the devstack client-args exercise | 21:06 |
mtreinish | otherwiseguy: http://git.openstack.org/cgit/openstack-dev/devstack/tree/exercises/client-args.sh?h=stable/grizzly#n136 | 21:07 |
*** pafuent has left #openstack-qa | 21:09 | |
otherwiseguy | so basically we are using a swiftclient version that is not compatible with the version of swift we are testing. | 21:10 |
*** ianw has quit IRC | 21:14 | |
*** ianw has joined #openstack-qa | 21:15 | |
*** afazekas has joined #openstack-qa | 21:21 | |
mtreinish | otherwiseguy: no I don't think that's the issue, because it's failing on an import in the bin script | 21:24 |
mtreinish | so it's not actually getting to the point were it will do anything | 21:24 |
*** salv-orlando has quit IRC | 21:25 | |
*** salv-orlando has joined #openstack-qa | 21:25 | |
openstackgerrit | Jorge Chai proposed a change to openstack/tempest: Add basic Delete Queue Marconi test https://review.openstack.org/74137 | 21:25 |
*** shwetaap has quit IRC | 21:26 | |
otherwiseguy | mtreinish: 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 bearings | 21:29 | |
mtreinish | otherwiseguy: yeah but we don't ever run those swift tests that call swiftclient during a grenade run | 21:30 |
mtreinish | otherwiseguy: the tests in a grenade run are the devstack exercises and tempest | 21:31 |
mtreinish | the 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 |
otherwiseguy | I 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 |
mtreinish | otherwiseguy: it could be a python install issue or a repo sync issue | 21:32 |
mtreinish | the last one is unlikely though | 21:33 |
otherwiseguy | I'll go ahead and abandon the backport. Not exactly where to go from there, though. | 21:34 |
otherwiseguy | lol, got the same error in the jenkins run with the backport. :p so, yeah... :) | 21:35 |
mtreinish | otherwiseguy: I think I might have found it: http://git.openstack.org/cgit/openstack/requirements/tree/tools/pip-requires?h=stable/grizzly#n48 | 21:40 |
*** dims has joined #openstack-qa | 21:41 | |
otherwiseguy | mtreinish: aha. | 21:41 |
mtreinish | I bet that's conflicting with the install from master | 21:42 |
mtreinish | mordred: ^^^ | 21:42 |
mtreinish | as one of the few who understands requirements and python installing well :0 | 21:43 |
mordred | aroo? | 21:43 |
*** cnesa has quit IRC | 21:43 | |
mordred | what's up? | 21:43 |
mtreinish | the stable havana grenade jobs are failing when using swiftclient | 21:43 |
mtreinish | there is an import for an exception class that doesn't exist in the master branch anymore | 21:44 |
mtreinish | which is what is installed during grenade | 21:44 |
*** coolsvap has quit IRC | 21:44 | |
mtreinish | but the g-r file has a cap on the version which master is greater than | 21:44 |
mtreinish | so 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 clone | 21:45 |
mtreinish | mordred: the fail logs: http://logs.openstack.org/76/72576/1/check/check-grenade-dsvm-neutron/bf9977e/console.html#_2014-02-17_18_36_40_289 | 21:46 |
*** ociuhandu has quit IRC | 21:47 | |
*** nati_ueno has quit IRC | 21:49 | |
*** nati_ueno has joined #openstack-qa | 21:49 | |
*** cnesa has joined #openstack-qa | 21:50 | |
mtreinish | well I guess I'll just propose the uncap and see if that fixes it | 21:51 |
mtreinish | instead of just guessing | 21:51 |
*** vkozhukalov has quit IRC | 21:53 | |
mtreinish | otherwiseguy: https://review.openstack.org/#/c/74176/ | 21:53 |
otherwiseguy | mtreinish: wouldn't uncapping it potentially break grizzly as swift in grizzly will import try to import HTTPException from swiftclient? | 21:53 |
mtreinish | otherwiseguy: yeah I guess but only for the swift tests that use swiftclient (which would only be on proposed swift backports) | 21:54 |
mtreinish | I don't think swift has a runtime dep on swiftclient | 21:54 |
mtreinish | otherwiseguy: we can cross that bridge when/if someone tries to backport a swift change to stable/grizzly | 21:55 |
*** shwetaap has joined #openstack-qa | 21:56 | |
*** mrodden has quit IRC | 21:59 | |
*** mrodden has joined #openstack-qa | 22:00 | |
*** nati_ueno has quit IRC | 22:04 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add basic Savanna CLI tests https://review.openstack.org/68151 | 22:06 |
openstackgerrit | andrea-frittoli proposed a change to openstack/tempest: Refactor credentials into an object https://review.openstack.org/73704 | 22:07 |
*** rfolco has quit IRC | 22:11 | |
*** shwetaap has quit IRC | 22:12 | |
*** fbo is now known as fbo_away | 22:13 | |
otherwiseguy | mtreinish: looks like the test is failing with the change, same error. https://jenkins03.openstack.org/job/check-devstack-dsvm-cells/1005/console | 22:13 |
mtreinish | otherwiseguy: oh, I wonder if grizzly is before we forced g-r in the gate | 22:15 |
*** shwetaap has joined #openstack-qa | 22:16 | |
*** markmcclain has quit IRC | 22:16 | |
mtreinish | because 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_851 | 22:16 |
*** jlibosva has quit IRC | 22:19 | |
mordred | mtreinish: hey - sorry, I stepped out for a second - looking | 22:20 |
mordred | mtreinish: yes. that can cause that sort of weirdness | 22:21 |
*** banix has quit IRC | 22:28 | |
*** protux has quit IRC | 22:33 | |
*** protux has joined #openstack-qa | 22:34 | |
*** shwetaap has quit IRC | 22:38 | |
*** mriedem has quit IRC | 22:38 | |
*** dkranz has quit IRC | 22:39 | |
*** shwetaap has joined #openstack-qa | 22:40 | |
*** jamielennox|away is now known as jamielennox | 22:45 | |
*** amotoki has quit IRC | 23:12 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds scenario for hotplug nic in network_basic_ops https://review.openstack.org/69361 | 23:14 |
*** bill_az_ has joined #openstack-qa | 23:15 | |
*** protux has quit IRC | 23:21 | |
*** otherwiseguy has quit IRC | 23:22 | |
*** mrodden has quit IRC | 23:22 | |
*** jgrimm has quit IRC | 23:26 | |
*** yassine has quit IRC | 23:28 | |
*** mrodden has joined #openstack-qa | 23:40 | |
*** mwagner_lap has quit IRC | 23:51 | |
*** mlavalle has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!