| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Replace remaining use of legacy fact https://review.opendev.org/c/openstack/puppet-horizon/+/942024 | 00:40 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Replace remaining use of legacy facts https://review.opendev.org/c/openstack/puppet-openstacklib/+/942025 | 00:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Replace remaining use of legacy fact https://review.opendev.org/c/openstack/puppet-horizon/+/942024 | 00:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstacklib master: Stop using absolute names for defined resource types https://review.opendev.org/c/openstack/puppet-openstacklib/+/941944 | 00:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Validate redirect_type using data type https://review.opendev.org/c/openstack/puppet-horizon/+/941943 | 00:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Horizon: Support customizing root_url https://review.opendev.org/c/openstack/puppet-openstack-integration/+/942026 | 00:48 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Use common implementation in litmus tests https://review.opendev.org/c/openstack/puppet-horizon/+/942027 | 00:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-horizon master: Use common implementation in litmus tests https://review.opendev.org/c/openstack/puppet-horizon/+/942027 | 01:24 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-aodh master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-aodh/+/942028 | 01:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-barbican master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-barbican/+/942029 | 01:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceilometer master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ceilometer/+/942030 | 01:40 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-cinder/+/942031 | 01:43 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-designate master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-designate/+/942032 | 01:45 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-glance/+/942033 | 01:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-heat/+/942034 | 01:48 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ironic/+/942035 | 01:51 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-keystone/+/942036 | 01:52 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-magnum master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-magnum/+/942037 | 01:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-manila/+/942038 | 01:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-mistral master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-mistral/+/942039 | 01:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-neutron/+/942040 | 01:58 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-nova/+/942041 | 02:00 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-octavia/+/942042 | 02:01 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-trove master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-trove/+/942043 | 02:04 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vitrage master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-vitrage/+/942044 | 02:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-watcher/+/942045 | 02:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-aodh master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-aodh/+/942028 | 02:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-barbican master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-barbican/+/942029 | 02:19 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-cinder/+/942031 | 02:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-designate master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-designate/+/942032 | 02:20 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-glance/+/942033 | 02:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-heat/+/942034 | 02:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ironic/+/942035 | 02:22 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-keystone/+/942036 | 02:29 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-magnum master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-magnum/+/942037 | 02:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-manila/+/942038 | 02:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-mistral master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-mistral/+/942039 | 02:31 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-neutron/+/942040 | 02:31 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-nova/+/942041 | 02:32 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-octavia/+/942042 | 02:34 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-octavia/+/942042 | 02:34 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-trove master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-trove/+/942043 | 02:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vitrage master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-vitrage/+/942044 | 02:35 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-watcher/+/942045 | 02:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ironic/+/942035 | 02:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceilometer master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ceilometer/+/942030 | 02:37 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-swift/+/942046 | 02:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Deprecate support for rabbit_heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-glance/+/942033 | 02:48 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Deprecate heartbeat_in_pthread parameter https://review.opendev.org/c/openstack/puppet-oslo/+/942047 | 02:50 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-aodh master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-aodh/+/942028 | 02:53 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-barbican master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-barbican/+/942029 | 02:53 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cinder master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-cinder/+/942031 | 02:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ceilometer master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ceilometer/+/942030 | 02:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-designate master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-designate/+/942032 | 02:54 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-glance master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-glance/+/942033 | 02:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-heat/+/942034 | 02:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-ironic master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-ironic/+/942035 | 02:55 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-keystone/+/942036 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-magnum master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-magnum/+/942037 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-manila master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-manila/+/942038 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-mistral master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-mistral/+/942039 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-neutron/+/942040 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-nova/+/942041 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-octavia/+/942042 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-trove master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-trove/+/942043 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-vitrage master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-vitrage/+/942044 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-watcher/+/942045 | 02:58 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-swift/+/942046 | 02:58 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-cloudkitty master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-cloudkitty/+/942049 | 02:59 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Deprecate heartbeat_in_pthread parameter https://review.opendev.org/c/openstack/puppet-oslo/+/942047 | 04:45 |
| d1nuc0m | Hi tkajinam I'm finally setting up again my testbed, tested https://review.opendev.org/c/openstack/keystone/+/925553 and it works (thanks again, ik it took time) | 08:44 |
| d1nuc0m | I'll probably propose some more changes as there are other hardcoded params such as OIDCScope and it's not possible to add other locations | 08:45 |
| d1nuc0m | (edit, I forgot I'm still using 2024.1) | 09:00 |
| tkajinam | d1nuc0m, ack and thank you, too ! | 09:18 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-keystone master: feat: allow to set OIDCScope https://review.opendev.org/c/openstack/puppet-keystone/+/942070 | 09:19 |
| d1nuc0m | Here we go :D | 09:20 |
| d1nuc0m | Btw, would it be possible to backport https://opendev.org/openstack/puppet-keystone/commit/e4acfc9f29e17d17ad736d733c4989b31a4d7799 and https://opendev.org/openstack/puppet-keystone/commit/81d2a80f0a75dccb0c7aabdef7abd91b1f375f20 to 2024.1? | 09:22 |
| d1nuc0m | I'll still have the testbed on it for a while | 09:22 |
| tkajinam | d1nuc0m, yeah we can try | 09:26 |
| tkajinam | because the change may not break the default behavior | 09:26 |
| d1nuc0m | tkajinam, thank you | 09:32 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-keystone master: feat: add openidc_scope param to set OIDCScope https://review.opendev.org/c/openstack/puppet-keystone/+/942070 | 09:44 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-openstacklib master: feat: add owner and group params for clouds.yaml https://review.opendev.org/c/openstack/puppet-openstacklib/+/942074 | 10:01 |
| d1nuc0m | As commented, I don't know how should I test for user/group ownership... | 10:02 |
| tkajinam | d1nuc0m, maybe you can add a test case with customized user/group to https://github.com/openstack/puppet-openstacklib/blob/master/spec/defines/openstacklib_clouds_spec.rb ? | 10:10 |
| tkajinam | Technically we can try adding an acceptance tests and assert file ownerhsip by the native serverspec interface but unit tests may be enough | 10:10 |
| d1nuc0m | tkajinam, yes, but shouldn't it require that the user is existing before the file is created and ownership assigned? | 10:12 |
| tkajinam | d1nuc0m, unit tests only checks interface of resource calls. Because file resource may require users/groups using implicit autorequire (instead of resource dependencies) it does not requite actual user/group resources defined | 10:16 |
| tkajinam | contrary to acceptance tests which actually runs manifests | 10:17 |
| tkajinam | https://opendev.org/openstack/puppet-openstacklib/src/branch/master/spec/defines/openstacklib_policy_default_spec.rb#L13-L14 | 10:17 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-keystone master: feat: add openidc_scope param to set OIDCScope https://review.opendev.org/c/openstack/puppet-keystone/+/942070 | 10:21 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-swift/+/942046 | 10:22 |
| d1nuc0m | tkajinam, thanks, I thought it actually checked for the user existance | 10:22 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-openstacklib master: feat: add owner and group params for clouds.yaml https://review.opendev.org/c/openstack/puppet-openstacklib/+/942074 | 10:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-cookiecutter master: Unpin bundler https://review.opendev.org/c/openstack/puppet-openstack-cookiecutter/+/942081 | 11:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: feat: add openidc_scope param to set OIDCScope https://review.opendev.org/c/openstack/puppet-keystone/+/942070 | 12:00 |
| d1nuc0m | tkajinam, thanks for the tests | 12:40 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-openstacklib master: feat: add owner and group params for clouds.yaml https://review.opendev.org/c/openstack/puppet-openstacklib/+/942074 | 12:44 |
| *** tkajinam is now known as Guest9449 | 13:10 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: feat: add openidc_scope param to set OIDCScope https://review.opendev.org/c/openstack/puppet-keystone/+/942070 | 13:59 |
| tkajinam | added the release note and made a minor format fix | 13:59 |
| d1nuc0m | I had forgotten the notes | 14:07 |
| opendevreview | Francesco Di Nucci proposed openstack/puppet-openstacklib master: feat: add owner and group params for clouds.yaml https://review.opendev.org/c/openstack/puppet-openstacklib/+/942074 | 14:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-cookiecutter master: Stop using absolute names for defined resource types https://review.opendev.org/c/openstack/puppet-openstack-cookiecutter/+/941966 | 14:44 |
| -opendevstatus- NOTICE: nominations for the OpenStack PTL and TC positions are closing soon, for details see https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/7DKEV7IEHOTHED7RVEFG7WIDVUC4MY3Z/ | 16:00 | |
| opendevreview | Merged openstack/puppet-manila master: Stop using absolute names for defined resource types https://review.opendev.org/c/openstack/puppet-manila/+/941956 | 16:24 |
| opendevreview | Merged openstack/puppet-manila master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-manila/+/942038 | 16:24 |
| opendevreview | Merged openstack/puppet-cinder master: Stop using absolute names for defined resource types https://review.opendev.org/c/openstack/puppet-cinder/+/941945 | 16:39 |
| opendevreview | Merged openstack/puppet-neutron master: Ubuntu: Add support for separate services https://review.opendev.org/c/openstack/puppet-neutron/+/941840 | 16:41 |
| opendevreview | Merged openstack/puppet-glance master: Stop using absolute names for defined resource types https://review.opendev.org/c/openstack/puppet-glance/+/941950 | 16:44 |
| opendevreview | Merged openstack/puppet-glance master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-glance/+/942033 | 16:47 |
| opendevreview | Merged openstack/puppet-cinder master: Deprecate support for [oslo_messaging_rabbit] heartbeat_in_pthread https://review.opendev.org/c/openstack/puppet-cinder/+/942031 | 16:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!