)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2a0479298a03e174db641b73f3b47808765fb2dd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"DellEMC SC: Added timeout to REST calls"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added a simple 30 second timeout to the rest calls to the Dell EMC"},{"line_number":10,"context_line":"StorageCenter API."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8e5857cc4a63c27ba240e0b4073aea42ef251595"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f93b717_e80aedcf","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":18},"updated":"2018-02-01 16:28:52.000000000","message":"should be just timeout. what default value you set to is internal to code.","commit_id":"b23786df82e4bb1dc3e0e84d8c304e90aec8f7ea"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"37731b9bea9b90e518cf5263cc0e508388af9d3d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"DellEMC SC: Added timeout to REST calls"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added a simple 30 second timeout to the rest calls to the Dell EMC"},{"line_number":10,"context_line":"StorageCenter API."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8e5857cc4a63c27ba240e0b4073aea42ef251595"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f93b717_a8a06543","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":18},"in_reply_to":"5f93b717_e80aedcf","updated":"2018-02-01 16:45:19.000000000","message":"ditto and can we also add a release note. And doc update?","commit_id":"b23786df82e4bb1dc3e0e84d8c304e90aec8f7ea"}],"cinder/tests/unit/volume/drivers/dell_emc/sc/test_scapi.py":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2a0479298a03e174db641b73f3b47808765fb2dd","unresolved":false,"context_lines":[{"line_number":8857,"context_line":"        expected_headers \u003d self.httpclient.header.copy()"},{"line_number":8858,"context_line":"        mock_get.assert_called_once_with(\u0027https://localhost:3033/api/rest/url\u0027,"},{"line_number":8859,"context_line":"                                         headers\u003dexpected_headers,"},{"line_number":8860,"context_line":"                                         timeout\u003d30,"},{"line_number":8861,"context_line":"                                         verify\u003dFalse)"},{"line_number":8862,"context_line":""},{"line_number":8863,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5f93b717_8826c967","line":8860,"range":{"start_line":8860,"start_character":1,"end_line":8860,"end_character":52},"updated":"2018-02-01 16:28:52.000000000","message":"should be configurable","commit_id":"b23786df82e4bb1dc3e0e84d8c304e90aec8f7ea"}],"cinder/volume/drivers/dell_emc/sc/storagecenter_api.py":[{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"fc8ee2bf45c77c4f9d6e9763203c8b3f61cc4cc7","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        self.header[\u0027Accept\u0027] \u003d \u0027application/json\u0027"},{"line_number":101,"context_line":"        self.header[\u0027x-dell-api-version\u0027] \u003d apiversion"},{"line_number":102,"context_line":"        self.verify \u003d verify"},{"line_number":103,"context_line":"        self.timeout \u003d 30"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"        # Verify is a configurable option.  So if this is false do not"},{"line_number":106,"context_line":"        # spam the c-vol log."}],"source_content_type":"text/x-python","patch_set":1,"id":"5f93b717_0ba65254","line":103,"updated":"2018-01-31 18:23:25.000000000","message":"Can we make this a config option please?","commit_id":"f10e1972e5c850898d485abd7db2a0ecbf3fe8dd"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"c91ade5c3c835b00ea2b6abf5b049a4af9708c51","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        self.header[\u0027Accept\u0027] \u003d \u0027application/json\u0027"},{"line_number":101,"context_line":"        self.header[\u0027x-dell-api-version\u0027] \u003d apiversion"},{"line_number":102,"context_line":"        self.verify \u003d verify"},{"line_number":103,"context_line":"        self.timeout \u003d 30"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"        # Verify is a configurable option.  So if this is false do not"},{"line_number":106,"context_line":"        # spam the c-vol log."}],"source_content_type":"text/x-python","patch_set":1,"id":"5f93b717_3c7cb61d","line":103,"in_reply_to":"5f93b717_0ba65254","updated":"2018-01-31 20:02:02.000000000","message":"Let me know if this fixes things for the customer first.","commit_id":"f10e1972e5c850898d485abd7db2a0ecbf3fe8dd"}],"cinder/volume/drivers/dell_emc/sc/storagecenter_common.py":[{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"37731b9bea9b90e518cf5263cc0e508388af9d3d","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                    \u0027Storage Center.\u0027),"},{"line_number":69,"context_line":"    cfg.IntOpt(\u0027dell_api_timeout\u0027,"},{"line_number":70,"context_line":"               default\u003d30,"},{"line_number":71,"context_line":"               help\u003d\u0027Timeout for Dell EMC Storage Center REST API calls.\u0027)"},{"line_number":72,"context_line":"]"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f93b717_08fcf93b","line":71,"updated":"2018-02-01 16:45:19.000000000","message":"Can we call out the time units ( seconds)","commit_id":"b23786df82e4bb1dc3e0e84d8c304e90aec8f7ea"}]}