Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-11-15
16:24:11 efried I guess that could work.
16:24:46 mriedem i would prefer just a simple unset --provider
16:25:11 mriedem than dork with the existing set --allocation <gdi this is a mess like bdms> format thing
16:25:16 mriedem but that's just me
16:25:57 efried I won't die on a hill for it, but the reasons I don't like that are:
16:25:57 efried 1) it's asymmetrical
16:25:57 efried 2) it makes it impossible to narrow down to RC granularity in the future
16:26:29 mriedem i don't think it's impossible, you add a --resource-class option
16:26:39 mriedem if specified, unset the resource class for the given provider,
16:26:51 efried not if you can specify multiple providers
16:26:51 mriedem if no providers are specified, remove the resource class from all providers on which the consumer has allocations
16:27:25 mriedem 1. openstack resource provider allocation unset --provider A --provider B --resource-class VGPU consumer1 == remove VGPU allocatoins from providers A and B for consumer1
16:27:53 mriedem 2. openstack resource provider allocation unset --resource-class VGPU consumer1 == remove VGPU allocations from all providers on which consumer1 has allocations
16:28:13 mriedem 3. openstack resource provider allocation unset --provider A consumer1 == remove all allocations consumer1 has on provider A
16:28:23 mriedem doesn't seem impossible to me
16:28:52 efried and --resource-class is repeatable too?
16:28:56 mriedem sure
16:29:04 efried Okay. Then I just hate the asymmetry.
16:29:35 efried 4. openstack resource provider allocation unset consumer1 ? Same as `delete`?
16:31:26 mriedem yeah i guess
16:31:34 mriedem well,
16:31:52 mriedem if you make --provider use nargs=+ to start, then that's not possible as at least one provider is required,
16:32:03 mriedem which you could do until you add support for --resource-class
16:32:14 mriedem but that's a behavior change and would require a major version release probably
16:32:19 efried Yeah, but the code is nearly identical because you still have to deal with having specified all --provider Z
16:32:23 mriedem anyway, sure
16:32:25 efried which is effectively the same.
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.

Earlier   Later