Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-10-01
16:00:18 efried ack
16:04:18 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: Fetch discovery documents with auth when needed https://review.opendev.org/685042
16:11:42 mordred gtema, efried: BWAHAHAHAHA. it's "not a bug" that we're requesting a later microversion from nova - we actually bumped the sdk layer microversion for the server resource
16:12:08 mordred but - because of the other bug, we never actually were getting 2.72 payloads in devstack and didn't notice we needed to update our flavor support
16:12:33 gtema life is perfect
16:12:48 gtema (unless one touches anything)
16:12:55 mordred yeah
16:14:04 gtema discovery and microversions are voodoo things in OpenStack
16:23:42 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix server for later microversion https://review.opendev.org/685999
16:24:42 mordred gtema: yah. although they wouldn't have been if the client libraries hadn't hardcoded around them and if people hadn't put versioned endpoints in the catalog for the last 9 years. like - there was this whole system that literally nobody actually used :(
16:24:59 gtema sure
16:25:27 gtema hope we will fix it now with Nova switched to SDK and others using it deeper
16:27:21 mordred gtema: or we've just opened ourselves to a new kind of pain
16:35:11 gtema that's of course as well
#openstack-sdks - 2019-10-02
09:17:24 openstackgerrit Artem Goncharov proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374
12:39:54 openstackgerrit Artem Goncharov proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374
15:38:01 openstackgerrit Stephen Finucane proposed openstack/cliff master: Blacklist bandit 1.6.0 due to directory exclusion bug https://review.opendev.org/658522
15:39:35 tosky stephenfin: I think also 1.6.1 and 1.6.2 are buggy, I had to stick to <1.6.0 in sahara
15:41:47 openstackgerrit Stephen Finucane proposed openstack/cliff master: Switch to stestr https://review.opendev.org/581564
15:41:57 stephenfin tosky: Hmm, really? That was just a rebase of amotoki's patch
15:42:35 tosky stephenfin: that was proposed in May, when 1.6.{1,2} have not been released yet
17:25:09 openstackgerrit Merged openstack/cliff master: Modify the help message of `-c`/`--column` parameter https://review.opendev.org/672852
19:34:33 openstackgerrit Merged openstack/cliff master: Add an errexit attribute to InteractiveApp to exit on command errors https://review.opendev.org/511971
19:36:48 openstackgerrit Merged openstack/cliff master: Allow finding command by partial name https://review.opendev.org/552970
#openstack-sdks - 2019-10-03
09:26:31 stephenfin amotoki: Morning. Could I ask you to look at this change for me, please? https://review.opendev.org/#/c/664393/ It's been around for some time and is blocking some doc rework I want to do in nova
09:27:16 mordred hey stephenfin
09:27:18 mordred looking
09:27:30 stephenfin mordred: Yo. Thanks :)
09:28:04 mordred stephenfin: done
09:28:16 stephenfin \o/ Appreciate it
09:28:38 mordred stephenfin: you know things ...
09:28:55 stephenfin don't you do it
09:28:59 mordred stephenfin: what's the python2 story for U again? support goes away, right?
09:29:20 stephenfin phew, thought you were threatening me with core on something I didn't want for a second
09:29:23 stephenfin Yup, kill it
09:29:53 stephenfin We haven't started removing stuff in nova yet and probably won't until Train GA
09:29:55 stephenfin but after that...
09:30:11 mordred stephenfin: cool. I have someone interested in adding some type annotations - which obviously need 2 to go away
09:30:32 stephenfin mordred: https://review.opendev.org/#/c/676209/
09:30:41 stephenfin ignore all the failures - I rushed it :)
09:30:45 mordred nice!
09:31:10 stephenfin We might want to put this in openstack-zuul-jobs though https://review.opendev.org/#/c/676210/
09:31:19 mordred stephenfin: zuul projects are already py3 - this one: https://review.opendev.org/#/c/686249/ added annotations but had to put in an ignore on the "import openstack" line
09:32:04 mordred yeah - probably so. we've actually caught bugs with mypy in zuul
09:32:09 stephenfin mordred: That's because we're using an old version of flake8 because hacking hasn't been updated. New versions support type hints
09:32:19 stephenfin or rather, detect that an import is necessary due to type hints
09:32:50 stephenfin alas, updating hacking to work with flake8 3.x is easier said that done because it was completely rewritten. I've tried and failed a few times :(
09:32:51 mordred oh - no - I mean we had to mypy ignore it becuase there's no type information from openstack I think
09:32:55 stephenfin ohhh
09:33:06 mordred oh goodie
09:33:31 mordred stephenfin: yeah - line 17: https://review.opendev.org/#/c/686249/3/zuul_registry/swift.py
09:33:46 stephenfin mordred: This should do the same thing, I think https://review.opendev.org/#/c/676208/1/setup.cfg@91
09:34:16 stephenfin but maybe not. I'm at about 3/10 on the "understanding how mypy works" scale rn
09:34:22 mordred stephenfin: ignore_missing_imports? or follow_imports=skip?
09:34:32 stephenfin ignore_missing_imports
09:34:50 stephenfin hmm, maybe not actually
09:35:04 mordred ah - no, follow_imports=skip
09:35:07 mordred https://mypy.readthedocs.io/en/latest/running_mypy.html#follow-imports
09:36:23 stephenfin Hmm, worth experimenting with those flags, I imagine
09:36:26 stephenfin That's cool though
09:37:04 mordred yeah
09:37:13 stephenfin I'd like to see them in most of oslo too, though I think I'll need to request help from the mypy folks for o.v.o. That's a weird one
09:37:19 stephenfin zuul doesn't use oslo though, does it?
09:37:24 mordred that said - it would be nice to add the annotations so that you could annotate things with signatures of the imported objects
09:37:35 mordred stephenfin: that's right - zuul does not use oslo
09:37:49 stephenfin cool. probably won't affect you much so :)
09:55:26 mordred stephenfin: oh - actually - fwiw, we just run mypy in our pep8 tox env
09:55:44 mordred which is really "static code analysis tox env"
09:57:05 stephenfin Hmm, I wanted to keep it separate because (a) pep8 is already _really_ slow in nova and I did not want to make it even slower, and (b) we're all luddites there and I didn't want to foist stuff onto people before they were ready :)
09:57:24 stephenfin I could go with pep8 though. Don't some project use a 'linters' target?
09:57:27 stephenfin *projects
09:57:45 stephenfin or 'style' or something? /me tries to recall where he saw it
10:07:12 mordred stephenfin: yeah - we tried switching to linters a while back, but didn't really get any traction
10:08:08 mordred but - you know - mostly thinking about it from a gate pov and not a local dev pov - I imagine pep8+mypy is still quicker than py36 unittests
10:08:22 mordred it's a good point re: local dev though, especially if pep8 is slow for y'all
10:14:47 mordred poo. the connection metaclass makes sdk almost impossible to type hint
10:15:36 mordred maybe we should replace it with a generated file and a zuul job that runs when os-service-types list updates
10:50:32 openstackgerrit Merged openstack/os-client-config master: Update master for stable/train https://review.opendev.org/683481
11:02:14 openstackgerrit Merged openstack/shade master: Update master for stable/train https://review.opendev.org/683658
11:02:14 openstackgerrit Merged openstack/shade master: fix "How To Contribute",bugs,review url https://review.opendev.org/682801
11:19:48 openstackgerrit Merged openstack/python-openstackclient master: Add 'openstack server migrate (confirm|revert)' commands https://review.opendev.org/664393
11:49:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use has_service in functional test's require_service https://review.opendev.org/685995
11:49:38 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix server for later microversion https://review.opendev.org/685999
13:11:41 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Start supporting type info https://review.opendev.org/686397
13:32:48 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: WIP Add typing flag file https://review.opendev.org/686401
13:50:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use has_service in functional test's require_service https://review.opendev.org/685995
13:56:25 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix server for later microversion https://review.opendev.org/685999
13:56:44 openstackgerrit Pierre Prinetti proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408
14:00:23 mordred dtantsur|afk: when you get a sec, I left a note in 685999 - I'm wondering if you have any ideas on how a "good" way to handle that is since you've been doing more with microversions so far
14:00:56 mordred I'm guessing I'd want to put something into the ServerGroup resource that controlled which mv it used when doing the creation call
14:08:11 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Fix server for later microversion https://review.opendev.org/685999
14:10:41 mordred dtantsur|afk: neverind - mriedem tells me that in this case it's easy - old mv never actually supported a list
14:11:10 mriedem child's play
14:19:33 openstackgerrit Pierre Prinetti proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408
14:23:19 openstackgerrit Pierre Prinetti proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408
15:33:45 openstackgerrit Pierre Prinetti proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408
15:37:32 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408
15:46:11 openstackgerrit Pierre Prinetti proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408
15:56:33 openstackgerrit Tristan Cacqueray proposed openstack/openstacksdk master: WIP: annotate some swift function used by the zuul_registry https://review.opendev.org/686433
15:59:43 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: versions: Fix 'versions show' help message https://review.opendev.org/686408

Earlier   Later