)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ba6183e252353b6a08d3e560e7a3ced2841444d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7c5b8270_6b6896bb","updated":"2023-06-27 01:05:22.000000000","message":"recheck tempest","commit_id":"854d79fddb1416c82715155ca80d4c3480c1c1cb"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"32ec5d4259e25cc96b736830af366fe17a82be49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8aa425c7_e7dfab75","updated":"2023-06-26 00:50:55.000000000","message":"recheck tempest","commit_id":"854d79fddb1416c82715155ca80d4c3480c1c1cb"}],"manifests/keystone/authtoken.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"253dab5d6507a78da83f4fcfcd96a922068cd7dc","unresolved":false,"context_lines":[{"line_number":9,"context_line":"#   Defaults to \u0027watcher\u0027"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*password*]"},{"line_number":12,"context_line":"#   (Optional) Password to create for the service user"},{"line_number":13,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]"},{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*auth_url*]"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"9b752812_3228a254","line":12,"updated":"2023-06-22 14:38:18.000000000","message":"So if this causes a failure if not provided, it\u0027s not exactly optional is it. I thought we dropped the os_service_default elsewhere for passwords when it was required.","commit_id":"dcb8730565079f5c521b84598430b2badfa94353"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c9ba073fdda362821423a3d56e7f5871455eea92","unresolved":false,"context_lines":[{"line_number":9,"context_line":"#   Defaults to \u0027watcher\u0027"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*password*]"},{"line_number":12,"context_line":"#   (Optional) Password to create for the service user"},{"line_number":13,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]"},{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*auth_url*]"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"76fc31f3_c701b24d","line":12,"in_reply_to":"3d0f3acf_fc56029b","updated":"2023-06-23 01:19:52.000000000","message":"I didn\u0027t make this parameter required following the implementations still left in the other modules but probably it\u0027s time to fix it in all modules. I\u0027ll push changes to the other modules.","commit_id":"dcb8730565079f5c521b84598430b2badfa94353"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"cf9721d475b9d28aad5e6dbdb5833445611087ae","unresolved":false,"context_lines":[{"line_number":9,"context_line":"#   Defaults to \u0027watcher\u0027"},{"line_number":10,"context_line":"#"},{"line_number":11,"context_line":"# [*password*]"},{"line_number":12,"context_line":"#   (Optional) Password to create for the service user"},{"line_number":13,"context_line":"#   Defaults to $facts[\u0027os_service_default\u0027]"},{"line_number":14,"context_line":"#"},{"line_number":15,"context_line":"# [*auth_url*]"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"3d0f3acf_fc56029b","line":12,"in_reply_to":"9b752812_3228a254","updated":"2023-06-22 15:02:05.000000000","message":"Let\u0027s just make it required I guess like in other places, it shouldn\u0027t really be optional not that it hurts if it is","commit_id":"dcb8730565079f5c521b84598430b2badfa94353"}]}