Thursday, 2014-03-20

*** GMann has joined #openstack-qa00:06
*** atiwari has quit IRC00:10
marunjogo, sdague: so, the proposed fix is running gate checks now: https://review.openstack.org/#/c/80413/00:13
marunIt 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
marunhopefully 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 IRC00:15
jogomarun: cool, this may be worth bumping to the top of the queue00:18
*** dkorn has quit IRC00:19
*** mmaglana has quit IRC00:19
marunjogo: if the failure rate is still high, I would agree.00:19
jogomarun: lets move discussion to -infra00:20
marunjogo: ok00:21
*** Aimon has quit IRC00:22
*** masayukig has joined #openstack-qa00:24
*** mrodden has quit IRC00:25
*** matsuhashi has joined #openstack-qa00:29
*** salv-orlando has joined #openstack-qa00:35
*** rhsu has joined #openstack-qa00:43
*** mmaglana has joined #openstack-qa00:45
*** yamahata has joined #openstack-qa00:48
*** zhikunliu has joined #openstack-qa00:51
openstackgerritMasayuki Igawa proposed a change to openstack/tempest: Make add_remove_fixed_ip tests executable in Nova gate  https://review.openstack.org/8167700:53
*** andreaf has quit IRC00:54
openstackgerritxu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates'  https://review.openstack.org/8148300:56
openstackgerritEiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes  https://review.openstack.org/8147700:57
*** sthaha has joined #openstack-qa01:12
*** sthaha has joined #openstack-qa01:12
*** malini_afk is now known as malini01:22
openstackgerritSean Dague proposed a change to openstack/tempest: add test call to the request log  https://review.openstack.org/8165401:22
openstackgerritSean Dague proposed a change to openstack/tempest: simplify rest_client logging  https://review.openstack.org/8162001:22
*** nosnos has joined #openstack-qa01:25
*** ken1ohmichi has joined #openstack-qa01:27
*** samuelbercovici has quit IRC01:32
*** Aimon has joined #openstack-qa01:33
*** Aimon has quit IRC01:34
*** salv-orlando has quit IRC01:35
*** Aimon has joined #openstack-qa01:37
*** Aimon has quit IRC01:38
*** Aimon has joined #openstack-qa01:40
openstackrecheckConsole logs not available after 13:23s for check-tempest-dsvm-trove-full 81668,2,a68887301:40
openstackgerritYuiko Takada proposed a change to openstack/tempest: Verify quotas attributes through Nova os-quota-sets API  https://review.openstack.org/8143901:52
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify the response status of create delete Image  https://review.openstack.org/8168701:53
*** chandan_kumar has joined #openstack-qa01:53
*** jufeng has joined #openstack-qa01:55
*** shwetaap has joined #openstack-qa02:03
openstackrecheckopenstack/tempest change: https://review.openstack.org/81603 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error02:12
openstackgerritxu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates'  https://review.openstack.org/8148302:12
*** nosnos_ has joined #openstack-qa02:16
openstackgerritEdgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6  https://review.openstack.org/6731202:18
*** bill_az_ has quit IRC02:19
*** nosnos has quit IRC02:19
openstackgerritEdgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6  https://review.openstack.org/6731202:21
*** ken1ohmichi has quit IRC02:26
*** che-arne has quit IRC02:29
*** mwagner__ is now known as mwagner__zzzz02:37
openstackrecheckopenstack/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 error02:39
*** mriedem has quit IRC02:48
*** mriedem has joined #openstack-qa02:56
openstackrecheckopenstack/nova change: https://review.openstack.org/76751 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/125487202:58
*** shwetaap has quit IRC03:04
*** shwetaap has joined #openstack-qa03:04
openstackgerritYuiko Takada proposed a change to openstack/tempest: Verify quotas attributes through Nova os-quota-sets API  https://review.openstack.org/8143903:07
openstackgerritYuuichi Fujioka proposed a change to openstack-dev/devstack: fix typo in running opendaylight command  https://review.openstack.org/8169303:11
*** chandan_kumar has quit IRC03:22
*** harlowja is now known as harlowja_away03:23
*** matsuhashi has quit IRC03:47
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/78587 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/128352203:52
HenryGIs there way to get devstack's stack.sh to reclone all the repos and then stop (not start openstack)?04:02
*** mriedem has quit IRC04:04
*** mspreitz has joined #openstack-qa04:05
mspreitzIs all OK in the gate?04:06
*** malini is now known as malini_afk04:10
*** Akshik has joined #openstack-qa04:11
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Validate image metadata attributes of Nova APIs  https://review.openstack.org/8169904:12
*** saju_m has joined #openstack-qa04:12
*** shwetaap1 has joined #openstack-qa04:17
*** shwetaap has quit IRC04:17
*** sweston has joined #openstack-qa04:21
*** matsuhashi has joined #openstack-qa04:31
*** Madan has joined #openstack-qa04:33
openstackgerritYuiko Takada proposed a change to openstack/tempest: Verify "get quotas detail" API response attributes  https://review.openstack.org/8170104:35
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify detail_list flavor attributes of V2/V3 APIs  https://review.openstack.org/8054804:38
openstackrecheckopenstack/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/125487204:39
*** yfujioka has joined #openstack-qa04:40
*** mmaglana has quit IRC04:44
*** mmaglana has joined #openstack-qa04:45
*** mmaglana has quit IRC04:49
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs  https://review.openstack.org/8054304:52
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify "create a server" API response attributes  https://review.openstack.org/7751704:52
*** Akshik_ has joined #openstack-qa04:56
*** Akshik has quit IRC05:00
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs  https://review.openstack.org/8054305:01
*** yfujioka has quit IRC05:02
openstackgerritxu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates'  https://review.openstack.org/8148305:02
*** chandan_kumar has joined #openstack-qa05:10
*** mspreitz has quit IRC05:16
*** chandan_kumar has quit IRC05:17
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify get flavor attributes of Nova V2/V3 APIs  https://review.openstack.org/8054305:19
*** nosnos has joined #openstack-qa05:19
*** nosnos_ has quit IRC05:23
openstackgerritA change was merged to openstack/tempest: Add missing client names to T102 hacking check  https://review.openstack.org/8081405:24
*** amotoki has joined #openstack-qa05:29
openstackrecheckopenstack-dev/pbr change: https://review.openstack.org/81322 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/129192605:33
*** vkozhukalov_ has joined #openstack-qa05:33
*** giulivo has joined #openstack-qa05:33
openstackgerritDaisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 1  https://review.openstack.org/8170305:40
openstackgerritDaisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 2  https://review.openstack.org/8170405:41
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Verify "create a server" API response attributes  https://review.openstack.org/7751705:42
openstackgerritDaisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 3  https://review.openstack.org/8170605:42
openstackgerritDaisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 4  https://review.openstack.org/8170705:43
raiesgiulivo: hi05:45
raiesI need some help on instance access05:45
raiesI created an instance a created network (created by me) in devstack05:46
*** nosnos_ has joined #openstack-qa05:49
*** nosnos has quit IRC05:51
raiesIn devstack I created VM using a self created network (created by me) and a created security group ping and ssh enabled05:59
raiesstill I am not able to ping this VM while it is in running state05:59
raieswhat exatra settings are required there ??05:59
raiesanyone there ^^06:00
raiesI am using devstack06:01
*** saju_m has quit IRC06:02
*** nati_ueno has joined #openstack-qa06:03
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Verify list Image attributes through Nova V2 API  https://review.openstack.org/7890806:04
*** saju_m has joined #openstack-qa06:04
*** saju_m has quit IRC06:07
*** saju_m has joined #openstack-qa06:08
*** vkozhukalov_ has quit IRC06:19
*** matsuhashi has quit IRC06:28
*** Madan has quit IRC06:29
*** matsuhashi has joined #openstack-qa06:30
openstackgerritA change was merged to openstack/tempest: Validate get limits attributes of Nova V2 API  https://review.openstack.org/8115106:33
*** saju_m has quit IRC06:35
openstackgerritEiichi Aikawa proposed a change to openstack/tempest: Verify "get version" API response attributes  https://review.openstack.org/8093606:37
openstackrecheckopenstack/tempest change: https://review.openstack.org/80890 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error06:37
Akshik_do we have any approval mechanism for creating and modifying a instance06:44
openstackgerritEiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes  https://review.openstack.org/8147706:49
*** nati_ueno has quit IRC06:51
*** masayukig has quit IRC06:53
*** sweston has quit IRC07:02
*** jlibosva has joined #openstack-qa07:12
openstackrecheckopenstack/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/125487207:12
*** jlibosva has quit IRC07:16
*** rhsu has quit IRC07:16
*** psharma has joined #openstack-qa07:18
*** Akshik_ has quit IRC07:19
*** Akshik_ has joined #openstack-qa07:19
*** Akshik has joined #openstack-qa07:22
*** Akshik_ has quit IRC07:22
*** Madan has joined #openstack-qa07:23
openstackgerritJerry Cai proposed a change to openstack/tempest: Fix to create volume with the image size if imageRef set  https://review.openstack.org/8147107:25
*** lukego has joined #openstack-qa07:27
openstackgerritMarc Koderer proposed a change to openstack/tempest: Add error handling if testscenarios aren't supported  https://review.openstack.org/8172207:28
*** nati_ueno has joined #openstack-qa07:39
*** rhsu has joined #openstack-qa07:41
*** tkammer has joined #openstack-qa07:41
*** jlibosva has joined #openstack-qa07:43
openstackgerritxu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates'  https://review.openstack.org/8148307:45
*** vkozhukalov_ has joined #openstack-qa07:49
*** rhsu has quit IRC07:49
*** nmagnezi has joined #openstack-qa07:55
*** saju_m has joined #openstack-qa07:56
*** andreaf has joined #openstack-qa07:56
*** nmagnezi has quit IRC08:25
*** Akshik_ has joined #openstack-qa08:25
*** Akshik has quit IRC08:29
*** nosnos has joined #openstack-qa08:33
*** nmagnezi has joined #openstack-qa08:35
*** nosnos_ has quit IRC08:37
*** nmagnezi has quit IRC08:42
*** jhenner1 has joined #openstack-qa08:43
andreafmkoderer: ping08:43
*** jhenner has quit IRC08:45
*** nati_ueno has quit IRC08:47
*** Akshik_ has quit IRC08:48
*** morganfainberg is now known as morganfainberg_Z08:54
openstackrecheckopenstack/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/125487208:55
*** jordanP has joined #openstack-qa08:56
openstackgerritAlexander Bochkarev proposed a change to openstack/tempest: Add compute pollsters tests for ceilometer  https://review.openstack.org/8096508:57
*** nati_ueno has joined #openstack-qa08:58
*** nati_uen_ has joined #openstack-qa08:59
mkodererandreaf: I am in many meetings today.. is it about my -1?09:01
*** yassine has joined #openstack-qa09:03
*** nati_ueno has quit IRC09:03
andreafmkoderer: yes, but I think I found a way to split my change in a nicer way, so I'll follow your advice09:04
*** Akshik has joined #openstack-qa09:06
openstackrecheckopenstack/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/127390809:06
*** Akshik_ has joined #openstack-qa09:07
openstackgerritSean Dague proposed a change to openstack/tempest: add test call to the request log  https://review.openstack.org/8165409:08
*** Madan has quit IRC09:09
*** Akshik has quit IRC09:11
*** jhenner1 has quit IRC09:12
mkodererandreaf: great thank you09:15
openstackgerritJerry Cai proposed a change to openstack/tempest: Fix to create volume with the image size if imageRef set  https://review.openstack.org/8147109:18
openstackgerritxu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'list_aggregates'  https://review.openstack.org/8148309:22
*** jhenner has joined #openstack-qa09:26
*** Madan has joined #openstack-qa09:27
*** safchain has joined #openstack-qa09:27
*** safchain has quit IRC09:37
*** psedlak_clone is now known as psedlak09:39
openstackrecheckopenstack/python-swiftclient change: https://review.openstack.org/81479 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/128352209:44
openstackgerritMh Raies proposed a change to openstack/tempest: Network fwaas API test  https://review.openstack.org/7885709:49
*** safchain has joined #openstack-qa09:50
openstackgerritGhanshyam Mann proposed a change to openstack/tempest: Check attributes of image meta item Nova APIs  https://review.openstack.org/8174809:55
*** nosnos has quit IRC09:59
*** yamahata has quit IRC10:02
*** jufeng has quit IRC10:03
*** nosnos has joined #openstack-qa10:09
mkoderercyeoh: if you have time for a review.. this fixes the pep8 thing properly: https://review.openstack.org/#/c/80926/10:11
*** rwsu has quit IRC10:18
openstackgerritSean Dague proposed a change to openstack/tempest: add test call to the request log  https://review.openstack.org/8165410:18
*** safchain has quit IRC10:18
*** safchain has joined #openstack-qa10:20
openstackgerritAleksandr Gordeev proposed a change to openstack-dev/devstack: Remove any lib/baremetal usage for ironic deploy  https://review.openstack.org/8095810:23
openstackgerritEiichi Aikawa proposed a change to openstack/tempest: Verify "get version" API response attributes  https://review.openstack.org/8093610:26
openstackrecheckConsole logs not available after 13:58s for gate-neutron-python27 78861,1,9787a2c10:26
*** Jaq1 has joined #openstack-qa10:29
openstackrecheckopenstack/nova change: https://review.openstack.org/74622 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/125489010:31
openstackgerritafazekas proposed a change to openstack/tempest: unskip test_integration_1  https://review.openstack.org/8176210:31
*** afazekas has joined #openstack-qa10:31
*** rwsu has joined #openstack-qa10:33
*** Jaq1 has left #openstack-qa10:34
openstackgerritEiichi Aikawa proposed a change to openstack/tempest: Verify certificate API response attributes  https://review.openstack.org/8147710:35
*** jacqm has joined #openstack-qa10:36
*** nati_uen_ has quit IRC10:43
*** nati_ueno has joined #openstack-qa10:44
*** tosky has joined #openstack-qa10:44
*** nati_ueno has quit IRC10:48
openstackgerritEiichi Aikawa proposed a change to openstack/tempest: Verify tenant usages API response attributes  https://review.openstack.org/8176610:51
*** yamahata has joined #openstack-qa10:53
*** yassine has quit IRC10:56
*** chandankumar_ has quit IRC11:00
*** chandankumar has joined #openstack-qa11:00
*** matsuhashi has quit IRC11:01
openstackgerritPranali Deore proposed a change to openstack/tempest: Added testcase for glance  https://review.openstack.org/8176911:08
openstackgerritafazekas proposed a change to openstack/tempest: Fix test_verify_created_server_ephemeral_disk  https://review.openstack.org/8062211:08
openstackgerritDmitry Tantsur proposed a change to openstack-dev/devstack: Configure SELinux to allow SSH to listen on $IRONIC_VM_SSH_PORT  https://review.openstack.org/8177011:11
openstackgerritDmitry Tantsur proposed a change to openstack-dev/devstack: Configure SELinux to allow SSH to listen on $IRONIC_VM_SSH_PORT  https://review.openstack.org/8177011:13
*** devvesa has joined #openstack-qa11:17
*** malini_afk is now known as malini11:19
openstackgerritPranali Deore proposed a change to openstack/tempest: Added testcase for glance  https://review.openstack.org/8176911:20
openstackrecheckopenstack/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 error11:23
*** matsuhashi has joined #openstack-qa11:24
*** zhikunliu has quit IRC11:26
openstackgerritMh Raies proposed a change to openstack/tempest: API test for 'create server with scheduler hints'  https://review.openstack.org/7990811:27
*** tkammer has quit IRC11:34
*** markmcclain has joined #openstack-qa11:35
*** markmcclain1 has joined #openstack-qa11:35
*** GMann has quit IRC11:35
*** markmcclain1 has quit IRC11:35
openstackgerritafazekas proposed a change to openstack/tempest: Account failure to the tearDown instead of setUp  https://review.openstack.org/5431811:36
*** markmcclain1 has joined #openstack-qa11:36
*** markmcclain1 has quit IRC11:37
*** markmcclain has quit IRC11:39
openstackgerritBartosz Górski proposed a change to openstack-dev/devstack: Adds support for multi-region  https://review.openstack.org/7931011:39
*** markmcclain has joined #openstack-qa11:41
openstackgerritSean Dague proposed a change to openstack/tempest: drop unused params for _log_request  https://review.openstack.org/8177611:42
openstackgerritSean Dague proposed a change to openstack/tempest: move the request_id logging into the expected place  https://review.openstack.org/8177711:42
openstackgerritSean Dague proposed a change to openstack/tempest: add request timing  https://review.openstack.org/8177811:42
*** safchain_ has joined #openstack-qa11:42
*** safchain has quit IRC11:42
*** salv-orlando has joined #openstack-qa11:47
*** tkammer has joined #openstack-qa11:50
*** rfolco has joined #openstack-qa11:53
nithyagafazekas: 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
nithyagafazekas: In that case, would it be ok to go ahead without https://blueprints.launchpad.net/tempest/+spec/ssh-auth-strategy?12:05
nithyagand submit the test with the current ssh verification using adminPass?12:07
*** matsuhashi has quit IRC12:08
*** saju_m has quit IRC12:09
openstackrecheckopenstack/horizon change: https://review.openstack.org/79736 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/128352212:11
*** matsuhashi has joined #openstack-qa12:12
*** yassine has joined #openstack-qa12:12
*** matsuhashi has quit IRC12:18
*** malini is now known as malini_afk12:21
*** matsuhashi has joined #openstack-qa12:24
*** saju_m has joined #openstack-qa12:27
openstackgerritMh Raies proposed a change to openstack/tempest: Network fwaas API test  https://review.openstack.org/7885712:27
openstackrecheckConsole logs not available after 14:19s for check-grenade-dsvm-partial-ncpu 81498,2,63e68bd12:28
openstackgerritJerry Cai proposed a change to openstack/tempest: Fix to create volume with the image size if imageRef set  https://review.openstack.org/8147112:32
*** jordanP has quit IRC12:34
*** weshay has joined #openstack-qa12:39
*** devvesa has quit IRC12:40
*** sthaha has quit IRC12:41
*** vkozhukalov_ has quit IRC12:42
*** sthaha has joined #openstack-qa12:43
*** nosnos has quit IRC12:46
*** matsuhashi has quit IRC12:47
*** jordanP has joined #openstack-qa12:49
*** aysyd has joined #openstack-qa12:51
*** saju_m has quit IRC12:53
*** vkozhukalov has joined #openstack-qa12:58
*** gfidente has joined #openstack-qa13:02
*** mwagner__zzzz is now known as mwagner_13:04
*** saju_m has joined #openstack-qa13:07
*** Madan has quit IRC13:10
*** zul has quit IRC13:12
*** adalbas has joined #openstack-qa13:14
*** zul has joined #openstack-qa13:15
*** mriedem has joined #openstack-qa13:15
openstackgerritAleksandr Gordeev proposed a change to openstack-dev/devstack: Configure ssh server properly for ironic  https://review.openstack.org/8180713:16
*** psharma has quit IRC13:16
Akshik_what is the format to specify the network id in heat template13:17
*** jordanP has quit IRC13:17
*** jordanP has joined #openstack-qa13:21
*** saju_m has quit IRC13:23
*** sthaha has quit IRC13:23
*** dkranz has joined #openstack-qa13:31
openstackrecheckopenstack/keystone change: https://review.openstack.org/81385 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error13:33
*** sthaha has joined #openstack-qa13:35
*** mkollaro has joined #openstack-qa13:36
*** salv-orlando has quit IRC13:38
*** saju_m has joined #openstack-qa13:39
*** andreaf has quit IRC13:46
*** yamahata has quit IRC13:48
*** saju_m has quit IRC13:48
*** yamahata has joined #openstack-qa13:49
*** tkammer has quit IRC13:53
*** malini_afk is now known as malini13:58
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Move to the python-saharaclient  https://review.openstack.org/8108514:01
*** zhikunliu has joined #openstack-qa14:02
*** mrodden has joined #openstack-qa14:02
*** andreaf has joined #openstack-qa14:05
*** tkammer has joined #openstack-qa14:06
*** jordan_ has joined #openstack-qa14:07
*** devvesa has joined #openstack-qa14:10
*** jordanP has quit IRC14:10
*** bill_az_ has joined #openstack-qa14:12
*** Akshik_ has quit IRC14:14
*** nati_ueno has joined #openstack-qa14:19
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Cleanup sahara cli tests  https://review.openstack.org/8181914:25
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Add sahara edp cli commands tests  https://review.openstack.org/8182014:25
*** gfidente has quit IRC14:30
openstackgerritIvan Berezovskiy proposed a change to openstack-infra/devstack-gate: Update Readme to setup jenkins accout  https://review.openstack.org/8182314:37
openstackgerritA change was merged to openstack-infra/devstack-gate: Temporary workaround for Time out errors from libvirt  https://review.openstack.org/7981614:39
openstackgerritDane LeBlanc proposed a change to openstack/tempest: Add network API test to create/update a port with 2 IP addresses  https://review.openstack.org/6754714:42
*** salv-orlando has joined #openstack-qa14:45
openstackrecheckopenstack/neutron change: https://review.openstack.org/81248 failed gate-tempest-dsvm-neutron, gate-tempest-dsvm-neutron in the gate queue with an unrecognized error14:46
openstackgerritSean Dague proposed a change to openstack-infra/elastic-recheck: add match for lbaas failure  https://review.openstack.org/8182514:48
*** BobBall has joined #openstack-qa14:51
*** nati_ueno has quit IRC14:52
openstackgerritDmitry Tantsur proposed a change to openstack-dev/devstack: Configure SELinux to allow SSH to listen on $IRONIC_VM_SSH_PORT  https://review.openstack.org/8177014:54
sdaguemriedem / mtreinish - https://review.openstack.org/#/c/81825/14:55
*** mwagner is now known as mwagner_notHere14:55
mkoderersdague: https://review.openstack.org/#/c/80926/ should be fine now14:56
*** otherwiseguy has joined #openstack-qa14:56
mriedemsdague: looking14:57
*** atiwari has joined #openstack-qa14:57
*** nati_ueno has joined #openstack-qa15:03
mtreinishsdague: +A15:03
*** rdekel has joined #openstack-qa15:04
openstackgerritA change was merged to openstack-infra/elastic-recheck: add match for lbaas failure  https://review.openstack.org/8182515:05
*** shwetaap1 has quit IRC15:05
*** mwagner_ has quit IRC15:06
*** jordan_ has quit IRC15:08
*** openstackrecheck has quit IRC15:11
*** openstackrecheck has joined #openstack-qa15:11
openstackgerritenikanorov proposed a change to openstack/tempest: Skip loadbalancer basic scenario test  https://review.openstack.org/8183115:14
*** openstackrecheck has quit IRC15:15
*** Aimon has quit IRC15:16
*** yfried1 has quit IRC15:16
*** shwetaap has joined #openstack-qa15:17
*** tosky has quit IRC15:23
*** yfried has joined #openstack-qa15:24
*** tosky has joined #openstack-qa15:27
*** krtaylor has quit IRC15:27
andreafmtreinish: ping15:31
mtreinishandreaf: hi15:32
openstackgerritafazekas proposed a change to openstack/tempest: Enable instance validation with the server_actions v2  https://review.openstack.org/8183415:32
andreafmtreinish: I'm changing Credentials unit tests to work with the config fixture15:32
*** jordanP has joined #openstack-qa15:33
andreafmtreinish: I run into a problem were running unit tests in parallel breaks the tests15:33
mtreinishooh that's a sign of unit test maturity15:33
mtreinishthat means we've been making progress this cycle :)15:33
andreafmtreinish: I have two fixtures, one setting auth_version=v2 and one v3. Any idea how to work around this?15:33
mtreinishandreaf: I would make the tests that use the v2 fixture be the parent class for the test class that uses v315:34
mtreinishthen add a lock fixture to the beginning of setUp()15:34
mtreinishthat way they're guarenteed not to run at the same time15:35
andreafmtreinish: ok the parent / child relationship is already there, I'll try the lock, thanks!15:36
mtreinishandreaf: for example: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_aggregates.py#n14315:37
*** kgriffs_afk is now known as kgriffs15:37
andreafmtreinish: self.useFixture(fixtures.LockFixture('auth_version')) ?15:38
andreafmtreinish: the name is just a name right?15:38
mtreinishandreaf: yeah15:38
mtreinishthat should work fine15:38
*** Aimon has joined #openstack-qa15:40
*** tosky_ has joined #openstack-qa15:40
*** tosky has quit IRC15:40
andreafmtreinish: trying....15:40
andreafmtreinish: RequiredOptError: value required for option: lock_path15:42
andreafmtreinish: lock_path is defined in tempest.conf15:43
*** tkammer has quit IRC15:43
*** otherwiseguy has quit IRC15:43
mtreinishhah, oops yeah I guess we should figure out a way to do that15:43
andreafmtreinish: but tempest.conf is not there.... maybe it'll work if I put it in the fake config15:43
mtreinishit will15:43
mtreinishor at least it should15:43
*** rdekel has quit IRC15:44
mtreinishwe'll have to think about setting that as a default in the base test class at some point15:44
mtreinishbut the config fixture isn't used everywhere so that can come later15:44
mtreinishandreaf: I would just make a tmpdir using the lib and make sure you clean it up after the test15:45
andreafright the problem with having the config in the fake config is that it's an hardcoded path15:45
andreafmtreinish: good idea15:45
andreafmtreinish: do you mean "import tempfile"15:47
mtreinishyeah and then using something like: tempfile.mkdtemp(prefix='tempest-unit-test-locks')15:48
mtreinishI did basically the same thing in test_wrappers.py15:49
*** rossella_s has quit IRC15:54
andreafmtreinish: ok, but I would do it in the fixture in this case as I need to setup the fake config with the tmp dir15:55
andreafmtreinish: so I'd override the fixture cleanup()  instead of using addCleanup()15:55
*** rossella_s has joined #openstack-qa15:56
mtreinishandreaf: I think I'd have to see your code, I don't have enough context15:56
*** zhikunliu has quit IRC15:58
mtreinishandreaf: this is what I was thinking about though: http://paste.openstack.org/show/73917/15:58
*** devvesa has quit IRC15:58
mtreinishbut I maybe missing something I didn't actually test it15:59
mtreinishoh you could bake that into the config fixture itself15:59
andreafmtreinish: I was doing something like http://paste.openstack.org/show/73918/16:00
mtreinishyeah that's probably the better way to do it16:01
mtreinishthat way if locking is ever needed the lock path is set16:02
mtreinishthe only downside is that it's making a tempdir for every test that needs config16:02
andreafmtreinish: I was just thinking that... so it's not actually locking as the tmp dir is different?16:03
andreafthe lock dir I mean16:03
mtreinishhmm yeah that's right16:03
sdaguemtreinish: you mind giving this series a look - https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:rest_client_logging,n,z16:04
andreafmtreinish: I just created a few 100s tmp dirs on my workstation16:04
mtreinishandreaf: I'm gonna run to lunch I'll be back in a bit16:04
sdaguemost of the test fails are upstream races and unrelated16:04
sdagueand I'd like your feedback on those in case we should restack them differently16:04
*** devvesa has joined #openstack-qa16:06
*** otherwiseguy has joined #openstack-qa16:07
openstackgerritsahid proposed a change to openstack-dev/devstack: Fix bug ext2: parent directory not found with guestfs  https://review.openstack.org/7957716:08
*** sthaha has quit IRC16:16
openstackgerritJerry Cai proposed a change to openstack/tempest: Add "capabilities:" to multiple_backends extra-spec key  https://review.openstack.org/8184216:17
*** mrodden has quit IRC16:17
*** yfried has quit IRC16:20
*** nithyag has quit IRC16:24
openstackgerritSergey Lukjanov proposed a change to openstack/tempest: Add sahara edp cli commands tests  https://review.openstack.org/8182016:27
openstackgerritSean Dague proposed a change to openstack-infra/devstack-gate: wip: DO NOT MERGE gather dstat data on ceilometer free env  https://review.openstack.org/8184316:27
*** yfried has joined #openstack-qa16:31
openstackgerritIvan Berezovskiy proposed a change to openstack-infra/devstack-gate: Update Readme to setup jenkins accout  https://review.openstack.org/8182316:31
*** mrodden has joined #openstack-qa16:31
*** rhsu has joined #openstack-qa16:33
*** dims has quit IRC16:37
*** mmaglana has joined #openstack-qa16:43
*** YorikSar has left #openstack-qa16:44
jogosdague: I see the gate isn't any better  today16:45
openstackgerritenikanorov proposed a change to openstack/tempest: Skip loadbalancer basic scenario test  https://review.openstack.org/8183116:48
*** dims has joined #openstack-qa16:49
openstackgerritafazekas proposed a change to openstack/tempest: Translate xml server tenantId /userId  https://review.openstack.org/8184716:50
*** otherwiseguy has quit IRC16:52
openstackgerritafazekas proposed a change to openstack/tempest: Enable instance validation with the server_actions v2  https://review.openstack.org/8183416:56
openstackgerritBob Ball proposed a change to openstack-dev/devstack: XenAPI:  Find location for XenAPI plugins  https://review.openstack.org/8184916:58
*** kgriffs is now known as kgriffs_afk17:03
openstackgerritAleksandr Gordeev proposed a change to openstack-dev/devstack: Configure ssh server properly for ironic  https://review.openstack.org/8180717:03
*** mkollaro has quit IRC17:04
*** devvesa has quit IRC17:05
*** mwagner_ has joined #openstack-qa17:07
*** nati_ueno has quit IRC17:08
*** nati_ueno has joined #openstack-qa17:08
*** tkammer has joined #openstack-qa17:08
*** nati_ueno has quit IRC17:12
*** vkozhukalov has quit IRC17:17
*** jordanP has quit IRC17:20
andreafmtreinish: ping17:21
*** mkollaro has joined #openstack-qa17:22
*** otherwiseguy has joined #openstack-qa17:23
*** morganfainberg_Z is now known as morganfainberg17:23
*** yassine has quit IRC17:23
*** krtaylor has joined #openstack-qa17:27
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1295261  https://review.openstack.org/8185517:30
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Fix elasticsearch URL for gate queue  https://review.openstack.org/8185617:30
*** afazekas has quit IRC17:30
*** tkammer has quit IRC17:37
mtreinishandreaf: hey17:52
*** mkollaro has quit IRC17:55
openstackgerritA change was merged to openstack-infra/elastic-recheck: Fix elasticsearch URL for gate queue  https://review.openstack.org/8185617:55
*** safchain_ has quit IRC17:55
mtreinishsdague: ok I'll take a look in a bit17:57
andreafmtreinish: I'm not managing to get locking working properly18:01
andreafmtreinish: 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 help18:02
mtreinishandreaf: sure I can give it a shot18:02
*** mriedem has quit IRC18:02
andreafmtreinish: thanks18:03
*** openstackrecheck has joined #openstack-qa18:04
*** mkollaro has joined #openstack-qa18:05
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1294971  https://review.openstack.org/8187118:06
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Define V3 Credentials  https://review.openstack.org/8187218:07
openstackgerritAndrea Frittoli  proposed a change to openstack/tempest: Defines a Credentials class  https://review.openstack.org/7438718:07
andreafmtreinish: https://review.openstack.org/8187218:07
*** mriedem has joined #openstack-qa18:14
*** otherwiseguy has quit IRC18:16
openstackrecheckopenstack/nova change: https://review.openstack.org/79195 failed gate-tempest-dsvm-postgres-full in the gate queue with an unrecognized error18:18
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1295261  https://review.openstack.org/8185518:19
*** yfried has quit IRC18:20
*** dkorn has joined #openstack-qa18:21
*** stack555 has joined #openstack-qa18:21
stack555Hello 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 all18:22
stack555Because of this my test case is run by jenkins all the time18:22
stack555Is there a way I can get around this?18:22
*** yfried has joined #openstack-qa18:23
*** packet has joined #openstack-qa18:26
*** jlibosva has quit IRC18:26
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1294874  https://review.openstack.org/8164918:31
*** markmcclain has quit IRC18:34
*** openstackrecheck has quit IRC18:34
*** openstackrecheck has joined #openstack-qa18:34
*** arosen has joined #openstack-qa18:36
*** mkollaro has quit IRC18:36
mtreinishmriedem: 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 IRC19:05
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1294971  https://review.openstack.org/8187119:09
mriedemmtreinish: i don't remember that19:12
mtreinishmriedem: ok, I think I figured out something anyway19:13
mtreinishandreaf: http://paste.openstack.org/show/73936/19:13
mtreinishI confirmed the locking works correctly there19:14
mtreinishbut the same 3 unit tests still fail19:14
*** vkozhukalov has joined #openstack-qa19:15
mtreinishandreaf: I'm going to push the lock fixture usage thing out as a separate patch19:16
andreafmtreinish: thanks, that's quite useful19:16
*** kgriffs_afk is now known as kgriffs19:17
andreafmtreinish: 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 missed19:18
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: provide more specific feedback to query authors  https://review.openstack.org/8189119:18
mtreinishandreaf: well that's why I added the lock to the tenant_isolation tests too19:18
mtreinishbecause you were using the V2 fixture there19:18
openstackgerritDolph Mathews proposed a change to openstack-infra/elastic-recheck: add query for bug 1295261  https://review.openstack.org/8185519:19
mtreinishbut when I added debug to the auth methods I saw that get valid stuff was getting called twice19:19
mtreinishand the second time had invalid results19:19
andreafmtreinish: how do you get a log file written by the unit test?19:20
andreafmtreinish: I tried adding logging statement, but I suppose that if the config is not there it won't write to the log file19:21
mtreinishdunno I've never tried19:21
mtreinishI just added prints19:21
andreafmtreinish: oh, ok19:21
mtreinishI'm sure there is a way19:22
andreafmtreinish: thanks for the help, it's really appreciated19:22
andreafmtreinish: I'm splitting the multi-auth patches down further, and in the process I'm also rebasing them, so it's taking me a while19:23
mtreinishyeah it's a big refactor I didn't expect it to be super fast19:24
andreafmtreinsh: I've got 7 patches I'm working on. And there's still quite some other stuff to be done19:24
openstackrecheckConsole logs not available after 13:24s for check-tempest-dsvm-full 54318,8,57a986f19:25
andreafmtreinish: btw. I won't make it to the meeting tonight (it's 10pm for me)19:25
openstackrecheckopenstack/python-heatclient change: https://review.openstack.org/75383 failed gate-tempest-dsvm-neutron-heat-slow in the gate queue with an unrecognized error19:26
andreafmtreinish: but you already have my updates on the multi auth stuff19:26
mtreinishandreaf: yeah that's fine this bp is one of the more visable ones too19:27
andreafmtreinish: 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 though19:28
mtreinishandreaf: yeah that's a lot easier to review19:29
mtreinishplus over half of it is unit testing19:29
mtreinishwhich is a really good thing19:29
andreafmtreinish: the last update I have is that I defined a couple of bp in the bp repo, both related to the multi-auth stuff19:29
andreafmtreinish: yep unit tests can be tricky but they are really worth the effort19:30
andreafmtreinish: ok going home now, have a good day19:30
*** kgriffs is now known as kgriffs_afk19:31
*** andreaf has quit IRC19:31
*** andreaf has joined #openstack-qa19:32
*** packet has quit IRC19:33
*** andreaf has quit IRC19:33
*** andreaf has joined #openstack-qa19:34
*** packet has joined #openstack-qa19:34
*** malini is now known as malini_afk19:34
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests  https://review.openstack.org/8189619:35
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests  https://review.openstack.org/8189619:36
*** dkorn has quit IRC19:37
*** andreaf_ has joined #openstack-qa19:45
*** andreaf has quit IRC19:48
*** jamielennox is now known as jamielennox|away19:48
*** andreaf has joined #openstack-qa19:52
*** andreaf_ has quit IRC19:53
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Add config fixture support to unit tests  https://review.openstack.org/8189619:57
*** otherwiseguy has joined #openstack-qa19:58
sdaguejogo: I don't think that's true, I think it's just taking time to clear19:59
sdaguemtreinish: so any thoughts on the log_request approach?20:04
*** andreaf has quit IRC20:08
openstackrecheckopenstack/nova change: https://review.openstack.org/53775 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/129192620:12
*** kgriffs_afk is now known as kgriffs20:16
jogosdague: agreed, I didn't realize the top two bugs were fixed early this AM20:21
*** kgriffs is now known as kgriffs_afk20:25
dkranzmtreinish: Seems that scenario tests always run with tenant isolation. Isn't that a bug?20:25
openstackgerritPablo Andres Fuente proposed a change to openstack/tempest: Add test for OS::Neutron::* Heat resource  https://review.openstack.org/8190920:29
*** kgriffs_afk is now known as kgriffs20:32
mriedemjogo: i'll let you +A https://review.openstack.org/#/c/81855/ if you're good with it now20:36
*** rwsu has quit IRC20:37
jogomriedem: done20:40
*** mkoderer has quit IRC20:41
*** che-arne has joined #openstack-qa20:41
sdaguedtroyer: thoughts on - https://review.openstack.org/#/c/81663/ ?20:44
openstackgerritA change was merged to openstack-infra/elastic-recheck: add query for bug 1295261  https://review.openstack.org/8185520:45
mtreinishdkranz: no all the tests always run with tenant isolation20:45
sdaguemtreinish: I think the question was is there a way to turn them off20:45
mtreinishsdague: just for the scenario tests? or in general20:45
mtreinishbecause there is a config flag20:46
*** dims has quit IRC20:46
mtreinishbut you can't run in parallel that way20:46
sdagueright, but that works right for scenario as well?20:46
mtreinishsdague: it should20:46
sdaguethat's how I interpretted his question20:46
mtreinishbut there might be a bug I dunno20:46
*** mwagner_ has quit IRC20:46
*** openstackrecheck has quit IRC20:46
*** openstackrecheck has joined #openstack-qa20:47
dansmithhttp://logs.openstack.org/62/69262/11/check/check-tempest-dsvm-full/b52c95d/logs/tempest_conf.txt.gz20:47
dansmithresize=false20:47
dansmithare we not testing resizes upstream anymore? or is it only enabled on one test?20:47
mtreinishsdague: on the logging one thing is that I think that having the resp headers and body is sometimes very useful for debugging20:47
mtreinishI can think of a couple of cases where I needed that to help debug a failure (mostly api bugs)20:48
sdaguemtreinish: those aren't on the server side?20:48
sdaguedansmith: it looks like we aren't testing it again20:49
dansmithsdague: unintentionally I assume?20:50
sdagueI assume20:50
dansmithlike, the default changed in tempest or something?20:50
sdagueyeh, or a refactor in devstack lost the flag20:50
sdagueit should be set from devstack20:50
dansmithI was about to smack the vmware guys around because resize on vmware is totally broken and they're not testing it,20:50
dansmithbut apparently nobody is20:50
sdagueyeh20:50
dansmithwhich is :((20:50
sdagueit looks like we are only running the negative tests20:50
sdagueagreed20:51
mtreinishsdague: I thought the request gets logged for some services but I'm not sure about the reply. Also not all the services log that20:51
dansmithwhat does that mean?20:51
dansmith(negative tests)20:51
sdaguethe attempts to do resize badly20:51
sdaguewhich nova prevents20:51
dansmithokay20:51
sdaguelet me go make a devstack patch to turn that back on20:51
dansmiththat would be awesome, thanks20:52
mtreinishsdague: shouldn't we also block the negative tests if resize isn't enabled?20:52
dansmithand I'll prepare to look at what is totally broken as a result :(20:52
*** marun has quit IRC20:53
*** jamielennox|away is now known as jamielennox20:55
openstackgerritSean Dague proposed a change to openstack-dev/devstack: turn on resize feature  https://review.openstack.org/8191720:55
mtreinishsdague: 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 there20:55
mtreinishnot that that's a bad thing20:55
mtreinishbut it's more verbose than what's there now20:56
sdagueyeh, I don't think that's an issue20:56
sdaguewe were at info before20:56
sdague_log_request and _log_response were both at info20:56
dtroyersdague: looks good...20:57
mtreinishfor request and response really? I thought it was just the req-id20:57
sdaguemtreinish: have you looked at our logs? :)20:57
*** krtaylor has quit IRC20:57
mtreinishI'm just going from memory I guess I should do that20:57
sdagueso 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
sdaguelike the fact that we dump out the config N times20:59
mtreinishsdague: oh I see where I was getting confused we'd log request: method url  and response: status code at info20:59
mtreinishand the headers and body at debug20:59
mtreinishyeah that config thing is weird21:00
*** rockyg has joined #openstack-qa21:01
mtreinishI 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 gate21:01
mtreinishand honestly for the most part we definitely are way too verbose21:01
*** dims has joined #openstack-qa21:01
mtreinishI think it was mriedem who told me doesn't ever open a tempest log because it kills his browser21:02
sdagueyeh21:02
*** ken1ohmichi has joined #openstack-qa21:02
sdagueand it wasn't really useful21:02
mriedemit was me21:02
mriedemsome joker just told me to pull the file down21:03
maurosrjaypipes: I guess you can answer this better than me https://bugs.launchpad.net/tempest/+bug/127825521:03
maurosrhad to go home now brb21:03
sdague2014-03-20 01:18:44.457 1979 INFO tempest.common.rest_client [-] Response Status: 20021:03
sdaguesuper useful21:03
mtreinishmaurosr: https://review.openstack.org/#/c/62101/21:03
mtreinishsdague: yeah I think the info changes are fine but maybe keep the debug level for the rest client around21:04
*** rwsu has joined #openstack-qa21:04
mtreinishmaurosr: that is related I think I just haven't ever found the time to fix it21:04
*** kgriffs is now known as kgriffs_afk21:05
sdagueok, what do you want in the debug, because my patch series stripped out the params at one point21:05
*** rfolco has quit IRC21:05
mtreinishsdague: just the resp body and headers (and maybe req too)21:05
sdagueso that's going to make it as enormous as it is today21:06
sdaguewhich is too big for anything but chrome (2.1 MB compressed currently)21:06
mtreinishsdague: the info level will have 1/2 as much because you're combing the response and req levels :)21:06
mtreinishsdague: ok, I wish there was a way to just log that on failures21:07
*** aysyd has quit IRC21:07
*** jaypipes has quit IRC21:07
mtreinishoh maybe just on the service clients because the auth stuff really isn't needed to be logged21:07
*** mdenny has quit IRC21:07
*** mdenny has joined #openstack-qa21:07
mtreinishanyway your change is fine I can always iterate on having useful debug info that's not a huge mess21:07
sdaguethat'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 out21:09
*** lukego has quit IRC21:09
sdaguelets see if it's a huge problem or not21:09
sdagueto not have it21:09
mtreinishI 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_89321:10
sdaguealso, logging keystone tokens, never useful :)21:10
mtreinishsorry wrong link21:10
sdaguemtreinish: agreed21:10
mtreinishthis 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_85621:10
sdagueI was going to go whack other things when I get a chance21:10
mtreinishsee that stuff is dumb to log except for when a cli test falls apart21:11
sdagueyep21:11
sdague100% agree21:11
sdaguemtreinish: you want me to collapse the patches at all?21:11
sdagueI have one more change on the _find_caller, as I figured out a better way to get the class for the _run_cleanups triggers21:11
sdagueso I have to do a rebase anyway21:12
mtreinishprobably it's not like they're large changes but it may help it get through review faster21:12
*** mdenny has quit IRC21:13
mtreinishand they are tightly coupled21:13
*** jaypipes has joined #openstack-qa21:14
*** ChanServ sets mode: +o jaypipes21:14
*** dkranz has quit IRC21:16
sdaguemtreinish: 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 distinct21:16
morganfainbergsdague, ++ please don't log the keystone token!21:17
morganfainberg:P21:17
morganfainbergi know we're guilty of it sometimes21:17
sdaguemorganfainberg: yeh, well that ship is sailed so far...21:17
sdaguego look at logs some times21:17
morganfainbergsdague, *sigh( i know :(21:18
sdaguelogging pki tokens is so fun too21:18
sdagueit's actually the only reason the horizonal scroll bar is still triggered on log.openstack.org21:18
sdagueglanceclient is the biggest offender I think21:18
*** mlavalle has joined #openstack-qa21:19
clarkbsdague: yeah the CRM114 stuff needs to be taught to treat all UUIDS and pki tokens as a single TOKEN identifier21:20
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/71841 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/129516521:20
openstackgerritMatthew Treinish proposed a change to openstack-dev/devstack: Fix incorrect tempest config groups  https://review.openstack.org/8192221:21
*** jamielennox is now known as jamielennox|away21:22
*** vkozhukalov has quit IRC21:25
sdagueclarkb: how much are we stripping before we pass data to it?.21:27
openstackgerritSean Dague proposed a change to openstack/tempest: add _find_caller to the request log  https://review.openstack.org/8165421:27
openstackgerritSean Dague proposed a change to openstack/tempest: simplify rest_client logging  https://review.openstack.org/8162021:27
openstackgerritSean Dague proposed a change to openstack/tempest: add request timing  https://review.openstack.org/8177821:27
clarkbsdague: right now it isn't really stripping anything but the timestamps21:28
clarkbsdague: 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 weekened21:28
sdagueso honestly, I be if you threw out the whole line up to the ]21:29
sdagueyou'd probably greatly increase it's usefulness21:29
sdaguebasically keep the message after the request id21:30
clarkbsdague: good point there is extra info there that is all meta21:30
sdagueclarkb:  https://review.openstack.org/81654 turned out pretty solid21:33
sdaguethat's the call stack tracking to tell where a rest call is coming from21:33
*** afazekas has joined #openstack-qa21:36
*** weshay has quit IRC21:40
sdaguedtroyer: 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
sdagueit seems to really work now21:45
sdagueand no bad side effects so far21:45
dtroyersdague: 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 difference21:48
dtroyersorry 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 urgent21:49
sdagueyep, no worries21:50
sdagueyeh, I have devstack-gate logs that showed that21:50
sdaguewhen I removed the > /dev/null I got log out put21:50
sdagueit was weird21:50
sdagueand I've yet to see a truncated log since21:50
dtroyercool.   maybe I got too aggressive closing file handles21:51
sdaguewe actually started to get the jobs trace in the logs, which we never had before21:51
sdagueno, this was in devstack-gate21:51
sdaguewe added a > /dev/null on the devstack run to keep the logs out of the console21:51
dtroyerah, ok.   is the stdbuff still in the stack.sh call?  I'm not sure that is still necessary21:51
sdagueyep21:52
dtroyerbut this may not be the time to try taking it out21:52
*** mriedem has quit IRC21:53
openstackgerritSean Dague proposed a change to openstack/tempest: don't log cli output on success  https://review.openstack.org/8192621:56
sdaguedtroyer: agreed21:56
sdaguemtreinish: there you go :)21:56
*** dkranz has joined #openstack-qa21:57
sdaguemorganfainberg: ... 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_72121:58
sdagueyou guys do it a lot :)21:58
*** jgrimm has quit IRC21:59
mtreinishmeeting time22:00
afazekasdtroyer, sdague: https://review.openstack.org/#/c/77630/22:02
afazekasAt 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 available22:05
*** masayuki_ has joined #openstack-qa22:05
openstackrecheckConsole logs not available after 13:22s for gate-horizon-pep8 71330,11,f18c5a122:05
openstackrecheckopenstack/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/125487222:05
*** yjiang5 has quit IRC22:05
*** marun has joined #openstack-qa22:07
*** atiwari has quit IRC22:08
dtroyerafazekas: just one small bit to do yet22:10
openstackgerritafazekas proposed a change to openstack-dev/devstack: Upgrade to cirros 0.3.2  https://review.openstack.org/8148622:13
afazekasdtroyer: can I use the  #NOPRIME with the scsi-target-utils as well ?22:16
dtroyerI didn't see that getting installed manually.22:17
dtroyerI think you always want that if cinder is configured, right?22:17
*** Sukhdev has joined #openstack-qa22:20
afazekasdtroyer: If you are using LIO it is not needed, it is required for the tgtd22:21
dtroyerok.  if you add #NORPIME than you will need to install it in install_cinder()22:22
afazekasok22:22
dtroyerusually we've only done that on packages that have side-effects, like servers.  I can go either way on that one22:23
dtroyernova compute uses it too, so it would be fine to leave it alone if you want22:24
afazekasdtroyer: Since the yum install now are strictly checked, otherwise it cause install issue on RHEL722:24
dtroyeroh, for missing deps, right22:24
dtroyeris nova going to have the same issue then?22:25
afazekasNow 5 packages is missing on RHEL7 these 3 https://review.openstack.org/#/c/79947/ and the python-mox and the  scsi-target-utils22:26
*** nati_ueno has joined #openstack-qa22:26
dtroyeroff-topic, has anything been announced about RHEL7's release date?  even if it just which quarter it is expected in?22:28
sdaguejogo: 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.html22:28
jogosdague: ' No distributions at all found for PrettyTable'  looks like the culprit22:31
afazekasdtroyer: I do not know about it22:31
dtroyerok, np22:31
sdaguejogo: yep22:31
sdagueER caught it22:31
sdaguestill funny22:31
jogoahh good22:32
sdagueis that a mirror bug?22:32
jogosdague: ohh that was your patch ;)22:32
sdague:)22:32
jogohttps://review.openstack.org/#/c/81665/22:33
jogopip socket timeout22:33
sdagueY U FAIL MY PACH!22:33
jogohaha22:33
clarkbjogo: sdague: derekh is working on upping that timeout22:33
sdagueclarkb: yeh, I saw there was a patch in progress22:34
jogooh if your in here clarkb, so that bug is marked as incomplete22:34
clarkbI was mean and -1'd it (but there is a totally easy way to do it :) )22:34
jogo127241722:34
sdagueyou sure a timeout will fix it, and it doesn't need retry22:34
sdaguelike we did on the git calls in devstack22:34
clarkbsdague: no we are not sure22:34
clarkbwhich is why that bug is marked incomplete. There isn't a whole lot of info22:34
clarkbI will change it to triaged so that it doesn't go away22:35
jogoclarkb: ahh thanks22:35
jogothis isn't related to  the mirror being updated is it?22:35
*** adalbas has quit IRC22:37
clarkbI don't think so, we use rsync for that which should be atomic renaming things right?22:37
clarkbmaybe it doesn't22:38
* clarkb tries to remember22:38
clarkbyeah it copies to a temp file then renames so we should be ok22:38
clarkbas any open connections would continue reading the old file and new connections would get the new file22:39
*** cyeoh is now known as cyeoh_away22:39
sdagueyeh, it's atomic22:41
sdagueeven on partials22:41
sdaguethere is often an issue though that the index comes first22:42
sdagueand the packages after22:42
sdaguedistro mirrors often screw up that way22:42
clarkbright but we shouldn't remove anything with rsync22:42
clarkbso if a pacakge was there before it will continue to show up in the index22:42
sdagueclarkb: sure, but if you point to a file that doesn't yet exist22:42
sdagueI assume decision is done at the index level22:43
clarkbright so there is a race around jobs trying to use packages that don't exist yet22:43
* sdague is honestly not sure about all the pypi mirror internals22:43
clarkbsdague: it is super simple. it basically rsyncs a pip package cache into an apache docroot that is served with mod autoindex22:44
clarkbsdague: so there isn't a race with rsync copying a new index file first22:44
clarkband shouldn't be a race with new files getting put int place22:44
*** andreaf has joined #openstack-qa22:44
openstackgerritSteve Baker proposed a change to openstack/tempest: Add a StackResourceBuildErrorException  https://review.openstack.org/8193522:56
openstackgerritSteve Baker proposed a change to openstack/tempest: Catch StackResourceBuildErrorException on WaitCondition error  https://review.openstack.org/8193622:56
*** andreaf_ has joined #openstack-qa22:56
*** andreaf has quit IRC22:58
dkranzsdague, 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
mtreinishdkranz: yeah that's a bug23:00
dkranzmtreinish: ok, np. I'll fix it.23:00
*** masayuki_ is now known as masayukig23:02
*** shwetaap has quit IRC23:05
*** andreaf_ has quit IRC23:06
*** salv-orlando has quit IRC23:06
*** Aimon has quit IRC23:06
*** salv-orlando has joined #openstack-qa23:07
*** rockyg has quit IRC23:08
*** jamielennox|away is now known as jamielennox23:08
*** mlavalle has quit IRC23:12
openstackgerritafazekas proposed a change to openstack-dev/devstack: Add LIO target / iscsi_helper support to cinder  https://review.openstack.org/7763023:16
openstackgerritA change was merged to openstack/tempest: Add server to clean up even in case of errors  https://review.openstack.org/8157323:17
afazekashttps://review.openstack.org/#/c/81834/ is anybody able to reproduce this failure ?23:20
afazekasAt least 1 from 4 n-net job fails23:21
openstackgerritA change was merged to openstack/tempest: Fix error trace induced by dhcp test actions  https://review.openstack.org/8041823:24
* afazekas TODO hack the ssh code at the lower level (ssh or restClient) to always report details23:24
openstackrecheckopenstack/requirements change: https://review.openstack.org/81319 failed because of: gate-grenade-dsvm: https://bugs.launchpad.net/bugs/126290623:26
openstackrecheckopenstack/python-heatclient change: https://review.openstack.org/76698 failed gate-tempest-dsvm-neutron in the gate queue with an unrecognized error23:29
*** che-arne has quit IRC23:31
*** mrodden has quit IRC23:35
*** mwagner_notHere has quit IRC23:37
openstackgerritMark Maglana proposed a change to openstack/tempest: Add server boot time measurement  https://review.openstack.org/8107723:40
openstackgerritDane LeBlanc proposed a change to openstack-dev/devstack: stack.sh fails for Cisco plugin with missing cisco_plugins.ini error  https://review.openstack.org/8194223:44
*** rockyg has joined #openstack-qa23:45
*** mrodden has joined #openstack-qa23:48
*** mmaglana has quit IRC23:51
*** nati_ueno has quit IRC23:56
*** nati_ueno has joined #openstack-qa23:57
*** mriedem has joined #openstack-qa23:58
*** rockyg has quit IRC23:58
*** cyeoh_away is now known as cyeoh23:58
mriedemmtreinish: 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!