)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28415,"name":"Lin Shuicheng","email":"shuicheng.lin@intel.com","username":"slin14"},"change_message_id":"df5a6d9477251a4bec7261d35d805705a16e7eda","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add test class for kube_app AppOperator"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add test class and setup method."},{"line_number":10,"context_line":"Mock method/operator will be added in indiviual commits which uses"},{"line_number":11,"context_line":"that method or operator."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Story: 2007082"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_705a0e83","line":10,"range":{"start_line":10,"start_character":48,"end_line":10,"end_character":55},"updated":"2020-01-14 05:08:56.000000000","message":"commit","commit_id":"fd0bd02f6a5bb594586a1bc77483a6bea7030e03"},{"author":{"_account_id":28415,"name":"Lin Shuicheng","email":"shuicheng.lin@intel.com","username":"slin14"},"change_message_id":"df5a6d9477251a4bec7261d35d805705a16e7eda","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add test class for kube_app AppOperator"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add test class and setup method."},{"line_number":10,"context_line":"Mock method/operator will be added in indiviual commits which uses"},{"line_number":11,"context_line":"that method or operator."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Story: 2007082"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_50559251","line":10,"range":{"start_line":10,"start_character":38,"end_line":10,"end_character":47},"updated":"2020-01-14 05:08:56.000000000","message":"individual","commit_id":"fd0bd02f6a5bb594586a1bc77483a6bea7030e03"}],"sysinv/sysinv/sysinv/sysinv/tests/conductor/test_kube_app_app_operator.py":[{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"e476dc53828327478214688161b7a7bf36f7f63e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4"},{"line_number":2,"context_line":"# coding\u003dutf-8"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# SPDX-License-Identifier: Apache-2.0"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_fc880d1b","line":2,"range":{"start_line":1,"start_character":0,"end_line":2,"end_character":14},"updated":"2020-01-14 17:24:55.000000000","message":"Don\u0027t put vim configuration in comment headers.","commit_id":"6a23af62dc3496608c630c092b6ecfb549b5caa1"},{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"e476dc53828327478214688161b7a7bf36f7f63e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"# Copyright (C) 2020 Intel Corporation"},{"line_number":8,"context_line":"#"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"\"\"\"Test class for Sysinv Kube_app AppOperator.\"\"\""},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"from sysinv.conductor import kube_app"},{"line_number":13,"context_line":"from sysinv.db import api as dbapi"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_9c99d96c","line":10,"range":{"start_line":10,"start_character":25,"end_line":10,"end_character":33},"updated":"2020-01-14 17:24:55.000000000","message":"Should be lower case.","commit_id":"6a23af62dc3496608c630c092b6ecfb549b5caa1"},{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"e476dc53828327478214688161b7a7bf36f7f63e","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        super(AppOperatorTestCase, self).setUp()"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"        # Set up objects for testing"},{"line_number":25,"context_line":"        self.service \u003d kube_app.AppOperator(dbapi.get_instance())"},{"line_number":26,"context_line":"        self.context \u003d context.get_admin_context()"},{"line_number":27,"context_line":"        self.dbapi \u003d dbapi.get_instance()"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_3cbf85ac","line":25,"range":{"start_line":25,"start_character":13,"end_line":25,"end_character":20},"updated":"2020-01-14 17:24:55.000000000","message":"This isn\u0027t a service. Maybe \"app_operator\" would be a better name.","commit_id":"6a23af62dc3496608c630c092b6ecfb549b5caa1"},{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"e476dc53828327478214688161b7a7bf36f7f63e","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"        # Set up objects for testing"},{"line_number":25,"context_line":"        self.service \u003d kube_app.AppOperator(dbapi.get_instance())"},{"line_number":26,"context_line":"        self.context \u003d context.get_admin_context()"},{"line_number":27,"context_line":"        self.dbapi \u003d dbapi.get_instance()"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_bce055cd","line":26,"range":{"start_line":26,"start_character":8,"end_line":26,"end_character":50},"updated":"2020-01-14 17:24:55.000000000","message":"I don\u0027t see the AppOperator methods using the context - why is this needed?","commit_id":"6a23af62dc3496608c630c092b6ecfb549b5caa1"},{"author":{"_account_id":14174,"name":"Bart Wensley","email":"bartwensley@hotmail.com","username":"bartwensley"},"change_message_id":"2f8dd3411ab28f6033ea9120b59829491b2f5243","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"        # Set up objects for testing"},{"line_number":25,"context_line":"        self.service \u003d kube_app.AppOperator(dbapi.get_instance())"},{"line_number":26,"context_line":"        self.context \u003d context.get_admin_context()"},{"line_number":27,"context_line":"        self.dbapi \u003d dbapi.get_instance()"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_fd6fa7c4","line":26,"range":{"start_line":26,"start_character":8,"end_line":26,"end_character":50},"in_reply_to":"3fa7e38b_3716be4c","updated":"2020-01-14 19:21:34.000000000","message":"Thanks Al - that makes sense. I\u0027m not OK with updating the private _context member on these objects, but will suggest another way to do it.","commit_id":"6a23af62dc3496608c630c092b6ecfb549b5caa1"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"3cb88167a78ea4655ac64762ec0d1833bb5158ab","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"        # Set up objects for testing"},{"line_number":25,"context_line":"        self.service \u003d kube_app.AppOperator(dbapi.get_instance())"},{"line_number":26,"context_line":"        self.context \u003d context.get_admin_context()"},{"line_number":27,"context_line":"        self.dbapi \u003d dbapi.get_instance()"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_3716be4c","line":26,"range":{"start_line":26,"start_character":8,"end_line":26,"end_character":50},"in_reply_to":"3fa7e38b_bce055cd","updated":"2020-01-14 17:43:01.000000000","message":"The context is used by some later unit tests\nhttps://review.opendev.org/#/c/702361/\n\nThe _context is declared  the objects/base and I think it is needed when updating the object","commit_id":"6a23af62dc3496608c630c092b6ecfb549b5caa1"}]}