)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8398e689ef4cd69c123a2b1ba058c5d346757794","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Nilesh Thathagar \u003cnilesh.thathagar@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-06-12 10:11:55 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dell PowerFlex Driver: Added default port value"},{"line_number":8,"context_line":"for the dell PowerFlex NAS backend"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Closes-Bug: #2066840"},{"line_number":11,"context_line":"Change-Id: Ifda8c0b179763fe0884ef3845b6836be861ca6a8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"cf98cac3_5fa1e3a0","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":34},"updated":"2024-06-12 23:50:28.000000000","message":"nit:\n\nPlease keep your commit message header to a single line under 50 characters; for example:\n\n\"Dell PowerFlex: Set default value for port config\"\n\nwould suffice","commit_id":"14d7adf5b2b967fec9b15da0fa8aa3096c954014"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"2d3cb9b41af2da2f5ce4d96e418da0b3b5181f47","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nilesh Thathagar \u003cnilesh.thathagar@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-06-12 10:11:55 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dell PowerFlex Driver: Added default port value"},{"line_number":8,"context_line":"for the dell PowerFlex NAS backend"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Closes-Bug: #2066840"},{"line_number":11,"context_line":"Change-Id: Ifda8c0b179763fe0884ef3845b6836be861ca6a8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a66b2135_e02fdd8b","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":34},"in_reply_to":"cf98cac3_5fa1e3a0","updated":"2024-06-13 03:21:54.000000000","message":"Done","commit_id":"14d7adf5b2b967fec9b15da0fa8aa3096c954014"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"527e957eb10a63b5696dbe37d0e432c3c1a69a81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e702ad3a_fac06bfc","updated":"2024-06-12 12:30:58.000000000","message":"Please do a code review","commit_id":"14d7adf5b2b967fec9b15da0fa8aa3096c954014"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c690654beb28a1cafab7d459ecddb63e9f9c703a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"87e01987_4872dbb1","updated":"2024-06-12 20:11:56.000000000","message":"Thanks; LGTM. please see the comment inline with a suggestion","commit_id":"14d7adf5b2b967fec9b15da0fa8aa3096c954014"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"6cc69a9d5d2e2e5d9ed13375450274b542030763","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"642a6b71_2a273341","updated":"2024-06-17 14:00:14.000000000","message":"Just a reminder: this fix can be backported to older branches as well","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0ec8e3990e24f932d04ad4f2af6386331b26c24a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7df5f597_f866ed90","updated":"2024-06-14 18:00:47.000000000","message":"LGTM - the Dell PowerFlex CI system is passing; thank you","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"502ba726c58e4f58a25308eb70216afe5bc24949","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9c9f6cde_0a965520","updated":"2024-06-17 13:59:44.000000000","message":"LGTM, thank you for the fix, Nilesh!","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"07794b641f0d661280121d795f943e1660b37097","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c070e921_8dbe5249","updated":"2024-06-17 17:25:57.000000000","message":"Nilesh, please hold rechecking this change for now, I believe something is broken in our gate after https://review.opendev.org/c/openstack/neutron/+/916406. We need neutron to start but it is failing: https://zuul.opendev.org/t/openstack/build/0bacd173408c48bb9572fc1cb2d63e3e/log/controller/logs/screen-q-svc.txt","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"e603b960e41c6686963882b172c4e06a9d203730","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7a5223df_7b58ca21","updated":"2024-06-17 17:29:55.000000000","message":"Thanks Carlos. Yes neutron service is not getting start.","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"34d53950f327dd75e2865eb19bb0282a88f18528","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f7d7be9f_b2ac92de","updated":"2024-06-17 14:06:31.000000000","message":"Thanks, will do a backporting for old branches.","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"d4941443b6920b414977a71721146f574cf1b683","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4960f627_5904baba","updated":"2024-06-17 15:32:30.000000000","message":"merge-check","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"d85b2c913fdbba507e5e7dca5603ac5ae2d1ddf7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b5967dd2_2741abc4","updated":"2024-06-17 15:01:44.000000000","message":"recheck","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b25e2be6396b9beccb9ea3ac82d8a773f277111e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"25eeb08b_f41edef0","updated":"2024-06-17 22:48:07.000000000","message":"recheck\n\nhttps://review.opendev.org/c/openstack/manila-tempest-plugin/+/922143 has merged","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"c205961d2d41c17b4d378365adedf69de3939cb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5b8bafde_2188d6c0","in_reply_to":"c070e921_8dbe5249","updated":"2024-06-17 17:26:41.000000000","message":"As soon as the gate is fixed, I can get back and recheck the change","commit_id":"d7f7b6312a3529f6f23cc541114594308f85e1af"}],"manila/share/drivers/dell_emc/plugins/powerflex/connection.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c690654beb28a1cafab7d459ecddb63e9f9c703a","unresolved":true,"context_lines":[{"line_number":44,"context_line":"               help\u003d\u0027Protection domain to use.\u0027),"},{"line_number":45,"context_line":"    cfg.StrOpt(\u0027dell_nas_backend_host\u0027,"},{"line_number":46,"context_line":"               help\u003d\u0027Dell NAS backend hostname or IP address.\u0027),"},{"line_number":47,"context_line":"    cfg.StrOpt(\u0027dell_nas_backend_port\u0027,"},{"line_number":48,"context_line":"               default\u003d443,"},{"line_number":49,"context_line":"               help\u003d\u0027Port number to use with the Dell NAS backend.\u0027),"},{"line_number":50,"context_line":"    cfg.StrOpt(\u0027dell_nas_server\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"038bc54e_97ec2b85","line":47,"range":{"start_line":47,"start_character":8,"end_line":47,"end_character":14},"updated":"2024-06-12 20:11:56.000000000","message":"why not change this to an IntOpt instead of type-casting this on line 104?","commit_id":"14d7adf5b2b967fec9b15da0fa8aa3096c954014"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"2d3cb9b41af2da2f5ce4d96e418da0b3b5181f47","unresolved":false,"context_lines":[{"line_number":44,"context_line":"               help\u003d\u0027Protection domain to use.\u0027),"},{"line_number":45,"context_line":"    cfg.StrOpt(\u0027dell_nas_backend_host\u0027,"},{"line_number":46,"context_line":"               help\u003d\u0027Dell NAS backend hostname or IP address.\u0027),"},{"line_number":47,"context_line":"    cfg.StrOpt(\u0027dell_nas_backend_port\u0027,"},{"line_number":48,"context_line":"               default\u003d443,"},{"line_number":49,"context_line":"               help\u003d\u0027Port number to use with the Dell NAS backend.\u0027),"},{"line_number":50,"context_line":"    cfg.StrOpt(\u0027dell_nas_server\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"d8e47f9a_79131a91","line":47,"range":{"start_line":47,"start_character":8,"end_line":47,"end_character":14},"in_reply_to":"038bc54e_97ec2b85","updated":"2024-06-13 03:21:54.000000000","message":"Yes,we can do that. Thanks.","commit_id":"14d7adf5b2b967fec9b15da0fa8aa3096c954014"}]}