Earlier  
Posted Nick Remark
#openstack-sdks - 2018-06-06
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
15:08:19 mriedem this looks like a bug https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server.py#L132 - why would show server get the server, then get it again?
15:09:31 mriedem oh i see: https://github.com/openstack/python-openstackclient/blob/4cd702c3610e4e6b40695ff9ea8eecf7c0735c68/openstackclient/compute/v2/server.py#L120
15:10:42 mriedem sort of
15:10:54 mriedem there is just so much new code to chew on
15:14:38 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: Use Server.to_dict() rather than Server._info https://review.openstack.org/573744
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)

Earlier   Later