*** GMann has joined #openstack-qa | 00:06 | |
*** atiwari has quit IRC | 00:10 | |
marun | jogo, sdague: so, the proposed fix is running gate checks now: https://review.openstack.org/#/c/80413/ | 00:13 |
---|---|---|
marun | It passed check and the side-effect may be testing taking longer due to not being able to unsafely run some things in parallel. | 00:14 |
marun | hopefully the impact is minimal but of course we should be prepared to revert if the failure rate is adversely affected. | 00:15 |
*** rhsu has quit IRC | 00:15 | |
jogo | marun: cool, this may be worth bumping to the top of the queue | 00:18 |
*** dkorn has quit IRC | 00:19 | |
*** mmaglana has quit IRC | 00:19 | |
marun | jogo: if the failure rate is still high, I would agree. | 00:19 |
jogo | marun: lets move discussion to -infra | 00:20 |
marun | jogo: ok | 00:21 |
*** Aimon has quit IRC | 00:22 | |
*** masayukig has joined #openstack-qa | 00:24 | |
*** mrodden has quit IRC | 00:25 | |
*** matsuhashi has joined #openstack-qa | 00:29 | |
*** salv-orlando has joined #openstack-qa | 00:35 | |
*** rhsu has joined #openstack-qa | 00:43 | |
*** mmaglana has joined #openstack-qa | 00:45 | |
*** yamahata has joined #openstack-qa | 00:48 | |
*** zhikunliu has joined #openstack-qa | 00:51 | |
openstackgerrit | Masayuki Igawa proposed a change to openstack/tempest: Make add_remove_fixed_ip tests executable in Nova gate https://review.openstack.org/81677 | 00:53 |
*** andreaf has quit IRC | 00:54 | |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates' https://review.openstack.org/81483 | 00:56 |
openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes https://review.openstack.org/81477 | 00:57 |
*** sthaha has joined #openstack-qa | 01:12 | |
*** sthaha has joined #openstack-qa | 01:12 | |
*** malini_afk is now known as malini | 01:22 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: add test call to the request log https://review.openstack.org/81654 | 01:22 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: simplify rest_client logging https://review.openstack.org/81620 | 01:22 |
*** nosnos has joined #openstack-qa | 01:25 | |
*** ken1ohmichi has joined #openstack-qa | 01:27 | |
*** samuelbercovici has quit IRC | 01:32 | |
*** Aimon has joined #openstack-qa | 01:33 | |
*** Aimon has quit IRC | 01:34 | |
*** salv-orlando has quit IRC | 01:35 | |
*** Aimon has joined #openstack-qa | 01:37 | |
*** Aimon has quit IRC | 01:38 | |
*** Aimon has joined #openstack-qa | 01:40 | |
openstackrecheck | Console logs not available after 13:23s for check-tempest-dsvm-trove-full 81668,2,a688873 | 01:40 |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify quotas attributes through Nova os-quota-sets API https://review.openstack.org/81439 | 01:52 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify the response status of create delete Image https://review.openstack.org/81687 | 01:53 |
*** chandan_kumar has joined #openstack-qa | 01:53 | |
*** jufeng has joined #openstack-qa | 01:55 | |
*** shwetaap has joined #openstack-qa | 02:03 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/81603 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 02:12 |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates' https://review.openstack.org/81483 | 02:12 |
*** nosnos_ has joined #openstack-qa | 02:16 | |
openstackgerrit | Edgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6 https://review.openstack.org/67312 | 02:18 |
*** bill_az_ has quit IRC | 02:19 | |
*** nosnos has quit IRC | 02:19 | |
openstackgerrit | Edgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6 https://review.openstack.org/67312 | 02:21 |
*** ken1ohmichi has quit IRC | 02:26 | |
*** che-arne has quit IRC | 02:29 | |
*** mwagner__ is now known as mwagner__zzzz | 02:37 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/81260 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1283522, gate-devstack-dsvm-cells: unrecognized error | 02:39 |
*** mriedem has quit IRC | 02:48 | |
*** mriedem has joined #openstack-qa | 02:56 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/76751 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254872 | 02:58 |
*** shwetaap has quit IRC | 03:04 | |
*** shwetaap has joined #openstack-qa | 03:04 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify quotas attributes through Nova os-quota-sets API https://review.openstack.org/81439 | 03:07 |
openstackgerrit | Yuuichi Fujioka proposed a change to openstack-dev/devstack: fix typo in running opendaylight command https://review.openstack.org/81693 | 03:11 |
*** chandan_kumar has quit IRC | 03:22 | |
*** harlowja is now known as harlowja_away | 03:23 | |
*** matsuhashi has quit IRC | 03:47 | |
openstackrecheck | openstack/ceilometer change: https://review.openstack.org/78587 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1283522 | 03:52 |
HenryG | Is there way to get devstack's stack.sh to reclone all the repos and then stop (not start openstack)? | 04:02 |
*** mriedem has quit IRC | 04:04 | |
*** mspreitz has joined #openstack-qa | 04:05 | |
mspreitz | Is all OK in the gate? | 04:06 |
*** malini is now known as malini_afk | 04:10 | |
*** Akshik has joined #openstack-qa | 04:11 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Validate image metadata attributes of Nova APIs https://review.openstack.org/81699 | 04:12 |
*** saju_m has joined #openstack-qa | 04:12 | |
*** shwetaap1 has joined #openstack-qa | 04:17 | |
*** shwetaap has quit IRC | 04:17 | |
*** sweston has joined #openstack-qa | 04:21 | |
*** matsuhashi has joined #openstack-qa | 04:31 | |
*** Madan has joined #openstack-qa | 04:33 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify "get quotas detail" API response attributes https://review.openstack.org/81701 | 04:35 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify detail_list flavor attributes of V2/V3 APIs https://review.openstack.org/80548 | 04:38 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/79511 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1283522, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254872 | 04:39 |
*** yfujioka has joined #openstack-qa | 04:40 | |
*** mmaglana has quit IRC | 04:44 | |
*** mmaglana has joined #openstack-qa | 04:45 | |
*** mmaglana has quit IRC | 04:49 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs https://review.openstack.org/80543 | 04:52 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify "create a server" API response attributes https://review.openstack.org/77517 | 04:52 |
*** Akshik_ has joined #openstack-qa | 04:56 | |
*** Akshik has quit IRC | 05:00 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs https://review.openstack.org/80543 | 05:01 |
*** yfujioka has quit IRC | 05:02 | |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates' https://review.openstack.org/81483 | 05:02 |
*** chandan_kumar has joined #openstack-qa | 05:10 | |
*** mspreitz has quit IRC | 05:16 | |
*** chandan_kumar has quit IRC | 05:17 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs https://review.openstack.org/80543 | 05:19 |
*** nosnos has joined #openstack-qa | 05:19 | |
*** nosnos_ has quit IRC | 05:23 | |
openstackgerrit | A change was merged to openstack/tempest: Add missing client names to T102 hacking check https://review.openstack.org/80814 | 05:24 |
*** amotoki has joined #openstack-qa | 05:29 | |
openstackrecheck | openstack-dev/pbr change: https://review.openstack.org/81322 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1291926 | 05:33 |
*** vkozhukalov_ has joined #openstack-qa | 05:33 | |
*** giulivo has joined #openstack-qa | 05:33 | |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 1 https://review.openstack.org/81703 | 05:40 |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 2 https://review.openstack.org/81704 | 05:41 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Verify "create a server" API response attributes https://review.openstack.org/77517 | 05:42 |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 3 https://review.openstack.org/81706 | 05:42 |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 4 https://review.openstack.org/81707 | 05:43 |
raies | giulivo: hi | 05:45 |
raies | I need some help on instance access | 05:45 |
raies | I created an instance a created network (created by me) in devstack | 05:46 |
*** nosnos_ has joined #openstack-qa | 05:49 | |
*** nosnos has quit IRC | 05:51 | |
raies | In devstack I created VM using a self created network (created by me) and a created security group ping and ssh enabled | 05:59 |
raies | still I am not able to ping this VM while it is in running state | 05:59 |
raies | what exatra settings are required there ?? | 05:59 |
raies | anyone there ^^ | 06:00 |
raies | I am using devstack | 06:01 |
*** saju_m has quit IRC | 06:02 | |
*** nati_ueno has joined #openstack-qa | 06:03 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify list Image attributes through Nova V2 API https://review.openstack.org/78908 | 06:04 |
*** saju_m has joined #openstack-qa | 06:04 | |
*** saju_m has quit IRC | 06:07 | |
*** saju_m has joined #openstack-qa | 06:08 | |
*** vkozhukalov_ has quit IRC | 06:19 | |
*** matsuhashi has quit IRC | 06:28 | |
*** Madan has quit IRC | 06:29 | |
*** matsuhashi has joined #openstack-qa | 06:30 | |
openstackgerrit | A change was merged to openstack/tempest: Validate get limits attributes of Nova V2 API https://review.openstack.org/81151 | 06:33 |
*** saju_m has quit IRC | 06:35 | |
openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify "get version" API response attributes https://review.openstack.org/80936 | 06:37 |
openstackrecheck | openstack/tempest change: https://review.openstack.org/80890 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 06:37 |
Akshik_ | do we have any approval mechanism for creating and modifying a instance | 06:44 |
openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes https://review.openstack.org/81477 | 06:49 |
*** nati_ueno has quit IRC | 06:51 | |
*** masayukig has quit IRC | 06:53 | |
*** sweston has quit IRC | 07:02 | |
*** jlibosva has joined #openstack-qa | 07:12 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/71529 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1254872, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254872 | 07:12 |
*** jlibosva has quit IRC | 07:16 | |
*** rhsu has quit IRC | 07:16 | |
*** psharma has joined #openstack-qa | 07:18 | |
*** Akshik_ has quit IRC | 07:19 | |
*** Akshik_ has joined #openstack-qa | 07:19 | |
*** Akshik has joined #openstack-qa | 07:22 | |
*** Akshik_ has quit IRC | 07:22 | |
*** Madan has joined #openstack-qa | 07:23 | |
openstackgerrit | Jerry Cai proposed a change to openstack/tempest: Fix to create volume with the image size if imageRef set https://review.openstack.org/81471 | 07:25 |
*** lukego has joined #openstack-qa | 07:27 | |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add error handling if testscenarios aren't supported https://review.openstack.org/81722 | 07:28 |
*** nati_ueno has joined #openstack-qa | 07:39 | |
*** rhsu has joined #openstack-qa | 07:41 | |
*** tkammer has joined #openstack-qa | 07:41 | |
*** jlibosva has joined #openstack-qa | 07:43 | |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates' https://review.openstack.org/81483 | 07:45 |
*** vkozhukalov_ has joined #openstack-qa | 07:49 | |
*** rhsu has quit IRC | 07:49 | |
*** nmagnezi has joined #openstack-qa | 07:55 | |
*** saju_m has joined #openstack-qa | 07:56 | |
*** andreaf has joined #openstack-qa | 07:56 | |
*** nmagnezi has quit IRC | 08:25 | |
*** Akshik_ has joined #openstack-qa | 08:25 | |
*** Akshik has quit IRC | 08:29 | |
*** nosnos has joined #openstack-qa | 08:33 | |
*** nmagnezi has joined #openstack-qa | 08:35 | |
*** nosnos_ has quit IRC | 08:37 | |
*** nmagnezi has quit IRC | 08:42 | |
*** jhenner1 has joined #openstack-qa | 08:43 | |
andreaf | mkoderer: ping | 08:43 |
*** jhenner has quit IRC | 08:45 | |
*** nati_ueno has quit IRC | 08:47 | |
*** Akshik_ has quit IRC | 08:48 | |
*** morganfainberg is now known as morganfainberg_Z | 08:54 | |
openstackrecheck | openstack/tempest change: https://review.openstack.org/80704 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1283522, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254872 | 08:55 |
*** jordanP has joined #openstack-qa | 08:56 | |
openstackgerrit | Alexander Bochkarev proposed a change to openstack/tempest: Add compute pollsters tests for ceilometer https://review.openstack.org/80965 | 08:57 |
*** nati_ueno has joined #openstack-qa | 08:58 | |
*** nati_uen_ has joined #openstack-qa | 08:59 | |
mkoderer | andreaf: I am in many meetings today.. is it about my -1? | 09:01 |
*** yassine has joined #openstack-qa | 09:03 | |
*** nati_ueno has quit IRC | 09:03 | |
andreaf | mkoderer: yes, but I think I found a way to split my change in a nicer way, so I'll follow your advice | 09:04 |
*** Akshik has joined #openstack-qa | 09:06 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/81396 failed because of: gate-tempest-dsvm-neutron: unrecognized error, gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254890 https://bugs.launchpad.net/bugs/1273908 | 09:06 |
*** Akshik_ has joined #openstack-qa | 09:07 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: add test call to the request log https://review.openstack.org/81654 | 09:08 |
*** Madan has quit IRC | 09:09 | |
*** Akshik has quit IRC | 09:11 | |
*** jhenner1 has quit IRC | 09:12 | |
mkoderer | andreaf: great thank you | 09:15 |
openstackgerrit | Jerry Cai proposed a change to openstack/tempest: Fix to create volume with the image size if imageRef set https://review.openstack.org/81471 | 09:18 |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates' https://review.openstack.org/81483 | 09:22 |
*** jhenner has joined #openstack-qa | 09:26 | |
*** Madan has joined #openstack-qa | 09:27 | |
*** safchain has joined #openstack-qa | 09:27 | |
*** safchain has quit IRC | 09:37 | |
*** psedlak_clone is now known as psedlak | 09:39 | |
openstackrecheck | openstack/python-swiftclient change: https://review.openstack.org/81479 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1283522 | 09:44 |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Network fwaas API test https://review.openstack.org/78857 | 09:49 |
*** safchain has joined #openstack-qa | 09:50 | |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Check attributes of image meta item Nova APIs https://review.openstack.org/81748 | 09:55 |
*** nosnos has quit IRC | 09:59 | |
*** yamahata has quit IRC | 10:02 | |
*** jufeng has quit IRC | 10:03 | |
*** nosnos has joined #openstack-qa | 10:09 | |
mkoderer | cyeoh: if you have time for a review.. this fixes the pep8 thing properly: https://review.openstack.org/#/c/80926/ | 10:11 |
*** rwsu has quit IRC | 10:18 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: add test call to the request log https://review.openstack.org/81654 | 10:18 |
*** safchain has quit IRC | 10:18 | |
*** safchain has joined #openstack-qa | 10:20 | |
openstackgerrit | Aleksandr Gordeev proposed a change to openstack-dev/devstack: Remove any lib/baremetal usage for ironic deploy https://review.openstack.org/80958 | 10:23 |
openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify "get version" API response attributes https://review.openstack.org/80936 | 10:26 |
openstackrecheck | Console logs not available after 13:58s for gate-neutron-python27 78861,1,9787a2c | 10:26 |
*** Jaq1 has joined #openstack-qa | 10:29 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/74622 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1254890 | 10:31 |
openstackgerrit | afazekas proposed a change to openstack/tempest: unskip test_integration_1 https://review.openstack.org/81762 | 10:31 |
*** afazekas has joined #openstack-qa | 10:31 | |
*** rwsu has joined #openstack-qa | 10:33 | |
*** Jaq1 has left #openstack-qa | 10:34 | |
openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes https://review.openstack.org/81477 | 10:35 |
*** jacqm has joined #openstack-qa | 10:36 | |
*** nati_uen_ has quit IRC | 10:43 | |
*** nati_ueno has joined #openstack-qa | 10:44 | |
*** tosky has joined #openstack-qa | 10:44 | |
*** nati_ueno has quit IRC | 10:48 | |
openstackgerrit | Eiichi Aikawa proposed a change to openstack/tempest: Verify tenant usages API response attributes https://review.openstack.org/81766 | 10:51 |
*** yamahata has joined #openstack-qa | 10:53 | |
*** yassine has quit IRC | 10:56 | |
*** chandankumar_ has quit IRC | 11:00 | |
*** chandankumar has joined #openstack-qa | 11:00 | |
*** matsuhashi has quit IRC | 11:01 | |
openstackgerrit | Pranali Deore proposed a change to openstack/tempest: Added testcase for glance https://review.openstack.org/81769 | 11:08 |
openstackgerrit | afazekas proposed a change to openstack/tempest: Fix test_verify_created_server_ephemeral_disk https://review.openstack.org/80622 | 11:08 |
openstackgerrit | Dmitry Tantsur proposed a change to openstack-dev/devstack: Configure SELinux to allow SSH to listen on $IRONIC_VM_SSH_PORT https://review.openstack.org/81770 | 11:11 |
openstackgerrit | Dmitry Tantsur proposed a change to openstack-dev/devstack: Configure SELinux to allow SSH to listen on $IRONIC_VM_SSH_PORT https://review.openstack.org/81770 | 11:13 |
*** devvesa has joined #openstack-qa | 11:17 | |
*** malini_afk is now known as malini | 11:19 | |
openstackgerrit | Pranali Deore proposed a change to openstack/tempest: Added testcase for glance https://review.openstack.org/81769 | 11:20 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/57954 failed gate-tempest-dsvm-neutron-pg, gate-tempest-dsvm-neutron-pg in the gate queue with an unrecognized error | 11:23 |
*** matsuhashi has joined #openstack-qa | 11:24 | |
*** zhikunliu has quit IRC | 11:26 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: API test for 'create server with scheduler hints' https://review.openstack.org/79908 | 11:27 |
*** tkammer has quit IRC | 11:34 | |
*** markmcclain has joined #openstack-qa | 11:35 | |
*** markmcclain1 has joined #openstack-qa | 11:35 | |
*** GMann has quit IRC | 11:35 | |
*** markmcclain1 has quit IRC | 11:35 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Account failure to the tearDown instead of setUp https://review.openstack.org/54318 | 11:36 |
*** markmcclain1 has joined #openstack-qa | 11:36 | |
*** markmcclain1 has quit IRC | 11:37 | |
*** markmcclain has quit IRC | 11:39 | |
openstackgerrit | Bartosz Górski proposed a change to openstack-dev/devstack: Adds support for multi-region https://review.openstack.org/79310 | 11:39 |
*** markmcclain has joined #openstack-qa | 11:41 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: drop unused params for _log_request https://review.openstack.org/81776 | 11:42 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: move the request_id logging into the expected place https://review.openstack.org/81777 | 11:42 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: add request timing https://review.openstack.org/81778 | 11:42 |
*** safchain_ has joined #openstack-qa | 11:42 | |
*** safchain has quit IRC | 11:42 | |
*** salv-orlando has joined #openstack-qa | 11:47 | |
*** tkammer has joined #openstack-qa | 11:50 | |
*** rfolco has joined #openstack-qa | 11:53 | |
nithyag | afazekas: Hi Attila, I had to leave yesterday and I could not respond to you. The test I intend to add would fit in https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_create_server.py as it is a test to verify positive api behavior. I see https://github.com/openstack/tempest/blob/master/tempest/scenario/test_snapshot_pattern.py to be very specific to snapshots. | 12:02 |
nithyag | afazekas: In that case, would it be ok to go ahead without https://blueprints.launchpad.net/tempest/+spec/ssh-auth-strategy? | 12:05 |
nithyag | and submit the test with the current ssh verification using adminPass? | 12:07 |
*** matsuhashi has quit IRC | 12:08 | |
*** saju_m has quit IRC | 12:09 | |
openstackrecheck | openstack/horizon change: https://review.openstack.org/79736 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1283522 | 12:11 |
*** matsuhashi has joined #openstack-qa | 12:12 | |
*** yassine has joined #openstack-qa | 12:12 | |
*** matsuhashi has quit IRC | 12:18 | |
*** malini is now known as malini_afk | 12:21 | |
*** matsuhashi has joined #openstack-qa | 12:24 | |
*** saju_m has joined #openstack-qa | 12:27 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Network fwaas API test https://review.openstack.org/78857 | 12:27 |
openstackrecheck | Console logs not available after 14:19s for check-grenade-dsvm-partial-ncpu 81498,2,63e68bd | 12:28 |
openstackgerrit | Jerry Cai proposed a change to openstack/tempest: Fix to create volume with the image size if imageRef set https://review.openstack.org/81471 | 12:32 |
*** jordanP has quit IRC | 12:34 | |
*** weshay has joined #openstack-qa | 12:39 | |
*** devvesa has quit IRC | 12:40 | |
*** sthaha has quit IRC | 12:41 | |
*** vkozhukalov_ has quit IRC | 12:42 | |
*** sthaha has joined #openstack-qa | 12:43 | |
*** nosnos has quit IRC | 12:46 | |
*** matsuhashi has quit IRC | 12:47 | |
*** jordanP has joined #openstack-qa | 12:49 | |
*** aysyd has joined #openstack-qa | 12:51 | |
*** saju_m has quit IRC | 12:53 | |
*** vkozhukalov has joined #openstack-qa | 12:58 | |
*** gfidente has joined #openstack-qa | 13:02 | |
*** mwagner__zzzz is now known as mwagner_ | 13:04 | |
*** saju_m has joined #openstack-qa | 13:07 | |
*** Madan has quit IRC | 13:10 | |
*** zul has quit IRC | 13:12 | |
*** adalbas has joined #openstack-qa | 13:14 | |
*** zul has joined #openstack-qa | 13:15 | |
*** mriedem has joined #openstack-qa | 13:15 | |
openstackgerrit | Aleksandr Gordeev proposed a change to openstack-dev/devstack: Configure ssh server properly for ironic https://review.openstack.org/81807 | 13:16 |
*** psharma has quit IRC | 13:16 | |
Akshik_ | what is the format to specify the network id in heat template | 13:17 |
*** jordanP has quit IRC | 13:17 | |
*** jordanP has joined #openstack-qa | 13:21 | |
*** saju_m has quit IRC | 13:23 | |
*** sthaha has quit IRC | 13:23 | |
*** dkranz has joined #openstack-qa | 13:31 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/81385 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 13:33 |
*** sthaha has joined #openstack-qa | 13:35 | |
*** mkollaro has joined #openstack-qa | 13:36 | |
*** salv-orlando has quit IRC | 13:38 | |
*** saju_m has joined #openstack-qa | 13:39 | |
*** andreaf has quit IRC | 13:46 | |
*** yamahata has quit IRC | 13:48 | |
*** saju_m has quit IRC | 13:48 | |
*** yamahata has joined #openstack-qa | 13:49 | |
*** tkammer has quit IRC | 13:53 | |
*** malini_afk is now known as malini | 13:58 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Move to the python-saharaclient https://review.openstack.org/81085 | 14:01 |
*** zhikunliu has joined #openstack-qa | 14:02 | |
*** mrodden has joined #openstack-qa | 14:02 | |
*** andreaf has joined #openstack-qa | 14:05 | |
*** tkammer has joined #openstack-qa | 14:06 | |
*** jordan_ has joined #openstack-qa | 14:07 | |
*** devvesa has joined #openstack-qa | 14:10 | |
*** jordanP has quit IRC | 14:10 | |
*** bill_az_ has joined #openstack-qa | 14:12 | |
*** Akshik_ has quit IRC | 14:14 | |
*** nati_ueno has joined #openstack-qa | 14:19 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Cleanup sahara cli tests https://review.openstack.org/81819 | 14:25 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add sahara edp cli commands tests https://review.openstack.org/81820 | 14:25 |
*** gfidente has quit IRC | 14:30 | |
openstackgerrit | Ivan Berezovskiy proposed a change to openstack-infra/devstack-gate: Update Readme to setup jenkins accout https://review.openstack.org/81823 | 14:37 |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Temporary workaround for Time out errors from libvirt https://review.openstack.org/79816 | 14:39 |
openstackgerrit | Dane LeBlanc proposed a change to openstack/tempest: Add network API test to create/update a port with 2 IP addresses https://review.openstack.org/67547 | 14:42 |
*** salv-orlando has joined #openstack-qa | 14:45 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/81248 failed gate-tempest-dsvm-neutron, gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 14:46 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/elastic-recheck: add match for lbaas failure https://review.openstack.org/81825 | 14:48 |
*** BobBall has joined #openstack-qa | 14:51 | |
*** nati_ueno has quit IRC | 14:52 | |
openstackgerrit | Dmitry Tantsur proposed a change to openstack-dev/devstack: Configure SELinux to allow SSH to listen on $IRONIC_VM_SSH_PORT https://review.openstack.org/81770 | 14:54 |
sdague | mriedem / mtreinish - https://review.openstack.org/#/c/81825/ | 14:55 |
*** mwagner is now known as mwagner_notHere | 14:55 | |
mkoderer | sdague: https://review.openstack.org/#/c/80926/ should be fine now | 14:56 |
*** otherwiseguy has joined #openstack-qa | 14:56 | |
mriedem | sdague: looking | 14:57 |
*** atiwari has joined #openstack-qa | 14:57 | |
*** nati_ueno has joined #openstack-qa | 15:03 | |
mtreinish | sdague: +A | 15:03 |
*** rdekel has joined #openstack-qa | 15:04 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: add match for lbaas failure https://review.openstack.org/81825 | 15:05 |
*** shwetaap1 has quit IRC | 15:05 | |
*** mwagner_ has quit IRC | 15:06 | |
*** jordan_ has quit IRC | 15:08 | |
*** openstackrecheck has quit IRC | 15:11 | |
*** openstackrecheck has joined #openstack-qa | 15:11 | |
openstackgerrit | enikanorov proposed a change to openstack/tempest: Skip loadbalancer basic scenario test https://review.openstack.org/81831 | 15:14 |
*** openstackrecheck has quit IRC | 15:15 | |
*** Aimon has quit IRC | 15:16 | |
*** yfried1 has quit IRC | 15:16 | |
*** shwetaap has joined #openstack-qa | 15:17 | |
*** tosky has quit IRC | 15:23 | |
*** yfried has joined #openstack-qa | 15:24 | |
*** tosky has joined #openstack-qa | 15:27 | |
*** krtaylor has quit IRC | 15:27 | |
andreaf | mtreinish: ping | 15:31 |
mtreinish | andreaf: hi | 15:32 |
openstackgerrit | afazekas proposed a change to openstack/tempest: Enable instance validation with the server_actions v2 https://review.openstack.org/81834 | 15:32 |
andreaf | mtreinish: I'm changing Credentials unit tests to work with the config fixture | 15:32 |
*** jordanP has joined #openstack-qa | 15:33 | |
andreaf | mtreinish: I run into a problem were running unit tests in parallel breaks the tests | 15:33 |
mtreinish | ooh that's a sign of unit test maturity | 15:33 |
mtreinish | that means we've been making progress this cycle :) | 15:33 |
andreaf | mtreinish: I have two fixtures, one setting auth_version=v2 and one v3. Any idea how to work around this? | 15:33 |
mtreinish | andreaf: I would make the tests that use the v2 fixture be the parent class for the test class that uses v3 | 15:34 |
mtreinish | then add a lock fixture to the beginning of setUp() | 15:34 |
mtreinish | that way they're guarenteed not to run at the same time | 15:35 |
andreaf | mtreinish: ok the parent / child relationship is already there, I'll try the lock, thanks! | 15:36 |
mtreinish | andreaf: for example: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_aggregates.py#n143 | 15:37 |
*** kgriffs_afk is now known as kgriffs | 15:37 | |
andreaf | mtreinish: self.useFixture(fixtures.LockFixture('auth_version')) ? | 15:38 |
andreaf | mtreinish: the name is just a name right? | 15:38 |
mtreinish | andreaf: yeah | 15:38 |
mtreinish | that should work fine | 15:38 |
*** Aimon has joined #openstack-qa | 15:40 | |
*** tosky_ has joined #openstack-qa | 15:40 | |
*** tosky has quit IRC | 15:40 | |
andreaf | mtreinish: trying.... | 15:40 |
andreaf | mtreinish: RequiredOptError: value required for option: lock_path | 15:42 |
andreaf | mtreinish: lock_path is defined in tempest.conf | 15:43 |
*** tkammer has quit IRC | 15:43 | |
*** otherwiseguy has quit IRC | 15:43 | |
mtreinish | hah, oops yeah I guess we should figure out a way to do that | 15:43 |
andreaf | mtreinish: but tempest.conf is not there.... maybe it'll work if I put it in the fake config | 15:43 |
mtreinish | it will | 15:43 |
mtreinish | or at least it should | 15:43 |
*** rdekel has quit IRC | 15:44 | |
mtreinish | we'll have to think about setting that as a default in the base test class at some point | 15:44 |
mtreinish | but the config fixture isn't used everywhere so that can come later | 15:44 |
mtreinish | andreaf: I would just make a tmpdir using the lib and make sure you clean it up after the test | 15:45 |
andreaf | right the problem with having the config in the fake config is that it's an hardcoded path | 15:45 |
andreaf | mtreinish: good idea | 15:45 |
andreaf | mtreinish: do you mean "import tempfile" | 15:47 |
mtreinish | yeah and then using something like: tempfile.mkdtemp(prefix='tempest-unit-test-locks') | 15:48 |
mtreinish | I did basically the same thing in test_wrappers.py | 15:49 |
*** rossella_s has quit IRC | 15:54 | |
andreaf | mtreinish: ok, but I would do it in the fixture in this case as I need to setup the fake config with the tmp dir | 15:55 |
andreaf | mtreinish: so I'd override the fixture cleanup() instead of using addCleanup() | 15:55 |
*** rossella_s has joined #openstack-qa | 15:56 | |
mtreinish | andreaf: I think I'd have to see your code, I don't have enough context | 15:56 |
*** zhikunliu has quit IRC | 15:58 | |
mtreinish | andreaf: this is what I was thinking about though: http://paste.openstack.org/show/73917/ | 15:58 |
*** devvesa has quit IRC | 15:58 | |
mtreinish | but I maybe missing something I didn't actually test it | 15:59 |
mtreinish | oh you could bake that into the config fixture itself | 15:59 |
andreaf | mtreinish: I was doing something like http://paste.openstack.org/show/73918/ | 16:00 |
mtreinish | yeah that's probably the better way to do it | 16:01 |
mtreinish | that way if locking is ever needed the lock path is set | 16:02 |
mtreinish | the only downside is that it's making a tempdir for every test that needs config | 16:02 |
andreaf | mtreinish: I was just thinking that... so it's not actually locking as the tmp dir is different? | 16:03 |
andreaf | the lock dir I mean | 16:03 |
mtreinish | hmm yeah that's right | 16:03 |
sdague | mtreinish: you mind giving this series a look - https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:rest_client_logging,n,z | 16:04 |
andreaf | mtreinish: I just created a few 100s tmp dirs on my workstation | 16:04 |
mtreinish | andreaf: I'm gonna run to lunch I'll be back in a bit | 16:04 |
sdague | most of the test fails are upstream races and unrelated | 16:04 |
sdague | and I'd like your feedback on those in case we should restack them differently | 16:04 |
*** devvesa has joined #openstack-qa | 16:06 | |
*** otherwiseguy has joined #openstack-qa | 16:07 | |
openstackgerrit | sahid proposed a change to openstack-dev/devstack: Fix bug ext2: parent directory not found with guestfs https://review.openstack.org/79577 | 16:08 |
*** sthaha has quit IRC | 16:16 | |
openstackgerrit | Jerry Cai proposed a change to openstack/tempest: Add "capabilities:" to multiple_backends extra-spec key https://review.openstack.org/81842 | 16:17 |
*** mrodden has quit IRC | 16:17 | |
*** yfried has quit IRC | 16:20 | |
*** nithyag has quit IRC | 16:24 | |
openstackgerrit | Sergey Lukjanov proposed a change to openstack/tempest: Add sahara edp cli commands tests https://review.openstack.org/81820 | 16:27 |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: wip: DO NOT MERGE gather dstat data on ceilometer free env https://review.openstack.org/81843 | 16:27 |
*** yfried has joined #openstack-qa | 16:31 | |
openstackgerrit | Ivan Berezovskiy proposed a change to openstack-infra/devstack-gate: Update Readme to setup jenkins accout https://review.openstack.org/81823 | 16:31 |
*** mrodden has joined #openstack-qa | 16:31 | |
*** rhsu has joined #openstack-qa | 16:33 | |
*** dims has quit IRC | 16:37 | |
*** mmaglana has joined #openstack-qa | 16:43 | |
*** YorikSar has left #openstack-qa | 16:44 | |
jogo | sdague: I see the gate isn't any better today | 16:45 |
openstackgerrit | enikanorov proposed a change to openstack/tempest: Skip loadbalancer basic scenario test https://review.openstack.org/81831 | 16:48 |
*** dims has joined #openstack-qa | 16:49 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Translate xml server tenantId /userId https://review.openstack.org/81847 | 16:50 |
*** otherwiseguy has quit IRC | 16:52 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Enable instance validation with the server_actions v2 https://review.openstack.org/81834 | 16:56 |
openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: XenAPI: Find location for XenAPI plugins https://review.openstack.org/81849 | 16:58 |
*** kgriffs is now known as kgriffs_afk | 17:03 | |
openstackgerrit | Aleksandr Gordeev proposed a change to openstack-dev/devstack: Configure ssh server properly for ironic https://review.openstack.org/81807 | 17:03 |
*** mkollaro has quit IRC | 17:04 | |
*** devvesa has quit IRC | 17:05 | |
*** mwagner_ has joined #openstack-qa | 17:07 | |
*** nati_ueno has quit IRC | 17:08 | |
*** nati_ueno has joined #openstack-qa | 17:08 | |
*** tkammer has joined #openstack-qa | 17:08 | |
*** nati_ueno has quit IRC | 17:12 | |
*** vkozhukalov has quit IRC | 17:17 | |
*** jordanP has quit IRC | 17:20 | |
andreaf | mtreinish: ping | 17:21 |
*** mkollaro has joined #openstack-qa | 17:22 | |
*** otherwiseguy has joined #openstack-qa | 17:23 | |
*** morganfainberg_Z is now known as morganfainberg | 17:23 | |
*** yassine has quit IRC | 17:23 | |
*** krtaylor has joined #openstack-qa | 17:27 | |
openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1295261 https://review.openstack.org/81855 | 17:30 |
openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Fix elasticsearch URL for gate queue https://review.openstack.org/81856 | 17:30 |
*** afazekas has quit IRC | 17:30 | |
*** tkammer has quit IRC | 17:37 | |
mtreinish | andreaf: hey | 17:52 |
*** mkollaro has quit IRC | 17:55 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Fix elasticsearch URL for gate queue https://review.openstack.org/81856 | 17:55 |
*** safchain_ has quit IRC | 17:55 | |
mtreinish | sdague: ok I'll take a look in a bit | 17:57 |
andreaf | mtreinish: I'm not managing to get locking working properly | 18:01 |
andreaf | mtreinish: I was wondering if I upload the patchset if you would mind have a look at it and see if you see the problem - I tried even with a fixed folder but it didn't help | 18:02 |
mtreinish | andreaf: sure I can give it a shot | 18:02 |
*** mriedem has quit IRC | 18:02 | |
andreaf | mtreinish: thanks | 18:03 |
*** openstackrecheck has joined #openstack-qa | 18:04 | |
*** mkollaro has joined #openstack-qa | 18:05 | |
openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1294971 https://review.openstack.org/81871 | 18:06 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Define V3 Credentials https://review.openstack.org/81872 | 18:07 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Defines a Credentials class https://review.openstack.org/74387 | 18:07 |
andreaf | mtreinish: https://review.openstack.org/81872 | 18:07 |
*** mriedem has joined #openstack-qa | 18:14 | |
*** otherwiseguy has quit IRC | 18:16 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/79195 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error | 18:18 |
openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1295261 https://review.openstack.org/81855 | 18:19 |
*** yfried has quit IRC | 18:20 | |
*** dkorn has joined #openstack-qa | 18:21 | |
*** stack555 has joined #openstack-qa | 18:21 | |
stack555 | Hello I am working on the folllowing review of adding test cases for nuage plugin https://review.openstack.org/#/c/81673/ .. I have added a decorator requires_ext='net-parition' in my test case so that it runs only when nuage plugin is enabled but I noticed that when i checked out code the tempest.conf by default has api_extension set to all | 18:22 |
stack555 | Because of this my test case is run by jenkins all the time | 18:22 |
stack555 | Is there a way I can get around this? | 18:22 |
*** yfried has joined #openstack-qa | 18:23 | |
*** packet has joined #openstack-qa | 18:26 | |
*** jlibosva has quit IRC | 18:26 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1294874 https://review.openstack.org/81649 | 18:31 |
*** markmcclain has quit IRC | 18:34 | |
*** openstackrecheck has quit IRC | 18:34 | |
*** openstackrecheck has joined #openstack-qa | 18:34 | |
*** arosen has joined #openstack-qa | 18:36 | |
*** mkollaro has quit IRC | 18:36 | |
mtreinish | mriedem: I remember you were looking at using locks in the nova unit tests what was the solution that you ended up coming with there? | 18:47 |
*** tosky_ has quit IRC | 19:05 | |
openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1294971 https://review.openstack.org/81871 | 19:09 |
mriedem | mtreinish: i don't remember that | 19:12 |
mtreinish | mriedem: ok, I think I figured out something anyway | 19:13 |
mtreinish | andreaf: http://paste.openstack.org/show/73936/ | 19:13 |
mtreinish | I confirmed the locking works correctly there | 19:14 |
mtreinish | but the same 3 unit tests still fail | 19:14 |
*** vkozhukalov has joined #openstack-qa | 19:15 | |
mtreinish | andreaf: I'm going to push the lock fixture usage thing out as a separate patch | 19:16 |
andreaf | mtreinish: thanks, that's quite useful | 19:16 |
*** kgriffs_afk is now known as kgriffs | 19:17 | |
andreaf | mtreinish: I'll check why the test fail - there may be something else - strange that when I run them without parallel they all work. There may be some odd dependency that I missed | 19:18 |
openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: provide more specific feedback to query authors https://review.openstack.org/81891 | 19:18 |
mtreinish | andreaf: well that's why I added the lock to the tenant_isolation tests too | 19:18 |
mtreinish | because you were using the V2 fixture there | 19:18 |
openstackgerrit | Dolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1295261 https://review.openstack.org/81855 | 19:19 |
mtreinish | but when I added debug to the auth methods I saw that get valid stuff was getting called twice | 19:19 |
mtreinish | and the second time had invalid results | 19:19 |
andreaf | mtreinish: how do you get a log file written by the unit test? | 19:20 |
andreaf | mtreinish: I tried adding logging statement, but I suppose that if the config is not there it won't write to the log file | 19:21 |
mtreinish | dunno I've never tried | 19:21 |
mtreinish | I just added prints | 19:21 |
andreaf | mtreinish: oh, ok | 19:21 |
mtreinish | I'm sure there is a way | 19:22 |
andreaf | mtreinish: thanks for the help, it's really appreciated | 19:22 |
andreaf | mtreinish: I'm splitting the multi-auth patches down further, and in the process I'm also rebasing them, so it's taking me a while | 19:23 |
mtreinish | yeah it's a big refactor I didn't expect it to be super fast | 19:24 |
andreaf | mtreinsh: I've got 7 patches I'm working on. And there's still quite some other stuff to be done | 19:24 |
openstackrecheck | Console logs not available after 13:24s for check-tempest-dsvm-full 54318,8,57a986f | 19:25 |
andreaf | mtreinish: btw. I won't make it to the meeting tonight (it's 10pm for me) | 19:25 |
openstackrecheck | openstack/python-heatclient change: https://review.openstack.org/75383 failed gate-tempest-dsvm-neutron-heat-slow in the gate queue with an unrecognized error | 19:26 |
andreaf | mtreinish: but you already have my updates on the multi auth stuff | 19:26 |
mtreinish | andreaf: yeah that's fine this bp is one of the more visable ones too | 19:27 |
andreaf | mtreinish: https://review.openstack.org/#/c/74387/ is now down to +343, -37 lines, I think that's as small as I'm able to make it. It's still running check though | 19:28 |
mtreinish | andreaf: yeah that's a lot easier to review | 19:29 |
mtreinish | plus over half of it is unit testing | 19:29 |
mtreinish | which is a really good thing | 19:29 |
andreaf | mtreinish: the last update I have is that I defined a couple of bp in the bp repo, both related to the multi-auth stuff | 19:29 |
andreaf | mtreinish: yep unit tests can be tricky but they are really worth the effort | 19:30 |
andreaf | mtreinish: ok going home now, have a good day | 19:30 |
*** kgriffs is now known as kgriffs_afk | 19:31 | |
*** andreaf has quit IRC | 19:31 | |
*** andreaf has joined #openstack-qa | 19:32 | |
*** packet has quit IRC | 19:33 | |
*** andreaf has quit IRC | 19:33 | |
*** andreaf has joined #openstack-qa | 19:34 | |
*** packet has joined #openstack-qa | 19:34 | |
*** malini is now known as malini_afk | 19:34 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests https://review.openstack.org/81896 | 19:35 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests https://review.openstack.org/81896 | 19:36 |
*** dkorn has quit IRC | 19:37 | |
*** andreaf_ has joined #openstack-qa | 19:45 | |
*** andreaf has quit IRC | 19:48 | |
*** jamielennox is now known as jamielennox|away | 19:48 | |
*** andreaf has joined #openstack-qa | 19:52 | |
*** andreaf_ has quit IRC | 19:53 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests https://review.openstack.org/81896 | 19:57 |
*** otherwiseguy has joined #openstack-qa | 19:58 | |
sdague | jogo: I don't think that's true, I think it's just taking time to clear | 19:59 |
sdague | mtreinish: so any thoughts on the log_request approach? | 20:04 |
*** andreaf has quit IRC | 20:08 | |
openstackrecheck | openstack/nova change: https://review.openstack.org/53775 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1291926 | 20:12 |
*** kgriffs_afk is now known as kgriffs | 20:16 | |
jogo | sdague: agreed, I didn't realize the top two bugs were fixed early this AM | 20:21 |
*** kgriffs is now known as kgriffs_afk | 20:25 | |
dkranz | mtreinish: Seems that scenario tests always run with tenant isolation. Isn't that a bug? | 20:25 |
openstackgerrit | Pablo Andres Fuente proposed a change to openstack/tempest: Add test for OS::Neutron::* Heat resource https://review.openstack.org/81909 | 20:29 |
*** kgriffs_afk is now known as kgriffs | 20:32 | |
mriedem | jogo: i'll let you +A https://review.openstack.org/#/c/81855/ if you're good with it now | 20:36 |
*** rwsu has quit IRC | 20:37 | |
jogo | mriedem: done | 20:40 |
*** mkoderer has quit IRC | 20:41 | |
*** che-arne has joined #openstack-qa | 20:41 | |
sdague | dtroyer: thoughts on - https://review.openstack.org/#/c/81663/ ? | 20:44 |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: add query for bug 1295261 https://review.openstack.org/81855 | 20:45 |
mtreinish | dkranz: no all the tests always run with tenant isolation | 20:45 |
sdague | mtreinish: I think the question was is there a way to turn them off | 20:45 |
mtreinish | sdague: just for the scenario tests? or in general | 20:45 |
mtreinish | because there is a config flag | 20:46 |
*** dims has quit IRC | 20:46 | |
mtreinish | but you can't run in parallel that way | 20:46 |
sdague | right, but that works right for scenario as well? | 20:46 |
mtreinish | sdague: it should | 20:46 |
sdague | that's how I interpretted his question | 20:46 |
mtreinish | but there might be a bug I dunno | 20:46 |
*** mwagner_ has quit IRC | 20:46 | |
*** openstackrecheck has quit IRC | 20:46 | |
*** openstackrecheck has joined #openstack-qa | 20:47 | |
dansmith | http://logs.openstack.org/62/69262/11/check/check-tempest-dsvm-full/b52c95d/logs/tempest_conf.txt.gz | 20:47 |
dansmith | resize=false | 20:47 |
dansmith | are we not testing resizes upstream anymore? or is it only enabled on one test? | 20:47 |
mtreinish | sdague: on the logging one thing is that I think that having the resp headers and body is sometimes very useful for debugging | 20:47 |
mtreinish | I can think of a couple of cases where I needed that to help debug a failure (mostly api bugs) | 20:48 |
sdague | mtreinish: those aren't on the server side? | 20:48 |
sdague | dansmith: it looks like we aren't testing it again | 20:49 |
dansmith | sdague: unintentionally I assume? | 20:50 |
sdague | I assume | 20:50 |
dansmith | like, the default changed in tempest or something? | 20:50 |
sdague | yeh, or a refactor in devstack lost the flag | 20:50 |
sdague | it should be set from devstack | 20:50 |
dansmith | I was about to smack the vmware guys around because resize on vmware is totally broken and they're not testing it, | 20:50 |
dansmith | but apparently nobody is | 20:50 |
sdague | yeh | 20:50 |
dansmith | which is :(( | 20:50 |
sdague | it looks like we are only running the negative tests | 20:50 |
sdague | agreed | 20:51 |
mtreinish | sdague: I thought the request gets logged for some services but I'm not sure about the reply. Also not all the services log that | 20:51 |
dansmith | what does that mean? | 20:51 |
dansmith | (negative tests) | 20:51 |
sdague | the attempts to do resize badly | 20:51 |
sdague | which nova prevents | 20:51 |
dansmith | okay | 20:51 |
sdague | let me go make a devstack patch to turn that back on | 20:51 |
dansmith | that would be awesome, thanks | 20:52 |
mtreinish | sdague: shouldn't we also block the negative tests if resize isn't enabled? | 20:52 |
dansmith | and I'll prepare to look at what is totally broken as a result :( | 20:52 |
*** marun has quit IRC | 20:53 | |
*** jamielennox|away is now known as jamielennox | 20:55 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: turn on resize feature https://review.openstack.org/81917 | 20:55 |
mtreinish | sdague: also back to the logging by doing it all at info every request after setUpClass will get picked up by the pylogger fixture so on a failure every request will be dumped there | 20:55 |
mtreinish | not that that's a bad thing | 20:55 |
mtreinish | but it's more verbose than what's there now | 20:56 |
sdague | yeh, I don't think that's an issue | 20:56 |
sdague | we were at info before | 20:56 |
sdague | _log_request and _log_response were both at info | 20:56 |
dtroyer | sdague: looks good... | 20:57 |
mtreinish | for request and response really? I thought it was just the req-id | 20:57 |
sdague | mtreinish: have you looked at our logs? :) | 20:57 |
*** krtaylor has quit IRC | 20:57 | |
mtreinish | I'm just going from memory I guess I should do that | 20:57 |
sdague | so honestly I'd rather at this point err on the side of throwing out as much as possible, and then pushing back on servers to log the right bits on their side. Because I feel a lot of verbosity in this log isn't all that useful. | 20:58 |
sdague | like the fact that we dump out the config N times | 20:59 |
mtreinish | sdague: oh I see where I was getting confused we'd log request: method url and response: status code at info | 20:59 |
mtreinish | and the headers and body at debug | 20:59 |
mtreinish | yeah that config thing is weird | 21:00 |
*** rockyg has joined #openstack-qa | 21:01 | |
mtreinish | I think it's fine to do this I'm just wary about wedging us in a place where it's impossible to debug tempest in the gate | 21:01 |
mtreinish | and honestly for the most part we definitely are way too verbose | 21:01 |
*** dims has joined #openstack-qa | 21:01 | |
mtreinish | I think it was mriedem who told me doesn't ever open a tempest log because it kills his browser | 21:02 |
sdague | yeh | 21:02 |
*** ken1ohmichi has joined #openstack-qa | 21:02 | |
sdague | and it wasn't really useful | 21:02 |
mriedem | it was me | 21:02 |
mriedem | some joker just told me to pull the file down | 21:03 |
maurosr | jaypipes: I guess you can answer this better than me https://bugs.launchpad.net/tempest/+bug/1278255 | 21:03 |
maurosr | had to go home now brb | 21:03 |
sdague | 2014-03-20 01:18:44.457 1979 INFO tempest.common.rest_client [-] Response Status: 200 | 21:03 |
sdague | super useful | 21:03 |
mtreinish | maurosr: https://review.openstack.org/#/c/62101/ | 21:03 |
mtreinish | sdague: yeah I think the info changes are fine but maybe keep the debug level for the rest client around | 21:04 |
*** rwsu has joined #openstack-qa | 21:04 | |
mtreinish | maurosr: that is related I think I just haven't ever found the time to fix it | 21:04 |
*** kgriffs is now known as kgriffs_afk | 21:05 | |
sdague | ok, what do you want in the debug, because my patch series stripped out the params at one point | 21:05 |
*** rfolco has quit IRC | 21:05 | |
mtreinish | sdague: just the resp body and headers (and maybe req too) | 21:05 |
sdague | so that's going to make it as enormous as it is today | 21:06 |
sdague | which is too big for anything but chrome (2.1 MB compressed currently) | 21:06 |
mtreinish | sdague: the info level will have 1/2 as much because you're combing the response and req levels :) | 21:06 |
mtreinish | sdague: ok, I wish there was a way to just log that on failures | 21:07 |
*** aysyd has quit IRC | 21:07 | |
*** jaypipes has quit IRC | 21:07 | |
mtreinish | oh maybe just on the service clients because the auth stuff really isn't needed to be logged | 21:07 |
*** mdenny has quit IRC | 21:07 | |
*** mdenny has joined #openstack-qa | 21:07 | |
mtreinish | anyway your change is fine I can always iterate on having useful debug info that's not a huge mess | 21:07 |
sdague | that's actually an interesting idea to follow on later. Add the debug into a class specific buffer. So if there is a fail it comes out | 21:09 |
*** lukego has quit IRC | 21:09 | |
sdague | lets see if it's a huge problem or not | 21:09 |
sdague | to not have it | 21:09 |
mtreinish | I mean we definitely still have stupid logs sometimes for example: http://logs.openstack.org/78/81778/1/check/check-tempest-dsvm-postgres-full/893ff25/logs/tempest.txt.gz#_2014-03-20_13_47_50_893 | 21:10 |
sdague | also, logging keystone tokens, never useful :) | 21:10 |
mtreinish | sorry wrong link | 21:10 |
sdague | mtreinish: agreed | 21:10 |
mtreinish | this one http://logs.openstack.org/78/81778/1/check/check-tempest-dsvm-postgres-full/893ff25/logs/tempest.txt.gz#_2014-03-20_13_47_31_856 | 21:10 |
sdague | I was going to go whack other things when I get a chance | 21:10 |
mtreinish | see that stuff is dumb to log except for when a cli test falls apart | 21:11 |
sdague | yep | 21:11 |
sdague | 100% agree | 21:11 |
sdague | mtreinish: you want me to collapse the patches at all? | 21:11 |
sdague | I have one more change on the _find_caller, as I figured out a better way to get the class for the _run_cleanups triggers | 21:11 |
sdague | so I have to do a rebase anyway | 21:12 |
mtreinish | probably it's not like they're large changes but it may help it get through review faster | 21:12 |
*** mdenny has quit IRC | 21:13 | |
mtreinish | and they are tightly coupled | 21:13 |
*** jaypipes has joined #openstack-qa | 21:14 | |
*** ChanServ sets mode: +o jaypipes | 21:14 | |
*** dkranz has quit IRC | 21:16 | |
sdague | mtreinish: ok, so it feels like 1, 3, 4 could probably go together. 2 is the _find_caller which seems unique enough to be it's own thing, and 5 is the timing add, which is small, but seems distinct | 21:16 |
morganfainberg | sdague, ++ please don't log the keystone token! | 21:17 |
morganfainberg | :P | 21:17 |
morganfainberg | i know we're guilty of it sometimes | 21:17 |
sdague | morganfainberg: yeh, well that ship is sailed so far... | 21:17 |
sdague | go look at logs some times | 21:17 |
morganfainberg | sdague, *sigh( i know :( | 21:18 |
sdague | logging pki tokens is so fun too | 21:18 |
sdague | it's actually the only reason the horizonal scroll bar is still triggered on log.openstack.org | 21:18 |
sdague | glanceclient is the biggest offender I think | 21:18 |
*** mlavalle has joined #openstack-qa | 21:19 | |
clarkb | sdague: yeah the CRM114 stuff needs to be taught to treat all UUIDS and pki tokens as a single TOKEN identifier | 21:20 |
openstackrecheck | openstack/ceilometer change: https://review.openstack.org/71841 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1295165 | 21:20 |
openstackgerrit | Matthew Treinish proposed a change to openstack-dev/devstack: Fix incorrect tempest config groups https://review.openstack.org/81922 | 21:21 |
*** jamielennox is now known as jamielennox|away | 21:22 | |
*** vkozhukalov has quit IRC | 21:25 | |
sdague | clarkb: how much are we stripping before we pass data to it?. | 21:27 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: add _find_caller to the request log https://review.openstack.org/81654 | 21:27 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: simplify rest_client logging https://review.openstack.org/81620 | 21:27 |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: add request timing https://review.openstack.org/81778 | 21:27 |
clarkb | sdague: right now it isn't really stripping anything but the timestamps | 21:28 |
clarkb | sdague: I keep meaning to do that substitution then get distracted by things that don't require me to learn a new programming language :) I should hunker down and do it for fun over the weekened | 21:28 |
sdague | so honestly, I be if you threw out the whole line up to the ] | 21:29 |
sdague | you'd probably greatly increase it's usefulness | 21:29 |
sdague | basically keep the message after the request id | 21:30 |
clarkb | sdague: good point there is extra info there that is all meta | 21:30 |
sdague | clarkb: https://review.openstack.org/81654 turned out pretty solid | 21:33 |
sdague | that's the call stack tracking to tell where a rest call is coming from | 21:33 |
*** afazekas has joined #openstack-qa | 21:36 | |
*** weshay has quit IRC | 21:40 | |
sdague | dtroyer: also, because I ran this through on the weekend, I wanted to make sure to highlight it to you - https://review.openstack.org/#/c/80792/ | 21:45 |
sdague | it seems to really work now | 21:45 |
sdague | and no bad side effects so far | 21:45 |
dtroyer | sdague: really? fflush doesn't close, but it apparently isn't happy if stdout is closed? if that's true than yeah, changing the order will make a difference | 21:48 |
dtroyer | sorry I'm not as attentive in devstack this week, it's spring break… so feel free to ping me on anything that is more than casually urgent | 21:49 |
sdague | yep, no worries | 21:50 |
sdague | yeh, I have devstack-gate logs that showed that | 21:50 |
sdague | when I removed the > /dev/null I got log out put | 21:50 |
sdague | it was weird | 21:50 |
sdague | and I've yet to see a truncated log since | 21:50 |
dtroyer | cool. maybe I got too aggressive closing file handles | 21:51 |
sdague | we actually started to get the jobs trace in the logs, which we never had before | 21:51 |
sdague | no, this was in devstack-gate | 21:51 |
sdague | we added a > /dev/null on the devstack run to keep the logs out of the console | 21:51 |
dtroyer | ah, ok. is the stdbuff still in the stack.sh call? I'm not sure that is still necessary | 21:51 |
sdague | yep | 21:52 |
dtroyer | but this may not be the time to try taking it out | 21:52 |
*** mriedem has quit IRC | 21:53 | |
openstackgerrit | Sean Dague proposed a change to openstack/tempest: don't log cli output on success https://review.openstack.org/81926 | 21:56 |
sdague | dtroyer: agreed | 21:56 |
sdague | mtreinish: there you go :) | 21:56 |
*** dkranz has joined #openstack-qa | 21:57 | |
sdague | morganfainberg: ... so keystone client and token logging - http://logs.openstack.org/78/81778/1/check/check-tempest-dsvm-postgres-full/893ff25/logs/tempest.txt.gz#_2014-03-20_13_48_19_721 | 21:58 |
sdague | you guys do it a lot :) | 21:58 |
*** jgrimm has quit IRC | 21:59 | |
mtreinish | meeting time | 22:00 |
afazekas | dtroyer, sdague: https://review.openstack.org/#/c/77630/ | 22:02 |
afazekas | At the moment the tgtd is not available for RHEL7, and probably the LIO will be the default for iSCSi anyway, on the other distros both are available | 22:05 |
*** masayuki_ has joined #openstack-qa | 22:05 | |
openstackrecheck | Console logs not available after 13:22s for gate-horizon-pep8 71330,11,f18c5a1 | 22:05 |
openstackrecheck | openstack/ceilometer change: https://review.openstack.org/71839 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1291926 https://bugs.launchpad.net/bugs/1254872 | 22:05 |
*** yjiang5 has quit IRC | 22:05 | |
*** marun has joined #openstack-qa | 22:07 | |
*** atiwari has quit IRC | 22:08 | |
dtroyer | afazekas: just one small bit to do yet | 22:10 |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Upgrade to cirros 0.3.2 https://review.openstack.org/81486 | 22:13 |
afazekas | dtroyer: can I use the #NOPRIME with the scsi-target-utils as well ? | 22:16 |
dtroyer | I didn't see that getting installed manually. | 22:17 |
dtroyer | I think you always want that if cinder is configured, right? | 22:17 |
*** Sukhdev has joined #openstack-qa | 22:20 | |
afazekas | dtroyer: If you are using LIO it is not needed, it is required for the tgtd | 22:21 |
dtroyer | ok. if you add #NORPIME than you will need to install it in install_cinder() | 22:22 |
afazekas | ok | 22:22 |
dtroyer | usually we've only done that on packages that have side-effects, like servers. I can go either way on that one | 22:23 |
dtroyer | nova compute uses it too, so it would be fine to leave it alone if you want | 22:24 |
afazekas | dtroyer: Since the yum install now are strictly checked, otherwise it cause install issue on RHEL7 | 22:24 |
dtroyer | oh, for missing deps, right | 22:24 |
dtroyer | is nova going to have the same issue then? | 22:25 |
afazekas | Now 5 packages is missing on RHEL7 these 3 https://review.openstack.org/#/c/79947/ and the python-mox and the scsi-target-utils | 22:26 |
*** nati_ueno has joined #openstack-qa | 22:26 | |
dtroyer | off-topic, has anything been announced about RHEL7's release date? even if it just which quarter it is expected in? | 22:28 |
sdague | jogo: you know you are a real-boy when your gate job is racing and failing other people's code - http://logs.openstack.org/54/81654/6/check/check-grenade-dsvm-partial-ncpu/f2b44ec/console.html | 22:28 |
jogo | sdague: ' No distributions at all found for PrettyTable' looks like the culprit | 22:31 |
afazekas | dtroyer: I do not know about it | 22:31 |
dtroyer | ok, np | 22:31 |
sdague | jogo: yep | 22:31 |
sdague | ER caught it | 22:31 |
sdague | still funny | 22:31 |
jogo | ahh good | 22:32 |
sdague | is that a mirror bug? | 22:32 |
jogo | sdague: ohh that was your patch ;) | 22:32 |
sdague | :) | 22:32 |
jogo | https://review.openstack.org/#/c/81665/ | 22:33 |
jogo | pip socket timeout | 22:33 |
sdague | Y U FAIL MY PACH! | 22:33 |
jogo | haha | 22:33 |
clarkb | jogo: sdague: derekh is working on upping that timeout | 22:33 |
sdague | clarkb: yeh, I saw there was a patch in progress | 22:34 |
jogo | oh if your in here clarkb, so that bug is marked as incomplete | 22:34 |
clarkb | I was mean and -1'd it (but there is a totally easy way to do it :) ) | 22:34 |
jogo | 1272417 | 22:34 |
sdague | you sure a timeout will fix it, and it doesn't need retry | 22:34 |
sdague | like we did on the git calls in devstack | 22:34 |
clarkb | sdague: no we are not sure | 22:34 |
clarkb | which is why that bug is marked incomplete. There isn't a whole lot of info | 22:34 |
clarkb | I will change it to triaged so that it doesn't go away | 22:35 |
jogo | clarkb: ahh thanks | 22:35 |
jogo | this isn't related to the mirror being updated is it? | 22:35 |
*** adalbas has quit IRC | 22:37 | |
clarkb | I don't think so, we use rsync for that which should be atomic renaming things right? | 22:37 |
clarkb | maybe it doesn't | 22:38 |
* clarkb tries to remember | 22:38 | |
clarkb | yeah it copies to a temp file then renames so we should be ok | 22:38 |
clarkb | as any open connections would continue reading the old file and new connections would get the new file | 22:39 |
*** cyeoh is now known as cyeoh_away | 22:39 | |
sdague | yeh, it's atomic | 22:41 |
sdague | even on partials | 22:41 |
sdague | there is often an issue though that the index comes first | 22:42 |
sdague | and the packages after | 22:42 |
sdague | distro mirrors often screw up that way | 22:42 |
clarkb | right but we shouldn't remove anything with rsync | 22:42 |
clarkb | so if a pacakge was there before it will continue to show up in the index | 22:42 |
sdague | clarkb: sure, but if you point to a file that doesn't yet exist | 22:42 |
sdague | I assume decision is done at the index level | 22:43 |
clarkb | right so there is a race around jobs trying to use packages that don't exist yet | 22:43 |
* sdague is honestly not sure about all the pypi mirror internals | 22:43 | |
clarkb | sdague: it is super simple. it basically rsyncs a pip package cache into an apache docroot that is served with mod autoindex | 22:44 |
clarkb | sdague: so there isn't a race with rsync copying a new index file first | 22:44 |
clarkb | and shouldn't be a race with new files getting put int place | 22:44 |
*** andreaf has joined #openstack-qa | 22:44 | |
openstackgerrit | Steve Baker proposed a change to openstack/tempest: Add a StackResourceBuildErrorException https://review.openstack.org/81935 | 22:56 |
openstackgerrit | Steve Baker proposed a change to openstack/tempest: Catch StackResourceBuildErrorException on WaitCondition error https://review.openstack.org/81936 | 22:56 |
*** andreaf_ has joined #openstack-qa | 22:56 | |
*** andreaf has quit IRC | 22:58 | |
dkranz | sdague, mtreinish : About isolated scenario it was just that the scenario code does not check the conf variable for isolation. I was going to fix that unless you said it was not a bug. | 23:00 |
mtreinish | dkranz: yeah that's a bug | 23:00 |
dkranz | mtreinish: ok, np. I'll fix it. | 23:00 |
*** masayuki_ is now known as masayukig | 23:02 | |
*** shwetaap has quit IRC | 23:05 | |
*** andreaf_ has quit IRC | 23:06 | |
*** salv-orlando has quit IRC | 23:06 | |
*** Aimon has quit IRC | 23:06 | |
*** salv-orlando has joined #openstack-qa | 23:07 | |
*** rockyg has quit IRC | 23:08 | |
*** jamielennox|away is now known as jamielennox | 23:08 | |
*** mlavalle has quit IRC | 23:12 | |
openstackgerrit | afazekas proposed a change to openstack-dev/devstack: Add LIO target / iscsi_helper support to cinder https://review.openstack.org/77630 | 23:16 |
openstackgerrit | A change was merged to openstack/tempest: Add server to clean up even in case of errors https://review.openstack.org/81573 | 23:17 |
afazekas | https://review.openstack.org/#/c/81834/ is anybody able to reproduce this failure ? | 23:20 |
afazekas | At least 1 from 4 n-net job fails | 23:21 |
openstackgerrit | A change was merged to openstack/tempest: Fix error trace induced by dhcp test actions https://review.openstack.org/80418 | 23:24 |
* afazekas TODO hack the ssh code at the lower level (ssh or restClient) to always report details | 23:24 | |
openstackrecheck | openstack/requirements change: https://review.openstack.org/81319 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/1262906 | 23:26 |
openstackrecheck | openstack/python-heatclient change: https://review.openstack.org/76698 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error | 23:29 |
*** che-arne has quit IRC | 23:31 | |
*** mrodden has quit IRC | 23:35 | |
*** mwagner_notHere has quit IRC | 23:37 | |
openstackgerrit | Mark Maglana proposed a change to openstack/tempest: Add server boot time measurement https://review.openstack.org/81077 | 23:40 |
openstackgerrit | Dane LeBlanc proposed a change to openstack-dev/devstack: stack.sh fails for Cisco plugin with missing cisco_plugins.ini error https://review.openstack.org/81942 | 23:44 |
*** rockyg has joined #openstack-qa | 23:45 | |
*** mrodden has joined #openstack-qa | 23:48 | |
*** mmaglana has quit IRC | 23:51 | |
*** nati_ueno has quit IRC | 23:56 | |
*** nati_ueno has joined #openstack-qa | 23:57 | |
*** mriedem has joined #openstack-qa | 23:58 | |
*** rockyg has quit IRC | 23:58 | |
*** cyeoh_away is now known as cyeoh | 23:58 | |
mriedem | mtreinish: do you know if the cinder backup tests are run in the gate? looking at devstack the cinder backup service is started, but not seeing any special configuration in cinder.conf happening for cinder backups. looks like it defaults to use swift but i'd expect swift credentials setup in cinder.conf. | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!