| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-01 | |||
| 18:46:58 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | |
| 21:03:39 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Shift swift segment async code out of adapter https://review.openstack.org/614832 | |
| 21:03:40 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Collect request stats with statsd https://review.openstack.org/614834 | |
| 21:03:40 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove unused Task classes https://review.openstack.org/614833 | |
| 21:03:41 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | |
| 23:11:46 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace TaskManager with a keystoneauth concurrency https://review.openstack.org/604926 | |
| 23:11:46 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Collect request stats with statsd https://review.openstack.org/614834 | |
| #openstack-sdks - 2018-11-02 | |||
| 06:52:47 | openstackgerrit | Vieri proposed openstack/openstackclient master: Update min tox version to 2.0 https://review.openstack.org/615036 | |
| 06:54:05 | openstackgerrit | Vieri proposed openstack/openstacksdk master: Update min tox version to 2.0 https://review.openstack.org/615037 | |
| 06:55:57 | openstackgerrit | Vieri proposed openstack/os-client-config master: Update min tox version to 2.0 https://review.openstack.org/615040 | |
| 07:10:15 | openstackgerrit | Kim Bao Long proposed openstack-infra/shade master: Update the min version of tox to 2.0 https://review.openstack.org/615056 | |
| 07:46:00 | openstackgerrit | Vieri proposed openstack-dev/os-http master: Update min tox version to 2.0 https://review.openstack.org/615079 | |
| 08:31:36 | openstackgerrit | Kim Bao Long proposed openstack-infra/shade master: Update the min version of tox to 2.0 https://review.openstack.org/615056 | |
| 11:57:16 | openstackgerrit | Vieri proposed openstack-infra/requestsexceptions master: Update min tox version to 2.0 https://review.openstack.org/615150 | |
| 12:06:22 | openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Allow returning munch on create and get object https://review.openstack.org/597198 | |
| 12:06:38 | samueldmq | morning sdk! | |
| 13:20:44 | mordred | morning samueldmq ! | |
| 13:26:08 | Shrews | mordred: what are the "verification commands" that we need tox 2.0 for in https://review.openstack.org/615056 ? | |
| 13:26:40 | Shrews | we haven't had any issues with shade tox that i know of | |
| 13:27:49 | Shrews | upgrading local tox (in my experience) usually results in a PITA | |
| 13:28:00 | mordred | Shrews: no clue. but I figure I'm running tox 3.4.0 locally, so requiring 2 vs 1 is probably fine? | |
| 13:28:10 | mordred | Shrews: what version are you running locally? | |
| 13:29:41 | Shrews | mordred: oh wow, didn't realize tox had advanced versions so quickly | |
| 13:30:16 | Shrews | i'm on 3.1.2 on this machine, so i suppose i shouldn't worry about that requirement :) | |
| 13:35:32 | dtantsur | Shrews, mordred, EPEL has 1.4.2.. | |
| 13:36:02 | mordred | dtantsur: do folks develop on centos+epel? | |
| 13:36:47 | dtantsur | mordred: I have devstack on centos, but I install tox from pypi | |
| 13:36:53 | mordred | sorry- that may have come out snarkier than intended - it's an honest question, I'm pretty clueless about how a bunch of that stuff works | |
| 13:37:21 | dtantsur | well, I do see overall issues with developing on centos. but I know people do it (ditto for rhel) | |
| 13:37:29 | mordred | dtantsur: yah - I install tox from pypi as well (although I'm on ubuntu bionic) | |
| 13:37:31 | mordred | wild | |
| 13:37:38 | dtantsur | in any case, since we started putting basepython=python3 in tox.ini, centos users are quite screwed :D | |
| 13:37:43 | mordred | yeah | |
| 13:37:59 | mordred | fingers crossed centos8/rhel8 have a good python3 version | |
| 13:38:17 | dtantsur | check what I have to do: https://github.com/dtantsur/config/blob/master/tasks/tox.yml | |
| 13:38:35 | mordred | dtantsur: :) | |
| 13:38:49 | mordred | dtantsur: fwiw - I gave up on distro python even and started using pyenv for python | |
| 13:39:18 | dtantsur | mordred: let us make devstack give up on distro python, and I'll be a much happier man | |
| 13:39:30 | dtantsur | :) | |
| 13:39:49 | mordred | I keep mildly floating the idea that we should just use pyenv - but I don't actually want to do the work to champion that :) | |
| 13:40:28 | dtantsur | I'd be fine with venv as well.. even if it just one huge venv with all services. | |
| 13:41:01 | dtantsur | another problem with us using the distro python is that it heavily screws 'yum update' by overwriting half of system python packages | |
| 13:43:31 | mordred | yes | |
| 13:43:56 | mordred | dtantsur: have you played with pyenv? it's really cool | |
| 13:44:03 | mordred | mordred@solace:~/src/git.openstack.org/openstack/openstacksdk$ pyenv versions | |
| 13:44:04 | mordred | system | |
| 13:44:06 | mordred | * 2.7.14 (set by /home/mordred/src/github.com/pyenv/pyenv/version) | |
| 13:44:08 | mordred | * 3.5.5 (set by /home/mordred/src/github.com/pyenv/pyenv/version) | |
| 13:44:30 | dtantsur | I have, yes. I don't have a pressing need for it on my laptop since fedora has plenty of python versions out of the box :) | |
| 13:44:40 | mordred | fair :) | |
| 13:45:20 | dtantsur | for centos - yes. I need to come up with some playbooks to configure it (I have a rule of never configuring my test environments by hand, only with these playbooks I linked above) | |
| 13:45:37 | mordred | that is an excellent rule | |
| 14:08:27 | mordred | Shrews: I made a few more patches related to nodepool/task-manager - https://review.openstack.org/#/c/614834/ and its ancestors | |
| 14:08:49 | mordred | the three leading up to it are just extracting some smaller pieces from the "remove task manager" patch | |
| 14:09:01 | mordred | the one I linked to is the one with a whole new thing in it | |
| 14:23:03 | openstackgerrit | Merged openstack-infra/shade master: Update the min version of tox to 2.0 https://review.openstack.org/615056 | |
| 14:24:16 | Shrews | mordred: *nod* | |
| 14:31:42 | openstackgerrit | Merged openstack/openstacksdk master: Update min tox version to 2.0 https://review.openstack.org/615037 | |
| 15:01:35 | openstackgerrit | Merged openstack/os-client-config master: Update min tox version to 2.0 https://review.openstack.org/615040 | |
| 15:01:36 | openstackgerrit | Merged openstack/keystoneauth master: fix wrong spelling of "unnecessary" https://review.openstack.org/614178 | |
| 15:47:43 | kmalloc | mordred: +1 on the rate limiting, going through the code one more time | |
| 15:48:00 | kmalloc | mordred: as you'd expect... because the whole KSA contract of "we wont break you... ever" | |
| 15:48:11 | kmalloc | this is a pretty big change in additional functionality | |
| 15:49:16 | kmalloc | dtantsur: i never understood why RHEL/Fedora doesn't do the user/local bit for pip where debian did. Overwriting the distro python seems like a bad idea. | |
| 15:49:30 | dtantsur | yeah, no idea either | |
| 15:49:43 | dtantsur | I just got used to add --user to any pip commands I execute | |
| 15:50:01 | kmalloc | it is a major reason i run ubuntu (*looks sideways at employer reference*) locally | |
| 15:50:23 | kmalloc | --user is good, except when it just doesn't work. | |
| 15:50:40 | kmalloc | or when i need to install things like 20 times because different users.... | |
| 15:51:02 | kmalloc | though these days i just run my python envs in docker containers | |
| 15:51:20 | kmalloc | it might even be worth utilizing docker containers for python versions within devstack. | |
| 15:51:45 | kmalloc | since it really doesn't take a ton of time to build if we have a local artifact repo | |
| 15:51:49 | kmalloc | local-ish | |
| 15:52:10 | kmalloc | but pyenv seems like way less work | |
| 15:53:33 | mordred | kmalloc: for zuul's container builds, we're just using the python base image from docker (although we're using python:alpine) | |
| 15:54:22 | kmalloc | mordred: i have so many concerns about alpine... but that is a different conversation | |
| 15:54:25 | mordred | kmalloc: and we're using the pbrx tool I wrote - which will make you container images of things using bindep and pbr without any additional in-repo config | |
| 15:55:02 | kmalloc | mordred: yeah i've been looking at pbrx it's awesome | |
| 15:55:04 | mordred | kmalloc: well, in this case it's because of three things - bmw was already using alpine containers in production for zuul, modern bubblewrap is already in alpine - no no need for ppas, and it's smaller | |
| 15:55:11 | mordred | kmalloc: but yeah | |
| 15:55:45 | kmalloc | mordred: oooh ooh i know lets use GENTOO! /s [nothing against gentoo, just not in this case] | |
| 15:55:51 | mordred | hahaha | |
| 15:57:07 | kmalloc | i have been working to use alpine locally, but i need to rip apart my network and re-build it first because i lost access to control things via vlans.... | |
| 15:57:23 | kmalloc | so some infrastructure bits for a local virt stack can't be stood up | |
| 15:57:36 | kmalloc | don't get me wrong, I like alpine's lightweight nature. | |
| 15:57:55 | kmalloc | I feel like it's a little too fast-and-loose with what is added to it. | |
| 15:58:15 | kmalloc | somewhere between alpine and debian is where I'd like to be. (Ubuntu is not the answer) | |
| 15:58:28 | kmalloc | debian stable* | |
| 16:46:56 | kmalloc | mordred: +2 on rate limit in KSA | |
| 16:47:09 | kmalloc | it feels like it really belongs there as much as I wish it could be higher up. | |
| 16:47:14 | mordred | kmalloc: woot. (pending tests, of course) | |
| 16:47:17 | kmalloc | and that is due to retry logic | |
| 16:47:18 | kmalloc | yes. | |
| 16:47:25 | mordred | kmalloc: yah. the retry logic is the real kicker | |
| 16:47:30 | kmalloc | must have tests before +A. | |
| 16:47:34 | mordred | kmalloc: sort of like why discovery needed to be there too | |
| 16:47:43 | kmalloc | functional tests are fine (doesn't need to be unit) | |
| 16:47:44 | mordred | since it needes to be there for keystone _anyway_ for auth to work :) | |
| 16:47:57 | kmalloc | it needs to be tested so we don't break | |
| 16:48:10 | kmalloc | unit tests of that is going to be hard(tm) | |