frickler | stephenfin: gtema: pls check https://bugs.launchpad.net/openstacksdk/+bug/2046645 | 07:30 |
---|---|---|
opendevreview | Thomas Goirand proposed openstack/keystoneauth master: Python 3.12: do not use datetime.utcnow() https://review.opendev.org/c/openstack/keystoneauth/+/903860 | 09:03 |
frickler | zigo: let's move the bandit discussion here to include gtema and stephenfin. the issue isn't related to your patch, see e.g. https://zuul.opendev.org/t/openstack/build/487d03418bcc4ef88fb39dba53523b4f , but I don't know offhand why this doesn't fail the job | 09:53 |
opendevreview | Thomas Goirand proposed openstack/keystoneauth master: Python 3.12: do not use datetime.utcnow() https://review.opendev.org/c/openstack/keystoneauth/+/903860 | 10:05 |
opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Bump bandit https://review.opendev.org/c/openstack/keystoneauth/+/903868 | 10:20 |
opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Bump hacking, other flake8 plugins https://review.opendev.org/c/openstack/keystoneauth/+/903869 | 10:20 |
stephenfin | frickler: zigo: it's an outdated bandit ^ idk why the job wasn't failing though | 10:21 |
stephenfin | the tox target also fails locally | 10:22 |
stephenfin | ah, any exceptions result in the file being skipped, even if exception came from bandit rather than the file https://github.com/PyCQA/bandit/blob/1.5.1/bandit/core/manager.py#L286-L293 | 10:25 |
stephenfin | seems like a not-so-great idea but what do I know 🤷♂️ | 10:26 |
frickler | stephenfin: thx | 11:05 |
opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Remove resource_key for dns floating IP https://review.opendev.org/c/openstack/openstacksdk/+/903879 | 12:14 |
opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Remove resource_key for dns floating IP https://review.opendev.org/c/openstack/openstacksdk/+/903879 | 14:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!