)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"0dd26a4b373b0a0555dd4dcbe425c4506e34985f","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2022-06-14 21:32:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add fail_reason to backup detail view."},{"line_number":8,"context_line":" see also https://bugs.launchpad.net/cinder/+bug/1978729"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I4e639211043270e814fac489f915588af03f966a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"40aa54ed_ad0488a2","line":8,"range":{"start_line":8,"start_character":1,"end_line":8,"end_character":9},"updated":"2022-08-22 04:42:23.000000000","message":"You can use Related bug tag here.","commit_id":"7895be8a7b2178ae79d322679523540ca9df7073"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"b3bcb21fed61d9a32b6234cf461ef293a802c07b","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add messages tab to backup detail view"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"  and add fail_reason to backup detail view"},{"line_number":10,"context_line":"  since messages does not seem always to contain"},{"line_number":11,"context_line":"  the fail_reason."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"  Use DetailTabs to improve top padding"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  Related-Bug https://bugs.launchpad.net/cinder/+bug/1978729"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I4e639211043270e814fac489f915588af03f966a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9b41aabb_9d687a2a","line":15,"range":{"start_line":9,"start_character":0,"end_line":15,"end_character":60},"updated":"2022-08-31 10:04:02.000000000","message":"Could you improve the commit-msg like below:\n\nAdd cinder-user-facing messages for Backup\n\nThis patch adds a tab for cinder user messages for volume backups. Cinder user messages show error details for cinder resources like if we are unable to create a volume backup due to some failure in cinder it will show us the reason for failure.\nIt also updates project and admin SnapshotDetailsTabs to use DetailTabsGroup instead of TabGroup to improve top padding.\nRelated-Bug https://bugs.launchpad.net/cinder/+bug/1978729","commit_id":"377dc29a94a5472b83b9b5fdfc420eaed5500df5"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"0dd26a4b373b0a0555dd4dcbe425c4506e34985f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"71b72ffa_77342b33","updated":"2022-08-22 04:42:23.000000000","message":"Thanks for the patch.\nOne question: There is already a Messages tab on the horizon for volumes and backup\nresources. So I am just wondering do we need this patch because if you want to see failure messages for any volume/backup, you can see the same in the failures messages tab for that volumes/backup. For more info. please refer [1].\n\n[1] https://review.opendev.org/c/openstack/horizon/+/734161","commit_id":"7895be8a7b2178ae79d322679523540ca9df7073"},{"author":{"_account_id":29260,"name":"Jesper Schmitz Mouridsen","email":"jesper@schmitz.computer","username":"jsmdk"},"change_message_id":"1ed2a23f6d0d540b9152ee34aa36be5254e96ba5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95b5c0f7_d3cd7d88","updated":"2022-06-15 17:58:50.000000000","message":"recheck","commit_id":"7895be8a7b2178ae79d322679523540ca9df7073"},{"author":{"_account_id":29260,"name":"Jesper Schmitz Mouridsen","email":"jesper@schmitz.computer","username":"jsmdk"},"change_message_id":"6fe64f74a7741aaa65c4e6fbcef4aec381efec3e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c941e028_e5b3ccad","in_reply_to":"71b72ffa_77342b33","updated":"2022-08-30 12:20:11.000000000","message":"I do not see any backup errros in that tab? An the backup list does not have a messages tab on recent devstack. I have added one in patchset 2","commit_id":"7895be8a7b2178ae79d322679523540ca9df7073"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"b3bcb21fed61d9a32b6234cf461ef293a802c07b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a3fbd2ee_32f4a2eb","in_reply_to":"c941e028_e5b3ccad","updated":"2022-08-31 10:04:02.000000000","message":"sorry, my bad.","commit_id":"7895be8a7b2178ae79d322679523540ca9df7073"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"d1b30352c3d17a45f6d1e121e171fe00325d44be","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"20767f03_0213b5ad","updated":"2022-08-30 12:11:53.000000000","message":"-1, Due to no response on my Previous comment.","commit_id":"377dc29a94a5472b83b9b5fdfc420eaed5500df5"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"b3bcb21fed61d9a32b6234cf461ef293a802c07b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6c55fcf1_8302039e","updated":"2022-08-31 10:04:02.000000000","message":"Thanks for the patch.\nCould you please add a release note for this patch?\n\nUnit test in missing for the change, you can take reference from https://review.opendev.org/c/openstack/horizon/+/750882/","commit_id":"377dc29a94a5472b83b9b5fdfc420eaed5500df5"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"ea9dfd25bac56198eaa339f695742a89d4393654","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"40b34af1_4e53e8bc","in_reply_to":"6c55fcf1_8302039e","updated":"2022-09-06 04:54:04.000000000","message":"any update on this patch, we are approaching feature-freeze, so it is good if we can merge this asap.","commit_id":"377dc29a94a5472b83b9b5fdfc420eaed5500df5"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"7f43281b26bcc1a1087d2649f4628a6a4aecd433","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a6fa68aa_810266a3","updated":"2022-09-06 13:57:33.000000000","message":"Hi Core team, Can we have 2nd review on this patch. I am planning to add this feature in cycle highlights patch.","commit_id":"20a571fdd299f18c497bbb22ab0af7ead1bc4690"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"8aa4d8a38831033c60eae021ab64b1130f0515ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3b99fbf4_5b712120","updated":"2022-09-06 08:10:52.000000000","message":"Thanks for the Update, LGTM.","commit_id":"20a571fdd299f18c497bbb22ab0af7ead1bc4690"}]}
