| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Clean up local variable usage - cinder https://review.openstack.org/109722 | 00:12 |
|---|---|---|
| *** rbak has joined #openstack-qa | 00:13 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Clean up local variable usage - Remainder https://review.openstack.org/113956 | 00:14 |
| *** marun has quit IRC | 00:15 | |
| *** dims has joined #openstack-qa | 00:18 | |
| *** dmorita has joined #openstack-qa | 00:26 | |
| *** rbak has quit IRC | 00:26 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Clean up local variable usage - Account setup https://review.openstack.org/113953 | 00:32 |
| *** marun has joined #openstack-qa | 00:33 | |
| *** marun has quit IRC | 00:33 | |
| *** mwagner_lap has joined #openstack-qa | 00:35 | |
| *** marun has joined #openstack-qa | 00:37 | |
| *** marun has quit IRC | 00:42 | |
| *** xianghui has quit IRC | 00:54 | |
| *** xianghui has joined #openstack-qa | 00:58 | |
| *** topshare has joined #openstack-qa | 00:59 | |
| *** markmcclain has joined #openstack-qa | 00:59 | |
| *** topshare_ has joined #openstack-qa | 00:59 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: Fix "NotFound" error in _clear_stacks() https://review.openstack.org/113466 | 01:01 |
| *** topshare has quit IRC | 01:03 | |
| *** suneelb has quit IRC | 01:06 | |
| *** xianghui has quit IRC | 01:07 | |
| *** xianghui has joined #openstack-qa | 01:07 | |
| *** wuhg has joined #openstack-qa | 01:09 | |
| *** yjiang5 has quit IRC | 01:13 | |
| *** mtreinish has quit IRC | 01:19 | |
| *** krtaylor has quit IRC | 01:21 | |
| *** mtreinish has joined #openstack-qa | 01:22 | |
| *** mtreinish has quit IRC | 01:23 | |
| *** mtreinish has joined #openstack-qa | 01:23 | |
| *** arnaud has quit IRC | 01:25 | |
| *** dims has quit IRC | 01:26 | |
| *** timello_ has quit IRC | 01:30 | |
| *** krtaylor has joined #openstack-qa | 01:32 | |
| *** mtreinish has quit IRC | 01:38 | |
| *** aimon has joined #openstack-qa | 01:39 | |
| *** jaypipes has quit IRC | 01:41 | |
| *** nosnos has joined #openstack-qa | 01:42 | |
| *** jaypipes has joined #openstack-qa | 01:42 | |
| *** ChanServ sets mode: +o jaypipes | 01:42 | |
| *** arnaud has joined #openstack-qa | 02:05 | |
| *** xianghui has quit IRC | 02:07 | |
| *** xianghui has joined #openstack-qa | 02:07 | |
| *** rfolco has joined #openstack-qa | 02:20 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: enable volumes v2 snapshot tests by sharing codes https://review.openstack.org/106664 | 02:22 |
| *** markmcclain has quit IRC | 02:22 | |
| *** harlowja is now known as harlowja_away | 02:35 | |
| *** markmcclain has joined #openstack-qa | 02:40 | |
| *** jecarey has joined #openstack-qa | 02:40 | |
| *** marun has joined #openstack-qa | 02:59 | |
| *** yjiang5 has joined #openstack-qa | 03:00 | |
| *** marun has quit IRC | 03:02 | |
| *** StevenK has quit IRC | 03:08 | |
| *** cpallares has joined #openstack-qa | 03:08 | |
| *** cyeoh has quit IRC | 03:08 | |
| *** StevenK has joined #openstack-qa | 03:08 | |
| *** xianghui has quit IRC | 03:09 | |
| *** xianghui has joined #openstack-qa | 03:09 | |
| *** cyeoh has joined #openstack-qa | 03:12 | |
| *** dims has joined #openstack-qa | 03:14 | |
| *** cpallares has quit IRC | 03:16 | |
| *** xianghui has quit IRC | 03:16 | |
| *** xianghui has joined #openstack-qa | 03:17 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Clean up local variable usage - Zaqar https://review.openstack.org/115485 | 03:19 |
| *** markmcclain has quit IRC | 03:20 | |
| *** dims has quit IRC | 03:23 | |
| *** cpallares has joined #openstack-qa | 03:24 | |
| *** krtaylor has quit IRC | 03:28 | |
| *** cpallares has quit IRC | 03:30 | |
| *** cpallares has joined #openstack-qa | 03:30 | |
| *** cpallares has quit IRC | 03:31 | |
| *** cmyster has joined #openstack-qa | 03:34 | |
| *** cmyster has quit IRC | 03:34 | |
| *** cmyster has joined #openstack-qa | 03:34 | |
| *** krtaylor has joined #openstack-qa | 03:41 | |
| *** krtaylor has quit IRC | 03:46 | |
| *** chandankumar has joined #openstack-qa | 03:59 | |
| *** oomichi has joined #openstack-qa | 04:01 | |
| *** yjiang5 has quit IRC | 04:02 | |
| *** krtaylor has joined #openstack-qa | 04:04 | |
| *** vaidy has joined #openstack-qa | 04:14 | |
| openstackgerrit | Zhi Kun Liu proposed a change to openstack/tempest: enable volumes v2 snapshot tests by sharing codes https://review.openstack.org/106664 | 04:14 |
| *** xianghui has quit IRC | 04:17 | |
| *** xianghui has joined #openstack-qa | 04:17 | |
| *** Madan has joined #openstack-qa | 04:27 | |
| *** pcrews has quit IRC | 04:46 | |
| *** chandankumar has quit IRC | 04:52 | |
| *** chandankumar has joined #openstack-qa | 04:59 | |
| *** chandankumar has quit IRC | 05:02 | |
| *** vaidy has quit IRC | 05:05 | |
| *** dims has joined #openstack-qa | 05:07 | |
| *** topshare_ has quit IRC | 05:09 | |
| *** dims has quit IRC | 05:11 | |
| *** topshare has joined #openstack-qa | 05:17 | |
| *** nmagnezi has joined #openstack-qa | 05:17 | |
| *** nmagnezi has quit IRC | 05:23 | |
| openstackgerrit | Emily Hugenbruch proposed a change to openstack/tempest: Add glance tag and test to test_server_actions https://review.openstack.org/111825 | 05:28 |
| *** topshare has quit IRC | 05:31 | |
| *** topshare has joined #openstack-qa | 05:32 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-dev/devstack: Give PyMySQL a go. https://review.openstack.org/115495 | 05:34 |
| *** vaidy-nick has joined #openstack-qa | 05:37 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-dev/devstack: Latest tox should work for openstack now, use it https://review.openstack.org/115496 | 05:38 |
| *** chandankumar has joined #openstack-qa | 05:40 | |
| coolsvap | gmann, hi | 05:53 |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Adds the clients and tests for CINDER QoS V1 & V2 APIs https://review.openstack.org/90306 | 05:55 |
| *** SridharG has joined #openstack-qa | 06:02 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-infra/devstack-gate: Try PyMySQL as Python MySQLDB alternative. https://review.openstack.org/115501 | 06:03 |
| *** bkopilov has joined #openstack-qa | 06:04 | |
| *** SridharG has quit IRC | 06:06 | |
| *** afazekas has quit IRC | 06:11 | |
| *** saurabh_ has quit IRC | 06:12 | |
| *** afazekas_ has quit IRC | 06:16 | |
| *** sdmonov has joined #openstack-qa | 06:18 | |
| *** ildikov_ has quit IRC | 06:18 | |
| *** sdmonov has quit IRC | 06:23 | |
| *** nmagnezi has joined #openstack-qa | 06:28 | |
| *** flaper87|afk is now known as flaper87 | 06:35 | |
| *** arnaud has quit IRC | 06:37 | |
| *** SridharG has joined #openstack-qa | 06:42 | |
| *** topshare has quit IRC | 06:43 | |
| *** ildikov has joined #openstack-qa | 06:46 | |
| *** niedbalski has quit IRC | 06:46 | |
| *** ildikov has quit IRC | 06:49 | |
| *** pcrews has joined #openstack-qa | 06:54 | |
| *** afazekas_ has joined #openstack-qa | 06:57 | |
| *** topshare has joined #openstack-qa | 06:58 | |
| *** topshare has quit IRC | 06:59 | |
| *** topshare has joined #openstack-qa | 06:59 | |
| *** topshare has quit IRC | 07:02 | |
| *** topshare has joined #openstack-qa | 07:05 | |
| *** topshare has quit IRC | 07:06 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/grenade: Make function err run echo with root https://review.openstack.org/115517 | 07:12 |
| *** gszasz has joined #openstack-qa | 07:16 | |
| *** ildikov has joined #openstack-qa | 07:20 | |
| *** tkammer has joined #openstack-qa | 07:21 | |
| *** sdmonov has joined #openstack-qa | 07:26 | |
| *** markus_z has joined #openstack-qa | 07:30 | |
| Santosh | afazekas_: hi | 07:30 |
| afazekas_ | Santosh: hi | 07:32 |
| *** afazekas_ is now known as afazekas | 07:32 | |
| afazekas | I did not get answer on #openstack-neutron | 07:32 |
| *** jlibosva has joined #openstack-qa | 07:32 | |
| afazekas | Available network statuses: 'ACTIVE' 'BUILD' 'DOWN' 'ERROR' | 07:34 |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/devstack: Pull dstat logic into its own function so grenade can use it https://review.openstack.org/115526 | 07:36 |
| afazekas | the NET_STATUS_BUILD is never used in the neutron code | 07:36 |
| openstackgerrit | Ashish Kumar Gupta proposed a change to openstack/tempest: Move API response success check to Neutron client https://review.openstack.org/114186 | 07:36 |
| openstackgerrit | Ashish Kumar Gupta proposed a change to openstack/tempest: Port API Tests Enhancement https://review.openstack.org/115527 | 07:36 |
| *** jlanoux has joined #openstack-qa | 07:36 | |
| *** mpaolino has joined #openstack-qa | 07:37 | |
| *** sdmonov has quit IRC | 07:38 | |
| *** topshare has joined #openstack-qa | 07:43 | |
| *** tkammer has quit IRC | 07:44 | |
| *** jordanP has joined #openstack-qa | 07:45 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Adds the clients and tests for CINDER QoS V1 & V2 APIs https://review.openstack.org/90306 | 07:47 |
| *** jpich has joined #openstack-qa | 07:54 | |
| afazekas | Santosh: FYI: I am asking initial state again https://review.openstack.org/#/c/115534/ | 08:05 |
| *** oomichi has quit IRC | 08:07 | |
| *** pcrews has quit IRC | 08:08 | |
| *** ildikov_ has joined #openstack-qa | 08:09 | |
| *** qba73 has joined #openstack-qa | 08:11 | |
| afazekas | sdague: ping | 08:11 |
| *** ildikov has quit IRC | 08:12 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: Install sqlalchemy-migrate from master if requested https://review.openstack.org/114224 | 08:21 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack-dev/devstack: (DO NOT MERGE) test run using mysqlconnector https://review.openstack.org/114207 | 08:21 |
| *** lordd_ has joined #openstack-qa | 08:30 | |
| coolsvap | gmann, please have a look at https://review.openstack.org/#/c/90306/ let me know if you have any further comments | 08:31 |
| *** topshare_ has joined #openstack-qa | 08:31 | |
| *** Dafna has quit IRC | 08:33 | |
| *** Dafna has joined #openstack-qa | 08:34 | |
| *** topshare has quit IRC | 08:34 | |
| *** gszasz has quit IRC | 08:35 | |
| coolsvap | afazekas, please have a look at https://review.openstack.org/#/c/90306/ once, i have addresses some comments from gmann regarding the base classes | 08:36 |
| *** tkammer has joined #openstack-qa | 08:37 | |
| coolsvap | s/addresses/addressed | 08:37 |
| gmann | coolsvap: Hi | 08:44 |
| coolsvap | gmann, I think i will need to revert to the way i had implemented the base admin classes | 08:44 |
| gmann | coolsvap: please check inline comment. You need to implement the admin tests base class on same line as BaseVolumeTest | 08:45 |
| coolsvap | gmann, I did that | 08:45 |
| gmann | only BaseVolumeAdminTest is needed BaseVolumeV1AdminTest is not needed. | 08:45 |
| *** BigQ has joined #openstack-qa | 08:46 | |
| coolsvap | what about admin tests for v1 apis? | 08:46 |
| *** Longgeek has joined #openstack-qa | 08:46 | |
| *** topshare_ has quit IRC | 08:47 | |
| gmann | for that you can keep BaseVolumeV1AdminTest which should be derived from BaseVolumeAdminTest. | 08:47 |
| *** Longgeek has quit IRC | 08:48 | |
| *** Longgeek has joined #openstack-qa | 08:48 | |
| gmann | class BaseVolumeV1AdminTest(BaseVolumeAdminTest): _api_version = 1 | 08:48 |
| mpaolino | hi guys, I have a build failed with the Hyper-V CI (http://64.119.130.115/96140/6/results.html.gz) | 08:49 |
| mpaolino | can you help me to solve it? | 08:49 |
| gmann | it should not have any code. it is just for tests using BaseVolumeV1AdminTest as base class. once all admin tests class is derived from BaseVolumeAdminTest, we need to remove BaseVolumeV1AdminTest. | 08:49 |
| mpaolino | is that something known? how can I check it? | 08:49 |
| mpaolino | the original patch is https://review.openstack.org/#/c/96140/ | 08:50 |
| coolsvap | gmann, let me try that, I was initializing the v1 and v2 clients saperately | 08:51 |
| coolsvap | gmann, but again the issue is with xml clients will come again | 08:51 |
| gmann | that was old approach but we need to share the code for V1, V2 tests and clients. | 08:51 |
| *** mkollaro has joined #openstack-qa | 08:51 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Adds the clients and tests for CINDER QoS V1 & V2 APIs https://review.openstack.org/90306 | 08:52 |
| gmann | same way as BaseVolumeTest is implemented | 08:52 |
| openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Migrate test_snapshot_pattern to tempest client https://review.openstack.org/110426 | 08:56 |
| openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Migrate test_volume_boot_pattern to tempest client https://review.openstack.org/112535 | 08:56 |
| openstackgerrit | Joseph Lanoux proposed a change to openstack/tempest: Migrate test_minimun_basic to tempest client https://review.openstack.org/107562 | 08:56 |
| *** topshare has joined #openstack-qa | 08:56 | |
| BigQ | Hey guys. I am currently on some research for my studies and I am looking at QA here. As far as I understand the tempest project defines and implements test for gate testing. Whats the workflow to include a new test case to this suite (contributor, reviewers...). Can anyone provide me some information?)? | 08:57 |
| *** niedbalski has joined #openstack-qa | 08:58 | |
| *** lordd_ has quit IRC | 09:01 | |
| *** lordd_ has joined #openstack-qa | 09:02 | |
| gmann | BigQ: Thanks for having interest in QA. That actually depend on what type of tests you want to add. | 09:03 |
| BigQ | what are categories in testing? | 09:04 |
| gmann | BigQ: have a look into https://github.com/openstack/tempest/blob/master/README.rst . https://github.com/openstack/tempest/blob/master/tempest/README.rst tells all catagory of tests type we have in tempest | 09:05 |
| gmann | There are 2 README files (link above) | 09:06 |
| *** vaidy-nick has quit IRC | 09:06 | |
| BigQ | great, thanks gmann. I will have a look | 09:09 |
| gmann | BigQ: np | 09:10 |
| *** mtreinish has joined #openstack-qa | 09:10 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 09:10 |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add security groups support https://review.openstack.org/104984 | 09:18 |
| coolsvap | gmann, i think the admin can be implemented same way as base directly | 09:21 |
| coolsvap | gmann, currently the patch is failing since we do not have xml clients for all components | 09:22 |
| gmann | coolsvap: implementing admin on same line should work. | 09:23 |
| coolsvap | gmann, have you checked the latest patch? | 09:23 |
| gmann | thats how non admin base tests class is implemented | 09:23 |
| coolsvap | gmann, i have done it according to what you are saying | 09:23 |
| gmann | coolsvap: ok, i will check. | 09:24 |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: implement resource destruction https://review.openstack.org/104986 | 09:24 |
| coolsvap | gmann, but the patch will fail with tempest.api.volume.admin.test_volumes_actions.VolumesActionsTestXML and so many tests where they expect to have the v1 xml client | 09:24 |
| coolsvap | since v1 client is inheriting from BaseVolumeAdminTest, it inherits all the v2 clients | 09:25 |
| gmann | coolsvap: i did not get. I will have look into your patch and let you know about failure too | 09:26 |
| coolsvap | gmann, https://jenkins06.openstack.org/job/check-tempest-dsvm-full/15144/consoleText | 09:26 |
| Santosh | afazekas: thanks afazekas | 09:28 |
| coolsvap | gmann, I am reverting the patch to the version where its working, i will submit a new patch set with the changes you mentioned about the merging on v1 and v2 admin base classes | 09:34 |
| coolsvap | gmann, since it will be affecting many other files apart from qos spec tests we are trying to add with this change set | 09:35 |
| gmann | coolsvap: you can keep this patch. all new cinder tests should go with sharing the tests code - https://blueprints.launchpad.net/tempest/+spec/cinder-v2-api-tests | 09:35 |
| *** gszasz has joined #openstack-qa | 09:36 | |
| gmann | coolsvap: it should not affect other files. let me check and let you know. give me some time | 09:36 |
| *** zz_johnthetubagu is now known as johnthetubaguy | 09:37 | |
| coolsvap | gmann, okay please, because the changeset has already been iterating in different directions | 09:37 |
| gmann | coolsvap: you are using qos v2 client in v1 tests also | 09:40 |
| coolsvap | gmann, thats what we need different clients for v1 and v2, so we have different clients | 09:41 |
| coolsvap | so once initialize v2 client in baseadmintest, its inherited in baseadminv1 | 09:41 |
| coolsvap | and it throws the error | 09:42 |
| gmann | coolsvap: check how BaseVolumeTest is implemented. you need to initialize the clients based on if cls._api_version . if _api_version is 1 inilize qos_client with V1 client ele if it is 2 then inilialize client with V2 qos client | 09:42 |
| *** tosky has joined #openstack-qa | 09:43 | |
| gmann | coolsvap: do you have if else condition there? | 09:43 |
| gmann | main idea in tests are we need to share the code between V1 and V2 tests. | 09:44 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/tempest: Adds Ironic set/get boot device scenario test https://review.openstack.org/113276 | 09:44 |
| gmann | and thats why we are removing the separate base class. | 09:44 |
| coolsvap | gmann, I agree, I am just trying to split the client initialization in different classes instead of if else | 09:44 |
| gmann | thats what you are doing against https://blueprints.launchpad.net/tempest/+spec/cinder-v2-api-tests | 09:45 |
| gmann | if you have diff base tests class how you will share the tests code | 09:46 |
| coolsvap | gmann, adding the if eise just a min | 09:47 |
| *** lordd_ has quit IRC | 09:49 | |
| *** lordd_ has joined #openstack-qa | 09:51 | |
| *** lordd_ has quit IRC | 09:52 | |
| gmann | coolsvap: I just gave the sample code there. | 09:54 |
| gmann | that should work fine. | 09:54 |
| *** raies has quit IRC | 09:55 | |
| *** lordd_ has joined #openstack-qa | 09:55 | |
| *** tkammer has quit IRC | 09:57 | |
| *** gszasz has quit IRC | 09:57 | |
| *** lordd_ has quit IRC | 09:58 | |
| *** cnesa7 has joined #openstack-qa | 09:58 | |
| *** jhenner1 has quit IRC | 10:05 | |
| *** openstackgerrit has quit IRC | 10:10 | |
| *** jhenner has joined #openstack-qa | 10:17 | |
| BigQ | gmann: I got through the readmes. I am a bit confused now. For me all the test's goal is to kind of validating your own cloud setup that should be managed by OpenStack. Of course this sounds very reasonable! But in which way are these tests related to regular OpenStack changes? Who decides which tests are applied for check/gate regarding the gerrit workflow (https://wiki.openstack.org/wiki/Gerrit_Workflow) | 10:18 |
| *** aix has joined #openstack-qa | 10:19 | |
| *** asselin_ has quit IRC | 10:19 | |
| BigQ | gmann: Or are all tempest types used in the gerrit workflow? | 10:19 |
| *** tkammer has joined #openstack-qa | 10:22 | |
| *** gszasz has joined #openstack-qa | 10:23 | |
| *** tkammer has quit IRC | 10:27 | |
| *** yamahata has quit IRC | 10:28 | |
| coolsvap | gmann, thanks for the comment | 10:30 |
| coolsvap | gmann, i did the same thing, but the xml client issue will always be there, even if i do that, qos does not have a xml client | 10:30 |
| gmann | BigQ: There are gate jobs for that. http://status.openstack.org/zuul/ - you cna find different gate jobs having their own set of tests. | 10:36 |
| gmann | BigQ: those run against each changes in openstack projects and verify the changes | 10:37 |
| BigQ | gmann: and thes are not related to your tests? | 10:38 |
| gmann | coolsvap: have you released the patch? It will be better if you release the patch then, i will have look into the xml issue | 10:38 |
| gmann | BigQ: those jobs run on any kind of changes you made in code. | 10:38 |
| *** jaypipes has quit IRC | 10:39 | |
| *** jhenner1 has joined #openstack-qa | 10:39 | |
| *** jaypipes has joined #openstack-qa | 10:40 | |
| *** ChanServ sets mode: +o jaypipes | 10:40 | |
| *** jhenner has quit IRC | 10:40 | |
| *** tkammer has joined #openstack-qa | 10:42 | |
| *** topshare has quit IRC | 10:43 | |
| coolsvap | gmann, got the xml issue resolved | 10:44 |
| *** Puneet_ has quit IRC | 10:44 | |
| coolsvap | sending the patch | 10:45 |
| *** vaidy-nick has joined #openstack-qa | 10:46 | |
| gmann | coolsvap: great :) | 10:47 |
| *** gszasz has quit IRC | 10:49 | |
| *** gszasz has joined #openstack-qa | 10:54 | |
| *** nosnos has quit IRC | 10:57 | |
| *** gszasz has quit IRC | 10:58 | |
| *** vaidy-nick has quit IRC | 10:59 | |
| *** stoner_ has joined #openstack-qa | 11:00 | |
| gmann | coolsvap: please check my comments inline. xml issues needs to handle as mentioned in inline comments/ | 11:02 |
| coolsvap | gmann, thx! updating the patch | 11:04 |
| stoner_ | has anyone here figured out how to use pycharm or pydev to debug a Tempest test? I found some directions how to do it with pdb like: python -m testtools.run some_test, but debugging with pdb is painful | 11:04 |
| gmann | coolsvap: np | 11:06 |
| *** tkammer has quit IRC | 11:09 | |
| *** ildikov_ has quit IRC | 11:10 | |
| *** gszasz has joined #openstack-qa | 11:12 | |
| *** lordd_ has joined #openstack-qa | 11:36 | |
| *** cmyster has quit IRC | 11:37 | |
| *** cdent has joined #openstack-qa | 11:42 | |
| *** dims has joined #openstack-qa | 11:50 | |
| *** ylobankov has joined #openstack-qa | 11:50 | |
| *** tkammer has joined #openstack-qa | 11:53 | |
| *** dkranz has joined #openstack-qa | 12:00 | |
| *** qba73_ has joined #openstack-qa | 12:01 | |
| *** qba73 has quit IRC | 12:01 | |
| *** nmagnezi has quit IRC | 12:04 | |
| afazekas | andreaf_, dkranz, mkoderer: https://review.openstack.org/#/c/115195/ | 12:05 |
| *** cdent has quit IRC | 12:08 | |
| *** bauzas has joined #openstack-qa | 12:09 | |
| *** weshay has joined #openstack-qa | 12:11 | |
| dkranz | afazekas: ok | 12:16 |
| afazekas | dkranz: thx | 12:16 |
| *** lordd__ has joined #openstack-qa | 12:20 | |
| *** lordd_ has quit IRC | 12:21 | |
| mkoderer | dkranz: could you have a look to https://review.openstack.org/#/c/115258/ | 12:22 |
| mkoderer | and https://review.openstack.org/#/c/113899/ | 12:22 |
| *** adalbas has joined #openstack-qa | 12:22 | |
| dkranz | mkoderer: what is wrong with empty string? | 12:22 |
| dkranz | mkoderer: Not opposed to the change, just curious | 12:23 |
| *** gszasz has quit IRC | 12:23 | |
| mkoderer | dkranz: the thing is that is doesn't create valid request | 12:23 |
| *** gszasz has joined #openstack-qa | 12:23 | |
| mkoderer | if you put an empty string in a name field for instance | 12:23 |
| dkranz | mkoderer: then shouldn't the schema say the min length is 1? | 12:23 |
| *** malini has quit IRC | 12:25 | |
| mkoderer | dkranz: that's right.. but don't you think that for string fields it should be one character in the vaild case as default? | 12:25 |
| dkranz | mkoderer: The only point of the valid case is to *not* be the cause of the failure. | 12:25 |
| dkranz | mkoderer: My only concern about the change is that it will prevent us from finding bugs where the schema says any string is ok but empty string actually causes failure | 12:26 |
| *** dmorita has quit IRC | 12:26 | |
| dkranz | mkoderer: But maybe that should be covered by a generated positive case | 12:27 |
| dkranz | mkoderer: that checks the boundaries | 12:27 |
| mkoderer | dkranz: but that's the case currently.. there are failing because of the empty string | 12:28 |
| mkoderer | dkranz: yep we should have a positve check to cross check if the schema is valid | 12:28 |
| mkoderer | I will work on the too | 12:28 |
| dkranz | mkoderer: ok, so we can make the change to move along but we should not lose this issues | 12:28 |
| dkranz | mkoderer: Great | 12:28 |
| dkranz | mkoderer: just a sec I'll +A | 12:28 |
| mkoderer | dkranz: I will raise a bug that we don't forget it | 12:29 |
| *** ArxCruz has joined #openstack-qa | 12:30 | |
| dkranz | mkoderer: ok, thanks. You could add the bug link to the patch (now +A) | 12:31 |
| mkoderer | dkranz: btw I am working on the "reduce complexity" at import level patch | 12:31 |
| mkoderer | dkranz: first WiP https://review.openstack.org/#/c/115273/ | 12:32 |
| *** aysyd has joined #openstack-qa | 12:32 | |
| *** rbak has joined #openstack-qa | 12:35 | |
| *** bkopilov has quit IRC | 12:36 | |
| *** rbak has quit IRC | 12:38 | |
| *** cdent has joined #openstack-qa | 12:39 | |
| *** cdent has quit IRC | 12:43 | |
| *** cdent_ has joined #openstack-qa | 12:43 | |
| *** Puneet_ has joined #openstack-qa | 12:47 | |
| *** lucasagomes has joined #openstack-qa | 12:49 | |
| lucasagomes | hi all, can I get some eyes at https://review.openstack.org/#/c/99677 ? this is been there for a while and keep rebasing it | 12:49 |
| lucasagomes | ipxe supported already landed in Ironic and that change is part of the blueprint for it, so I need that to merge in order to mark the bp as implemented | 12:50 |
| lucasagomes | I keep* | 12:50 |
| lucasagomes | thanks | 12:50 |
| *** andreaf has joined #openstack-qa | 12:51 | |
| *** andreaf has quit IRC | 12:51 | |
| *** andreaf has joined #openstack-qa | 12:51 | |
| *** andreaf has quit IRC | 12:51 | |
| *** andreaf has joined #openstack-qa | 12:52 | |
| *** andreaf has quit IRC | 12:52 | |
| *** andreaf has joined #openstack-qa | 12:52 | |
| *** andreaf has quit IRC | 12:53 | |
| *** andreaf_ has quit IRC | 12:53 | |
| *** andreaf has joined #openstack-qa | 12:53 | |
| *** andreaf has quit IRC | 12:53 | |
| *** andreaf has joined #openstack-qa | 12:54 | |
| *** andreaf has quit IRC | 12:54 | |
| *** andreaf has joined #openstack-qa | 12:54 | |
| *** aix has quit IRC | 13:00 | |
| *** aix has joined #openstack-qa | 13:03 | |
| *** mwagner_lap has quit IRC | 13:04 | |
| *** dkranz has quit IRC | 13:04 | |
| *** malini has joined #openstack-qa | 13:05 | |
| *** qba73_ has quit IRC | 13:15 | |
| *** qba73 has joined #openstack-qa | 13:17 | |
| *** Madan has quit IRC | 13:18 | |
| *** qba73 has quit IRC | 13:18 | |
| *** qba73 has joined #openstack-qa | 13:19 | |
| *** dustins has joined #openstack-qa | 13:19 | |
| *** jecarey has quit IRC | 13:21 | |
| *** mriedem has joined #openstack-qa | 13:23 | |
| *** topshare has joined #openstack-qa | 13:25 | |
| *** toure_ has joined #openstack-qa | 13:32 | |
| *** lordd__ has quit IRC | 13:35 | |
| *** timello has joined #openstack-qa | 13:39 | |
| *** adalbas has quit IRC | 13:41 | |
| *** krtaylor has quit IRC | 13:42 | |
| *** pcrews has joined #openstack-qa | 13:45 | |
| mriedem | mtreinish: devstack specs go under qa-specs right? | 13:46 |
| *** sdmonov has joined #openstack-qa | 13:47 | |
| *** adalbas has joined #openstack-qa | 13:53 | |
| *** SridharG has quit IRC | 13:54 | |
| *** Longgeek has quit IRC | 13:55 | |
| *** Longgeek has joined #openstack-qa | 13:56 | |
| *** mwagner_lap has joined #openstack-qa | 13:57 | |
| *** mkollaro has quit IRC | 13:57 | |
| coolsvap | gmann, afazekas andreaf please have a look at https://review.openstack.org/#/c/90306/ | 13:58 |
| *** rbak has joined #openstack-qa | 14:00 | |
| *** zz_dimtruck is now known as dimtruck | 14:01 | |
| *** cdent_ is now known as cdent | 14:02 | |
| *** dkranz has joined #openstack-qa | 14:06 | |
| *** markmcclain has joined #openstack-qa | 14:06 | |
| *** mestery has joined #openstack-qa | 14:07 | |
| *** mestery has quit IRC | 14:07 | |
| *** mestery has joined #openstack-qa | 14:08 | |
| *** aix has quit IRC | 14:08 | |
| *** Poornima has joined #openstack-qa | 14:15 | |
| *** topshare has quit IRC | 14:18 | |
| *** jecarey has joined #openstack-qa | 14:20 | |
| *** aix has joined #openstack-qa | 14:20 | |
| *** rwsu has quit IRC | 14:24 | |
| *** yamahata has joined #openstack-qa | 14:25 | |
| boris-42 | dtroyer hi | 14:27 |
| *** Poornima has quit IRC | 14:30 | |
| jgriffith | dtroyer: I'm thinking of something like this: https://gist.github.com/0db8f7a3fe53b491a50d | 14:30 |
| jgriffith | boris-42: ^^ | 14:30 |
| boris-42 | jgriffith yep but what I think is to have one argument in localrc | 14:31 |
| *** marun has joined #openstack-qa | 14:31 | |
| boris-42 | jgriffith that will enable/disable profiler in all projects | 14:31 |
| jgriffith | boris-42: oh... good idea | 14:31 |
| boris-42 | jgriffith if somebody don't want to set it at all | 14:31 |
| *** jecarey has quit IRC | 14:31 | |
| jgriffith | boris-42: yeah, I just threw that together to illustrate | 14:32 |
| jgriffith | boris-42: but yes, having it read a local.conf var is the way to go | 14:32 |
| *** topshare has joined #openstack-qa | 14:32 | |
| *** Sukhdev has joined #openstack-qa | 14:33 | |
| *** niedbalski has quit IRC | 14:33 | |
| *** Poornima has joined #openstack-qa | 14:34 | |
| *** rwsu has joined #openstack-qa | 14:36 | |
| *** lordd_ has joined #openstack-qa | 14:39 | |
| mtreinish | mriedem: yeah it's part of the qa program now | 14:40 |
| mtreinish | but I'm not sure they do bps on devstack | 14:40 |
| mtreinish | dtroyer: ^^^ | 14:40 |
| mriedem | mtreinish: ok. unrelated, wanna +W this? https://review.openstack.org/#/c/115665/ | 14:42 |
| *** mkollaro has joined #openstack-qa | 14:46 | |
| *** jecarey has joined #openstack-qa | 14:47 | |
| *** hemna_ is now known as hemna | 14:47 | |
| *** ArxCruz has quit IRC | 14:50 | |
| *** jgrimm has joined #openstack-qa | 14:58 | |
| *** rbak has quit IRC | 14:59 | |
| *** topshare has quit IRC | 15:00 | |
| *** rbak has joined #openstack-qa | 15:00 | |
| *** mwagner_lap has quit IRC | 15:03 | |
| *** BigQ has quit IRC | 15:04 | |
| *** dkranz has quit IRC | 15:05 | |
| *** toure_ has quit IRC | 15:05 | |
| *** toure_ has joined #openstack-qa | 15:08 | |
| dtroyer | mriedem: there are devstack blueprints but honestly we don't really use them much..I see I'm way behind on status updates | 15:10 |
| andreaf | mtreinish: what do you think about https://review.openstack.org/#/c/112474/? The change proposed by Ken'ichi would work on stable branches as well, but it would allow for any additional quota item... | 15:11 |
| *** openstackgerrit has joined #openstack-qa | 15:12 | |
| *** Longgeek_ has joined #openstack-qa | 15:14 | |
| lucasagomes | dtroyer, hi there, when you get some time if you could rereview https://review.openstack.org/#/c/99677 it would be great. All the work has been already merged in Ironic, but I have to keep the devstack patch updated and rebasing it etc... so it would be great if we could merge in devstack soon. Thanks! | 15:16 |
| *** yamahata has quit IRC | 15:16 | |
| lucasagomes | (it's also blocking the bp in ironic to be marked as implemented, since devstack support was part of the spec) | 15:17 |
| *** BobBall has joined #openstack-qa | 15:17 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Update Marconi to Zaqar https://review.openstack.org/115384 | 15:17 |
| *** Longgeek has quit IRC | 15:17 | |
| *** dkranz has joined #openstack-qa | 15:19 | |
| aix | y | 15:20 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts.py https://review.openstack.org/114636 | 15:21 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 15:21 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 15:21 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 15:21 |
| *** marun has quit IRC | 15:21 | |
| *** ArxCruz has joined #openstack-qa | 15:21 | |
| *** marun has joined #openstack-qa | 15:21 | |
| dtroyer | lucasagomes: [looking]…FWIW, you mentioned in a comment waiting on an ironic review to merge but there is no note that it did…I would have probably looked at it sooner if there had been one… | 15:21 |
| BobBall | Can I set a logstash query to show only failed jobs somehow? I've got a problem that seems to be showing in a large number of succeeded jobs but I think is the cause of my failure and I want to look at some other jobs that failed with this message too | 15:22 |
| mtreinish | andreaf: hmm, let me take a look | 15:22 |
| lucasagomes | dtroyer, oh :( my bad... yeah it got merged a couple of days ago | 15:22 |
| lucasagomes | and thanks for looking into it now | 15:22 |
| andreaf | mtreinish: please don't tell me I like large patches :D https://review.openstack.org/#/c/115123/3 | 15:22 |
| mtreinish | andreaf: heh, every single test file... | 15:23 |
| *** mestery has quit IRC | 15:23 | |
| mtreinish | BobBall: yeah you can use the build_status tag | 15:24 |
| *** markmcclain has quit IRC | 15:27 | |
| BobBall | ahhhh - I was trying to put it into the query somehow (and clearly getting the syntax wrong) but if I just filter using the stuff on the left it works great, thansk. | 15:27 |
| BobBall | thanks* | 15:27 |
| *** lordd_ has quit IRC | 15:29 | |
| mtreinish | ok, yeah that works too | 15:30 |
| mtreinish | normally all you need is 'AND build_status:failure' or something like that | 15:31 |
| BobBall | colon! | 15:33 |
| BobBall | thanks mtreinish. | 15:33 |
| openstackgerrit | gordon chung proposed a change to openstack/tempest: missing network configuration in telemetry tests https://review.openstack.org/115683 | 15:36 |
| *** mestery has joined #openstack-qa | 15:42 | |
| *** mestery has quit IRC | 15:42 | |
| *** mestery has joined #openstack-qa | 15:43 | |
| *** marun has quit IRC | 15:44 | |
| *** marun has joined #openstack-qa | 15:47 | |
| openstackgerrit | Emilien Macchi proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 15:48 |
| *** marun has quit IRC | 15:51 | |
| *** marun has joined #openstack-qa | 15:52 | |
| *** wuhg has quit IRC | 15:52 | |
| *** mestery has quit IRC | 15:56 | |
| *** tosky has quit IRC | 15:56 | |
| *** aimon has quit IRC | 15:56 | |
| *** jlanoux has quit IRC | 15:58 | |
| *** dkranz has quit IRC | 15:59 | |
| *** yjiang5 has joined #openstack-qa | 16:05 | |
| *** marun has quit IRC | 16:06 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated the baremetal client for bp client-checks-success https://review.openstack.org/115238 | 16:08 |
| coolsvap | mtreinish, ^^ got the issue | 16:09 |
| coolsvap | andreaf, u there | 16:09 |
| *** arnaud__ has joined #openstack-qa | 16:10 | |
| *** marun has joined #openstack-qa | 16:11 | |
| *** Obulpath_ has joined #openstack-qa | 16:12 | |
| openstackgerrit | Swapnil Kulkarni proposed a change to openstack/tempest: Updated database clients for bp client-checks-success https://review.openstack.org/115671 | 16:13 |
| *** dkranz has joined #openstack-qa | 16:14 | |
| *** markmcclain has joined #openstack-qa | 16:16 | |
| coolsvap | dkranz, hi | 16:16 |
| *** Obulpath_ has quit IRC | 16:18 | |
| coolsvap | dkranz, is anyone actively working on bp client-checks-success apart from you and andreaf ? | 16:18 |
| *** mestery has joined #openstack-qa | 16:19 | |
| andreaf | coolsvap, hi | 16:19 |
| andreaf | coolsvap, I'm not actively working on that bp | 16:19 |
| coolsvap | andreaf, hi, I was thinking if i could pickup the compute part from the bp client-checks-success, like I did for volume I will add a single change set | 16:20 |
| coolsvap | andreaf, great I will do it then for compute | 16:20 |
| andreaf | coolsvap, it would be great, thank you | 16:21 |
| *** mestery has quit IRC | 16:21 | |
| *** mestery has joined #openstack-qa | 16:22 | |
| *** bauzas has left #openstack-qa | 16:22 | |
| *** k4n0 has quit IRC | 16:25 | |
| *** qba73 has quit IRC | 16:27 | |
| *** mestery has quit IRC | 16:27 | |
| *** afazekas has quit IRC | 16:29 | |
| *** aimon has joined #openstack-qa | 16:30 | |
| *** jordanP has quit IRC | 16:33 | |
| *** Sukhdev has quit IRC | 16:33 | |
| *** Sukhdev has joined #openstack-qa | 16:34 | |
| *** krtaylor has joined #openstack-qa | 16:35 | |
| *** chandankumar has quit IRC | 16:37 | |
| *** mestery has joined #openstack-qa | 16:38 | |
| openstackgerrit | A change was merged to openstack/tempest: Convert request schema (flavors_admin) https://review.openstack.org/112783 | 16:38 |
| openstackgerrit | A change was merged to openstack/tempest: Change default size for strings to 1 https://review.openstack.org/115258 | 16:38 |
| *** Poornima has quit IRC | 16:39 | |
| *** arnaud__ has quit IRC | 16:42 | |
| *** mkollaro has quit IRC | 16:42 | |
| *** tosky has joined #openstack-qa | 16:44 | |
| *** jpich has quit IRC | 16:45 | |
| dkranz | coolsvap: Yes: https://review.openstack.org/#/c/114186/ | 16:45 |
| dkranz | coolsvap: You can look at the whiteboard in the launchpad blueprint and see which patches have referenced it | 16:46 |
| andreaf | mtreinish: the large patch adding safe_setup only changed the failure slightly :( | 16:46 |
| mtreinish | andreaf: sigh... | 16:47 |
| mtreinish | I'm wondering if the teardown side is working correctly | 16:47 |
| andreaf | mtreinish: something strange is happening now http://logs.openstack.org/78/114578/12/check/check-tempest-dsvm-full/7852bda/console.html | 16:47 |
| andreaf | mtreinish: we get 4 nested InvalidConfiguration exceptions or so it seems | 16:48 |
| *** gszasz has quit IRC | 16:48 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack-dev/devstack: Ironic: Setup/Configure iPXE https://review.openstack.org/99677 | 16:50 |
| *** mestery_ has joined #openstack-qa | 16:51 | |
| *** mestery has quit IRC | 16:51 | |
| andreaf | mtreinish: do you mean the teardown in the tests or the clear_isolated_creds from accounts.py | 16:52 |
| mtreinish | the pseudo-lock file remove | 16:52 |
| mtreinish | andreaf: we probably should add more logging details in the test patch to see if it's doing the right thing | 16:53 |
| mtreinish | I'd do it but my damn 'a' key is sticking again... | 16:53 |
| *** mestery has joined #openstack-qa | 16:55 | |
| *** mestery_ has quit IRC | 16:58 | |
| andreaf | mtreinish, ok I'll git it a try | 16:58 |
| mtreinish | andreaf: http://logs.openstack.org/78/114578/12/check/check-tempest-dsvm-full/7852bda/logs/tempest.txt.gz#_2014-08-20_15_52_50_794 | 16:58 |
| mtreinish | that's where things start to fall apart | 16:59 |
| *** mestery has quit IRC | 16:59 | |
| mtreinish | it's really the stack traces after that | 16:59 |
| coolsvap | dkranz, I checked the whiteboard and also the etherpad, I have been updating it for sometime, just wanted to confirm | 17:00 |
| coolsvap | dkranz, i am aware about neutron patch :) | 17:00 |
| andreaf | mtreinish: so the tearDownClass itself is failing | 17:01 |
| coolsvap | dkranz, wanted to confirm about compute since andreaf had updated the etherpad | 17:01 |
| mtreinish | andreaf: yeah it looks that way | 17:01 |
| coolsvap | dkranz, i am updating the etherpad for WIP items | 17:01 |
| mtreinish | because it failed to find a hash for a given cred object | 17:02 |
| *** markus_z has quit IRC | 17:03 | |
| *** lucasagomes is now known as lucas-dinner | 17:03 | |
| *** mkollaro has joined #openstack-qa | 17:03 | |
| andreaf | mtreinish: the get_hash seems broken | 17:03 |
| *** lucas-dinner has left #openstack-qa | 17:03 | |
| mtreinish | yep | 17:03 |
| *** dkranz has quit IRC | 17:03 | |
| mtreinish | it seems to fail only on the admin tests though | 17:04 |
| andreaf | mtreinish: it assumes the hash_dict will only contain username tenant and password | 17:04 |
| andreaf | mtreinish: while it will contain all the details including ids | 17:04 |
| andreaf | mtreinish: I see also non-admin tests failing | 17:05 |
| mtreinish | no that's just the logging | 17:05 |
| mtreinish | I think | 17:05 |
| mtreinish | failing on get hash? | 17:05 |
| andreaf | cred_dict = { | 17:05 |
| andreaf | 'username': creds.username, | 17:05 |
| andreaf | 'tenant_name': creds.tenant_name, | 17:05 |
| andreaf | 'password': creds.password | 17:05 |
| andreaf | } | 17:05 |
| andreaf | if self.hash_dict[hash] == cred_dict: | 17:05 |
| andreaf | return hash | 17:05 |
| andreaf | sorry multiline paste | 17:05 |
| andreaf | so it should do something like if self.hash_dict[hash] == cred_dict.to_dict() | 17:06 |
| mtreinish | yeah, but all of creds is what's being logged | 17:06 |
| mtreinish | which is where the ids are | 17:06 |
| *** atiwari has joined #openstack-qa | 17:07 | |
| mtreinish | I'm thinking we should just make a subset dict from hash_dict[hash] and only compare keys that need to be there | 17:07 |
| mtreinish | username, tenant, and pass | 17:07 |
| mtreinish | also to_dict() didn't exist when I wrote that bit :) | 17:08 |
| andreaf | mtreinish: I know but why not doing it that way? at the end the hash is created from the dict loaded from the YAML, and there is no restriction on what's in the YAML | 17:09 |
| andreaf | mtreinish: if we keep it generic people can decide to use names or ids or whatever works as valid creds | 17:09 |
| *** arnaud has joined #openstack-qa | 17:09 | |
| andreaf | andreaf, the test function is generating creds using isolated creds, and thus those creds include ids as well, which are written in the YAML along with names | 17:10 |
| andreaf | and so they go in the hash_dict | 17:10 |
| mtreinish | because I don't think we want to allow free form data in the yaml | 17:10 |
| mtreinish | it'll make things more difficult when we add networking | 17:11 |
| *** suneelb has joined #openstack-qa | 17:11 | |
| mtreinish | and role and domain support | 17:11 |
| mtreinish | I thought we were going to have a strict format on the yaml, or was I mistaken? | 17:12 |
| andreaf | mtreinish: well we can still structure it, so credentials, roles, networking | 17:12 |
| *** markmcclain has quit IRC | 17:12 | |
| andreaf | mtreinish: yes I guess that's what we discussed initially - but after seeing the implementation I see no reason for restricting which credentials fields are used | 17:13 |
| *** jlibosva has quit IRC | 17:13 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/tempest: javelin: add network resources https://review.openstack.org/100169 | 17:13 |
| andreaf | mtreinish: if we allow everything which is defined as a valid field in Credentials we can simply use the existing validation there and automatically support v2 and v3 creds | 17:14 |
| mtreinish | well I think having the ids in there is fine, but if we do to_dict() won't that require they're always there | 17:14 |
| andreaf | mtreinish: as well as leave it open for ids | 17:14 |
| mtreinish | that's my concern, I don't want to require everyone include the ids | 17:14 |
| mtreinish | because names should be sufficient for uniqueness | 17:15 |
| andreaf | mtreinish: uhm | 17:15 |
| mtreinish | well the username, pass, tenant_name triplet | 17:15 |
| andreaf | mtreinish: well in v3 case names won't be unique unless you give the domain | 17:15 |
| *** dkranz has joined #openstack-qa | 17:16 | |
| mtreinish | heh, well that answersa my inline comment :) | 17:16 |
| andreaf | so the clear credentials will take credentials from the internal dict and call the remove for each of them | 17:17 |
| andreaf | the credentials dict is created from the YAML and so is the hash_dict (created from the YAML) | 17:17 |
| andreaf | so whatever was in the YAML it should match | 17:18 |
| mtreinish | sort of self.credentials is a dict of the actual cred objects | 17:19 |
| mtreinish | which are generated from the get_credentials() method | 17:19 |
| *** harlowja_away is now known as harlowja | 17:19 | |
| mtreinish | the question is how do we compare that with the dict used to generate it | 17:20 |
| mtreinish | but, you're familiar with the credsa object code | 17:22 |
| andreaf | mtreinish: uhm right so we call auth.get_credentials() | 17:22 |
| andreaf | which has fill_in=True be default | 17:23 |
| andreaf | which means that auth will try to talk to the identity service and fill all missing fields so you have names and ids | 17:23 |
| mtreinish | ahh, so if we set fill_in=False then we can use to_dict() to compare | 17:24 |
| mtreinish | but then we don't know if the creds are valid until we usea them | 17:24 |
| mtreinish | hmm | 17:25 |
| andreaf | mtreinish: right | 17:25 |
| andreaf | mtreinish: we could also fill_in all creds before creating the hash but that would be an overhead - e.g. 8 identity calls for using maybe one or two creds only | 17:27 |
| mtreinish | oh, yeah I guess that's an option | 17:27 |
| mtreinish | well, we've got 3 ways to do it, I'll leave it up to you because I can barely type on this stupid laptop :) | 17:28 |
| andreaf | mtreinish: sigh still no new keyboard | 17:29 |
| mtreinish | personally I'd compare on the minimum subset and handle the v3 if domain is in the dict | 17:29 |
| mtreinish | andreaf: nope, they won't send me a new one I have to ship them the whole laptop | 17:30 |
| mtreinish | which is difficult because I'm constantly using it | 17:30 |
| andreaf | mtreinish: one thing we could improve is to only parse the YAML once and fill creds and put that in a singleton | 17:30 |
| *** afazekas has joined #openstack-qa | 17:30 | |
| andreaf | mtreinish: then the overhead of using the whole dict would be minimal and we would get immediate validation of all configured creds | 17:30 |
| mtreinish | but it would really be N times because in parallel it's separate processes | 17:31 |
| mtreinish | which is why I opted to do it this way | 17:32 |
| mtreinish | but sure we can give a singleton a try I guessa | 17:32 |
| andreaf | mtreinish: ok but N times with N == concurrency is still better than once per test class? | 17:32 |
| andreaf | mtreinish: so I'd start with comparing a subset of fields so we can see if things are working | 17:34 |
| andreaf | mtreinish: doing the singleton is a bit more work so we can do that once we have a working first version | 17:34 |
| mtreinish | ok, sounds like a plan | 17:35 |
| *** asselin has joined #openstack-qa | 17:35 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts.py https://review.openstack.org/114636 | 17:38 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 17:38 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 17:38 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 17:38 |
| andreaf | mtreinish, ok first simple implementation ^^^ in https://review.openstack.org/114636 | 17:38 |
| andreaf | afk - prob until tomorrow now | 17:40 |
| *** gfidente has quit IRC | 17:51 | |
| mtreinish | andreaf: ok cool, thanks | 17:55 |
| *** mpaolino has quit IRC | 17:58 | |
| *** dims has quit IRC | 17:59 | |
| *** dims has joined #openstack-qa | 17:59 | |
| *** johnthetubaguy is now known as zz_johnthetubagu | 18:02 | |
| *** lordd_ has joined #openstack-qa | 18:03 | |
| *** marun has quit IRC | 18:11 | |
| *** mkollaro has quit IRC | 18:11 | |
| *** marun has joined #openstack-qa | 18:14 | |
| *** bkopilov has joined #openstack-qa | 18:16 | |
| mkoderer | mtreinish: could I get your +A to https://review.openstack.org/#/c/113787/ | 18:16 |
| *** harlowja has quit IRC | 18:20 | |
| *** harlowja_ has joined #openstack-qa | 18:20 | |
| *** tosky has quit IRC | 18:24 | |
| *** samuelbercovici has joined #openstack-qa | 18:31 | |
| openstackgerrit | Ashish Kumar Gupta proposed a change to openstack/tempest: Floating IP Negative Tests https://review.openstack.org/109529 | 18:31 |
| *** Sukhdev has quit IRC | 18:33 | |
| *** marun has quit IRC | 18:39 | |
| *** aix has quit IRC | 18:40 | |
| openstackgerrit | Ashish Kumar Gupta proposed a change to openstack/tempest: Floating IP Negative Tests https://review.openstack.org/109529 | 18:47 |
| *** Sukhdev has joined #openstack-qa | 18:51 | |
| *** tkammer has quit IRC | 18:55 | |
| *** mriedem1 has joined #openstack-qa | 19:02 | |
| *** chandankumar has joined #openstack-qa | 19:02 | |
| *** mriedem has quit IRC | 19:02 | |
| *** kmartin has quit IRC | 19:03 | |
| *** mkollaro has joined #openstack-qa | 19:03 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 19:05 |
| *** shardy is now known as shardy_afk | 19:06 | |
| *** cdent_ has joined #openstack-qa | 19:07 | |
| *** cdent has quit IRC | 19:07 | |
| *** cdent_ is now known as cdent | 19:07 | |
| *** chandankumar has quit IRC | 19:09 | |
| *** kmartin has joined #openstack-qa | 19:19 | |
| *** cnesa7 has quit IRC | 19:25 | |
| *** tkammer has joined #openstack-qa | 19:27 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Add Ironic sideways migration testing https://review.openstack.org/111859 | 19:29 |
| *** malini has quit IRC | 19:34 | |
| *** marun has joined #openstack-qa | 19:36 | |
| *** stoner_ has quit IRC | 19:37 | |
| *** marun has quit IRC | 19:40 | |
| *** dims has quit IRC | 19:45 | |
| *** dims has joined #openstack-qa | 19:45 | |
| *** dims has quit IRC | 19:51 | |
| *** dims has joined #openstack-qa | 19:51 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack-infra/subunit2sql: Switch sample sqlalchemy url to use mysql https://review.openstack.org/115761 | 19:51 |
| dkranz | eglynn-office: I noticed that 'curl http://HOSTNAME:877' which is for ceilometer behaves differently than other services in two ways | 19:52 |
| dkranz | eglynn-office: I mean 8777 | 19:52 |
| openstackgerrit | K Jonathan Harker proposed a change to openstack-dev/devstack: Use tox for bashate testing https://review.openstack.org/115762 | 19:53 |
| dkranz | eglynn-office: First, this call requires auth, and second, even if you give it auth it returns 406 | 19:53 |
| dkranz | eglynn-office: Any idea why? | 19:54 |
| dkranz | eglynn-office: also, is it a bug that the catalog type in keystone for telemetry is 'metering' and not 'telemetry'? | 19:57 |
| *** marun has joined #openstack-qa | 19:57 | |
| cdent | dkranz: I think there's a bug reported for that first bit. | 19:58 |
| * cdent goes looking | 19:58 | |
| *** harlowja_ is now known as harlowja_away | 19:58 | |
| cdent | dkranz: this look like the same stuff: https://bugs.launchpad.net/ceilometer/+bug/1350076 ? | 19:59 |
| dkranz | cdent: thanks, checking... | 19:59 |
| cdent | also, btw, eglynn is on holiday | 19:59 |
| dkranz | cdent: ok, I will add to the ticket about the 406 and catalog type | 20:00 |
| *** lordd_ has quit IRC | 20:00 | |
| *** dims has quit IRC | 20:05 | |
| *** dims has joined #openstack-qa | 20:06 | |
| *** dims_ has joined #openstack-qa | 20:07 | |
| *** dims has quit IRC | 20:07 | |
| *** dims_ has quit IRC | 20:14 | |
| dkranz | notmyname: How do you query the swift endpoint for what swift versions are supported? Seems to be GET of http://HOSTNAME:port for other services | 20:15 |
| *** dims has joined #openstack-qa | 20:15 | |
| *** marun has quit IRC | 20:16 | |
| *** mkollaro has quit IRC | 20:17 | |
| *** marun has joined #openstack-qa | 20:18 | |
| *** dims has quit IRC | 20:19 | |
| *** Sukhdev has quit IRC | 20:20 | |
| *** mkollaro has joined #openstack-qa | 20:22 | |
| *** toure_ has quit IRC | 20:22 | |
| *** dims has joined #openstack-qa | 20:26 | |
| *** Longgeek_ has quit IRC | 20:31 | |
| *** dustins has quit IRC | 20:32 | |
| notmyname | dkranz: the short-term answer is that there is only one version (v1) to the API | 20:32 |
| notmyname | dkranz: but in general, GET the /info endpoint and you'll see what's supported in cluster | 20:33 |
| *** stevebaker has quit IRC | 20:33 | |
| *** stevebaker has joined #openstack-qa | 20:33 | |
| dkranz | notmyname: ah, ok. I saw the extensions there but did not realize it was for versions also | 20:33 |
| *** jamielennox is now known as jamielennox|away | 20:33 | |
| notmyname | dkranz: it's not. today. that is, there's only one version, so there isn't anything to report. but if we someday add another, it will be there | 20:34 |
| dkranz | notmyname: ok | 20:34 |
| dkranz | notmyname: thanks | 20:34 |
| cdent | sdague: can we chat javelin2 for a mo? | 20:40 |
| cdent | or jogo, if you're about | 20:42 |
| *** stevebaker has quit IRC | 20:43 | |
| *** stevebaker has joined #openstack-qa | 20:43 | |
| *** mriedem has joined #openstack-qa | 20:45 | |
| cdent | jogo, sdague I gotta dash, but I'm trying to resolve these two related things: http://lists.openstack.org/pipermail/openstack-dev/2014-August/043372.html https://review.openstack.org/#/c/102354/ If you could hop on that thread and that review (again) that would be great. thanks. | 20:47 |
| * cdent waves | 20:47 | |
| *** cdent has quit IRC | 20:47 | |
| *** mriedem1 has quit IRC | 20:48 | |
| stevebaker | dtroyer: thanks for those heat image building reviews, this is the start of the series | 20:55 |
| stevebaker | dtroyer: https://review.openstack.org/#/c/92257/7 | 20:55 |
| dtroyer | stevebaker: np…I already +2 that one…oh, rebase…+2 again | 20:58 |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add multi-provider extension positive test cases https://review.openstack.org/104526 | 20:59 |
| openstackgerrit | Cedric Brandily proposed a change to openstack/tempest: Add partial specs (multi-)provider extension positive test cases https://review.openstack.org/105234 | 21:00 |
| stevebaker | ianw, chmouel_: Any chance we could push this series home? Its getting old enough to grow facial hair ;) https://review.openstack.org/#/c/92257/7 | 21:00 |
| openstackgerrit | Robbie Harwood proposed a change to openstack-dev/devstack: Add and document $NOVA_VNC_ENABLED for forcing VNC on https://review.openstack.org/107073 | 21:07 |
| openstackgerrit | Robbie Harwood proposed a change to openstack-dev/devstack: Add and document $NOVA_VNC_ENABLED for forcing VNC on https://review.openstack.org/107073 | 21:10 |
| openstackgerrit | A change was merged to openstack/tempest: Convert request schema (servers) https://review.openstack.org/113787 | 21:15 |
| *** rfolco has quit IRC | 21:15 | |
| *** atiwari has quit IRC | 21:15 | |
| *** harlowja_away is now known as harlowja_ | 21:17 | |
| *** jecarey has quit IRC | 21:17 | |
| *** atiwari has joined #openstack-qa | 21:23 | |
| *** ekhugen_ has joined #openstack-qa | 21:26 | |
| *** dims has quit IRC | 21:34 | |
| *** dims has joined #openstack-qa | 21:34 | |
| *** dkranz has quit IRC | 21:37 | |
| *** dims has quit IRC | 21:39 | |
| *** marun has quit IRC | 21:42 | |
| *** mriedem has quit IRC | 21:48 | |
| *** jecarey has joined #openstack-qa | 21:50 | |
| *** weshay has quit IRC | 21:50 | |
| *** tkammer has quit IRC | 21:53 | |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Fix use of nonexistent class variable in accounts.py https://review.openstack.org/114636 | 21:57 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Make class setup safe in compute base admin class https://review.openstack.org/115123 | 21:57 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Add Credentials Provider factory https://review.openstack.org/107685 | 21:57 |
| openstackgerrit | Andrea Frittoli proposed a change to openstack/tempest: Force use of Accounts for test purposes https://review.openstack.org/114578 | 21:57 |
| *** harlowja_ has quit IRC | 21:59 | |
| andreaf | mtreinish: new version ^^^ hopefully this one makes more sense | 22:00 |
| *** harlowja has joined #openstack-qa | 22:02 | |
| *** flaper87 is now known as flaper87|afk | 22:02 | |
| *** weshay has joined #openstack-qa | 22:03 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-infra/devstack-gate: Disable cinder with ironic https://review.openstack.org/115803 | 22:04 |
| *** mkollaro has quit IRC | 22:04 | |
| *** dims has joined #openstack-qa | 22:06 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack-infra/devstack-gate: Add Ironic-python-agent support to devstack-gate https://review.openstack.org/112143 | 22:09 |
| openstackgerrit | Joe Gordon proposed a change to openstack-dev/devstack: Pull dstat logic into its own function so grenade can use it https://review.openstack.org/115526 | 22:13 |
| *** pcrews has quit IRC | 22:14 | |
| *** atiwari has quit IRC | 22:19 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-dev/devstack: Give PyMySQL a go. https://review.openstack.org/115495 | 22:20 |
| *** sdmonov has quit IRC | 22:21 | |
| jgriffith | coolsvap: ping | 22:21 |
| *** jamielennox|away is now known as jamielennox | 22:22 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Latest tox should work for openstack now, use it https://review.openstack.org/115496 | 22:25 |
| *** jgrimm has quit IRC | 22:33 | |
| openstackgerrit | Clark Boylan proposed a change to openstack-dev/devstack: Give PyMySQL a go. https://review.openstack.org/115495 | 22:35 |
| openstackgerrit | Joe Gordon proposed a change to openstack/tempest: Move javelin2 over to use oslo logging https://review.openstack.org/107609 | 22:37 |
| clarkb | mikal: are you awake yet? do you happen to know how angus is testing that particular neutron issue? | 22:37 |
| clarkb | mikal: I think 115496 will work in the gate at this point with only one somewhat unrelated change to ceilometer so I am curious to see if we can check that pymysql fixes the neutron thing you guys were hitting | 22:38 |
| *** ekhugen_ has quit IRC | 22:44 | |
| *** lordd_ has joined #openstack-qa | 22:46 | |
| ianw | stevebaker: what was the difference between 6 & 7 in https://review.openstack.org/#/c/92258/ ? | 22:51 |
| ianw | (is it just me, or is "reference version" mostly useless in gerrit?) | 22:51 |
| *** dims has quit IRC | 22:52 | |
| stevebaker | ianw: it was just a rebase. Using the Patch Set numbers can be useful: https://review.openstack.org/#/c/92258/6..7/lib/heat | 22:53 |
| *** pcrews has joined #openstack-qa | 22:54 | |
| *** dimtruck is now known as zz_dimtruck | 22:54 | |
| ianw | stevebaker: yeah ... you'd think that would be what the "reference version" pulldown would do for you. putting all the intervening unrelated changes in is annoying | 22:54 |
| stevebaker | true, it requires knowledge of the patchset, or recent other changes. not ideal | 22:55 |
| clarkb | ianw: it is how git works | 22:55 |
| clarkb | ianw: when you rebase those intervening unrelated changes are in your new change | 22:55 |
| clarkb | and are part of the diff. you can't actually know if they are unrelated | 22:55 |
| clarkb | that said checkout git reivew -m | 22:56 |
| ianw | clarkb: what i mean is that what people surely want to see is that patch-set difference view you get with X..Y | 22:58 |
| clarkb | ianw: I know | 22:58 |
| clarkb | ianw: and that is actually what you get | 22:58 |
| clarkb | if there is a rebase in the middle then that diff includes all that extra stuff | 22:59 |
| clarkb | the way git review -m works is what you really want I think | 22:59 |
| clarkb | it rebases both sides to a common base then diffs | 22:59 |
| ianw | clarkb: yeah, ok i guess I'm thinking of X..Y/specific/file which didn't change in the rebase. | 23:02 |
| ianw | stevebaker: how does https://review.openstack.org/#/c/92256/9 work with the pip_install of d-i-b in the previous patch? | 23:04 |
| ianw | stevebaker: does it really remove the need for it? | 23:04 |
| stevebaker | ianw: nope, because the aim is to build pip packages on the fly based on git checkouts, with the eventual aim of gating on changes to these agent projects, so zuul will set what git ref to checkout and a pip archive will be built off that | 23:07 |
| *** rbak has quit IRC | 23:07 | |
| ianw | stevebaker: ahh, so basically https://review.openstack.org/92257 should be using the d-i-b installed by https://review.openstack.org/#/c/92256/9 when enabled? | 23:09 |
| stevebaker | ianw: Yes, which btw is *not* what ironic want, so they just get the d-i-b from pypi, or they can pin to another non-latest version if necessary. | 23:10 |
| *** aysyd has quit IRC | 23:13 | |
| ianw | stevebaker: will there be conflicts though, if d-i-b is enabled, then https://review.openstack.org/#/c/92257/7/lib/ironic will be using the git version? should there be some workaround here? | 23:14 |
| *** lordd_ has quit IRC | 23:14 | |
| stevebaker | ianw: yes, don't enable the dib service by default, specifically don't enable it for the ironic jobs | 23:14 |
| ianw | stevebaker: that seems a little like an undocumented gotcha ... what if lib/ironic refused to go ahead if dib was installed, to ensure it was installing from pip? | 23:16 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Add within-juno dir and within-master symlink https://review.openstack.org/112731 | 23:17 |
| stevebaker | ianw: who knows, they might choose to gate on dib in the future ;) unlikely, but I don't see this as a huge issue | 23:17 |
| stevebaker | ianw: currently the dib service will only be enabled for the heat functional tests job. It might also be enabled for a future job which just gates on d-i-b and the agent projects | 23:18 |
| ianw | stevebaker: what would actually happen if ironic was using the git version? or why don't they want to use it? | 23:20 |
| *** lordd_ has joined #openstack-qa | 23:20 | |
| stevebaker | ianw: things would work as normal until there was a d-i-b commit which broke something (unlikely, hasn't happened yet). Ironic asked to only consume d-i-b releases by default, and to give them the option of pinning to a specific release just in case there is a d-i-b release with a breaking change | 23:23 |
| *** amotoki has quit IRC | 23:23 | |
| *** lordd_ has quit IRC | 23:24 | |
| stevebaker | ianw: this is something which devananda asked for as a condition of these changes going in. given that they were consuming d-i-b from their own git checkout before this change, I think their situation is now improved | 23:24 |
| mikal | clarkb: sorry, I am travelling tomorrow so have been in and out | 23:25 |
| clarkb | mikal: no problem | 23:25 |
| mikal | clarkb: Angus is on IRC... Trying pinging "gus" in #openstack? | 23:25 |
| clarkb | mikal: ya jhesketh got me sorted out | 23:25 |
| mikal | Oh, cool | 23:25 |
| mikal | Excellent | 23:25 |
| clarkb | I was looking for another nick for some reason and thank you | 23:25 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Add Ironic sideways migration testing https://review.openstack.org/111859 | 23:30 |
| *** rbak has joined #openstack-qa | 23:31 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Add dib-utils to projects list https://review.openstack.org/106810 | 23:32 |
| *** zz_dimtruck is now known as dimtruck | 23:34 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Updates to README.rst https://review.openstack.org/107427 | 23:38 |
| *** rbak has quit IRC | 23:39 | |
| *** rbak has joined #openstack-qa | 23:40 | |
| *** rbak has quit IRC | 23:42 | |
| *** adalbas has quit IRC | 23:44 | |
| *** zaro has quit IRC | 23:45 | |
| *** bkopilov has quit IRC | 23:46 | |
| *** bkopilov has joined #openstack-qa | 23:46 | |
| *** ilyashakhat_ has joined #openstack-qa | 23:47 | |
| *** ilyashakhat has quit IRC | 23:49 | |
| *** dhellmann has quit IRC | 23:49 | |
| *** hemna is now known as hemna_ | 23:49 | |
| *** dhellmann has joined #openstack-qa | 23:50 | |
| *** adalbas has joined #openstack-qa | 23:51 | |
| *** eezhova has quit IRC | 23:53 | |
| *** eezhova has joined #openstack-qa | 23:53 | |
| *** openstackgerrit has quit IRC | 23:54 | |
| *** openstackgerrit has joined #openstack-qa | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!