)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"cb482e8a77d41f7b0385539a83e0cad1e465ce10","unresolved":true,"context_lines":[{"line_number":9,"context_line":"In MemcachedDebug mode, also log debug for cache keys,"},{"line_number":10,"context_line":"get/set/delete/etc calls, with the keys/values."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"As this may expose token strings in the log, detach that from the"},{"line_number":13,"context_line":"global Debug paramter."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also switch multinode CI scenarios to use MemcachedDebug."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"1e77a1b9_07f185e5","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":22},"updated":"2022-03-17 12:45:54.000000000","message":"Debug logs from oslo.cache is disabled by default, and you should set oslo.cache\u003dDEBUG in [DEFAULT] default_log_levels","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"1add43ad9d2d561e71112bfaf52835588f28f813","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In MemcachedDebug mode, also log debug for cache keys,"},{"line_number":10,"context_line":"get/set/delete/etc calls, with the keys/values."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"As this may expose token strings in the log, detach that from the"},{"line_number":13,"context_line":"global Debug paramter."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also switch multinode CI scenarios to use MemcachedDebug."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"87986655_cc375bc6","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":22},"in_reply_to":"1e77a1b9_07f185e5","updated":"2022-03-17 16:57:06.000000000","message":"Done","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"7cfad339b15a493a626cbbfc28df1220dd03c118","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1c1439cd_878b3f56","updated":"2022-03-08 23:19:58.000000000","message":"It seems cache backend for nova is not properly enabled now, which I\u0027m fixing in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/832708 .\n\nAlso, I can\u0027t find out the debug log enabled by this change in keystone . Do you mind pointing some examples ? My concern is that we use cache backend for keystone as well and this might expose token string in log.","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"2a47791ce84bd313a05c5fe3bf4bdffe500c2fb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"78b03958_fd3dc71d","updated":"2022-03-08 18:43:42.000000000","message":"Looks like a good addition","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"95df8e1635c3ad42db52c3398977758e84d19b3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aec1dee9_8f0e03f3","updated":"2022-03-14 14:08:11.000000000","message":"recheck to see how it goes after https://review.opendev.org/c/openstack/tripleo-heat-templates/+/832708 merged","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"88a6d63a72b14a168d65f810f8c9de7345b54a5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a65ad565_2a7cdb13","in_reply_to":"1c1439cd_878b3f56","updated":"2022-03-14 14:07:29.000000000","message":"Debug settings are not for production use cases normally... If there are token strings shown in debug logs, that should be fixed on the keystone side.","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"1b93e1e514831337568c9cc9cad9b33f3de684ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"08732ae3_1e587d1b","updated":"2022-03-17 09:34:20.000000000","message":"PTAL","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"c37ff488f1ac68fc84c170b4205fcec749c3aa5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3fdfb6c5_429fecc4","updated":"2022-03-15 12:45:16.000000000","message":"PTAL","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"13b9dcb7a048d61714dcf7a5e16a543f6b02ec38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e570527c_1957b9d0","updated":"2022-03-15 12:46:56.000000000","message":"full-ci-recheck experimental","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"e58342b471f79d0d9a560224032756b372ec5286","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"abddd303_c77b08c0","updated":"2022-03-18 14:57:20.000000000","message":"@Takashi, folks, it works now [0], but is far from ideal (it overrides default log levels for all services that use oslo.log). But that\u0027s for debug purposes only anyway... We\u0027ll smooth this out as Takashi suggested to allow oslo.log merging from extra_log_level_defaults (today this is internal and not configurable for services)\n\n[0] https://d4afc528cd07cb57fe41-bedf8339247cc3e592c35ad8566e99fe.ssl.cf2.rackcdn.com/832506/8/check/tripleo-ci-centos-9-scenario012-standalone/cb990eb/logs/undercloud/var/log/containers/memcached/memcached.log","commit_id":"cfd7b4b5326429ae8f256a3df69d89724ddc0efe"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"48e91bd9d73fb00db3bf0f48ad0bcd5bba5fb8fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1a1d4b38_1d84dcc0","updated":"2022-03-18 15:02:48.000000000","message":"hm, actually unpatched memcached debug logs look the same and already provide get/set calls!\ncompare it to https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_db5/833164/1/check/tripleo-ci-centos-9-standalone/db51f05/logs/undercloud/var/log/containers/memcached/memcached.log","commit_id":"cfd7b4b5326429ae8f256a3df69d89724ddc0efe"}],"deployment/memcached/memcached-container-puppet.yaml":[{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"b86798b9f186be06600ac831e9b40d906d25ed3e","unresolved":true,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"4a44ea94_09cd69a5","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"updated":"2022-03-08 11:24:38.000000000","message":"We should set these in the respective services I guess.","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"31a82e72eb1c2b11f824c60bb480826bb08de4c9","unresolved":true,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e75264b0_865f4bd1","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"in_reply_to":"05eb7821_f02336d4","updated":"2022-03-08 12:25:29.000000000","message":"do you mean the service_config_settings here, or in the corresponding services templates?","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"2d8c091a3ddd3ff6935ede2c6318a9eddc05db96","unresolved":false,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"2ebbc55c_7d7efafa","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"in_reply_to":"48beba3a_0040e187","updated":"2022-03-14 14:29:26.000000000","message":"Done","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"dd3b6cfccbb9d44019d1a4469c2817e870fc9c65","unresolved":true,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"05eb7821_f02336d4","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"in_reply_to":"4a44ea94_09cd69a5","updated":"2022-03-08 12:22:13.000000000","message":"fair point, although we still use a flat hiera data for all roles iirc?","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"cf7b0b801f74d3532b1f8da9bb8e4353f7169f41","unresolved":true,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"eee1ccfa_f562ecd0","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"in_reply_to":"75fe843c_601f7844","updated":"2022-03-08 23:04:02.000000000","message":"This should be added to service_config_settings or global_config_settings, otherwise these parameters have no effect when memecached and api services are running in different nodes.\n\nAs dvd mentioned, implementing this in each template is another option.","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"90857e163a2911d227ed8d7d6f698e8dec8bf5ba","unresolved":true,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"75fe843c_601f7844","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"in_reply_to":"e75264b0_865f4bd1","updated":"2022-03-08 15:05:24.000000000","message":"I believe we should add this in the common deployment template for each one of the services, including the condition [1]. I guess we can keep oslo::cache here though.\n\nOther than that, nice idea as usual Bogdan!\n\n[1]\n~~~\n  memcached_debug:\n    or:\n      - {get_param: Debug}\n      - {get_param: MemcachedDebug}\n~~~","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"3a3ab40cd7ed22a8edf3bd908630eebee808297b","unresolved":true,"context_lines":[{"line_number":246,"context_line":"              tripleo::profile::base::memcached::certificate_specs:"},{"line_number":247,"context_line":"                service_certificate: \u0027/etc/pki/tls/certs/memcached.crt\u0027"},{"line_number":248,"context_line":"                service_key: \u0027/etc/pki/tls/private/memcached.key\u0027"},{"line_number":249,"context_line":"          - if:"},{"line_number":250,"context_line":"            - memcached_debug"},{"line_number":251,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":252,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":253,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":254,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":255,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":256,"context_line":"      service_config_settings:"},{"line_number":257,"context_line":"        collectd:"},{"line_number":258,"context_line":"            tripleo.collectd.plugins.memcached:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"48beba3a_0040e187","line":255,"range":{"start_line":249,"start_character":1,"end_line":255,"end_character":52},"in_reply_to":"eee1ccfa_f562ecd0","updated":"2022-03-14 14:22:38.000000000","message":"I\u0027ll use global hiera config, since the service config one is too much granular (tens of services to list there, like nova_foo, neutron_bar, etc)","commit_id":"256968e652e08ea78e9176aa01c02cd8176a32e9"},{"author":{"_account_id":8833,"name":"Rabi Mishra","email":"ramishra@redhat.com","username":"rabi"},"change_message_id":"1c356bfc2535f9c13680c256e3cfc969ae4c76e4","unresolved":true,"context_lines":[{"line_number":272,"context_line":"              memcached_authtoken_port: {get_param: MemcachedPort}"},{"line_number":273,"context_line":"          - if:"},{"line_number":274,"context_line":"            - {get_param: MemcachedDebug}"},{"line_number":275,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":276,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":277,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":278,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":279,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":280,"context_line":"      # BEGIN DOCKER SETTINGS"},{"line_number":281,"context_line":"      puppet_config:"},{"line_number":282,"context_line":"        config_volume: \u0027memcached\u0027"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"faf2e0c4_f804d89b","line":279,"range":{"start_line":275,"start_character":1,"end_line":279,"end_character":52},"updated":"2022-03-17 10:24:35.000000000","message":"So we decided to keep them here and not in service base templates? We would be setting the hieras irrespective of services in role or not.","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"1add43ad9d2d561e71112bfaf52835588f28f813","unresolved":false,"context_lines":[{"line_number":272,"context_line":"              memcached_authtoken_port: {get_param: MemcachedPort}"},{"line_number":273,"context_line":"          - if:"},{"line_number":274,"context_line":"            - {get_param: MemcachedDebug}"},{"line_number":275,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":276,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":277,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":278,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":279,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":280,"context_line":"      # BEGIN DOCKER SETTINGS"},{"line_number":281,"context_line":"      puppet_config:"},{"line_number":282,"context_line":"        config_volume: \u0027memcached\u0027"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"431e6bc4_44a97595","line":279,"range":{"start_line":275,"start_character":1,"end_line":279,"end_character":52},"in_reply_to":"2c2e036b_112d4e58","updated":"2022-03-17 16:57:06.000000000","message":"Done","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"71a092e4882e5bc8cbab59bcabfb3d2ba4d00c8a","unresolved":true,"context_lines":[{"line_number":272,"context_line":"              memcached_authtoken_port: {get_param: MemcachedPort}"},{"line_number":273,"context_line":"          - if:"},{"line_number":274,"context_line":"            - {get_param: MemcachedDebug}"},{"line_number":275,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":276,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":277,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":278,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":279,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":280,"context_line":"      # BEGIN DOCKER SETTINGS"},{"line_number":281,"context_line":"      puppet_config:"},{"line_number":282,"context_line":"        config_volume: \u0027memcached\u0027"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"2c2e036b_112d4e58","line":279,"range":{"start_line":275,"start_character":1,"end_line":279,"end_character":52},"in_reply_to":"ab1be168_afd13b9c","updated":"2022-03-17 12:36:34.000000000","message":"Usage of oslo::cache::* is invalid because oslo::cache is not a class but a resource type. A resource type is used as a kind of template in individual modules, and its parameters can\u0027t be directly set using hieradata.\n\nAlso, we don\u0027t include neutron::cache so the defintion has no effect.","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"4d7d2622766be1f826ef6596cc19be7d39b7b6c3","unresolved":true,"context_lines":[{"line_number":272,"context_line":"              memcached_authtoken_port: {get_param: MemcachedPort}"},{"line_number":273,"context_line":"          - if:"},{"line_number":274,"context_line":"            - {get_param: MemcachedDebug}"},{"line_number":275,"context_line":"            - nova::cache::debug_cache_backend: true"},{"line_number":276,"context_line":"              neutron::cache::debug_cache_backend: true"},{"line_number":277,"context_line":"              oslo::cache::debug_cache_backend: true"},{"line_number":278,"context_line":"              keystone::cache::debug_cache_backend: true"},{"line_number":279,"context_line":"              heat::cache::debug_cache_backend: true"},{"line_number":280,"context_line":"      # BEGIN DOCKER SETTINGS"},{"line_number":281,"context_line":"      puppet_config:"},{"line_number":282,"context_line":"        config_volume: \u0027memcached\u0027"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ab1be168_afd13b9c","line":279,"range":{"start_line":275,"start_character":1,"end_line":279,"end_character":52},"in_reply_to":"faf2e0c4_f804d89b","updated":"2022-03-17 10:35:19.000000000","message":"I would go this path for simplicity sake.","commit_id":"aec02a6336357d427dd28ef6372f0bcb78b80dde"}]}