Monday, 2019-08-26

openstackgerritMerged openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/67838900:37
*** hongbin has joined #openstack-requirements00:51
*** brinzhang has joined #openstack-requirements00:57
*** brinzhang has quit IRC03:26
*** brinzhang has joined #openstack-requirements03:27
*** hongbin has quit IRC04:02
*** udesale has joined #openstack-requirements04:05
*** beekneemech has quit IRC05:16
*** bnemec has joined #openstack-requirements05:20
openstackgerritOpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints  https://review.opendev.org/67846406:19
*** mnasiadka has joined #openstack-requirements08:31
mnasiadkaHi there08:31
mnasiadkaIt seems ansible-runner===1.3.5 is in upper-constraints.txt - but does not exist...08:32
mnasiadkaok, found that: https://review.opendev.org/#/c/678317/08:33
*** ralonsoh has joined #openstack-requirements08:40
*** strigazi has joined #openstack-requirements08:58
*** brinzhang_ has joined #openstack-requirements09:17
*** brinzhang has quit IRC09:21
*** udesale has quit IRC11:04
*** ccamacho has joined #openstack-requirements11:21
*** jroll has quit IRC11:44
*** jroll has joined #openstack-requirements11:45
*** e0ne has joined #openstack-requirements12:01
*** e0ne has quit IRC12:59
*** e0ne has joined #openstack-requirements13:05
*** shilpasd has joined #openstack-requirements13:46
shilpasdHi 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 issue13:49
shilpasdpl throw few pointers to resolve this13:49
*** e0ne has quit IRC13:51
efriedshilpasd: Have you tried asking about this in -infra?14:16
shilpasdefried & prometheanfire: nope, will do that14:18
shilpasdefried: thanks will check with infra team and get back to you14:18
efriedshilpasd: 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-requirements14:22
*** e0ne has quit IRC14:23
*** e0ne has joined #openstack-requirements14:24
prometheanfiresmcginnis: mind looking at https://review.opendev.org/67831714:26
smcginnisPulling releases, great.14:28
shilpasdefried: 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 you14:28
prometheanfirethanks14:30
prometheanfiresmcginnis: yep, I kinda want to make that criteria for advcotating it's removal from openstack (global-reqs)14:30
*** zigo has joined #openstack-requirements14:33
*** e0ne has quit IRC15:11
openstackgerritAndrii Ostapenko proposed openstack/requirements master: Roll back ansible-runner to 1.3.4  https://review.opendev.org/67859015:13
openstackgerritEric Fried proposed openstack/requirements master: Blacklist eventlet 0.25.0  https://review.opendev.org/67807815:21
openstackgerritEric Fried proposed openstack/requirements master: Uncap kombu  https://review.opendev.org/67824515:21
*** e0ne has joined #openstack-requirements15:23
*** brinzhang_ has quit IRC15:35
*** brinzhang_ has joined #openstack-requirements15:36
*** e0ne has quit IRC15:38
*** e0ne has joined #openstack-requirements15:40
*** efried has quit IRC15:42
*** efried has joined #openstack-requirements15:51
*** e0ne has quit IRC16:02
prometheanfiresmcginnis: mind looking at https://review.opendev.org/678078 too ?16:39
*** shilpasd has quit IRC17:16
*** ralonsoh has quit IRC17:26
*** e0ne has joined #openstack-requirements17:59
*** e0ne has quit IRC18:04
openstackgerritMerged openstack/requirements stable/queens: Retire swauth  https://review.opendev.org/67837518:46
openstackgerritMerged openstack/requirements stable/rocky: Retire swauth  https://review.opendev.org/67837618:46
openstackgerritMerged openstack/requirements stable/ocata: Retire swauth  https://review.opendev.org/67837318:46
openstackgerritMerged openstack/requirements stable/pike: Retire swauth  https://review.opendev.org/67837418:46
openstackgerritMerged openstack/requirements stable/newton: Retire swauth  https://review.opendev.org/67837118:46
openstackgerritMerged openstack/requirements master: Retire swauth  https://review.opendev.org/67836718:46
openstackgerritMerged openstack/requirements master: update constraint for openstacksdk to new release 0.35.0  https://review.opendev.org/67820718:47
openstackgerritMerged openstack/requirements master: ansible-runner release 1.3.5 was pulled. 1.3.6 will be released next  https://review.opendev.org/67831719:13
*** efried has quit IRC19:24
*** efried has joined #openstack-requirements20:02
*** efried has quit IRC20:07
*** efried has joined #openstack-requirements20:13
*** lbragstad has joined #openstack-requirements21:24
lbragstado/ we're seeing some issues with a version of bandit in keystonemiddleware's stable/queens gate21:25
lbragstadafter checking https://opendev.org/openstack/requirements/raw/branch/stable/queens/upper-constraints.txt i see that bandit isn't in that list21:26
lbragstadif we were to cap bandit for stable/queens, is that where we'd do it?21:26
*** tonyb[m] has quit IRC21:35
*** tonyb has quit IRC21:39
lbragstadcc prometheanfire smcginnis ^21:55
openstackgerritMerged openstack/requirements stable/stein: Retire swauth  https://review.opendev.org/67837721:57
smcginnislbragstad: That's considered a linter, so I think we've typically avoided adding those.22:07
smcginnisLet's see what prometheanfire says though.22:07
lbragstadsmcginnis ah - so should we just fix the errors we're seeing in stable/queens instead?22:12
lbragstadfor example - https://review.opendev.org/#/c/678636/122:13
smcginnislbragstad: That change looks fine as far as stable policy goes, so that might be the easiest approach.22:14
smcginnislbragstad: Was this due to bandit adding stuff that used to pass but now doesn't?22:14
smcginnisIf 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
prometheanfirelbragstad: iirc that's a linter22:19
prometheanfireyou can add it to your requirements (a cap or whatever) and the bot should be happy with it22:20
lbragstadprometheanfire so - i can put that in test-requirements instead?22:22
lbragstadsmcginnis yeah - 1.6.0 introduced a couple of additional security checks that trigger some false positives in keystonemiddleware22:22
lbragstad(naming convention stuff)22:22
lbragstadcc kmalloc22:24
kmallocwfm22:24
kmallochowever you need to get it in our linters22:24
prometheanfireyep22:27
lbragstadsweet - thanks smcginnis prometheanfire and kmalloc22:45
lbragstadupdated - https://review.opendev.org/#/c/678696/122:45

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!