)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"6bbcc05bee99c6a7c295b0388ddfb3cfd23c1b59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7860d410_1d4e561c","updated":"2022-06-07 12:35:41.000000000","message":"recheck","commit_id":"78fa4397bd88db5b364660098066aec013d009be"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"45c5f857426ca6c5a96a14ec6123219a78b67644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b180514a_151d678f","updated":"2022-06-07 19:38:30.000000000","message":"recheck","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"687ccbe4fc9f5eb7adca1876ed5b57e3e1415b1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e19b8f68_8e044283","updated":"2022-06-10 05:48:22.000000000","message":"recheck\n","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"}],"deployment/cinder/cinder-volume-container-puppet.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f9564b5ef993f360417d408e298c7e4aea4427cb","unresolved":true,"context_lines":[{"line_number":96,"context_line":"  CinderNfsBackendName:"},{"line_number":97,"context_line":"    type: comma_delimited_list"},{"line_number":98,"context_line":"    default: \u0027tripleo_nfs\u0027"},{"line_number":99,"context_line":"    description: A list of Cinder Nfs backend names."},{"line_number":100,"context_line":"  CinderNfsMultiConfig:"},{"line_number":101,"context_line":"    type: json"},{"line_number":102,"context_line":"    default: {}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"deea0df1_06241b3c","line":99,"range":{"start_line":99,"start_character":34,"end_line":99,"end_character":37},"updated":"2022-06-14 15:05:23.000000000","message":"nit: NFS","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"90d001f202ef60bc1f404e4b477f9ed2ef043698","unresolved":false,"context_lines":[{"line_number":96,"context_line":"  CinderNfsBackendName:"},{"line_number":97,"context_line":"    type: comma_delimited_list"},{"line_number":98,"context_line":"    default: \u0027tripleo_nfs\u0027"},{"line_number":99,"context_line":"    description: A list of Cinder Nfs backend names."},{"line_number":100,"context_line":"  CinderNfsMultiConfig:"},{"line_number":101,"context_line":"    type: json"},{"line_number":102,"context_line":"    default: {}"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"d1a93ff0_292d3ac3","line":99,"range":{"start_line":99,"start_character":34,"end_line":99,"end_character":37},"in_reply_to":"deea0df1_06241b3c","updated":"2022-06-14 21:32:41.000000000","message":"Done","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f9564b5ef993f360417d408e298c7e4aea4427cb","unresolved":true,"context_lines":[{"line_number":101,"context_line":"    type: json"},{"line_number":102,"context_line":"    default: {}"},{"line_number":103,"context_line":"    description: \u003e"},{"line_number":104,"context_line":"      Dictionary of settings when configuring multiple Nfs backends. The"},{"line_number":105,"context_line":"      hash key is the backend name, and the value is a dictionary of parameter"},{"line_number":106,"context_line":"      values unique to that backend."},{"line_number":107,"context_line":"  CinderNfsAvailabilityZone:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f1aef4a7_e350b3ed","line":104,"range":{"start_line":104,"start_character":55,"end_line":104,"end_character":58},"updated":"2022-06-14 15:05:23.000000000","message":"NFS","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"90d001f202ef60bc1f404e4b477f9ed2ef043698","unresolved":false,"context_lines":[{"line_number":101,"context_line":"    type: json"},{"line_number":102,"context_line":"    default: {}"},{"line_number":103,"context_line":"    description: \u003e"},{"line_number":104,"context_line":"      Dictionary of settings when configuring multiple Nfs backends. The"},{"line_number":105,"context_line":"      hash key is the backend name, and the value is a dictionary of parameter"},{"line_number":106,"context_line":"      values unique to that backend."},{"line_number":107,"context_line":"  CinderNfsAvailabilityZone:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"35e4b36c_7b5975ef","line":104,"range":{"start_line":104,"start_character":55,"end_line":104,"end_character":58},"in_reply_to":"f1aef4a7_e350b3ed","updated":"2022-06-14 21:32:41.000000000","message":"Done","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f9564b5ef993f360417d408e298c7e4aea4427cb","unresolved":true,"context_lines":[{"line_number":110,"context_line":"      The availability zone of the NFS Cinder backend."},{"line_number":111,"context_line":"      When set, it overrides the default CinderStorageAvailabilityZone."},{"line_number":112,"context_line":"    type: string"},{"line_number":113,"context_line":"  CinderNfsSharesConfig:"},{"line_number":114,"context_line":"    type: string"},{"line_number":115,"context_line":"    default: \u0027/etc/cinder/shares-nfs.conf\u0027"},{"line_number":116,"context_line":"  CinderNfsMountOptions:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9b31a6c6_a8f7d107","line":113,"updated":"2022-06-14 15:05:23.000000000","message":"Please add a parameter description. Whereas the cinder parameter\u0027s description is \"File with the list of available NFS shares,\" the idea behind this tripleo parameter is it\u0027s the name of the file where tripleo will write the list of NFS shares.\n\nI also think this is a good place to repeat the fact that, when configuring multiple backends, the parameter values must be unique across all of the backends.","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"90d001f202ef60bc1f404e4b477f9ed2ef043698","unresolved":false,"context_lines":[{"line_number":110,"context_line":"      The availability zone of the NFS Cinder backend."},{"line_number":111,"context_line":"      When set, it overrides the default CinderStorageAvailabilityZone."},{"line_number":112,"context_line":"    type: string"},{"line_number":113,"context_line":"  CinderNfsSharesConfig:"},{"line_number":114,"context_line":"    type: string"},{"line_number":115,"context_line":"    default: \u0027/etc/cinder/shares-nfs.conf\u0027"},{"line_number":116,"context_line":"  CinderNfsMountOptions:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"8c796932_a15c49f1","line":113,"in_reply_to":"9b31a6c6_a8f7d107","updated":"2022-06-14 21:32:41.000000000","message":"Done","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"}],"environments/cinder-nfs-config.yaml":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f9564b5ef993f360417d408e298c7e4aea4427cb","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# A Heat environment file which can be used to enable a"},{"line_number":2,"context_line":"# a Cinder NFS backend, configured via puppet"},{"line_number":3,"context_line":"parameter_defaults:"},{"line_number":4,"context_line":"  CinderNfsBackendName: [\u0027tripleo_nfs_1\u0027]"},{"line_number":5,"context_line":"  CinderEnableNfsBackend: True"},{"line_number":6,"context_line":"  CinderNfsMountOptions: context\u003dsystem_u:object_r:container_file_t:s0"},{"line_number":7,"context_line":"  CinderNfsServers: 192.168.24.2:/var/nfs/cinder"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2232f074_de62609c","line":4,"range":{"start_line":4,"start_character":26,"end_line":4,"end_character":39},"updated":"2022-06-14 15:05:23.000000000","message":"This should match the default value, which is just \u0027tripleo_nfs\u0027","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"90d001f202ef60bc1f404e4b477f9ed2ef043698","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# A Heat environment file which can be used to enable a"},{"line_number":2,"context_line":"# a Cinder NFS backend, configured via puppet"},{"line_number":3,"context_line":"parameter_defaults:"},{"line_number":4,"context_line":"  CinderNfsBackendName: [\u0027tripleo_nfs_1\u0027]"},{"line_number":5,"context_line":"  CinderEnableNfsBackend: True"},{"line_number":6,"context_line":"  CinderNfsMountOptions: context\u003dsystem_u:object_r:container_file_t:s0"},{"line_number":7,"context_line":"  CinderNfsServers: 192.168.24.2:/var/nfs/cinder"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"d5db7ff9_92361bc2","line":4,"range":{"start_line":4,"start_character":26,"end_line":4,"end_character":39},"in_reply_to":"2232f074_de62609c","updated":"2022-06-14 21:32:41.000000000","message":"Done","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"f9564b5ef993f360417d408e298c7e4aea4427cb","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  CinderNfsBackendName: [\u0027tripleo_nfs_1\u0027]"},{"line_number":5,"context_line":"  CinderEnableNfsBackend: True"},{"line_number":6,"context_line":"  CinderNfsMountOptions: context\u003dsystem_u:object_r:container_file_t:s0"},{"line_number":7,"context_line":"  CinderNfsServers: 192.168.24.2:/var/nfs/cinder"},{"line_number":8,"context_line":"  CinderNfsSharesConfig: /etc/cinder/shares-nfs.conf"},{"line_number":9,"context_line":"# To configure multiple NFS backends, use CinderNfsMultiConfig to"},{"line_number":10,"context_line":"# assign parameter values specific to that backend. For example:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5aaa68db_5fb78c57","line":7,"updated":"2022-06-14 15:05:23.000000000","message":"Leave this a blank string \u0027\u0027","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"90d001f202ef60bc1f404e4b477f9ed2ef043698","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  CinderNfsBackendName: [\u0027tripleo_nfs_1\u0027]"},{"line_number":5,"context_line":"  CinderEnableNfsBackend: True"},{"line_number":6,"context_line":"  CinderNfsMountOptions: context\u003dsystem_u:object_r:container_file_t:s0"},{"line_number":7,"context_line":"  CinderNfsServers: 192.168.24.2:/var/nfs/cinder"},{"line_number":8,"context_line":"  CinderNfsSharesConfig: /etc/cinder/shares-nfs.conf"},{"line_number":9,"context_line":"# To configure multiple NFS backends, use CinderNfsMultiConfig to"},{"line_number":10,"context_line":"# assign parameter values specific to that backend. For example:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"90d22f90_88a7205b","line":7,"in_reply_to":"5aaa68db_5fb78c57","updated":"2022-06-14 21:32:41.000000000","message":"Done","commit_id":"c24640355fd726c07c8731bb713216c8dca4b672"}]}
