openstackgerrit | Merged openstack/requirements master: Updated from generate-constraints https://review.opendev.org/678389 | 00:37 |
---|---|---|
*** hongbin has joined #openstack-requirements | 00:51 | |
*** brinzhang has joined #openstack-requirements | 00:57 | |
*** brinzhang has quit IRC | 03:26 | |
*** brinzhang has joined #openstack-requirements | 03:27 | |
*** hongbin has quit IRC | 04:02 | |
*** udesale has joined #openstack-requirements | 04:05 | |
*** beekneemech has quit IRC | 05:16 | |
*** bnemec has joined #openstack-requirements | 05:20 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/678464 | 06:19 |
*** mnasiadka has joined #openstack-requirements | 08:31 | |
mnasiadka | Hi there | 08:31 |
mnasiadka | It seems ansible-runner===1.3.5 is in upper-constraints.txt - but does not exist... | 08:32 |
mnasiadka | ok, found that: https://review.opendev.org/#/c/678317/ | 08:33 |
*** ralonsoh has joined #openstack-requirements | 08:40 | |
*** strigazi has joined #openstack-requirements | 08:58 | |
*** brinzhang_ has joined #openstack-requirements | 09:17 | |
*** brinzhang has quit IRC | 09:21 | |
*** udesale has quit IRC | 11:04 | |
*** ccamacho has joined #openstack-requirements | 11:21 | |
*** jroll has quit IRC | 11:44 | |
*** jroll has joined #openstack-requirements | 11:45 | |
*** e0ne has joined #openstack-requirements | 12:01 | |
*** e0ne has quit IRC | 12:59 | |
*** e0ne has joined #openstack-requirements | 13:05 | |
*** shilpasd has joined #openstack-requirements | 13:46 | |
shilpasd | Hi All: need discussion related to python3-train patch for masakari-monitors https://review.opendev.org/#/c/669387/ for py27/py36 and py37, here observed 'import libvirt' is an issue, seems libvirt-python package is an issue | 13:49 |
shilpasd | pl throw few pointers to resolve this | 13:49 |
*** e0ne has quit IRC | 13:51 | |
efried | shilpasd: Have you tried asking about this in -infra? | 14:16 |
shilpasd | efried & prometheanfire: nope, will do that | 14:18 |
shilpasd | efried: thanks will check with infra team and get back to you | 14:18 |
efried | shilpasd: Clearly the generic python tox template is bringing in something that masakari isn't set up to handle. It may be a matter of adding a new requirement in your test-requirements.txt, but I'm not sure. | 14:19 |
*** e0ne has joined #openstack-requirements | 14:22 | |
*** e0ne has quit IRC | 14:23 | |
*** e0ne has joined #openstack-requirements | 14:24 | |
prometheanfire | smcginnis: mind looking at https://review.opendev.org/678317 | 14:26 |
smcginnis | Pulling releases, great. | 14:28 |
shilpasd | efried: ok, on #openstack-infra, got the solution https://opendev.org/openstack/project-config/src/branch/master/nodepool/elements/bindep-fallback.txt#L51, will check and get back to you | 14:28 |
prometheanfire | thanks | 14:30 |
prometheanfire | smcginnis: yep, I kinda want to make that criteria for advcotating it's removal from openstack (global-reqs) | 14:30 |
*** zigo has joined #openstack-requirements | 14:33 | |
*** e0ne has quit IRC | 15:11 | |
openstackgerrit | Andrii Ostapenko proposed openstack/requirements master: Roll back ansible-runner to 1.3.4 https://review.opendev.org/678590 | 15:13 |
openstackgerrit | Eric Fried proposed openstack/requirements master: Blacklist eventlet 0.25.0 https://review.opendev.org/678078 | 15:21 |
openstackgerrit | Eric Fried proposed openstack/requirements master: Uncap kombu https://review.opendev.org/678245 | 15:21 |
*** e0ne has joined #openstack-requirements | 15:23 | |
*** brinzhang_ has quit IRC | 15:35 | |
*** brinzhang_ has joined #openstack-requirements | 15:36 | |
*** e0ne has quit IRC | 15:38 | |
*** e0ne has joined #openstack-requirements | 15:40 | |
*** efried has quit IRC | 15:42 | |
*** efried has joined #openstack-requirements | 15:51 | |
*** e0ne has quit IRC | 16:02 | |
prometheanfire | smcginnis: mind looking at https://review.opendev.org/678078 too ? | 16:39 |
*** shilpasd has quit IRC | 17:16 | |
*** ralonsoh has quit IRC | 17:26 | |
*** e0ne has joined #openstack-requirements | 17:59 | |
*** e0ne has quit IRC | 18:04 | |
openstackgerrit | Merged openstack/requirements stable/queens: Retire swauth https://review.opendev.org/678375 | 18:46 |
openstackgerrit | Merged openstack/requirements stable/rocky: Retire swauth https://review.opendev.org/678376 | 18:46 |
openstackgerrit | Merged openstack/requirements stable/ocata: Retire swauth https://review.opendev.org/678373 | 18:46 |
openstackgerrit | Merged openstack/requirements stable/pike: Retire swauth https://review.opendev.org/678374 | 18:46 |
openstackgerrit | Merged openstack/requirements stable/newton: Retire swauth https://review.opendev.org/678371 | 18:46 |
openstackgerrit | Merged openstack/requirements master: Retire swauth https://review.opendev.org/678367 | 18:46 |
openstackgerrit | Merged openstack/requirements master: update constraint for openstacksdk to new release 0.35.0 https://review.opendev.org/678207 | 18:47 |
openstackgerrit | Merged openstack/requirements master: ansible-runner release 1.3.5 was pulled. 1.3.6 will be released next https://review.opendev.org/678317 | 19:13 |
*** efried has quit IRC | 19:24 | |
*** efried has joined #openstack-requirements | 20:02 | |
*** efried has quit IRC | 20:07 | |
*** efried has joined #openstack-requirements | 20:13 | |
*** lbragstad has joined #openstack-requirements | 21:24 | |
lbragstad | o/ we're seeing some issues with a version of bandit in keystonemiddleware's stable/queens gate | 21:25 |
lbragstad | after checking https://opendev.org/openstack/requirements/raw/branch/stable/queens/upper-constraints.txt i see that bandit isn't in that list | 21:26 |
lbragstad | if we were to cap bandit for stable/queens, is that where we'd do it? | 21:26 |
*** tonyb[m] has quit IRC | 21:35 | |
*** tonyb has quit IRC | 21:39 | |
lbragstad | cc prometheanfire smcginnis ^ | 21:55 |
openstackgerrit | Merged openstack/requirements stable/stein: Retire swauth https://review.opendev.org/678377 | 21:57 |
smcginnis | lbragstad: That's considered a linter, so I think we've typically avoided adding those. | 22:07 |
smcginnis | Let's see what prometheanfire says though. | 22:07 |
lbragstad | smcginnis ah - so should we just fix the errors we're seeing in stable/queens instead? | 22:12 |
lbragstad | for example - https://review.opendev.org/#/c/678636/1 | 22:13 |
smcginnis | lbragstad: That change looks fine as far as stable policy goes, so that might be the easiest approach. | 22:14 |
smcginnis | lbragstad: Was this due to bandit adding stuff that used to pass but now doesn't? | 22:14 |
smcginnis | If that's the case, that does seem like something that could happen over time and maybe we really should get that added to upper-constraints. | 22:15 |
prometheanfire | lbragstad: iirc that's a linter | 22:19 |
prometheanfire | you can add it to your requirements (a cap or whatever) and the bot should be happy with it | 22:20 |
lbragstad | prometheanfire so - i can put that in test-requirements instead? | 22:22 |
lbragstad | smcginnis yeah - 1.6.0 introduced a couple of additional security checks that trigger some false positives in keystonemiddleware | 22:22 |
lbragstad | (naming convention stuff) | 22:22 |
lbragstad | cc kmalloc | 22:24 |
kmalloc | wfm | 22:24 |
kmalloc | however you need to get it in our linters | 22:24 |
prometheanfire | yep | 22:27 |
lbragstad | sweet - thanks smcginnis prometheanfire and kmalloc | 22:45 |
lbragstad | updated - https://review.opendev.org/#/c/678696/1 | 22:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!