Tuesday, 2014-01-28

openstackgerritA change was merged to openstack-infra/devstack-gate: Add support for grenade-dsvm-rolling test  https://review.openstack.org/6419900:00
openstackgerritA change was merged to openstack-infra/devstack-gate: Add Trove testing support  https://review.openstack.org/6504000:01
*** openstack has joined #openstack-qa00:02
*** ken1ohmichi has joined #openstack-qa00:07
*** ken1ohmichi has quit IRC00:07
*** ken1ohmichi has joined #openstack-qa00:11
openstackgerritEdgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6  https://review.openstack.org/6731200:12
openstackgerritA change was merged to openstack/tempest: Use unittest2.TestSuite with py26  https://review.openstack.org/6913400:12
openstackgerritA change was merged to openstack/tempest: Specify 'active' status for deleting situations  https://review.openstack.org/6936900:14
openstackgerritA change was merged to openstack/tempest: Fix parameter type spelling tempest HOT template  https://review.openstack.org/6784400:14
*** markmcclain has quit IRC00:16
*** matsuhashi has joined #openstack-qa00:27
*** morganfainberg is now known as morganfainberg|z00:27
openstackgerritHenry Gessau proposed a change to openstack/tempest: Network API: default to ipv4, add ipv6 tests  https://review.openstack.org/6721000:30
openstackgerritEdgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6  https://review.openstack.org/6731200:32
*** dims has quit IRC00:34
*** dims has joined #openstack-qa00:36
*** hemna_ is now known as hemnafk00:37
*** zz_ewindisch is now known as ewindisch00:45
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: DO NOT MERGE: set NOVA_COMPUTE_ROLLING_UPGRADE=True  https://review.openstack.org/6407500:47
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: Add rolling upgrade test for nova-compute  https://review.openstack.org/6407400:47
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: Remove stop nova processes from upgrade-nova  https://review.openstack.org/6396500:47
openstackgerritKen'ichi Ohmichi proposed a change to openstack/tempest: Remove Nova v3 XML tests completely  https://review.openstack.org/6951700:53
*** iwamoto has joined #openstack-qa00:59
*** mrodden has quit IRC01:03
*** masayukig has quit IRC01:06
*** masayukig has joined #openstack-qa01:09
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: DO NOT MERGE: set NOVA_COMPUTE_ROLLING_UPGRADE=True  https://review.openstack.org/6407501:15
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: Add rolling upgrade test for nova-compute  https://review.openstack.org/6407401:15
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: Remove stop nova processes from upgrade-nova  https://review.openstack.org/6396501:15
*** _ruhe is now known as ruhe01:16
openstackgerritA change was merged to openstack/tempest: Add a check for compute api versions to verify_tempest_config  https://review.openstack.org/6537101:34
*** nosnos has joined #openstack-qa01:36
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Fix volume creation race condition in test_server_rescue tests  https://review.openstack.org/6945501:44
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Move common _delete_volume cleanup method in base compute test class  https://review.openstack.org/6945401:44
*** ianw has quit IRC01:59
*** ianw has joined #openstack-qa02:00
openstackgerritRonak Shah proposed a change to openstack/tempest: Running SNAT specific test cases only when ext-gw-mode extension is enabled  https://review.openstack.org/6952802:01
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: DO NOT MERGE: set NOVA_COMPUTE_ROLLING_UPGRADE=True  https://review.openstack.org/6407502:02
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: Add rolling upgrade test for nova-compute  https://review.openstack.org/6407402:02
openstackgerritJoe Gordon proposed a change to openstack-dev/grenade: Remove stop nova processes from upgrade-nova  https://review.openstack.org/6396502:02
*** ruhe is now known as _ruhe02:03
openstackgerritA change was merged to openstack-infra/devstack-gate: Remove configure_tempest.sh  https://review.openstack.org/6692202:03
openstackgerritRonak Shah proposed a change to openstack/tempest: Run SNAT specific test cases only with ext-gw-mode extension  https://review.openstack.org/6952802:04
clarkbjog0: comments on https://review.openstack.org/#/c/68799/02:11
*** vkozhukalov has joined #openstack-qa02:15
jog0clarkb: thanks, responded02:17
clarkband responded :)02:18
jog0clarkb: and one more back and forth02:22
jog0I marked it as a WOP02:22
jog0WIP02:22
jog0but if your looking at fun e-r patches to review https://review.openstack.org/#/c/68811/02:22
jog0btw great job on the zuul window stuff02:23
clarkbjog0: thanks. I am currently trying to make a dent in the review backlog02:24
openstackgerritMatt Riedemann proposed a change to openstack-infra/elastic-recheck: Add query for test_server_rescue race bug 1226412  https://review.openstack.org/6953002:24
clarkbI think I have reviewed like 60 something changes today and the list is still massive02:24
jog0clarkb: yeah, openstack has 1812 open reviews02:26
clarkbjog0: quick question https://review.openstack.org/#/c/68811/2/elastic_recheck/elasticRecheck.py in the split and slice in queue() is that looking at the job name to get the name?02:27
clarkbthe job name doesn't always indicate the correct queue02:27
clarkbwould be better to look at the vote jenkins left. +/-1 = check +/-2 = gate02:28
clarkboh nevermind I see what it is doing02:29
jog0yeah02:32
jog0it look at the queue name in the log url02:32
jog0http://logs.openstack.org/86/59686/1/gate/gate-tempest-dsvm-large-ops/f6f933b/logs/02:33
jog0/gate/02:33
clarkbyup02:33
jog0anyway time to call it a day o/ happy reviewing02:35
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Enable tenant isolation for the boto tests  https://review.openstack.org/6950002:35
*** ewindisch is now known as zz_ewindisch02:43
*** nati_ueno has quit IRC03:02
*** morganfainberg|z is now known as morganfainberg03:11
*** stack555 has joined #openstack-qa03:12
*** matsuhashi has quit IRC03:13
*** matsuhashi has joined #openstack-qa03:14
openstackgerritYuiko Takada proposed a change to openstack/tempest: Separate negative tests for test_templates  https://review.openstack.org/6929103:15
*** zz_ewindisch is now known as ewindisch03:16
*** ewindisch is now known as zz_ewindisch03:17
*** rwsu has quit IRC03:18
*** mrodden has joined #openstack-qa03:20
*** masayukig has quit IRC03:27
*** morganfainberg is now known as morganfainberg|z03:36
Anjumtreinish:  there ?03:45
*** matsuhashi has quit IRC03:49
*** mriedem has quit IRC03:55
*** harlowja is now known as harlowja_away04:01
*** harlowja_away is now known as harlowja04:06
openstackgerritYuiko Takada proposed a change to openstack/tempest: Separate negative tests for test_templates  https://review.openstack.org/6929104:14
Anjusdague:  ping04:19
*** masayukig has joined #openstack-qa04:31
*** matsuhashi has joined #openstack-qa04:45
*** Madan has joined #openstack-qa04:45
*** weshay has quit IRC04:49
*** yfried has quit IRC05:07
*** Madan has quit IRC05:35
*** psharma has joined #openstack-qa05:40
*** Madan has joined #openstack-qa05:51
openstackgerritChristopher Yeoh proposed a change to openstack-infra/elastic-recheck: Adds query for bug 1273556  https://review.openstack.org/6954206:01
*** harlowja is now known as harlowja_away06:10
*** vkozhukalov has quit IRC06:51
*** nati_ueno has joined #openstack-qa06:54
*** nati_ueno has quit IRC07:07
*** nati_ueno has joined #openstack-qa07:08
*** Madan has quit IRC07:10
*** coolsvap has joined #openstack-qa07:17
*** afazekas has joined #openstack-qa07:17
*** yrabl has joined #openstack-qa07:23
openstackgerritYuiko Takada proposed a change to openstack/tempest: Separate negative tests for test_templates  https://review.openstack.org/6929107:24
*** Madan has joined #openstack-qa07:24
*** vkozhukalov has joined #openstack-qa07:33
*** yfried has joined #openstack-qa07:33
Anjuafazekas:  ping07:58
*** nati_uen_ has joined #openstack-qa08:02
openstackgerritSergey Nikitin proposed a change to openstack/tempest: Expand baremetal port coverage  https://review.openstack.org/6955508:02
afazekasAnju: pong08:03
*** luqas has joined #openstack-qa08:04
Anjuafazekas:  https://github.com/openstack/tempest/blob/master/tempest/api/compute/test_live_block_migration.py#L11708:04
AnjuI want to make  a different test as it s negative..but I want to check it ..how can set my conf foe the same08:05
*** nati_ueno has quit IRC08:05
afazekasevery dvsm gate job prints the tempest config08:06
Anjuafazekas:  cant I cehck the ame in devstack08:07
afazekasyou can also read the localrc from the logs, and several additional environment variable is set by the devstack-gate08:10
Anjuafazekas:  if I dont want to skip this ..08:10
afazekasLIVE_MIGRATION_AVAILABLE=True08:11
Anjuafazekas:  in which file08:12
afazekaslocalrc (or local.conf)08:12
afazekashttps://github.com/openstack-dev/devstack/blob/master/lib/tempest#L27508:13
*** bdperkin has quit IRC08:13
*** bdperkin has joined #openstack-qa08:14
Anjuafazekas:  so we need to skip it ? I am creating a new file for this test08:15
openstackgerritBhuvan Arumugam proposed a change to openstack/tempest: SSH login to openstack server using proxy  https://review.openstack.org/6929808:15
*** nati_uen_ has quit IRC08:15
*** nati_ueno has joined #openstack-qa08:16
*** mrodden has quit IRC08:19
*** mrodden has joined #openstack-qa08:19
*** yfried has quit IRC08:20
*** yfried has joined #openstack-qa08:22
*** yassine has joined #openstack-qa08:23
raieswhat is the service type for the swift in devstack running swift ??08:30
raiesafazekas: ^^08:30
raiesafazekas: I checked keystone endpoint-get --service object-store but it returns error08:30
*** tkammer has joined #openstack-qa08:31
*** iwamoto has quit IRC08:32
afazekasAnju: FYI https://github.com/openstack/nova/blob/master/etc/nova/nova.conf.sample#L61408:32
*** bdperkin has quit IRC08:36
*** nmagnezi has joined #openstack-qa08:42
*** safchain has joined #openstack-qa08:51
*** _ruhe is now known as ruhe08:51
*** vkozhukalov has quit IRC08:51
*** jlibosva has joined #openstack-qa08:53
*** jlibosva has quit IRC08:54
*** jlibosva has joined #openstack-qa08:57
*** jpich has joined #openstack-qa08:59
openstackgerritEdgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6  https://review.openstack.org/6731209:01
*** bdperkin_gone has joined #openstack-qa09:02
*** bdperkin_gone is now known as bdperkin09:03
*** bdperkin has quit IRC09:03
*** bdperkin has joined #openstack-qa09:03
*** ken1ohmichi has quit IRC09:05
afazekasraies: http://www.fpaste.org/72277/00012139/09:07
afazekasobject-store09:07
raiesafazekas: yes tried that. it is working :) thanks09:07
*** giulivo has joined #openstack-qa09:08
*** SergeyLukjanov_ is now known as SergeyLukjanov09:08
*** nati_uen_ has joined #openstack-qa09:09
*** matsuhashi has quit IRC09:09
*** nati_uen_ has joined #openstack-qa09:09
*** nati_ueno has quit IRC09:12
vponomaryovafazekas: Hello, You have left comments on https://review.openstack.org/#/c/62429/ , rebase is done. If you have time, please look at it.09:13
*** matsuhas_ has joined #openstack-qa09:14
*** rossella_s has joined #openstack-qa09:18
*** marun has quit IRC09:19
openstackgerritElena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer  https://review.openstack.org/5869709:20
*** ruhe is now known as _ruhe09:20
openstackgerritA change was merged to openstack/tempest: Fix glance_http HTTPS response issues  https://review.openstack.org/6948709:25
openstackgerritVincent Untz proposed a change to openstack/tempest: Use BaseComputeTest.create_test_server  https://review.openstack.org/6344609:26
openstackgerritMh Raies proposed a change to openstack/tempest: API tests for neutron metering label and rules  https://review.openstack.org/6956009:26
afazekasI wonder how will it looks like together with  https://review.openstack.org/#/c/65311/09:28
openstackgerritnayna-patel proposed a change to openstack/tempest: Test to update neutron security group  https://review.openstack.org/6956109:28
*** amotoki has joined #openstack-qa09:33
vponomaryovafazekas: if your question to me, so my answer is next: just manual rebase, for patch, that is merging as second09:36
*** nati_ueno has joined #openstack-qa09:51
*** nati_uen_ has quit IRC09:53
*** gszasz has joined #openstack-qa09:55
*** luqas has quit IRC09:57
*** nati_ueno has quit IRC09:57
*** safchain has quit IRC10:00
*** safchain has joined #openstack-qa10:01
openstackgerritYair Fried proposed a change to openstack/tempest: Rename cross_tenant to security_groups_basic_ops  https://review.openstack.org/6956710:12
*** luqas has joined #openstack-qa10:19
*** Madan1 has joined #openstack-qa10:21
*** matsuhas_ has quit IRC10:22
*** Madan has quit IRC10:23
*** max_lobur_afk is now known as max_lobur10:29
*** matsuhashi has joined #openstack-qa10:31
*** vkozhukalov has joined #openstack-qa10:37
openstackgerritElena Ezhova proposed a change to openstack/tempest: Add scenario test for load balancer  https://review.openstack.org/5869710:39
*** andreaf has joined #openstack-qa10:46
*** coolsvap has quit IRC10:47
*** coolsvap has joined #openstack-qa11:00
*** matsuhashi has quit IRC11:00
openstackgerritAnn Kamyshnikova proposed a change to openstack/tempest: Add bool and integer support to XML parser  https://review.openstack.org/6629711:01
openstackgerritAnn Kamyshnikova proposed a change to openstack/tempest: Verify more information in API tests for LBaaS  https://review.openstack.org/6111811:02
openstackgerritAnn Kamyshnikova proposed a change to openstack/tempest: Verify more information for pools in lbaas tests  https://review.openstack.org/6594311:03
openstackgerritAnn Kamyshnikova proposed a change to openstack/tempest: Vefify more information for member in lbaas api tests  https://review.openstack.org/6654111:05
openstackgerritAnn Kamyshnikova proposed a change to openstack/tempest: Add namespaces to xml in Network client  https://review.openstack.org/6413011:06
openstackgerritElena Ezhova proposed a change to openstack/tempest: Refactor create_ and update_ methods for VIPs and health_monitors  https://review.openstack.org/6651611:07
openstackgerritElena Ezhova proposed a change to openstack/tempest: Add api tests for load balancer's VIPs and health monitors  https://review.openstack.org/6679611:07
openstackgerritVladislav Kuzmin proposed a change to openstack/tempest: Fixed up an error message  https://review.openstack.org/6957711:08
openstackgerritAnn Kamyshnikova proposed a change to openstack/tempest: Verify more information in floating ip tests  https://review.openstack.org/6372311:09
*** boris-42 has quit IRC11:11
*** matsuhashi has joined #openstack-qa11:14
*** matsuhashi has quit IRC11:17
*** SergeyLukjanov is now known as SergeyLukjanov_a11:19
*** SergeyLukjanov_a is now known as SergeyLukjanov11:19
*** coolsvap has quit IRC11:30
*** rfolco has joined #openstack-qa11:36
raiesafazekas: hi11:37
openstackgerritMh Raies proposed a change to openstack/tempest: API tests for neutron metering label and rules  https://review.openstack.org/6956011:40
openstackgerritSean Dague proposed a change to openstack/tempest: tighten up isolated creds create  https://review.openstack.org/6919711:42
openstackgerritnayna-patel proposed a change to openstack/tempest: Test to update neutron security group  https://review.openstack.org/6956111:48
openstackgerritSean Dague proposed a change to openstack/tempest: tighten up isolated creds create  https://review.openstack.org/6919711:48
afazekasraies: hi11:53
*** masayukig has quit IRC11:57
openstackgerritJakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron  https://review.openstack.org/5869511:58
openstackgerritA change was merged to openstack/tempest: Fixing typo in test_volumes_get.py  https://review.openstack.org/6683012:28
*** jasondotstar has joined #openstack-qa12:28
*** boris-42 has joined #openstack-qa12:31
*** luqas has quit IRC12:36
*** Madan1 has quit IRC12:40
*** max_lobur is now known as max_lobur_afk12:41
*** dkranz has joined #openstack-qa12:45
dkranzmkoderer: ping12:45
mkodererdkranz: hey david12:45
dkranzmkoderer: I restored the change12:46
dkranzmkoderer: did you read the minutes from the last meeting?12:46
mkodererok cool I will update it12:47
dkranzmkoderer: We came up with:12:47
dkranz1. No XML12:47
mkodererdkranz: ahm yes I browsed trough12:47
dkranz2. Keep old xml tests12:47
mkodererok this was exactly what I did12:47
dkranz3. Move schema defs to separate files in each subdir: compute/flavor, etc.12:48
dkranzmkoderer: ok, great. I just got up and have not had a chance to look yet12:48
dkranzmkoderer: I will do so this morning12:48
dkranzmkoderer: Thanks for doing that12:48
mkodererdkranz: Ok I didn't move them12:48
dkranzmkoderer: We are then promised a real review :)12:48
mkodererthat's nice :)12:48
dkranzmkoderer: Folks felt it would be more readable in separate files and I agree.12:49
dkranzmkoderer: But I would still keep them as global defs rather than as a json file that has to be read in12:49
mkodererdkranz: ok I will do that in the next hours and push a new version12:50
dkranzmkoderer: thanks. I am going to look at the current state now but have meeting in 10min12:50
*** gszasz has quit IRC12:50
mkodererdkranz: ok cu later12:51
*** amotoki has quit IRC12:51
*** _ruhe is now known as ruhe12:51
openstackgerritSergey Lukjanov proposed a change to openstack-dev/devstack: Expose Savanna infrastructure engine type  https://review.openstack.org/6959112:57
*** psharma has quit IRC12:59
*** markmcclain has joined #openstack-qa13:00
*** nmagnezi_ has joined #openstack-qa13:02
*** nmagnezi has quit IRC13:02
*** afazekas has quit IRC13:02
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Multiversion authentication part1  https://review.openstack.org/6531113:02
*** max_lobur_afk is now known as max_lobur13:04
*** gszasz has joined #openstack-qa13:05
*** jhenner has quit IRC13:05
*** nosnos has quit IRC13:08
*** om has joined #openstack-qa13:09
*** luqas has joined #openstack-qa13:10
*** openstackrecheck has joined #openstack-qa13:14
openstackgerritnayna-patel proposed a change to openstack/tempest: Test to update neutron security group  https://review.openstack.org/6956113:15
*** afazekas has joined #openstack-qa13:15
openstackgerritJakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron  https://review.openstack.org/5869513:18
*** luqas has quit IRC13:18
*** jhenner has joined #openstack-qa13:25
openstackgerritanju Tiwari proposed a change to openstack/tempest: Removed unused lock from a aggregate test  https://review.openstack.org/6959513:26
*** Anju1 has joined #openstack-qa13:30
Anju1afazekas : ping13:30
Anju1afazekas:  https://review.openstack.org/#/c/69595/13:31
*** afazekas is now known as afazekas|mtg13:31
SergeyLukjanovfolks, could you please take a look at initial API tests for Savanna in Tempest? https://review.openstack.org/#/c/61114/13:32
openstackrecheckConsole logs not available after 13:27s for gate-ceilometer-docs 69582,1,bbf22a813:32
openstackrecheckopenstack/keystone change: https://review.openstack.org/66317 failed with an unrecognized error13:33
*** yfried has quit IRC13:33
openstackrecheckopenstack/horizon change: https://review.openstack.org/64839 failed with an unrecognized error13:33
openstackrecheckopenstack/python-cinderclient change: https://review.openstack.org/63147 failed tempest because of: https://bugs.launchpad.net/bugs/121048313:33
afazekas|mtgAnju1: I added mteinish to the review13:34
Anju1afazekas|mtg:  but the link I added there is wrong13:34
openstackrecheckopenstack/tempest change: https://review.openstack.org/66541 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1273386 and https://bugs.launchpad.net/bugs/125389613:34
*** om has quit IRC13:35
Anju1afazekas|mtg:  you provide me that..can you plaese tell me how could I check13:35
*** NithyaG has joined #openstack-qa13:35
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Multiversion authentication part1  https://review.openstack.org/6531113:36
openstackrecheckopenstack/requirements change: https://review.openstack.org/66349 failed with an unrecognized error13:37
*** yfried has joined #openstack-qa13:37
openstackrecheckopenstack/python-neutronclient change: https://review.openstack.org/68603 failed tempest because of: https://bugs.launchpad.net/bugs/125489013:42
openstackgerritSean Dague proposed a change to openstack/tempest: Revert "skip test_volume_boot_pattern"  https://review.openstack.org/6959913:44
openstackgerritanju Tiwari proposed a change to openstack/tempest: Removed unused lock from a aggregate test  https://review.openstack.org/6959513:46
openstackrecheckopenstack/keystone change: https://review.openstack.org/69219 failed with an unrecognized error13:48
openstackrecheckopenstack/keystone change: https://review.openstack.org/69218 failed with an unrecognized error13:50
*** dims has quit IRC13:50
*** dims has joined #openstack-qa13:52
*** om has joined #openstack-qa13:54
*** markmcclain has quit IRC13:57
*** markmcclain has joined #openstack-qa13:57
mkodererdkranz: should we rename the *_negative_new file in *_negative_json?14:00
dkranzmkoderer: I don't think we need to call things json since xml is I think going away eventually14:01
mkodererdkranz: should we keep _new?14:01
mkodererdkranz: or auto?14:01
dkranzmkoderer: Perhaps we should just rename the other file to be _xml?14:01
dkranzmkoderer: Cause that was actually the real change to that file14:01
mkodererdkranz: ok that's better14:01
*** dims has quit IRC14:03
*** mestery has quit IRC14:03
*** dims has joined #openstack-qa14:04
*** dkranz has quit IRC14:09
openstackrecheckConsole logs not available after 14:06s for gate-heat-python27 67532,3,2173da614:09
openstackgerritJakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron  https://review.openstack.org/5869514:09
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69591 failed tempest because of: https://bugs.launchpad.net/bugs/125489014:10
openstackrecheckopenstack/heat change: https://review.openstack.org/68800 failed with an unrecognized error14:10
*** afazekas|mtg has quit IRC14:11
*** yfried has quit IRC14:11
*** gszasz has quit IRC14:13
*** jhenner has quit IRC14:13
*** mestery has joined #openstack-qa14:15
*** afazekas has joined #openstack-qa14:18
openstackgerritMarc Koderer proposed a change to openstack/tempest: Negative test autogeneration framework  https://review.openstack.org/6473314:19
*** nmagnezi has joined #openstack-qa14:22
*** mriedem has joined #openstack-qa14:23
*** zz_ewindisch is now known as ewindisch14:23
openstackrecheckopenstack/glance change: https://review.openstack.org/69392 failed with an unrecognized error14:24
*** nmagnezi_ has quit IRC14:26
*** yfried has joined #openstack-qa14:27
openstackrecheckopenstack/glance change: https://review.openstack.org/69594 failed with an unrecognized error14:29
*** jhenner has joined #openstack-qa14:30
sdaguemkoderer, mtreinish, afazekas: https://review.openstack.org/#/c/66878/ ?14:33
sdaguekilling another xml client14:33
mkoderersdague: cool14:35
afazekas:)14:35
openstackrecheckopenstack/tempest change: https://review.openstack.org/66516 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1235435 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1273386 and https://bugs.launchpad.net/bugs/127038214:37
*** gszasz has joined #openstack-qa14:37
mkoderersdague, mtreinish, afazekas: btw https://review.openstack.org/64733 negative tests are ready for review14:38
*** mestery_ has joined #openstack-qa14:40
andreafafazekas, giulivo, sdague, mtreinish: multi auth version bp part1: passes check now: https://review.openstack.org/#/c/65311/14:41
andreafit affects a lot of files so I had to do a lot of re-basing14:42
*** mestery__ has joined #openstack-qa14:42
*** meste____ has joined #openstack-qa14:43
*** mestery has quit IRC14:43
*** mestery_ has quit IRC14:46
*** mestery__ has quit IRC14:46
*** tkammer has quit IRC14:47
openstackgerritLeandro Ignacio Costantino proposed a change to openstack/tempest: Add tempest test for heat resource OS::Nova::KeyPair  https://review.openstack.org/6900014:48
mkoderermtreinish: is it cruel to -1 a patch due to missing unit tests?14:52
openstackrecheckopenstack/glance change: https://review.openstack.org/69614 failed with an unrecognized error14:52
openstackrecheckopenstack/glance change: https://review.openstack.org/58997 failed tempest because of: https://bugs.launchpad.net/bugs/1209086 and https://bugs.launchpad.net/bugs/124025614:52
mtreinishmkoderer: no :)14:52
mkoderermtreinish: https://review.openstack.org/#/c/66297 gives no real usage of the code14:52
mtreinishI think we should start doing it more14:52
openstackrecheckopenstack/heat change: https://review.openstack.org/69248 failed with an unrecognized error14:53
mkoderermtreinish: that's a -1 for me14:53
mtreinishmkoderer: I'm also not sure what the use case is there14:53
mkoderermtreinish: yep ... and I think a unit test would make the usage clearer14:54
openstackrecheckopenstack/tempest change: https://review.openstack.org/69599 failed tempest because of: https://bugs.launchpad.net/bugs/127060814:56
*** ruhe is now known as _ruhe14:56
*** mestery has joined #openstack-qa14:57
andreafI think having unit tests for https://review.openstack.org/#/c/66297 is fair enough. The use case for that is inconsistency between JSON and XML. json.loads() will give ints and bools, while XML gives only strings. So a test that looks into an int/bool value won't work with both JSON/XML unless this is implemented14:57
*** mestery_ has joined #openstack-qa14:58
*** meste____ has quit IRC15:00
*** om has quit IRC15:00
*** tkammer has joined #openstack-qa15:01
*** mestery__ has joined #openstack-qa15:01
*** ewindisch is now known as zz_ewindisch15:01
*** mestery has quit IRC15:02
*** mrodden has quit IRC15:02
*** NithyaG has quit IRC15:02
*** NithyaG has joined #openstack-qa15:03
*** mestery_ has quit IRC15:03
*** yfried1 has joined #openstack-qa15:05
*** weshay has joined #openstack-qa15:05
*** mestery has joined #openstack-qa15:05
*** tkammer has quit IRC15:06
*** yfried has quit IRC15:06
*** nmagnezi has quit IRC15:06
*** mestery__ has quit IRC15:09
*** rwsu has joined #openstack-qa15:12
openstackrecheckConsole logs not available after 13:23s for gate-nova-pep8 56381,12,27fd34115:14
*** mrodden has joined #openstack-qa15:16
*** coolsvap has joined #openstack-qa15:16
openstackgerritandrea-frittoli proposed a change to openstack/tempest: Set content length to 0 in case of empty body  https://review.openstack.org/6962515:17
*** zz_ewindisch is now known as ewindisch15:19
*** jgrimm has joined #openstack-qa15:20
afazekassdague: https://review.openstack.org/#/c/65964/315:21
*** nmagnezi has joined #openstack-qa15:21
*** morganfainberg|z is now known as morganfainberg15:22
*** tkammer has joined #openstack-qa15:22
openstackgerritYair Fried proposed a change to openstack/tempest: Rename cross_tenant to security_groups_basic_ops  https://review.openstack.org/6956715:23
*** dkranz has joined #openstack-qa15:24
openstackrecheckConsole logs not available after 13:27s for gate-nova-pep8 69603,1,4b1df9815:28
openstackrecheckopenstack/tempest change: https://review.openstack.org/64733 failed tempest because of: https://bugs.launchpad.net/bugs/125489015:28
openstackrecheckopenstack/tempest change: https://review.openstack.org/69203 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/127338615:29
openstackrecheckopenstack/tempest change: https://review.openstack.org/66541 failed tempest because of: https://bugs.launchpad.net/bugs/125144815:29
openstackrecheckopenstack/nova change: https://review.openstack.org/67855 failed with an unrecognized error15:29
*** _ruhe is now known as ruhe15:30
jlibosvachmouel: Hi15:34
*** HInfoForIRC has joined #openstack-qa15:36
HInfoForIRCHere is info about irc  http://p.pw/DLV15:36
*** HInfoForIRC has left #openstack-qa15:36
*** yfried1 has quit IRC15:36
*** yfried has joined #openstack-qa15:36
yfriedmtreinish: sdague dkranz: salv-orlando: afazekas: jaypipes: are you sure that this https://review.openstack.org/69361 (with all of it's copied code) is preferable to refactoring network basic ops?15:40
openstackrecheckConsole logs not available after 13:25s for gate-python-neutronclient-python33 68603,2,a30c35215:43
openstackrecheckopenstack/glance change: https://review.openstack.org/69605 failed with an unrecognized error15:43
openstackrecheckopenstack/glance change: https://review.openstack.org/69594 failed with an unrecognized error15:43
openstackrecheckopenstack/heat change: https://review.openstack.org/67759 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/125884815:45
*** amotoki has joined #openstack-qa15:47
*** nmagnezi has quit IRC15:50
mriedemjgriffith: have you seen this? https://review.openstack.org/#/c/69599/15:51
Anju1mtreinish:  hi,,https://review.openstack.org/#/c/69595/..needs your review, I change it as per my knowledge15:52
*** andreaf has quit IRC15:52
*** andreaf has joined #openstack-qa15:52
openstackrecheckopenstack/tempest change: https://review.openstack.org/69625 failed tempest because of: https://bugs.launchpad.net/bugs/1257641 and https://bugs.launchpad.net/bugs/125144815:54
afazekasyfried: the network basic ops should be simplified in order to make more functions common in the network scenario tests15:55
openstackrecheckopenstack/heat change: https://review.openstack.org/58877 failed with an unrecognized error15:56
openstackrecheckopenstack/nova change: https://review.openstack.org/40467 failed with an unrecognized error15:57
yfriedyfried: something like this? https://review.openstack.org/#/c/66879/15:59
openstackrecheckopenstack/nova change: https://review.openstack.org/61987 failed tempest because of: https://bugs.launchpad.net/bugs/110114716:01
*** nmagnezi has joined #openstack-qa16:01
openstackrecheckopenstack/nova change: https://review.openstack.org/59167 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1101147 and https://bugs.launchpad.net/bugs/125016816:03
openstackrecheckopenstack/tempest change: https://review.openstack.org/69000 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/127338616:03
*** yrabl has quit IRC16:04
openstackrecheckopenstack/cinder change: https://review.openstack.org/69506 failed with an unrecognized error16:04
*** tkammer has quit IRC16:04
*** boris-42 has quit IRC16:07
*** SergeyLukjanov is now known as SergeyLukjanov_16:09
openstackrecheckopenstack/nova change: https://review.openstack.org/69622 failed with an unrecognized error16:12
*** samuelbercovici has joined #openstack-qa16:13
*** yfried has quit IRC16:15
openstackrecheckopenstack/nova change: https://review.openstack.org/61990 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1101147 and https://bugs.launchpad.net/bugs/125016816:16
*** tkammer has joined #openstack-qa16:16
openstackrecheckopenstack/heat change: https://review.openstack.org/69536 failed with an unrecognized error16:17
openstackgerritEric Windisch proposed a change to openstack-dev/devstack: Upload docker images directly into glance  https://review.openstack.org/6644016:17
*** atiwari has joined #openstack-qa16:18
*** vkozhukalov has quit IRC16:19
*** nmagnezi has quit IRC16:21
*** tkammer has quit IRC16:21
*** IzikP has joined #openstack-qa16:28
openstackgerritJulien Leloup proposed a change to openstack/tempest: New parameter "leave_dirty_stack" available in tempest.conf to prevent (or not) the execution of tearDownClass() after an exception during a stress test.  https://review.openstack.org/6964116:30
*** ruhe is now known as _ruhe16:31
openstackrecheckConsole logs not available after 13:26s for check-tempest-dsvm-postgres-full 69567,2,ec1891416:31
openstackrecheckopenstack/nova change: https://review.openstack.org/58432 failed with an unrecognized error16:31
*** Anju1 has quit IRC16:31
openstackgerritMatthew Treinish proposed a change to openstack/tempest: Remove super call in overloaded close()  https://review.openstack.org/6964316:31
openstackrecheckopenstack/heat change: https://review.openstack.org/69612 failed with an unrecognized error16:32
IzikPHi, I've added a change in tempest api loadbalancer test. could someone review ? https://review.openstack.org/#/c/68190/16:33
mtreinishsdague: ^^^ that was a fun bug I introdcuded16:33
* mtreinish can't type today16:34
sdaguemtreinish: do we need to rush that?16:34
openstackgerritEric Windisch proposed a change to openstack-dev/devstack: Install glance images before starting Nova  https://review.openstack.org/6964716:35
openstackgerritEric Windisch proposed a change to openstack-dev/devstack: Upload docker images directly into glance  https://review.openstack.org/6644016:35
openstackrecheckopenstack/nova change: https://review.openstack.org/58921 failed with an unrecognized error16:37
mtreinishsdague: I don't know it we don't run against an https endpoint in the gate.16:37
mtreinishbut it will fail if you try to do that16:37
sdagueso we probably don't, but we should probably change that16:37
openstackrecheckopenstack/requirements change: https://review.openstack.org/68424 failed with an unrecognized error16:41
mtreinishsdague: oh I really can't type this morning. We definitely don't because otherwise the original patch would have failed the gate if we did16:41
openstackrecheckopenstack/nova change: https://review.openstack.org/52687 failed with an unrecognized error16:41
sdaguemtreinish: ok, then I'll just +2 and leave for someone else16:41
openstackrecheckopenstack/glance change: https://review.openstack.org/69594 failed with an unrecognized error16:41
openstackgerritA change was merged to openstack-infra/elastic-recheck: Add fingerprint for bug 1268631  https://review.openstack.org/6906816:43
*** openstackrecheck has quit IRC16:47
*** openstackrecheck has joined #openstack-qa16:47
*** Anju1 has joined #openstack-qa16:52
afazekasyfried: not splitting, we just need to simplify several over engineered method16:53
mkodererdkranz: I already updated https://review.openstack.org/#/c/64733/ and it's trough jenkins now16:56
*** afazekas has quit IRC17:00
*** yfried has joined #openstack-qa17:02
*** marun has joined #openstack-qa17:03
openstackrecheckConsole logs not available after 13:28s for check-tempest-dsvm-ironic-postgres-nv 60008,3,657e4d217:04
*** marun has quit IRC17:05
*** mlavalle has joined #openstack-qa17:07
*** boris-42 has joined #openstack-qa17:07
*** marun has joined #openstack-qa17:08
*** _ruhe is now known as ruhe17:15
openstackrecheckConsole logs not available after 13:23s for check-grenade-dsvm 69605,1,b23611617:18
openstackrecheckopenstack/nova change: https://review.openstack.org/66652 failed tempest because of: https://bugs.launchpad.net/bugs/124025617:18
openstackrecheckopenstack/nova change: https://review.openstack.org/67206 failed tempest because of: https://bugs.launchpad.net/bugs/125884817:18
openstackrecheckopenstack/glance change: https://review.openstack.org/69614 failed with an unrecognized error17:19
openstackrecheckopenstack/tempest change: https://review.openstack.org/65533 failed tempest because of: https://bugs.launchpad.net/bugs/125884817:19
openstackgerritPeter Portante proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1266616  https://review.openstack.org/6967017:20
yfriedsalv-orlando: ping17:23
dkranzmkoderer: Great! Thanks and let's hope this can get in soon.17:27
*** ruhe is now known as _ruhe17:28
*** Anju1 has quit IRC17:32
openstackrecheckConsole logs not available after 13:22s for gate-nova-pylint 66634,12,bbdac8817:33
openstackrecheckopenstack/nova change: https://review.openstack.org/46588 failed with an unrecognized error17:33
openstackrecheckopenstack/nova change: https://review.openstack.org/66685 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/125884817:34
openstackrecheckopenstack/keystone change: https://review.openstack.org/69632 failed with an unrecognized error17:34
openstackrecheckopenstack/python-neutronclient change: https://review.openstack.org/59192 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/127338617:34
*** yassine has quit IRC17:34
openstackrecheckopenstack/python-cinderclient change: https://review.openstack.org/59813 failed tempest because of: https://bugs.launchpad.net/bugs/125884817:34
bnemecsdague: About https://review.openstack.org/#/c/54403/8 , I'm not sure what else I would unit test beyond the doc tests.  All of the exceptions I'm aware of are already covered.17:35
bnemecI suppose I could do a POC change in a project like Nova to fix these errors and try to find more exceptions.17:35
*** SergeyLukjanov_ is now known as SergeyLukjanov17:36
openstackgerritSteve Martinelli proposed a change to openstack-dev/devstack: Change all keystoneclient commands to openstacklient in libs  https://review.openstack.org/6910717:36
*** Anju1 has joined #openstack-qa17:38
openstackrecheckopenstack/tempest change: https://review.openstack.org/66541 failed tempest because of: https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/127338617:38
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/69644 failed with an unrecognized error17:40
openstackrecheckopenstack/horizon change: https://review.openstack.org/61032 failed with an unrecognized error17:40
openstackrecheckopenstack/tempest change: https://review.openstack.org/69641 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/127338617:43
Anju1mtreinish:  around ?17:45
openstackgerritEdgar Magana proposed a change to openstack/tempest: Add test for subnet gateway IPv4 and IPv6  https://review.openstack.org/6731217:49
*** markd has joined #openstack-qa17:49
*** nmagnezi has joined #openstack-qa17:51
*** IzikP has left #openstack-qa17:53
*** jlibosva has quit IRC17:54
*** gszasz has quit IRC17:56
*** jlibosva has joined #openstack-qa17:56
*** ewindisch is now known as zz_ewindisch17:57
*** zz_ewindisch is now known as ewindisch17:58
*** samuelbercovici has quit IRC18:02
*** tkammer has joined #openstack-qa18:02
openstackrecheckConsole logs not available after 13:26s for gate-nova-python27 69603,3,0fa251718:02
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/69648 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:03
openstackrecheckopenstack/nova change: https://review.openstack.org/69631 failed tempest because of: https://bugs.launchpad.net/bugs/125868218:03
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/62779 failed with an unrecognized error18:03
openstackrecheckopenstack/heat change: https://review.openstack.org/69655 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:04
openstackrecheckopenstack/nova change: https://review.openstack.org/52733 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:11
openstackrecheckopenstack/nova change: https://review.openstack.org/63472 failed with an unrecognized error18:11
openstackrecheckopenstack/nova change: https://review.openstack.org/69652 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:13
*** Anju1 has left #openstack-qa18:14
*** harlowja_away is now known as harlowja18:17
*** vkozhukalov has joined #openstack-qa18:22
*** nati_ueno has joined #openstack-qa18:25
yfrieddkranz: I was offline for a few hours. did you happen to notice if someone answered my question?18:30
*** jpich has quit IRC18:30
dkranzyfried: I didn't see your question18:31
dkranzyfried: I see the question now and don't see an answer following18:32
openstackrecheckConsole logs not available after 13:28s for gate-keystone-python27 69531,3,1fa600618:33
openstackrecheckopenstack/horizon change: https://review.openstack.org/68979 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:33
dkranzyfried: Still there?18:34
yfrieddkranz: yeah. what about you - any opinion on the subject?18:35
dkranzyfried: Yes18:35
dkranzyfried: I think we need to separate the notion of conventional code re-use from the question of what is and is not a separate test case.18:36
yfriedI'll re-post it are you sure that this https://review.openstack.org/69361 (with all of it's copied code) is preferable to refactoring network basic ops?18:36
yfrieddkranz: care to explain?18:36
dkranzyfried: I thought the objection to the "refactoring" was about creating test cases that depend or don't depend on other test cases, right?18:37
openstackgerritPeter Portante proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1266616  https://review.openstack.org/6967018:37
dkranzyfried: But even if we accept that test cases will not be dependent on other test cases that doesn't mean that methods like there are in https://review.openstack.org/69361 cannot be shared between test classes/methods18:38
yfrieddkranz: IDK. I still can't understand the objection - the whole point was that single test case create artificial dependencies18:38
dkranzyfried: Only between test cases18:38
dkranzyfried: I am just saying there is a difference between sharing code and making sure shared code only gets executed a minimum number of times18:39
openstackgerritSergey Lukjanov proposed a change to openstack-dev/devstack: Include SAVANNA_CONF_DIR into SAVANNA_CONF_FILE  https://review.openstack.org/6969218:39
yfrieddkranz: how would you do that? move all shared code to scenario.manager?18:39
dkranzyfried: Yes, or somewhere like that18:40
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/69675 failed with an unrecognized error18:40
yfrieddkranz: take a look at this - maybe it's a better example of what I had in mind - https://review.openstack.org/#/c/69567/18:40
dkranzyfried: Once there is shared code for the important functions it is much easier to deal with the ordering/dependence because the tests cases themmselves become trivial18:40
dkranzyfried: ok, just a sec18:41
*** rossella_s has quit IRC18:41
yfrieddkranz: IMO manger is too crowded as it is. I'm having trouble with the methods that are already there18:41
dkranzyfried: They don't have to be in manager. Or even in a class at all. They could be functions.18:42
dkranzyfried: In some util file or other grouping18:42
dkranzyfried: The point is to just keep the tests themselves simple18:42
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/69674 failed with an unrecognized error18:43
dkranzyfried: But I was not a strong objector to the other way either18:43
yfrieddkranz: well, please let me know what you think about my example. it shows the just what kind of unwanted coupling a single test case creates18:44
dkranzyfried: You mean the original patch you pointed to?18:44
openstackgerritDennyZhang proposed a change to openstack-dev/devstack: Skip curl pip package, if exists  https://review.openstack.org/5508618:44
dkranzyfried: afazekas already gave  it +2 :)18:45
yfriedno. this one https://review.openstack.org/#/c/69567/18:45
yfrieddkranz: the original shows the bad way, with code duplication18:46
*** ewindisch is now known as zz_ewindisch18:46
dkranzyfried: Can you explain why code needs to be duplicated in either case?18:46
yfrieddkranz: are you referring to hotplug_nic (the one afazekas reviewed)?18:49
dkranzyfried: Yes18:49
yfriedit does everything that network_basic does.18:50
dkranzyfried: So why can't they share the code?18:50
*** safchain has quit IRC18:50
openstackrecheckopenstack/nova change: https://review.openstack.org/51328 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:51
*** sdague has quit IRC18:51
openstackrecheckopenstack/nova change: https://review.openstack.org/61717 failed with an unrecognized error18:51
dkranzyfried: Can't there be more methods on NetworkScenarioTest?18:51
dkranzyfried: or stand-alone functions?18:51
openstackrecheckopenstack/nova change: https://review.openstack.org/60621 failed tempest because of: https://bugs.launchpad.net/bugs/125884818:51
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69107 failed with an unrecognized error18:52
*** jmeridth has quit IRC18:53
yfrieddkranz: don't know. I would like to see it as just another test_method in network basic_ops but I got shot down on the mail thread. I was hoping this will bring everyone to my side of the fence, but instead it the fastest +2 I ever got18:53
dkranzyfried: :)18:54
mriedemmtreinish: when calling self.addCleanup on a classmethod, does one need to pass anything for the cls arg?18:54
dkranzyfried: I would not +2 it if those _xxx methods are going to be duplicated in other tests.18:54
mriedemmtreinish: i.e. i'm making delete_volume a class method here now: https://review.openstack.org/#/c/69454/5/tempest/api/compute/base.py18:54
mriedemso tearDownClass can call it18:54
mriedembut not sure how addCleanup will work here: https://review.openstack.org/#/c/69454/5/tempest/api/compute/volumes/test_volumes_get.py18:55
yfrieddkranz: let's say we move all the helper methods to an outside module. this still leaves us with the setUpClass method and its very bad way of collecting resources.18:56
*** sdague has joined #openstack-qa18:56
dkranzyfried: I have to go to a meeting now.18:56
yfrieddkranz: enjoy18:56
dkranzyfried: What if there were a new class that had the ugly setUpClass stuff so it did not have to be duplicated?18:56
openstackrecheckopenstack/nova change: https://review.openstack.org/68659 failed with an unrecognized error18:57
dkranzyfried: ANd these test classes inherited from that?18:57
mtreinishmriedem: can't you just call self.addCleanup(self._delete_volume, volume['id']) or am I missing something18:57
dkranzyfried: Shoot me an email if you have more thoughts and I'll look when I come back.18:57
yfrieddkranz: how is it different from using setUp instead???18:57
*** yrabl has joined #openstack-qa19:00
mriedemmtreinish: that's what i had in a previous patch and it was failing, https://review.openstack.org/#/c/69454/3/tempest/api/compute/volumes/test_volumes_get.py19:04
openstackrecheckopenstack/nova change: https://review.openstack.org/56381 failed with an unrecognized error19:04
mriedemmrodden: looks like you can pass the class or an instance of the class to a classmethod: http://docs.python.org/2/library/functions.html#classmethod19:05
mriedemso should be good to pass self to addCleanup19:05
*** mrodden has quit IRC19:07
mlavalleakamyshnikova: ping19:09
openstackgerritPeter Portante proposed a change to openstack-infra/elastic-recheck: Update fingerprint for bug 1270608  https://review.openstack.org/6970219:10
*** mrodden has joined #openstack-qa19:12
*** jlibosva has quit IRC19:15
openstackrecheckopenstack/nova change: https://review.openstack.org/40467 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:17
openstackrecheckopenstack/nova change: https://review.openstack.org/67351 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:17
openstackrecheckopenstack/nova change: https://review.openstack.org/66652 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:17
*** max_lobur is now known as max_lobur_afk19:17
mtreinishmriedem: yeah that's what I was suggesting (assuming mrodden was supposed to be me)19:18
mtreinishis that you just add the classmethod19:18
mriedemmtreinish: i actually meant mrodden :) i was asking him about it in real life19:19
mriedemmrodden in real life, like that movie19:19
*** andreaf_ has joined #openstack-qa19:19
*** andreaf_ has quit IRC19:19
mriedemmtreinish: did you see cyeoh's comments in that patch, i.e. he didn't think moving the volume creation into the tests will make a difference19:19
mriedemi didn't realize the test cases ran in serial19:20
*** andreaf_ has joined #openstack-qa19:20
mtreinishI guess it would be too crazy if any of the 3 of us confused each other19:20
openstackrecheckopenstack/nova change: https://review.openstack.org/67559 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:20
mtreinishmriedem: yeah the test classes are serialized that's the only way we could work, because a lot of tests have shared resources in setUpClass19:20
mtreinishthat's also why tenant isolation protects us against most contention issues19:21
mriedemmtreinish: so for this one: https://review.openstack.org/#/c/69455/ - that basically means i can leave the volume creation in setUpClass19:21
mriedemstill probably good to have the wait for deletion on teardown though19:21
mriedemand it looks like i could work that down to 1 volume for the test rather than 219:22
mtreinishmriedem: yeah it is the other thing to consider is that using a shared volume in setUpClass if something happens to it19:22
mtreinishyou've screwed up the second test19:22
mriedemmtreinish: that's what i don't like about the shared resources19:22
*** andreaf has quit IRC19:22
mriedemonce something blows up, it snowballs badly19:22
openstackrecheckopenstack/nova change: https://review.openstack.org/67207 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:23
mriedemso if you're ok with keeping the resources isolated to the tests that need them, i'll just leave my patch that way19:23
openstackrecheckopenstack/nova change: https://review.openstack.org/66941 failed with an unrecognized error19:23
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/67199 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:23
mtreinishmriedem: yeah I'm fine with that, it might not fix the bug but because the tests actually do operations on the resources I think it's a good idea19:23
mtreinishhaving shared resources is ok for things like lists tests where you do a bunch of read only operations19:24
mriedemmtreinish: yeah, ok. i'll clean up the commit message since i misunderstood how the parallelization was happening19:24
mriedemthere is more in this test class that needs the same kind of cleanup19:24
mtreinishmriedem: it was probably done that way because volume operations take a long time19:24
mriedemlike the resc_server in setUpClass19:24
mriedemyeah, but in this case it's not even a shared volume19:25
mriedemso doing it in setUpClass or in the test cases should take the same amount of time19:25
mriedemnow i'm flip flopping, we could get away with one volume for the 2 test cases19:26
mriedemso i'll do that, and leave it in setUpClass, and add the wait_for_deletion tear down in case it is causing any issues with parallel volume tests from other classes19:27
mtreinishmriedem: tenant isolation doesn't protect us?19:27
openstackrecheckopenstack/heat change: https://review.openstack.org/68312 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:28
mriedemmtreinish: i mean more in terms of overloading cinder unnecessarily19:28
mriedemcausing timeouts waiting for volumes to be available19:28
mtreinishmriedem: oh ok19:28
openstackrecheckopenstack/python-cinderclient change: https://review.openstack.org/59813 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/125884819:31
openstackrecheckopenstack/nova change: https://review.openstack.org/67581 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:32
openstackrecheckopenstack/keystone change: https://review.openstack.org/69226 failed with an unrecognized error19:33
*** _ruhe is now known as ruhe19:34
openstackgerritLeandro Ignacio Costantino proposed a change to openstack/tempest: Add tempest test for heat resource OS::Nova::KeyPair  https://review.openstack.org/6900019:34
openstackrecheckopenstack/nova change: https://review.openstack.org/67578 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:35
*** rwsu has quit IRC19:36
openstackrecheckopenstack/nova change: https://review.openstack.org/66972 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:36
mtreinishmriedem: one thing about your comment 69455 is that you can't make any assumptions about test ordering19:39
mtreinishthat isn't fixed19:39
mriedemoh, well gdi19:39
mriedem:)19:39
mriedemno wonder it detaches the volume in the last test even though i didn't think that cleanup was needed19:40
mtreinishthey're serialized at the class level but the order in the class level could be random (in practice they aren't normally)19:40
openstackgerritSteve Martinelli proposed a change to openstack-dev/devstack: Convert keystoneclient commands in lib/keystone to openstackclient  https://review.openstack.org/6970519:40
mriedemmtreinish: ok, well i still think a single shared volume will work19:40
*** rwsu has joined #openstack-qa19:40
mriedemtest_rescued_vm_attach_volume is a negative test that tries to attach an unattached volume to a rescued server19:40
mriedemthat fails b/c the server is in the wrong state19:40
openstackrecheckopenstack/cinder change: https://review.openstack.org/69687 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:41
mriedemand test_rescued_vm_detach_volume has a _detach cleanup, so that should be ok also19:41
mriedemfor shared volume19:41
openstackrecheckopenstack/nova change: https://review.openstack.org/63472 failed tempest because of: https://bugs.launchpad.net/bugs/1223469 and https://bugs.launchpad.net/bugs/125884819:41
openstackrecheckopenstack/requirements change: https://review.openstack.org/69496 failed tempest because of: https://bugs.launchpad.net/bugs/125884819:41
mtreinishmriedem: ok that should be fine then19:42
openstackrecheckopenstack/python-heatclient change: https://review.openstack.org/69690 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/125884819:53
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Remove unnecessary volume creation in test_server_rescue  https://review.openstack.org/6945519:53
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Move common _delete_volume cleanup method in base compute test class  https://review.openstack.org/6945419:53
openstackrecheckopenstack/swift change: https://review.openstack.org/69666 failed tempest because of: https://bugs.launchpad.net/bugs/125884820:00
*** Sukhdev has joined #openstack-qa20:02
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69692 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/127338620:02
*** vkozhukalov has quit IRC20:03
*** jlibosva has joined #openstack-qa20:04
*** rfolco has quit IRC20:05
*** stack555 has quit IRC20:08
*** raies has quit IRC20:08
openstackrecheckopenstack/tempest change: https://review.openstack.org/69625 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1251448 and https://bugs.launchpad.net/bugs/1273386 and https://bugs.launchpad.net/bugs/125764120:08
*** nmagnezi has quit IRC20:11
*** mrodden1 has joined #openstack-qa20:12
*** mrodden has quit IRC20:13
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69322 failed tempest because of: https://bugs.launchpad.net/bugs/125884820:20
openstackrecheckopenstack/python-heatclient change: https://review.openstack.org/67153 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/125884820:27
*** mrodden1 is now known as mrodden20:29
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/67199 failed tempest because of: https://bugs.launchpad.net/bugs/125884820:34
*** safchain has joined #openstack-qa20:37
jgriffithmriedem: FYI I'm working up a patch that changes how we do targets20:48
jgriffithmriedem: unit tests may take a while but it's on the way20:48
openstackrecheckopenstack/heat change: https://review.openstack.org/68312 failed tempest because of: https://bugs.launchpad.net/bugs/125884820:49
mriedemjgriffith: ok, thanks20:53
*** ruhe is now known as _ruhe20:54
*** tkammer has quit IRC20:57
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69705 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/125884820:58
openstackrecheckopenstack/horizon change: https://review.openstack.org/66603 failed with an unrecognized error21:00
*** _ruhe is now known as ruhe21:00
openstackrecheckopenstack/nova change: https://review.openstack.org/56381 failed tempest because of: https://bugs.launchpad.net/bugs/125884821:03
*** safchain has quit IRC21:05
sdaguestevebaker: just as an out of band follow up, the devstack changes to have heat bind on all addresses didn't address the port issue?21:07
openstackrecheckopenstack/keystone change: https://review.openstack.org/69632 failed with an unrecognized error21:08
*** NewInformator has joined #openstack-qa21:08
*** NewInformator has left #openstack-qa21:08
*** yassine has joined #openstack-qa21:12
openstackrecheckopenstack/tempest change: https://review.openstack.org/69455 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1258848 and https://bugs.launchpad.net/bugs/127338621:12
openstackgerritA change was merged to openstack-dev/devstack: Fix Swift process kill  https://review.openstack.org/6944021:13
mtreinishmriedem: ^^^ ooh a triple :)21:13
mriedemha21:14
openstackrecheckopenstack/nova change: https://review.openstack.org/63472 failed tempest because of: https://bugs.launchpad.net/bugs/1253896 and https://bugs.launchpad.net/bugs/125884821:14
openstackrecheckopenstack/python-heatclient change: https://review.openstack.org/69690 failed with an unrecognized error21:14
openstackrecheckopenstack/tempest change: https://review.openstack.org/69454 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1258848 and https://bugs.launchpad.net/bugs/127338621:28
openstackrecheckopenstack/ceilometer change: https://review.openstack.org/67850 failed tempest because of: https://bugs.launchpad.net/bugs/125884821:30
mriedemmtreinish: interesting that e-r hit those, since it really failed due to code bugs21:34
*** coolsvap has quit IRC21:35
mriedemapparently i'm not callin my classmethod correctly21:35
openstackrecheckopenstack/nova change: https://review.openstack.org/40467 failed with an unrecognized error21:38
mriedemmtreinish: i'm seeing this in the logs for 69454: "RuntimeError: ("tearDownClass does not calls the super's tearDownClass in these classes: set([<class 'tempest.api.compute.volumes.test_volumes_list.VolumesTestXML'>, <class 'tempest.api.compute.volumes.test_volumes_list.VolumesTestJSON'>])\nIf you see the exception, with another exception please do not report this one!If you are changing tempest code, make sure you", "are calling the super21:41
mriedemhttp://logs.openstack.org/54/69454/6/check/check-tempest-dsvm-full/29cb2fa/console.html#_2014-01-28_21_09_06_84621:41
mriedembut i didn't remove a tearDownClass in that class21:41
mriedem?21:41
mriedemoh, my guess is it's just because tearDownClass puked21:42
mriedemin test_volumes_list21:42
mtreinishmriedem: yeah that get's raised whenever the base classes tearDownClass doesn't run on a subclass21:42
openstackgerritJeff Peeler proposed a change to openstack/tempest: Return body output after given status reached  https://review.openstack.org/6972521:44
openstackgerritJeff Peeler proposed a change to openstack/tempest: Enhance templates test  https://review.openstack.org/6972621:44
openstackrecheckopenstack/cinder change: https://review.openstack.org/69719 failed tempest because of: https://bugs.launchpad.net/bugs/125489021:46
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69692 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/1258848 and https://bugs.launchpad.net/bugs/127338621:47
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Remove unnecessary volume creation in test_server_rescue  https://review.openstack.org/6945521:50
openstackgerritMatt Riedemann proposed a change to openstack/tempest: Move common _delete_volume cleanup method in base compute test class  https://review.openstack.org/6945421:50
openstackrecheckopenstack/python-keystoneclient change: https://review.openstack.org/67199 failed tempest because of: https://bugs.launchpad.net/bugs/125884821:51
*** packet has joined #openstack-qa21:57
*** zz_ewindisch is now known as ewindisch21:59
openstackgerritJakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron  https://review.openstack.org/5869521:59
*** SergeyLukjanov is now known as SergeyLukjanov_21:59
openstackrecheckopenstack/oslo.messaging change: https://review.openstack.org/68959 failed tempest because of: https://bugs.launchpad.net/bugs/125884822:01
*** Sukhdev has quit IRC22:01
openstackgerritJakub Libosvar proposed a change to openstack-dev/devstack: Stop all neutron-ns-metadata-proxy with stop_neutron  https://review.openstack.org/6972922:03
*** jlibosva has quit IRC22:05
*** harlowja is now known as harlowja_away22:10
*** amotoki has quit IRC22:14
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Only plot gate failures in status.o.o/elastic-search  https://review.openstack.org/6973522:16
openstackgerritJoe Gordon proposed a change to openstack-infra/elastic-recheck: Only plot gate failures in status.o.o/elastic-recheck/  https://review.openstack.org/6973522:17
*** harlowja_away is now known as harlowja22:18
openstackgerritA change was merged to openstack-dev/devstack: Repeatedly add log_filters,log_outputs to libvirtd.conf when restart  https://review.openstack.org/6928322:30
openstackgerritA change was merged to openstack-dev/devstack: Convert trove to plugin  https://review.openstack.org/6949922:30
mriedemmtreinish: sdague: can i get a -2 on this until jgriffith works up his cinder patch? https://review.openstack.org/#/c/69203/22:30
mriedemthat should also apply here: https://review.openstack.org/#/c/69599/22:31
mriedemthere can be only one22:31
sdaguemriedem: done22:32
sdagueand done22:33
sdaguemriedem: thanks for being on top of that22:33
mriedemon that note, i'm out of here for the day and leaving the laptop at work, i've done a terrible job of parenting the last few days22:34
mriedemcartoons and mac n' cheese22:35
sdagueenjoy22:35
mriedemmy daughter will soon look like that feral kid from the road warrior22:35
*** mriedem has quit IRC22:36
*** dkranz has quit IRC22:36
openstackrecheckopenstack/gantt change: https://review.openstack.org/68520 failed with an unrecognized error22:45
openstackgerritA change was merged to openstack/tempest: Add test for volume list with all-tenants  https://review.openstack.org/6677422:48
openstackrecheckopenstack/tempest change: https://review.openstack.org/69528 failed tempest because of: https://bugs.launchpad.net/bugs/125884822:48
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69705 failed with an unrecognized error22:49
openstackgerritMax Lobur proposed a change to openstack/tempest: Remove baremetal xml support  https://review.openstack.org/6687822:51
openstackrecheckopenstack/horizon change: https://review.openstack.org/66603 failed tempest because of: https://bugs.launchpad.net/bugs/125884822:54
*** openstack has joined #openstack-qa23:04
openstackrecheckopenstack/nova change: https://review.openstack.org/40467 failed with an unrecognized error23:04
*** andreaf_ has quit IRC23:08
*** ruhe is now known as _ruhe23:10
openstackrecheckopenstack/tempest change: https://review.openstack.org/69455 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:16
openstackrecheckopenstack-dev/devstack change: https://review.openstack.org/69729 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:23
*** dims has quit IRC23:24
openstackrecheckopenstack/oslo.messaging change: https://review.openstack.org/59714 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:24
openstackrecheckopenstack/nova change: https://review.openstack.org/69731 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:24
openstackrecheckopenstack/nova change: https://review.openstack.org/69733 failed with an unrecognized error23:25
openstackrecheckopenstack/swift change: https://review.openstack.org/69509 failed with an unrecognized error23:28
openstackrecheckopenstack/nova change: https://review.openstack.org/68319 failed tempest because of: https://bugs.launchpad.net/bugs/124906523:29
*** giulivo has quit IRC23:36
openstackrecheckopenstack/oslo.messaging change: https://review.openstack.org/68959 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:36
openstackrecheckopenstack/tempest change: https://review.openstack.org/69454 failed tempest because of: https://bugs.launchpad.net/bugs/1254890 and https://bugs.launchpad.net/bugs/125884823:38
*** dims has joined #openstack-qa23:39
openstackrecheckopenstack/nova change: https://review.openstack.org/52733 failed with an unrecognized error23:46
openstackgerritJoe Gordon proposed a change to openstack-infra/devstack-gate: Add support to run nova-api-metadata as separate binary  https://review.openstack.org/6830423:48
openstackrecheckopenstack/keystone change: https://review.openstack.org/54647 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:51
openstackgerritChenZheng proposed a change to openstack/tempest: Add extra list servers by host test  https://review.openstack.org/6523423:53
*** mlavalle has quit IRC23:57
openstackgerritA change was merged to openstack/tempest: Separate negative tests for test_templates  https://review.openstack.org/6929123:59
openstackgerritA change was merged to openstack-dev/devstack: allow for upgrade of the precise kernel  https://review.openstack.org/6946423:59
openstackrecheckopenstack/horizon change: https://review.openstack.org/69581 failed tempest because of: https://bugs.launchpad.net/bugs/125884823:59

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