)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"88f0a071bd8a8138170631426f3c7848efb8b772","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"83bc920b_7ad53446","updated":"2022-05-31 21:57:05.000000000","message":"Just some whitespace issues to clean up, but otherwise lgtm.","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"dfca21b85703fb8489a643370c0ca8224f064fa9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"aef72575_b61ad9d6","updated":"2022-06-02 13:34:08.000000000","message":"LGTM. Good job !","commit_id":"18ef55f83b39c4aeec0c9e228c26bca778155709"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"06de8162def0775d2246d5059c72dfd2a88807af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"47b17203_b891f621","updated":"2022-06-02 17:13:48.000000000","message":"recheck","commit_id":"18ef55f83b39c4aeec0c9e228c26bca778155709"}],"manifests/profile/base/cinder/volume/dellemc_unity.pp":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"88f0a071bd8a8138170631426f3c7848efb8b772","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  if $step \u003e\u003d 4 {"},{"line_number":42,"context_line":"    $backend_defaults \u003d {"},{"line_number":43,"context_line":"      \u0027CinderDellEMCUnityAvailabilityZone\u0027       \u003d\u003e lookup(\u0027cinder::backend::dellemc_unity::backend_availability_zone\u0027,"},{"line_number":44,"context_line":"                                                          undef, undef, undef),"},{"line_number":45,"context_line":"      \u0027CinderDellEMCUnitySanIp\u0027                  \u003d\u003e lookup(\u0027cinder::backend::dellemc_unity::san_ip\u0027, undef, undef, undef),"},{"line_number":46,"context_line":"      \u0027CinderDellEMCUnitySanLogin\u0027               \u003d\u003e lookup(\u0027cinder::backend::dellemc_unity::san_login\u0027, undef, undef, undef),"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"1f1da5b0_a0925c09","line":43,"range":{"start_line":43,"start_character":42,"end_line":43,"end_character":48},"updated":"2022-05-31 21:57:05.000000000","message":"Remove extra spaces, so that there\u0027s only one space for the longest item (CinderDellEMCUnityAvailabilityZone and CinderDellEMCUnityStoragePoolNames are the longest).\n\nOnly add extra spaces for the shorter items in order to align things.","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"95b44fd90e227eab8c9ce88b7e85fee56150482d","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  if $step \u003e\u003d 4 {"},{"line_number":42,"context_line":"    $backend_defaults \u003d {"},{"line_number":43,"context_line":"      \u0027CinderDellEMCUnityAvailabilityZone\u0027       \u003d\u003e lookup(\u0027cinder::backend::dellemc_unity::backend_availability_zone\u0027,"},{"line_number":44,"context_line":"                                                          undef, undef, undef),"},{"line_number":45,"context_line":"      \u0027CinderDellEMCUnitySanIp\u0027                  \u003d\u003e lookup(\u0027cinder::backend::dellemc_unity::san_ip\u0027, undef, undef, undef),"},{"line_number":46,"context_line":"      \u0027CinderDellEMCUnitySanLogin\u0027               \u003d\u003e lookup(\u0027cinder::backend::dellemc_unity::san_login\u0027, undef, undef, undef),"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"6608b0e3_7e17a7ba","line":43,"range":{"start_line":43,"start_character":42,"end_line":43,"end_character":48},"in_reply_to":"1f1da5b0_a0925c09","updated":"2022-06-01 06:50:49.000000000","message":"Done","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"}],"spec/classes/tripleo_profile_base_cinder_volume_spec.rb":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"88f0a071bd8a8138170631426f3c7848efb8b772","unresolved":true,"context_lines":[{"line_number":208,"context_line":"      context \u0027with only unity\u0027 do"},{"line_number":209,"context_line":"        before :each do"},{"line_number":210,"context_line":"          params.merge!({"},{"line_number":211,"context_line":"            :cinder_enable_dellemc_unity_backend  \u003d\u003e true,"},{"line_number":212,"context_line":"            :cinder_enable_iscsi_backend       \u003d\u003e false,"},{"line_number":213,"context_line":"          })"},{"line_number":214,"context_line":"        end"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"dca45b6b_7faa6d75","line":211,"range":{"start_line":211,"start_character":49,"end_line":211,"end_character":50},"updated":"2022-05-31 21:57:05.000000000","message":"extra space should be removed","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"95b44fd90e227eab8c9ce88b7e85fee56150482d","unresolved":false,"context_lines":[{"line_number":208,"context_line":"      context \u0027with only unity\u0027 do"},{"line_number":209,"context_line":"        before :each do"},{"line_number":210,"context_line":"          params.merge!({"},{"line_number":211,"context_line":"            :cinder_enable_dellemc_unity_backend  \u003d\u003e true,"},{"line_number":212,"context_line":"            :cinder_enable_iscsi_backend       \u003d\u003e false,"},{"line_number":213,"context_line":"          })"},{"line_number":214,"context_line":"        end"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"1f0b56e4_e316512c","line":211,"range":{"start_line":211,"start_character":49,"end_line":211,"end_character":50},"in_reply_to":"dca45b6b_7faa6d75","updated":"2022-06-01 06:50:49.000000000","message":"Done","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"88f0a071bd8a8138170631426f3c7848efb8b772","unresolved":true,"context_lines":[{"line_number":209,"context_line":"        before :each do"},{"line_number":210,"context_line":"          params.merge!({"},{"line_number":211,"context_line":"            :cinder_enable_dellemc_unity_backend  \u003d\u003e true,"},{"line_number":212,"context_line":"            :cinder_enable_iscsi_backend       \u003d\u003e false,"},{"line_number":213,"context_line":"          })"},{"line_number":214,"context_line":"        end"},{"line_number":215,"context_line":"        it \u0027should configure only unity\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"9a421996_a5294bc8","line":212,"range":{"start_line":212,"start_character":42,"end_line":212,"end_character":44},"updated":"2022-05-31 21:57:05.000000000","message":"add spaces to align with L211","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"95b44fd90e227eab8c9ce88b7e85fee56150482d","unresolved":false,"context_lines":[{"line_number":209,"context_line":"        before :each do"},{"line_number":210,"context_line":"          params.merge!({"},{"line_number":211,"context_line":"            :cinder_enable_dellemc_unity_backend  \u003d\u003e true,"},{"line_number":212,"context_line":"            :cinder_enable_iscsi_backend       \u003d\u003e false,"},{"line_number":213,"context_line":"          })"},{"line_number":214,"context_line":"        end"},{"line_number":215,"context_line":"        it \u0027should configure only unity\u0027 do"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"1b215eb3_550d134d","line":212,"range":{"start_line":212,"start_character":42,"end_line":212,"end_character":44},"in_reply_to":"9a421996_a5294bc8","updated":"2022-06-01 06:50:49.000000000","message":"Done","commit_id":"fe6b5143c73214120b6f0661ddf1aa5146dec868"}]}