| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-27 | |||
| 16:10:46 | mordred | yah - and we've since added support for telling if the server sent paginated results automatically, so the parameter isn't super useful anymore - we just haven't gone through and gotten rid of it yet | |
| 16:12:40 | tobias-urdin | ack, thanks | |
| 16:40:11 | larsks | What is the typical latency between "fix merging" and "fix shows up on pypi"? | |
| 16:41:23 | mordred | larsks: depends ... we need to tag a release for it to show up on pypi - which can usually happen pretty quickly. I'm fighting a broken laptop at the moment though and have been out for US Thanksgiving, so I'm a little behind on grokking the current state of things | |
| 16:41:43 | mordred | larsks: I'd imagine we could get something cut by the end of the week at least though | |
| 16:43:08 | larsks | No worries. I've got a trio of problems that have conspired to give me a set of "unsharable" ansible playbooks because they require fixes to openstacksdk, the ansible os_server plugin, and the openstack inventory plugin. I'm hoping to get to the point that I don't need to provide people with manual patching instructions, but it's not critical. | |
| 16:44:27 | mordred | larsks: I would also love for you to not need to give people manual patching instructions | |
| 16:44:28 | mordred | larsks: :) | |
| 16:44:43 | mordred | larsks: I think that inventory plugin patch looks backportable | |
| 16:45:10 | larsks | Hopefully also https://github.com/ansible/ansible/pull/48798 | |
| 16:48:22 | mordred | larsks: yah. that one too (and I have shipit'd that one too) | |
| 16:48:29 | larsks | \o/ | |
| 17:06:07 | openstackgerrit | Merged openstack/openstacksdk master: do not force interface=admin for identity api v3 https://review.openstack.org/619949 | |
| 17:26:14 | larsks | mordred: another openstacksdk review, but not critical (rarely used code path, I think; only tripped over it due to a misconfiguration): https://review.openstack.org/#/c/618519/ | |
| 17:36:18 | openstackgerrit | Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/614786 | |
| 17:37:21 | mordred | larsks: sweet. thanks | |
| 20:27:05 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Register proxy directly in add_service https://review.openstack.org/620398 | |
| #openstack-sdks - 2018-11-28 | |||
| 07:47:26 | openstackgerrit | Vieri proposed openstack/python-openstackclient master: add python 3.6 unit test job https://review.openstack.org/620511 | |
| 08:15:50 | dtantsur|mtg | mordred: morning! could you check https://review.openstack.org/#/c/619594/ as well? it may be a bit controversial, so ping me if you need to discuss :) | |
| 14:06:54 | mordred | dtantsur|mtg: haven't read the patch yet, but I agree with the commit message | |
| 14:07:01 | dtantsur|mtg | good start :) | |
| 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 | dtantsur|mtg | or I can do it here, but only for ironic for now (to avoid bloating the patch) | |
| 14:15:14 | gtema | dtantsur|msg: I also think a follow-up would be better | |
| 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: Slightly refactor vendor profile loading https://review.openstack.org/616227 | |
| 14:43:36 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Support remote vendor profiles https://review.openstack.org/616228 | |
| 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 | |