)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"4b9faa82e40bf04442faed02ce85d5b3821fc57f","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This change adds the ability to configure the connection_pool_size in"},{"line_number":14,"context_line":"Nova.  The default is 10 and the minimum value is also 10 as it is not"},{"line_number":15,"context_line":"recommended to run in that state."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug: #1494688"},{"line_number":18,"context_line":"Co-Authored-By: rgerganov@vmware.com"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ff346bd7_e505a3c3","line":15,"updated":"2017-07-26 12:39:45.000000000","message":"this last sentence doesn\u0027t quite align with what\u0027s said in the help message for the config setting. This makes it sound like the default is too low, whereas the help message says to only raise things if the default is not good enough.","commit_id":"fc0a9e559ef362c97f2b67043252b43a36f61b23"}],"nova/virt/vmwareapi/driver.py":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"4a39b6b9d56c8e1123a6098225fe61a9641a8ecc","unresolved":false,"context_lines":[{"line_number":513,"context_line":"                 scheme\u003d\"https\","},{"line_number":514,"context_line":"                 cacert\u003dCONF.vmware.ca_file,"},{"line_number":515,"context_line":"                 insecure\u003dCONF.vmware.insecure,"},{"line_number":516,"context_line":"                 pool_size\u003dCONF.vmware.connection_pool_size):"},{"line_number":517,"context_line":"        super(VMwareAPISession, self).__init__("},{"line_number":518,"context_line":"                host\u003dhost_ip,"},{"line_number":519,"context_line":"                port\u003dhost_port,"}],"source_content_type":"text/x-python","patch_set":6,"id":"7f515b1d_93d7dc4b","line":516,"updated":"2017-10-02 18:50:31.000000000","message":"Since it\u0027s already being done, it\u0027s probably fine.\n\nBut using CONF in parameter defaults is weird. Sure, CONF is a big fat ugly global, but treating it as such means that fixing that horribleness becomes harder later.\n\n(Feel free to ignore this comment, I\u0027m mostly just shaking my tiny fist at the universe.)","commit_id":"d5c29e1be5ff12045c6dd324ca73509a5213adc3"}]}