| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-28 | |||
| 14:07:21 | mordred | dtantsur|mtg: in the shade layer we've been only working with detail records as well | |
| 14:07:29 | dtantsur|mtg | yeah | |
| 14:13:25 | mordred | dtantsur|mtg: if we go ahead and apply it as a pattern across the board, perhaps we merge your ListMixin into the base Resource and add a flag like has_detail or something? | |
| 14:14:02 | gtema | I would also vote for that, since it is quite broadly spreaded | |
| 14:14:32 | dtantsur|mtg | mordred: sounds good. I can do it, thought I'd prefer in a follow-up :) this thing blocks migrating metalsmith to openstacksdk. | |
| 14:15:14 | gtema | dtantsur|msg: I also think a follow-up would be better | |
| 14:15:14 | dtantsur|mtg | or I can do it here, but only for ironic for now (to avoid bloating the patch) | |
| 14:15:15 | mordred | oh - yeah - I just +Ad your patch ... more thinking out loud | |
| 14:15:27 | mordred | so yeah, follow up sounds great | |
| 14:15:40 | dtantsur|mtg | good :) | |
| 14:43:36 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support remote vendor profiles https://review.openstack.org/616228 | |
| 14:43:36 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Slightly refactor vendor profile loading https://review.openstack.org/616227 | |
| 14:48:48 | openstackgerrit | Merged openstack/openstacksdk master: Make timeouts in functional tests configurable https://review.openstack.org/619008 | |
| 14:57:02 | mordred | dtantsur|mtg: ^^ I updated 616227 from your comment about preferring a global variable | |
| 15:01:55 | dtantsur|mtg | cool, thnx | |
| 15:09:24 | mordred | gtema: comments and questions on https://review.openstack.org/#/c/618993 | |
| 15:10:00 | gtema | mordred: ok, thanks. Will look at it | |
| 15:11:47 | cdent | mordred: I recall you had a script that would parse the max (micro)versions out of the services in a catalog and produce a little report. Is that around somewhere and consumable? | |
| 15:15:12 | mordred | cdent: it's definitely around somewhere ... | |
| 15:15:18 | mordred | cdent: the question is ... WHERE | |
| 15:15:40 | cdent | i'm certain you can find it because you used all caps. that will give extra power. | |
| 15:16:28 | mordred | cdent: oh - right. I added it as a command to osc | |
| 15:16:32 | mordred | cdent: openstack versions show | |
| 15:16:43 | cdent | oh, nice | |
| 15:17:21 | mordred | even better than a script! | |
| 15:18:32 | cdent | totes | |
| 15:18:44 | cdent | those all caps were super powerful today | |
| 15:23:20 | mordred | SO POWERFUL | |
| 15:29:02 | frickler | oh, that's a nice one indeed. makes it pretty obvious that I should tackle moving neutron and designate away from using specific ports in their devstack deployment ;) | |
| 15:30:27 | mordred | frickler: our obsession with running services on specific ports is so strange to me | |
| 15:33:20 | frickler | mordred: well, the origin of that to me always was that you cannot bind two different processes to the same port. and after that inertia takes over | |
| 15:34:54 | mordred | ++ | |
| 15:37:38 | frickler | mordred: I just noticed that for the image service, it lists eight different versions instead of microversions. is that a bug or is glance doing weird stuff (again)? | |
| 15:39:26 | dtantsur|mtg | frickler: glance has many major versions looking like microversions IIRC | |
| 15:43:23 | openstackgerrit | Merged openstack/openstacksdk master: Change approach to detailed listings of baremetal resources https://review.openstack.org/619594 | |
| 15:44:10 | mordred | frickler: glance doing weird things | |
| 15:44:35 | mordred | frickler: they are not selectable - so you can't use them like microversions and say "hey, I see you have 2.1 and 2.2, I want to talk to 2.1" | |
| 15:44:50 | dtantsur|mtg | mordred: do you think you could release the SDK with the just merged patch? I'd give the metalsmith migration another try. | |
| 15:45:04 | mordred | dtantsur|mtg: sure thing | |
| 15:46:26 | mordred | dtantsur|mtg: remote: https://review.openstack.org/620639 Release 0.20.0 of openstacksdk | |
| 15:47:32 | dtantsur|mtg | thnx! | |
| 15:47:47 | mordred | larsks: ^^ release request I think interests you too | |
| 16:05:06 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043 | |
| 16:07:28 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for client-side rate limiting https://review.openstack.org/605043 | |
| 16:38:28 | openstackgerrit | Merged openstack/osc-lib master: Remove pypy job https://review.openstack.org/617336 | |
| 16:46:11 | larsks | mordred: ooo, nifty. Thanks! | |
| 17:39:54 | openstackgerrit | Merged openstack/openstacksdk master: Update link address for vendor support https://review.openstack.org/619830 | |
| 18:01:24 | nicolasochem__ | Hello, I would like to use the sdk to validate a heat stack update. It's possible with the CLI (--dry-run command). | |
| 18:01:32 | nicolasochem__ | This is now the CLI does it : https://github.com/openstack/python-heatclient/blob/master/heatclient/v1/stacks.py#L185-L189 | |
| 18:02:08 | nicolasochem__ | But I don't see this call anywhere in the openstacksdk source code. Am I missing anything, or is it missing from sdk altogether ? | |
| 18:02:38 | nicolasochem__ | Note that you can do a dry-run of stack creation with the sdk. My question is about stack update. | |
| 18:21:06 | mordred | nicolasochem__: doesn't look like it's in the sdk - but we should definitely add it | |
| 18:51:00 | nicolasochem__ | mordred: thank you. I have opened a ticket and I am happy to QA it if needed. https://storyboard.openstack.org/#!/story/2004450 | |
| 18:58:39 | mordred | awesome! thanks | |
| #openstack-sdks - 2018-11-29 | |||
| 07:54:08 | openstackgerrit | Merged openstack/python-openstackclient master: Updated the take_actions for unified limits https://review.openstack.org/612283 | |
| 09:19:01 | openstackgerrit | Merged openstack/openstacksdk master: Add missing seperator between words https://review.openstack.org/618704 | |
| 09:25:17 | openstackgerrit | Merged openstack/openstacksdk master: implement block-storage backup resource https://review.openstack.org/591445 | |
| 14:33:08 | openstackgerrit | Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139 | |
| 14:38:36 | gtema | oh god zuul is slow today - one change recheck is already since 5 hours open and one task still queued | |
| 15:57:17 | edleafe | API-SIG Office Hour begins at the top of the hour. I will be here, but also in another meeting, so don't everyone ask your questions at once | |
| 15:58:00 | elmiko | i'll be here too | |
| 15:58:04 | elmiko | but ask edleafe first XD | |
| 16:02:06 | cdent | the lack of formality with you two is why I had to stop coming to these things | |
| 16:02:08 | cdent | :D | |
| 16:03:14 | elmiko | fair | |
| 16:58:43 | elmiko | catch ya next week edleafe, if not sooner o/ | |
| 16:59:07 | edleafe | elmiko: Leaving a minute early? Where's your dedication, man?? | |
| 16:59:25 | elmiko | haha | |
| 18:55:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990 | |
| 19:05:34 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990 | |
| 19:31:47 | openstackgerrit | Merged openstack/openstacksdk master: Add CRUD methods for Neutron Port Forwarding https://review.openstack.org/616710 | |
| 20:25:08 | mriedem | just wanted to come by and say https://docs.openstack.org/keystoneauth/latest/using-sessions.html is nice | |
| 20:25:09 | mriedem | thanks | |
| 20:28:32 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Strawman stats abstraction https://review.openstack.org/620990 | |
| 20:32:30 | mordred | mriedem: \o/ | |
| 21:13:25 | openstackgerrit | Merged openstack/openstacksdk master: Check result of server metadata operation https://review.openstack.org/619478 | |
| #openstack-sdks - 2018-11-30 | |||
| 06:52:08 | openstackgerrit | Rabi Mishra proposed openstack/osc-lib master: Fix formatter handling for python 3.7 https://review.openstack.org/618137 | |
| 07:13:53 | openstackgerrit | Filippo Inzaghi proposed openstack/python-openstackclient master: Add Python 3.6 classifier to setup.cfg https://review.openstack.org/621096 | |
| 07:36:08 | openstackgerrit | Radoslaw Smigielski proposed openstack/python-openstackclient master: Fix --limit option in image list sub-command https://review.openstack.org/617139 | |
| 08:26:15 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993 | |
| 08:27:48 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Refactor tag support https://review.openstack.org/618993 | |
| 09:23:03 | gtema | dtantsur|msg, are you here? | |
| 09:24:34 | dtantsur|mtg | gtema: I am | |
| 09:25:09 | gtema | dtantsur|msg: as a continue to removal of xxxDetails I have found similar issue in orchestrate | |
| 09:25:19 | gtema | but the difference is that it is not list, but create/update | |
| 09:25:30 | dtantsur|mtg | I think all Details are broken more or less | |
| 09:25:36 | gtema | basically StackPreview | |
| 09:26:14 | gtema | I feel it would be good to apply the same approach as for Details here, but this would mean extension for also other request types | |
| 09:26:30 | gtema | and extending basic proxy/resource/test_proxy | |
| 09:26:43 | gtema | this is however becoming relatively big | |
| 09:26:55 | gtema | I mean everywhere adding optional base_path | |
| 09:27:08 | gtema | wanted to ask, what is your opinion here? | |
| 09:28:04 | dtantsur|mtg | I need to think about it, I'm quite sleepy right now :) and I'm not familiar with orchestration enough | |
| 09:28:54 | gtema | it's not about orchestration itself. I am asking more about adding base_path optional parameter not only for "list", what you did for Details, but also for other request types | |
| 09:30:00 | gtema | use case: upon some parameter to proxy (i.e. preview) the endpoint for "create" is different | |
| 09:44:28 | dtantsur|mtg | gtema: this can be done in _prepare_request (used by everything except for list) | |
| 09:45:12 | gtema | sure, this is what I have done. But you would first need to also extend Proxy._create (and _update) and pass this optional parameter into Resource | |
| 09:45:46 | gtema | and this mean that test_proxy_base should be also extended with additional check of base_path for such calls | |
| 09:46:35 | gtema | that's what I ment "is getting big" | |
| 09:47:29 | gtema | and wanted to check before I land a patch not to face resistence | |