)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"37a0b1266b796d480e31b3dce522bd5dec312531","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"8485f05e_528fa613","updated":"2021-10-20 08:30:01.000000000","message":"it looks good enough to go in; backport till Wallaby","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e60bd1bc51f6b1ac4a02796f57ecf02628debb12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"e0286298_f8265b93","updated":"2021-10-19 23:10:10.000000000","message":"recheck","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"}],"ansible/group_vars/all.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"4fb3748cc380a9c45f90f8b8ada5df2d8511abf0","unresolved":true,"context_lines":[{"line_number":1064,"context_line":""},{"line_number":1065,"context_line":"# External Ceph FS name."},{"line_number":1066,"context_line":"# By default this is undefined to allow Manila to auto-find the first FS available."},{"line_number":1067,"context_line":"#ceph_manila_fs_name: \"manila\""},{"line_number":1068,"context_line":""},{"line_number":1069,"context_line":"# External Ceph keyrings"},{"line_number":1070,"context_line":"ceph_cinder_keyring: \"ceph.client.cinder.keyring\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"6ff61061_91e27ddb","line":1067,"range":{"start_line":1067,"start_character":22,"end_line":1067,"end_character":30},"updated":"2021-08-03 09:15:29.000000000","message":"I\u0027d suggest dropping \"manila\", since it\u0027s not the default.","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"1ea65f1e7664a4e06d45967337c8406901b7f32f","unresolved":false,"context_lines":[{"line_number":1064,"context_line":""},{"line_number":1065,"context_line":"# External Ceph FS name."},{"line_number":1066,"context_line":"# By default this is undefined to allow Manila to auto-find the first FS available."},{"line_number":1067,"context_line":"#ceph_manila_fs_name: \"manila\""},{"line_number":1068,"context_line":""},{"line_number":1069,"context_line":"# External Ceph keyrings"},{"line_number":1070,"context_line":"ceph_cinder_keyring: \"ceph.client.cinder.keyring\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d9bd7031_982be363","line":1067,"range":{"start_line":1067,"start_character":22,"end_line":1067,"end_character":30},"in_reply_to":"6ff61061_91e27ddb","updated":"2021-08-06 10:12:11.000000000","message":"Done","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":1063,"context_line":"ceph_nova_user: \"nova\""},{"line_number":1064,"context_line":""},{"line_number":1065,"context_line":"# External Ceph FS name."},{"line_number":1066,"context_line":"# By default this is undefined to allow Manila to auto-find the first FS available."},{"line_number":1067,"context_line":"# ceph_manila_fs_name:"},{"line_number":1068,"context_line":""},{"line_number":1069,"context_line":"# External Ceph keyrings"},{"line_number":1070,"context_line":"ceph_cinder_keyring: \"ceph.client.cinder.keyring\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"5ba1b670_bc1a88a4","line":1067,"range":{"start_line":1066,"start_character":0,"end_line":1067,"end_character":22},"updated":"2021-08-13 14:44:09.000000000","message":"We don\u0027t rely on undefined globals and should not - to make it possible to enumerate all available globals. Also, this variable is role-specific, please move it into the manila role, with default of an empty string.","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":1063,"context_line":"ceph_nova_user: \"nova\""},{"line_number":1064,"context_line":""},{"line_number":1065,"context_line":"# External Ceph FS name."},{"line_number":1066,"context_line":"# By default this is undefined to allow Manila to auto-find the first FS available."},{"line_number":1067,"context_line":"# ceph_manila_fs_name:"},{"line_number":1068,"context_line":""},{"line_number":1069,"context_line":"# External Ceph keyrings"},{"line_number":1070,"context_line":"ceph_cinder_keyring: \"ceph.client.cinder.keyring\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"bfcfcf26_8c3a0a92","line":1067,"range":{"start_line":1066,"start_character":0,"end_line":1067,"end_character":22},"in_reply_to":"1d9ac062_c591fb4e","updated":"2021-08-16 09:24:06.000000000","message":"I\u0027ve moved it to the manila role with an empty string default","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"29304ed940340ce71880982633913d3487f1939d","unresolved":true,"context_lines":[{"line_number":1063,"context_line":"ceph_nova_user: \"nova\""},{"line_number":1064,"context_line":""},{"line_number":1065,"context_line":"# External Ceph FS name."},{"line_number":1066,"context_line":"# By default this is undefined to allow Manila to auto-find the first FS available."},{"line_number":1067,"context_line":"# ceph_manila_fs_name:"},{"line_number":1068,"context_line":""},{"line_number":1069,"context_line":"# External Ceph keyrings"},{"line_number":1070,"context_line":"ceph_cinder_keyring: \"ceph.client.cinder.keyring\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"1d9ac062_c591fb4e","line":1067,"range":{"start_line":1066,"start_character":0,"end_line":1067,"end_character":22},"in_reply_to":"5ba1b670_bc1a88a4","updated":"2021-08-14 08:45:32.000000000","message":"Finally, it has occurred to me now, we could also just refrain from adding this variable as there is no valuable logic at Kolla Ansible level and users can just add it via a config override like any other customisation they require.","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"}],"ansible/roles/manila/tasks/fix_cephfs_owner.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"4fb3748cc380a9c45f90f8b8ada5df2d8511abf0","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Check /tmp/cephfs path"},{"line_number":3,"context_line":"  become: true"},{"line_number":4,"context_line":"  command: \"docker exec -u 0 {{ manila_services[\u0027manila-share\u0027][\u0027container_name\u0027] }} ls /tmp/cephfs/\""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9317f9bf_e3d22b70","side":"PARENT","line":1,"updated":"2021-08-03 09:15:29.000000000","message":"Quite happy to see the back of this file.","commit_id":"aad331069def7faf09b046c95d9dca827a7bede2"}],"ansible/roles/manila/templates/manila-share.conf.j2":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"4fb3748cc380a9c45f90f8b8ada5df2d8511abf0","unresolved":true,"context_lines":[{"line_number":116,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":117,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":118,"context_line":"{% endif %}"},{"line_number":119,"context_line":"cephfs_enable_snapshots \u003d False"},{"line_number":120,"context_line":"cephfs_ganesha_server_is_remote\u003d False"},{"line_number":121,"context_line":"cephfs_ganesha_server_ip \u003d {{ api_interface_address }}"},{"line_number":122,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"a95ef1c5_b12d5e7f","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":31},"updated":"2021-08-03 09:15:29.000000000","message":"According to this bug report [1] for Tripleo, this option was removed in Wallaby.\n\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1973373","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"a8e631e2f09295eb967a94412f76874a27350267","unresolved":false,"context_lines":[{"line_number":116,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":117,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":118,"context_line":"{% endif %}"},{"line_number":119,"context_line":"cephfs_enable_snapshots \u003d False"},{"line_number":120,"context_line":"cephfs_ganesha_server_is_remote\u003d False"},{"line_number":121,"context_line":"cephfs_ganesha_server_ip \u003d {{ api_interface_address }}"},{"line_number":122,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"b80524b5_78a9f28d","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":31},"in_reply_to":"3c2e5689_01b65f28","updated":"2021-08-06 14:33:53.000000000","message":"Ah my apologies I see now!\nGood spot, I\u0027ve added that change","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"6819a67b702792e732e669fcfe52ea9fd1cfc941","unresolved":false,"context_lines":[{"line_number":116,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":117,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":118,"context_line":"{% endif %}"},{"line_number":119,"context_line":"cephfs_enable_snapshots \u003d False"},{"line_number":120,"context_line":"cephfs_ganesha_server_is_remote\u003d False"},{"line_number":121,"context_line":"cephfs_ganesha_server_ip \u003d {{ api_interface_address }}"},{"line_number":122,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"3c2e5689_01b65f28","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":31},"in_reply_to":"85a2240f_6c929697","updated":"2021-08-06 13:21:36.000000000","message":"By \"this option\", I was referring to cephfs_enable_snapshots (highlighted by gerrit)","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"1ea65f1e7664a4e06d45967337c8406901b7f32f","unresolved":false,"context_lines":[{"line_number":116,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":117,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":118,"context_line":"{% endif %}"},{"line_number":119,"context_line":"cephfs_enable_snapshots \u003d False"},{"line_number":120,"context_line":"cephfs_ganesha_server_is_remote\u003d False"},{"line_number":121,"context_line":"cephfs_ganesha_server_ip \u003d {{ api_interface_address }}"},{"line_number":122,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"85a2240f_6c929697","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":31},"in_reply_to":"a95ef1c5_b12d5e7f","updated":"2021-08-06 10:12:11.000000000","message":"That report states this was introduced in wallaby, not removed","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"f25e025586f5a099b1cf57feb7c36bdf591684d7","unresolved":true,"context_lines":[{"line_number":100,"context_line":"share_driver \u003d manila.share.drivers.cephfs.driver.CephFSDriver"},{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% endif %}"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"{% if enable_manila_backend_cephfs_nfs | bool %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"49d9a6ef_05fd5341","line":103,"updated":"2021-08-09 09:26:02.000000000","message":"I suppose we also need cephfs_filesystem_name here?","commit_id":"b3821801d8cb9c5a83be7d9bad500708ee92c36a"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"3c088e96fc912a7574f3939c76330a55ce7ad00d","unresolved":false,"context_lines":[{"line_number":100,"context_line":"share_driver \u003d manila.share.drivers.cephfs.driver.CephFSDriver"},{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% endif %}"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"{% if enable_manila_backend_cephfs_nfs | bool %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"e506db9b_aeddccf9","line":103,"in_reply_to":"49d9a6ef_05fd5341","updated":"2021-08-09 10:31:46.000000000","message":"Done","commit_id":"b3821801d8cb9c5a83be7d9bad500708ee92c36a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"3c52de10_60954b58","line":104,"range":{"start_line":104,"start_character":37,"end_line":104,"end_character":38},"updated":"2021-08-13 14:44:09.000000000","message":"nit: extra space (also, this line will need adapting due to the globals comment; we could also just pipe the empty name to manila and it should work the same - that would be best)","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"29a75763_bf73c143","line":104,"range":{"start_line":104,"start_character":37,"end_line":104,"end_character":38},"in_reply_to":"3c52de10_60954b58","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"},{"line_number":108,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"ad924c7d_a7bfea5f","line":105,"range":{"start_line":105,"start_character":27,"end_line":105,"end_character":46},"updated":"2021-08-13 14:44:09.000000000","message":"nit: lack of spaces around the variable name (I wonder where the linter is)","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"},{"line_number":108,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7bfde9fb_b41b0478","line":105,"range":{"start_line":105,"start_character":27,"end_line":105,"end_character":46},"in_reply_to":"ad924c7d_a7bfea5f","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":115,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":116,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":117,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":118,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":119,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":120,"context_line":"{% endif %}"},{"line_number":121,"context_line":"cephfs_ganesha_server_is_remote\u003d False"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"277c7889_09ba77b0","line":118,"range":{"start_line":118,"start_character":37,"end_line":118,"end_character":38},"updated":"2021-08-13 14:44:09.000000000","message":"nit: extra space (also, this line will need adapting due to the globals comment; we could also just pipe the empty name to manila and it should work the same - that would be best)","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":115,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":116,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":117,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":118,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":119,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":120,"context_line":"{% endif %}"},{"line_number":121,"context_line":"cephfs_ganesha_server_is_remote\u003d False"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"749e4f69_e4731b5c","line":118,"range":{"start_line":118,"start_character":37,"end_line":118,"end_character":38},"in_reply_to":"277c7889_09ba77b0","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":116,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":117,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":118,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":119,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":120,"context_line":"{% endif %}"},{"line_number":121,"context_line":"cephfs_ganesha_server_is_remote\u003d False"},{"line_number":122,"context_line":"cephfs_ganesha_server_ip \u003d {{ api_interface_address }}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"0906943e_af33b54e","line":119,"range":{"start_line":119,"start_character":27,"end_line":119,"end_character":46},"updated":"2021-08-13 14:44:09.000000000","message":"nit: lack of spaces around the variable name (I wonder where the linter is)","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":116,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":117,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":118,"context_line":"{% if ceph_manila_fs_name is defined  %}"},{"line_number":119,"context_line":"cephfs_filesystem_name \u003d {{ceph_manila_fs_name}}"},{"line_number":120,"context_line":"{% endif %}"},{"line_number":121,"context_line":"cephfs_ganesha_server_is_remote\u003d False"},{"line_number":122,"context_line":"cephfs_ganesha_server_ip \u003d {{ api_interface_address }}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"2ed839e3_dc2d2d5d","line":119,"range":{"start_line":119,"start_character":27,"end_line":119,"end_character":46},"in_reply_to":"0906943e_af33b54e","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"4da0da36a3d145bea310af6af55972b4ad11dc55","unresolved":true,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if manila_cephfs_filesystem_name | length %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ manila_cephfs_filesystem_name }}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"4380bcb7_8a3da843","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":47},"updated":"2021-08-27 09:58:29.000000000","message":"this is not needed","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"31222ec0ad77ac11ef1ad5608e60ff3e831863a8","unresolved":true,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if manila_cephfs_filesystem_name | length %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ manila_cephfs_filesystem_name }}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"d728792f_b0c476d7","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":47},"in_reply_to":"0a92a6ca_a11d0028","updated":"2021-08-31 09:56:52.000000000","message":"Not in general; the default is an empty string here. And empty means Manila will decide.","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ad7c5413ace89c11586b5bf67e0a6f6b49c65aff","unresolved":true,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if manila_cephfs_filesystem_name | length %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ manila_cephfs_filesystem_name }}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"7bf34dc0_dc52701d","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":47},"in_reply_to":"2e2c917a_a362960f","updated":"2021-09-10 08:34:55.000000000","message":"OTOH, this code is ready to go, and it seems likely that users using manila/ceph would need to set this option.","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"3bc720779ec39df87aa3f7926af7b368f7aeb18e","unresolved":true,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if manila_cephfs_filesystem_name | length %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ manila_cephfs_filesystem_name }}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"0a92a6ca_a11d0028","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":47},"in_reply_to":"4380bcb7_8a3da843","updated":"2021-08-31 09:26:26.000000000","message":"Are you sure? Empty option \u003d\u003d unset option?","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"be086f1db9fbc599a0590b4da37f06fbb642ff20","unresolved":true,"context_lines":[{"line_number":101,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":102,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":103,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":104,"context_line":"{% if manila_cephfs_filesystem_name | length %}"},{"line_number":105,"context_line":"cephfs_filesystem_name \u003d {{ manila_cephfs_filesystem_name }}"},{"line_number":106,"context_line":"{% endif %}"},{"line_number":107,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"2e2c917a_a362960f","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":47},"in_reply_to":"d728792f_b0c476d7","updated":"2021-08-31 10:15:19.000000000","message":"Nota bene, my original suggestion was not to claim ownership at all and let users use the overrides.","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"4da0da36a3d145bea310af6af55972b4ad11dc55","unresolved":true,"context_lines":[{"line_number":115,"context_line":"cephfs_conf_path \u003d /etc/ceph/ceph.conf"},{"line_number":116,"context_line":"cephfs_auth_id \u003d {{ ceph_manila_user }}"},{"line_number":117,"context_line":"cephfs_cluster_name \u003d ceph"},{"line_number":118,"context_line":"{% if manila_cephfs_filesystem_name | length %}"},{"line_number":119,"context_line":"cephfs_filesystem_name \u003d {{ manila_cephfs_filesystem_name }}"},{"line_number":120,"context_line":"{% endif %}"},{"line_number":121,"context_line":"cephfs_ganesha_server_is_remote\u003d False"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"f18222ec_a1af6d41","line":118,"range":{"start_line":118,"start_character":0,"end_line":118,"end_character":47},"updated":"2021-08-27 09:58:29.000000000","message":"ditto","commit_id":"8d5dde3723c75958b7583d30b525667e99ec2496"}],"doc/source/reference/storage/external-ceph-guide.rst":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"4fb3748cc380a9c45f90f8b8ada5df2d8511abf0","unresolved":true,"context_lines":[{"line_number":206,"context_line":"#. Copy Ceph configuration file to ``/etc/kolla/config/manila/ceph.conf``"},{"line_number":207,"context_line":"#. Copy Ceph keyring to ``/etc/kolla/config/manila/\u003cceph_manila_keyring\u003e``"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"#. If using multiple filesystems (Ceph Pacific+) ``/etc/kolla/globals.yml``:"},{"line_number":210,"context_line":"    * ``ceph_manila_fs_name`` (default: undefined, default to the first FS)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"#. Setup Manila in the usual way"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"e754a964_68b9adc5","line":210,"range":{"start_line":209,"start_character":0,"end_line":210,"end_character":75},"updated":"2021-08-03 09:15:29.000000000","message":"How about:\n\n #. If using multiple filesystems (Ceph Pacific+), set ``ceph_manila_fs_name`` in \n    ``/etc/kolla/globals.yml`` (default: undefined, meaning use the first found\n    filesystem)","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"1ea65f1e7664a4e06d45967337c8406901b7f32f","unresolved":false,"context_lines":[{"line_number":206,"context_line":"#. Copy Ceph configuration file to ``/etc/kolla/config/manila/ceph.conf``"},{"line_number":207,"context_line":"#. Copy Ceph keyring to ``/etc/kolla/config/manila/\u003cceph_manila_keyring\u003e``"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"#. If using multiple filesystems (Ceph Pacific+) ``/etc/kolla/globals.yml``:"},{"line_number":210,"context_line":"    * ``ceph_manila_fs_name`` (default: undefined, default to the first FS)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"#. Setup Manila in the usual way"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"f4eeaacb_2c0bdaaa","line":210,"range":{"start_line":209,"start_character":0,"end_line":210,"end_character":75},"in_reply_to":"e754a964_68b9adc5","updated":"2021-08-06 10:12:11.000000000","message":"Done","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":208,"context_line":""},{"line_number":209,"context_line":"#. If using multiple filesystems (Ceph Pacific+), set ``ceph_manila_fs_name``"},{"line_number":210,"context_line":"    in ``/etc/kolla/globals.yml`` (default: undefined, meaning use the first"},{"line_number":211,"context_line":"    found filesystem)"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"#. Setup Manila in the usual way"},{"line_number":214,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"8573dd88_730543b5","line":211,"updated":"2021-08-13 14:44:09.000000000","message":"two unaligned lines, renders improperly","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":208,"context_line":""},{"line_number":209,"context_line":"#. If using multiple filesystems (Ceph Pacific+), set ``ceph_manila_fs_name``"},{"line_number":210,"context_line":"    in ``/etc/kolla/globals.yml`` (default: undefined, meaning use the first"},{"line_number":211,"context_line":"    found filesystem)"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"#. Setup Manila in the usual way"},{"line_number":214,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"723cb179_9c5761bb","line":211,"in_reply_to":"8573dd88_730543b5","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"}],"etc/kolla/globals.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"4fb3748cc380a9c45f90f8b8ada5df2d8511abf0","unresolved":true,"context_lines":[{"line_number":458,"context_line":"# Manila"},{"line_number":459,"context_line":"#ceph_manila_keyring: \"ceph.client.manila.keyring\""},{"line_number":460,"context_line":"#ceph_manila_user: \"manila\""},{"line_number":461,"context_line":"#ceph_manila_fs_name: \"manila\""},{"line_number":462,"context_line":""},{"line_number":463,"context_line":"#############################"},{"line_number":464,"context_line":"# Keystone - Identity Options"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9589ea40_31cf68ad","line":461,"range":{"start_line":461,"start_character":22,"end_line":461,"end_character":30},"updated":"2021-08-03 09:15:29.000000000","message":"I\u0027d suggest dropping \"manila\", since that\u0027s not the default.","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"1ea65f1e7664a4e06d45967337c8406901b7f32f","unresolved":false,"context_lines":[{"line_number":458,"context_line":"# Manila"},{"line_number":459,"context_line":"#ceph_manila_keyring: \"ceph.client.manila.keyring\""},{"line_number":460,"context_line":"#ceph_manila_user: \"manila\""},{"line_number":461,"context_line":"#ceph_manila_fs_name: \"manila\""},{"line_number":462,"context_line":""},{"line_number":463,"context_line":"#############################"},{"line_number":464,"context_line":"# Keystone - Identity Options"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"6cb57842_636e6030","line":461,"range":{"start_line":461,"start_character":22,"end_line":461,"end_character":30},"in_reply_to":"9589ea40_31cf68ad","updated":"2021-08-06 10:12:11.000000000","message":"Done","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"1bd1ab3b329943c12e0ace8cd40d66b7c7b611a0","unresolved":true,"context_lines":[{"line_number":621,"context_line":"#hnas_evs_ip:"},{"line_number":622,"context_line":"#hnas_file_system_name:"},{"line_number":623,"context_line":""},{"line_number":624,"context_line":"# If the glusterfs server requires remote ssh, then you need to fill"},{"line_number":625,"context_line":"# in \u0027manila_glus"},{"line_number":626,"context_line":"# CephFS backend configuration."},{"line_number":627,"context_line":"# External Ceph FS name."},{"line_number":628,"context_line":"# By default this is empty to allow Manila to auto-find the first FS available."}],"source_content_type":"text/x-yaml","patch_set":13,"id":"c8378db2_5c9eae18","line":625,"range":{"start_line":624,"start_character":0,"end_line":625,"end_character":17},"updated":"2021-08-17 08:54:22.000000000","message":"erroneous","commit_id":"b023198e5ea5a15f10f093b8b858daf183a300b4"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"f0691bedda91b76e1f4b8d0045d8a4308e4c3d45","unresolved":false,"context_lines":[{"line_number":621,"context_line":"#hnas_evs_ip:"},{"line_number":622,"context_line":"#hnas_file_system_name:"},{"line_number":623,"context_line":""},{"line_number":624,"context_line":"# If the glusterfs server requires remote ssh, then you need to fill"},{"line_number":625,"context_line":"# in \u0027manila_glus"},{"line_number":626,"context_line":"# CephFS backend configuration."},{"line_number":627,"context_line":"# External Ceph FS name."},{"line_number":628,"context_line":"# By default this is empty to allow Manila to auto-find the first FS available."}],"source_content_type":"text/x-yaml","patch_set":13,"id":"461b3225_6154a1dc","line":625,"range":{"start_line":624,"start_character":0,"end_line":625,"end_character":17},"in_reply_to":"c8378db2_5c9eae18","updated":"2021-08-17 09:02:56.000000000","message":"Done","commit_id":"b023198e5ea5a15f10f093b8b858daf183a300b4"}],"releasenotes/notes/support-manila-wallaby-2e29e866af0d6287.yaml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"4fb3748cc380a9c45f90f8b8ada5df2d8511abf0","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Removes fix_cephfs_owner yaml which related to pre-wallaby Manila\u0027s"},{"line_number":9,"context_line":"    use of subfolders. Post-wallaby Manila now uses cephfs volumes instead,"},{"line_number":10,"context_line":"    as such this file is no longer required and is causing"},{"line_number":11,"context_line":"    bugs 1938285 and 1935784"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"20875106_3160100e","line":11,"range":{"start_line":10,"start_character":43,"end_line":11,"end_character":28},"updated":"2021-08-03 09:15:29.000000000","message":"Typically we link to bugs with this syntax:\n\n`LP#1938285 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1938285\u003e`__","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"1ea65f1e7664a4e06d45967337c8406901b7f32f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Removes fix_cephfs_owner yaml which related to pre-wallaby Manila\u0027s"},{"line_number":9,"context_line":"    use of subfolders. Post-wallaby Manila now uses cephfs volumes instead,"},{"line_number":10,"context_line":"    as such this file is no longer required and is causing"},{"line_number":11,"context_line":"    bugs 1938285 and 1935784"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d4266ff7_819facec","line":11,"range":{"start_line":10,"start_character":43,"end_line":11,"end_character":28},"in_reply_to":"20875106_3160100e","updated":"2021-08-06 10:12:11.000000000","message":"Done","commit_id":"1f519eca58fea2d9c440f8511cb1ac4ba4d5464f"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds ceph_manila_fs_name variable to support multi-fs"},{"line_number":5,"context_line":"    Ceph Pacific+ deloyments."},{"line_number":6,"context_line":"fixes:"},{"line_number":7,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"c5c48a52_b6bb5a1d","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":29},"updated":"2021-08-13 14:44:09.000000000","message":"nit: put in `` ``","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds ceph_manila_fs_name variable to support multi-fs"},{"line_number":5,"context_line":"    Ceph Pacific+ deloyments."},{"line_number":6,"context_line":"fixes:"},{"line_number":7,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"432dc578_37f4e05a","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":29},"in_reply_to":"c5c48a52_b6bb5a1d","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    Ceph Pacific+ deloyments."},{"line_number":6,"context_line":"fixes:"},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Removes fix_cephfs_owner yaml which related to pre-wallaby Manila\u0027s"},{"line_number":9,"context_line":"    use of subfolders. Post-wallaby Manila now uses cephfs volumes instead,"},{"line_number":10,"context_line":"    as such this file is no longer required."},{"line_number":11,"context_line":"    `LP#1938285 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1938285\u003e`__"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"f593e3ea_8e1f7e72","line":8,"range":{"start_line":8,"start_character":12,"end_line":8,"end_character":29},"updated":"2021-08-13 14:44:09.000000000","message":"nit: put in `` ``","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    Ceph Pacific+ deloyments."},{"line_number":6,"context_line":"fixes:"},{"line_number":7,"context_line":"  - |"},{"line_number":8,"context_line":"    Removes fix_cephfs_owner yaml which related to pre-wallaby Manila\u0027s"},{"line_number":9,"context_line":"    use of subfolders. Post-wallaby Manila now uses cephfs volumes instead,"},{"line_number":10,"context_line":"    as such this file is no longer required."},{"line_number":11,"context_line":"    `LP#1938285 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1938285\u003e`__"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"c7cd82a4_7e06dad1","line":8,"range":{"start_line":8,"start_character":12,"end_line":8,"end_character":29},"in_reply_to":"f593e3ea_8e1f7e72","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f50b11beb5d43ccc8bb49aa688daa8c964fecab6","unresolved":true,"context_lines":[{"line_number":11,"context_line":"    `LP#1938285 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1938285\u003e`__"},{"line_number":12,"context_line":"    `LP#1935784 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1935784\u003e`__"},{"line_number":13,"context_line":"  - |"},{"line_number":14,"context_line":"    Removes use of \u0027cephfs_enable_snapshots\u0027 in Manila config as this"},{"line_number":15,"context_line":"    option was removed from Manila in the Wallaby release."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"d43db445_9918f88b","line":14,"range":{"start_line":14,"start_character":20,"end_line":14,"end_character":43},"updated":"2021-08-13 14:44:09.000000000","message":"nit: put in `` `` instead of \u0027 \u0027","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"},{"author":{"_account_id":33694,"name":"Skylar Tristan Kelty","display_name":"Skylar Kelty","email":"skylar.kelty@kcl.ac.uk","username":"skylarkelty"},"change_message_id":"24616e3ccd77ae05c6305cb850af91bd6a28ff5e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    `LP#1938285 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1938285\u003e`__"},{"line_number":12,"context_line":"    `LP#1935784 \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1935784\u003e`__"},{"line_number":13,"context_line":"  - |"},{"line_number":14,"context_line":"    Removes use of \u0027cephfs_enable_snapshots\u0027 in Manila config as this"},{"line_number":15,"context_line":"    option was removed from Manila in the Wallaby release."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"abb14c71_f8c97154","line":14,"range":{"start_line":14,"start_character":20,"end_line":14,"end_character":43},"in_reply_to":"d43db445_9918f88b","updated":"2021-08-16 09:24:06.000000000","message":"Done","commit_id":"b131032ec4e4cdd6a29d4c7986cc4f2067541680"}]}
