)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"30d1becc84bb597916b18ad509d968412b1e2f18","unresolved":false,"context_lines":[{"line_number":10,"context_line":"session settings like cafile and insecure, to make use of non-system TLS"},{"line_number":11,"context_line":"certificates (or disable certificate checks at all). Also client"},{"line_number":12,"context_line":"certificates and timeout values can be specified, too."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I510a2683958fc8c3aaca9293b4280f325b9551fc"},{"line_number":15,"context_line":"Signed-off-by: Gyorgy Szombathelyi \u003cgyurco@freemail.hu\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da36d5c6_fe1b1b19","line":13,"updated":"2017-02-22 12:57:33.000000000","message":"Please open a bug report for that problem so we could backport that change.","commit_id":"92f8ad7d4cc7fcf31b07df7d46cf8baa797a46ec"}],"nova/scheduler/client/report.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"30d1becc84bb597916b18ad509d968412b1e2f18","unresolved":false,"context_lines":[{"line_number":195,"context_line":"        auth_plugin \u003d keystone.load_auth_from_conf_options("},{"line_number":196,"context_line":"            CONF, \u0027placement\u0027)"},{"line_number":197,"context_line":"        self._client \u003d keystone.load_session_from_conf_options("},{"line_number":198,"context_line":"            CONF, \u0027placement\u0027, auth\u003dauth_plugin)"},{"line_number":199,"context_line":"        # NOTE(danms): Keep track of how naggy we\u0027ve been"},{"line_number":200,"context_line":"        self._warn_count \u003d 0"},{"line_number":201,"context_line":"        self.ks_filter \u003d {\u0027service_type\u0027: \u0027placement\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_3ea213af","line":198,"updated":"2017-02-22 12:57:33.000000000","message":"Looks good to me https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/session.py#L229-L230","commit_id":"92f8ad7d4cc7fcf31b07df7d46cf8baa797a46ec"}],"nova/tests/unit/scheduler/client/test_report.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"30d1becc84bb597916b18ad509d968412b1e2f18","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"class TestConstructor(test.NoDBTestCase):"},{"line_number":135,"context_line":"    @mock.patch(\u0027keystoneauth1.session.Session\u0027)"},{"line_number":136,"context_line":"    @mock.patch(\u0027keystoneauth1.loading.load_auth_from_conf_options\u0027)"},{"line_number":137,"context_line":"    def test_constructor(self, load_auth_mock, ks_sess_mock):"},{"line_number":138,"context_line":"        client \u003d report.SchedulerReportClient()"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_fea07ba1","line":135,"updated":"2017-02-22 12:57:33.000000000","message":"I\u0027d rather have preferred you mock the module helper directly and not the internal keystone object.","commit_id":"92f8ad7d4cc7fcf31b07df7d46cf8baa797a46ec"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"9f3879561fbd4392de7d22c1eeb0f5e35d9f4f39","unresolved":false,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"        load_auth_mock.assert_called_once_with(CONF, \u0027placement\u0027)"},{"line_number":141,"context_line":"        load_sess_mock.assert_called_once_with(CONF, \u0027placement\u0027,"},{"line_number":142,"context_line":"                                              auth\u003dload_auth_mock.return_value)"},{"line_number":143,"context_line":"        self.assertIsNone(client.ks_filter[\u0027interface\u0027])"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"    @mock.patch(\u0027keystoneauth1.loading.load_session_from_conf_options\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_585b5287","line":142,"range":{"start_line":142,"start_character":45,"end_line":142,"end_character":46},"updated":"2017-02-22 16:58:20.000000000","message":"nit: wrong alignment but okay.","commit_id":"a377fc5988e9a6057bd14617879a7cbcc3c8bb81"},{"author":{"_account_id":18940,"name":"Gyorgy Szombathelyi","email":"gyurco@freemail.hu","username":"gyurco78"},"change_message_id":"b590d75dd1330da69efc3238c1e3f7f6ee25ea56","unresolved":false,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"        load_auth_mock.assert_called_once_with(CONF, \u0027placement\u0027)"},{"line_number":141,"context_line":"        load_sess_mock.assert_called_once_with(CONF, \u0027placement\u0027,"},{"line_number":142,"context_line":"                                              auth\u003dload_auth_mock.return_value)"},{"line_number":143,"context_line":"        self.assertIsNone(client.ks_filter[\u0027interface\u0027])"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"    @mock.patch(\u0027keystoneauth1.loading.load_session_from_conf_options\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_78481688","line":142,"range":{"start_line":142,"start_character":45,"end_line":142,"end_character":46},"in_reply_to":"da36d5c6_585b5287","updated":"2017-02-22 17:02:25.000000000","message":"The good alignment will make the line one character longer than allowed :)\nPep8 accepted this one.","commit_id":"a377fc5988e9a6057bd14617879a7cbcc3c8bb81"}]}