)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32841,"name":"Reinildes Oliveira","display_name":"Rei Oliveira","email":"Reinildes.JoseMateusOliveira@windriver.com","username":"rjosemat"},"change_message_id":"d3d27b112712771d3501971bd519f17d488c8f07","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Story: 2009669"},{"line_number":14,"context_line":"Task: 43901"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Thiago Brito \u003cthiago.brito@windriver.com\u003e"},{"line_number":17,"context_line":"Change-Id: I5faa17acf8de8649f8ad3a0de5c6fc85cf76836b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"11b0202a_5c08d08d","line":15,"updated":"2022-03-15 18:24:47.000000000","message":"nit: Missing Test Cases section","commit_id":"1dade6471e16ab26d6b4dc4584bd21995391d400"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"13ac3fa829838d409c0aeb14a389381fb9cfd531","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"67588f50_0b17176b","updated":"2021-11-02 17:42:36.000000000","message":"looks like this failed zuul, even though the parent commit https://review.opendev.org/c/starlingx/config/+/806288/   adds the missing STEVEDORE_PUPPET_PLUGINS_NS  constant.\nWill see if rebasing after the parent merges fixes zuul","commit_id":"5b7232febaf06c7603e4c668ec97760b7ff5c1a3"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"fc443e0a98abb2bd884637bd29179cbc284df724","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"9f6bb1af_3996c0a7","in_reply_to":"67588f50_0b17176b","updated":"2021-11-02 17:47:09.000000000","message":"looks like the problem here is that the relation chain (parent commit) is backwards.\n\nThat other commit is failing because of the pylint error introduced in this commit.\n\nsysinv/conductor/kube_app.py:146:4: W0102: Dangerous default value {} as argument (dangerous-default-value)","commit_id":"5b7232febaf06c7603e4c668ec97760b7ff5c1a3"},{"author":{"_account_id":32349,"name":"Thiago Paiva Brito","email":"thiago.brito@windriver.com","username":"outbrito"},"change_message_id":"2ff3a3726b07e0bf9231d353264ed54b27cc5861","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"bb0c7b70_803b3712","in_reply_to":"9f6bb1af_3996c0a7","updated":"2021-11-03 13:43:51.000000000","message":"Thanks for pointing those out, Al.","commit_id":"5b7232febaf06c7603e4c668ec97760b7ff5c1a3"},{"author":{"_account_id":32841,"name":"Reinildes Oliveira","display_name":"Rei Oliveira","email":"Reinildes.JoseMateusOliveira@windriver.com","username":"rjosemat"},"change_message_id":"2ac707f24701a932aaab80dfd871f5c9149260d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"0ee4b9f5_c9c28cfb","updated":"2022-03-15 18:26:50.000000000","message":"I have peeked into this change because it has a merge conflict with one of mine. \n\nI took the liberty of commenting on one issue in the exception class. It\u0027s an issue that I have just run into, please see if it\u0027s of any help.","commit_id":"1dade6471e16ab26d6b4dc4584bd21995391d400"},{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"3ee796f09c808b9a00869344157c438955700a4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"c3fec616_78bb0e09","updated":"2022-03-18 17:27:13.000000000","message":"this needs the merge conflict to be resolved","commit_id":"1dade6471e16ab26d6b4dc4584bd21995391d400"}],"sysinv/sysinv/sysinv/sysinv/common/exception.py":[{"author":{"_account_id":32841,"name":"Reinildes Oliveira","display_name":"Rei Oliveira","email":"Reinildes.JoseMateusOliveira@windriver.com","username":"rjosemat"},"change_message_id":"d3d27b112712771d3501971bd519f17d488c8f07","unresolved":true,"context_lines":[{"line_number":1625,"context_line":""},{"line_number":1626,"context_line":""},{"line_number":1627,"context_line":"class PluginLoadedException(SysinvException):"},{"line_number":1628,"context_line":"    message \u003d _(\"Plugin already loaded: %(plugin_name)\")"}],"source_content_type":"text/x-python","patch_set":24,"id":"b0cf42d0_4c0a317d","line":1628,"updated":"2022-03-15 18:24:47.000000000","message":"I have peeked into this change because it has a merge conflict with one of mine.\n\nI think you are missing a \u0027s\u0027 in plugin_name.From %(plugin_name) to %(plugin_name)s","commit_id":"1dade6471e16ab26d6b4dc4584bd21995391d400"}],"sysinv/sysinv/sysinv/sysinv/conductor/manager.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"eea6f5f220be5c8ec1d839632cd9fb4a7ca3f99a","unresolved":true,"context_lines":[{"line_number":303,"context_line":"        # ceph for the initial unlock."},{"line_number":304,"context_line":"        self._helm \u003d helm.HelmOperator(self.dbapi)"},{"line_number":305,"context_line":"        self._app \u003d kube_app.AppOperator(self.dbapi, self._helm, self._puppet,"},{"line_number":306,"context_line":"                                         self.apps_metadata)"},{"line_number":307,"context_line":"        self._docker \u003d kube_app.DockerHelper(self.dbapi)"},{"line_number":308,"context_line":"        self._kube \u003d kubernetes.KubeOperator()"},{"line_number":309,"context_line":"        self._armada \u003d kube_app.ArmadaHelper(self._kube)"}],"source_content_type":"text/x-python","patch_set":23,"id":"30d08dd7_538d4256","line":306,"updated":"2021-11-05 17:39:00.000000000","message":"nit: consider passing this as: apps_metadata\u003dself.apps_metadata  since its a named argument.\nthe change is not needed, its just protective in case the argument list changes again","commit_id":"0a403f797ba8b5164dcb6e74f32a7e785e554581"}],"sysinv/sysinv/sysinv/sysinv/exceptions.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"76970b2b2a52704c58b4ce662f1e070755364923","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Copyright (c) 2021 Wind River Systems, Inc."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# SPDX-License-Identifier: Apache-2.0"},{"line_number":5,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":22,"id":"9ee9795f_20b0d6c6","line":2,"updated":"2021-11-03 18:25:02.000000000","message":"We actually already have sysinv/common/exception.py\n\nIt defines  what we typically consider the base level exception in sysinv.\n\nclass SysinvException(Exception):\n    \"\"\"Base Sysinv Exception\"\"\"","commit_id":"0b1f5a1658c18dad98046f3897f2f9f06d5cd07a"},{"author":{"_account_id":32349,"name":"Thiago Paiva Brito","email":"thiago.brito@windriver.com","username":"outbrito"},"change_message_id":"85870c015d0665c2809211b90c5647f26198629c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#"},{"line_number":2,"context_line":"# Copyright (c) 2021 Wind River Systems, Inc."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# SPDX-License-Identifier: Apache-2.0"},{"line_number":5,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":22,"id":"42831744_70151bda","line":2,"in_reply_to":"9ee9795f_20b0d6c6","updated":"2021-11-03 18:37:44.000000000","message":"Awesome. Done.","commit_id":"0b1f5a1658c18dad98046f3897f2f9f06d5cd07a"}],"sysinv/sysinv/sysinv/sysinv/puppet/puppet.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"eea6f5f220be5c8ec1d839632cd9fb4a7ca3f99a","unresolved":true,"context_lines":[{"line_number":93,"context_line":"            LOG.info(\"Couldn\u0027t find endpoint distribution located at %s for \""},{"line_number":94,"context_line":"                     \"%s\" % (install_location, puppet_entrypoint.dist))"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def purge_cache(self):"},{"line_number":97,"context_line":"        \"\"\"Purge the stevedore entry point cache.\"\"\""},{"line_number":98,"context_line":"        try:"},{"line_number":99,"context_line":"            del extension.ExtensionManager.ENTRY_POINT_CACHE["}],"source_content_type":"text/x-python","patch_set":23,"id":"38f807d3_e37500c8","line":96,"range":{"start_line":96,"start_character":8,"end_line":96,"end_character":19},"updated":"2021-11-05 17:39:00.000000000","message":"nit: consider renaming this _purge_cache to indicate it is a private method.  ie: I dont think you want anything calling this because it not protected by a lock.","commit_id":"0a403f797ba8b5164dcb6e74f32a7e785e554581"}]}