| openstackgerrit | Jim Rollenhagen proposed a change to openstack-dev/devstack: Fix ironic-python-agent tarball unpack https://review.openstack.org/117384 | 00:04 |
|---|---|---|
| openstackgerrit | Steve Baker proposed a change to openstack/tempest: Use cirros for orchestration test_neutron_resources https://review.openstack.org/110507 | 00:06 |
| *** dims has quit IRC | 00:07 | |
| *** bkopilov has joined #openstack-qa | 00:10 | |
| *** mlavalle has quit IRC | 00:14 | |
| *** topshare has joined #openstack-qa | 00:17 | |
| openstackgerrit | Ian Wienand proposed a change to openstack-dev/devstack: only use lsb_release for distro tags https://review.openstack.org/117104 | 00:17 |
| *** topshare has quit IRC | 00:18 | |
| *** aimon has quit IRC | 00:27 | |
| jogo | sdague mtreinish: https://review.openstack.org/#/c/117390/ | 00:28 |
| jogo | first stab at making it easier to understand what logs are what | 00:28 |
| *** qba73 has quit IRC | 00:31 | |
| *** dmorita has joined #openstack-qa | 00:33 | |
| *** aimon has joined #openstack-qa | 00:39 | |
| *** aimon has quit IRC | 00:45 | |
| *** yamahata has joined #openstack-qa | 00:54 | |
| *** yamahata has quit IRC | 00:57 | |
| *** yamahata has joined #openstack-qa | 00:57 | |
| *** dims has joined #openstack-qa | 01:05 | |
| *** dims_ has joined #openstack-qa | 01:09 | |
| *** wuhg has joined #openstack-qa | 01:10 | |
| openstackgerrit | A change was merged to openstack-dev/devstack: Move sahara client installation to lib/sahara https://review.openstack.org/112000 | 01:11 |
| *** dims__ has joined #openstack-qa | 01:11 | |
| *** dims has quit IRC | 01:11 | |
| *** dkranz has joined #openstack-qa | 01:14 | |
| *** dims_ has quit IRC | 01:14 | |
| *** aimon has joined #openstack-qa | 01:31 | |
| *** adalbas has joined #openstack-qa | 01:31 | |
| *** mriedem has joined #openstack-qa | 01:45 | |
| *** mriedem has quit IRC | 01:46 | |
| *** adalbas has quit IRC | 01:49 | |
| *** nosnos has joined #openstack-qa | 01:49 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: clean up readme https://review.openstack.org/115011 | 01:59 |
| *** openstackgerrit has quit IRC | 02:02 | |
| *** openstackgerrit has joined #openstack-qa | 02:02 | |
| *** dkranz has quit IRC | 02:06 | |
| *** dims__ has quit IRC | 02:13 | |
| *** dims has joined #openstack-qa | 02:14 | |
| *** topshare has joined #openstack-qa | 02:25 | |
| *** topshare has quit IRC | 02:36 | |
| openstackgerrit | shihanzhang proposed a change to openstack-dev/devstack: Support ipset for security group https://review.openstack.org/113453 | 02:50 |
| *** topshare has joined #openstack-qa | 02:54 | |
| *** topshare has quit IRC | 03:19 | |
| *** topshare has joined #openstack-qa | 03:22 | |
| *** nosnos has quit IRC | 03:26 | |
| *** nosnos has joined #openstack-qa | 03:27 | |
| *** Longgeek has joined #openstack-qa | 03:31 | |
| *** nosnos has quit IRC | 03:32 | |
| *** topshare has quit IRC | 03:37 | |
| *** mitz_ has quit IRC | 03:59 | |
| *** mitz has joined #openstack-qa | 03:59 | |
| *** Santosh has joined #openstack-qa | 04:07 | |
| *** topshare has joined #openstack-qa | 04:08 | |
| *** dims has quit IRC | 04:17 | |
| *** pcrews has quit IRC | 04:19 | |
| *** nosnos has joined #openstack-qa | 04:27 | |
| *** amotoki has joined #openstack-qa | 04:36 | |
| *** yfried has quit IRC | 04:48 | |
| *** Longgeek_ has joined #openstack-qa | 04:52 | |
| *** Longgeek has quit IRC | 04:56 | |
| *** dims has joined #openstack-qa | 05:18 | |
| *** topshare has quit IRC | 05:22 | |
| *** dims has quit IRC | 05:22 | |
| *** Puneet has quit IRC | 05:25 | |
| *** topshare has joined #openstack-qa | 05:27 | |
| openstackgerrit | A change was merged to openstack-infra/devstack-gate: Add Ironic-python-agent support to devstack-gate https://review.openstack.org/112143 | 05:39 |
| *** oomichi has quit IRC | 05:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/tempest: Updated from global requirements https://review.openstack.org/110110 | 05:42 |
| *** yfried has joined #openstack-qa | 05:44 | |
| *** neeti has joined #openstack-qa | 05:44 | |
| *** k4n0 has joined #openstack-qa | 06:03 | |
| *** nmagnezi__ has joined #openstack-qa | 06:09 | |
| openstackgerrit | Ian Wienand proposed a change to openstack-dev/devstack: Remove usage of $[ for arithmetic https://review.openstack.org/117432 | 06:16 |
| *** topshare has quit IRC | 06:17 | |
| *** qba73 has joined #openstack-qa | 06:20 | |
| *** bkopilov has quit IRC | 06:22 | |
| *** gmann has quit IRC | 06:27 | |
| *** flaper87|afk is now known as flaper87 | 06:27 | |
| openstackgerrit | A change was merged to openstack/tempest: Remove functionality to load json files https://review.openstack.org/115918 | 06:27 |
| *** dmorita has quit IRC | 06:30 | |
| openstackgerrit | Swami Reddy proposed a change to openstack/tempest: Add "reboot_instance" EC2 API test case https://review.openstack.org/115944 | 06:30 |
| *** gmann has joined #openstack-qa | 06:31 | |
| *** ildikov has quit IRC | 06:32 | |
| *** e0ne has joined #openstack-qa | 06:37 | |
| *** tkammer has joined #openstack-qa | 06:39 | |
| *** jlibosva has joined #openstack-qa | 06:41 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-infra/devstack-gate: Fix sideways grenade local tmp variables https://review.openstack.org/117435 | 06:41 |
| *** afazekas has joined #openstack-qa | 06:41 | |
| *** e0ne has quit IRC | 06:43 | |
| *** rdekel1 has joined #openstack-qa | 06:46 | |
| *** qba73 has quit IRC | 06:50 | |
| yfried | coolsvap: since you are online, https://review.openstack.org/#/c/90306/38/tempest/api/volume/base.py,cm | 06:51 |
| yfried | coolsvap: why do you need method "clear_qos_specs"? | 06:51 |
| yfried | coolsvap: I can't see it being used anywhere? | 06:54 |
| *** mpaolino has joined #openstack-qa | 06:55 | |
| yfried | gmann: ^ | 06:57 |
| *** e0ne has joined #openstack-qa | 06:57 | |
| yfried | gmann: maybe you know... | 06:57 |
| *** e0ne has quit IRC | 06:58 | |
| coolsvap | yfried, just amin | 07:00 |
| *** ildikov has joined #openstack-qa | 07:01 | |
| coolsvap | yfried, I think the clear_qos_specs is missed in the teardown class :( will need one more patch set | 07:04 |
| coolsvap | the base class impl has been changed so many times :( | 07:04 |
| *** marun has joined #openstack-qa | 07:14 | |
| *** harlowja_ is now known as harlowja_away | 07:14 | |
| 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:15 |
| *** SridharG has joined #openstack-qa | 07:19 | |
| *** gmann_ has joined #openstack-qa | 07:20 | |
| *** markus_z has joined #openstack-qa | 07:20 | |
| gmann_ | yfried: yes, that actually should be called from teardownclass | 07:22 |
| *** topshare has joined #openstack-qa | 07:23 | |
| gmann_ | yfried: Nice catch Thanks. I missed that :) | 07:26 |
| yfried | gmann_: coolsvap: in that case I'd -1 it. why aren't you using cleanup? | 07:28 |
| coolsvap | yfried, the cleanup is there | 07:28 |
| *** topshare has quit IRC | 07:28 | |
| yfried | coolsvap: if you are using cleanup then you don't need to call "clear" method from teardown as well. unless you have a good reason. | 07:29 |
| gmann_ | yfried: those resources needs to be shared among many tests and needs to be deleted at tearDownClass. | 07:31 |
| gmann_ | yfried: self.created_qos is the one needed by may tests | 07:32 |
| gmann_ | may -> many | 07:32 |
| *** jhenner has joined #openstack-qa | 07:32 | |
| *** topshare has joined #openstack-qa | 07:35 | |
| *** topshare has quit IRC | 07:38 | |
| *** jlanoux has joined #openstack-qa | 07:44 | |
| *** jpich has joined #openstack-qa | 07:53 | |
| *** jordanP has joined #openstack-qa | 07:55 | |
| *** topshare has joined #openstack-qa | 07:56 | |
| *** mpaolino has quit IRC | 07:58 | |
| *** cnesa8 has joined #openstack-qa | 08:00 | |
| *** e0ne has joined #openstack-qa | 08:19 | |
| openstackgerrit | Mathieu Rohon proposed a change to openstack-dev/devstack: Detect when Debian is used to launch libvirtd https://review.openstack.org/116657 | 08:21 |
| *** flaper87 is now known as flaper87|afk | 08:21 | |
| *** boris-42 has quit IRC | 08:21 | |
| *** neeti has quit IRC | 08:28 | |
| *** flaper87|afk is now known as flaper87 | 08:30 | |
| *** neeti has joined #openstack-qa | 08:30 | |
| *** boris-42 has joined #openstack-qa | 08:31 | |
| *** jamielennox is now known as jamielennox|away | 08:36 | |
| *** zz_johnthetubagu is now known as johnthetubaguy | 08:43 | |
| *** tkammer has quit IRC | 08:45 | |
| *** nosnos has quit IRC | 08:47 | |
| *** lucasagomes has joined #openstack-qa | 08:47 | |
| *** nosnos has joined #openstack-qa | 08:47 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 08:48 |
| *** bkopilov has joined #openstack-qa | 08:48 | |
| lucasagomes | hi all, I need another tempest core to take a look at https://review.openstack.org/#/c/113276/ (there's a +2 already) it's a fairly small patch adding some tests to Ironic, it's the last patch we need to mark one blueprint in J3 as Implemented | 08:48 |
| lucasagomes | thanks | 08:48 |
| *** nosnos has quit IRC | 08:52 | |
| *** tkammer has joined #openstack-qa | 09:00 | |
| openstackgerrit | Nir Magnezi proposed a change to openstack/tempest: Adding port status validation upon interface addition to a router https://review.openstack.org/117454 | 09:01 |
| openstackgerrit | Nir Magnezi proposed a change to openstack/tempest: Adding port status validation upon interface addition to a router https://review.openstack.org/117454 | 09:04 |
| openstackgerrit | Sergey Shnaidman proposed a change to openstack/tempest: DHCPv6 network tests https://review.openstack.org/117458 | 09:07 |
| *** tkammer has quit IRC | 09:09 | |
| yfried | afazekas: why did you enabled this patch again? https://review.openstack.org/#/c/93369/ this isn't needed anymore... | 09:12 |
| *** qba73 has joined #openstack-qa | 09:14 | |
| yfried | gmann_: coolsvap: still here? | 09:18 |
| *** tkammer has joined #openstack-qa | 09:22 | |
| *** gmann_ has quit IRC | 09:22 | |
| *** nosnos has joined #openstack-qa | 09:24 | |
| gmann | yfried: yup | 09:24 |
| *** BobBall_Away has left #openstack-qa | 09:29 | |
| *** luqas has joined #openstack-qa | 09:42 | |
| *** yamahata has quit IRC | 09:45 | |
| *** sdmonov has joined #openstack-qa | 09:46 | |
| *** oomichi has joined #openstack-qa | 09:47 | |
| *** mpaolino has joined #openstack-qa | 09:54 | |
| *** gmann has quit IRC | 10:00 | |
| *** gmann has joined #openstack-qa | 10:01 | |
| *** tosky has joined #openstack-qa | 10:01 | |
| gmann | yfried: qas spec created in setUpClass needs to be clean up in tearDownClass | 10:03 |
| coolsvap | yfried, lost connectivity for a bit | 10:03 |
| kashyap | afazekas, SSH time outs from " [tempest.scenario.test_snapshot_pattern]" -- http://logs.openstack.org/04/117104/2/check/check-grenade-dsvm-partial-ncpu/d3829fe/console.html | 10:04 |
| *** e0ne has quit IRC | 10:05 | |
| *** mpaolino has quit IRC | 10:06 | |
| kashyap | Hmm, seems like it's one of the top Gate bugs | 10:07 |
| yfried | gmann: coolsvap: I get it now. doesn't mean I like it, but I can't think of a better way. is it so costly to create it in setUp (as in - create a new resource for each test)? I'm less familiar with storage api | 10:10 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/elastic-recheck: Add footprint for bug 1362538 https://review.openstack.org/117468 | 10:12 |
| *** e0ne has joined #openstack-qa | 10:13 | |
| gmann | yfried: IMO, if many tests case use that resource as read only then, its better to create once and share among them irrespective of resource creation is heavy or light. | 10:15 |
| *** kmartin has quit IRC | 10:15 | |
| openstackgerrit | Sergey Shnaidman proposed a change to openstack/tempest: DHCPv6 network tests https://review.openstack.org/117458 | 10:17 |
| gmann | yfried: here i can understand your concern about resource cleanup (addCleanup is always better) :) | 10:17 |
| *** topshare has quit IRC | 10:18 | |
| coolsvap | yfried, I just pasted the output of the tmpest test results on http://paste.openstack.org/show/101625/ for test_qos | 10:18 |
| coolsvap | if you see the slowest api's are where we are still not creating new resources | 10:19 |
| coolsvap | wherever we are deleting the resouces the creation and cleanup is handled in the same request, for request which can share resources the resources are created in setupClass | 10:20 |
| *** nosnos has quit IRC | 10:20 | |
| coolsvap | yfried, ^^ | 10:20 |
| *** nosnos has joined #openstack-qa | 10:21 | |
| *** nosnos has quit IRC | 10:25 | |
| *** jordanP has quit IRC | 10:28 | |
| yfried | coolsvap: I'm sorry, I fail to understand you. if you think it's better, than I'm ok with it. I have other concerns with the patch. posting review now | 10:30 |
| coolsvap | yfried, nothing like that, its good to raise the concerns and I have more of less updated the patch to the current tempest status (client updates n all), and so many revisions, thanks gmann for helping out with the details | 10:32 |
| *** nosnos has joined #openstack-qa | 10:34 | |
| *** mkollaro has joined #openstack-qa | 10:36 | |
| yfried | gmann: coolsvap: is cls.created_qos a read-only resource? | 10:36 |
| yfried | gmann: coolsvap: why is it used in set_unset test? | 10:37 |
| yfried | and in associate_disassociate? | 10:37 |
| *** luqas has quit IRC | 10:38 | |
| openstackgerrit | Shuichiro MAKIGAKI proposed a change to openstack-dev/devstack: Fix condition of screen process check https://review.openstack.org/117475 | 10:40 |
| coolsvap | yfried set will set the qos for a volume type | 10:43 |
| coolsvap | volume type is updated | 10:43 |
| *** HenryG has quit IRC | 10:43 | |
| coolsvap | yfried, /v2/{tenant_id}/qos-specs/{qos_id}/associate : Associates a QoS specification with a specified volume type. | 10:43 |
| coolsvap | yfried, similar with disassociate | 10:43 |
| yfried | gmann: coolsvap: this seems wrong to me. if you are touching this resource (other than show/list) then it can't be shared. you don't know that disassociate/unset worked fine and your resource is in unknown | 10:44 |
| yfried | state | 10:45 |
| coolsvap | yfried, the associations are tested in tests | 10:47 |
| coolsvap | with asserts in addition to the response checks in clients | 10:47 |
| gmann | yfried: association/dissociation does not alter this resource so that it can cause any harm to other tests. | 10:48 |
| yfried | gmann: coolsvap ^ those the same apply for set_unset? | 10:49 |
| coolsvap | yfried, I said the same thing gmann is saying, quo resource is not updated, volume_type is | 10:49 |
| coolsvap | s/quo/qos | 10:50 |
| gmann | yfried: test_set_unset_qos_key set and then unset the key on qos spec in same test and tests runs in parallel at class level. this also does not harm other tests operation. | 10:51 |
| yfried | gmann: coolsvap: ok. then see my last -1 and I'm done with this review. fix the decorator on the test methods and I'm ok with it | 10:52 |
| *** aix has joined #openstack-qa | 10:52 | |
| gmann | yfried: decorator on the test you means tagging those as gate etc? | 10:55 |
| yfried | gmann: yes | 10:56 |
| yfried | gmann: I think that otherwise they won't run at the gate | 10:56 |
| gmann | yfried: well that is not being used in gate now a days. | 10:57 |
| yfried | gmann: what do you mean? these tests shouldn't run at the gate? | 10:57 |
| gmann | yfried: this tag was used very long back when gate used to filter the tests which to run or not. Now gate run all tests matching the configuration option. | 10:58 |
| gmann | yfried: i means tag "gate" is not being used in gate | 10:58 |
| yfried | gmann: and smoke? | 10:58 |
| yfried | mtreinish: sdague: assuming clients are validating response, do they really need to return it as well? most tests are doing | 11:02 |
| yfried | _, body = client_call() | 11:02 |
| gmann | yfried: that mainly for 2 purpose. 1 for neutron as they had issue with running all tests in parallel due to stability . 2 for grenade case to avoid running all the tests for functionality verification | 11:03 |
| gmann | yfried: here tests case can be tagged as smoke if you think it is needed. | 11:03 |
| *** ildikov has quit IRC | 11:05 | |
| yfried | gmann: I don't know. I was unfamiliar with the reasons you brought up. thank you for the new info. don't tag them if you don't think they need to be tagged. could you still fix the placement of the tests in the module? I find it easier to read if we group actual tests togather. esp for thos unfamiliar with the test | 11:06 |
| coolsvap | yfried, gmann lost connectivity again, but i believe all the tests need to run in gate | 11:06 |
| yfried | coolsvap: as gmann said, apparently they will run regardless of tag. only need the smoke tag for grenade. if you don't want them to run on grenade then ignore my comment | 11:07 |
| gmann | yfried: for tagging as smoke i agree. I have provided the list of tests which needed tag as smoke for functional verification in grenade case | 11:07 |
| gmann | yfried: I think those tests can tests the most/basic QoS spec functionality during smoke kind of testing | 11:08 |
| coolsvap | gmann, yfried thanks for the comments | 11:09 |
| openstackgerrit | A change was merged to openstack/tempest: Catch BadRequest exception from correct module https://review.openstack.org/115851 | 11:09 |
| gmann | coolsvap: np :) | 11:11 |
| openstackgerrit | A change was merged to openstack-dev/grenade: Add within-juno dir and within-master symlink https://review.openstack.org/112731 | 11:11 |
| openstackgerrit | A change was merged to openstack-dev/grenade: Source all devstack service libs for Tempest https://review.openstack.org/114087 | 11:11 |
| openstackgerrit | A change was merged to openstack-dev/grenade: Fix check-sanity + save-state /w disabled services https://review.openstack.org/113405 | 11:12 |
| yfried | coolsvap: sure. thank you for being so patient | 11:13 |
| *** tkammer has quit IRC | 11:14 | |
| 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 | 11:16 |
| *** nosnos has quit IRC | 11:18 | |
| yfried | coolsvap: I really don't want to do another -1. could you please move L108-L1329 so all tests are grouped together? | 11:20 |
| *** nosnos has joined #openstack-qa | 11:20 | |
| coolsvap | yfried, just a min doing it | 11:20 |
| 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 | 11:22 |
| *** tkammer has joined #openstack-qa | 11:27 | |
| openstackgerrit | Sean Dague proposed a change to openstack-dev/devstack: remove kombu from package lists https://review.openstack.org/117482 | 11:27 |
| *** sdmonov has quit IRC | 11:29 | |
| *** mkollaro1 has joined #openstack-qa | 11:31 | |
| sdague | yfried: I think some times you want the response, honestly I had an early proposal to change the body into an object that included the response as an attribute, so you could get it if you wanted it | 11:32 |
| sdague | that would be my preference | 11:32 |
| *** mkollaro has quit IRC | 11:33 | |
| yfried | sdague: tnx. can you think of a single place where the response is used? and assuming it's validated correctly, and code other than excepted response could be retrieved by catching the exception, no? | 11:34 |
| sdague | yfried: I don't right now, but with the way we are expected to do versioning in nova having access to the headers on the response is going to be important | 11:37 |
| sdague | so if we converted to class Body(dict): with a .resp attr we'd build a reasonable interface here | 11:38 |
| sdague | and I think it would give you all the cleanups that you want in the main path code | 11:39 |
| openstackgerrit | Ghanshyam Mann proposed a change to openstack/tempest: Use safe_setup instead try block in setUpClass https://review.openstack.org/110844 | 11:42 |
| openstackgerrit | Sean Dague proposed a change to openstack/tempest: Cleanup neutron default security groups on tenant deletion https://review.openstack.org/79275 | 11:46 |
| yfried | sdague: what do you mean "give you all the cleanups that you want in the main path code"? | 11:46 |
| sdague | I'm assuming the issue you are trying to address is _, body = foo() being everywhere | 11:46 |
| yfried | sdague: yeah | 11:47 |
| sdague | right | 11:47 |
| yfried | sdague: ok. tnx. this seems like a spec is required, though... | 11:47 |
| sdague | yeh, so originally I was thinking about it the other way it appears - https://review.openstack.org/#/c/81276/4/specs/rest-clients-return-obj.rst | 11:49 |
| sdague | but I'd actually say dict + resp object would make more sense | 11:49 |
| yfried | sdague: did you mean about times where body is ignored? | 11:51 |
| sdague | yeh, I did at the time | 11:51 |
| yfried | sdague: resp, _ = client_call()? | 11:51 |
| sdague | but thigns have changed | 11:51 |
| sdague | that was 6 months ago | 11:51 |
| *** HenryG has joined #openstack-qa | 11:52 | |
| yfried | sdague: ok. tnx for the link | 11:52 |
| yfried | sdague: where you by chance able to catch the conv I had with mtreinish yesterday. about OOwrapper for resource dicts? | 11:52 |
| yfried | seems like combining the output of the client inot an object holding both resp and body_dict could be upgraded to resource object allowing resource related ops to be carried out via resource | 11:54 |
| yfried | server = create_server() | 11:54 |
| yfried | server.resp holds the response you mentioned | 11:54 |
| yfried | server.id would point to body_dict['id'] | 11:55 |
| yfried | and server.reboot() would do self.servers_client.reboot_server(body_dict['id']) | 11:55 |
| yfried | sdague: what do you think? | 11:56 |
| sdague | so, I think that's fine. That being said, this being a testing library it's somewhat helpful for it to be really explicit (and for people to realize what the rest calls are getting called) | 11:57 |
| sdague | I'd say before going down that path providing some guidelines about interface allowances, for instance, that instance methods must only ever call 1 rest call under the covers | 11:58 |
| *** cdent has joined #openstack-qa | 11:59 | |
| *** neeti has quit IRC | 11:59 | |
| openstackgerrit | John Davidge proposed a change to openstack-dev/devstack: Add IPv6 support for tenant data network https://review.openstack.org/87987 | 12:01 |
| *** tosky has quit IRC | 12:01 | |
| *** jordanP has joined #openstack-qa | 12:02 | |
| *** topshare has joined #openstack-qa | 12:04 | |
| *** e0ne has quit IRC | 12:04 | |
| *** jhenner1 has joined #openstack-qa | 12:06 | |
| *** nmagnezi__ has quit IRC | 12:06 | |
| *** yfried has quit IRC | 12:06 | |
| *** afazekas has quit IRC | 12:06 | |
| *** jhenner has quit IRC | 12:07 | |
| *** yfried has joined #openstack-qa | 12:07 | |
| *** yfried has quit IRC | 12:08 | |
| *** yfried has joined #openstack-qa | 12:08 | |
| *** yfried_ has joined #openstack-qa | 12:10 | |
| *** masayukig has quit IRC | 12:10 | |
| *** jecarey has joined #openstack-qa | 12:11 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 12:12 |
| openstackgerrit | Puneet Arora proposed a change to openstack/tempest: Added keystone cli test case https://review.openstack.org/114885 | 12:12 |
| maurosr | ls | 12:13 |
| *** yfried has quit IRC | 12:13 | |
| openstackgerrit | Nir Magnezi proposed a change to openstack/tempest: Adding port status validation upon interface addition to a router https://review.openstack.org/117454 | 12:15 |
| *** masayukig has joined #openstack-qa | 12:15 | |
| *** weshay has joined #openstack-qa | 12:16 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 12:17 |
| *** e0ne has joined #openstack-qa | 12:18 | |
| *** cdent has quit IRC | 12:19 | |
| yfried_ | sdague: so this seems like specs as well. tnx. I'd write one (or update yours - what do you think is better?) | 12:20 |
| sdague | yfried_: probably start from scratch, mine is kind of old in it's approach | 12:21 |
| *** adalbas has joined #openstack-qa | 12:21 | |
| *** nmagnezi__ has joined #openstack-qa | 12:21 | |
| *** nosnos has quit IRC | 12:22 | |
| *** nosnos has joined #openstack-qa | 12:23 | |
| *** aysyd has joined #openstack-qa | 12:24 | |
| *** nosnos has quit IRC | 12:27 | |
| *** topshare has quit IRC | 12:28 | |
| *** acfleury has joined #openstack-qa | 12:28 | |
| *** yamahata has joined #openstack-qa | 12:30 | |
| *** topshare has joined #openstack-qa | 12:30 | |
| openstackgerrit | A change was merged to openstack/tempest: Add REVIEWING.rst https://review.openstack.org/99451 | 12:31 |
| *** acfleury has quit IRC | 12:36 | |
| *** jecarey has quit IRC | 12:37 | |
| openstackgerrit | Nir Magnezi proposed a change to openstack/tempest: Adding port status validation upon interface addition to a router https://review.openstack.org/117454 | 12:39 |
| *** dpaterson has joined #openstack-qa | 12:39 | |
| *** marun has quit IRC | 12:45 | |
| openstackgerrit | gordon chung proposed a change to openstack-dev/grenade: add new config file to upgrade https://review.openstack.org/114278 | 12:50 |
| *** zz_dimtruck is now known as dimtruck | 12:51 | |
| *** marun has joined #openstack-qa | 12:55 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/elastic-recheck: iAdd footprint for bug 1362538 https://review.openstack.org/117468 | 12:55 |
| *** sdmonov has joined #openstack-qa | 12:56 | |
| *** topshare has quit IRC | 13:01 | |
| *** mkollaro has joined #openstack-qa | 13:05 | |
| *** mkollaro1 has quit IRC | 13:07 | |
| *** yfried_ has quit IRC | 13:07 | |
| *** jhenner1 has quit IRC | 13:09 | |
| *** jaypipes has joined #openstack-qa | 13:09 | |
| *** ChanServ sets mode: +o jaypipes | 13:09 | |
| *** tkammer has quit IRC | 13:09 | |
| *** k4n0 has quit IRC | 13:10 | |
| *** qba73 has quit IRC | 13:10 | |
| *** qba73 has joined #openstack-qa | 13:13 | |
| *** markmcclain has joined #openstack-qa | 13:14 | |
| *** sballe_ has quit IRC | 13:16 | |
| *** sballe_ has joined #openstack-qa | 13:16 | |
| *** afazekas has joined #openstack-qa | 13:16 | |
| *** qba73 has quit IRC | 13:17 | |
| *** nmagnezi__ has quit IRC | 13:18 | |
| openstackgerrit | Yuriy Taraday proposed a change to openstack-dev/devstack: Add rootwrap daemon mode support for Neutron https://review.openstack.org/107386 | 13:18 |
| *** dustins has joined #openstack-qa | 13:20 | |
| *** oomichi has quit IRC | 13:20 | |
| *** tosky has joined #openstack-qa | 13:20 | |
| *** qba73 has joined #openstack-qa | 13:21 | |
| *** tkammer has joined #openstack-qa | 13:22 | |
| *** jhenner has joined #openstack-qa | 13:23 | |
| *** qba73 has quit IRC | 13:24 | |
| *** qba73 has joined #openstack-qa | 13:25 | |
| *** mriedem has joined #openstack-qa | 13:26 | |
| *** qba73 has quit IRC | 13:28 | |
| *** qba73 has joined #openstack-qa | 13:32 | |
| *** cdent has joined #openstack-qa | 13:32 | |
| *** qba73 has quit IRC | 13:35 | |
| *** qba73 has joined #openstack-qa | 13:37 | |
| *** evgenyf has joined #openstack-qa | 13:38 | |
| *** adalbas has quit IRC | 13:40 | |
| *** salv-orlando has quit IRC | 13:43 | |
| *** rbak has quit IRC | 13:43 | |
| *** dustins has quit IRC | 13:45 | |
| *** afazekas has quit IRC | 13:46 | |
| *** tosky_ has joined #openstack-qa | 13:46 | |
| *** SridharG1 has joined #openstack-qa | 13:47 | |
| *** nmagnezi has joined #openstack-qa | 13:48 | |
| *** SridharG has quit IRC | 13:49 | |
| *** malini1 has joined #openstack-qa | 13:49 | |
| *** dkranz has joined #openstack-qa | 13:49 | |
| *** tosky has quit IRC | 13:50 | |
| *** markus_z has quit IRC | 13:50 | |
| *** amotoki has quit IRC | 13:51 | |
| *** sdmonov has quit IRC | 13:51 | |
| *** amotoki has joined #openstack-qa | 13:51 | |
| *** SridharG1 has quit IRC | 13:52 | |
| *** adalbas has joined #openstack-qa | 13:52 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Reduce the number of workers needed particularly when planning to run Tempest. https://review.openstack.org/117517 | 13:55 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Reduce the number of workers needed when running Tempest. https://review.openstack.org/117517 | 13:56 |
| *** pcrews has joined #openstack-qa | 13:59 | |
| *** SridharG has joined #openstack-qa | 14:00 | |
| *** andreaf has joined #openstack-qa | 14:03 | |
| *** rbak has joined #openstack-qa | 14:06 | |
| openstackgerrit | Roey Dekel proposed a change to openstack/tempest: Enhanced connectivity check using multiple protocols https://review.openstack.org/117520 | 14:06 |
| *** dustins has joined #openstack-qa | 14:08 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Reduce the number of workers needed when running Tempest. https://review.openstack.org/117517 | 14:09 |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Reduce the number of workers needed when running Tempest. https://review.openstack.org/117517 | 14:10 |
| *** dimtruck is now known as zz_dimtruck | 14:10 | |
| *** zz_dimtruck is now known as dimtruck | 14:11 | |
| *** tosky_ has quit IRC | 14:11 | |
| openstackgerrit | Bob Ball proposed a change to openstack-dev/devstack: Reduce the number of workers needed when running Tempest. https://review.openstack.org/117517 | 14:11 |
| *** tosky_ has joined #openstack-qa | 14:11 | |
| *** SridharG has left #openstack-qa | 14:11 | |
| *** rdekel1 has quit IRC | 14:12 | |
| *** samuelbercovici has joined #openstack-qa | 14:14 | |
| *** andreaf has quit IRC | 14:14 | |
| *** Longgeek_ has quit IRC | 14:24 | |
| *** jgrimm_zz is now known as jgrimm | 14:24 | |
| *** tkammer has quit IRC | 14:28 | |
| *** andreaf has joined #openstack-qa | 14:28 | |
| *** yamahata has quit IRC | 14:29 | |
| *** salv-orlando has joined #openstack-qa | 14:30 | |
| openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Fix regression in Cinder volume type creation https://review.openstack.org/117522 | 14:32 |
| *** markmcclain has quit IRC | 14:32 | |
| *** masayukig has quit IRC | 14:34 | |
| *** masayukig has joined #openstack-qa | 14:40 | |
| *** tnurlygayanov has quit IRC | 14:42 | |
| *** andreaf has quit IRC | 14:43 | |
| *** andreaf has joined #openstack-qa | 14:43 | |
| *** atiwari has joined #openstack-qa | 14:45 | |
| *** qba73 has quit IRC | 14:46 | |
| openstackgerrit | Mathieu Rohon proposed a change to openstack-dev/devstack: Detect when Debian is used to launch libvirtd https://review.openstack.org/116657 | 14:48 |
| openstackgerrit | Ala Rezmerita proposed a change to openstack/tempest: Cleanup neutron default security groups on tenant deletion https://review.openstack.org/79275 | 14:50 |
| *** ildikov has joined #openstack-qa | 14:53 | |
| *** andreaf has quit IRC | 14:54 | |
| *** andreaf has joined #openstack-qa | 14:54 | |
| *** kmartin has joined #openstack-qa | 15:00 | |
| *** balles has joined #openstack-qa | 15:01 | |
| *** balles has quit IRC | 15:01 | |
| *** nmagnezi has quit IRC | 15:03 | |
| *** coolsvap has quit IRC | 15:07 | |
| *** salv-orlando has quit IRC | 15:14 | |
| *** coolsvap has joined #openstack-qa | 15:19 | |
| *** mwagner_lap has joined #openstack-qa | 15:20 | |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Migrate test_network_basic_ops to tempest clients https://review.openstack.org/111719 | 15:24 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Adds status check for FloatingIP in scenarios https://review.openstack.org/102700 | 15:24 |
| *** evgenyf has quit IRC | 15:25 | |
| *** sdmonov has joined #openstack-qa | 15:27 | |
| *** HenryG has quit IRC | 15:28 | |
| *** samuelbercovici has quit IRC | 15:33 | |
| *** markmcclain has joined #openstack-qa | 15:37 | |
| openstackgerrit | Morgan Fainberg proposed a change to openstack-dev/devstack: Use the apache 2.4 ErrorLogFormat directive https://review.openstack.org/102425 | 15:40 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Migrate advanced server network to tempest clients https://review.openstack.org/113124 | 15:40 |
| openstackgerrit | Yair Fried proposed a change to openstack/tempest: Migrate security_groups_basic to tempest clients https://review.openstack.org/112199 | 15:40 |
| *** jlanoux has quit IRC | 15:46 | |
| *** bkopilov has quit IRC | 15:49 | |
| *** jordanP has quit IRC | 16:04 | |
| *** dustins has quit IRC | 16:08 | |
| *** mlavalle has joined #openstack-qa | 16:08 | |
| *** e0ne has quit IRC | 16:12 | |
| openstackgerrit | meera-belur proposed a change to openstack/tempest: Adding basic tests for the barbican - key management service https://review.openstack.org/117032 | 16:27 |
| *** aimon has quit IRC | 16:28 | |
| *** sdmonov has quit IRC | 16:29 | |
| *** jaypipes has quit IRC | 16:30 | |
| jlibosva | sdague: hi, I know you must be busy before release:) Any chance you can look at https://review.openstack.org/#/c/117146/ ? | 16:33 |
| *** lucasagomes has quit IRC | 16:34 | |
| jlibosva | sdague: it's related to https://review.openstack.org/#/c/100169/ which basically will add support for network in javelin in order to not fail in neutron grenade | 16:34 |
| sdague | jlibosva: seems sane | 16:34 |
| *** jlanoux has joined #openstack-qa | 16:35 | |
| *** jpich has quit IRC | 16:37 | |
| *** aix has quit IRC | 16:40 | |
| *** mwagner_lap has quit IRC | 16:41 | |
| *** jaypipes has joined #openstack-qa | 16:43 | |
| *** ChanServ sets mode: +o jaypipes | 16:43 | |
| *** jaypipes has quit IRC | 16:43 | |
| *** andreaf_ has joined #openstack-qa | 16:43 | |
| *** andreaf has quit IRC | 16:44 | |
| *** jaypipes has joined #openstack-qa | 16:46 | |
| *** ChanServ sets mode: +o jaypipes | 16:46 | |
| *** mpaolino has joined #openstack-qa | 16:46 | |
| *** suneelb has joined #openstack-qa | 16:51 | |
| morganfainberg | ianw, so re fedora and httpd restarts sometimes failing, new FC20 build (this is that bug from a while ago where stop/start vs restart tends to make things fail less), I have confirmed it still exists | 16:51 |
| morganfainberg | ianw, i'm not sure how we can fix this, it's a very long standing bug. I'll see what we can do about reporting it upstream...but my guess is the answer will be "work around it" | 16:53 |
| *** qba73 has joined #openstack-qa | 16:55 | |
| *** aimon has joined #openstack-qa | 16:57 | |
| jlibosva | dtroyer: jogo Hi, can you please look at https://review.openstack.org/#/c/117146/1 ? We need it to validate patch for javelin that supposed to fix neutron grenade | 16:58 |
| *** sdmonov has joined #openstack-qa | 16:59 | |
| *** tosky_ is now known as tosky | 17:01 | |
| *** suneelb has quit IRC | 17:02 | |
| *** suneelb has joined #openstack-qa | 17:02 | |
| *** suneelb has quit IRC | 17:02 | |
| *** suneelb has joined #openstack-qa | 17:03 | |
| *** suneelb has quit IRC | 17:03 | |
| *** harlowja_away is now known as harlowja_ | 17:03 | |
| *** suneelb has joined #openstack-qa | 17:03 | |
| *** harlowja_ is now known as harlowja_away | 17:04 | |
| *** suneelb has quit IRC | 17:04 | |
| *** suneelb has joined #openstack-qa | 17:04 | |
| *** Santosh has quit IRC | 17:06 | |
| *** suneelb has quit IRC | 17:07 | |
| *** suneelb has joined #openstack-qa | 17:07 | |
| *** harlowja_away is now known as harlowja_ | 17:09 | |
| *** mpaolino has quit IRC | 17:09 | |
| *** andreaf_ has quit IRC | 17:09 | |
| *** andreaf_ has joined #openstack-qa | 17:10 | |
| *** e0ne has joined #openstack-qa | 17:10 | |
| *** markmcclain has quit IRC | 17:17 | |
| *** jlibosva has quit IRC | 17:19 | |
| *** dustins has joined #openstack-qa | 17:20 | |
| *** tosky has quit IRC | 17:30 | |
| *** dkranz is now known as dkranz_afk | 17:33 | |
| openstackgerrit | Alexei Vinogradov proposed a change to openstack-dev/devstack: Add support of hbase database https://review.openstack.org/86683 | 17:33 |
| *** suneelb has quit IRC | 17:35 | |
| *** suneelb has joined #openstack-qa | 17:36 | |
| *** andreaf_ has quit IRC | 17:36 | |
| *** sabeen has quit IRC | 17:36 | |
| *** andreaf has joined #openstack-qa | 17:37 | |
| *** dkranz_afk is now known as dkranz | 17:38 | |
| *** Sukhdev has joined #openstack-qa | 17:40 | |
| *** markmcclain has joined #openstack-qa | 17:42 | |
| *** rfolco has quit IRC | 17:43 | |
| *** andreaf has quit IRC | 17:47 | |
| *** andreaf has joined #openstack-qa | 17:47 | |
| *** tosky has joined #openstack-qa | 17:48 | |
| *** yfried_ has joined #openstack-qa | 17:48 | |
| *** jlanoux has quit IRC | 17:49 | |
| mtreinish | asselin: anyway the difference for the script I alluded to before is that there they'll be taking a subset of an existing repo to make a new one | 17:50 |
| mtreinish | while for tempest lib migration we need to move from one existing repo into another | 17:51 |
| mtreinish | it's slightly different, although most of the steps are the same | 17:51 |
| *** andreaf has quit IRC | 17:51 | |
| asselin | mtreinish, I don't really see the difference. Isn't the tempest-lib a subset of tempest? | 17:52 |
| mtreinish | oslo actually already has a good script for graduating a library which would probably be useful for that puppet spec | 17:52 |
| mtreinish | asselin: no, because we're moving pieces over piecemeal | 17:52 |
| *** marun has quit IRC | 17:52 | |
| asselin | ok i see | 17:52 |
| mtreinish | for example, look at the WIP repo | 17:52 |
| dhellmann | mtreinish: a few steps of our script are specific to oslo, but I've been meaning to refactor it to make a tool to just export part of the history of a repo | 17:52 |
| dhellmann | mtreinish: in the mean time, just comment out the steps you don't want :-) | 17:52 |
| mtreinish | dhellmann: heh, yeah I just stole the git bits to figure out what to do manually | 17:53 |
| mtreinish | it was really useful | 17:53 |
| mtreinish | asselin: that repo has merged a few files over but eventually we need to move a lot more | 17:53 |
| mtreinish | like the rest client | 17:53 |
| mtreinish | asselin: the steps for extracting the history will be basically the same | 17:54 |
| mtreinish | it's just the merge side into tempest-lib that's different | 17:54 |
| *** andreaf has joined #openstack-qa | 17:54 | |
| asselin | mtreinish, is there any reason why it has to be peicemeal? | 17:54 |
| mtreinish | asselin: mostly time. Doing the migration also involves some refactoring. | 17:55 |
| *** rfolco has joined #openstack-qa | 17:55 | |
| mtreinish | we don't want to hold up dev while working on that | 17:55 |
| mtreinish | so doing it by functional chunks made sense here | 17:56 |
| asselin | ok | 17:56 |
| *** morganfainberg is now known as morganfainberg_Z | 17:57 | |
| ekhugen_alt | mtreinish, I've submitted patches for all the places I could find where we were missing the service tags for the add-service-tags bp. I don't know if that's enough to mark the bp complete | 17:59 |
| mtreinish | ekhugen_alt: I think it is. Honestly, it'll never be 100% because things are always changing | 17:59 |
| *** johnthetubaguy is now known as zz_johnthetubagu | 17:59 | |
| mtreinish | but if you've looked and think you've got them all that's fine | 18:00 |
| mtreinish | the BP has been open for almost a year | 18:00 |
| mtreinish | so when the last of your patches merges I think we can mark it as closed :) | 18:00 |
| ekhugen_alt | okay, thanks mtreinish | 18:00 |
| *** mestery has quit IRC | 18:08 | |
| *** mestery has joined #openstack-qa | 18:08 | |
| *** andreaf has quit IRC | 18:15 | |
| *** qba73 has quit IRC | 18:17 | |
| *** mwagner_lap has joined #openstack-qa | 18:18 | |
| *** andreaf has joined #openstack-qa | 18:19 | |
| *** salv-orlando has joined #openstack-qa | 18:28 | |
| *** andreaf has quit IRC | 18:29 | |
| *** dustins has quit IRC | 18:30 | |
| *** wuhg has quit IRC | 18:32 | |
| *** suneelb has quit IRC | 18:35 | |
| *** suneelb has joined #openstack-qa | 18:36 | |
| openstackgerrit | Joe Gordon proposed a change to openstack-infra/elastic-recheck: Make list of open reviews bold https://review.openstack.org/117588 | 18:38 |
| *** linkid has joined #openstack-qa | 18:38 | |
| *** linkid has left #openstack-qa | 18:40 | |
| *** suneelb has quit IRC | 18:41 | |
| *** dustins has joined #openstack-qa | 18:41 | |
| *** e0ne has quit IRC | 18:46 | |
| *** cdent has quit IRC | 18:49 | |
| *** e0ne has joined #openstack-qa | 18:50 | |
| jogo | dtroyer sdague mtreinish: your thoughts on https://review.openstack.org/#/c/117390/2 would be helpful | 18:51 |
| jogo | https://review.openstack.org/#/c/117390/2 imrpoves the apache footer to explain what logs are what | 18:51 |
| *** mkollaro has quit IRC | 18:52 | |
| mtreinish | jogo: do you want inline comments flushing out the missing files, or should I just push a patch on top of it? | 18:54 |
| *** dustins has quit IRC | 18:55 | |
| jogo | mtreinish: you can just ammend the patch if you want | 18:56 |
| mtreinish | jogo: ok | 18:56 |
| openstackgerrit | Adam Gandelman proposed a change to openstack-dev/grenade: Add Ironic sideways migration testing https://review.openstack.org/111859 | 19:00 |
| jogo | mtreinish: thanks | 19:00 |
| *** mkollaro has joined #openstack-qa | 19:00 | |
| *** suneelb has joined #openstack-qa | 19:01 | |
| *** amotoki_ has joined #openstack-qa | 19:02 | |
| *** dustins has joined #openstack-qa | 19:02 | |
| *** masayukig has quit IRC | 19:02 | |
| *** amotoki has quit IRC | 19:03 | |
| *** cnesa8 has quit IRC | 19:05 | |
| *** masayukig has joined #openstack-qa | 19:08 | |
| *** balles has joined #openstack-qa | 19:15 | |
| *** mkollaro has quit IRC | 19:17 | |
| *** balles has quit IRC | 19:18 | |
| *** sballe has joined #openstack-qa | 19:18 | |
| dtroyer | jogo: pinging you since you brought it up last, ;) I collected my notes on the DevStack service name and logging in https://etherpad.openstack.org/p/devstack-logging | 19:20 |
| *** flaper87 is now known as flaper87|afk | 19:21 | |
| dtroyer | I'd like to get some feedback particularly around the compatibility needs. I know what the gate does but don't have a good sense on what folk do locally | 19:21 |
| jogo | dtroyer: looks good | 19:23 |
| jogo | dtroyer: yeah not sure what folks do locally | 19:23 |
| jogo | being this is a dev/test tool I don't think its very bad if we unintentionally break backwards compat | 19:24 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack-infra/elastic-recheck: Add footprint for bug 1362528 https://review.openstack.org/117468 | 19:26 |
| *** e0ne has quit IRC | 19:27 | |
| *** e0ne has joined #openstack-qa | 19:29 | |
| *** e0ne has quit IRC | 19:30 | |
| *** openstackgerrit has quit IRC | 19:31 | |
| *** openstackgerrit has joined #openstack-qa | 19:32 | |
| *** mestery has quit IRC | 19:35 | |
| *** mestery has joined #openstack-qa | 19:36 | |
| *** Sukhdev has quit IRC | 19:40 | |
| *** dims has joined #openstack-qa | 19:46 | |
| openstackgerrit | A change was merged to openstack/tempest: Add functools.wraps on all function decorators https://review.openstack.org/113899 | 19:51 |
| *** mwagner_lap has quit IRC | 19:52 | |
| *** dims has quit IRC | 19:53 | |
| *** e0ne has joined #openstack-qa | 19:53 | |
| openstackgerrit | Sean M. Collins proposed a change to openstack-dev/devstack: Ubuntu: Start openvswitch-switch service after install https://review.openstack.org/117603 | 19:54 |
| openstackgerrit | Sean M. Collins proposed a change to openstack-dev/devstack: Ubuntu: Start openvswitch-switch service after install https://review.openstack.org/117603 | 19:54 |
| sc68cal | always forget the closes-bug syntax for gerrit... | 19:57 |
| *** qba73 has joined #openstack-qa | 20:01 | |
| *** e0ne has quit IRC | 20:03 | |
| *** malini1 has quit IRC | 20:04 | |
| *** dimtruck is now known as zz_dimtruck | 20:06 | |
| *** e0ne has joined #openstack-qa | 20:07 | |
| *** zz_dimtruck is now known as dimtruck | 20:08 | |
| *** sballe has quit IRC | 20:14 | |
| *** tkammer has joined #openstack-qa | 20:16 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-infra/elastic-recheck: Add fingerprint for bug 1347692 https://review.openstack.org/117607 | 20:17 |
| *** Dafna has quit IRC | 20:23 | |
| *** qba73 has quit IRC | 20:27 | |
| *** qba73 has joined #openstack-qa | 20:27 | |
| *** e0ne has quit IRC | 20:30 | |
| *** amotoki_ has quit IRC | 20:32 | |
| *** e0ne has joined #openstack-qa | 20:33 | |
| *** catohornet1 has joined #openstack-qa | 20:35 | |
| *** dustins has quit IRC | 20:38 | |
| *** e0ne has quit IRC | 20:42 | |
| *** e0ne has joined #openstack-qa | 20:46 | |
| *** e0ne has quit IRC | 20:49 | |
| *** jaypipes has quit IRC | 20:50 | |
| *** e0ne has joined #openstack-qa | 20:53 | |
| openstackgerrit | yunhong-jiang proposed a change to openstack/tempest: Test boot from volume checking https://review.openstack.org/116739 | 20:53 |
| openstackgerrit | yunhong-jiang proposed a change to openstack/tempest: Passing parameter to create image https://review.openstack.org/116738 | 20:53 |
| ianw | morganfainberg: huh, ok, will look out for that issue | 20:55 |
| *** mlavalle_ has joined #openstack-qa | 20:56 | |
| *** mlavalle has quit IRC | 20:57 | |
| *** mlavalle_ is now known as mlavalle | 20:57 | |
| openstackgerrit | David Kranz proposed a change to openstack/tempest: Move success response checking to the token client https://review.openstack.org/117621 | 20:59 |
| *** mlavalle has quit IRC | 21:00 | |
| *** mlavalle has joined #openstack-qa | 21:01 | |
| *** malini1 has joined #openstack-qa | 21:02 | |
| *** malini1 has quit IRC | 21:04 | |
| *** malini1 has joined #openstack-qa | 21:05 | |
| *** andreaf has joined #openstack-qa | 21:06 | |
| *** aysyd has quit IRC | 21:11 | |
| *** Sukhdev has joined #openstack-qa | 21:15 | |
| *** mlavalle has quit IRC | 21:16 | |
| *** dhellmann is now known as dhellmann_ | 21:17 | |
| *** mlavalle has joined #openstack-qa | 21:17 | |
| *** qba73 has quit IRC | 21:19 | |
| *** sdmonov has quit IRC | 21:19 | |
| *** sdmonov has joined #openstack-qa | 21:20 | |
| *** e0ne has quit IRC | 21:27 | |
| *** e0ne has joined #openstack-qa | 21:27 | |
| *** flwang_ has joined #openstack-qa | 21:28 | |
| *** dpaterson has quit IRC | 21:29 | |
| *** flwang_ has quit IRC | 21:29 | |
| *** andreaf has quit IRC | 21:37 | |
| *** dkranz has quit IRC | 21:37 | |
| *** andreaf has joined #openstack-qa | 21:37 | |
| *** andreaf has quit IRC | 21:43 | |
| *** adalbas has quit IRC | 21:46 | |
| *** mlavalle has quit IRC | 21:48 | |
| *** mlavalle has joined #openstack-qa | 21:49 | |
| *** dimtruck is now known as zz_dimtruck | 21:52 | |
| *** sdmonov has quit IRC | 21:53 | |
| *** zz_dimtruck is now known as dimtruck | 21:53 | |
| *** sdmonov has joined #openstack-qa | 21:55 | |
| *** e0ne has quit IRC | 21:56 | |
| *** mriedem has quit IRC | 21:57 | |
| *** e0ne has joined #openstack-qa | 22:00 | |
| *** gmann_ has joined #openstack-qa | 22:02 | |
| *** gmann_ has left #openstack-qa | 22:02 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack-infra/devstack-gate: Point TEMPEST_HTTP_IMAGE at git.openstack.org https://review.openstack.org/117637 | 22:03 |
| openstackgerrit | A change was merged to openstack-dev/devstack: scope warnings to only display if the services are enabled https://review.openstack.org/112725 | 22:04 |
| *** jgrimm is now known as zz_jgrimm | 22:06 | |
| *** e0ne has quit IRC | 22:09 | |
| *** tkammer has quit IRC | 22:10 | |
| mtreinish | sdague, jogo: any thoughts on?: https://github.com/mtreinish/tempest-lib I'm not sure I got the git history quite right (I might have added too many commits to the log) | 22:19 |
| mtreinish | I did the merges there before I wrote my script to do it | 22:19 |
| mtreinish | so the options weren't the same | 22:19 |
| *** dimtruck is now known as zz_dimtruck | 22:22 | |
| jogo | mtreinish: oh you did that fancy history copy thing | 22:24 |
| jogo | mtreinish: will have to look at it tomorrow | 22:25 |
| jogo | mtreinish: it would be easier to review what commands you ran then the result actually | 22:25 |
| mtreinish | jogo: ehh, I'd have to dig it up in my command history | 22:25 |
| mtreinish | I was following some guide to do the first 3 merges | 22:25 |
| mtreinish | but then I wrote a script based on dhellmann_'s oslo graduation script | 22:26 |
| mtreinish | which seemed fancier, and like he knew what he was doing | 22:26 |
| clarkb | mtreinish: jogo: depending on the layout of your git tree using git subtree is an easy way to split out a dir from one repo into another repo | 22:26 |
| jogo | clarkb: yeah, I am wondering what subtrees mtreinish spit out | 22:27 |
| mtreinish | jogo, clarkb: for the cli unit tests I used: http://paste.openstack.org/show/102018/ | 22:28 |
| mtreinish | on tempest | 22:29 |
| mtreinish | I'm still trying to dig up the commands I used for the other migrated files | 22:29 |
| jogo | why are cli tests in tempest-lib? | 22:29 |
| mtreinish | those are the unit tests for the cli test framework | 22:30 |
| mtreinish | the cli tests are in tempest/cli/simple_read_only | 22:30 |
| jogo | ohh cli framework | 22:30 |
| jogo | makes sense | 22:30 |
| jogo | overall it looks good | 22:32 |
| jogo | it may be worth porting one of the CLI tests to use your tempest_lib to make sure it works | 22:32 |
| mtreinish | jogo, clarkb: oh nm, I used the same command as http://git.openstack.org/cgit/openstack/oslo-incubator/tree/tools/graduate.sh for the other files I migrated | 22:32 |
| jogo | before makign this public | 22:32 |
| mtreinish | I forgot that this morning I hard coded a local copy of it for that first merge | 22:33 |
| mtreinish | jogo: yeah that's on my todo list for tomorrow | 22:33 |
| *** amotoki has joined #openstack-qa | 22:33 | |
| *** dkranz has joined #openstack-qa | 22:33 | |
| *** markmcclain has quit IRC | 22:34 | |
| *** morganfainberg_Z is now known as morganfainberg | 22:34 | |
| mtreinish | jogo: although I think if the history looks sane I should un-WIP: https://review.openstack.org/117640 | 22:35 |
| mtreinish | and migrate it over | 22:35 |
| jogo | mtreinish: history looks sane enough yeah | 22:35 |
| jogo | I would write a small thing that uses tempest_lib before un WIPing | 22:36 |
| jogo | but that is just me | 22:36 |
| mtreinish | ok, sure that makes sense | 22:37 |
| *** amotoki has quit IRC | 22:38 | |
| *** torandu has quit IRC | 22:47 | |
| *** malini2 has joined #openstack-qa | 22:56 | |
| *** torandu has joined #openstack-qa | 22:56 | |
| *** malini1 has quit IRC | 22:57 | |
| *** aimon has quit IRC | 22:59 | |
| *** malini2 has quit IRC | 23:00 | |
| *** rbak has quit IRC | 23:01 | |
| *** mriedem has joined #openstack-qa | 23:02 | |
| *** aimon has joined #openstack-qa | 23:05 | |
| *** atiwari has quit IRC | 23:09 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Make list of open reviews bold https://review.openstack.org/117588 | 23:12 |
| *** Sukhdev has quit IRC | 23:15 | |
| openstackgerrit | A change was merged to openstack-infra/elastic-recheck: Add query for bug 1362347 https://review.openstack.org/117358 | 23:17 |
| *** sdmonov has quit IRC | 23:18 | |
| openstackgerrit | Matthew Treinish proposed a change to openstack/tempest: Migrate cli test framework to tempest-lib https://review.openstack.org/117649 | 23:26 |
| *** topshare has joined #openstack-qa | 23:30 | |
| jgriffith | mtreinish: ping | 23:34 |
| mtreinish | jgriffith: pong | 23:35 |
| jgriffith | mtreinish: so I'm trying to figure out how to fix a little timing problem | 23:35 |
| jgriffith | mtreinish: https://github.com/openstack/tempest/blob/master/tempest/scenario/manager.py#L251 | 23:35 |
| jgriffith | mtreinish: so I ran into a case where the encryption test that is a sub-class of this | 23:36 |
| jgriffith | mtreinish: can fail to attach | 23:36 |
| jgriffith | mtreinish: the problem is we end up calling delete multiple times | 23:36 |
| jgriffith | mtreinish: hmm... wait, that can't be where it's coming from | 23:37 |
| mtreinish | yeah that code is just a bit too clever so it's hard to follow | 23:37 |
| jgriffith | mtreinish: indeed :) | 23:37 |
| mtreinish | it's exploiting that addCleanup is a LIFO | 23:37 |
| jgriffith | Ok... hmm, so I'll have to look elsewhere as to the cause of the issue | 23:38 |
| mtreinish | if wait for delete is enabled, you do end up doing an extra get on a nonexistant volume | 23:38 |
| mtreinish | jgriffith: do you have logs I can look at? | 23:38 |
| jgriffith | mtreinish: Looking for something meaningful to share now :) | 23:38 |
| jgriffith | mtreinish: there's a ton of stuff... | 23:39 |
| jgriffith | mtreinish: but what you see is that the volume was deleted out of the DB and on the backend | 23:39 |
| jgriffith | but new calls to get and delete follow up | 23:39 |
| jgriffith | mtreinish: looks like 3 additional calls to delete | 23:39 |
| jgriffith | mtreinish: doesn't necessarily hurt anything I don't think | 23:39 |
| mtreinish | hmm, but that is weird. Did you find the tempest log snippet making the extra requests? | 23:40 |
| jgriffith | mtreinish: ahh... let me look there | 23:40 |
| *** topshare has quit IRC | 23:42 | |
| jgriffith | mtreinish: actually no... just a pile of GET's and only the one delete | 23:43 |
| jgriffith | mtreinish: which means something's funky in Cinder... | 23:44 |
| jgriffith | mtreinish: hmm... ok, sorry for the distraction | 23:44 |
| mtreinish | jgriffith: no worries. Good luck | 23:44 |
| mtreinish | jgriffith: is this on a local run? | 23:44 |
| jgriffith | mtreinish: yeah, it's on my 3'rd party ci rig | 23:45 |
| jgriffith | mtreinish: been thrashing it internally before I publish | 23:45 |
| jgriffith | mtreinish: if I don't figure it out i'll point you to the web-server with my logs and enlist some help :) | 23:45 |
| mtreinish | jgriffith: heh, sure :) | 23:46 |
| jgriffith | :) | 23:46 |
| *** openstackgerrit has quit IRC | 23:46 | |
| *** jamielennox|away is now known as jamielennox | 23:47 | |
| *** openstackgerrit has joined #openstack-qa | 23:47 | |
| *** tosky has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!