Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-06-08
15:19:11 slaweq dtroyer: hi, can You maybe take a look at https://review.openstack.org/#/c/567620/ - I answered to Your comment there and I would like to know what You think about it
15:49:03 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Optimize _prep_server_detail to avoid redundant find_resource https://review.openstack.org/573760
16:22:07 dtantsur mordred: back to the topic of retries, ironic and rate-limiting taskmanager. can we implement the taskmanager so that it takes over the retries parameters and implements them instead of passing into Session?
16:26:07 mordred dtantsur: potentially - or I think what I'd suggest is implementing it in the adapter layer so that the adapter is submitting the retries into the taskmanager
16:26:28 mordred which would make sure that a retry doesn't double-hit an api you're rate limiting against
16:26:56 mordred dtantsur: how about I get the complex task manager at least proposed into the sdk today so we've got a sane thing to look at
16:27:01 dtantsur mordred: I guess my question boils down to whether we'll be able to simulate the same interface (session.do_something(status_code_retries=5)) even with rate limiting
16:27:07 dtantsur +++
16:27:30 dtantsur another random thought: wdyt about wiring things like baremetal_status_code_retries in the configuration? so that users can adjust it?
16:28:01 dtantsur similar to how you wire in default microversions
16:30:11 mordred dtantsur: yes, absolute can do that same interface - that's how it works now at the adapter layer
16:31:12 mordred dtantsur: yes. I think that's a great idea (wiring retries) - I don't think we should make the codes themselves configurable (since those are pretty specific per-service) - but retry count and delay seem like reasonable end-user tunables, yeah?
16:31:43 dtantsur yep (we cannot tweak delays, they're just exponential, but we can tweak retries)
16:50:43 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Allow configuring status_code_retries and connect_retries via cloud config https://review.openstack.org/573780
16:50:45 dtantsur mordred: something like this ^^^ won't pass the CI until keystoneauth is released
18:46:08 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Fix server show for microversion 2.47 https://review.openstack.org/573813
19:52:48 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Fix server show for microversion 2.47 https://review.openstack.org/573813
20:40:35 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Mention 2.51 and beautify events output for openstack server event show https://review.openstack.org/573830
21:11:34 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Mention compute API 2.50 in openstack quota show --class https://review.openstack.org/573834
21:19:58 mriedem on these last 2 changes ^ for some reason when i build the docs, the updated command help isn't showing up in the docs
21:20:03 mriedem via tox -e docs
21:20:12 mriedem maybe how the docs job works for publications is different?
21:23:47 mriedem i guess i still can't add a 2nd task to https://storyboard.openstack.org/#!/story/1751104 - same 500 timeout
#openstack-sdks - 2018-06-09
02:41:43 openstackgerrit kaka proposed openstack/microversion-parse master: fix tox python3 overrides https://review.openstack.org/573880
04:24:09 openstackgerrit Merged openstack/keystoneauth master: fix tox python3 overrides https://review.openstack.org/572906
11:17:08 openstackgerrit Alessandro Nesta proposed openstack/os-service-types master: fix tox python3 overrides https://review.openstack.org/573950
13:16:25 openstackgerrit Alessandro Nesta proposed openstack/python-openstackclient master: fix tox python3 overrides https://review.openstack.org/573975
13:26:54 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Mention 2.51 in help for openstack server event show https://review.openstack.org/573830
13:32:18 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Mention compute API 2.50 in openstack quota show --class https://review.openstack.org/573834
14:05:49 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Honor service-type aliases in config https://review.openstack.org/566929
14:15:25 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
14:15:36 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049
14:15:53 mordred dtantsur|afk: ^^ I have hopefully addressed your concerns - and thanks!
14:31:26 openstackgerrit Mohammed Naser proposed openstack/python-openstackclient master: Fix incorrect functio name for listing role inferences https://review.openstack.org/573990
14:31:38 openstackgerrit Mohammed Naser proposed openstack/python-openstackclient master: Fix incorrect function name for listing role inferences https://review.openstack.org/573990
14:51:29 openstackgerrit Mohammed Naser proposed openstack/python-openstackclient master: Switch to InferenceRuleManager for inferred roles https://review.openstack.org/573997
17:11:26 openstackgerrit Merged openstack/python-openstackclient master: Add system role functionality https://review.openstack.org/524416
17:11:28 openstackgerrit Merged openstack/python-openstackclient master: Update role document to include system parameter https://review.openstack.org/573303
17:11:29 openstackgerrit Merged openstack/python-openstackclient master: Fix urls in README.rst https://review.openstack.org/573140
17:11:30 openstackgerrit Merged openstack/python-openstackclient master: Add bgp commands to neutron decoder https://review.openstack.org/559742
17:38:59 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Support filtering port with IP address substring https://review.openstack.org/532324
18:16:44 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Fix RuntimeError when showing project which has extra properties https://review.openstack.org/530207
18:43:45 openstackgerrit Merged openstack/python-openstackclient master: Remove deprecated ip floating commands https://review.openstack.org/546924
#openstack-sdks - 2018-06-10
04:35:24 openstackgerrit Alessandro Nesta proposed openstack/python-openstackclient master: fix tox python3 overrides https://review.openstack.org/573975
#openstack-sdks - 2018-06-11
09:10:30 kimamisa Hi all. Just doing some advertisement for change, which now includes some functionnal tests ! https://review.openstack.org/#/c/557359/
09:10:42 kimamisa *my change
14:55:38 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Import rate limiting TaskManager from nodepool https://review.openstack.org/574285
15:02:03 mordred dtantsur: ^^
15:02:37 dtantsur nice! will check after the current meeting
15:22:18 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Import rate limiting TaskManager from nodepool https://review.openstack.org/574285
16:12:27 arkadefr hello, I'm facing a problem with CONFIG_SEARCH_PATH here: https://github.com/openstack/openstacksdk/blob/master/openstack/config/loader.py#L48 . I'm running an ansible command with my user X , and the search path are mainly composed by /root/.config/openstack/clouds.yaml instead of my user directory: /home/X/.config/openstack/clouds.yaml . is it a known issue?
16:14:26 arkadefr my library appdirs and openstack etc are installed as system-wide with pip as root.
16:26:13 mordred arkadefr: so - in ansible the context in which you are executing can become confusing ... if you run ansible as you, it's going to connect to the remote system (even if the remote system is localhost) as whatever user is defined in the inventory
16:26:43 mordred arkadefr: the clouds.yaml doesn't get read in the context of the user executing ansible, but as the remote user ansible is using
16:41:03 dtantsur mordred: do you know how often keystoneauth is released? and whom to ping about it?
16:50:35 mordred dtantsur: it's released whenver we need it to be - kmalloc, lbragstad and I are good people to ping
16:50:49 dtantsur PIIIING! :)
16:51:02 lbragstad dtantsur: do you need a release?
16:51:20 dtantsur yeah, with the patch introducing status_code_retries to Session
16:51:46 lbragstad this one ? https://review.openstack.org/#/c/570934/
16:51:57 kmalloc oh yeah
16:51:59 kmalloc lets release that
16:52:39 kmalloc mordred: i look forward to the email we get to send saying "keystoneclient is deprecated, use SDK instead" :P just FYI
16:52:57 kmalloc mordred: whenever that actually happens, i'm looking forward to it
16:53:15 dtantsur lbragstad: yep
16:53:34 lbragstad dtantsur: mordred kmalloc feel free to double check https://review.openstack.org/#/c/574337/
16:54:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Throw an error on conflicting microversion config https://review.openstack.org/574338
16:54:49 kmalloc looked and +1 lbragstad
16:54:54 mordred kmalloc: ++
16:55:04 mordred Shrews, dtantsur ^^ there's the followup with throwing the error
16:55:13 dtantsur thnx
17:14:24 openstackgerrit qingszhao proposed openstack-infra/requestsexceptions master: fix tox python3 overrides https://review.openstack.org/574350
17:44:47 openstackgerrit qingszhao proposed openstack-dev/os-http master: fix tox python3 overrides https://review.openstack.org/574363
17:45:25 openstackgerrit Eric Fried proposed openstack/os-service-types master: fix tox python3 overrides https://review.openstack.org/573950
18:05:18 openstackgerrit Merged openstack-infra/requestsexceptions master: fix tox python3 overrides https://review.openstack.org/574350
18:14:30 kimamisa Hi ! Any core reviewer for my change ?? :D  Thank you ! https://review.openstack.org/#/c/557359/7
18:14:39 mordred dtantsur|afk: I have sent you some futurist patches :)
18:41:31 openstackgerrit Merged openstack/os-service-types master: fix tox python3 overrides https://review.openstack.org/573950
18:54:07 arkadefr mordred: thanks for the information, I'll continue investigating because I thought I was connecting to localhost as the same user as the one I'm launching the playbook. Thanks again
20:15:22 Shrews arkadefr: if it helps, http://dshrewsbury.blogspot.com/2016/02/common-problems-using-ansible-20.html
22:04:31 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: WIP: Implement support for limits https://review.openstack.org/574475
22:05:09 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: WIP: Implement support for limits https://review.openstack.org/574475
#openstack-sdks - 2018-06-12
00:23:18 openstackgerrit Merged openstack/openstacksdk master: Honor service-type aliases in config https://review.openstack.org/566929
00:48:00 openstackgerrit Merged openstack/python-openstackclient master: Use Server.to_dict() rather than Server._info https://review.openstack.org/573744
01:14:17 openstackgerrit Merged openstack/python-openstackclient master: Optimize _prep_server_detail to avoid redundant find_resource https://review.openstack.org/573760
03:37:04 openstackgerrit qingszhao proposed openstack-dev/os-http master: change default python 3 env in tox to 3.5 https://review.openstack.org/574537
04:44:38 openstackgerrit Merged openstack/python-openstackclient master: Fix server show for microversion 2.47 https://review.openstack.org/573813
05:44:51 arkadefr Shrews: thanks for the reference. It was a "become" in ansible which change my user to root. I explicitly set become=no and it works :)
08:35:57 openstackgerrit XiaojueGuan proposed openstack/os-client-config master: Update links in README https://review.openstack.org/574645
08:49:54 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Allow configuring status_code_retries and connect_retries via cloud config https://review.openstack.org/573780
08:59:55 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Add set_node_provision_state to baremetal Proxy https://review.openstack.org/570895
10:24:13 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: [WIP] Add set_node_provision_state to baremetal Proxy https://review.openstack.org/570895
11:53:50 rabel_ hi. isn't https://bugs.launchpad.net/python-openstackclient the correct place to file bugs against OSC?
11:57:14 kimamisa rabel: Try here https://bugs.launchpad.net/ubuntu/+source/python-openstackclient
11:58:45 rabel_ kimamisa: thanks. but isn't that just for the ubuntu package?
12:00:37 kimamisa rabel: true. try here: https://storyboard.openstack.org/#!/project/975
12:10:51 rabel_ thanks
12:12:51 dtantsur cdent, edleafe, I wonder if we should point graphql enthusiasts to storyboard. it just occurred to me that it's a perfect use case.
12:14:55 cdent dtantsur: that does make quite a bit of sense, but I'm not sure it satisfies what might be requirement: demonstrating usefulness (and providing feedback space) to openstack users
12:15:35 dtantsur well, it would allow to collect developers' feedback before throwing users under the train :)

Earlier   Later