)]}'
{"shade/openstackcloud.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"b498117149889f39909bc7339b2d0fb7abb3d1ed","unresolved":false,"context_lines":[{"line_number":5702,"context_line":"                \"Unavailable feature: security groups\""},{"line_number":5703,"context_line":"            )"},{"line_number":5704,"context_line":""},{"line_number":5705,"context_line":"        group \u003d None"},{"line_number":5706,"context_line":"        if self._use_neutron_secgroups():"},{"line_number":5707,"context_line":"            with _utils.neutron_exceptions("},{"line_number":5708,"context_line":"                    \"Error creating security group {0}\".format(name)):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_0d8e3478","line":5705,"updated":"2016-10-05 11:20:48.000000000","message":"Shouldn\u0027t this be \"group \u003d {}\" ? I don\"t see how the _normalize_secgroup method can cope with a None value.","commit_id":"3aa63eb9145811383aa2bbec5555949b4c98e5fb"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"535249053bdf44716e3988316afff86fec68bf99","unresolved":false,"context_lines":[{"line_number":5702,"context_line":"                \"Unavailable feature: security groups\""},{"line_number":5703,"context_line":"            )"},{"line_number":5704,"context_line":""},{"line_number":5705,"context_line":"        group \u003d None"},{"line_number":5706,"context_line":"        if self._use_neutron_secgroups():"},{"line_number":5707,"context_line":"            with _utils.neutron_exceptions("},{"line_number":5708,"context_line":"                    \"Error creating security group {0}\".format(name)):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a95cdbc_f66bf9b4","line":5705,"in_reply_to":"1a95cdbc_0d8e3478","updated":"2016-10-09 11:05:50.000000000","message":"I can make that change for ease of reading. It\u0027s really just to make the variable exist at this scope level so that the return outside of the if/else can see it. There should never be an exit point below the if/else that hasn\u0027t replaced this. However ... might as well - thanks!","commit_id":"3aa63eb9145811383aa2bbec5555949b4c98e5fb"}]}