)]}'
{"octavia/tests/unit/amphorae/drivers/haproxy/test_ssh_driver.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"62cb9372109919f7bbb7377f856ad22377dab493","unresolved":false,"context_lines":[{"line_number":240,"context_line":"            self.assertEqual(tls.intermediates,"},{"line_number":241,"context_line":"                             self.driver._map_cert_tls_container("},{"line_number":242,"context_line":"                                 cert).intermediates)"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @mock.patch.object(ssh_driver.HaproxyManager, \u0027_execute_command\u0027)"},{"line_number":245,"context_line":"    def test_post_vip_plug_no_down_links(self, exec_command):"},{"line_number":246,"context_line":"        amps \u003d [data_models.Amphora(id\u003dMOCK_AMP_ID, compute_id\u003dMOCK_COMPUTE_ID,"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a4dcd0f_20dd3fe0","line":243,"updated":"2015-08-05 05:47:53.000000000","message":"Wait, is this a gerrit glitch, or did test_build_pem get deleted as part of this patchset? Seems unrelated...","commit_id":"ab8c56b41964a31652d3e146dceef626e4f51c2e"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"62cb9372109919f7bbb7377f856ad22377dab493","unresolved":false,"context_lines":[{"line_number":332,"context_line":""},{"line_number":333,"context_line":"    def test_is_root(self):"},{"line_number":334,"context_line":"        cfg.CONF.set_override(\u0027username\u0027, \u0027root\u0027, group\u003d\u0027haproxy_amphora\u0027)"},{"line_number":335,"context_line":"        self.assertEqual(True, self.driver._is_root())"},{"line_number":336,"context_line":"        cfg.CONF.set_override(\u0027username\u0027, \u0027blah\u0027, group\u003d\u0027haproxy_amphora\u0027)"},{"line_number":337,"context_line":"        self.assertEqual(False, self.driver._is_root())"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a4dcd0f_a0ef0f86","line":335,"updated":"2015-08-05 05:47:53.000000000","message":"Isn\u0027t there an assertTrue/assertFalse?","commit_id":"ab8c56b41964a31652d3e146dceef626e4f51c2e"},{"author":{"_account_id":6951,"name":"Brandon Logan","email":"brandon.logan@rackspace.com","username":"brandon-logan"},"change_message_id":"82347d6fcdc020b09d6517c7c7b99c384129788d","unresolved":false,"context_lines":[{"line_number":332,"context_line":""},{"line_number":333,"context_line":"    def test_is_root(self):"},{"line_number":334,"context_line":"        cfg.CONF.set_override(\u0027username\u0027, \u0027root\u0027, group\u003d\u0027haproxy_amphora\u0027)"},{"line_number":335,"context_line":"        self.assertEqual(True, self.driver._is_root())"},{"line_number":336,"context_line":"        cfg.CONF.set_override(\u0027username\u0027, \u0027blah\u0027, group\u003d\u0027haproxy_amphora\u0027)"},{"line_number":337,"context_line":"        self.assertEqual(False, self.driver._is_root())"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a4dcd0f_ca8cb237","line":335,"in_reply_to":"1a4dcd0f_a0ef0f86","updated":"2015-08-05 08:19:24.000000000","message":"there is and I use it, but the default fail message is terrible \"False is not True\" or \"True is not False\", this at least gives a better message.  I\u0027d still use assertTrue/False as well though.  i\u0027m just ranting","commit_id":"ab8c56b41964a31652d3e146dceef626e4f51c2e"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"e04cf59276888394fdd265d7624ec45c7327bdbf","unresolved":false,"context_lines":[{"line_number":208,"context_line":"                        mock.call(mock.ANY, mock.ANY),"},{"line_number":209,"context_line":"                    ])"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def test_build_pem(self):"},{"line_number":212,"context_line":"        expected \u003d \u0027imainter\\nimainter2\\nimacert\\nimakey\u0027"},{"line_number":213,"context_line":"        tls_tupe \u003d sample_configs.sample_tls_container_tuple("},{"line_number":214,"context_line":"            certificate\u003d\u0027imacert\u0027, private_key\u003d\u0027imakey\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a4dcd0f_fee910e7","line":211,"updated":"2015-08-05 17:00:46.000000000","message":"Now it moved, so it might cause a merge conflict for someone else later, but whatever. Not going to be a show-stopper. :P","commit_id":"cdffd85709b0a4b585f2d357db4f87c66ce4f010"}]}