| opendevreview | Takashi Kajinami proposed openstack/keystone master: Fix missing decoding https://review.opendev.org/c/openstack/keystone/+/1011168 | 01:32 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/python-keystoneclient master: Fix 403 caused by domain id mismatch https://review.opendev.org/c/openstack/python-keystoneclient/+/1011196 | 03:36 |
| opendevreview | Takashi Kajinami proposed openstack/python-keystoneclient master: Add minimum pyproject.toml https://review.opendev.org/c/openstack/python-keystoneclient/+/1011161 | 03:49 |
| opendevreview | Takashi Kajinami proposed openstack/python-keystoneclient master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/python-keystoneclient/+/1011162 | 03:49 |
| opendevreview | Takashi Kajinami proposed openstack/python-keystoneclient master: Add openstack-python3-next-jobs job template https://review.opendev.org/c/openstack/python-keystoneclient/+/1011163 | 03:50 |
| opendevreview | Takashi Kajinami proposed openstack/python-keystoneclient master: Simplify tox.ini https://review.opendev.org/c/openstack/python-keystoneclient/+/1011164 | 03:50 |
| tkajinam | fyi: I happened to discover broken functional tests in keystoneclient https://review.opendev.org/c/openstack/python-keystoneclient/+/1011196 fixes it | 10:46 |
| tkajinam | xek, if you are still around, do you mind reviewing https://review.opendev.org/c/openstack/python-keystoneclient/+/1011196 ? | 14:40 |
| opendevreview | Nathan Oyler proposed openstack/keystone master: Handle projects acting as domains in include_names https://review.opendev.org/c/openstack/keystone/+/1011331 | 15:13 |
| notq | hello, what is the process for getting a bug in? I have made an issue, as well as an open change. Just looking for any advice on process. https://review.opendev.org/c/openstack/keystone/+/1011331 | 15:20 |
| opendevreview | Mauricio Harley proposed openstack/keystone master: Increase PBKDF2-SHA512 default iterations to 600,000 https://review.opendev.org/c/openstack/keystone/+/991059 | 16:08 |
| tkajinam | notq, the general guidance is filing a bug and associate your fix with the bug using closes-bug tag, but quickly checking the commit I think bug and the fix are well-written and have nothing obviously wrong. | 16:14 |
| tkajinam | notq, so my suggestion would be just be patient for a while until keystone cores get chance/time to review it. keep in mind that most of the core teams are very small these days and don't have quite large review bandwidth. | 16:16 |
| tkajinam | notq, in case you haven't heard any feedback for long them bringing it to weekly irc meeting sometimes helps | 16:16 |
| tkajinam | We had a busy PTG week last week so I assume people are taking some break this week, though | 16:17 |
| opendevreview | Merged openstack/python-keystoneclient master: Fix 403 caused by domain id mismatch https://review.opendev.org/c/openstack/python-keystoneclient/+/1011196 | 16:21 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Remove unused catalog conversion logic for v3 API https://review.opendev.org/c/openstack/keystonemiddleware/+/1011352 | 17:29 |
| notq | tkajinma - thank you kindly for explaining the process. | 17:32 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Fix outdated comment about catalog data https://review.opendev.org/c/openstack/keystonemiddleware/+/1011353 | 17:41 |
| opendevreview | Merged openstack/keystonemiddleware master: Make cache encryption options generic https://review.opendev.org/c/openstack/keystonemiddleware/+/1006513 | 17:45 |
| opendevreview | Merged openstack/keystonemiddleware master: Add oslo.cache backend support https://review.opendev.org/c/openstack/keystonemiddleware/+/979374 | 17:56 |
| notq | question regarding if a new pr would be reasonable, for 10 years keystonemiddleware has returned nonstandard cadf events. In that time, I have always transformed them after to match the openstack audit middleware which uses the DMTF CADF Specification. Due to complainance, I am no longer allowed to transform them to maintain consistency. | 18:19 |
| notq | example Keystonemiddleware "initiator.user_id" - CADF compliant "initiator.id", also username (name), domain_name (domain), action Created.user (action create/user) etc. Is this something I can reasonable expect would be received okay? Should I go ahead and make the issue and code change? | 18:21 |
| cardoe | I would open an issue. | 20:03 |
| opendevreview | Merged openstack/keystonemiddleware master: Fix outdated comment about catalog data https://review.opendev.org/c/openstack/keystonemiddleware/+/1011353 | 20:33 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!