| *** bobh has joined #openstack-sdks | 02:37 | |
| *** bobh has quit IRC | 02:57 | |
| openstackgerrit | Vieri proposed openstack/openstackclient master: Update min tox version to 2.0 https://review.openstack.org/615036 | 06:52 |
|---|---|---|
| openstackgerrit | Vieri proposed openstack/openstacksdk master: Update min tox version to 2.0 https://review.openstack.org/615037 | 06:54 |
| openstackgerrit | Vieri proposed openstack/os-client-config master: Update min tox version to 2.0 https://review.openstack.org/615040 | 06:55 |
| openstackgerrit | Kim Bao Long proposed openstack-infra/shade master: Update the min version of tox to 2.0 https://review.openstack.org/615056 | 07:10 |
| *** gildub has quit IRC | 07:40 | |
| *** gildub has joined #openstack-sdks | 07:40 | |
| openstackgerrit | Vieri proposed openstack-dev/os-http master: Update min tox version to 2.0 https://review.openstack.org/615079 | 07:46 |
| *** slaweq has joined #openstack-sdks | 08:17 | |
| openstackgerrit | Kim Bao Long proposed openstack-infra/shade master: Update the min version of tox to 2.0 https://review.openstack.org/615056 | 08:31 |
| *** jpich has joined #openstack-sdks | 09:00 | |
| *** cdent has joined #openstack-sdks | 09:04 | |
| *** cdent has quit IRC | 09:06 | |
| *** cdent has joined #openstack-sdks | 09:10 | |
| *** ttsiouts has joined #openstack-sdks | 09:17 | |
| *** dtantsur|afk is now known as dtantsur | 09:48 | |
| *** e0ne has joined #openstack-sdks | 09:51 | |
| *** ttsiouts has quit IRC | 10:16 | |
| *** ttsiouts has joined #openstack-sdks | 10:32 | |
| *** stephenfin is now known as finucannot | 10:44 | |
| *** dave-mccowan has joined #openstack-sdks | 11:31 | |
| openstackgerrit | Vieri proposed openstack-infra/requestsexceptions master: Update min tox version to 2.0 https://review.openstack.org/615150 | 11:57 |
| *** cdent has quit IRC | 12:06 | |
| openstackgerrit | Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: Allow returning munch on create and get object https://review.openstack.org/597198 | 12:06 |
| samueldmq | morning sdk! | 12:06 |
| *** ttsiouts has quit IRC | 12:39 | |
| *** bobh has joined #openstack-sdks | 12:59 | |
| *** gildub has quit IRC | 13:06 | |
| mordred | morning samueldmq ! | 13:20 |
| Shrews | mordred: what are the "verification commands" that we need tox 2.0 for in https://review.openstack.org/615056 ? | 13:26 |
| Shrews | we haven't had any issues with shade tox that i know of | 13:26 |
| Shrews | upgrading local tox (in my experience) usually results in a PITA | 13:27 |
| 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 |
| mordred | Shrews: what version are you running locally? | 13:28 |
| *** mriedem has joined #openstack-sdks | 13:28 | |
| *** hjensas has quit IRC | 13:29 | |
| Shrews | mordred: oh wow, didn't realize tox had advanced versions so quickly | 13:29 |
| Shrews | i'm on 3.1.2 on this machine, so i suppose i shouldn't worry about that requirement :) | 13:30 |
| *** ttsiouts has joined #openstack-sdks | 13:32 | |
| dtantsur | Shrews, mordred, EPEL has 1.4.2.. | 13:35 |
| mordred | dtantsur: do folks develop on centos+epel? | 13:36 |
| dtantsur | mordred: I have devstack on centos, but I install tox from pypi | 13:36 |
| 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:36 |
| dtantsur | well, I do see overall issues with developing on centos. but I know people do it (ditto for rhel) | 13:37 |
| mordred | dtantsur: yah - I install tox from pypi as well (although I'm on ubuntu bionic) | 13:37 |
| mordred | wild | 13:37 |
| dtantsur | in any case, since we started putting basepython=python3 in tox.ini, centos users are quite screwed :D | 13:37 |
| mordred | yeah | 13:37 |
| mordred | fingers crossed centos8/rhel8 have a good python3 version | 13:37 |
| dtantsur | check what I have to do: https://github.com/dtantsur/config/blob/master/tasks/tox.yml | 13:38 |
| mordred | dtantsur: :) | 13:38 |
| mordred | dtantsur: fwiw - I gave up on distro python even and started using pyenv for python | 13:38 |
| * mordred is such a terrible employee of a company that produces a linux distro | 13:39 | |
| dtantsur | mordred: let us make devstack give up on distro python, and I'll be a much happier man | 13:39 |
| dtantsur | :) | 13:39 |
| 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:39 |
| dtantsur | I'd be fine with venv as well.. even if it just one huge venv with all services. | 13:40 |
| *** d0ugal has quit IRC | 13:40 | |
| dtantsur | another problem with us using the distro python is that it heavily screws 'yum update' by overwriting half of system python packages | 13:41 |
| mordred | yes | 13:43 |
| mordred | dtantsur: have you played with pyenv? it's really cool | 13:43 |
| mordred | mordred@solace:~/src/git.openstack.org/openstack/openstacksdk$ pyenv versions | 13:44 |
| mordred | system | 13:44 |
| mordred | * 2.7.14 (set by /home/mordred/src/github.com/pyenv/pyenv/version) | 13:44 |
| mordred | * 3.5.5 (set by /home/mordred/src/github.com/pyenv/pyenv/version) | 13:44 |
| 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 |
| mordred | fair :) | 13:44 |
| 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 |
| mordred | that is an excellent rule | 13:45 |
| mordred | Shrews: I made a few more patches related to nodepool/task-manager - https://review.openstack.org/#/c/614834/ and its ancestors | 14:08 |
| mordred | the three leading up to it are just extracting some smaller pieces from the "remove task manager" patch | 14:08 |
| mordred | the one I linked to is the one with a whole new thing in it | 14:09 |
| *** ttsiouts_ has joined #openstack-sdks | 14:09 | |
| *** ttsiouts has quit IRC | 14:12 | |
| openstackgerrit | Merged openstack-infra/shade master: Update the min version of tox to 2.0 https://review.openstack.org/615056 | 14:23 |
| Shrews | mordred: *nod* | 14:24 |
| *** ttsiouts_ has quit IRC | 14:25 | |
| *** d0ugal has joined #openstack-sdks | 14:27 | |
| openstackgerrit | Merged openstack/openstacksdk master: Update min tox version to 2.0 https://review.openstack.org/615037 | 14:31 |
| *** ttsiouts has joined #openstack-sdks | 14:35 | |
| *** nmimi has quit IRC | 14:49 | |
| openstackgerrit | Merged openstack/os-client-config master: Update min tox version to 2.0 https://review.openstack.org/615040 | 15:01 |
| openstackgerrit | Merged openstack/keystoneauth master: fix wrong spelling of "unnecessary" https://review.openstack.org/614178 | 15:01 |
| *** ttsiouts has quit IRC | 15:04 | |
| *** ttsiouts has joined #openstack-sdks | 15:05 | |
| *** dave-mccowan has quit IRC | 15:25 | |
| *** dave-mccowan has joined #openstack-sdks | 15:26 | |
| kmalloc | mordred: +1 on the rate limiting, going through the code one more time | 15:47 |
| kmalloc | mordred: as you'd expect... because the whole KSA contract of "we wont break you... ever" | 15:48 |
| kmalloc | this is a pretty big change in additional functionality | 15:48 |
| * kmalloc needs to wrangle another Keystone core to weigh in as well. | 15:48 | |
| 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 |
| dtantsur | yeah, no idea either | 15:49 |
| dtantsur | I just got used to add --user to any pip commands I execute | 15:49 |
| kmalloc | it is a major reason i run ubuntu (*looks sideways at employer reference*) locally | 15:50 |
| kmalloc | --user is good, except when it just doesn't work. | 15:50 |
| kmalloc | or when i need to install things like 20 times because different users.... | 15:50 |
| kmalloc | though these days i just run my python envs in docker containers | 15:51 |
| kmalloc | it might even be worth utilizing docker containers for python versions within devstack. | 15:51 |
| kmalloc | since it really doesn't take a ton of time to build if we have a local artifact repo | 15:51 |
| kmalloc | local-ish | 15:51 |
| kmalloc | but pyenv seems like way less work | 15:52 |
| mordred | kmalloc: for zuul's container builds, we're just using the python base image from docker (although we're using python:alpine) | 15:53 |
| kmalloc | mordred: i have so many concerns about alpine... but that is a different conversation | 15:54 |
| 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:54 |
| * kmalloc grabs a crowbar to pry off the security hat | 15:54 | |
| kmalloc | mordred: yeah i've been looking at pbrx it's awesome | 15:55 |
| 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 |
| mordred | kmalloc: but yeah | 15:55 |
| kmalloc | mordred: oooh ooh i know lets use GENTOO! /s [nothing against gentoo, just not in this case] | 15:55 |
| mordred | hahaha | 15:55 |
| 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 |
| kmalloc | so some infrastructure bits for a local virt stack can't be stood up | 15:57 |
| kmalloc | don't get me wrong, I like alpine's lightweight nature. | 15:57 |
| kmalloc | I feel like it's a little too fast-and-loose with what is added to it. | 15:57 |
| kmalloc | somewhere between alpine and debian is where I'd like to be. (Ubuntu is not the answer) | 15:58 |
| kmalloc | debian stable* | 15:58 |
| *** e0ne has quit IRC | 16:33 | |
| *** ttsiouts has quit IRC | 16:38 | |
| *** ttsiouts has joined #openstack-sdks | 16:39 | |
| *** ttsiouts has quit IRC | 16:43 | |
| kmalloc | mordred: +2 on rate limit in KSA | 16:46 |
| kmalloc | it feels like it really belongs there as much as I wish it could be higher up. | 16:47 |
| mordred | kmalloc: woot. (pending tests, of course) | 16:47 |
| kmalloc | and that is due to retry logic | 16:47 |
| kmalloc | yes. | 16:47 |
| mordred | kmalloc: yah. the retry logic is the real kicker | 16:47 |
| kmalloc | must have tests before +A. | 16:47 |
| mordred | kmalloc: sort of like why discovery needed to be there too | 16:47 |
| kmalloc | functional tests are fine (doesn't need to be unit) | 16:47 |
| mordred | since it needes to be there for keystone _anyway_ for auth to work :) | 16:47 |
| kmalloc | it needs to be tested so we don't break | 16:47 |
| kmalloc | unit tests of that is going to be hard(tm) | 16:48 |
| mordred | oh - yeah. seriously | 16:48 |
| mordred | I *think* I have an idea of how to unit test it too | 16:48 |
| kmalloc | cool | 16:48 |
| kmalloc | but like i said, i'm happy with that simply being functional | 16:49 |
| kmalloc | you *might* be able to get away with unit tests utilizing freezegun | 16:49 |
| kmalloc | and stepping forward on ticks and a very low rate (e.g. 1 per multiple seconds) | 16:49 |
| kmalloc | concurrency could be tested via just objects that explicitly manage when they take action. | 16:50 |
| kmalloc | but functional might be so much easier | 16:50 |
| Shrews | mordred: comment on https://review.openstack.org/614832 | 17:00 |
| openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add --name-lookup-one-by-one option to server list https://review.openstack.org/568345 | 17:01 |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| *** jpich has quit IRC | 17:22 | |
| *** cdent has joined #openstack-sdks | 17:29 | |
| *** mriedem has quit IRC | 17:30 | |
| *** mriedem has joined #openstack-sdks | 17:35 | |
| * dtroyer finally made it through the last 2 months of osc queue… | 17:48 | |
| dtroyer | smcginnis: thanks for all that cleanup, I wish we could merge it today :) | 17:49 |
| dtroyer | I'm thinking to release master, rocky and queens early-mid next week… | 17:50 |
| openstackgerrit | Merged openstack/python-openstackclient master: Address issues from volume backend commands https://review.openstack.org/610161 | 18:00 |
| cdent | dtroyer: see you in berlin? | 18:03 |
| -openstackstatus- NOTICE: OpenStack infra's mirror nodes stopped accepting connections on ports 8080, 8081, and 8082. We will notify when this is fixed and jobs can be rechecked if they failed to communicate with a mirror on these ports. | 18:10 | |
| dtroyer | cdent: yes, I land Sunday | 18:10 |
| cdent | sweet | 18:10 |
| *** cdent has quit IRC | 18:24 | |
| *** e0ne has joined #openstack-sdks | 18:34 | |
| mordred | Shrews: well, clearly I'm just stupid | 18:34 |
| smcginnis | dtroyer: No worries. I'll try to keep those cleanup's rebased as conflicts come up so they are ready to go when the time is right. | 18:36 |
| dtroyer | smcginnis: there is an osc4 feature branch that has some work in it already. I need to merge master back in to that, but after that is done we could just merge those into the feature branch | 18:38 |
| smcginnis | dtroyer: Yeah, that makes sense if we have a whole batch of major-version inducing changes. | 18:38 |
| smcginnis | If you remember, let me know when a good point would be for that and I can rebase those over to the feature branch. | 18:39 |
| dtroyer | will do. I keep threatening to do it tomorrow :) | 18:47 |
| openstackgerrit | Merged openstack/python-openstackclient master: Make use of keystoneauth service-type filtering for versions https://review.openstack.org/585030 | 18:48 |
| -openstackstatus- NOTICE: The firewall situation with ports 8080, 8081, and 8082 on mirror nodes has been resolved. You can recheck jobs that have failed to communicate to the mirrors on those ports now. | 18:54 | |
| *** bobh has quit IRC | 19:04 | |
| *** e0ne has quit IRC | 19:05 | |
| *** gtmanfred has quit IRC | 19:17 | |
| *** gtmanfred has joined #openstack-sdks | 19:20 | |
| *** bobh has joined #openstack-sdks | 19:25 | |
| Shrews | mordred: yeah, well i didn't know if that was some sort of python trickery i wasn't aware of, or... your reason :-P | 19:32 |
| mordred | Shrews: definitely my reason | 19:33 |
| *** finucannot is now known as stephenfin | 19:35 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Add --key-name and --key-unset option for server rebuild API. https://review.openstack.org/576422 | 19:41 |
| *** cdent has joined #openstack-sdks | 20:24 | |
| *** bobh has quit IRC | 20:59 | |
| *** bobh has joined #openstack-sdks | 21:03 | |
| *** bobh has quit IRC | 21:09 | |
| *** cdent has quit IRC | 21:13 | |
| *** dave-mccowan has quit IRC | 21:35 | |
| *** ttsiouts has joined #openstack-sdks | 21:38 | |
| *** ttsiouts has quit IRC | 21:43 | |
| *** ttsiouts has joined #openstack-sdks | 21:53 | |
| *** gildub has joined #openstack-sdks | 22:00 | |
| openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | 22:28 |
| *** bobh has joined #openstack-sdks | 22:44 | |
| openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | 22:47 |
| *** bobh has quit IRC | 22:50 | |
| openstackgerrit | Merged openstack/osc-lib master: add lib-forward-testing-python3 test job https://review.openstack.org/594285 | 23:17 |
| *** ttsiouts has quit IRC | 23:17 | |
| *** mriedem has quit IRC | 23:18 | |
| *** markvoelker has quit IRC | 23:41 | |
| *** markvoelker has joined #openstack-sdks | 23:41 | |
| *** markvoelker has quit IRC | 23:46 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!