)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"29e431bebf6268eb625b3b32e6f7e680b2e2d202","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c1fab555_9befe116","updated":"2024-07-11 21:26:54.000000000","message":"Hey, Christian! Thank you for working on this change.\nCould you please take a look at the unit tests that are failing and also add a release note?","commit_id":"a7c4f2f0ac6354e5961b31ed00326f803da0ccd3"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"7e65254ad2a04273d7864381a2685a41e61bd4c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a0542b6a_9d3314cd","updated":"2024-07-19 09:28:33.000000000","message":"Resolved","commit_id":"001f191121a906d3706f283549e8b37372be0642"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"9623b57bb5037ce4d5aaf49ec5ee5a43b21424eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a737f37c_e88f07a8","updated":"2024-07-21 12:05:19.000000000","message":"Resolved","commit_id":"9a7203b5af854c7162161d836ff36fd78b9b8cbe"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"cdd04e7ad49939c2acc41add90f08906deef23f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0f19d9e2_9824016b","updated":"2024-07-21 12:50:32.000000000","message":"https://opendev.org/openstack/python-manilaclient/src/branch/master/manilaclient/v2/share_snapshots.py#L91\n\nI have added the metadata textbox but I got error message below when trying to create a snapshot from share with share_type snapshot_support \u003d True extra spec.\n```\n Error: Unable to create share snapshot. Details\nShareSnapshotManager.create() got an unexpected keyword argument \u0027metadata\u0027 \n```\nHow do I specify the api version in manila-ui","commit_id":"9a7203b5af854c7162161d836ff36fd78b9b8cbe"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"3b3f7d8d98e756149552cc45fd108a35f5cd6836","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0bdaab1f_141d7bca","updated":"2024-07-24 13:18:37.000000000","message":"Resolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2886b39cf092b9ba0b63bf4a7822e835aaa44c47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8d1bf4e2_c7a6dd21","updated":"2024-07-24 20:46:04.000000000","message":"Thanks for this change Christian; were you planning to work on allowing editing snapshot metadata in a different change?\n\nPlease see some comments inline","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36889,"name":"tspyderboy","display_name":"tspyderboy","email":"tspyderboy@gmail.com","username":"tspyderboy"},"change_message_id":"0d38bb514c9b84e62d062156fd708f53e9fcead4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1f59fe5e_04434799","updated":"2024-07-31 14:37:19.000000000","message":"When I put metadata as: hello\n\nINFO django.server \"GET /project/share_snapshots/489d427f-2125-4fdf-9cdf-6ef4796a20dc/share_snapshot_create/ HTTP/1.1\" 200 3657\nERROR django.request Internal Server Error: /project/share_snapshots/489d427f-2125-4fdf-9cdf-6ef4796a20dc/share_snapshot_create/\nTraceback (most recent call last):\n  File \"/home/tspyderboy-vm/Desktop/outreachy/horizon/.tox/venv/lib/python3.10/site-packages/django/core/handlers/exception.py\", line 55, in inner\n.\n.\n.\nvm/Desktop/outreachy/horizon/.tox/venv/lib/python3.10/site-packages/django/urls/base.py\", line 88, in reverse\n    return resolver._reverse_with_prefix(view, prefix, *args, **kwargs)\n  File \"/home/tspyderboy-vm/Desktop/outreachy/horizon/.tox/venv/lib/python3.10/site-packages/django/urls/resolvers.py\", line 828, in _reverse_with_prefix\n    raise NoReverseMatch(msg)\ndjango.urls.exceptions.NoReverseMatch: Reverse for \u0027/project/share_snapshots/489d427f-2125-4fdf-9cdf-6ef4796a20dc/share_snapshot_create/\u0027 not found. \u0027/project/share_snapshots/489d427f-2125-4fdf-9cdf-6ef4796a20dc/share_snapshot_create/\u0027 is not a valid view function or pattern name.\nERROR django.server \"POST /project/share_snapshots/489d427f-2125-4fdf-9cdf-6ef4796a20dc/share_snapshot_create/ HTTP/1.1\" 500 594713","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"7165a0572a5ea6d09f250eab171db9e0ada491dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"337181d5_6ef813ca","updated":"2024-07-24 20:52:47.000000000","message":"You will need to add metadata to the snapshot details page, as well as add a functionality to edit snapshot metadata in a different change","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"764abd9a_7996d708","in_reply_to":"1f59fe5e_04434799","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"75a582283dfce304a956e9477586584574fb5bb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"001f7959_70c384c2","in_reply_to":"337181d5_6ef813ca","updated":"2024-08-05 19:00:52.000000000","message":"I believe this comment would still be relevant. The edit and the details page must contain the metadata related stuff.","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"381492a3_df524e87","in_reply_to":"8d1bf4e2_c7a6dd21","updated":"2024-08-04 16:49:28.000000000","message":"Yes, I will add it in the next patchset","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"4541c5b1987cddbb867e41e7cd76ba0d260fea21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"fd14b6e4_364b937a","updated":"2024-07-31 17:55:41.000000000","message":"When I tried creating a snapshot with metadata “foo” to get an error message I received “Danger: There was an error submitting the form. Please try again.” which is vague and does not help the user address their mistake. Let\u0027s enhance this message, by handling cases like incorrect metadata submissions. These cases are handled correctly on the api layer: https://github.com/openstack/manila/blob/master/manila/api/v2/metadata.py","commit_id":"ded99092376676bb4fcd1eae9fab3e1d3c8a1a46"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"562bc107a91b266b7d952a867622fc82a7d2d324","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a768e78c_a1f05dc3","in_reply_to":"c13a5ac9_2eebbed2","updated":"2024-08-09 19:26:19.000000000","message":"\u003e Resolved\n\nStill encountering this messge when trying to create metadata \"foo\" with no value.","commit_id":"ded99092376676bb4fcd1eae9fab3e1d3c8a1a46"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c13a5ac9_2eebbed2","in_reply_to":"fd14b6e4_364b937a","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"ded99092376676bb4fcd1eae9fab3e1d3c8a1a46"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"478db432_7365a989","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"10e4bd428167344397cba8dd4871fc9be8d76f67"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"aadebcade13624e4b63271945814bc74fbd68e20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"accece66_b7e208da","updated":"2024-08-05 18:58:16.000000000","message":"Thanks for implementing the suggested changes. Some jobs are still failing (i.e. unit tests one). Could you please take a look at the tests and fix them?","commit_id":"e9ad4becb4792caec057cb9d88069000b28098a8"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"562bc107a91b266b7d952a867622fc82a7d2d324","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"8583a2fa_3f038446","updated":"2024-08-09 19:26:19.000000000","message":"Missing edit existing metadata caplibity.","commit_id":"71d5cf036d3bce1e067409702e1fa8b743aa931b"}],"doc/requirements.txt":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"29e431bebf6268eb625b3b32e6f7e680b2e2d202","unresolved":true,"context_lines":[{"line_number":1,"context_line":"openstackdocstheme\u003e\u003d2.2.1 # Apache-2.0"},{"line_number":2,"context_line":"reno\u003e\u003d3.1.0 # Apache-2.0"},{"line_number":3,"context_line":"doc8\u003e\u003d0.6.0 # Apache-2.0"},{"line_number":4,"context_line":"sphinx\u003e\u003d2.1.1 # BSD"},{"line_number":5,"context_line":"ddt\u003e\u003d1.0.1 # MIT"},{"line_number":6,"context_line":"fixtures\u003e\u003d3.0.0 # Apache-2.0/BSD"},{"line_number":7,"context_line":"oslotest\u003e\u003d3.2.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"6a36e923_b9dd1ad6","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":19},"updated":"2024-07-11 21:26:54.000000000","message":"any specific reason behind the requirements bump?","commit_id":"a7c4f2f0ac6354e5961b31ed00326f803da0ccd3"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"7e65254ad2a04273d7864381a2685a41e61bd4c1","unresolved":false,"context_lines":[{"line_number":1,"context_line":"openstackdocstheme\u003e\u003d2.2.1 # Apache-2.0"},{"line_number":2,"context_line":"reno\u003e\u003d3.1.0 # Apache-2.0"},{"line_number":3,"context_line":"doc8\u003e\u003d0.6.0 # Apache-2.0"},{"line_number":4,"context_line":"sphinx\u003e\u003d2.1.1 # BSD"},{"line_number":5,"context_line":"ddt\u003e\u003d1.0.1 # MIT"},{"line_number":6,"context_line":"fixtures\u003e\u003d3.0.0 # Apache-2.0/BSD"},{"line_number":7,"context_line":"oslotest\u003e\u003d3.2.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"d0587ac9_4531ca99","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":19},"in_reply_to":"6a36e923_b9dd1ad6","updated":"2024-07-19 09:28:33.000000000","message":"It was mistakenly added I will remove it","commit_id":"a7c4f2f0ac6354e5961b31ed00326f803da0ccd3"}],"manila_ui/api/manila.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"43c7e5930336b04912ed5cfefc243ffa0cf67e36","unresolved":true,"context_lines":[{"line_number":28,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"MANILA_UI_USER_AGENT_REPR \u003d \"manila_ui_plugin_for_horizon\""},{"line_number":31,"context_line":"MANILA_VERSION \u003d \"2.52\""},{"line_number":32,"context_line":"MANILA_SERVICE_TYPE \u003d \"sharev2\""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# API static values"}],"source_content_type":"text/x-python","patch_set":6,"id":"78cc15d3_b4dba4da","line":31,"range":{"start_line":31,"start_character":0,"end_line":31,"end_character":23},"updated":"2024-07-23 12:21:02.000000000","message":"were you able to fully test the change and create metadata for snapshots in this version? I am asking because metadata for share snapshots was introduced in the version 2.73 [1], as I shared with you on IRC. If it worked, then we have a major bug in the manila API.\n\n[1] https://docs.openstack.org/api-ref/shared-file-system/#snapshot-metadata-since-api-v2-73","commit_id":"d234915e0e58b83f337dc6a6865d1bf2fb01261f"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"3b3f7d8d98e756149552cc45fd108a35f5cd6836","unresolved":false,"context_lines":[{"line_number":28,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"MANILA_UI_USER_AGENT_REPR \u003d \"manila_ui_plugin_for_horizon\""},{"line_number":31,"context_line":"MANILA_VERSION \u003d \"2.52\""},{"line_number":32,"context_line":"MANILA_SERVICE_TYPE \u003d \"sharev2\""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# API static values"}],"source_content_type":"text/x-python","patch_set":6,"id":"49a3c233_4c615c00","line":31,"range":{"start_line":31,"start_character":0,"end_line":31,"end_character":23},"in_reply_to":"78cc15d3_b4dba4da","updated":"2024-07-24 13:18:37.000000000","message":"Resolved","commit_id":"d234915e0e58b83f337dc6a6865d1bf2fb01261f"}],"manila_ui/dashboards/project/share_snapshots/forms.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"43c7e5930336b04912ed5cfefc243ffa0cf67e36","unresolved":true,"context_lines":[{"line_number":49,"context_line":"        try:"},{"line_number":50,"context_line":"            metadata \u003d {}"},{"line_number":51,"context_line":"            try:"},{"line_number":52,"context_line":"                set_dict, unset_list \u003d utils.parse_str_meta(data[\u0027metadata\u0027])"},{"line_number":53,"context_line":"                if unset_list:"},{"line_number":54,"context_line":"                    msg \u003d _(\"Expected only pairs of key\u003dvalue.\")"},{"line_number":55,"context_line":"                    raise ValidationError(message\u003dmsg)"},{"line_number":56,"context_line":"                metadata \u003d set_dict"},{"line_number":57,"context_line":"            except ValidationError as e:"},{"line_number":58,"context_line":"                self.api_error(e.messages[0])"}],"source_content_type":"text/x-python","patch_set":6,"id":"2c1d9241_9ee08b8c","line":55,"range":{"start_line":52,"start_character":15,"end_line":55,"end_character":54},"updated":"2024-07-23 12:21:02.000000000","message":"This is missing unit test coverage. Could you please add some more unit tests?\n\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_ef9/923597/6/check/openstack-tox-cover/ef9fc83/cover/z_a56d8698c5b66e43_forms_py.html#t53","commit_id":"d234915e0e58b83f337dc6a6865d1bf2fb01261f"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"3b3f7d8d98e756149552cc45fd108a35f5cd6836","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        try:"},{"line_number":50,"context_line":"            metadata \u003d {}"},{"line_number":51,"context_line":"            try:"},{"line_number":52,"context_line":"                set_dict, unset_list \u003d utils.parse_str_meta(data[\u0027metadata\u0027])"},{"line_number":53,"context_line":"                if unset_list:"},{"line_number":54,"context_line":"                    msg \u003d _(\"Expected only pairs of key\u003dvalue.\")"},{"line_number":55,"context_line":"                    raise ValidationError(message\u003dmsg)"},{"line_number":56,"context_line":"                metadata \u003d set_dict"},{"line_number":57,"context_line":"            except ValidationError as e:"},{"line_number":58,"context_line":"                self.api_error(e.messages[0])"}],"source_content_type":"text/x-python","patch_set":6,"id":"728553b5_354f0e31","line":55,"range":{"start_line":52,"start_character":15,"end_line":55,"end_character":54},"in_reply_to":"2c1d9241_9ee08b8c","updated":"2024-07-24 13:18:37.000000000","message":"Resolved","commit_id":"d234915e0e58b83f337dc6a6865d1bf2fb01261f"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2886b39cf092b9ba0b63bf4a7822e835aaa44c47","unresolved":true,"context_lines":[{"line_number":50,"context_line":"            metadata \u003d {}"},{"line_number":51,"context_line":"            try:"},{"line_number":52,"context_line":"                set_dict, unset_list \u003d utils.parse_str_meta(data[\u0027metadata\u0027])"},{"line_number":53,"context_line":"                if unset_list:"},{"line_number":54,"context_line":"                    msg \u003d _(\"Expected only pairs of key\u003dvalue.\")"},{"line_number":55,"context_line":"                    raise ValidationError(message\u003dmsg)"},{"line_number":56,"context_line":"                metadata \u003d set_dict"},{"line_number":57,"context_line":"            except ValidationError as e:"},{"line_number":58,"context_line":"                self.api_error(e.messages[0])"},{"line_number":59,"context_line":"                return False"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"            snapshot \u003d manila.share_snapshot_create("},{"line_number":62,"context_line":"                request,"}],"source_content_type":"text/x-python","patch_set":7,"id":"a047069b_88ebffd8","line":59,"range":{"start_line":53,"start_character":16,"end_line":59,"end_character":28},"updated":"2024-07-24 20:46:04.000000000","message":"Can you add some unit test coverage for this?\n\nhttps://dcf871499e4a3740c523-eb1364e9f57f07996b71e7658d0ea6b7.ssl.cf5.rackcdn.com/923597/7/check/openstack-tox-cover/ef739a8/cover/z_a56d8698c5b66e43_forms_py.html","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"7165a0572a5ea6d09f250eab171db9e0ada491dc","unresolved":true,"context_lines":[{"line_number":50,"context_line":"            metadata \u003d {}"},{"line_number":51,"context_line":"            try:"},{"line_number":52,"context_line":"                set_dict, unset_list \u003d utils.parse_str_meta(data[\u0027metadata\u0027])"},{"line_number":53,"context_line":"                if unset_list:"},{"line_number":54,"context_line":"                    msg \u003d _(\"Expected only pairs of key\u003dvalue.\")"},{"line_number":55,"context_line":"                    raise ValidationError(message\u003dmsg)"},{"line_number":56,"context_line":"                metadata \u003d set_dict"},{"line_number":57,"context_line":"            except ValidationError as e:"},{"line_number":58,"context_line":"                self.api_error(e.messages[0])"},{"line_number":59,"context_line":"                return False"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"            snapshot \u003d manila.share_snapshot_create("},{"line_number":62,"context_line":"                request,"}],"source_content_type":"text/x-python","patch_set":7,"id":"a5e5701d_7a0d29b9","line":59,"range":{"start_line":53,"start_character":16,"end_line":59,"end_character":28},"in_reply_to":"a047069b_88ebffd8","updated":"2024-07-24 20:52:47.000000000","message":"I tried using the following metadata and expected to see an error notification, instead, i got HTTP 500:\n\n```\nname: testsnap\ndescription: testing\nmetadata:\nh\u003dl\nk\n```\n\nThe error in horizon logs:\n\n```\n2024-07-24 20:48:32.246715   File \"/opt/stack/manila-ui/manila_ui/dashboards/project/share_snapshots/views.py\", line 115, in get_context_data\n2024-07-24 20:48:32.246717     context \u003d super(self.__class__, self).get_context_data(**kwargs)\n2024-07-24 20:48:32.246719   File \"/opt/stack/horizon/horizon/forms/views.py\", line 144, in get_context_data\n2024-07-24 20:48:32.246721     context \u003d super().get_context_data(**kwargs)\n2024-07-24 20:48:32.246722   File \"/opt/stack/horizon/horizon/forms/views.py\", line 77, in get_context_data\n2024-07-24 20:48:32.246724     context \u003d super().get_context_data(**kwargs)\n2024-07-24 20:48:32.246726   File \"/opt/stack/horizon/horizon/forms/views.py\", line 56, in get_context_data\n2024-07-24 20:48:32.246728     context \u003d super().get_context_data(**kwargs)\n2024-07-24 20:48:32.246729   File \"/opt/stack/data/venv/lib/python3.10/site-packages/django/views/generic/edit.py\", line 74, in get_context_data\n2024-07-24 20:48:32.246731     kwargs[\"form\"] \u003d self.get_form()\n2024-07-24 20:48:32.246733   File \"/opt/stack/horizon/horizon/forms/views.py\", line 179, in get_form\n2024-07-24 20:48:32.246735     return form_class(self.request, **self.get_form_kwargs())\n2024-07-24 20:48:32.246736   File \"/opt/stack/data/venv/lib/python3.10/site-packages/django/views/generic/edit.py\", line 44, in get_form_kwargs\n2024-07-24 20:48:32.246738     \"initial\": self.get_initial(),\n2024-07-24 20:48:32.246740   File \"/opt/stack/manila-ui/manila_ui/dashboards/project/share_snapshots/views.py\", line 124, in get_initial\n2024-07-24 20:48:32.246742     self.submit_url \u003d reverse(self.submit_url, kwargs\u003dself.kwargs)\n2024-07-24 20:48:32.246744   File \"/opt/stack/data/venv/lib/python3.10/site-packages/django/urls/base.py\", line 88, in reverse\n2024-07-24 20:48:32.246746     return resolver._reverse_with_prefix(view, prefix, *args, **kwargs)\n2024-07-24 20:48:32.246750   File \"/opt/stack/data/venv/lib/python3.10/site-packages/django/urls/resolvers.py\", line 828, in _reverse_with_prefix\n2024-07-24 20:48:32.246752     raise NoReverseMatch(msg)\n2024-07-24 20:48:32.246754 django.urls.exceptions.NoReverseMatch: Reverse for \u0027/dashboard/project/share_snapshots/aae9f9ec-5629-45f3-b8de-39a3d2daa78c/share_snapshot_create/\u0027 not found. \u0027/dashboard/project/share_snapshots/aae9f9ec-5629-45f3-b8de-39a3d2daa78c/share_snapshot_create/\u0027 is not a valid view function or pattern name.\n```","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            metadata \u003d {}"},{"line_number":51,"context_line":"            try:"},{"line_number":52,"context_line":"                set_dict, unset_list \u003d utils.parse_str_meta(data[\u0027metadata\u0027])"},{"line_number":53,"context_line":"                if unset_list:"},{"line_number":54,"context_line":"                    msg \u003d _(\"Expected only pairs of key\u003dvalue.\")"},{"line_number":55,"context_line":"                    raise ValidationError(message\u003dmsg)"},{"line_number":56,"context_line":"                metadata \u003d set_dict"},{"line_number":57,"context_line":"            except ValidationError as e:"},{"line_number":58,"context_line":"                self.api_error(e.messages[0])"},{"line_number":59,"context_line":"                return False"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"            snapshot \u003d manila.share_snapshot_create("},{"line_number":62,"context_line":"                request,"}],"source_content_type":"text/x-python","patch_set":7,"id":"6a4a7436_931f21fb","line":59,"range":{"start_line":53,"start_character":16,"end_line":59,"end_character":28},"in_reply_to":"a5e5701d_7a0d29b9","updated":"2024-08-04 16:49:28.000000000","message":"REsolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2886b39cf092b9ba0b63bf4a7822e835aaa44c47","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"            snapshot \u003d manila.share_snapshot_create("},{"line_number":62,"context_line":"                request,"},{"line_number":63,"context_line":"                share_id\u003ddata[\u0027share_id\u0027],"},{"line_number":64,"context_line":"                name\u003ddata[\u0027name\u0027],"},{"line_number":65,"context_line":"                description\u003ddata[\u0027description\u0027],"},{"line_number":66,"context_line":"                metadata\u003dmetadata,"}],"source_content_type":"text/x-python","patch_set":7,"id":"f05e3cc5_3d26afad","line":63,"range":{"start_line":63,"start_character":16,"end_line":63,"end_character":25},"updated":"2024-07-24 20:46:04.000000000","message":"this isn\u0027t a kwarg","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"            snapshot \u003d manila.share_snapshot_create("},{"line_number":62,"context_line":"                request,"},{"line_number":63,"context_line":"                share_id\u003ddata[\u0027share_id\u0027],"},{"line_number":64,"context_line":"                name\u003ddata[\u0027name\u0027],"},{"line_number":65,"context_line":"                description\u003ddata[\u0027description\u0027],"},{"line_number":66,"context_line":"                metadata\u003dmetadata,"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f9ddd75_a93e384b","line":63,"range":{"start_line":63,"start_character":16,"end_line":63,"end_character":25},"in_reply_to":"f05e3cc5_3d26afad","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"}],"manila_ui/tests/api/test_manila.py":[{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"9623b57bb5037ce4d5aaf49ec5ee5a43b21424eb","unresolved":false,"context_lines":[{"line_number":334,"context_line":"            self.request, self.id, name, description, force)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"        self.manilaclient.share_snapshots.create.assert_called_once_with("},{"line_number":337,"context_line":"            self.id, name\u003dname, description\u003ddescription, force\u003dforce, metadata\u003dTrue)"},{"line_number":338,"context_line":""},{"line_number":339,"context_line":"    def test_snapshot_delete(self):"},{"line_number":340,"context_line":"        snapshot_id \u003d \u0027fake_snapshot_id\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"bfdbc7d8_0e0ed887","line":337,"in_reply_to":"ed1453fd_cf34d027","updated":"2024-07-21 12:05:19.000000000","message":"\u003e pep8: E501 line too long (84 \u003e 79 characters)\n\nPlease fix.","commit_id":"bab5a7ea315cacb54259f8b18a4548ddc66cd6cc"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2886b39cf092b9ba0b63bf4a7822e835aaa44c47","unresolved":true,"context_lines":[{"line_number":329,"context_line":"    def test_snapshot_create(self, force):"},{"line_number":330,"context_line":"        name \u003d \u0027fake_snapshot_name\u0027"},{"line_number":331,"context_line":"        description \u003d \"fake_snapshot_description\""},{"line_number":332,"context_line":"        metadata \u003d None"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        api.share_snapshot_create("},{"line_number":335,"context_line":"            self.request, self.id, name, description, metadata, force)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3f26cbe0_7850510c","line":332,"range":{"start_line":332,"start_character":19,"end_line":332,"end_character":23},"updated":"2024-07-24 20:46:04.000000000","message":"please set this to a dict","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[{"line_number":329,"context_line":"    def test_snapshot_create(self, force):"},{"line_number":330,"context_line":"        name \u003d \u0027fake_snapshot_name\u0027"},{"line_number":331,"context_line":"        description \u003d \"fake_snapshot_description\""},{"line_number":332,"context_line":"        metadata \u003d None"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"        api.share_snapshot_create("},{"line_number":335,"context_line":"            self.request, self.id, name, description, metadata, force)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7dfc14d2_50511693","line":332,"range":{"start_line":332,"start_character":19,"end_line":332,"end_character":23},"in_reply_to":"3f26cbe0_7850510c","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"}],"manila_ui/tests/dashboards/project/share_snapshots/tests.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2886b39cf092b9ba0b63bf4a7822e835aaa44c47","unresolved":true,"context_lines":[{"line_number":57,"context_line":"            \u0027method\u0027: \u0027CreateForm\u0027,"},{"line_number":58,"context_line":"            \u0027size\u0027: 1,"},{"line_number":59,"context_line":"            \u0027type\u0027: \u0027NFS\u0027,"},{"line_number":60,"context_line":"            \u0027metadata\u0027: {},"},{"line_number":61,"context_line":"            \u0027share_id\u0027: share.id,"},{"line_number":62,"context_line":"        }"},{"line_number":63,"context_line":"        self.mock_object("}],"source_content_type":"text/x-python","patch_set":7,"id":"11892358_2eee67e5","line":60,"range":{"start_line":60,"start_character":24,"end_line":60,"end_character":26},"updated":"2024-07-24 20:46:04.000000000","message":"please set some fake metadata here","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[{"line_number":57,"context_line":"            \u0027method\u0027: \u0027CreateForm\u0027,"},{"line_number":58,"context_line":"            \u0027size\u0027: 1,"},{"line_number":59,"context_line":"            \u0027type\u0027: \u0027NFS\u0027,"},{"line_number":60,"context_line":"            \u0027metadata\u0027: {},"},{"line_number":61,"context_line":"            \u0027share_id\u0027: share.id,"},{"line_number":62,"context_line":"        }"},{"line_number":63,"context_line":"        self.mock_object("}],"source_content_type":"text/x-python","patch_set":7,"id":"e45e8693_4bc8635a","line":60,"range":{"start_line":60,"start_character":24,"end_line":60,"end_character":26},"in_reply_to":"11892358_2eee67e5","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"}],"releasenotes/notes/Add-metadata-textbox-to-snapshot-create-8b64456d767c71fa.yaml":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2886b39cf092b9ba0b63bf4a7822e835aaa44c47","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8d33e163_333a4d32","updated":"2024-07-24 20:46:04.000000000","message":"nit: please rename this file without capitalizing \"A\" at the beginning of the filename","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"},{"author":{"_account_id":36853,"name":"christian okeke","email":"eceokekechima@gmail.com","username":"ccokeke"},"change_message_id":"97fa28643554341e3869a0f8257993468f437ce6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cff6586f_77f0b07f","in_reply_to":"8d33e163_333a4d32","updated":"2024-08-04 16:49:28.000000000","message":"Resolved","commit_id":"3bf4ccc3e665a50b86d1a8930e3e83a472847753"}]}
