)]}'
{"deployment/cinder/cinder-backend-pure-puppet.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d8abf739562a4e1931b8b219b68a5ffa221ff9d9","unresolved":false,"context_lines":[{"line_number":92,"context_line":"      config_settings:"},{"line_number":93,"context_line":"        map_merge:"},{"line_number":94,"context_line":"          - cinder_pure_backends_hash: {get_param: CinderPureBackendsHash}"},{"line_number":95,"context_line":"            cinder_pure_default_volume_backend_name: {get_param: CinderPureBackendName}"},{"line_number":96,"context_line":"            cinder_pure_default_pure_storage_protocol: {get_param: CinderPureStorageProtocol}"},{"line_number":97,"context_line":"            cinder_pure_default_san_ip: {get_param: CinderPureSanIp}"},{"line_number":98,"context_line":"            cinder_pure_default_pure_api_token: {get_param: CinderPureAPIToken}"},{"line_number":99,"context_line":"            cinder_pure_default_use_chap_auth: {get_param: CinderPureUseChap}"},{"line_number":100,"context_line":"            cinder_pure_default_use_multipath_for_image_xfer: {get_param: CinderPureMultipathXfer}"},{"line_number":101,"context_line":"            cinder_default_image_volume_cache_enabled : {get_param: CinderPureImageCache}"},{"line_number":102,"context_line":"          -"},{"line_number":103,"context_line":"            if:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5faad753_efb6f6d9","line":100,"range":{"start_line":95,"start_character":0,"end_line":100,"end_character":98},"updated":"2019-09-09 22:58:16.000000000","message":"Did you mean to use these as defaults in https://review.opendev.org/#/c/681138/1/manifests/profile/base/cinder/volume/pure.pp@68 \n\nfor L-68 to L\u003d74?","commit_id":"ae21cf0d2e793a4b2df6dee80e14f32ab44ef6ea"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"63fd33691fa348ec136bed4bf748eed1662327ab","unresolved":false,"context_lines":[{"line_number":92,"context_line":"      config_settings:"},{"line_number":93,"context_line":"        map_merge:"},{"line_number":94,"context_line":"          - cinder_pure_backends_hash: {get_param: CinderPureBackendsHash}"},{"line_number":95,"context_line":"            cinder_pure_default_volume_backend_name: {get_param: CinderPureBackendName}"},{"line_number":96,"context_line":"            cinder_pure_default_pure_storage_protocol: {get_param: CinderPureStorageProtocol}"},{"line_number":97,"context_line":"            cinder_pure_default_san_ip: {get_param: CinderPureSanIp}"},{"line_number":98,"context_line":"            cinder_pure_default_pure_api_token: {get_param: CinderPureAPIToken}"},{"line_number":99,"context_line":"            cinder_pure_default_use_chap_auth: {get_param: CinderPureUseChap}"},{"line_number":100,"context_line":"            cinder_pure_default_use_multipath_for_image_xfer: {get_param: CinderPureMultipathXfer}"},{"line_number":101,"context_line":"            cinder_default_image_volume_cache_enabled : {get_param: CinderPureImageCache}"},{"line_number":102,"context_line":"          -"},{"line_number":103,"context_line":"            if:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5faad753_9e208c4a","line":100,"range":{"start_line":95,"start_character":0,"end_line":100,"end_character":98},"in_reply_to":"5faad753_9e688cb9","updated":"2019-09-12 14:49:08.000000000","message":"Emilien and I agreed that I will take this over. I will soon post an update that fixes this (in a slightly different manner).","commit_id":"ae21cf0d2e793a4b2df6dee80e14f32ab44ef6ea"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"fe2e0e992e2bd4240391caf4f01319d6c902a049","unresolved":false,"context_lines":[{"line_number":92,"context_line":"      config_settings:"},{"line_number":93,"context_line":"        map_merge:"},{"line_number":94,"context_line":"          - cinder_pure_backends_hash: {get_param: CinderPureBackendsHash}"},{"line_number":95,"context_line":"            cinder_pure_default_volume_backend_name: {get_param: CinderPureBackendName}"},{"line_number":96,"context_line":"            cinder_pure_default_pure_storage_protocol: {get_param: CinderPureStorageProtocol}"},{"line_number":97,"context_line":"            cinder_pure_default_san_ip: {get_param: CinderPureSanIp}"},{"line_number":98,"context_line":"            cinder_pure_default_pure_api_token: {get_param: CinderPureAPIToken}"},{"line_number":99,"context_line":"            cinder_pure_default_use_chap_auth: {get_param: CinderPureUseChap}"},{"line_number":100,"context_line":"            cinder_pure_default_use_multipath_for_image_xfer: {get_param: CinderPureMultipathXfer}"},{"line_number":101,"context_line":"            cinder_default_image_volume_cache_enabled : {get_param: CinderPureImageCache}"},{"line_number":102,"context_line":"          -"},{"line_number":103,"context_line":"            if:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5faad753_9e688cb9","line":100,"range":{"start_line":95,"start_character":0,"end_line":100,"end_character":98},"in_reply_to":"5faad753_efb6f6d9","updated":"2019-09-12 14:42:29.000000000","message":"I think so , those should dhave the _default_ part in the name","commit_id":"ae21cf0d2e793a4b2df6dee80e14f32ab44ef6ea"}]}
