)]}'
{"manifests/backend/dellsc_iscsi.pp":[{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"c25196227b181429c16d01c15ce06a05cb27d901","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    \"${name}/secondary_san_password\": value \u003d\u003e $secondary_san_password;"},{"line_number":127,"context_line":"    \"${name}/secondary_sc_api_port\":  value \u003d\u003e $secondary_sc_api_port;"},{"line_number":128,"context_line":"  }"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"  if $excluded_domain_ip {"},{"line_number":131,"context_line":"    cinder_config {"},{"line_number":132,"context_line":"      \"${name}/excluded_domain_ip\": value \u003d\u003e $excluded_domain_ip;"},{"line_number":133,"context_line":"    }"},{"line_number":134,"context_line":"  }"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  if $manage_volume_type {"},{"line_number":137,"context_line":"    cinder_type { $volume_backend_name:"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bf659307_611afa7d","line":134,"range":{"start_line":129,"start_character":0,"end_line":134,"end_character":3},"updated":"2018-03-27 16:00:24.000000000","message":"Does this handle like 2 excluded_domain_ip\u0027s being specified?","commit_id":"a897fe3b715ae16ad1b649cd6315e0b8279ef7e7"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"c0521fb5bea1f03f982b9981645659e0ff732445","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    \"${name}/secondary_san_password\": value \u003d\u003e $secondary_san_password;"},{"line_number":127,"context_line":"    \"${name}/secondary_sc_api_port\":  value \u003d\u003e $secondary_sc_api_port;"},{"line_number":128,"context_line":"  }"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"  if $excluded_domain_ip {"},{"line_number":131,"context_line":"    cinder_config {"},{"line_number":132,"context_line":"      \"${name}/excluded_domain_ip\": value \u003d\u003e $excluded_domain_ip;"},{"line_number":133,"context_line":"    }"},{"line_number":134,"context_line":"  }"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"  if $manage_volume_type {"},{"line_number":137,"context_line":"    cinder_type { $volume_backend_name:"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bf659307_593b2aea","line":134,"range":{"start_line":129,"start_character":0,"end_line":134,"end_character":3},"in_reply_to":"bf659307_611afa7d","updated":"2018-03-27 17:03:59.000000000","message":"It does, as a comma separated list. It always worked that way, and this change just ensures an empty string doesn\u0027t get added to cinder.conf. At some point (I think it may have been in Ocata) olso changed and it no longer accepts an empty string.","commit_id":"a897fe3b715ae16ad1b649cd6315e0b8279ef7e7"}]}