)]}'
{"cue/manage/broker.py":[{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"c558d0b27009521ae6f1e5b91bef36e732ea7bb3","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        }"},{"line_number":53,"context_line":"        broker \u003d objects.Broker(**broker_values)"},{"line_number":54,"context_line":"        new_broker \u003d broker.create_broker(self.context)"},{"line_number":55,"context_line":"        print(new_broker.id)"},{"line_number":56,"context_line":"        return new_broker.id"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def list(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afb71cf_75a52ba0","line":55,"updated":"2015-06-09 21:22:07.000000000","message":"don\u0027t need print statement here","commit_id":"fb41b3a08deb4b0ac0f4f3990c0472dc1b537a58"},{"author":{"_account_id":14955,"name":"Abitha Palaniappan","email":"abitha.palaniappan@hp.com","username":"Abitha"},"change_message_id":"c3ae748a3351d96a6edfce33c8197c1589d1ab30","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        }"},{"line_number":53,"context_line":"        broker \u003d objects.Broker(**broker_values)"},{"line_number":54,"context_line":"        new_broker \u003d broker.create_broker(self.context)"},{"line_number":55,"context_line":"        print(new_broker.id)"},{"line_number":56,"context_line":"        return new_broker.id"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def list(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afb71cf_d8cd3632","line":55,"in_reply_to":"3afb71cf_75a52ba0","updated":"2015-06-09 21:56:14.000000000","message":"thats added for the ansible role to grab the id. I will modify it in future as a separate patch, to print a table with new broker values.","commit_id":"fb41b3a08deb4b0ac0f4f3990c0472dc1b537a58"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"6a09c5666f535f0a0300c08cc2a64bf8bcc011c6","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        }"},{"line_number":53,"context_line":"        broker \u003d objects.Broker(**broker_values)"},{"line_number":54,"context_line":"        new_broker \u003d broker.create_broker(self.context)"},{"line_number":55,"context_line":"        print(new_broker.id)"},{"line_number":56,"context_line":"        return new_broker.id"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def list(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afb71cf_023e94c0","line":55,"in_reply_to":"3afb71cf_d8cd3632","updated":"2015-06-10 16:16:31.000000000","message":"Could you also modify the unit test in https://github.com/stackforge/cue/blob/master/cue/tests/unit/manage/broker/test_broker.py#L70 to check the returned object?","commit_id":"fb41b3a08deb4b0ac0f4f3990c0472dc1b537a58"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"cab45964a5411f4222e76f9d76d33bd35392910e","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        }"},{"line_number":53,"context_line":"        broker \u003d objects.Broker(**broker_values)"},{"line_number":54,"context_line":"        new_broker \u003d broker.create_broker(self.context)"},{"line_number":55,"context_line":"        print(new_broker.id)"},{"line_number":56,"context_line":"        return new_broker.id"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def list(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afb71cf_427b6c7b","line":55,"in_reply_to":"3afb71cf_d8cd3632","updated":"2015-06-10 16:18:33.000000000","message":"ok, create a bug for that so we can track this outstanding work","commit_id":"fb41b3a08deb4b0ac0f4f3990c0472dc1b537a58"}]}