)]}'
{"sysinv/sysinv/sysinv/sysinv/helm/flux.py":[{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"0fd0e0fbb1faacb347b231a3ec6c78d51acc058e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":13,"id":"4bcbb2a5_ecd1d82c","updated":"2025-11-12 18:15:52.000000000","message":"Need:\n\n    #\n    # Copyright (c) 2025 Wind River Systems, Inc.\n    #\n    # SPDX-License-Identifier: Apache-2.0\n    #","commit_id":"d9380dcd1bd0d6999af9a4ce9c3e9f97aea68d5e"},{"author":{"_account_id":35446,"name":"Igor Pires Soares","display_name":"Igor Soares","email":"Igor.PiresSoares@windriver.com","username":"ipiresso"},"change_message_id":"2b3aff880e0db689e872843f33ed988d159972d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"215cc738_98232d9f","in_reply_to":"4bcbb2a5_ecd1d82c","updated":"2025-11-12 18:53:13.000000000","message":"Done","commit_id":"d9380dcd1bd0d6999af9a4ce9c3e9f97aea68d5e"},{"author":{"_account_id":28459,"name":"Bob Church","email":"robert.church@windriver.com","username":"rchurch"},"change_message_id":"0fd0e0fbb1faacb347b231a3ec6c78d51acc058e","unresolved":true,"context_lines":[{"line_number":17,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class FluxDeploymentManager(object):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    def __init__(self, dbapi):"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"436b7e46_7d88cb29","line":20,"updated":"2025-11-12 18:15:52.000000000","message":"Eventually, I\u0027d like to incorporate this into a k8s KOPF based ApplicationOperator, where the operator is deployed in a fresh k8s cluster.  As part of its initial startup it will bootstrap what it needs for operation. \n\nSo this operator would probably define a CRD and consume a CR for the desired configuration of Flux Controllers (chart version, chart repo/location, overrides,...) as this class would consume the incormation to initially install and/or upgrade based on the details provided.","commit_id":"d9380dcd1bd0d6999af9a4ce9c3e9f97aea68d5e"},{"author":{"_account_id":35446,"name":"Igor Pires Soares","display_name":"Igor Soares","email":"Igor.PiresSoares@windriver.com","username":"ipiresso"},"change_message_id":"2b3aff880e0db689e872843f33ed988d159972d3","unresolved":false,"context_lines":[{"line_number":17,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class FluxDeploymentManager(object):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    def __init__(self, dbapi):"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"d7424e51_ac870607","line":20,"in_reply_to":"436b7e46_7d88cb29","updated":"2025-11-12 18:53:13.000000000","message":"Acknowledged","commit_id":"d9380dcd1bd0d6999af9a4ce9c3e9f97aea68d5e"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"ca00bb556eb8d14dd03fa4f0897b57439a1e5cf7","unresolved":true,"context_lines":[{"line_number":145,"context_line":"    def rollback_controllers(self):"},{"line_number":146,"context_line":"        \"\"\" Rollback Flux controllers to the previous version \"\"\""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"        raise NotImplementedError"}],"source_content_type":"text/x-python","patch_set":14,"id":"8edb6b90_31d83d2e","line":148,"range":{"start_line":148,"start_character":14,"end_line":148,"end_character":33},"updated":"2025-11-13 14:59:04.000000000","message":"Just confirming, rollback is coming in a follow up commit? We\u0027ll need it to activate-rollback an upgrade.","commit_id":"0c98b2139475a072e515f7df6fbd4201701615f5"},{"author":{"_account_id":35446,"name":"Igor Pires Soares","display_name":"Igor Soares","email":"Igor.PiresSoares@windriver.com","username":"ipiresso"},"change_message_id":"48937769bb7ead97403446b30429be7a3765f14d","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    def rollback_controllers(self):"},{"line_number":146,"context_line":"        \"\"\" Rollback Flux controllers to the previous version \"\"\""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"        raise NotImplementedError"}],"source_content_type":"text/x-python","patch_set":14,"id":"a127a7a9_12813f01","line":148,"range":{"start_line":148,"start_character":14,"end_line":148,"end_character":33},"in_reply_to":"8edb6b90_31d83d2e","updated":"2025-11-13 15:51:08.000000000","message":"Yep, rollback is coming in a follow up commit. This review won\u0027t change upgrade and rollback behaviors once it is merged. A review for the update repo under this same topic will be provided later on to actually enable the new upgrade/rollback code developed on the sysinv side.","commit_id":"0c98b2139475a072e515f7df6fbd4201701615f5"}]}
