| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-11-15 | |||
| 16:32:44 | mriedem | so yes to 1-4 | |
| 16:33:09 | efried | are you going to skip implementing --resource-class for now? | |
| 16:34:14 | mriedem | i would yes | |
| 16:34:38 | mriedem | unlike this irc conversation, i don't really want to boil the ocean on providing something simple to just remove a single provider from a set of allocations :) | |
| 16:35:18 | mriedem | this started as like a "this should be relatively simple" kind of thing for something useful | |
| 16:35:33 | mriedem | i appreciate forward-thinking, but i'm also close to just abandoning | |
| 16:37:10 | efried | "remove *multiple* providers from a set of allocations" yeah? | |
| 16:39:11 | mriedem | it started as one, | |
| 16:39:20 | mriedem | tetsuro and mel asked for multiple which was simple to accomodate | |
| 16:44:32 | efried | or so you thought | |
| 16:45:15 | mriedem | yeah this eric guy got involved | |
| 16:45:18 | mriedem | blew it all up! | |
| 16:46:42 | efried | You'll thank me later. | |
| 16:46:44 | efried | In therapy. | |
| 16:53:53 | mnaser | mordred: i hope you're resting well after your entertainment-less flight --- can you leave some comments on https://review.opendev.org/#/c/684740/ please? | |
| 16:54:05 | mnaser | we kinda discussed that briefly at ansiblefest and i think ansible-sig might be a better home | |
| 19:33:55 | johnsom | efried It seems Dean is ok with that osc-lib hack for the neutron docs. Any chance we can move that from WIP? | |
| 19:35:02 | efried | johnsom: which one? | |
| 19:35:46 | johnsom | efried lol, https://review.opendev.org/#/c/693267 | |
| 19:36:28 | efried | johnsom: oh, yeah, needs tests, thought you might be willing to, ahem, co-author :P | |
| 19:36:39 | johnsom | Um, no | |
| 19:37:18 | efried | okay then, I'll see what I can work up. | |
| 19:37:44 | johnsom | Ok, thank you | |
| 20:15:47 | openstackgerrit | Eric Fried proposed openstack/osc-lib master: Add enhance_help kwarg to tags option generators https://review.opendev.org/693267 | |
| 20:15:49 | efried | johnsom: ^ | |
| 20:15:51 | efried | dtroyer: ^ | |
| 20:58:18 | openstackgerrit | Nate Johnston proposed openstack/python-openstackclient master: Handle NotFoundException when listing floating IPs https://review.opendev.org/694571 | |
| 21:10:48 | johnsom | Thanks | |
| 22:07:48 | openstackgerrit | Ruby Loo proposed openstack/openstacksdk master: baremetal node: 'error' is a failed state https://review.opendev.org/694634 | |
| #openstack-sdks - 2019-11-17 | |||
| 10:47:51 | mordred | mnaser: replied. happy to make sig deliverable - is there a pointer somewhere to what I need to do to do that? | |
| #openstack-sdks - 2019-11-18 | |||
| 09:16:08 | Horrorcat | moin. we’re currently troubleshooting keystone-to-keystone federation not working with recent openstackclient versions. the problem manifests itself in that `openstack catalog list` shows the correct endpoints, but for example `openstack server list` claims that no compute endpoint exists in the default region | |
| 09:16:35 | Horrorcat | We reported this issue a while back https://storyboard.openstack.org/#!/story/2004016 and haven’t gotten any feedback. | |
| 09:17:17 | Horrorcat | I now found a super-hacky solution, which is adding `self._cli_options._auth = self.auth` after the `if self._cli_options.service_provider` block in `ClientManager.setup_auth` in osc_lib/clientmanager.py | |
| 09:17:51 | Horrorcat | this feels so wrong, but from the control flow it doesn’t appear to me that this can be solved in a different way. the setup_auth method essentially *has* to replace the auth plugin of the _cli_options (really a CloudRegion object) thing | |
| 09:17:59 | Horrorcat | because it detects k2k federation, which requires replacing the auth plugin. | |
| 09:33:35 | openstackgerrit | Daniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694484 | |
| 09:35:28 | Horrorcat | here’s the patch: https://paste.debian.net/hidden/224116b2/ | |
| 10:36:29 | openstackgerrit | Merged openstack/openstacksdk master: baremetal node: 'error' is a failed state https://review.opendev.org/694634 | |
| 11:53:25 | openstackgerrit | Merged openstack/openstacksdk master: Add router add/remove route operations https://review.opendev.org/674324 | |
| #openstack-sdks - 2019-11-19 | |||
| 15:41:56 | tobiash | mordred: am I right here for python-openstackclient issues too? | |
| 15:42:25 | tobiash | I just stumbled over a non working 'openstack image add project' with the current python-openstackclient combined with a recent osc-lib | |
| 15:42:53 | gtema | tobiash: yes, you are right here | |
| 15:43:23 | tobiash | it looks like there was an interface change in osc-lib that was not fully finished yet, see https://review.opendev.org/683118 and https://review.opendev.org/683119 | |
| 15:43:59 | tobiash | as far as I can see in order to make this work again, both changes need to be merged and need a release right? | |
| 15:44:00 | gtema | this might be, dtroyer was saying last week they hurry up with osc-lib stuff | |
| 15:44:18 | tobiash | ah ok, thanks for the info | |
| 15:44:27 | gtema | welcome | |
| 16:00:52 | openstackgerrit | Daniel Bengtsson proposed openstack/osc-lib master: Fix the requirement of keystoneauth1. https://review.opendev.org/695022 | |
| 17:25:02 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Create Volume v3 functional tests https://review.opendev.org/695044 | |
| 18:28:04 | openstackgerrit | Nate Johnston proposed openstack/python-openstackclient master: Handle NotFoundException when listing floating IPs https://review.opendev.org/694571 | |
| 19:48:26 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Copy Volume v2 commands and unit tests to v3 https://review.opendev.org/695069 | |
| 21:07:25 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673725 | |
| #openstack-sdks - 2019-11-20 | |||
| 01:11:53 | openstackgerrit | Merged openstack/python-openstackclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673725 | |
| 02:15:26 | openstackgerrit | wangfaxin proposed openstack/api-sig master: Fix wrong openstack review link . https://review.opendev.org/695126 | |
| 05:10:28 | eandersson | Wondering if we are hitting another openstacksdk related memory leak in Senlin | |
| 05:11:25 | eandersson | > OpenStackConfig 18234 | |
| 05:12:39 | eandersson | After running the senlin-health-manager for a few hours with objgraph enabled I see a ton of OpenStackConfig, HTTP pools etc. | |
| 05:18:59 | eandersson | Suspecting something like this again https://storyboard.openstack.org/#!/story/2005306 | |
| 08:31:39 | eandersson | I think the pattern used in Senlin is incorrect | |
| 08:32:24 | eandersson | or something is wrong with the openstack sdk | |
| 08:35:08 | eandersson | https://review.opendev.org/#/c/695139/4/senlin/profiles/base.py | |
| 08:35:12 | eandersson | This seems to fix the memory leak | |
| 09:34:56 | openstackgerrit | Daniel Bengtsson proposed openstack/osc-lib master: Fix the requirement of keystoneauth1. https://review.opendev.org/695022 | |
| 13:39:40 | openstackgerrit | Matt Riedemann proposed openstack/python-openstackclient master: Provide stderr in exception when check_parser fails https://review.opendev.org/682939 | |
| 17:04:15 | efried | dtantsur, mordred: is sdk set up for swift such that it would standard ksa conf opts through our shiny new oslo_config kwarg? | |
| 17:04:21 | efried | would accept* | |
| 17:05:24 | dtantsur | efried: it *seems* so. at least we're already using it in this manner in ironic/ironic-inspector. | |
| 17:05:53 | efried | okay, neat. And... we decided the conf section needs to be [swift], not [object-store], I think. | |
| 17:06:03 | efried | which is slightly sad-making | |
| 17:08:40 | dtantsur | yep, I think so | |
| 17:08:54 | dtantsur | it can be easily hacked around, lemme show | |
| 17:09:00 | efried | confirmed (openstack.config.cloud_region.from_conf) | |
| 17:09:06 | efried | nah, sokay. | |
| 17:09:20 | efried | Hacked around without changing code? (Now I'm curious) | |
| 17:09:39 | dtantsur | similar hack https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/inspector.py#L54-L57 | |
| 17:09:46 | dtantsur | no, with code | |
| 17:10:18 | efried | oh, yeah, if we wanted to support it in code, we would just do it in openstack.config.cloud_region.from_conf, the logic would be easy, look for a conf section by service type or project | |
| 17:10:41 | efried | but I don't really care to do that now. | |
| 17:36:28 | openstackgerrit | Merged openstack/python-openstackclient master: Provide stderr in exception when check_parser fails https://review.opendev.org/682939 | |
| 18:32:40 | eandersson | mordrerd | |
| 18:32:57 | eandersson | you here? | |
| 19:07:51 | eandersson | * mordred | |
| #openstack-sdks - 2019-11-21 | |||
| 13:21:07 | openstackgerrit | Daniel Bengtsson proposed openstack/python-openstackclient master: Stop testing python 2 in tox. https://review.opendev.org/695427 | |
| 14:06:05 | openstackgerrit | Daniel Bengtsson proposed openstack/python-openstackclient master: Stop testing python 2 in tox. https://review.opendev.org/695427 | |
| 14:07:31 | openstackgerrit | Daniel Bengtsson proposed openstack/python-openstackclient master: Stop testing python 2 in tox and zuul. https://review.opendev.org/695427 | |
| 15:44:32 | openstackgerrit | Merged openstack/api-sig master: Fix wrong openstack review link . https://review.opendev.org/695126 | |
| 16:00:34 | elmiko | API SIG office hour is now open! | |
| 16:44:25 | dtantsur | wow, lonely elmiko! | |
| 16:44:30 | elmiko | hehe | |
| 16:44:35 | dtantsur | hey-hey, how is life? | |
| 16:44:47 | elmiko | not bad, got an acceptance for devconf, so yay \o/ | |
| 16:44:57 | dtantsur | devconf.cz? | |
| 16:44:59 | elmiko | yeah! | |
| 16:45:11 | dtantsur | congrats :) (if you see travelling to Brno as a good thing) | |
| 16:45:20 | elmiko | i actually like brno | |
| 16:45:24 | dtantsur | heh | |
| 16:45:28 | dtantsur | I lived there for too long ;) | |
| 16:45:28 | elmiko | the travelling part not so much lol | |
| 16:45:31 | elmiko | fair | |
| 16:45:46 | elmiko | how's life for you? new apartment all settled? | |