Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-06-06
18:04:16 mordred dtantsur: well... we have support for it, but the class that implements it is currently sitting in nodepool. I've been planning on moving the nodepool rate-limiting taskmanager into the sdk tree so that the whole picture is present (this plays in to the caching story too)
18:04:44 mordred dtantsur: so - it's a thing we support, but you woulnd't know it by looking at the code that's there - which needs to be fixed
18:04:56 dtantsur much complexity not wow
18:05:20 dtantsur I guess "just switch from ironicclient to openstacksdk" is not as easy as it seemed initially :)
18:05:36 mordred dtantsur: sure it is :)
18:06:02 mordred dtantsur: http://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/task_manager.py#n65 is the TaskManager that needs to be added to the openstacksdk tree - nodepool currently passes it in to the sdk connection constructor
18:06:05 dtantsur I wonder if it's a problem for other services, but ironic better has retries
18:06:08 mordred everyone else gets the direct-action taskmanager
18:06:26 dtantsur aha, taskmanagers is something I haven't figured out yet, to be honest
18:07:04 mordred you can mostly ignore them - until this - the one that's on by default just executes the calls when you make them
18:09:42 dtantsur mordred: do these work only in the former shade bits or in the other half as well?
18:09:51 dtantsur since connection.baremetal things seem to just use the session
18:09:59 dtantsur (unless I'm completely clueless, which is not unlikely)
18:10:00 mordred the other half - the way it's connected through is that it happens on the adapter calls
18:10:46 mordred dtantsur: this next thing I'm going to say will make you table flip - but the sessions laying around are all actually adapters - we need a gloal search-and-replace
18:11:37 mordred but - the Adapter created for each service by the sdk puts submits every external rest call into the taskmanager (which by default just executes it right then) - so if you did use a different taskmanager it would apply to any calls made on the adapter
18:12:05 mordred :)
18:12:13 dtantsur okay, so MAAAGIIIC happens, and it just works somewhere inside, right?
18:12:19 mordred yup
18:13:12 mordred ah - we have generalized support for retrying on keystoneauth1.exceptions.RetriableConnectionFailure
18:13:20 mordred but only on that one
18:13:29 mordred I like your approach in ksa better :)
18:13:58 dtantsur :)
18:29:51 dtantsur okay, I'll think about all this in the morning.. ironic really needs retries on 503 and 409 though. just not sure how to play nicely with rate limiting.
18:33:51 kmalloc ratelimiting is ... hard(tm)
19:27:26 openstackgerrit Doug Hellmann proposed openstack/cliff master: fix tox python3 overrides https://review.openstack.org/572901
19:27:38 openstackgerrit Doug Hellmann proposed openstack/keystoneauth master: fix tox python3 overrides https://review.openstack.org/572906
19:27:57 openstackgerrit Doug Hellmann proposed openstack/openstacksdk master: fix tox python3 overrides https://review.openstack.org/572913
19:28:02 openstackgerrit Doug Hellmann proposed openstack/osc-lib master: fix tox python3 overrides https://review.openstack.org/572915
19:28:04 openstackgerrit Doug Hellmann proposed openstack/os-client-config master: fix tox python3 overrides https://review.openstack.org/572916
19:28:29 openstackgerrit Doug Hellmann proposed openstack-infra/requestsexceptions master: fix tox python3 overrides https://review.openstack.org/572926
19:28:34 openstackgerrit Doug Hellmann proposed openstack-infra/shade master: fix tox python3 overrides https://review.openstack.org/572928
19:31:41 Shrews dhellmann: i left a comment on 572928
19:31:51 Shrews i'm not sure that will work, but we'll see
19:35:25 dhellmann Shrews : yeah, I responded on the patch, but this was all auto-generated (I guess I should have included that in the commit message)
19:35:30 dhellmann I'll fix up anything that fails hard
19:58:46 openstackgerrit Merged openstack/keystoneauth master: Add optional support for retrying certain HTTP codes https://review.openstack.org/570934
21:22:23 edleafe elmiko: cdent: dtantsur|afk: As it turns out, I should be available for the API-SIG meeting tomorrow
#openstack-sdks - 2018-06-07
00:16:07 openstackgerrit Merged openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974
07:15:07 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit ddd5f58d5eda742e908dd505de80654348c359ed Author: Popescu Catalin Date: Mon Jun 07 10:16:10 2018 +0300 https://review.openstack.org/572693
07:17:17 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit ddd5f58d5eda742e908dd505de80654348c359ed Author: Popescu Catalin Date: Mon Jun 07 10:16:10 2018 +0300 https://review.openstack.org/572693
07:38:57 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit ddd5f58d5eda742e908dd505de80654348c359ed Author: Popescu Catalin Date: Mon Jun 07 10:16:10 2018 +0300 https://review.openstack.org/573117
08:00:21 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit 7b1e758f3870ad62c2478250daa7cf4d00614689 Author: Popescu Catalin Date: Mon Jun 07 11:00:10 2018 +0300 https://review.openstack.org/573122
08:19:17 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit 7b1e758f3870ad62c2478250daa7cf4d00614689 Author: Popescu Catalin Date: Mon Jun 07 11:20:10 2018 +0300 https://review.openstack.org/573126
08:20:01 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit 7b1e758f3870ad62c2478250daa7cf4d00614689 Author: Popescu Catalin Date: Mon Jun 07 11:20:10 2018 +0300 https://review.openstack.org/573126
08:25:26 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: commit 7b1e758f3870ad62c2478250daa7cf4d00614689 Author: Popescu Catalin Date: Mon Jun 07 11:24:10 2018 +0300 https://review.openstack.org/573127
08:47:13 openstackgerrit Catalin Popescu proposed openstack/openstacksdk master: Modify live migrate in order to work https://review.openstack.org/573134
09:05:14 openstackgerrit Chen proposed openstack/python-openstackclient master: Fix urls in README.rst https://review.openstack.org/573140
10:19:17 openstackgerrit Jens Harbott (frickler) proposed openstack-infra/shade master: WIP: Test job based on devstack-new-neutron https://review.openstack.org/552459
10:58:09 openstackgerrit Merged openstack/cliff master: fix tox python3 overrides https://review.openstack.org/572901
12:34:23 openstackgerrit Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359
12:35:37 openstackgerrit Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359
14:07:24 openstackgerrit Merged openstack-infra/shade master: fix tox python3 overrides https://review.openstack.org/572928
14:16:28 openstackgerrit Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359
14:55:01 openstackgerrit Chen proposed openstack/python-openstackclient master: Fix urls in README.rst https://review.openstack.org/573140
15:58:58 openstackgerrit Lance Bragstad proposed openstack/python-openstackclient master: Update role document to include system parameter https://review.openstack.org/573303
16:30:11 elmiko edleafe cdent dtantsur ready for review, https://etherpad.openstack.org/p/api-sig-newsletter
16:31:43 edleafe elmiko: lgtm with the minor changes
16:31:57 elmiko thanks as always for the helping hand edleafe =)
16:36:40 elmiko sent
17:54:01 openstackgerrit Merged openstack/os-client-config master: fix tox python3 overrides https://review.openstack.org/572916
23:44:38 openstackgerrit Tim Burke proposed openstack/python-openstackclient master: Before writing object data to stdout, re-open it in binary mode https://review.openstack.org/573465
#openstack-sdks - 2018-06-08
07:57:07 openstackgerrit Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359
09:23:15 viktor_t Hi, what is the correction policy for python-openstackclient? Is there any chance to get https://bugs.launchpad.net/python-openstackclient/+bug/1747862 corrected into 3.14.x (for queens)?
09:23:16 openstack Launchpad bug 1747862 in python-openstackclient "encoding error when doing console log show" [Undecided,Fix released]
09:38:21 openstackgerrit Yang Youseok proposed openstack/openstacksdk master: Support port binding extended attributes for querying port https://review.openstack.org/573645
09:41:29 Luzi victor_t: Is there anyone active in osc? I tried to propose something on the mailinglist, but got no answer.
09:42:47 viktor_t Luzi: no idea...
09:43:58 Luzi victor_t: well, all I can see ist, that they don't use Launchpad anymore. Instead they use Storyboard.
14:00:11 mriedem is this where osc talk happens?
14:00:18 mriedem hard to know from https://wiki.openstack.org/wiki/IRC
14:04:44 frickler mriedem: if you want to talk osc, this is the place. not much happening, though
14:07:15 openstackgerrit Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359
14:07:36 mriedem frickler: oh i want to talk osc alright
14:08:21 mriedem i'm just going through https://etherpad.openstack.org/p/compute-api-microversion-gap-in-osc so i'm likely to have random questions, and just wanted to make sure this was the right place
14:08:58 mriedem for example, what's the process on changing how options work, i.e. https://review.openstack.org/#/c/460059/
14:09:23 mriedem openstack server migrate --live <host> requires a host - but we don't actually want people specifying a host during live migration. i suggest we make --live just mean live migration, and add a --host option.
14:10:12 mriedem the problem with live migration with a host is that before microversion 2.30, (which is the default in osc, uses 2.1), it will force the host and bypass the scheduler
14:10:17 mriedem which likely means breaking stuff
14:11:20 mriedem changing those options is clearly a behavior change, but is that ok? with a major version?
14:11:28 dtroyer mriedem: live migrate has a number of problems and needs major cleaning up. there are at least two reviews addressing that and someone who understands how it actually needs to reconcile it all
14:11:49 dtroyer We're planning v4 which will break things like this, I've been sauying that for way too long
14:12:13 mriedem "someone who understands how it actually needs to reconcile it all" yeah that's me
14:12:15 dtroyer IIRC there was at least one proposal to clean up live migrate without breaking currently-working behaviour
14:12:19 mriedem i've commented on chen rui's change above
14:12:25 mriedem that's chenrui's
14:12:35 mriedem it adds a --live-auto option for no host cases
14:12:42 mriedem that's a bit clunky though imo
14:13:31 dtroyer its all clunky. this would help me: write up the way you would handle live migrate for 4.0 (no back-compat) and lets see how much of that can be done without breaking things
14:14:05 dtroyer some of what is there now doesn't work, I'll break that, after all nobody is using it, right?
14:16:23 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Add note about version 2.5 when listing servers using --ip6 https://review.openstack.org/573709
14:16:42 mriedem dtroyer: yup, already did in the commit message of https://review.openstack.org/#/c/460059/
14:16:47 mriedem suggest a --live option and a --host option
14:16:53 mriedem and then we can add a --cold option too
14:17:09 dtroyer thanks
14:52:00 mriedem is there an easy-ish way to wrap fields in a show output? http://paste.openstack.org/show/722985/
14:52:08 mriedem the *_details lists in there wrap hard in a console
14:53:35 mriedem i guess i can use --fit-width
14:55:01 mriedem ooo format_list_of_dicts
15:00:35 mriedem i wish i had abandon auth on these osc compute patches :) lots of really old merge conflict cruft in here

Earlier   Later