)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"658d0f4a6f125c8a10b6ccf3e5618d3af8349cce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"4b3d9465_72469954","updated":"2021-10-13 17:28:34.000000000","message":"Thank you, for your patch!\nI added a few comments, check them inline :)","commit_id":"b4a0e9bf87707a0e2886c9fe32c848c3f007a8e1"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"5dedc08d99470e1e9391023d2a8db9a26b997c88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"8eac63a4_22b4506c","updated":"2021-11-09 15:27:52.000000000","message":"Thank you, Archana!\n\nI totally missed a index.rst detail, you will need just a small addition there\nmb :)","commit_id":"8e8a4420bf8de14e6583f101764d6c28e5dc1917"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"b2a339c9934eee8efb2ab47c766820fd72e57d05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"85bfbe43_d59fe831","updated":"2021-11-07 05:53:31.000000000","message":"Thanks, Fábio for the review.\nI\u0027ve followed your suggestions and made some changes.","commit_id":"8e8a4420bf8de14e6583f101764d6c28e5dc1917"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"bfc45606fb93abe62589a2483d85a2830a2c5ab2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"cb190f2a_392a63a8","updated":"2021-11-17 19:27:30.000000000","message":"My bad, I might confused you.\n\nYou can check how the index.rst is being built.\n\n1. Expand the zuul verified comment line\n2. click on openstack-tox-docs link\n3. select the tab \u0027Logs\u0027 -- between \u0027Artifacts\u0027 and \u0027Console\u0027\n4. click on \u0027(raw)\u0027 link from \u0027docs\u0027 -- first of list\n5. on the navbar, click on \u0027openstack share command-line\u0027\n\nIt will open the built docs for openstack share, there you should have all the available share group command under share group and also all the share group snapshot commands under share group snapshot.\n\nTake a look on the comments, and feel free to ping me on irc. Thank you once again :D","commit_id":"932eb3980f6d1ed8a6b58420eb550cb34d0ccc81"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"bc6e232480dd1e675ef1dcb7baa429671ef74140","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"38c66061_cd06899b","updated":"2021-11-16 17:39:01.000000000","message":"Thanks Fábio for the review. I\u0027ve updated in the latest patchset. Thanks :)","commit_id":"932eb3980f6d1ed8a6b58420eb550cb34d0ccc81"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"7e43bbdd77ffbaf4d163e644a4c6fb46b5e56204","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"cca9f1ac_72eeae98","updated":"2021-11-22 09:19:03.000000000","message":"Hi Fábio, Thanks a lot for the detailed review and really sorry messing things up here. I have updated a new patchset. You can take a look now :) Thanks again!","commit_id":"9f470282f90c013ba7da8be488a998d2cab5324e"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"6730bde2dfb08a57006fafead883809775856a46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"9a477bd0_d82873b0","updated":"2021-11-25 13:22:53.000000000","message":"recheck\nIssue in the CI with the share type creation has been fixed","commit_id":"9f470282f90c013ba7da8be488a998d2cab5324e"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"eb517e21_adf68d29","updated":"2022-02-16 17:36:37.000000000","message":"Thank you for working on this :)\n\nI added some comments, please check them inline.","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"667f2e6f15f09383a273d2e3cffd54aab2e99fb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"5fe5296f_1ef784ee","updated":"2022-02-10 16:17:49.000000000","message":"recheck","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"12fadbcccdce80e4fa24be5356f1271a5af561d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7049005b_798de719","updated":"2022-01-18 17:35:09.000000000","message":"recheck\n","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"7e766215_64c5ac32","updated":"2022-04-09 10:09:30.000000000","message":"Hi Archana, as discussed in the PTG session this week, I added myself as a co-author to this patch to take some pressure off you.\nFabio, thanks for a great review, I believe while testing and fixing I\u0027ve addressed most of your comments as well :) ","commit_id":"9738429067d0c77753223deb7d5046923f16c64a"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"f19e47e634cbc3b889396436ee8fea48dea96b2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"87d3c102_d1e379d2","in_reply_to":"7e766215_64c5ac32","updated":"2022-05-17 16:10:32.000000000","message":"Hi Maari, Thank you so much for adding yourself here on this patch and making zuul happy too :) Also thanks for all the reviews and help!","commit_id":"9738429067d0c77753223deb7d5046923f16c64a"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"cade55cf6eb610389786de59affa65f72a3757c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"9969dca0_0d2068e9","updated":"2022-07-14 21:57:45.000000000","message":"Actually voting +1, since I agree the message mentioned by haixin could use a change :)","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0ecaec1902751a3239c2e8be0b78a6bb20ff9fb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"e253ac4e_180e4b4d","updated":"2022-07-14 21:57:21.000000000","message":"Thank you for working on this change, Archana and Maari!\nI had a chance to test this change and it is looking great. Thank you!","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"93fcbb62_72749659","updated":"2022-07-12 04:42:35.000000000","message":"thanks for your change, here are some comments.","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"90285f76a8f400d1d596629706e4ab7844d97ccd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"de73dd77_2f93b6e5","updated":"2022-07-15 14:31:02.000000000","message":"Code looks very good, thanks Archana and Maari!","commit_id":"301274dbfe4466329916c589ddf671d38c4d3689"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"e112f71e5d368db516ffe4c537235cdc16976af2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"c0171f3b_ef03464e","updated":"2022-07-15 18:30:50.000000000","message":"Looking good! Thank you for the work on this change","commit_id":"301274dbfe4466329916c589ddf671d38c4d3689"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"3233790efef9a3078a8f98498d5ba420eb03e60e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"e1756cb4_ebf63ee3","updated":"2022-07-15 10:13:20.000000000","message":"Thanks Haixin and Carlos! I\u0027ve updated the patch, please have another look :) ","commit_id":"301274dbfe4466329916c589ddf671d38c4d3689"}],"doc/source/cli/osc/v2/index.rst":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"658d0f4a6f125c8a10b6ccf3e5618d3af8349cce","unresolved":true,"context_lines":[{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":153,"context_line":"    :command: share group *"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":156,"context_line":"share group snapshots"}],"source_content_type":"text/x-rst","patch_set":12,"id":"d0db4ee8_13d4f20d","line":153,"range":{"start_line":153,"start_character":14,"end_line":153,"end_character":27},"updated":"2021-10-13 17:28:34.000000000","message":"The items under share group snapshots will be displayed here.\nyou might need to something like this: `:command: share group [!s]*`\nthis :command: syntax accepts fnmatch style https://docs.python.org/3/library/fnmatch.html","commit_id":"b4a0e9bf87707a0e2886c9fe32c848c3f007a8e1"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"b2a339c9934eee8efb2ab47c766820fd72e57d05","unresolved":false,"context_lines":[{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":153,"context_line":"    :command: share group *"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":156,"context_line":"share group snapshots"}],"source_content_type":"text/x-rst","patch_set":12,"id":"4213b36b_0284d2f9","line":153,"range":{"start_line":153,"start_character":14,"end_line":153,"end_character":27},"in_reply_to":"d0db4ee8_13d4f20d","updated":"2021-11-07 05:53:31.000000000","message":"Ack","commit_id":"b4a0e9bf87707a0e2886c9fe32c848c3f007a8e1"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"5dedc08d99470e1e9391023d2a8db9a26b997c88","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"2d8b5cd8_a658bed3","line":154,"updated":"2021-11-09 15:27:52.000000000","message":"Sorry, I totally forgot about the \"share group set\", you might need to add this in order to have it included on the docs.\n\n.. autoprogram-cliff:: openstack.share.v2\n    :command: share group set","commit_id":"8e8a4420bf8de14e6583f101764d6c28e5dc1917"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"bc6e232480dd1e675ef1dcb7baa429671ef74140","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"dcf76ea2_227a3d2e","line":154,"in_reply_to":"08d87e57_5fdeff70","updated":"2021-11-16 17:39:01.000000000","message":"Done","commit_id":"8e8a4420bf8de14e6583f101764d6c28e5dc1917"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"8a235e9620de60db364b878cdb1abf85873898dc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"08d87e57_5fdeff70","line":154,"in_reply_to":"2d8b5cd8_a658bed3","updated":"2021-11-09 15:34:35.000000000","message":"https://docs.openstack.org/python-manilaclient/latest/cli/osc_plugin_cli.html\n\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_853/802791/14/check/openstack-tox-docs/853fc0c/docs/cli/osc_plugin_cli.html","commit_id":"8e8a4420bf8de14e6583f101764d6c28e5dc1917"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"bfc45606fb93abe62589a2483d85a2830a2c5ab2","unresolved":true,"context_lines":[{"line_number":149,"context_line":"share groups"},{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":153,"context_line":"    :command: share group set"}],"source_content_type":"text/x-rst","patch_set":15,"id":"0caf885d_80a45c38","line":153,"range":{"start_line":152,"start_character":0,"end_line":153,"end_character":29},"updated":"2021-11-17 19:27:30.000000000","message":"You will need both blocks of code:\n\n  .. autoprogram-cliff:: openstack.share.v2\n      :command: share group [!s]* \n\n  .. autoprogram-cliff:: openstack.share.v2\n      :command: share group set\n\nThe first one will take any share group command that doesnt have a word starting with \u0027s\u0027 - avoiding the duplication of share group snapshot commands but also excluding share group set.\n\nThe second one will take only the share group set, since the first one will exclude him from the list.","commit_id":"932eb3980f6d1ed8a6b58420eb550cb34d0ccc81"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":false,"context_lines":[{"line_number":149,"context_line":"share groups"},{"line_number":150,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":153,"context_line":"    :command: share group set"}],"source_content_type":"text/x-rst","patch_set":15,"id":"c5e88ffe_ea8bc4c8","line":153,"range":{"start_line":152,"start_character":0,"end_line":153,"end_character":29},"in_reply_to":"0caf885d_80a45c38","updated":"2022-02-16 17:36:37.000000000","message":"Done","commit_id":"932eb3980f6d1ed8a6b58420eb550cb34d0ccc81"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"bfc45606fb93abe62589a2483d85a2830a2c5ab2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"2555f5e3_d20d2103","line":154,"updated":"2021-11-17 19:27:30.000000000","message":"You might need to add a section for Share Group Snapshots, otherwise it wont appear on the docs.\n\n  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n  Share Group Snapshot\n  \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\n  .. autoprogram-cliff:: openstack.share.v2\n      :command: share group snapshot *","commit_id":"932eb3980f6d1ed8a6b58420eb550cb34d0ccc81"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"774bd46e_d2487302","line":154,"in_reply_to":"2555f5e3_d20d2103","updated":"2022-02-16 17:36:37.000000000","message":"Done","commit_id":"932eb3980f6d1ed8a6b58420eb550cb34d0ccc81"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":47,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":48,"context_line":"    :command: share revert"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":51,"context_line":"    :command: share group set"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":54,"context_line":"share instances"}],"source_content_type":"text/x-rst","patch_set":17,"id":"e439e64f_81c416e2","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":29},"updated":"2022-02-16 17:36:37.000000000","message":"this shouldn\u0027t be here","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[{"line_number":47,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":48,"context_line":"    :command: share revert"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":51,"context_line":"    :command: share group set"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":54,"context_line":"share instances"}],"source_content_type":"text/x-rst","patch_set":17,"id":"7f175013_5c87ed55","line":51,"range":{"start_line":50,"start_character":0,"end_line":51,"end_character":29},"in_reply_to":"e439e64f_81c416e2","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":165,"context_line":"share groups"},{"line_number":166,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":169,"context_line":"    :command: share group [!t]*"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":172,"context_line":"share group types"}],"source_content_type":"text/x-rst","patch_set":17,"id":"c1b8d2fb_672d0abb","line":169,"range":{"start_line":168,"start_character":0,"end_line":169,"end_character":31},"updated":"2022-02-16 17:36:37.000000000","message":"this section should be:\n\n.. autoprogram-cliff:: openstack.share.v2\n    :command: share group [!ts]*\n\n.. autoprogram-cliff:: openstack.share.v2\n    :command: share group set","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[{"line_number":165,"context_line":"share groups"},{"line_number":166,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":169,"context_line":"    :command: share group [!t]*"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":172,"context_line":"share group types"}],"source_content_type":"text/x-rst","patch_set":17,"id":"cb992fe8_653321b8","line":169,"range":{"start_line":168,"start_character":0,"end_line":169,"end_character":31},"in_reply_to":"c1b8d2fb_672d0abb","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":174,"context_line":""},{"line_number":175,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":176,"context_line":"    :command: share group type *"},{"line_number":177,"context_line":"      :command: share group [!s]*"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":180,"context_line":"share group snapshots"}],"source_content_type":"text/x-rst","patch_set":17,"id":"df6546c7_071f62e5","line":177,"range":{"start_line":177,"start_character":6,"end_line":177,"end_character":33},"updated":"2022-02-16 17:36:37.000000000","message":"this shouldn\u0027t be here also","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[{"line_number":174,"context_line":""},{"line_number":175,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":176,"context_line":"    :command: share group type *"},{"line_number":177,"context_line":"      :command: share group [!s]*"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":180,"context_line":"share group snapshots"}],"source_content_type":"text/x-rst","patch_set":17,"id":"19bef3ba_d854d369","line":177,"range":{"start_line":177,"start_character":6,"end_line":177,"end_character":33},"in_reply_to":"df6546c7_071f62e5","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"}],"manilaclient/osc/v2/share_group_snapshots.py":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"658d0f4a6f125c8a10b6ccf3e5618d3af8349cce","unresolved":true,"context_lines":[{"line_number":214,"context_line":"                **kwargs"},{"line_number":215,"context_line":"            )"},{"line_number":216,"context_line":"        except Exception as e:"},{"line_number":217,"context_line":"            result +\u003d 1"},{"line_number":218,"context_line":"            LOG.error(_("},{"line_number":219,"context_line":"                \"Failed to set share group snapshot properties \""},{"line_number":220,"context_line":"                \"\u0027%(properties)s\u0027: %(exception)s\"),"}],"source_content_type":"text/x-python","patch_set":12,"id":"224e9c2f_3905c5b8","line":217,"range":{"start_line":217,"start_character":12,"end_line":217,"end_character":23},"updated":"2021-10-13 17:28:34.000000000","message":"I can be mistaken, but since we are not inside a loop, you don\u0027t need an exception counter and the exception can be raised here inside the `except` also. Just like the Unset method","commit_id":"b4a0e9bf87707a0e2886c9fe32c848c3f007a8e1"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"b2a339c9934eee8efb2ab47c766820fd72e57d05","unresolved":false,"context_lines":[{"line_number":214,"context_line":"                **kwargs"},{"line_number":215,"context_line":"            )"},{"line_number":216,"context_line":"        except Exception as e:"},{"line_number":217,"context_line":"            result +\u003d 1"},{"line_number":218,"context_line":"            LOG.error(_("},{"line_number":219,"context_line":"                \"Failed to set share group snapshot properties \""},{"line_number":220,"context_line":"                \"\u0027%(properties)s\u0027: %(exception)s\"),"}],"source_content_type":"text/x-python","patch_set":12,"id":"ffcdc0b2_5c732750","line":217,"range":{"start_line":217,"start_character":12,"end_line":217,"end_character":23},"in_reply_to":"224e9c2f_3905c5b8","updated":"2021-11-07 05:53:31.000000000","message":"Done","commit_id":"b4a0e9bf87707a0e2886c9fe32c848c3f007a8e1"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":319,"context_line":"                   \"(default: name:asc), multiple keys and directions can be \""},{"line_number":320,"context_line":"                   \"specified separated by comma\")"},{"line_number":321,"context_line":"        )"},{"line_number":322,"context_line":"        parser.add_argument("},{"line_number":323,"context_line":"            \"--detailed\","},{"line_number":324,"context_line":"            action\u003d\"store_true\","},{"line_number":325,"context_line":"            help\u003d_(\"Show detailed information about share group snapshot. \")"},{"line_number":326,"context_line":"        )"},{"line_number":327,"context_line":"        return parser"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":17,"id":"08ece7b5_84407319","line":326,"range":{"start_line":322,"start_character":8,"end_line":326,"end_character":9},"updated":"2022-02-16 17:36:37.000000000","message":"there is no difference between having detailed as true or false. I canse that this is a problem from shell file [0] itself. we could take a look on it now or address a new patch to fix it in the future.\n\n[0] https://opendev.org/openstack/python-manilaclient/src/branch/master/manilaclient/v2/shell.py#L5817","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":true,"context_lines":[{"line_number":319,"context_line":"                   \"(default: name:asc), multiple keys and directions can be \""},{"line_number":320,"context_line":"                   \"specified separated by comma\")"},{"line_number":321,"context_line":"        )"},{"line_number":322,"context_line":"        parser.add_argument("},{"line_number":323,"context_line":"            \"--detailed\","},{"line_number":324,"context_line":"            action\u003d\"store_true\","},{"line_number":325,"context_line":"            help\u003d_(\"Show detailed information about share group snapshot. \")"},{"line_number":326,"context_line":"        )"},{"line_number":327,"context_line":"        return parser"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":17,"id":"2750afff_d6d14cf0","line":326,"range":{"start_line":322,"start_character":8,"end_line":326,"end_character":9},"in_reply_to":"08ece7b5_84407319","updated":"2022-04-09 10:09:30.000000000","message":"Thanks for pointing this out, I saw that additional options we could possibly display are \"Created At\" and \"Share Group ID\" so I added these here.","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0ecaec1902751a3239c2e8be0b78a6bb20ff9fb2","unresolved":false,"context_lines":[{"line_number":319,"context_line":"                   \"(default: name:asc), multiple keys and directions can be \""},{"line_number":320,"context_line":"                   \"specified separated by comma\")"},{"line_number":321,"context_line":"        )"},{"line_number":322,"context_line":"        parser.add_argument("},{"line_number":323,"context_line":"            \"--detailed\","},{"line_number":324,"context_line":"            action\u003d\"store_true\","},{"line_number":325,"context_line":"            help\u003d_(\"Show detailed information about share group snapshot. \")"},{"line_number":326,"context_line":"        )"},{"line_number":327,"context_line":"        return parser"},{"line_number":328,"context_line":""},{"line_number":329,"context_line":"    def take_action(self, parsed_args):"}],"source_content_type":"text/x-python","patch_set":17,"id":"f5eae2f9_fa53a6a3","line":326,"range":{"start_line":322,"start_character":8,"end_line":326,"end_character":9},"in_reply_to":"2750afff_d6d14cf0","updated":"2022-07-14 21:57:21.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":true,"context_lines":[{"line_number":246,"context_line":"                )"},{"line_number":247,"context_line":"            except Exception as e:"},{"line_number":248,"context_line":"                result +\u003d 1"},{"line_number":249,"context_line":"                LOG.error(\u0027Failed to update name the status for \u0027"},{"line_number":250,"context_line":"                          \u0027share group snapshot with ID \u0027"},{"line_number":251,"context_line":"                          f\u0027{share_group_snapshot.id}: {e}\u0027)"},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":21,"id":"85d98d0e_0aff39e8","line":249,"range":{"start_line":249,"start_character":27,"end_line":249,"end_character":63},"updated":"2022-07-12 04:42:35.000000000","message":"Failed to set status for the share group snapshot with Id .....","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":true,"context_lines":[{"line_number":246,"context_line":"                )"},{"line_number":247,"context_line":"            except Exception as e:"},{"line_number":248,"context_line":"                result +\u003d 1"},{"line_number":249,"context_line":"                LOG.error(\u0027Failed to update name the status for \u0027"},{"line_number":250,"context_line":"                          \u0027share group snapshot with ID \u0027"},{"line_number":251,"context_line":"                          f\u0027{share_group_snapshot.id}: {e}\u0027)"},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":21,"id":"eda1e8eb_f3305e89","line":249,"updated":"2022-07-12 04:42:35.000000000","message":"Same as set function above. maybe looks better.\n\nLOG.error(\u0027Failed to unset name or description for \u0027\n          \u0027share group snapshot with ID \u0027\n          f\u0027{share_group_snapshot.id}: {e}\u0027)","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"3233790efef9a3078a8f98498d5ba420eb03e60e","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                )"},{"line_number":247,"context_line":"            except Exception as e:"},{"line_number":248,"context_line":"                result +\u003d 1"},{"line_number":249,"context_line":"                LOG.error(\u0027Failed to update name the status for \u0027"},{"line_number":250,"context_line":"                          \u0027share group snapshot with ID \u0027"},{"line_number":251,"context_line":"                          f\u0027{share_group_snapshot.id}: {e}\u0027)"},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":21,"id":"32b244c6_d73b04ee","line":249,"range":{"start_line":249,"start_character":27,"end_line":249,"end_character":63},"in_reply_to":"5828b561_79ee82e1","updated":"2022-07-15 10:13:20.000000000","message":"Done","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"a496b9ebde6c79010f846257e96bbc334922f919","unresolved":true,"context_lines":[{"line_number":246,"context_line":"                )"},{"line_number":247,"context_line":"            except Exception as e:"},{"line_number":248,"context_line":"                result +\u003d 1"},{"line_number":249,"context_line":"                LOG.error(\u0027Failed to update name the status for \u0027"},{"line_number":250,"context_line":"                          \u0027share group snapshot with ID \u0027"},{"line_number":251,"context_line":"                          f\u0027{share_group_snapshot.id}: {e}\u0027)"},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":21,"id":"5828b561_79ee82e1","line":249,"range":{"start_line":249,"start_character":27,"end_line":249,"end_character":63},"in_reply_to":"85d98d0e_0aff39e8","updated":"2022-07-14 21:59:21.000000000","message":"++ - as this exception will only be thrown for status reset failure","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"3233790efef9a3078a8f98498d5ba420eb03e60e","unresolved":false,"context_lines":[{"line_number":246,"context_line":"                )"},{"line_number":247,"context_line":"            except Exception as e:"},{"line_number":248,"context_line":"                result +\u003d 1"},{"line_number":249,"context_line":"                LOG.error(\u0027Failed to update name the status for \u0027"},{"line_number":250,"context_line":"                          \u0027share group snapshot with ID \u0027"},{"line_number":251,"context_line":"                          f\u0027{share_group_snapshot.id}: {e}\u0027)"},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":21,"id":"7a95d3a5_f91dd434","line":249,"in_reply_to":"eda1e8eb_f3305e89","updated":"2022-07-15 10:13:20.000000000","message":"Done","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":true,"context_lines":[{"line_number":342,"context_line":"            action\u003dparseractions.NonNegativeAction,"},{"line_number":343,"context_line":"            help\u003d_(\"Limit the number of share groups returned\")"},{"line_number":344,"context_line":"        )"},{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \"--marker\","},{"line_number":347,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","},{"line_number":348,"context_line":"            help\u003d_(\"The last share group snapshot ID of the \""},{"line_number":349,"context_line":"                   \"previous page\")"},{"line_number":350,"context_line":"        )"},{"line_number":351,"context_line":"        parser.add_argument("},{"line_number":352,"context_line":"            \u0027--sort\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"961c8a91_246f2e38","line":349,"range":{"start_line":345,"start_character":7,"end_line":349,"end_character":35},"updated":"2022-07-12 04:42:35.000000000","message":"no query share group snaps not support marker, you can see:\nmanila/api/v2/share_group_snapshots.py#129   and \nmanila/api/common.py#167\nhere shoud be offset.\n\nparser.add_argument(\n            \"--offset\",\n            metavar\u003d\"\u003coffset\u003e\",\n            type\u003dint,\n            default\u003dNone,\n            action\u003dparseractions.NonNegativeAction,\n            help\u003d_(\"Start position of share group snapshot listing.\")\n        )","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"4600baca7a34f13f6d39dd45541c45ff297664d0","unresolved":true,"context_lines":[{"line_number":342,"context_line":"            action\u003dparseractions.NonNegativeAction,"},{"line_number":343,"context_line":"            help\u003d_(\"Limit the number of share groups returned\")"},{"line_number":344,"context_line":"        )"},{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \"--marker\","},{"line_number":347,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","},{"line_number":348,"context_line":"            help\u003d_(\"The last share group snapshot ID of the \""},{"line_number":349,"context_line":"                   \"previous page\")"},{"line_number":350,"context_line":"        )"},{"line_number":351,"context_line":"        parser.add_argument("},{"line_number":352,"context_line":"            \u0027--sort\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"decca626_68b1635b","line":349,"range":{"start_line":345,"start_character":7,"end_line":349,"end_character":35},"in_reply_to":"04aa4290_b1187366","updated":"2022-07-15 01:58:42.000000000","message":"ok, It looks like we\u0027re using marker as offset in OSC, but i think we also need add type\u003dint and action\u003dparseractions.NonNegativeAction,.","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"662be28d961b200b947cf158a8f452f595c7bb8a","unresolved":false,"context_lines":[{"line_number":342,"context_line":"            action\u003dparseractions.NonNegativeAction,"},{"line_number":343,"context_line":"            help\u003d_(\"Limit the number of share groups returned\")"},{"line_number":344,"context_line":"        )"},{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \"--marker\","},{"line_number":347,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","},{"line_number":348,"context_line":"            help\u003d_(\"The last share group snapshot ID of the \""},{"line_number":349,"context_line":"                   \"previous page\")"},{"line_number":350,"context_line":"        )"},{"line_number":351,"context_line":"        parser.add_argument("},{"line_number":352,"context_line":"            \u0027--sort\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"af213749_a1c20241","line":349,"range":{"start_line":345,"start_character":7,"end_line":349,"end_character":35},"in_reply_to":"73ab1f9f_667c7d7f","updated":"2022-07-18 04:11:34.000000000","message":"hi, Maari Tamm, thanks. I know what you mean, My point is that Manila does not support Maker[1][2]. only support offset and limit. \nhere in line 388, \n\u0027offset\u0027: parsed_args.marker,\nwe just pass the parameter value of the marker as offset to Manila. so if we give\nshare group snapshot ID(not int). i think we will get error in api/common.py.\n\n[1] https://opendev.org/openstack/manila/src/branch/master/manila/api/v2/share_group_snapshots.py#L129\n[2] https://opendev.org/openstack/manila/src/branch/master/manila/api/common.py#L167","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0ecaec1902751a3239c2e8be0b78a6bb20ff9fb2","unresolved":true,"context_lines":[{"line_number":342,"context_line":"            action\u003dparseractions.NonNegativeAction,"},{"line_number":343,"context_line":"            help\u003d_(\"Limit the number of share groups returned\")"},{"line_number":344,"context_line":"        )"},{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \"--marker\","},{"line_number":347,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","},{"line_number":348,"context_line":"            help\u003d_(\"The last share group snapshot ID of the \""},{"line_number":349,"context_line":"                   \"previous page\")"},{"line_number":350,"context_line":"        )"},{"line_number":351,"context_line":"        parser.add_argument("},{"line_number":352,"context_line":"            \u0027--sort\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"04aa4290_b1187366","line":349,"range":{"start_line":345,"start_character":7,"end_line":349,"end_character":35},"in_reply_to":"961c8a91_246f2e38","updated":"2022-07-14 21:57:21.000000000","message":"Haixin, I understand your point here, but in OSC we have been changing this nomenclature, see [1], [2] and [3]. Shares, snapshots and share groups now have --marker instead of offset :)\n\n[1] https://github.com/openstack/python-manilaclient/blob/89f7f2728b10f318b1dfb4ed50c0d3f6b209e60c/manilaclient/osc/v2/share_groups.py#L296\n[2] https://github.com/openstack/python-manilaclient/blob/91c0d7980713aecc220b7adaed1c97b4d88b3cc8/manilaclient/osc/v2/share_snapshots.py#L379\n[3] https://github.com/openstack/python-manilaclient/blob/3c25e506cf8cdfc0be6f5cd7dd1c4348893449bc/manilaclient/osc/v2/share.py#L474","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"3233790efef9a3078a8f98498d5ba420eb03e60e","unresolved":false,"context_lines":[{"line_number":342,"context_line":"            action\u003dparseractions.NonNegativeAction,"},{"line_number":343,"context_line":"            help\u003d_(\"Limit the number of share groups returned\")"},{"line_number":344,"context_line":"        )"},{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \"--marker\","},{"line_number":347,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","},{"line_number":348,"context_line":"            help\u003d_(\"The last share group snapshot ID of the \""},{"line_number":349,"context_line":"                   \"previous page\")"},{"line_number":350,"context_line":"        )"},{"line_number":351,"context_line":"        parser.add_argument("},{"line_number":352,"context_line":"            \u0027--sort\u0027,"}],"source_content_type":"text/x-python","patch_set":21,"id":"73ab1f9f_667c7d7f","line":349,"range":{"start_line":345,"start_character":7,"end_line":349,"end_character":35},"in_reply_to":"decca626_68b1635b","updated":"2022-07-15 10:13:20.000000000","message":"Haixin, I think you might be mixing up offset/marker and limit. Marker is not an int, it\u0027s the \"The last share group snapshot ID of the previous page\" and limit is the number of items to be returned :)","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":true,"context_lines":[{"line_number":372,"context_line":"                share_client.share_groups,"},{"line_number":373,"context_line":"                parsed_args.share_group).id"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        columns \u003d ["},{"line_number":376,"context_line":"            \u0027ID\u0027,"},{"line_number":377,"context_line":"            \u0027Name\u0027,"},{"line_number":378,"context_line":"            \u0027Status\u0027,"},{"line_number":379,"context_line":"            \u0027Description\u0027,"},{"line_number":380,"context_line":"        ]"},{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        search_opts \u003d {"},{"line_number":383,"context_line":"            \u0027all_tenants\u0027: parsed_args.all_projects,"}],"source_content_type":"text/x-python","patch_set":21,"id":"046d6389_681ad30b","line":380,"range":{"start_line":375,"start_character":0,"end_line":380,"end_character":9},"updated":"2022-07-12 04:42:35.000000000","message":"in manila/api/views/share_group_snapshots.py   line 57---\u003eline  84\nwe can see summary only  have id and name.\n\ncolumns \u003d [\n            \u0027ID\u0027,\n            \u0027Name\u0027,\n        ]","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0ecaec1902751a3239c2e8be0b78a6bb20ff9fb2","unresolved":false,"context_lines":[{"line_number":372,"context_line":"                share_client.share_groups,"},{"line_number":373,"context_line":"                parsed_args.share_group).id"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        columns \u003d ["},{"line_number":376,"context_line":"            \u0027ID\u0027,"},{"line_number":377,"context_line":"            \u0027Name\u0027,"},{"line_number":378,"context_line":"            \u0027Status\u0027,"},{"line_number":379,"context_line":"            \u0027Description\u0027,"},{"line_number":380,"context_line":"        ]"},{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        search_opts \u003d {"},{"line_number":383,"context_line":"            \u0027all_tenants\u0027: parsed_args.all_projects,"}],"source_content_type":"text/x-python","patch_set":21,"id":"b357f332_c8257389","line":380,"range":{"start_line":375,"start_character":0,"end_line":380,"end_character":9},"in_reply_to":"046d6389_681ad30b","updated":"2022-07-14 21:57:21.000000000","message":"Actually I think the code is right the way it is. The view will return the fields according to the type of listing you have requested. In this case, it is actually returning all of the 4 mentioned fields. That\u0027s even what is happening in manilaclient currently, see [1]\n\n[1] https://paste.opendev.org/show/b5E0euUf4FBsaaxcIfqP/","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"4600baca7a34f13f6d39dd45541c45ff297664d0","unresolved":false,"context_lines":[{"line_number":372,"context_line":"                share_client.share_groups,"},{"line_number":373,"context_line":"                parsed_args.share_group).id"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        columns \u003d ["},{"line_number":376,"context_line":"            \u0027ID\u0027,"},{"line_number":377,"context_line":"            \u0027Name\u0027,"},{"line_number":378,"context_line":"            \u0027Status\u0027,"},{"line_number":379,"context_line":"            \u0027Description\u0027,"},{"line_number":380,"context_line":"        ]"},{"line_number":381,"context_line":""},{"line_number":382,"context_line":"        search_opts \u003d {"},{"line_number":383,"context_line":"            \u0027all_tenants\u0027: parsed_args.all_projects,"}],"source_content_type":"text/x-python","patch_set":21,"id":"e45cd062_809b997c","line":380,"range":{"start_line":375,"start_character":0,"end_line":380,"end_character":9},"in_reply_to":"b357f332_c8257389","updated":"2022-07-15 01:58:42.000000000","message":"ack.","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":true,"context_lines":[{"line_number":385,"context_line":"            \u0027status\u0027: parsed_args.status,"},{"line_number":386,"context_line":"            \u0027share_group_id\u0027: share_group_id,"},{"line_number":387,"context_line":"            \u0027limit\u0027: parsed_args.limit,"},{"line_number":388,"context_line":"            \u0027offset\u0027: parsed_args.marker,"},{"line_number":389,"context_line":"        }"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        if parsed_args.detailed:"}],"source_content_type":"text/x-python","patch_set":21,"id":"bd396b7a_39627dd1","line":388,"range":{"start_line":388,"start_character":34,"end_line":388,"end_character":40},"updated":"2022-07-12 04:42:35.000000000","message":"offset","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"4600baca7a34f13f6d39dd45541c45ff297664d0","unresolved":false,"context_lines":[{"line_number":385,"context_line":"            \u0027status\u0027: parsed_args.status,"},{"line_number":386,"context_line":"            \u0027share_group_id\u0027: share_group_id,"},{"line_number":387,"context_line":"            \u0027limit\u0027: parsed_args.limit,"},{"line_number":388,"context_line":"            \u0027offset\u0027: parsed_args.marker,"},{"line_number":389,"context_line":"        }"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        if parsed_args.detailed:"}],"source_content_type":"text/x-python","patch_set":21,"id":"c82473a9_e6ca5877","line":388,"range":{"start_line":388,"start_character":34,"end_line":388,"end_character":40},"in_reply_to":"0d916a98_a94d7d4a","updated":"2022-07-15 01:58:42.000000000","message":"Done","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0ecaec1902751a3239c2e8be0b78a6bb20ff9fb2","unresolved":true,"context_lines":[{"line_number":385,"context_line":"            \u0027status\u0027: parsed_args.status,"},{"line_number":386,"context_line":"            \u0027share_group_id\u0027: share_group_id,"},{"line_number":387,"context_line":"            \u0027limit\u0027: parsed_args.limit,"},{"line_number":388,"context_line":"            \u0027offset\u0027: parsed_args.marker,"},{"line_number":389,"context_line":"        }"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        if parsed_args.detailed:"}],"source_content_type":"text/x-python","patch_set":21,"id":"0d916a98_a94d7d4a","line":388,"range":{"start_line":388,"start_character":34,"end_line":388,"end_character":40},"in_reply_to":"bd396b7a_39627dd1","updated":"2022-07-14 21:57:21.000000000","message":"Same answer as above","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"a5d74d33664af1be82ee21c5d1098273b00d4da7","unresolved":true,"context_lines":[{"line_number":388,"context_line":"            \u0027offset\u0027: parsed_args.marker,"},{"line_number":389,"context_line":"        }"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        if parsed_args.detailed:"},{"line_number":392,"context_line":"            columns.extend(["},{"line_number":393,"context_line":"                \u0027Created At\u0027,"},{"line_number":394,"context_line":"                \u0027Share Group ID\u0027,"},{"line_number":395,"context_line":"            ])"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        if parsed_args.all_projects:"},{"line_number":398,"context_line":"            columns.append(\u0027Project ID\u0027)"}],"source_content_type":"text/x-python","patch_set":21,"id":"c5cedd28_c1f00c5f","line":395,"range":{"start_line":391,"start_character":8,"end_line":395,"end_character":14},"updated":"2022-07-12 04:42:35.000000000","message":"in manila/api/views/share_group_snapshots.py  \ndef detail(self, request, share_group_snap):\n\nif parsed_args.detailed:\n            columns.extend([\n                \u0027Created At\u0027,\n                \u0027Share Group ID\u0027,\n                \u0027Status\u0027,\n                \u0027Description\u0027,\n                \u0027Project ID\u0027,\n                \u0027Share Group ID\u0027,\n                \u0027Members\u0027,\n            ])","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"4600baca7a34f13f6d39dd45541c45ff297664d0","unresolved":false,"context_lines":[{"line_number":388,"context_line":"            \u0027offset\u0027: parsed_args.marker,"},{"line_number":389,"context_line":"        }"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        if parsed_args.detailed:"},{"line_number":392,"context_line":"            columns.extend(["},{"line_number":393,"context_line":"                \u0027Created At\u0027,"},{"line_number":394,"context_line":"                \u0027Share Group ID\u0027,"},{"line_number":395,"context_line":"            ])"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        if parsed_args.all_projects:"},{"line_number":398,"context_line":"            columns.append(\u0027Project ID\u0027)"}],"source_content_type":"text/x-python","patch_set":21,"id":"f952be9f_3a6ba2d8","line":395,"range":{"start_line":391,"start_character":8,"end_line":395,"end_character":14},"in_reply_to":"8a368ad3_00760f94","updated":"2022-07-15 01:58:42.000000000","message":"ack.","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"0ecaec1902751a3239c2e8be0b78a6bb20ff9fb2","unresolved":false,"context_lines":[{"line_number":388,"context_line":"            \u0027offset\u0027: parsed_args.marker,"},{"line_number":389,"context_line":"        }"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        if parsed_args.detailed:"},{"line_number":392,"context_line":"            columns.extend(["},{"line_number":393,"context_line":"                \u0027Created At\u0027,"},{"line_number":394,"context_line":"                \u0027Share Group ID\u0027,"},{"line_number":395,"context_line":"            ])"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        if parsed_args.all_projects:"},{"line_number":398,"context_line":"            columns.append(\u0027Project ID\u0027)"}],"source_content_type":"text/x-python","patch_set":21,"id":"8a368ad3_00760f94","line":395,"range":{"start_line":391,"start_character":8,"end_line":395,"end_character":14},"in_reply_to":"c5cedd28_c1f00c5f","updated":"2022-07-14 21:57:21.000000000","message":"I think only adding the created at and ID would be enough, as currently manilaclient does not add members as well, nor project_id... Project ID would only be in case of all_tenants set to 1","commit_id":"5d375871e1d6078c449f6d234a9791232a9f35ec"}],"manilaclient/tests/unit/osc/v2/fakes.py":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"bd40a2340a522cf57c7214c27f8fa7d68ba2e450","unresolved":true,"context_lines":[{"line_number":36,"context_line":"        self.quotas \u003d mock.Mock()"},{"line_number":37,"context_line":"        self.share_snapshots \u003d mock.Mock()"},{"line_number":38,"context_line":"        self.share_snapshot_export_locations \u003d mock.Mock()"},{"line_number":39,"context_line":"        self.share_replicas \u003d mock.Mock()"},{"line_number":40,"context_line":"        self.shares.resource_class \u003d osc_fakes.FakeResource(None, {})"},{"line_number":41,"context_line":"        self.share_export_locations \u003d mock.Mock()"},{"line_number":42,"context_line":"        self.share_export_locations.resource_class \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"67efe0b4_7901cd81","side":"PARENT","line":39,"updated":"2021-08-10 08:50:53.000000000","message":"This shouldn\u0027t be removed","commit_id":"a658526cdbeade53b5bb05b83030970755272213"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"268565068fbf987c280f69a9a879f15cef3b36a6","unresolved":false,"context_lines":[{"line_number":36,"context_line":"        self.quotas \u003d mock.Mock()"},{"line_number":37,"context_line":"        self.share_snapshots \u003d mock.Mock()"},{"line_number":38,"context_line":"        self.share_snapshot_export_locations \u003d mock.Mock()"},{"line_number":39,"context_line":"        self.share_replicas \u003d mock.Mock()"},{"line_number":40,"context_line":"        self.shares.resource_class \u003d osc_fakes.FakeResource(None, {})"},{"line_number":41,"context_line":"        self.share_export_locations \u003d mock.Mock()"},{"line_number":42,"context_line":"        self.share_export_locations.resource_class \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"a1f30a1e_94af7967","side":"PARENT","line":39,"in_reply_to":"67efe0b4_7901cd81","updated":"2021-08-11 07:54:27.000000000","message":"Ack","commit_id":"a658526cdbeade53b5bb05b83030970755272213"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"bd40a2340a522cf57c7214c27f8fa7d68ba2e450","unresolved":true,"context_lines":[{"line_number":611,"context_line":"        attrs \u003d attrs or {}"},{"line_number":612,"context_line":"        methods \u003d methods or {}"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"        share_replica \u003d {"},{"line_number":615,"context_line":"            \u0027availability_zone\u0027: None,"},{"line_number":616,"context_line":"            \u0027cast_rules_to_readonly\u0027: True,"},{"line_number":617,"context_line":"            \u0027created_at\u0027: datetime.datetime.now().isoformat(),"}],"source_content_type":"text/x-python","patch_set":3,"id":"5be62c26_a5adfe21","side":"PARENT","line":614,"updated":"2021-08-10 08:50:53.000000000","message":"You shouldn\u0027t modify this class. You need to create a new fake class for share group snapshots with similar functions","commit_id":"a658526cdbeade53b5bb05b83030970755272213"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"268565068fbf987c280f69a9a879f15cef3b36a6","unresolved":false,"context_lines":[{"line_number":611,"context_line":"        attrs \u003d attrs or {}"},{"line_number":612,"context_line":"        methods \u003d methods or {}"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"        share_replica \u003d {"},{"line_number":615,"context_line":"            \u0027availability_zone\u0027: None,"},{"line_number":616,"context_line":"            \u0027cast_rules_to_readonly\u0027: True,"},{"line_number":617,"context_line":"            \u0027created_at\u0027: datetime.datetime.now().isoformat(),"}],"source_content_type":"text/x-python","patch_set":3,"id":"f3b625f1_a24601db","side":"PARENT","line":614,"in_reply_to":"5be62c26_a5adfe21","updated":"2021-08-11 07:54:27.000000000","message":"Done","commit_id":"a658526cdbeade53b5bb05b83030970755272213"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"bd40a2340a522cf57c7214c27f8fa7d68ba2e450","unresolved":true,"context_lines":[{"line_number":652,"context_line":"        return share_group_snapshots"},{"line_number":653,"context_line":""},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"class FakeShareGroupSnapshot(object):"},{"line_number":656,"context_line":"    \"\"\"Fake a share group snapshot\"\"\""},{"line_number":657,"context_line":""},{"line_number":658,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"55ccabce_ac044227","line":655,"updated":"2021-08-10 08:50:53.000000000","message":"Well, actually it is here. All the functions needed to manipulate fake share group snapshots should be here.","commit_id":"e18496498e740ad61d8a0c032f88024749062013"},{"author":{"_account_id":33380,"name":"Archana Kumari","display_name":"Archana Kumari","email":"ak31960466@gmail.com","username":"archanaserver"},"change_message_id":"c6608762d1e569d16fe805d6515eefaa71e99eb5","unresolved":false,"context_lines":[{"line_number":652,"context_line":"        return share_group_snapshots"},{"line_number":653,"context_line":""},{"line_number":654,"context_line":""},{"line_number":655,"context_line":"class FakeShareGroupSnapshot(object):"},{"line_number":656,"context_line":"    \"\"\"Fake a share group snapshot\"\"\""},{"line_number":657,"context_line":""},{"line_number":658,"context_line":"    @staticmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"6b9b80eb_6b7e737e","line":655,"in_reply_to":"55ccabce_ac044227","updated":"2021-08-11 08:11:04.000000000","message":"Done","commit_id":"e18496498e740ad61d8a0c032f88024749062013"}],"manilaclient/tests/unit/osc/v2/test_share_group_snapshot.py":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"a9ea5d41_6acb3c02","updated":"2022-02-16 17:36:37.000000000","message":"nit: renaming it to test_share_group_snapshots","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"d20a1779_6c0176e7","in_reply_to":"a9ea5d41_6acb3c02","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":143,"context_line":"            name\u003dNone,"},{"line_number":144,"context_line":"            description\u003dNone,"},{"line_number":145,"context_line":"        )"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        self.group_snapshot_mocks.get.assert_called_with("},{"line_number":148,"context_line":"            self.share_group_snapshot.id)"},{"line_number":149,"context_line":"        self.assertCountEqual(self.columns, columns)"}],"source_content_type":"text/x-python","patch_set":17,"id":"0f0d542b_4b056fcd","line":146,"updated":"2022-02-16 17:36:37.000000000","message":"it would be good to mock osc_lib.utils.wait_for_status in here","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[{"line_number":143,"context_line":"            name\u003dNone,"},{"line_number":144,"context_line":"            description\u003dNone,"},{"line_number":145,"context_line":"        )"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        self.group_snapshot_mocks.get.assert_called_with("},{"line_number":148,"context_line":"            self.share_group_snapshot.id)"},{"line_number":149,"context_line":"        self.assertCountEqual(self.columns, columns)"}],"source_content_type":"text/x-python","patch_set":17,"id":"356a2f1f_674e43b3","line":146,"in_reply_to":"0f0d542b_4b056fcd","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        self.data \u003d tuple(self.share_group_snapshot._info.values())"},{"line_number":333,"context_line":"        self.columns \u003d tuple(self.share_group_snapshot._info.keys())"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"    def test_share_group_show(self):"},{"line_number":336,"context_line":"        arglist \u003d ["},{"line_number":337,"context_line":"            self.share_group_snapshot.id"}],"source_content_type":"text/x-python","patch_set":17,"id":"2d97863e_6a8edbf8","line":334,"updated":"2022-02-16 17:36:37.000000000","message":"not a real problem, but it would be good to have the parser exception","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        self.data \u003d tuple(self.share_group_snapshot._info.values())"},{"line_number":333,"context_line":"        self.columns \u003d tuple(self.share_group_snapshot._info.keys())"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"    def test_share_group_show(self):"},{"line_number":336,"context_line":"        arglist \u003d ["},{"line_number":337,"context_line":"            self.share_group_snapshot.id"}],"source_content_type":"text/x-python","patch_set":17,"id":"91ac42b7_12d12d3b","line":334,"in_reply_to":"2d97863e_6a8edbf8","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"e85b721aa4f77f32fc736791d531dc95ef4e77ca","unresolved":true,"context_lines":[{"line_number":407,"context_line":"            self.share_group_snapshot,"},{"line_number":408,"context_line":"            display_description\u003dparsed_args.description)"},{"line_number":409,"context_line":"        self.assertIsNone(result)"},{"line_number":410,"context_line":""},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"class TestUnsetShareGroupSnapshot(TestShareGroupSnapshot):"},{"line_number":413,"context_line":""}],"source_content_type":"text/x-python","patch_set":17,"id":"3c2e4d4a_93ebb1b2","line":410,"updated":"2022-02-16 17:36:37.000000000","message":"we should add a test case with the exception","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"f5279c220e87da6078c619cec8ca7a21d6ed7025","unresolved":false,"context_lines":[{"line_number":407,"context_line":"            self.share_group_snapshot,"},{"line_number":408,"context_line":"            display_description\u003dparsed_args.description)"},{"line_number":409,"context_line":"        self.assertIsNone(result)"},{"line_number":410,"context_line":""},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"class TestUnsetShareGroupSnapshot(TestShareGroupSnapshot):"},{"line_number":413,"context_line":""}],"source_content_type":"text/x-python","patch_set":17,"id":"d27ba77f_3763f52b","line":410,"in_reply_to":"3c2e4d4a_93ebb1b2","updated":"2022-04-09 10:09:30.000000000","message":"Done","commit_id":"d1eee4e9c75f19e7a2e534b7678afb1d21c4c924"}]}