| *** gildub_ has joined #openstack-sdks | 00:09 | |
| *** gildub has quit IRC | 00:10 | |
| *** edleafe has quit IRC | 00:13 | |
| *** bobh has joined #openstack-sdks | 00:38 | |
| *** slaweq has joined #openstack-sdks | 01:11 | |
| *** bobh has quit IRC | 01:15 | |
| *** slaweq has quit IRC | 01:15 | |
| *** bobh has joined #openstack-sdks | 01:16 | |
| *** bobh has quit IRC | 01:34 | |
| *** ignatenkobrain has quit IRC | 01:37 | |
| *** ignatenkobrain has joined #openstack-sdks | 01:43 | |
| openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: [WIP]Add registered limit CRUD support https://review.openstack.org/609572 | 02:00 |
|---|---|---|
| *** slaweq has joined #openstack-sdks | 02:11 | |
| *** slaweq has quit IRC | 02:16 | |
| *** dave-mccowan has quit IRC | 02:21 | |
| *** bobh has joined #openstack-sdks | 02:22 | |
| *** dave-mccowan has joined #openstack-sdks | 02:23 | |
| *** bobh has quit IRC | 02:30 | |
| *** lbragstad has quit IRC | 02:40 | |
| *** chenyb4 has joined #openstack-sdks | 02:59 | |
| *** slaweq has joined #openstack-sdks | 03:11 | |
| *** slaweq has quit IRC | 03:15 | |
| *** dave-mccowan has quit IRC | 03:16 | |
| *** slaweq has joined #openstack-sdks | 05:11 | |
| *** slaweq has quit IRC | 05:16 | |
| *** olivierb has joined #openstack-sdks | 05:21 | |
| *** chenyb4 has quit IRC | 05:27 | |
| *** chenyb4 has joined #openstack-sdks | 06:01 | |
| *** e0ne has joined #openstack-sdks | 06:02 | |
| *** olivierb has quit IRC | 06:11 | |
| *** olivierb has joined #openstack-sdks | 06:12 | |
| *** dims has quit IRC | 06:29 | |
| *** e0ne has quit IRC | 06:29 | |
| *** slaweq has joined #openstack-sdks | 06:29 | |
| *** dims has joined #openstack-sdks | 06:33 | |
| *** slaweq has quit IRC | 06:34 | |
| *** dims has quit IRC | 06:38 | |
| *** dims has joined #openstack-sdks | 06:39 | |
| *** slaweq has joined #openstack-sdks | 06:41 | |
| *** slaweq has quit IRC | 06:46 | |
| *** slaweq has joined #openstack-sdks | 06:50 | |
| *** Luzi has joined #openstack-sdks | 07:00 | |
| *** slaweq has quit IRC | 07:03 | |
| *** slaweq has joined #openstack-sdks | 07:05 | |
| openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add registered limit CRUD support https://review.openstack.org/609572 | 07:14 |
| *** ttsiouts has joined #openstack-sdks | 07:15 | |
| *** ttsiouts has quit IRC | 07:17 | |
| openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add registered limit CRUD support https://review.openstack.org/609572 | 07:32 |
| openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add limit CRUD support https://review.openstack.org/609604 | 07:32 |
| *** ttsiouts has joined #openstack-sdks | 07:38 | |
| *** ttsiouts has quit IRC | 07:43 | |
| *** jpich has joined #openstack-sdks | 07:59 | |
| *** ttsiouts has joined #openstack-sdks | 08:02 | |
| *** ttsiouts has quit IRC | 08:07 | |
| *** ttsiouts has joined #openstack-sdks | 08:13 | |
| *** ttsiouts has quit IRC | 08:28 | |
| *** ttsiouts has joined #openstack-sdks | 08:34 | |
| *** dtantsur|afk is now known as dtantsur| | 08:44 | |
| *** dtantsur| is now known as dtantsur | 08:44 | |
| *** e0ne has joined #openstack-sdks | 09:05 | |
| dtantsur | API-SIG office hour here and now :) | 09:05 |
| *** ttsiouts has quit IRC | 09:05 | |
| *** tosky has joined #openstack-sdks | 09:06 | |
| openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add registered limit CRUD support https://review.openstack.org/609572 | 09:22 |
| openstackgerrit | wangxiyuan proposed openstack/openstacksdk master: Add limit CRUD support https://review.openstack.org/609604 | 09:22 |
| *** gkadam has quit IRC | 09:25 | |
| *** e0ne has quit IRC | 09:27 | |
| *** ttsiouts has joined #openstack-sdks | 09:33 | |
| *** e0ne has joined #openstack-sdks | 09:43 | |
| *** imacdonn has quit IRC | 09:52 | |
| *** imacdonn has joined #openstack-sdks | 09:53 | |
| *** gouthamr has quit IRC | 10:14 | |
| *** dmellado has quit IRC | 10:17 | |
| *** chenyb4 has quit IRC | 10:21 | |
| *** e0ne has quit IRC | 10:35 | |
| *** e0ne has joined #openstack-sdks | 10:38 | |
| *** ttsiouts has quit IRC | 10:58 | |
| *** e0ne has quit IRC | 11:04 | |
| *** dtantsur is now known as dtantsur|brb | 11:21 | |
| *** ttsiouts has joined #openstack-sdks | 11:25 | |
| *** e0ne has joined #openstack-sdks | 11:51 | |
| *** gildub_ has quit IRC | 11:59 | |
| *** jpena|off has joined #openstack-sdks | 12:02 | |
| *** gouthamr has joined #openstack-sdks | 12:05 | |
| *** dave-mccowan has joined #openstack-sdks | 12:06 | |
| *** dmellado has joined #openstack-sdks | 12:10 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | 12:15 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Move image methods to sdk image proxy https://review.openstack.org/609684 | 12:15 |
| mordred | samueldmq, dtantsur|brb: ^^ that last patch is definitely non-functional - but I wanted to push it up to see what you thought about the general idea | 12:16 |
| *** dave-mccowan has quit IRC | 12:19 | |
| *** mriedem has joined #openstack-sdks | 12:21 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | 12:54 |
| *** mriedem is now known as mriedem_afk | 12:54 | |
| samueldmq | mordred: I like it | 12:55 |
| samueldmq | mordred: the code in opesntackcloud will be much cleaner, and if you want to look at the specific craziness for each operation/version go to look at the proxies | 12:56 |
| *** edleafe has joined #openstack-sdks | 12:57 | |
| samueldmq | mordred: I was afraid we'd be creating a lot of files just for a little amount of code. multiple _proxy files... but the proxy files already exist today anyways | 12:57 |
| mordred | samueldmq: sweet - and yeah, they're already there ... hopefully this can make people less afraid of the size of that openstackcloud file :) | 12:59 |
| samueldmq | mordred: exactly, only the function contracts will be kept there | 13:00 |
| samueldmq | 12.5k loc in a single file is not human-friendly | 13:01 |
| mordred | samueldmq: it does make it easy to search for something from inside vim :) | 13:02 |
| samueldmq | that overflows my brain several times when I scroll it down | 13:02 |
| samueldmq | mordred: ++ | 13:02 |
| *** ttsiouts has quit IRC | 13:02 | |
| *** bobh has joined #openstack-sdks | 13:05 | |
| *** ttsiouts has joined #openstack-sdks | 13:05 | |
| mordred | Shrews: I know you love waking up to huge patches from me | 13:10 |
| mordred | Shrews: but also ... ^^ 609684- totally unfunctional - but you might have an opinion on the general idea | 13:10 |
| *** e0ne has quit IRC | 13:11 | |
| *** dims has quit IRC | 13:12 | |
| Shrews | oh, i've got opinions for ya.... | 13:13 |
| *** dims has joined #openstack-sdks | 13:14 | |
| *** dtantsur|brb is now known as dtantsur | 13:15 | |
| dtantsur | mordred: ack, I'll try to get to it soon-ish | 13:16 |
| dtantsur | mordred and all: the bifrost job was fixed, feel free to recheck things | 13:17 |
| mordred | woot! | 13:18 |
| *** dims has quit IRC | 13:19 | |
| *** e0ne has joined #openstack-sdks | 13:21 | |
| *** lbragstad has joined #openstack-sdks | 13:31 | |
| ml | openstackcloud raises Exception from openstack.cloud.exc, it doesn't use anything from openstack.exception. | 13:41 |
| ml | Why is that? | 13:41 |
| mordred | ml: it's just historical - we haven't finished merging all the things yet | 13:44 |
| mordred | ml: geting the openstackcloud code to raise exceptions from openstack.exception instead would be great | 13:45 |
| ml | Can I start using them in my change already? or is it desired to be handled as one change? | 13:46 |
| ml | probably a bad idea to keep both imports at the same time, because one can't import it as "exc" | 13:48 |
| mordred | ml: we could just import it as exceptions ... I think either way you want to go is fine | 13:50 |
| ml | ok, cool | 13:50 |
| *** mriedem_afk is now known as mriedem | 13:56 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | 14:06 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Move image methods to sdk image proxy https://review.openstack.org/609684 | 14:07 |
| *** dims has joined #openstack-sdks | 14:22 | |
| *** knikolla has joined #openstack-sdks | 14:23 | |
| dtantsur | mordred: is it okay that we ignore the error_message argument in https://github.com/openstack/openstacksdk/blob/master/openstack/_adapter.py#L129 ? | 14:25 |
| dtantsur | it's my third (?) attempt to understand error handling in https://review.openstack.org/#/c/604645/ and I'm still confused... | 14:26 |
| *** ttsiouts has quit IRC | 14:26 | |
| *** e0ne has quit IRC | 14:33 | |
| openstackgerrit | Matthias Lisin proposed openstack/openstacksdk master: openstackcloud.py: Implement FWaaS wrapper methods. https://review.openstack.org/604811 | 14:37 |
| mordred | dtantsur: probably not - and you're probably confused because it's probably broken | 14:38 |
| mordred | dtantsur: lemme see if I can wrap my head around it | 14:39 |
| dtantsur | cool. your head has much higher chances of figuring it out than mine :) | 14:39 |
| Shrews | mordred: overall direction on 609684 looks pretty good to me. Maybe make BaseImageProxy abstract for clarity and to enforce _upload_image() is declared in other classes | 14:39 |
| mordred | Shrews: yah - I tried that and hit "TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases" | 14:40 |
| Shrews | O.o | 14:40 |
| mordred | Shrews: and at that point I just gav up and decided to farm goats | 14:40 |
| Shrews | oh, maybe because it derives from Proxy | 14:41 |
| Shrews | poo | 14:41 |
| mordred | Shrews: although - I keep thinking that maybe that proxy metaclass is a bad idea | 14:42 |
| *** e0ne has joined #openstack-sdks | 14:42 | |
| *** dims has quit IRC | 14:43 | |
| Shrews | mordred: you could make BaseImageProxy more of an interface (call it BaseImageInterface), and make v1._proxy.Proxy derive from that *and* proxy.Proxy | 14:44 |
| *** Luzi has quit IRC | 14:47 | |
| Shrews | there are other ways around it, but I'm meh about it | 14:47 |
| *** dims has joined #openstack-sdks | 14:48 | |
| *** ttsiouts has joined #openstack-sdks | 14:50 | |
| dtantsur | we need some metaclasses for metaclasses | 14:51 |
| * dtantsur ducks | 14:51 | |
| *** openstackgerrit has quit IRC | 14:58 | |
| *** openstackgerrit has joined #openstack-sdks | 15:00 | |
| *** mriedem has quit IRC | 15:03 | |
| *** lbragstad has quit IRC | 15:04 | |
| *** lbragstad has joined #openstack-sdks | 15:05 | |
| *** e0ne has quit IRC | 15:09 | |
| smcginnis | dtantsur: Wouldn't that be metametaclasses? | 15:12 |
| dtantsur | awesome, yeah? | 15:12 |
| *** lbragstad has quit IRC | 15:18 | |
| *** lbragstad has joined #openstack-sdks | 15:19 | |
| *** e0ne has joined #openstack-sdks | 15:20 | |
| *** e0ne has quit IRC | 15:20 | |
| *** openstackgerrit has quit IRC | 15:22 | |
| *** mugsie has joined #openstack-sdks | 15:24 | |
| *** mriedem has joined #openstack-sdks | 15:32 | |
| *** ttsiouts has quit IRC | 15:34 | |
| *** ttsiouts has joined #openstack-sdks | 15:34 | |
| *** ttsiouts has quit IRC | 15:39 | |
| *** openstackgerrit has joined #openstack-sdks | 15:41 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rearrange shade image code https://review.openstack.org/609683 | 15:41 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Move image methods to sdk image proxy https://review.openstack.org/609684 | 15:41 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Revert the Proxy metaclass https://review.openstack.org/609747 | 15:41 |
| mordred | Shrews, dtantsur: ^^ how bout them metaapples? | 15:41 |
| dtantsur | heh | 15:45 |
| * dtantsur has to run soon, will check tomorrow | 15:45 | |
| mordred | dtantsur: maybe by then I will have figured out the error message scenario | 15:49 |
| elmiko | api sig office hours, HOOOoooooo | 16:00 |
| *** dtantsur is now known as dtantsur|afk | 16:00 | |
| elmiko | lol, i need to screenshot that | 16:00 |
| edleafe | Office hour for API-SIG starts now | 16:00 |
| edleafe | Try not to flood the channel with all your inquiries | 16:01 |
| elmiko | ++ | 16:02 |
| edleafe | hey, pipe down, elmiko! | 16:02 |
| mordred | edleafe: are we there yet? | 16:06 |
| edleafe | mordred: I swear, if you don't settle down, I'm turning this car around and going back home!! | 16:07 |
| mordred | edleafe: but but but | 16:08 |
| elmiko | edleafe: lol | 16:13 |
| elmiko | edleafe: are we there yet? | 16:13 |
| edleafe | elmiko: I'm going to count to 3... | 16:14 |
| * elmiko hides in back seat | 16:14 | |
| mordred | edleafe: elmiko keeps hitting me! | 16:17 |
| * edleafe sees that these office hours are going to be super-productive! | 16:17 | |
| mordred | if people are bored in office hours and want to review some fairly insane python patches - I've got plenty I can point you at ... | 16:18 |
| elmiko | best office hours ever! | 16:18 |
| *** gouthamr has quit IRC | 16:34 | |
| *** dmellado has quit IRC | 16:34 | |
| *** jpich has quit IRC | 16:38 | |
| *** mriedem is now known as mriedem_afk | 16:43 | |
| edleafe | Well, that was underwhelming. :) | 17:02 |
| *** olivierb has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/openstacksdk master: Correct updating baremetal nodes by name or ID https://review.openstack.org/608674 | 17:29 |
| *** mriedem_afk is now known as mriedem | 18:09 | |
| elmiko | edleafe: lol | 18:14 |
| *** openstackgerrit has quit IRC | 18:20 | |
| *** openstackgerrit has joined #openstack-sdks | 18:21 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Handle not having cinderclient.v1 available https://review.openstack.org/609473 | 18:21 |
| *** gouthamr_ has joined #openstack-sdks | 18:25 | |
| *** dmellado has joined #openstack-sdks | 18:27 | |
| *** openstackstatus has quit IRC | 18:28 | |
| *** openstackstatus has joined #openstack-sdks | 18:30 | |
| *** ChanServ sets mode: +v openstackstatus | 18:30 | |
| *** dave-mccowan has joined #openstack-sdks | 18:58 | |
| *** dave-mccowan has quit IRC | 19:03 | |
| *** dave-mccowan has joined #openstack-sdks | 19:04 | |
| openstackgerrit | Merged openstack/openstacksdk master: Use discovery instead of config to create proxies https://review.openstack.org/600292 | 20:02 |
| openstackgerrit | Merged openstack/openstacksdk master: Remove api version default values https://review.openstack.org/600330 | 20:16 |
| *** tosky__ has joined #openstack-sdks | 20:16 | |
| *** tosky has quit IRC | 20:17 | |
| *** tosky__ is now known as tosky | 20:23 | |
| *** gildub_ has joined #openstack-sdks | 20:46 | |
| openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Handle not having cinderclient.v1 available https://review.openstack.org/609473 | 21:02 |
| *** bobh has quit IRC | 21:05 | |
| *** bobh has joined #openstack-sdks | 21:07 | |
| *** bobh has quit IRC | 21:11 | |
| *** dave-mccowan has quit IRC | 21:28 | |
| *** dave-mccowan has joined #openstack-sdks | 21:30 | |
| *** gildub_ has quit IRC | 21:39 | |
| *** bobh has joined #openstack-sdks | 21:40 | |
| *** bobh has quit IRC | 21:44 | |
| *** bobh has joined #openstack-sdks | 21:49 | |
| *** gouthamr_ is now known as gouthamr | 21:53 | |
| *** gildub_ has joined #openstack-sdks | 21:57 | |
| *** bobh has quit IRC | 22:12 | |
| *** openstackgerrit has quit IRC | 22:19 | |
| *** openstackgerrit has joined #openstack-sdks | 22:21 | |
| *** mriedem is now known as mriedem_gone | 22:23 | |
| *** mriedem_gone has quit IRC | 22:24 | |
| *** openstackstatus has quit IRC | 22:28 | |
| *** openstackstatus has joined #openstack-sdks | 22:29 | |
| *** ChanServ sets mode: +v openstackstatus | 22:29 | |
| *** tosky has quit IRC | 23:15 | |
| *** lbragstad has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!