Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-03-14
16:57:15 elmiko edleafe: LOL yes!!!
16:57:53 gtema that's what API-SIG is doing, isn't it ;-)
16:57:59 edleafe gtema: yeah, let's re-invent distributed databases!
16:58:16 elmiko gtema: well, we provide guidance but we have always said that we are not the api police ;)
16:58:58 gtema and that's sad. Sometimes the police (or even S.W.A.T.) is necessary to bring things back
16:59:04 edleafe Although we might look pretty snazzy in our API Police uniforms!
16:59:05 elmiko haha
16:59:45 edleafe gtema: unfortunately, OpenStack has always been more about not hurting anyone's feelings than creating great software
17:00:19 gtema thanks
17:00:29 edleafe Well, the office hour is over, and I have to run out for a bit.
17:00:54 elmiko well, also, in the beginning there was no api guidance and to empower open collaboration and development you kinda have to let everyone in if you don't have standards ;)
17:00:59 elmiko later edleafe o/
17:01:28 gtema yeah
17:01:39 elmiko it's a tough issue
17:02:16 gtema I know. Closed-source is bad, but opensource has also problems
17:02:27 elmiko yup, no magic solutions
17:02:56 elmiko i'm gonna grab some lunch, take care o/
17:03:29 gtema yup. have a nice day
23:15:03 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Add "name" filter in "list" call when retrieving a single register https://review.openstack.org/637238
#openstack-sdks - 2019-03-15
04:21:21 openstackgerrit Rabi Mishra proposed openstack/keystoneauth master: Set Content-Type header explictly for LegacyJsonAdapter https://review.openstack.org/643505
13:22:01 openstackgerrit sean mooney proposed openstack/python-openstackclient master: add openstack server recreate command https://review.openstack.org/643578
13:42:42 openstackgerrit sean mooney proposed openstack/python-openstackclient master: add openstack server recreate command https://review.openstack.org/643578
14:47:22 gtema mordred: are you here?
14:50:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
14:53:28 mordred gtema: sort of - about to go on a call
14:53:53 gtema ah, ok. I would need a deeper help.
14:54:46 gtema I am basically writing unit tests for a connection.add_service and again found, that add_service causes additional authorization
14:55:37 gtema and while I was using keystonev2 - it was ok. But when decided to switch to v3 (since it gives possibility to select result catalog) - face this dramatically
14:56:07 gtema found_version = temp_adapter.get_api_major_version() (from service_description) - causes again authorization
15:01:48 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP Add unit tests for connection.add_service https://review.openstack.org/643604
15:43:22 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP Add unit tests for connection.add_service https://review.openstack.org/643604
15:52:47 openstackgerrit Merged openstack/python-openstackclient master: Fix: set invalid None project_id on range creation https://review.openstack.org/642707
15:52:52 openstackgerrit Merged openstack/python-openstackclient master: Fix service discovery in functional tests https://review.openstack.org/642074
16:44:44 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: implement the correct update of the maintenance_reason field https://review.openstack.org/643639
17:22:21 brtknr Does openstacksdk allow you to create a specific floating ip?
17:22:45 brtknr I looked at cloud.network.create_floating_ip() but it only creates random ones...
17:34:11 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: implement the correct update of the maintenance_reason field https://review.openstack.org/643639
17:50:08 mordred brtknr: it doesn't look like we have support for creating a specific floating ip anywhere. would be a welcome feature add
18:54:08 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: implement the correct update of the maintenance_reason field https://review.openstack.org/643639
18:55:17 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
19:10:02 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
19:40:24 openstackgerrit Merged openstack/openstacksdk master: Collect request stats https://review.openstack.org/614834
19:40:25 openstackgerrit Merged openstack/openstacksdk master: Use auth_url as identity endpoint when not project scoped https://review.openstack.org/643037
20:01:53 fried_rice mordred: couple more bugs flushed out as I've been playing.
20:03:45 mordred fried_rice: sweet - feel free to push up changes and I can put docs up later
20:04:03 mordred fried_rice: (I realized I didn't want to push up new patches while the nova patch was still in flight anyway)
20:04:20 fried_rice ack
20:04:39 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
20:04:49 fried_rice mordred: ^ nothing big
20:05:22 fried_rice wait what
20:05:43 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
20:06:07 fried_rice there
20:06:19 mordred fried_rice: yes! I agree with those changes :P)
20:06:47 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
20:06:55 mordred fried_rice: there it is updated with docs
20:07:02 mordred or, at least, what I've got so far
20:07:10 mordred I'll get some tests added in a bit
20:09:55 fried_rice nice
20:15:40 openstackgerrit Daniel Speichert proposed openstack/openstacksdk master: Deduplicate next-page URL's query params https://review.openstack.org/643675
20:53:40 openstackgerrit sean mooney proposed openstack/python-openstackclient master: add openstack server recreate command https://review.openstack.org/643578
21:54:33 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Add proxy API reference to baremetal user guide https://review.openstack.org/643683
#openstack-sdks - 2019-03-16
01:05:18 openstackgerrit Merged openstack/python-openstackclient master: Fix: incorrect check when no shared/private input https://review.openstack.org/642708
#openstack-sdks - 2019-03-17
00:08:49 openstackgerrit Merged openstack/openstacksdk master: Add proxy API reference to baremetal user guide https://review.openstack.org/643683
18:38:29 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
18:40:47 efried mordred: Updated ^ (please refresh if you had local edits)
20:20:42 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
20:20:55 efried mordred: and again (added oslo.config to requirements)
22:09:48 adriant mordred: I guess at some point I should add Adjutant to the SDK now that I have a reasonable service type
23:42:25 mordred efried: sweet. left a comment about adding oslo.config.
23:42:27 mordred adriant: yes!
23:50:19 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
23:50:27 mordred efried: ^^ took a stab at it
#openstack-sdks - 2019-03-18
00:22:35 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
00:22:39 mordred I'm stupid
00:31:08 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.openstack.org/643601
10:47:44 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add unit tests for connection.add_service https://review.openstack.org/643604
11:03:33 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Create runtime descriptor for new service https://review.openstack.org/643918
12:37:39 gtema mordred, dtantsur, Shrews: I would like to ask you to have a look at https://review.openstack.org/#/c/642218/ I am currently not very happy how it feels and want to know your opinions
12:49:18 mordred gtema: I think it looks great!
12:50:15 gtema ok, I just dislike usage of stuff, which is not belonging to the object and is only present in runtime
12:50:53 gtema but if you guys are ok with that we can push it and do probably some followup
12:51:38 gtema thanks mordred
12:56:27 mordred gtema: well, what do you mean - maybe I'm missing the thing that's bothering you
12:56:52 mordred gtema: the mixin classes do make the things part of the object
12:57:38 gtema sure, and that is exactly the "problem" for me. It is ok when the class use stuff from Mixin, but I do no like when one Mixin use stuff from another Mixin
12:58:35 gtema so I am more a fan of a regular inheritance, which is clear from the object description
12:59:51 gtema something what I have done with _network_common. The most "clear" way would be if _compute, _network, _floating_ip would inherit from it, and not indirectly use stuff, which is added on a higher level
13:01:01 gtema this could have also solved initialization of mixins in the connection - how it is currently done is not nice
13:04:33 gtema ok, letme quickly remove WIP from the title
13:05:01 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Split OpenStackCloud into reasonable pieces https://review.openstack.org/642218
13:07:59 mordred gtema: ah - yes. I understand your concern there
13:08:45 gtema mordred: great
13:09:44 mordred gtema: I think we'll be fighting to balance a desire to organize stuff into files with the mixin mess just because python doesn't have a better way to organize implementation
13:10:00 gtema agreed
13:10:41 gtema ok, will continue "thinking" about that issue
13:11:05 mordred all that said - I was originally thinking that once we got things moved into the proxy objects that potentially most of the remaining methods could just turn in to very small wrapper methods - and we could just shove all of OpenStackCloud directly on to Connection
13:11:18 mordred but it would still be thousands of lines long even in the best case
13:12:40 gtema yes, that's clear. With this change you have only "basic" logic left in the OpenStackCloud and it can be further merged into Connection. But here it is exactly the most prominent that mixins access methods, which they do not really belong
13:12:59 mordred yeah
13:15:24 gtema mordred: any plans for SDK on PTG?

Earlier   Later