| *** calebb has quit IRC | 01:29 | |
| *** rods has quit IRC | 01:33 | |
| *** calebb has joined #openstack-shade | 01:38 | |
| *** gouthamr has quit IRC | 03:24 | |
| *** gkadam has joined #openstack-shade | 03:58 | |
| *** gkadam_ has joined #openstack-shade | 04:01 | |
| *** gkadam has quit IRC | 04:03 | |
| *** gkadam_ is now known as gkadam | 04:04 | |
| *** openstack has quit IRC | 04:42 | |
| *** openstack has joined #openstack-shade | 04:43 | |
| *** gkadam has quit IRC | 04:43 | |
| *** olaph has joined #openstack-shade | 04:43 | |
| *** pabelanger has joined #openstack-shade | 04:43 | |
| *** yolanda has joined #openstack-shade | 04:43 | |
| *** calebb has joined #openstack-shade | 04:44 | |
| *** spredzy has joined #openstack-shade | 04:44 | |
| *** lastmikoi has joined #openstack-shade | 04:44 | |
| *** purplerbot has joined #openstack-shade | 04:45 | |
| *** openstackstatus has joined #openstack-shade | 04:46 | |
| *** zigo has joined #openstack-shade | 04:46 | |
| *** ChanServ sets mode: +v openstackstatus | 04:46 | |
| *** jlk has joined #openstack-shade | 04:49 | |
| *** ianw has joined #openstack-shade | 04:49 | |
| *** jlk has quit IRC | 04:49 | |
| *** jlk has joined #openstack-shade | 04:49 | |
| *** rcarrill1 has joined #openstack-shade | 04:53 | |
| *** slaweq has joined #openstack-shade | 06:44 | |
| *** slaweq has quit IRC | 06:48 | |
| *** rods has joined #openstack-shade | 07:19 | |
| *** gkadam has joined #openstack-shade | 07:26 | |
| *** iogg has joined #openstack-shade | 07:50 | |
| *** rods has quit IRC | 08:26 | |
| *** iogg has quit IRC | 08:44 | |
| *** ioggstream has joined #openstack-shade | 08:44 | |
| *** rods has joined #openstack-shade | 09:02 | |
| *** gkadam is now known as gkadam-afk | 09:32 | |
| *** spredzy has quit IRC | 09:33 | |
| *** RuiChen has joined #openstack-shade | 09:39 | |
| *** spredzy has joined #openstack-shade | 09:42 | |
| *** rods has quit IRC | 11:04 | |
| *** ioggstream has quit IRC | 11:22 | |
| *** rods has joined #openstack-shade | 12:00 | |
| *** ioggstream has joined #openstack-shade | 12:06 | |
| *** openstackgerrit has joined #openstack-shade | 12:27 | |
| openstackgerrit | Samuel de Medeiros Queiroz proposed openstack-infra/shade master: Use new keystoneauth version discovery https://review.openstack.org/493582 | 12:27 |
|---|---|---|
| *** pleia2 has joined #openstack-shade | 12:42 | |
| openstackgerrit | Samuel de Medeiros Queiroz proposed openstack-infra/shade master: Use new keystoneauth version discovery https://review.openstack.org/493582 | 12:44 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Use new keystoneauth version discovery https://review.openstack.org/493582 | 12:47 |
| mordred | samueldmq: I also ^^ made the patch better yesterday but forgot to upload it :) | 12:48 |
| mordred | samueldmq: that passes pep8, py27 and py35 locally if I install those keystoneauth patches - let's see how it does with devstack | 12:48 |
| samueldmq | mordred: ah, great! | 12:49 |
| samueldmq | I am feeling I need to step back and review all the version discovery things again | 12:49 |
| samueldmq | :-) | 12:49 |
| cmurphy | only if you want your brain to bleed | 12:52 |
| samueldmq | that would be an attempt to fix the current bleeding | 12:53 |
| samueldmq | cmurphy: ^ :-) | 12:53 |
| openstackgerrit | Rosario Di Somma proposed openstack-infra/shade master: WIP: Use direct calls to get_<resource>_by_id https://review.openstack.org/494535 | 13:05 |
| rods | mordred ^^ stil a WIP but should pass existing unit tests. Added a couple of comments, I'd like your opinion when you have a sec. | 13:37 |
| openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Consume new mapping fields from service-type-authority https://review.openstack.org/493366 | 13:42 |
| openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Sync from latest service-types-authority https://review.openstack.org/494550 | 13:42 |
| mordred | rods: awesome! | 13:46 |
| mordred | rods: responded - and left a crazy idea for you :) | 13:59 |
| *** slaweq has joined #openstack-shade | 14:05 | |
| slaweq | mordred: hello | 14:05 |
| slaweq | mordred: can You click W+1 in https://review.openstack.org/#/c/494319/ and https://review.openstack.org/#/c/494321/ if both are fine for You? | 14:06 |
| slaweq | thx | 14:06 |
| mordred | slaweq: oh - yup. meant to do that yesterday :) | 14:08 |
| slaweq | mordred: thx | 14:08 |
| openstackgerrit | Merged openstack/os-service-types master: Consume new mapping fields from service-type-authority https://review.openstack.org/493366 | 14:14 |
| rods | mordred generating those methods is an interesting idea, will give us a reliable/consistent interface | 14:15 |
| rods | mordred Looking at all the list or search methods(but even many or the create and update) we have, looks like they are all almost the same. Mostly just calls to ..._client.(post|get|put) | 14:18 |
| mordred | rods: yah. it might even be easier if we wind up doing the sdk merge - since there is a consistent underlying object interface then that we can query (at least after my current set of patches) - so the shade layer can be mostly auto-generated with only the places that need special logic written out | 14:20 |
| rods | what if we move all those methods in specific Resource managers and use these managers in the OpenStackCloud.methods? | 14:20 |
| mordred | rods: yah - that's where I think an sdk merge actually might make things nicer and potentially even less complex for both sides | 14:21 |
| * mordred needs to publish his local patches relatd to that | 14:21 | |
| mordred | there's one more layer of abstraction they've got right now which I think needs to be removed because it's one layer too many - but digging in it's pretty close to being what we'd wind up with if we abstracted resource managers ourselves | 14:22 |
| openstackgerrit | Monty Taylor proposed openstack/os-service-types master: Sync from latest service-types-authority https://review.openstack.org/494550 | 14:26 |
| rods | mordred that's cool. Should we wait on the methods generation thing and put more effort on the sdk merge? I wonder if that code would stil be useful if we end up merging | 14:33 |
| rods | anything I can help with with the sdk merging? | 14:33 |
| mordred | well - I shold probably propose the governance patch and then publish my current work in progress | 14:34 |
| mordred | so - yeah - I think that's maybe a good idea | 14:34 |
| mordred | rods: for now the thing you have in this patch gets us far enough I think | 14:34 |
| rods | mordred ok, I'll tweak it a little and add some tests | 14:36 |
| mordred | rods: thanks! | 14:36 |
| openstackgerrit | Merged openstack/os-service-types master: Sync from latest service-types-authority https://review.openstack.org/494550 | 17:09 |
| *** SamYaple has joined #openstack-shade | 17:25 | |
| *** slaweq_ has joined #openstack-shade | 17:47 | |
| *** slaweq_ has quit IRC | 17:53 | |
| *** purplerbot has quit IRC | 18:06 | |
| *** purplerbot has joined #openstack-shade | 18:07 | |
| *** purplerbot has quit IRC | 18:09 | |
| *** ioggstream has quit IRC | 18:11 | |
| *** purplerbot has joined #openstack-shade | 18:12 | |
| *** rods has quit IRC | 18:45 | |
| *** rods has joined #openstack-shade | 18:51 | |
| *** ioggstream has joined #openstack-shade | 19:02 | |
| *** slaweq_ has joined #openstack-shade | 19:24 | |
| *** gkadam-afk has quit IRC | 19:35 | |
| *** calebb has quit IRC | 19:59 | |
| *** calebb has joined #openstack-shade | 20:01 | |
| slaweq_ | mordred: hello | 20:16 |
| slaweq_ | mordred: can I ask You a question? | 20:16 |
| slaweq_ | mordred: I'm checking little bit issue with test_list_volumes_pagination in gate-shade-functional-legacy (it failed for me at least twice today) | 20:17 |
| slaweq_ | is it some "known issue" or it just should be rechecked and will be fine? | 20:18 |
| mordred | slaweq_: yah - unfortunately sometimes the volume tests tend to timeout | 20:19 |
| mordred | slaweq_: we have not yet been able to come up with a full fix for it | 20:20 |
| slaweq_ | yes, so I looked to it little bit and I found that it's failing on cleaning after test | 20:20 |
| slaweq_ | not in test itself | 20:20 |
| mordred | yes - this is correct | 20:20 |
| mordred | unforunately - if we just ignore it - we wind up running out of quota - or disk space | 20:21 |
| slaweq_ | and what I found is that in test there is volume in status "deleting": http://logs.openstack.org/21/494321/1/check/gate-shade-functional-legacy/9ce964d/console.html#_2017-08-17_19_47_33_392189 | 20:21 |
| slaweq_ | but in cinder logs it looks that this volume was deleted sucessfully: http://logs.openstack.org/21/494321/1/check/gate-shade-functional-legacy/9ce964d/logs/screen-c-api.txt.gz#_2017-08-17_19_43_23_833 | 20:22 |
| slaweq_ | and cinder-volume log: http://logs.openstack.org/21/494321/1/check/gate-shade-functional-legacy/9ce964d/logs/screen-c-vol.txt.gz#_2017-08-17_19_44_33_380 | 20:23 |
| slaweq_ | mordred: so my question is, can it be some issue with caching in list_volumes() on shade's side? | 20:23 |
| slaweq_ | mordred: or it is some issue in cinder? | 20:23 |
| slaweq_ | do You know maybe? | 20:24 |
| mordred | slaweq_: I think it's a cinder issue - I know we've had issues in with nova data cache being stale before | 20:28 |
| slaweq_ | mordred: ok, I will try to read about cinder and such issue, thx | 20:29 |
| mordred | cool - thanks! oh - and yes, I think the fact that we can see in http://logs.openstack.org/21/494321//1/check/gate-shade-functional-legacy/9ce964d/console.html#_2017-08-17_19_47_33_392189 | 20:30 |
| mordred | (which you pasted a little earlier) | 20:30 |
| mordred | that's the REST response back from keystoneauth - so a remote call was made and that was cinder's answer - so we can rule out caching | 20:31 |
| mordred | at leats shade-side | 20:31 |
| *** ioggstream has quit IRC | 21:06 | |
| openstackgerrit | Slawek Kaplonski proposed openstack-infra/shade master: DNM Test force cinder volume delete https://review.openstack.org/494705 | 21:08 |
| mordred | ooh. neat | 21:09 |
| slaweq_ | mordred: I just pushed some small patch with force delete of volume added: https://developer.openstack.org/api-ref/block-storage/v3/index.html#volumes-volumes | 21:11 |
| mordred | slaweq_: yah - I like it :) | 21:11 |
| slaweq_ | I will try to recheck few times and check if maybe it will help | 21:11 |
| mordred | slaweq_: it'll be exciting if that solves that problem! | 21:11 |
| slaweq_ | if yes, I will add it with some tests | 21:11 |
| *** slaweq_ has quit IRC | 21:40 | |
| *** slaweq_ has joined #openstack-shade | 21:41 | |
| *** slaweq_ has quit IRC | 21:45 | |
| *** slaweq_ has joined #openstack-shade | 22:10 | |
| *** slaweq_ has quit IRC | 22:19 | |
| *** gouthamr has joined #openstack-shade | 22:25 | |
| *** openstackgerrit has quit IRC | 22:33 | |
| *** slaweq_ has joined #openstack-shade | 22:50 | |
| *** slaweq_ has quit IRC | 22:59 | |
| *** slaweq_ has joined #openstack-shade | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!