| *** benj_88 is now known as benj_8 | 00:39 | |
| notq | cardoe - thank you https://bugs.launchpad.net/keystone/+bug/2171456 | 03:02 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/keystonemiddleware master: Imported Translations from Zanata https://review.opendev.org/c/openstack/keystonemiddleware/+/1007362 | 03:57 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Adopt pre-commit (redux) https://review.opendev.org/c/openstack/keystonemiddleware/+/1006308 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Set id for services in catalog https://review.opendev.org/c/openstack/keystonemiddleware/+/879503 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Enable ignored ruff rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006497 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Add hacking to pre-commit rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006498 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Enable ruff G, LOG, UP rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006499 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Enable ignored ruff rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006500 | 13:59 |
| opendevreview | Stephen Finucane proposed openstack/keystonemiddleware master: Enable ignored ruff rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006500 | 14:04 |
| opendevreview | Grzegorz Grasza proposed openstack/keystone stable/2026.1: credentials: reject delegated tokens, restrict PATCH to blob only https://review.opendev.org/c/openstack/keystone/+/1002333 | 15:12 |
| opendevreview | Merged openstack/keystone master: ec2: enforce timestamp presence in _check_timestamp https://review.opendev.org/c/openstack/keystone/+/993907 | 16:56 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Generalize cache factory https://review.opendev.org/c/openstack/keystonemiddleware/+/1008576 | 17:03 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Deprecate built-in caching https://review.opendev.org/c/openstack/keystonemiddleware/+/1011521 | 17:03 |
| opendevreview | Merged openstack/keystonemiddleware master: Adopt pre-commit (redux) https://review.opendev.org/c/openstack/keystonemiddleware/+/1006308 | 17:21 |
| opendevreview | Merged openstack/keystonemiddleware master: Enable ignored ruff rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006497 | 17:21 |
| opendevreview | Merged openstack/keystonemiddleware master: Add hacking to pre-commit rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006498 | 17:21 |
| opendevreview | Merged openstack/keystonemiddleware master: Enable ruff G, LOG, UP rules https://review.opendev.org/c/openstack/keystonemiddleware/+/1006499 | 17:21 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Generalize cache factory https://review.opendev.org/c/openstack/keystonemiddleware/+/1008576 | 17:39 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Deprecate built-in caching https://review.opendev.org/c/openstack/keystonemiddleware/+/1011521 | 17:41 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Generalize cache factory https://review.opendev.org/c/openstack/keystonemiddleware/+/1008576 | 17:44 |
| opendevreview | Takashi Kajinami proposed openstack/keystonemiddleware master: Deprecate built-in caching https://review.opendev.org/c/openstack/keystonemiddleware/+/1011521 | 17:44 |
| Matko[m] | Hi! Question about keystonemiddleware.audit: were audit_map_file, service_name, ignore_req_list and ignore_path_list left unregistered in _AUDIT_OPTS on purpose? https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/audit/__init__.py#L44 | 17:50 |
| Matko[m] | AuditMiddleware reads them from the raw paste conf, but Config() first passes every paste key through _conf_values_type_convert() to convert it to the option's type. https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/_common/config.py#L83 | 17:50 |
| Matko[m] | Any key that isn't a registered option logs 'The option "audit_map_file" is not known to keystonemiddleware', so every deployment (like nova/cinder/neutron) with [filter:audit] gets that warning even though the value is used correctly. | 17:50 |
| tkajinam | Matko[m], these options are for audit middleware, not keystonemiddleware | 17:58 |
| tkajinam | s/keystonemiddleware/authtoken middleware/ | 17:58 |
| Matko[m] | Sorry, I mislinked | 17:58 |
| Matko[m] | or not? | 17:58 |
| tkajinam | oh wait I did misunderstand it | 17:59 |
| tkajinam | https://github.com/openstack/keystonemiddleware/blob/593e08bd2c1deaab620cb96efaf74d1dadb07e93/keystonemiddleware/audit/__init__.py#L76 | 17:59 |
| Matko[m] | When setting audit_map_file in [filter:audit] in paste config, there is a warning. | 18:00 |
| tkajinam | but I think we do register these here | 18:00 |
| tkajinam | in api-paste.ini, right ? | 18:00 |
| Matko[m] | yes | 18:01 |
| tkajinam | I vaguely remembered that's a common harmless symptom caused by the handling of both *.conf and *.ini . we typically see that in swift but the same can appear in the other services | 18:02 |
| Matko[m] | Although it's harmless, is there a way to not log a warning? or change it to debug maybe? As there is nothing an operator can do to address the warning. (Or register the option) | 18:04 |
| tkajinam | maybe we can change the log level of https://github.com/openstack/keystonemiddleware/blob/master/keystonemiddleware/_common/config.py#L59 | 18:08 |
| Matko[m] | yes, I think that would be reasonable. | 18:08 |
| tkajinam | the problem is not that keystonemiddleware does not register these options, but it attempts to load all options to paste.ini and warn about anything it doesn't know | 18:08 |
| tkajinam | keystonemiddleware should not register these audit options by itself becuase it doesn't use these options | 18:09 |
| tkajinam | but it should not complain that it finds that audit options in paste.ini. | 18:09 |
| Matko[m] | right, so it's back to what you said earlier: "symptom caused by the handling of both *.conf and *.ini" | 18:09 |
| tkajinam | yeah | 18:10 |
| tkajinam | if you want to silence that warning you can probably try setting these options in config files in stead. ([audit] section in nova.conf, for example) | 18:11 |
| tkajinam | but maybe useful if you could file a short bug so that we fix the log level eventually. | 18:11 |
| tkajinam | hmm sorry I noticed ignore_req_list is not loaded from nova.conf so it doesn't work | 18:12 |
| tkajinam | s/nova.conf/service conf/ | 18:12 |
| Matko[m] | ok, lets change the log level then? | 18:12 |
| Matko[m] | I need to run some errands. I'll open a bug later and create the change. thanks for your help! | 18:13 |
| tkajinam | +1 | 18:13 |
| opendevreview | Grzegorz Grasza proposed openstack/keystone stable/2025.2: credentials: reject delegated tokens, restrict PATCH to blob only https://review.opendev.org/c/openstack/keystone/+/1002334 | 19:49 |
| opendevreview | Grzegorz Grasza proposed openstack/keystone stable/2025.1: credentials: reject delegated tokens, restrict PATCH to blob only https://review.opendev.org/c/openstack/keystone/+/1002335 | 19:50 |
| opendevreview | Mathieu Gagné proposed openstack/keystonemiddleware master: Log unknown paste options at debug level https://review.opendev.org/c/openstack/keystonemiddleware/+/1011558 | 20:36 |
| Matko[m] | Any keystone core that can review this 1 byte change in keystone? thanks https://review.opendev.org/c/openstack/keystone/+/990574 | 20:39 |
| opendevreview | Mathieu Gagné proposed openstack/keystonemiddleware master: Log unknown paste options at debug level https://review.opendev.org/c/openstack/keystonemiddleware/+/1011558 | 22:08 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!