*** lyxus has joined #openstack-qa | 00:01 | |
lyxus | Hello Folks, I want to write some test unit for neutron. Is there a doc on how to write a decent test ? I saw only reference to tempest | 00:02 |
---|---|---|
*** GMann has quit IRC | 00:03 | |
*** sthaha has joined #openstack-qa | 00:04 | |
*** andreaf2 has quit IRC | 00:06 | |
*** GMann has joined #openstack-qa | 00:06 | |
openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Remove created routers as part of test cleanup https://review.openstack.org/84904 | 00:12 |
*** mmaglana has quit IRC | 00:14 | |
*** maru_afk is now known as marun | 00:20 | |
*** yamahata has joined #openstack-qa | 00:22 | |
*** mlavalle has quit IRC | 00:22 | |
openstackrecheck | Console logs not available after 13:21s for gate-swift-pep8 81207,9,60de005 | 00:25 |
*** matsuhashi has joined #openstack-qa | 00:28 | |
*** markmcclain has quit IRC | 00:36 | |
*** zhikunliu has joined #openstack-qa | 00:44 | |
*** mriedem has joined #openstack-qa | 00:48 | |
mriedem | clarkb: can you educate me on if tags:console here is better than tags:"console" or if it doesn't matter? i'm still confused on tags | 00:48 |
mriedem | https://review.openstack.org/#/c/84558/3/queries/1301027.yaml | 00:48 |
*** bill_az has quit IRC | 00:51 | |
openstackgerrit | Daisuke Morita proposed a change to openstack/tempest: Add parametric tests of Swift object API, part 1 https://review.openstack.org/81703 | 00:52 |
*** GMann has quit IRC | 01:01 | |
*** harlowja is now known as harlowja_away | 01:01 | |
*** matsuhashi has quit IRC | 01:05 | |
*** matsuhashi has joined #openstack-qa | 01:06 | |
*** yamahata has quit IRC | 01:11 | |
*** mrodden has quit IRC | 01:11 | |
*** TravT has quit IRC | 01:14 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify "list interfaces" Nova v2/v3 API response attributes https://review.openstack.org/83269 | 01:28 |
*** nosnos has joined #openstack-qa | 01:29 | |
*** rfolco has quit IRC | 01:32 | |
*** dmorita has joined #openstack-qa | 01:38 | |
openstackgerrit | Jerry Cai proposed a change to openstack/tempest: Volume size could be specified to create volume https://review.openstack.org/81471 | 01:48 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify create/get flavor attributes of Nova APIs https://review.openstack.org/80543 | 01:48 |
*** GMann has joined #openstack-qa | 01:48 | |
*** sahumada has quit IRC | 01:53 | |
*** sahumada has joined #openstack-qa | 01:57 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify create agent attributes of V2/V3 APIs https://review.openstack.org/84344 | 01:57 |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify create agent attributes of V2/V3 APIs https://review.openstack.org/84344 | 01:57 |
openstackrecheck | Console logs not available after 13:23s for check-grenade-dsvm 84639,1,8fe4e9e | 01:59 |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify update agent attributes of V2/V3 APIs https://review.openstack.org/84663 | 02:00 |
openstackrecheck | openstack/nova change: https://review.openstack.org/68093 failed gate-grenade-dsvm-partial-ncpu in the gate queue with an unrecognized error | 02:03 |
*** malini_afk is now known as malini | 02:04 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add V2 Nova API os-agents tests https://review.openstack.org/78111 | 02:06 |
*** HenryG has quit IRC | 02:08 | |
*** otherwiseguy has joined #openstack-qa | 02:23 | |
clarkb | mriedem: I prefer tags:"console" because it is a bit more explicit, but tags:console is equivalent in that case | 02:30 |
clarkb | mriedem: unquoted allows you to do globs and searches. quoted is verbatim in this case they are equivalent | 02:30 |
clarkb | mriedem: as far as tags goes think of it as a classification mechanism | 02:30 |
clarkb | mriedem: in this case the console tag indicates it came from console data somewhere. Could be a unittest job or the devstack log or a grenade log | 02:31 |
mriedem | clarkb: ok that make sense with what you were saying about not using project:openstack/neutron then and devstack logs | 02:32 |
mriedem | i kept thinking of tags:console == filename:console.html | 02:32 |
clarkb | mriedem: ya not wuite as it covers more | 02:33 |
mriedem | clarkb: i'd like to get something in the e-r readme about tags, just haven't gotten around to it | 02:33 |
mriedem | clarkb: and if you're ok with https://review.openstack.org/#/c/84558/ now it's all yours | 02:33 |
mriedem | the bug is fixed, so just to get classification rates up | 02:33 |
clarkb | mriedem: done | 02:34 |
mriedem | clarkb: cool, thanks - time for me to sign off, ttyl | 02:34 |
*** GMann has quit IRC | 02:34 | |
openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add a fingerprint for bug 1301027 https://review.openstack.org/84558 | 02:37 |
*** GMann has joined #openstack-qa | 02:37 | |
*** mriedem has quit IRC | 02:37 | |
*** mrodden has joined #openstack-qa | 02:39 | |
*** malini is now known as malini_afk | 02:47 | |
*** weshay has joined #openstack-qa | 02:53 | |
*** mrodden has quit IRC | 03:00 | |
*** mrodden has joined #openstack-qa | 03:02 | |
*** matsuhashi has quit IRC | 03:05 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/84869 failed because of: gate-tempest-dsvm-postgres-full: https://bugs.launchpad.net/bugs/1253896 | 03:08 |
*** shwetaap has joined #openstack-qa | 03:09 | |
*** mrodden has quit IRC | 03:22 | |
*** nosnos has quit IRC | 03:23 | |
*** otherwiseguy has quit IRC | 03:28 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add V2 Nova API os-agents tests https://review.openstack.org/78111 | 03:29 |
*** GMann_ has joined #openstack-qa | 03:29 | |
openstackrecheck | Console logs not available after 13:22s for gate-nova-python26 83256,11,7b833b5 | 03:31 |
*** GMann has quit IRC | 03:31 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/84907 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 03:43 |
*** Madan has joined #openstack-qa | 03:55 | |
*** harlowja_away is now known as harlowja | 04:00 | |
*** mmaglana has joined #openstack-qa | 04:02 | |
*** nosnos has joined #openstack-qa | 04:09 | |
*** matsuhashi has joined #openstack-qa | 04:15 | |
*** psharma has joined #openstack-qa | 04:18 | |
*** shwetaap has quit IRC | 04:20 | |
*** oomichi has joined #openstack-qa | 04:21 | |
*** oomichi has quit IRC | 04:23 | |
*** mrodden has joined #openstack-qa | 04:24 | |
*** yfried__ has quit IRC | 04:24 | |
*** ken1ohmichi has joined #openstack-qa | 04:32 | |
*** masayukig has quit IRC | 04:32 | |
*** masayukig has joined #openstack-qa | 04:32 | |
*** ken1ohmichi has quit IRC | 04:33 | |
*** oomichi has joined #openstack-qa | 04:35 | |
*** matsuhashi has quit IRC | 04:40 | |
*** weshay has quit IRC | 04:43 | |
*** ken1ohmichi has joined #openstack-qa | 04:43 | |
*** ken1ohmichi has left #openstack-qa | 04:44 | |
*** rohitk has joined #openstack-qa | 04:44 | |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify attributes through Nova os-security-groups API https://review.openstack.org/82998 | 04:45 |
*** matsuhashi has joined #openstack-qa | 04:46 | |
openstackgerrit | A change was merged to openstack-dev/devstack: bring back oslo_clean https://review.openstack.org/84891 | 04:47 |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify update agent attributes of V2/V3 APIs https://review.openstack.org/84663 | 04:47 |
*** ashishsnigam has joined #openstack-qa | 04:50 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Missing baremetal NodeState API test https://review.openstack.org/84643 | 04:51 |
*** yamahata has joined #openstack-qa | 05:02 | |
*** mmaglana has quit IRC | 05:05 | |
*** yamahata has quit IRC | 05:12 | |
*** yamahata has joined #openstack-qa | 05:12 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Network fwaas API test https://review.openstack.org/78857 | 05:12 |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify delete quota response of V2/V3 APIs https://review.openstack.org/84944 | 05:17 |
*** catohornet has quit IRC | 05:30 | |
*** harlowja is now known as harlowja_away | 05:32 | |
*** yfried__ has joined #openstack-qa | 05:36 | |
openstackgerrit | xu-haiwei proposed a change to openstack/tempest: Verify the response attributes of 'aggregate-set-metadata' API https://review.openstack.org/84948 | 05:47 |
*** andreaf2 has joined #openstack-qa | 05:57 | |
enikanorov_ | sdague: hi | 05:57 |
*** jlibosva has joined #openstack-qa | 06:04 | |
*** andreaf_ has joined #openstack-qa | 06:05 | |
*** andreaf_ has quit IRC | 06:07 | |
*** andreaf_ has joined #openstack-qa | 06:07 | |
*** andreaf has quit IRC | 06:08 | |
*** andreaf_ has quit IRC | 06:09 | |
*** andreaf has joined #openstack-qa | 06:09 | |
*** andreaf has quit IRC | 06:10 | |
*** andreaf has joined #openstack-qa | 06:10 | |
*** andreaf2 has quit IRC | 06:12 | |
*** rohitk has quit IRC | 06:12 | |
openstackgerrit | Ann Kamyshnikova proposed a change to openstack/tempest: Verify more information in floating ip tests https://review.openstack.org/63723 | 06:16 |
*** psharma has quit IRC | 06:16 | |
*** tkammer has joined #openstack-qa | 06:20 | |
*** evgenyf has joined #openstack-qa | 06:21 | |
*** rohitk has joined #openstack-qa | 06:31 | |
*** psharma has joined #openstack-qa | 06:31 | |
*** evgenyf has quit IRC | 06:33 | |
*** matsuhashi has quit IRC | 06:36 | |
*** matsuhashi has joined #openstack-qa | 06:39 | |
*** masayukig has quit IRC | 06:52 | |
*** jpich has joined #openstack-qa | 06:52 | |
*** afazekas has joined #openstack-qa | 06:56 | |
*** luqas has joined #openstack-qa | 06:58 | |
*** luqas has quit IRC | 07:00 | |
*** luqas has joined #openstack-qa | 07:01 | |
*** psharma has quit IRC | 07:03 | |
raies | afazekas: hi please check it needs approval https://review.openstack.org/#/c/83663/ | 07:03 |
*** luqas has quit IRC | 07:03 | |
*** zigo has quit IRC | 07:04 | |
*** sdague has quit IRC | 07:05 | |
*** sdague has joined #openstack-qa | 07:05 | |
*** zigo has joined #openstack-qa | 07:08 | |
*** rohitk has quit IRC | 07:10 | |
*** mikal has quit IRC | 07:10 | |
*** mikal has joined #openstack-qa | 07:10 | |
*** evgenyf has joined #openstack-qa | 07:19 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 07:19 |
*** marun is now known as marun_afk | 07:29 | |
*** qba73 has joined #openstack-qa | 07:29 | |
*** psharma has joined #openstack-qa | 07:30 | |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add decorator for safe tearDown https://review.openstack.org/84645 | 07:34 |
*** luqas has joined #openstack-qa | 07:45 | |
openstackgerrit | Amit Ugol proposed a change to openstack/tempest: Fixed a few places as per the comments https://review.openstack.org/84786 | 07:47 |
*** luqas has quit IRC | 07:47 | |
*** jordanP has joined #openstack-qa | 07:47 | |
raies | afazekas: please give your opinion on my comment in https://review.openstack.org/84643 patch 3 | 07:49 |
openstackgerrit | Adam Gandelman proposed a change to openstack/tempest: Ignore unbound ports when testing binding attrs https://review.openstack.org/84954 | 07:50 |
*** luqas has joined #openstack-qa | 07:51 | |
*** safchain has joined #openstack-qa | 07:53 | |
*** VinodGupta has joined #openstack-qa | 07:57 | |
*** yassine has joined #openstack-qa | 08:08 | |
*** locke105 has quit IRC | 08:16 | |
*** fbo_away is now known as fbo | 08:16 | |
*** [1]evgenyf has joined #openstack-qa | 08:17 | |
*** evgenyf has quit IRC | 08:19 | |
*** [1]evgenyf is now known as evgenyf | 08:19 | |
*** GMann_ has quit IRC | 08:31 | |
openstackgerrit | Elena Ezhova proposed a change to openstack/tempest: Remove test_create_port_with_no_ip https://review.openstack.org/83002 | 08:35 |
*** GMann has joined #openstack-qa | 08:36 | |
*** vrovachev has joined #openstack-qa | 08:40 | |
openstackgerrit | Nikita Konovalov proposed a change to openstack/tempest: Add Sahara client for scenarios tests https://review.openstack.org/83317 | 08:49 |
*** ylobankov has joined #openstack-qa | 08:51 | |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Remove separate tearDown server cleanup https://review.openstack.org/84959 | 08:53 |
openstackgerrit | Marc Koderer proposed a change to openstack/tempest: Add decorator for safe tearDown https://review.openstack.org/84645 | 08:53 |
*** psharma has quit IRC | 08:57 | |
*** matsuhashi has quit IRC | 09:07 | |
openstackgerrit | Yair Fried proposed a change to openstack/tempest: Enable security_groups_basic_ops blocking tests https://review.openstack.org/62702 | 09:07 |
*** GMann has quit IRC | 09:11 | |
*** VinodGupta has quit IRC | 09:11 | |
*** matsuhashi has joined #openstack-qa | 09:12 | |
*** ams0 has joined #openstack-qa | 09:13 | |
*** morganfainberg is now known as morganfainberg_Z | 09:15 | |
openstackgerrit | rendh proposed a change to openstack/tempest: Add multi role/tenant to user and delete them from user https://review.openstack.org/72547 | 09:21 |
*** luqas has quit IRC | 09:23 | |
*** GMann has joined #openstack-qa | 09:23 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add V2 Nova API os-agents tests https://review.openstack.org/78111 | 09:25 |
*** cyeoh has quit IRC | 09:42 | |
*** cyeoh has joined #openstack-qa | 09:43 | |
*** Madan has quit IRC | 09:45 | |
openstackgerrit | rendh proposed a change to openstack/tempest: Add multi role/tenant to user and delete them from user https://review.openstack.org/72547 | 09:45 |
*** luqas has joined #openstack-qa | 09:45 | |
*** psharma has joined #openstack-qa | 09:46 | |
*** tosky has joined #openstack-qa | 09:48 | |
*** yassine has quit IRC | 09:48 | |
*** mkollaro has joined #openstack-qa | 09:53 | |
openstackgerrit | afazekas proposed a change to openstack/tempest: Wating for ACTIVE state in rescue tests https://review.openstack.org/84975 | 09:54 |
openstackgerrit | Abhijeet Jain proposed a change to openstack/tempest: Nova API server-group https://review.openstack.org/84977 | 09:56 |
raies | afazekas: I had confirmed that on IRC too | 09:59 |
raies | fake driver is by default creted during installation | 09:59 |
raies | afazekas: ^^ | 09:59 |
*** matsuhashi has quit IRC | 09:59 | |
*** Madan has joined #openstack-qa | 09:59 | |
*** matsuhashi has joined #openstack-qa | 09:59 | |
*** fbo is now known as fbo_away | 10:09 | |
*** mkollaro has quit IRC | 10:10 | |
*** zhikunliu has quit IRC | 10:13 | |
openstackgerrit | Ralf Haferkamp proposed a change to openstack-dev/devstack: Use iniset for changing my.conf and avoid deprecated config options https://review.openstack.org/79635 | 10:13 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify Server Actions attributes of Nova APIs https://review.openstack.org/84380 | 10:18 |
*** chandankumar_ has joined #openstack-qa | 10:22 | |
*** dmorita has quit IRC | 10:22 | |
*** ashishsnigam has quit IRC | 10:27 | |
afazekas | raies: it is created, but is it always configured ? | 10:28 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify Server Actions attributes of Nova APIs https://review.openstack.org/84380 | 10:28 |
afazekas | raies: https://review.openstack.org/#/c/84643/ I see logic in both argument | 10:32 |
raies | afazekas: Then what you say ?? | 10:33 |
raies | I think taking it separate will be better | 10:33 |
afazekas | raies: If I really need to pick choose about organization, or resource saving, I would prefer the resource saving | 10:33 |
*** Madan has quit IRC | 10:34 | |
raies | taking it with node file will be better ?? | 10:34 |
raies | In my opinion API tests should be separated as per module | 10:35 |
raies | but still will go with what you suggest | 10:35 |
raies | afazekas: ^^ | 10:35 |
afazekas | looks like it would create the same resources even if it would be in the test_nodes.py | 10:38 |
*** Madan has joined #openstack-qa | 10:39 | |
afazekas | GMann: Can you reevaluate this https://review.openstack.org/#/c/84643/ ? At the first look it does not looks more efficient in the test_nodes.py | 10:40 |
*** ashishsnigam has joined #openstack-qa | 10:43 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: Missing baremetal NodeState API test https://review.openstack.org/84643 | 10:44 |
openstackgerrit | Sergey Lukjanov proposed a change to openstack-infra/devstack-gate: Prepare for Juno dev https://review.openstack.org/83075 | 10:48 |
*** matsuhashi has quit IRC | 10:48 | |
openstackgerrit | A change was merged to openstack/tempest: Server create - JSON schema validation: adminPass is optional https://review.openstack.org/84673 | 10:49 |
*** Madan has quit IRC | 10:49 | |
*** obondarev1 has joined #openstack-qa | 10:50 | |
*** obondarev has quit IRC | 10:50 | |
*** VinodGupta has joined #openstack-qa | 10:50 | |
*** oomichi is now known as ken1ohmichi | 10:52 | |
openstackgerrit | Abhijeet Jain proposed a change to openstack/tempest: Nova Service API Negative Test https://review.openstack.org/84353 | 10:53 |
openstackgerrit | afazekas proposed a change to openstack/tempest: Waiting for ACTIVE state in rescue tests https://review.openstack.org/84975 | 10:55 |
*** VinodGupta has quit IRC | 10:55 | |
*** Madan has joined #openstack-qa | 10:55 | |
*** matsuhashi has joined #openstack-qa | 10:55 | |
openstackgerrit | enikanorov proposed a change to openstack/tempest: Unskip load balancer basic scenario test https://review.openstack.org/82480 | 10:58 |
GMann | afazekas: regarding - https://review.openstack.org/#/c/84643/. I still prefer to merge. because this API gets the information from the resource (node in this case) which is already created in test_nodes.py. | 11:06 |
GMann | afazekas: Logically if we go i think we should run this test in test_nodes.py only as resource will be reused and test execution will be fast. | 11:06 |
enikanorov_ | afazekas: could you please +2 https://review.openstack.org/82480 again? updated commit msg per sdague request | 11:07 |
GMann | afazekas: whats your point of view? | 11:08 |
jlibosva | dtroyer: hi, sorry for bothering again :) Any news on grenade stuff? If you need any assistance, please let me know | 11:08 |
*** afazekas has quit IRC | 11:09 | |
openstackgerrit | A change was merged to openstack-infra/devstack-gate: Correcting the right path to mysql log in Fedora 19 https://review.openstack.org/79967 | 11:09 |
*** VinodGupta has joined #openstack-qa | 11:11 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add V2 Nova API os-agents tests https://review.openstack.org/78111 | 11:14 |
*** ashishsnigam has quit IRC | 11:14 | |
*** matsuhashi has quit IRC | 11:15 | |
*** yassine has joined #openstack-qa | 11:20 | |
*** mkollaro has joined #openstack-qa | 11:21 | |
*** matsuhashi has joined #openstack-qa | 11:21 | |
*** afazekas has joined #openstack-qa | 11:24 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: normalize the services for a post grizzly world https://review.openstack.org/84287 | 11:25 |
openstackgerrit | Yuiko Takada proposed a change to openstack/tempest: Verify update agent attributes of V2/V3 APIs https://review.openstack.org/84663 | 11:27 |
*** luqas has quit IRC | 11:36 | |
openstackgerrit | Mh Raies proposed a change to openstack/tempest: VPNaas IPSec policies tests https://review.openstack.org/47816 | 11:36 |
openstackgerrit | Abhijeet Jain proposed a change to openstack/tempest: Add tests for server-group Nova V2 APIs https://review.openstack.org/84977 | 11:39 |
*** rfolco has joined #openstack-qa | 11:45 | |
*** HenryG has joined #openstack-qa | 11:47 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Remove the unused _get_unused_flavor_id() https://review.openstack.org/85001 | 11:50 |
*** matsuhashi has quit IRC | 11:52 | |
*** weshay has joined #openstack-qa | 11:56 | |
openstackgerrit | Julien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH https://review.openstack.org/74067 | 11:56 |
openstackgerrit | Julien Leloup proposed a change to openstack/tempest: Stress Test for server availability through SSH https://review.openstack.org/74067 | 12:00 |
*** ashishsnigam has joined #openstack-qa | 12:00 | |
*** mriedem has joined #openstack-qa | 12:00 | |
openstackgerrit | Sébastien Han proposed a change to openstack-dev/devstack: Implement Ceph backend for Glance / Cinder / Nova https://review.openstack.org/65113 | 12:03 |
openstackgerrit | Sébastien Han proposed a change to openstack-dev/devstack: Implement Ceph backend for Glance / Cinder / Nova https://review.openstack.org/65113 | 12:04 |
openstackgerrit | Sébastien Han proposed a change to openstack-dev/devstack: Implement Ceph backend for Glance / Cinder / Nova https://review.openstack.org/65113 | 12:06 |
*** ashishsnigam has quit IRC | 12:06 | |
*** yamahata has quit IRC | 12:10 | |
*** StevenK_ has joined #openstack-qa | 12:10 | |
*** fbo_away is now known as fbo | 12:10 | |
*** mrodden has quit IRC | 12:10 | |
*** mrodden has joined #openstack-qa | 12:10 | |
*** StevenK has quit IRC | 12:10 | |
*** Madan has quit IRC | 12:20 | |
*** aysyd has joined #openstack-qa | 12:26 | |
*** luqas has joined #openstack-qa | 12:29 | |
openstackgerrit | Aleksandr Gordeev proposed a change to openstack-dev/devstack: Delete unused neutron port https://review.openstack.org/85013 | 12:29 |
*** shwetaap has joined #openstack-qa | 12:42 | |
anteaya | mtreinish: we don't care if they have signed an agreement for specs-cookiecutter? https://review.openstack.org/#/c/83192/3/modules/openstack_project/files/gerrit/acls/openstack-dev/specs-cookiecutter.config | 12:43 |
anteaya | and usually refs/tags/* is just -ptl not -core | 12:44 |
openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Verify Server Actions attributes of Nova APIs https://review.openstack.org/84380 | 12:45 |
*** nosnos has quit IRC | 12:50 | |
*** GMann has quit IRC | 12:53 | |
*** psharma has quit IRC | 12:55 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Add python-dev to general requirements https://review.openstack.org/80547 | 12:58 |
*** StevenK_ is now known as StevenK | 13:02 | |
openstackgerrit | A change was merged to openstack-dev/devstack: Makes error message easier to understand. https://review.openstack.org/79464 | 13:02 |
*** adalbas has quit IRC | 13:03 | |
*** adalbas has joined #openstack-qa | 13:04 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: setup_install should pip install -U https://review.openstack.org/85027 | 13:04 |
*** samuelbercovici has joined #openstack-qa | 13:05 | |
*** dims_ has quit IRC | 13:07 | |
dkranz | sdague, mtreinish : So here is an initial result master against stable/havana with nova-network (no heat/ceilo/swift) http://paste.openstack.org/show/74966/ | 13:07 |
enikanorov_ | sdague: ping | 13:07 |
dkranz | sdague, mtreinish : Looks like lots of api changes to me. | 13:07 |
sdague | dkranz: did you did into the fails? | 13:08 |
sdague | I'm not convinced that's API changes | 13:08 |
sdague | that's configuration failure most like | 13:08 |
sdague | dkranz: setUpClass (tempest.api.image.v1.test_images.ListSnapshotImagesTest) ... FAIL | 13:09 |
sdague | 13:09 | |
sdague | there is no way that api actually changed | 13:09 |
sdague | enikanorov_: what's up? | 13:09 |
enikanorov_ | sdague: fixed commit msg in https://review.openstack.org/#/c/82480/ please take a look | 13:10 |
dkranz | sdague: Just starting. Some are config. But a bunch are from the new validation checks and some where JSON broke but not xml. | 13:10 |
dkranz | sdague: I'll post the whole log with stacktraces if I can. I am starting to look at them. | 13:11 |
sdague | cool | 13:11 |
sdague | enikanorov_: that still doesn't explain why 9 ports are openned up | 13:12 |
sdague | I'm very curious why the range is required | 13:12 |
sdague | and not just 80 | 13:12 |
enikanorov_ | well, we use ports 80 and 88. could create two rules intead of one | 13:12 |
enikanorov_ | i don't think it's a big deal | 13:12 |
sdague | I think it's good to be explicit | 13:13 |
sdague | where is 88 used? or is that private implementation in lbaas? | 13:13 |
enikanorov_ | no, it's just to create another backend http on the same VM | 13:13 |
enikanorov_ | just to save on spawning another VM | 13:14 |
sdague | ok | 13:14 |
enikanorov_ | so are you suggesting to add two rules for 80 and 88 ports separately? | 13:14 |
sdague | so please change the sec groups to be explict for the ports in question instead of a range | 13:14 |
sdague | yes | 13:14 |
enikanorov_ | ok | 13:14 |
openstackgerrit | enikanorov proposed a change to openstack/tempest: Unskip load balancer basic scenario test https://review.openstack.org/82480 | 13:18 |
*** samuelbercovici1 has joined #openstack-qa | 13:19 | |
*** chandankumar_ has quit IRC | 13:19 | |
*** dims_ has joined #openstack-qa | 13:20 | |
openstackgerrit | enikanorov proposed a change to openstack/tempest: Unskip load balancer basic scenario test https://review.openstack.org/82480 | 13:20 |
*** rfolco has quit IRC | 13:20 | |
*** samuelbercovici has quit IRC | 13:20 | |
*** samuelbercovici1 is now known as samuelbercovici | 13:20 | |
*** rfolco has joined #openstack-qa | 13:22 | |
afazekas | sdague, dkranz: https://review.openstack.org/#/c/84975/2 | 13:27 |
*** weshay has quit IRC | 13:27 | |
sdague | afazekas: nice fine | 13:28 |
sdague | find | 13:28 |
sdague | +2 | 13:29 |
*** weshay has joined #openstack-qa | 13:30 | |
afazekas | thx | 13:30 |
*** ken1ohmichi has quit IRC | 13:39 | |
*** VinodGupta has quit IRC | 13:39 | |
*** samuelbercovici1 has joined #openstack-qa | 13:43 | |
*** jgrimm has quit IRC | 13:45 | |
*** malini_afk is now known as malini | 13:46 | |
*** samuelbercovici has quit IRC | 13:47 | |
*** samuelbercovici1 is now known as samuelbercovici | 13:47 | |
*** otherwiseguy has joined #openstack-qa | 13:55 | |
*** GMann has joined #openstack-qa | 13:56 | |
yfried__ | afazekas: sdague: dkranz: how can I check gate history of a specific test? | 13:56 |
sdague | you can get the last 2 weeks out of logstash | 13:57 |
*** obondarev1 is now known as obondarev | 13:59 | |
yfried__ | sdague: could you please explain/send link? | 13:59 |
*** kgriffs|afk is now known as kgriffs | 13:59 | |
sdague | yfried__: all the output from all the test runs for the last 2 weeks are indexed here - http://logstash.openstack.org/#eyJzZWFyY2giOiIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6OTAwLCJncmFwaG1vZGUiOiJjb3VudCJ9 | 13:59 |
sdague | it's a search engine | 13:59 |
sdague | so you could search on the test name | 14:00 |
sdague | and see what the results were | 14:00 |
*** mwagner_lap has joined #openstack-qa | 14:01 | |
mtreinish | anteaya: I just copied it from the cookiecutter repo | 14:04 |
mtreinish | I didn't really look at it too closely | 14:04 |
anteaya | mtreinish: oh okay | 14:04 |
mtreinish | anteaya: is there something I should change on the permissions? | 14:05 |
*** mkollaro has quit IRC | 14:05 | |
*** mkollaro1 has joined #openstack-qa | 14:05 | |
*** mkollaro1 is now known as mkollaro | 14:05 | |
anteaya | well usually only -ptl has permissions on refs/tags/* | 14:05 |
anteaya | rather than core | 14:05 |
mtreinish | heh, actually I just noticed I won't have +2 on it when it merges :) | 14:06 |
anteaya | since creating and pushing tags doesn't need to be a group responsibility | 14:06 |
mtreinish | anteaya: oh, I wonder who is in the cookiecutter-core group | 14:06 |
mtreinish | I can't imagine that it's very big | 14:06 |
anteaya | good question | 14:06 |
anteaya | monty made it up | 14:06 |
anteaya | probably just him | 14:06 |
* anteaya looks | 14:06 | |
sdague | yeh, I'd expect mordred | 14:06 |
mtreinish | https://review.openstack.org/#/admin/groups/212,members | 14:07 |
openstackgerrit | afazekas proposed a change to openstack-infra/elastic-recheck: Add query for a heat db error https://review.openstack.org/85039 | 14:07 |
mtreinish | looks like mordred and oslo-core | 14:07 |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds more testcases to test_telemetry_alarming_api https://review.openstack.org/82491 | 14:11 |
tosky | mtreinish: about the meeting email on openstack-dev, shouldn't the time be 24:00 CEST instead of 23:00 CET now? (and maybe the same for other TZ after Daylight change) | 14:19 |
*** rushiagr has joined #openstack-qa | 14:21 | |
*** zhikunliu has joined #openstack-qa | 14:23 | |
mtreinish | tosky: probably, I forgot about the switch on Sunday there. I'll update for next weeks | 14:23 |
tosky | mtreinish: thanks! | 14:23 |
*** markmcclain has joined #openstack-qa | 14:24 | |
*** jgrimm has joined #openstack-qa | 14:24 | |
openstackgerrit | A change was merged to openstack/tempest: Verify "get version" API response attributes https://review.openstack.org/80936 | 14:25 |
*** rushiagr has left #openstack-qa | 14:25 | |
*** bill_az has joined #openstack-qa | 14:25 | |
openstackgerrit | Paul Michali proposed a change to openstack-dev/devstack: Devstack support for Neutron VPNaaS https://review.openstack.org/84842 | 14:26 |
*** chandan_kumar has quit IRC | 14:27 | |
*** chandan_kumar has joined #openstack-qa | 14:27 | |
*** chandan_kumar has quit IRC | 14:30 | |
*** afazekas has quit IRC | 14:30 | |
*** TravT has joined #openstack-qa | 14:35 | |
*** shwetaap has quit IRC | 14:37 | |
*** weshay has quit IRC | 14:38 | |
*** mkollaro1 has joined #openstack-qa | 14:39 | |
*** mkollaro has quit IRC | 14:39 | |
*** GMann has quit IRC | 14:43 | |
*** nmagnezi has joined #openstack-qa | 14:44 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: download fedora x86_64 image if heat is enabled https://review.openstack.org/85057 | 14:45 |
*** shwetaap has joined #openstack-qa | 14:45 | |
openstackgerrit | A change was merged to openstack/tempest: Waiting for ACTIVE state in rescue tests https://review.openstack.org/84975 | 14:49 |
openstackgerrit | nayna-patel proposed a change to openstack/tempest: Adds more testcases to test_telemetry_alarming_api https://review.openstack.org/82491 | 14:53 |
mtreinish | enikanorov_: on https://review.openstack.org/#/c/82480 shouldn't the commit message say 'Closes-Bug: #1294603' | 14:59 |
mtreinish | because you have the bug open against tempest too | 14:59 |
*** mmaglana has joined #openstack-qa | 14:59 | |
yfried__ | sdague: do you have an example to query a specific test through all of its executions? | 15:02 |
enikanorov_ | mtreinish: ok | 15:03 |
*** ashishsnigam has joined #openstack-qa | 15:03 | |
openstackgerrit | enikanorov proposed a change to openstack/tempest: Unskip load balancer basic scenario test https://review.openstack.org/82480 | 15:04 |
*** mmaglana has quit IRC | 15:04 | |
*** weshay has joined #openstack-qa | 15:08 | |
*** atiwari has joined #openstack-qa | 15:11 | |
openstackgerrit | Kaitlin Farr proposed a change to openstack/tempest: Adds scenario tests for volume encryption https://review.openstack.org/58018 | 15:11 |
*** zhikunliu has quit IRC | 15:15 | |
*** tkammer has quit IRC | 15:18 | |
*** ashishsnigam has quit IRC | 15:21 | |
*** nmagnezi has quit IRC | 15:21 | |
openstackgerrit | Pablo Andres Fuente proposed a change to openstack/tempest: Add test for OS::Neutron::* Heat resource https://review.openstack.org/81909 | 15:26 |
*** yamahata has joined #openstack-qa | 15:28 | |
*** shwetaap has quit IRC | 15:29 | |
*** ams0 has quit IRC | 15:30 | |
andreaf | mtreinish: ping | 15:32 |
mtreinish | andreaf: hey | 15:32 |
andreaf | mtreinish: I won't make it for the metting tonight, so I just wanted to provide a quick update on my bp. I created my bps in the qa-specs repo. I added a new one for cross-domain testing as well | 15:33 |
andreaf | mtreinish: https://blueprints.launchpad.net/tempest/+spec/cross-domain-testing | 15:34 |
andreaf | mtreinish: and I'm working on addressing the reviews I got so far on the Credentials changes. | 15:34 |
andreaf | s/metting/meeting | 15:34 |
mtreinish | andreaf: ok cool | 15:35 |
andreaf | mtreinish: have a good meeting! | 15:35 |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: increase heat guests https://review.openstack.org/85078 | 15:35 |
openstackgerrit | A change was merged to openstack/tempest: Validate get_instance_usage_audit_log Nova V2 API https://review.openstack.org/83303 | 15:36 |
mtreinish | andreaf: yeah I guess the 2200UTC time is kinda late for you isn't it | 15:36 |
sdague | andreaf: which TZ are you actually in? | 15:36 |
andreaf | mtreinish: yes it's not ideal - I'm in GMT (UTC+1 atm) | 15:36 |
andreaf | mtreinish, sdague: I try to make it when I can for the meeting at 10pm, but today I have guests at home | 15:37 |
sdague | andreaf: yep, no worries | 15:37 |
sdague | for some reason I thought you were in california | 15:37 |
andreaf | sdague: nope, UK - but I'm not British as you can guess from my name :D | 15:38 |
*** andreaf has quit IRC | 15:41 | |
sdague | heh | 15:41 |
*** sthaha has quit IRC | 15:43 | |
*** shwetaap has joined #openstack-qa | 15:46 | |
*** jgriffith has quit IRC | 15:47 | |
*** yfried__ has quit IRC | 15:48 | |
*** yfried__ has joined #openstack-qa | 15:48 | |
openstackgerrit | A change was merged to openstack/tempest: Add sahara to list of clients for T102 https://review.openstack.org/82519 | 15:49 |
*** mlavalle has joined #openstack-qa | 15:50 | |
*** samuelbercovici has quit IRC | 15:51 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: download fedora x86_64 image if heat is enabled https://review.openstack.org/85057 | 15:52 |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: increase heat guests https://review.openstack.org/85078 | 15:52 |
*** jlibosva has quit IRC | 15:52 | |
*** yfried__ has quit IRC | 15:54 | |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Move exceptions back into one place https://review.openstack.org/85085 | 15:56 |
*** jgriffith has joined #openstack-qa | 15:58 | |
*** jlibosva has joined #openstack-qa | 16:01 | |
*** evgenyf has quit IRC | 16:01 | |
*** marun_afk is now known as marun | 16:01 | |
*** andreaf has joined #openstack-qa | 16:03 | |
*** mdenny has quit IRC | 16:05 | |
enikanorov_ | > download fedora x86_64 image if heat is enabled | 16:05 |
enikanorov_ | wow, i wonder how large is that | 16:05 |
*** mdenny has joined #openstack-qa | 16:05 | |
*** yrabl has quit IRC | 16:07 | |
*** qba73 has quit IRC | 16:08 | |
*** krtaylor has quit IRC | 16:13 | |
openstackrecheck | openstack/python-heatclient change: https://review.openstack.org/77938 failed gate-tempest-dsvm-full in the gate queue with an unrecognized error | 16:16 |
openstackgerrit | Kaitlin Farr proposed a change to openstack/tempest: Adds scenario tests for volume encryption https://review.openstack.org/58018 | 16:16 |
enikanorov_ | mtreinish: sdague: https://review.openstack.org/#/c/82480/ changed per all your requests | 16:25 |
*** fbo is now known as fbo_away | 16:26 | |
*** yassine has quit IRC | 16:26 | |
*** locke105 has joined #openstack-qa | 16:27 | |
*** jpich has quit IRC | 16:27 | |
mriedem | clarkb: question/thought here on pinning this to a specific couple build_names, wondering if that makes the query faster https://review.openstack.org/#/c/85039/ | 16:27 |
clarkb | mriedem: yup that should make it quicker | 16:29 |
*** mrodden has quit IRC | 16:29 | |
clarkb | mriedem: reducing search space with filters is ++ good | 16:29 |
mriedem | ok, -1'ed | 16:29 |
mriedem | thanks | 16:29 |
mriedem | is it mmm mmm good? | 16:29 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Official client based auth https://review.openstack.org/82911 | 16:47 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Use auth data to fill credentials https://review.openstack.org/82111 | 16:47 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Get credential IDs from Credentials class https://review.openstack.org/80246 | 16:47 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Enforces the use of Credentials (part1) https://review.openstack.org/77594 | 16:47 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Enforces the use of Credentials (part2) https://review.openstack.org/82112 | 16:47 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Access credential fields as attributes https://review.openstack.org/82113 | 16:47 |
openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Defines a Credentials class https://review.openstack.org/74387 | 16:47 |
openstackgerrit | Aleksandr Gordeev proposed a change to openstack-dev/devstack: Configure ssh server properly for ironic https://review.openstack.org/81807 | 16:47 |
*** krtaylor has joined #openstack-qa | 16:50 | |
*** Abhijeet has quit IRC | 16:53 | |
*** andreaf has quit IRC | 16:59 | |
openstackrecheck | openstack/swift change: https://review.openstack.org/84456 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1295876 | 17:14 |
*** harlowja_away is now known as harlowja | 17:14 | |
*** jordanP has quit IRC | 17:19 | |
*** luqas has quit IRC | 17:21 | |
*** luqas has joined #openstack-qa | 17:21 | |
*** safchain has quit IRC | 17:25 | |
openstackgerrit | Luigi Toscano proposed a change to openstack-dev/devstack: Enable colors for Sahara log https://review.openstack.org/85115 | 17:26 |
*** mrodden has joined #openstack-qa | 17:27 | |
*** rockyg has joined #openstack-qa | 17:29 | |
*** morganfainberg_Z is now known as morganfainberg | 17:34 | |
*** shwetaap has quit IRC | 17:36 | |
*** tosky has quit IRC | 17:36 | |
*** ashishsnigam has joined #openstack-qa | 17:36 | |
*** otherwiseguy has quit IRC | 17:42 | |
*** yfried__ has joined #openstack-qa | 17:42 | |
jlibosva | dtroyer: hello, did you get my irc message earlier today? :) | 17:44 |
*** jlibosva has quit IRC | 17:45 | |
*** jlibosva has joined #openstack-qa | 17:46 | |
dtroyer | jlibosva: sorry, no. and I haven't gotten grenade to run locally for me, I don't think it has to do with your review, though. | 17:47 |
dtroyer | I do have one question in upgrade-neutron | 17:47 |
jlibosva | dtroyer: do you need any assistance to run it locally? | 17:47 |
jlibosva | dtroyer: I was using devstack-gate repo while running it on my env | 17:48 |
openstackgerrit | Ryota MIBU proposed a change to openstack-dev/devstack: Use sudo privilege to touch config files https://review.openstack.org/85120 | 17:48 |
dtroyer | You are stopping the running neutron processes twice, once at line 54 directly via screen and again at line 93 via stop_neutron() | 17:48 |
dtroyer | Is the first one needed? I'd prefer to just use the method in stop_neutron() if possible | 17:48 |
jlibosva | dtroyer: I was inspired by glance for example. Let me check | 17:49 |
ashishsnigam | can any one help me setting tempest for trove? | 17:50 |
openstackgerrit | Bartosz Górski proposed a change to openstack-dev/devstack: Adds support for multi-region https://review.openstack.org/79310 | 17:51 |
jlibosva | dtroyer: stop_neutron doesn't stop the neutron-server and third-party | 17:51 |
jlibosva | oh, there is explicit call for third-party | 17:52 |
jlibosva | in upgrade-neutron | 17:52 |
dtroyer | right. the main reason I care is we have found the method on line 55 to not always be reliable. but if there killed later anyway, it can be removed. | 17:53 |
jlibosva | dtroyer: ok, I'll try to remove it but first I need to find out how to stop neutron-server | 17:54 |
dtroyer | ok. that may require a devstack change, in which case we can probably go ahead with this and follow up later | 17:56 |
jlibosva | that's weird, I believe unstack.sh stops neutron, doesn't it? | 17:56 |
jlibosva | probably not :) | 17:57 |
jlibosva | also l3 agent seems not being stopped | 17:58 |
dtroyer | it should | 17:58 |
*** evgenyf has joined #openstack-qa | 17:59 | |
jlibosva | dtroyer: I'll add a comment to L55 that stop_neutron doesn't stop some services and will look into devstack tomorrow | 17:59 |
dtroyer | how about this: let's merge the grenade change and then you can follow up and make sure we're stopping neutron properly in devstack, then update grenade to do the right thing? | 17:59 |
dkranz | sdague: So here is an example. The aggregate fails are caused by the fallout from https://review.openstack.org/#/c/43061/ | 17:59 |
sdague | mtreinish: is there a reason we've got heat running serial? | 17:59 |
dkranz | sdague: Do you think that api change should have been allowed? | 18:00 |
jlibosva | dtroyer: agreed, should I add the comment or are you ok with this patch? | 18:00 |
dtroyer | ok, so if you're going to rev 58695, let me add a couple of minor comments to it before you re-submit | 18:00 |
jlibosva | dtroyer: ok | 18:00 |
dkranz | sdague: tempest was changed so that the json client does not always pass the az but the xml does pass it to the server even if the test is calling without that argument specified. | 18:00 |
sdague | dkranz: I'd have to dig a little deeper | 18:00 |
dkranz | sdague: No need to now | 18:00 |
sdague | dkranz: you want to start building an etherpad with your findings? | 18:01 |
sdague | so we can keep a list | 18:01 |
dkranz | sdague: Yes, I put it on the agenda for later. | 18:01 |
dkranz | sdague: But I can start it now | 18:01 |
dkranz | sdague: I have the feeling that most of the issues are going to be like this but we shall see. | 18:02 |
sdague | sure | 18:02 |
dkranz | sdague: We will have to become much clearer about what is allowed and what is not. The fallout from nova v2/3 was taking a pretty strong position on compatibility. | 18:02 |
sdague | yep, I actually think a master tempest, that requires changes in both stable and master on changes like this will encourage that though | 18:03 |
*** rwsu has quit IRC | 18:03 | |
mtreinish | sdague: ask dkranz he set that job up. I said before we made it voting it should be parallel though. | 18:03 |
dkranz | mtreinish: Which job? | 18:04 |
mtreinish | the heat slow | 18:04 |
dtroyer | jlibosva: ok, done. thanks | 18:05 |
jlibosva | dtroyer: thanks for comment | 18:05 |
jlibosva | s | 18:05 |
sdague | mtreinish: want to now? | 18:05 |
mtreinish | sdague: sure I'll push out a change | 18:05 |
sdague | we should recheck it a few times | 18:05 |
sdague | though there honestly isn't a lot in it at the moment | 18:05 |
sdague | so I doubt we'd see collisions | 18:05 |
dkranz | mtreinish: If it is serial that was just to help avoid flakies while getting it all working. | 18:05 |
dkranz | mtreinish: There is no reason for it to be serial once enabled to vote. | 18:06 |
*** evgenyf has quit IRC | 18:06 | |
*** shwetaap has joined #openstack-qa | 18:06 | |
ylobankov | sdague: dkranz: mtreinish: dtroyer: Hello, guys! could you please take a look at https://review.openstack.org/#/c/84487/3 when you have free time. Thanks. | 18:07 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Make heat-slow job run in parallel https://review.openstack.org/85129 | 18:07 |
mtreinish | sdague: ^^^ | 18:07 |
sdague | mtreinish: thanks | 18:08 |
openstackgerrit | Jakub Libosvar proposed a change to openstack-dev/grenade: Support for upgrading neutron https://review.openstack.org/58695 | 18:09 |
openstackrecheck | Console logs not available after 13:28s for check-tempest-dsvm-neutron-full 82911,3,a29888f | 18:09 |
openstackgerrit | Bartosz Górski proposed a change to openstack-dev/devstack: Adds support for multi-region https://review.openstack.org/79310 | 18:10 |
openstackgerrit | Adam Gandelman proposed a change to openstack-dev/devstack: Set flag in Tempest enabling Ironic scenario tests https://review.openstack.org/85132 | 18:15 |
openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Move exceptions back into one place https://review.openstack.org/85085 | 18:18 |
*** rockyg has quit IRC | 18:21 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/72252 failed because of: gate-neutron-python26: https://bugs.launchpad.net/bugs/1292506 | 18:23 |
*** mwagner_lap has quit IRC | 18:27 | |
*** shwetaap has quit IRC | 18:34 | |
*** shwetaap has joined #openstack-qa | 18:39 | |
*** pafuent has joined #openstack-qa | 18:47 | |
pafuent | Hi. Could someone review this https://review.openstack.org/#/c/81909/? | 18:48 |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: add an image_list tool https://review.openstack.org/85142 | 18:49 |
sdague | dtroyer: what do you think of that? | 18:49 |
*** marun is now known as marun_afk | 18:50 | |
*** mmaglana has joined #openstack-qa | 18:54 | |
*** shwetaap has quit IRC | 18:54 | |
*** otherwiseguy has joined #openstack-qa | 19:02 | |
jlibosva | sdague: dtroyer Hi, I pushed new patchset to grenade reacting on comments, could you please review? | 19:07 |
*** mwagner_lap has joined #openstack-qa | 19:08 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: allow for generic branch overrides https://review.openstack.org/84829 | 19:11 |
*** rockyg has joined #openstack-qa | 19:13 | |
*** shwetaap has joined #openstack-qa | 19:17 | |
*** jlibosva has quit IRC | 19:23 | |
*** malini is now known as malini_afk | 19:26 | |
*** fbo_away is now known as fbo | 19:29 | |
*** jlibosva has joined #openstack-qa | 19:31 | |
openstackgerrit | Sean Dague proposed a change to openstack-infra/devstack-gate: allow for generic branch overrides https://review.openstack.org/84829 | 19:39 |
*** ashishsnigam has quit IRC | 19:42 | |
openstackgerrit | David Kranz proposed a change to openstack/tempest: Don't output auth tokens with trace output https://review.openstack.org/85155 | 19:42 |
*** jlibosva has quit IRC | 19:42 | |
dtroyer | sdague: looks useful | 19:49 |
sdague | yeh, the complexity of image url creation in devstack has gotten to the point that it's cat and mouse in nodepool to keep up | 19:52 |
*** evgenyf has joined #openstack-qa | 19:53 | |
*** otherwiseguy has quit IRC | 19:56 | |
*** rwsu has joined #openstack-qa | 19:56 | |
*** mkollaro1 has quit IRC | 19:56 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: download fedora x86_64 image if heat is enabled https://review.openstack.org/85057 | 19:58 |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: increase heat guests https://review.openstack.org/85078 | 19:58 |
*** mkollaro has joined #openstack-qa | 20:01 | |
*** TravT|2 has joined #openstack-qa | 20:02 | |
*** TravT has quit IRC | 20:02 | |
*** harlowja has quit IRC | 20:02 | |
*** harlowja has joined #openstack-qa | 20:03 | |
*** HenryG has quit IRC | 20:06 | |
*** evgenyf has quit IRC | 20:07 | |
*** rwsu has quit IRC | 20:07 | |
*** rwsu has joined #openstack-qa | 20:09 | |
*** rwsu has quit IRC | 20:11 | |
*** rwsu has joined #openstack-qa | 20:11 | |
*** malini_afk is now known as malini | 20:15 | |
*** jhenner has quit IRC | 20:21 | |
*** rwsu has quit IRC | 20:27 | |
*** rwsu has joined #openstack-qa | 20:31 | |
lyxus | Hello Folks, I want to write some test unit for neutron. Is there a doc on how to write a decent test ? I saw only references to tempest | 20:31 |
sdague | lyxus: you should check the #openstack-neutron channel | 20:32 |
sdague | unit tests are project specific | 20:32 |
lyxus | sdague, ok thanks | 20:33 |
sdague | tempest is cross project integration and api testing | 20:33 |
pafuent | sdague: Sorry to bother you with this, but could you take a look at https://review.openstack.org/#/c/81909/? | 20:34 |
pafuent | pafuent: I rebase it to use the extracted templates | 20:34 |
pafuent | sdague: I rebase it to use the extracted templates | 20:35 |
sdague | pafuent: cool, I'll look in a bit. | 20:36 |
pafuent | sdague: Thanks | 20:36 |
*** shwetaap has quit IRC | 20:45 | |
*** marun_afk is now known as marun | 20:45 | |
openstackgerrit | meera-belur proposed a change to openstack/tempest: Keystone - Add test to filter endpoints results based on service id https://review.openstack.org/85174 | 20:53 |
*** mmaglana has quit IRC | 20:57 | |
*** shwetaap has joined #openstack-qa | 20:57 | |
*** TravT|2 has quit IRC | 20:59 | |
openstackrecheck | openstack/swift change: https://review.openstack.org/84332 failed gate-devstack-dsvm-cells in the gate queue with an unrecognized error | 21:00 |
*** mmaglana has joined #openstack-qa | 21:01 | |
*** pafuent has left #openstack-qa | 21:03 | |
*** rfolco has quit IRC | 21:03 | |
*** aysyd has quit IRC | 21:04 | |
*** dkranz has quit IRC | 21:10 | |
*** rockyg has quit IRC | 21:11 | |
enikanorov_ | cores, please unskip or LB test: https://review.openstack.org/#/c/82480/ | 21:17 |
enikanorov_ | thank you | 21:17 |
enikanorov_ | s/or/our | 21:17 |
*** weshay has quit IRC | 21:19 | |
openstackgerrit | meera-belur proposed a change to openstack/tempest: Add keystone test to filter endpoints results based on serviceid https://review.openstack.org/85179 | 21:21 |
*** rwsu has quit IRC | 21:22 | |
*** yassine has joined #openstack-qa | 21:25 | |
*** yassine has quit IRC | 21:27 | |
*** yassine has joined #openstack-qa | 21:27 | |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: download fedora x86_64 image if heat is enabled https://review.openstack.org/85057 | 21:28 |
openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: increase heat guests https://review.openstack.org/85078 | 21:28 |
*** Aimon has joined #openstack-qa | 21:30 | |
*** yassine has quit IRC | 21:30 | |
*** malini is now known as malini_afk | 21:30 | |
*** yassine has joined #openstack-qa | 21:31 | |
*** yassine has quit IRC | 21:31 | |
*** weshay has joined #openstack-qa | 21:37 | |
*** HenryG has joined #openstack-qa | 21:39 | |
*** zhikunliu has joined #openstack-qa | 21:46 | |
openstackgerrit | A change was merged to openstack/qa-specs: Mirror tox and sphinx fixes from nova-specs repo https://review.openstack.org/84502 | 21:48 |
*** fbo is now known as fbo_away | 21:48 | |
openstackrecheck | openstack/neutron change: https://review.openstack.org/82837 failed because of: gate-tempest-dsvm-neutron-heat-slow: https://bugs.launchpad.net/bugs/1297560 | 21:49 |
*** masayukig has joined #openstack-qa | 21:53 | |
mtreinish | sdague: umm, 84502 will cause issues building the docs until we actually have specs in both dirs | 21:54 |
mtreinish | because no rst files in the glob path is a warning | 21:54 |
mtreinish | which that patch makes build_sphinx fail on | 21:54 |
sdague | mtreinish: shouldn't it fail itself then? | 21:54 |
mtreinish | sdague: we don't gate on anything yet | 21:55 |
sdague | oh, ok, so what do we have to land to be ok? | 21:55 |
mtreinish | push a spec through | 21:55 |
mtreinish | I'll push out a patch that drops the implemented dir from the index | 21:56 |
mtreinish | until we get to that point | 21:56 |
mtreinish | then we should be good | 21:56 |
sdague | well, we'll push through one today | 21:56 |
sdague | I think this is ready - https://review.openstack.org/#/c/83264/ | 21:56 |
sdague | I just went through all the qa-specs and provided feedback | 21:56 |
*** mmaglana has quit IRC | 21:57 | |
*** Aimon has quit IRC | 21:57 | |
mtreinish | sdague: yeah I was just looking at them | 21:58 |
*** dkranz has joined #openstack-qa | 22:00 | |
*** oomichi has joined #openstack-qa | 22:02 | |
*** marun is now known as marun_afk | 22:03 | |
*** oomichi is now known as ken1ohmichi | 22:04 | |
openstackgerrit | A change was merged to openstack/qa-specs: Add spec for bp:add-service-tags https://review.openstack.org/83264 | 22:10 |
*** HenryG_ has joined #openstack-qa | 22:14 | |
*** HenryG has quit IRC | 22:15 | |
*** HenryG_ has quit IRC | 22:15 | |
*** HenryG has joined #openstack-qa | 22:16 | |
*** krtaylor has quit IRC | 22:22 | |
*** rwsu has joined #openstack-qa | 22:26 | |
*** weshay has quit IRC | 22:26 | |
*** mkollaro has quit IRC | 22:30 | |
adalbas | can anyone tell me if it is ok to have negative numbers for vcpu in AUDIT messages, or this could represent an issue? | 22:33 |
adalbas | as in 2014-03-28 14:50:15.645 AUDIT nova.compute.resource_tracker [-] Free VCPUS: -2 | 22:33 |
openstackrecheck | openstack/neutron change: https://review.openstack.org/85051 failed because of: gate-tempest-dsvm-neutron: https://bugs.launchpad.net/bugs/1295876 | 22:39 |
*** yamahata has quit IRC | 22:44 | |
*** jaypipes has joined #openstack-qa | 22:45 | |
*** ChanServ sets mode: +o jaypipes | 22:45 | |
*** marun_afk is now known as marun | 22:52 | |
*** atiwari has quit IRC | 22:54 | |
cyeoh | sdague: hi | 23:00 |
mtreinish | sdague: see the Aftermath section: http://en.wikipedia.org/wiki/United_States_presidential_election,_1828 | 23:01 |
mtreinish | that's what I was referring to | 23:01 |
sdague | cyeoh: yeh | 23:01 |
sdague | :) | 23:01 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/tempest: Add V2 Nova API os-agents tests https://review.openstack.org/78111 | 23:01 |
cyeoh | sdague: hey I was just wanting to ask you about how we might do testing of the v2.1 on V3 API if that gets the green light (trying to map out what will be needed) | 23:01 |
sdague | dtroyer: you still kicking around? I could use - https://review.openstack.org/#/c/85057/ and https://review.openstack.org/#/c/85142/ to land tonight if possible | 23:02 |
sdague | cyeoh: sure | 23:02 |
cyeoh | sdague: essentially we just want to run the V2 tests against the V2.1 (non voting until it is completed) | 23:02 |
sdague | do you have a base proposal? | 23:02 |
cyeoh | sdague: well I"m not sure I understand what is possible | 23:02 |
cyeoh | v2.1 gets exported on /v2.1 rather than /v2 | 23:02 |
sdague | ok, so you'd need a devstack set of patches to optionally enable it | 23:03 |
cyeoh | ken1ohmichi has a patch which just puts the v2.1 on /v2 instead, but obviously we can't merge that | 23:03 |
mtreinish | cyeoh: we can just have a separate job that will register the keystone compute endpoint as v2.1 | 23:03 |
mtreinish | tempest uses the catalog to get the endpoint | 23:03 |
sdague | yeh, I would think so | 23:03 |
cyeoh | mtreinish: ok I just wanted to make sure that wasn't considered too ugly :-) | 23:03 |
mtreinish | you do it for v3 :) | 23:03 |
cyeoh | but we can definitely go that route | 23:03 |
sdague | you could probably do the whole thing with a devstack flag honestly | 23:03 |
*** yamahata has joined #openstack-qa | 23:04 | |
cyeoh | so the issue with a devstack flag is that the change required is to the api-paste file | 23:04 |
sdague | so, we modify paste all the time in devstack | 23:04 |
cyeoh | oh ok then | 23:04 |
cyeoh | and I guess then lastly I just wanted to check that there were no concerns about the overhead of having to run another non voting tempest job? | 23:05 |
sdague | I'd start with it in experimental | 23:05 |
sdague | so you can run it on demand | 23:05 |
*** matsuhashi has joined #openstack-qa | 23:05 | |
sdague | and when it's regularly passing, bring it up to nv | 23:05 |
sdague | especially if it's going to be on nova changes | 23:06 |
sdague | as there are a *ton* of them | 23:06 |
cyeoh | oh that'd be great - can you point me to some docs to how do that? (setting up experimental jobs) | 23:06 |
sdague | it's just another queue | 23:06 |
sdague | https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/zuul/layout.yaml#L898 | 23:07 |
*** mlavalle has quit IRC | 23:07 | |
*** shwetaap has quit IRC | 23:07 | |
cyeoh | sdague: thanks! | 23:08 |
sdague | when you run check experimental, those jobs run on demand | 23:08 |
mtreinish | masayukig: so I was looking at your unit test patch | 23:08 |
mtreinish | which led me to push: https://review.openstack.org/#/c/85085/ | 23:08 |
sdague | ok, time for me to call it a day, have a good one folks | 23:08 |
mtreinish | there's nothing wrong with the unit tests | 23:08 |
cyeoh | sdague: nice, thanks for your help! | 23:08 |
mtreinish | masayukig: but how did you want to handle the merge conflict? | 23:08 |
masayukig | mtreinish: np, I'll rebase/depend on your patch :) | 23:10 |
mtreinish | ok, I would have been fine with you wanting to do it the other way too :) | 23:11 |
mtreinish | masayukig: also I remember a unit test patch on the commands stuff from someone at nec | 23:11 |
mtreinish | should I pick that up or will it be restored? | 23:12 |
masayukig | mtreinish: https://review.openstack.org/#/c/82700/ ? | 23:12 |
mtreinish | masayukig: https://review.openstack.org/#/c/79572/ | 23:13 |
masayukig | mtreinish: sorry, thanks. | 23:13 |
masayukig | I'll talk to Takada-san later. | 23:14 |
mtreinish | thanks | 23:14 |
mtreinish | it's not a problem I was just wondering where it went | 23:14 |
masayukig | mtreinish: sure, np. | 23:15 |
*** jgrimm has quit IRC | 23:18 | |
openstackrecheck | openstack/keystone change: https://review.openstack.org/84050 failed because of: gate-tempest-dsvm-full: https://bugs.launchpad.net/bugs/1254872 | 23:20 |
*** kgriffs is now known as kgriffs|afk | 23:22 | |
*** sthaha has joined #openstack-qa | 23:29 | |
*** HenryG_ has joined #openstack-qa | 23:30 | |
*** markmcclain has quit IRC | 23:31 | |
*** HenryG has quit IRC | 23:32 | |
*** Sukhdev has joined #openstack-qa | 23:34 | |
*** HenryG has joined #openstack-qa | 23:38 | |
*** yamahata has quit IRC | 23:38 | |
openstackgerrit | Bartosz Górski proposed a change to openstack-dev/devstack: Adds support for multi-region https://review.openstack.org/79310 | 23:41 |
*** HenryG_ has quit IRC | 23:41 | |
*** HenryG has quit IRC | 23:42 | |
*** mriedem has quit IRC | 23:48 | |
*** zhikunliu has quit IRC | 23:48 | |
*** mrodden has quit IRC | 23:51 | |
*** rockyg has joined #openstack-qa | 23:52 | |
dtroyer | sdague: done | 23:55 |
*** mdenny has quit IRC | 23:59 | |
*** GMann has joined #openstack-qa | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!