)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33631,"name":"Fabricio Henrique Ramos","email":"fabriciohenrique.ramos@windriver.com","username":"fhramos"},"change_message_id":"9b21d6fd76add713d1bc9aa1b2f17bb01d440e92","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Luis Eduardo Bonatti \u003cLuizEduardo.Bonatti@windriver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-05 17:07:51 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Data migration fluxcd"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the controller-0 needs to be unlocked after the"},{"line_number":10,"context_line":"upgrade, it never been inventoried cause the error"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"873d5639_a0e95174","line":7,"updated":"2022-08-05 18:32:23.000000000","message":"I think this could be improved to better express what this commit is doing,\na suggestion: Add logic to migrate fluxcd config","commit_id":"c74d05c29cbd7a9076d12eaa324b3e747701fc16"},{"author":{"_account_id":33684,"name":"Luis Eduardo Bonatti","email":"LuizEduardo.Bonatti@windriver.com","username":"lbonatti"},"change_message_id":"daea36b6da1481d57264988a961fd223afd82c45","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Luis Eduardo Bonatti \u003cLuizEduardo.Bonatti@windriver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-08-05 17:07:51 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Data migration fluxcd"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the controller-0 needs to be unlocked after the"},{"line_number":10,"context_line":"upgrade, it never been inventoried cause the error"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a20e1242_6be7bce3","line":7,"in_reply_to":"873d5639_a0e95174","updated":"2022-08-08 13:08:14.000000000","message":"Done","commit_id":"c74d05c29cbd7a9076d12eaa324b3e747701fc16"},{"author":{"_account_id":33631,"name":"Fabricio Henrique Ramos","email":"fabriciohenrique.ramos@windriver.com","username":"fhramos"},"change_message_id":"9b21d6fd76add713d1bc9aa1b2f17bb01d440e92","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Data migration fluxcd"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the controller-0 needs to be unlocked after the"},{"line_number":10,"context_line":"upgrade, it never been inventoried cause the error"},{"line_number":11,"context_line":"SysinvException Mandatory FluxCD yaml file doesn\u0027t exist"},{"line_number":12,"context_line":"that do not exist on the controller-1 cause the fluxcd"},{"line_number":13,"context_line":"wasn\u0027t migrate on data-migration."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Test Plan:"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1dbf77bb_3930bde1","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":33},"updated":"2022-08-05 18:32:23.000000000","message":"nit: the way this description is worded is a little confusing, would you mind rewording?","commit_id":"c74d05c29cbd7a9076d12eaa324b3e747701fc16"},{"author":{"_account_id":33684,"name":"Luis Eduardo Bonatti","email":"LuizEduardo.Bonatti@windriver.com","username":"lbonatti"},"change_message_id":"daea36b6da1481d57264988a961fd223afd82c45","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Data migration fluxcd"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the controller-0 needs to be unlocked after the"},{"line_number":10,"context_line":"upgrade, it never been inventoried cause the error"},{"line_number":11,"context_line":"SysinvException Mandatory FluxCD yaml file doesn\u0027t exist"},{"line_number":12,"context_line":"that do not exist on the controller-1 cause the fluxcd"},{"line_number":13,"context_line":"wasn\u0027t migrate on data-migration."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Test Plan:"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6de244c9_f420d7ea","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":33},"in_reply_to":"1dbf77bb_3930bde1","updated":"2022-08-08 13:08:14.000000000","message":"Thank you for the feedback!","commit_id":"c74d05c29cbd7a9076d12eaa324b3e747701fc16"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"9a5e7d127b0663cc2266597ffc567547c30496e2","unresolved":true,"context_lines":[{"line_number":14,"context_line":"Test Plan:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"PASS:"},{"line_number":17,"context_line":"CentOS -\u003e CentOS upgrade:"},{"line_number":18,"context_line":"- 22.06 -\u003e 22.12"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Story: 2009303"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bb14c090_d744de07","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":25},"updated":"2022-08-08 15:34:54.000000000","message":"Does this apply to both DX and SX test scenarios? I assume this test was for DX, right?","commit_id":"e8d8c61e21293036d3cb1832c0f2d6d8c0c4cbd8"},{"author":{"_account_id":33684,"name":"Luis Eduardo Bonatti","email":"LuizEduardo.Bonatti@windriver.com","username":"lbonatti"},"change_message_id":"e35ca5307aaed93c448f9cf31f8b2a108a44492c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"Test Plan:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"PASS:"},{"line_number":17,"context_line":"CentOS -\u003e CentOS upgrade:"},{"line_number":18,"context_line":"- 22.06 -\u003e 22.12"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Story: 2009303"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"412c3493_7b73651f","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":25},"in_reply_to":"bb14c090_d744de07","updated":"2022-08-08 15:42:17.000000000","message":"Yeah, Duplex","commit_id":"e8d8c61e21293036d3cb1832c0f2d6d8c0c4cbd8"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"21875f6305841883e032ef51965b89cd7ce1cbab","unresolved":true,"context_lines":[{"line_number":16,"context_line":"PASS:"},{"line_number":17,"context_line":"CentOS -\u003e CentOS DX upgrade:"},{"line_number":18,"context_line":"- 22.06 -\u003e 22.12"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Story: 2009303"},{"line_number":21,"context_line":"Task: 45962"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"2f948ad2_9668c45d","line":19,"updated":"2022-08-08 17:34:52.000000000","message":"Please also verify AIO-SX upgrade as that code path is also updated.","commit_id":"f8e0276101fc01031b077c062691f093c3bf6cf4"}],"controllerconfig/controllerconfig/controllerconfig/upgrades/controller.py":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"21875f6305841883e032ef51965b89cd7ce1cbab","unresolved":true,"context_lines":[{"line_number":329,"context_line":"def migrate_fluxcd_config(from_release, to_release):"},{"line_number":330,"context_line":"    \"\"\" Migrates fluxcd configuration. \"\"\""},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"    # Check if the folder exist before migration"},{"line_number":333,"context_line":"    if not os.path.exists(os.path.join(PLATFORM_PATH, \"fluxcd\")):"},{"line_number":334,"context_line":"        LOG.info(\"Skipping fluxcd migration, the directory doesn\u0027t exist\")"},{"line_number":335,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":5,"id":"2dfa4f4d_ff0e49b0","line":332,"range":{"start_line":332,"start_character":26,"end_line":332,"end_character":32},"updated":"2022-08-08 17:34:52.000000000","message":"nit: exists","commit_id":"f8e0276101fc01031b077c062691f093c3bf6cf4"},{"author":{"_account_id":33684,"name":"Luis Eduardo Bonatti","email":"LuizEduardo.Bonatti@windriver.com","username":"lbonatti"},"change_message_id":"3bec077625993bf9947eb2d379a615722469d0fe","unresolved":false,"context_lines":[{"line_number":329,"context_line":"def migrate_fluxcd_config(from_release, to_release):"},{"line_number":330,"context_line":"    \"\"\" Migrates fluxcd configuration. \"\"\""},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"    # Check if the folder exist before migration"},{"line_number":333,"context_line":"    if not os.path.exists(os.path.join(PLATFORM_PATH, \"fluxcd\")):"},{"line_number":334,"context_line":"        LOG.info(\"Skipping fluxcd migration, the directory doesn\u0027t exist\")"},{"line_number":335,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":5,"id":"7e0b3c12_312d0832","line":332,"range":{"start_line":332,"start_character":26,"end_line":332,"end_character":32},"in_reply_to":"2dfa4f4d_ff0e49b0","updated":"2022-08-08 20:02:15.000000000","message":"Done","commit_id":"f8e0276101fc01031b077c062691f093c3bf6cf4"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"21875f6305841883e032ef51965b89cd7ce1cbab","unresolved":true,"context_lines":[{"line_number":338,"context_line":"    devnull \u003d open(os.devnull, \u0027w\u0027)"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    # Copy the entire fluxcd.cfg directory to pick up any changes made"},{"line_number":341,"context_line":"    # after the data was migrated (i.e. updates to the controller-1 load)."},{"line_number":342,"context_line":"    source_fluxcd \u003d os.path.join(PLATFORM_PATH, \"fluxcd\", from_release, \"\")"},{"line_number":343,"context_line":"    dest_fluxcd \u003d os.path.join(PLATFORM_PATH, \"fluxcd\", to_release)"},{"line_number":344,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":5,"id":"984ca9a5_90d97de7","line":341,"range":{"start_line":341,"start_character":55,"end_line":341,"end_character":67},"updated":"2022-08-08 17:34:52.000000000","message":"comment doesnt apply to SX case","commit_id":"f8e0276101fc01031b077c062691f093c3bf6cf4"},{"author":{"_account_id":33684,"name":"Luis Eduardo Bonatti","email":"LuizEduardo.Bonatti@windriver.com","username":"lbonatti"},"change_message_id":"3bec077625993bf9947eb2d379a615722469d0fe","unresolved":false,"context_lines":[{"line_number":338,"context_line":"    devnull \u003d open(os.devnull, \u0027w\u0027)"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"    # Copy the entire fluxcd.cfg directory to pick up any changes made"},{"line_number":341,"context_line":"    # after the data was migrated (i.e. updates to the controller-1 load)."},{"line_number":342,"context_line":"    source_fluxcd \u003d os.path.join(PLATFORM_PATH, \"fluxcd\", from_release, \"\")"},{"line_number":343,"context_line":"    dest_fluxcd \u003d os.path.join(PLATFORM_PATH, \"fluxcd\", to_release)"},{"line_number":344,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":5,"id":"a3f13f2f_704be1a1","line":341,"range":{"start_line":341,"start_character":55,"end_line":341,"end_character":67},"in_reply_to":"984ca9a5_90d97de7","updated":"2022-08-08 20:02:15.000000000","message":"Done","commit_id":"f8e0276101fc01031b077c062691f093c3bf6cf4"}]}