)]}'
{"manifests/init.pp":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"d17d1ecb75a620570596eb52093189a4c4ea3f31","unresolved":false,"context_lines":[{"line_number":418,"context_line":"  }"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"  # NOTE(abishop): Remove the picks when cinder::api::keymgr_* are removed."},{"line_number":421,"context_line":"  $keymgr_backend_real \u003d pick($cinder::api::keymgr_backend,"},{"line_number":422,"context_line":"                              hiera(\u0027cinder::api::keymgr_backend\u0027, undef),"},{"line_number":423,"context_line":"                              $keymgr_backend)"},{"line_number":424,"context_line":"  $keymgr_encryption_api_url_real \u003d pick($cinder::api::keymgr_encryption_api_url,"},{"line_number":425,"context_line":"                                          hiera(\u0027cinder::api::keymgr_encryption_api_url\u0027, undef),"},{"line_number":426,"context_line":"                                          $keymgr_encryption_api_url)"},{"line_number":427,"context_line":"  $keymgr_encryption_auth_url_real \u003d pick($cinder::api::keymgr_encryption_auth_url,"},{"line_number":428,"context_line":"                                          hiera(\u0027cinder::api::keymgr_encryption_auth_url\u0027, undef),"},{"line_number":429,"context_line":"                                          $keymgr_encryption_auth_url)"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"  cinder_config {"},{"line_number":432,"context_line":"    \u0027DEFAULT/report_interval\u0027:                  value \u003d\u003e $report_interval;"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf51134e_1a624717","line":429,"range":{"start_line":421,"start_character":0,"end_line":429,"end_character":70},"updated":"2020-07-02 19:33:52.000000000","message":"hiera() part shouldn\u0027t be here","commit_id":"364addf42055c243de5aee8759e82c7e14c926b7"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"24183a93eac6806d5325388f0a0ece92abf2cfc8","unresolved":false,"context_lines":[{"line_number":418,"context_line":"  }"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"  # NOTE(abishop): Remove the picks when cinder::api::keymgr_* are removed."},{"line_number":421,"context_line":"  $keymgr_backend_real \u003d pick($cinder::api::keymgr_backend,"},{"line_number":422,"context_line":"                              hiera(\u0027cinder::api::keymgr_backend\u0027, undef),"},{"line_number":423,"context_line":"                              $keymgr_backend)"},{"line_number":424,"context_line":"  $keymgr_encryption_api_url_real \u003d pick($cinder::api::keymgr_encryption_api_url,"},{"line_number":425,"context_line":"                                          hiera(\u0027cinder::api::keymgr_encryption_api_url\u0027, undef),"},{"line_number":426,"context_line":"                                          $keymgr_encryption_api_url)"},{"line_number":427,"context_line":"  $keymgr_encryption_auth_url_real \u003d pick($cinder::api::keymgr_encryption_auth_url,"},{"line_number":428,"context_line":"                                          hiera(\u0027cinder::api::keymgr_encryption_auth_url\u0027, undef),"},{"line_number":429,"context_line":"                                          $keymgr_encryption_auth_url)"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"  cinder_config {"},{"line_number":432,"context_line":"    \u0027DEFAULT/report_interval\u0027:                  value \u003d\u003e $report_interval;"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf51134e_b531acef","line":429,"range":{"start_line":421,"start_character":0,"end_line":429,"end_character":70},"in_reply_to":"bf51134e_1a624717","updated":"2020-07-02 19:46:17.000000000","message":"I initially didn\u0027t have these (I also didn\u0027t think they would be necessar), but early attempts at this made me think the hiera binding wasn\u0027t happening when referencing vars in another module (cinder::api in this case).\n\nI will do some more tests.","commit_id":"364addf42055c243de5aee8759e82c7e14c926b7"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"77b7179f9c660be546598072b5c745fc8bc2cec8","unresolved":false,"context_lines":[{"line_number":418,"context_line":"  }"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":"  # NOTE(abishop): Remove the picks when cinder::api::keymgr_* are removed."},{"line_number":421,"context_line":"  $keymgr_backend_real \u003d pick($cinder::api::keymgr_backend,"},{"line_number":422,"context_line":"                              hiera(\u0027cinder::api::keymgr_backend\u0027, undef),"},{"line_number":423,"context_line":"                              $keymgr_backend)"},{"line_number":424,"context_line":"  $keymgr_encryption_api_url_real \u003d pick($cinder::api::keymgr_encryption_api_url,"},{"line_number":425,"context_line":"                                          hiera(\u0027cinder::api::keymgr_encryption_api_url\u0027, undef),"},{"line_number":426,"context_line":"                                          $keymgr_encryption_api_url)"},{"line_number":427,"context_line":"  $keymgr_encryption_auth_url_real \u003d pick($cinder::api::keymgr_encryption_auth_url,"},{"line_number":428,"context_line":"                                          hiera(\u0027cinder::api::keymgr_encryption_auth_url\u0027, undef),"},{"line_number":429,"context_line":"                                          $keymgr_encryption_auth_url)"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"  cinder_config {"},{"line_number":432,"context_line":"    \u0027DEFAULT/report_interval\u0027:                  value \u003d\u003e $report_interval;"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"bf51134e_8c6f25c4","line":429,"range":{"start_line":421,"start_character":0,"end_line":429,"end_character":70},"in_reply_to":"bf51134e_b531acef","updated":"2020-07-06 17:04:20.000000000","message":"You\u0027re right, they shouldn\u0027t be here. I was imagining a situation where the hiera data was present, but the cinder::api class wasn\u0027t included. But that\u0027s not a valid use case.","commit_id":"364addf42055c243de5aee8759e82c7e14c926b7"}]}