)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"be33e22d4e47a4446cb78ba445bd865d71554140","unresolved":false,"context_lines":[{"line_number":12,"context_line":"if any of their L3 agents is still using this config option."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/567369/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I92b1e30e806dfd20d99716d31c518845fd296a3d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"5fc1f717_82a428bb","line":15,"updated":"2019-03-19 10:19:36.000000000","message":"Add the related bug please (bug/1820763 I think)","commit_id":"979b5aa672af04cd0a92b22f57f941ac8d33eb6b"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e7e2c8cc433de9549c315a316b0591318d07ac45","unresolved":false,"context_lines":[{"line_number":12,"context_line":"if any of their L3 agents is still using this config option."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.openstack.org/#/c/567369/"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I92b1e30e806dfd20d99716d31c518845fd296a3d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"5fc1f717_b952badd","line":15,"in_reply_to":"5fc1f717_82a428bb","updated":"2019-03-19 11:22:20.000000000","message":"yes, I forgot about it. Thx","commit_id":"979b5aa672af04cd0a92b22f57f941ac8d33eb6b"}],"neutron/cmd/upgrade_checks/checks.py":[{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"5410a04f3f4966c625be1e64922ad284c6b89b86","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    def get_checks(self):"},{"line_number":39,"context_line":"        return ["},{"line_number":40,"context_line":"            (_(\"Worker counts configured\"), self.worker_count_check),"},{"line_number":41,"context_line":"            (_(\"External network bridge\"),"},{"line_number":42,"context_line":"             self.external_network_bridge_check)"},{"line_number":43,"context_line":"        ]"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5fc1f717_de100c51","line":41,"range":{"start_line":41,"start_character":16,"end_line":41,"end_character":24},"updated":"2019-03-15 02:01:34.000000000","message":"nit: alphabetical?","commit_id":"f0ee93009dc064b5a3fae98e0fbd12449058a597"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"dbc74a282003c0404376f1c6b48714c914271a1f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    def get_checks(self):"},{"line_number":39,"context_line":"        return ["},{"line_number":40,"context_line":"            (_(\"Worker counts configured\"), self.worker_count_check),"},{"line_number":41,"context_line":"            (_(\"External network bridge\"),"},{"line_number":42,"context_line":"             self.external_network_bridge_check)"},{"line_number":43,"context_line":"        ]"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5fc1f717_52088dd8","line":41,"range":{"start_line":41,"start_character":16,"end_line":41,"end_character":24},"in_reply_to":"5fc1f717_de100c51","updated":"2019-03-15 08:43:43.000000000","message":"Done","commit_id":"f0ee93009dc064b5a3fae98e0fbd12449058a597"},{"author":{"_account_id":4187,"name":"Ryan Tidwell","email":"rtidwell@suse.com","username":"ryan-tidwell"},"change_message_id":"bc2927f9bd65ef64cf6ed4b65b9cd960cf6d472f","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            if config.get(\"external_network_bridge\"):"},{"line_number":68,"context_line":"                return upgradecheck.Result("},{"line_number":69,"context_line":"                    upgradecheck.Code.WARNING,"},{"line_number":70,"context_line":"                    _(\"At least one of the L3 agents still use \""},{"line_number":71,"context_line":"                      \"\u0027external_network_bridge\u0027 config option to provide \""},{"line_number":72,"context_line":"                      \"gateway connectivity. This option is now removed. \""},{"line_number":73,"context_line":"                      \"Migration of routers from those L3 agents will be \""},{"line_number":74,"context_line":"                      \"required to connect them to external network through \""},{"line_number":75,"context_line":"                      \"integration bridge.\"))"},{"line_number":76,"context_line":"        return upgradecheck.Result("},{"line_number":77,"context_line":"            upgradecheck.Code.SUCCESS,"},{"line_number":78,"context_line":"            _(\"L3 agents use integration bridge to connect external gateways\"))"}],"source_content_type":"text/x-python","patch_set":6,"id":"5fc1f717_0ddb914f","line":75,"range":{"start_line":70,"start_character":20,"end_line":75,"end_character":42},"updated":"2019-03-18 22:44:23.000000000","message":"We should be able to identify which agent is misconfigured using agent.host, I think it would be a nice touch to compile a list of the agents that are misconfigured. This way operators don\u0027t have to hunt down which agents need to be re-configured.","commit_id":"f698ffe83498c1ee0b8d2a3fc007d80dfc999b46"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5d5d457dfc52ac58459d2751dcde3356ba284dd6","unresolved":false,"context_lines":[{"line_number":67,"context_line":"            if config.get(\"external_network_bridge\"):"},{"line_number":68,"context_line":"                return upgradecheck.Result("},{"line_number":69,"context_line":"                    upgradecheck.Code.WARNING,"},{"line_number":70,"context_line":"                    _(\"At least one of the L3 agents still use \""},{"line_number":71,"context_line":"                      \"\u0027external_network_bridge\u0027 config option to provide \""},{"line_number":72,"context_line":"                      \"gateway connectivity. This option is now removed. \""},{"line_number":73,"context_line":"                      \"Migration of routers from those L3 agents will be \""},{"line_number":74,"context_line":"                      \"required to connect them to external network through \""},{"line_number":75,"context_line":"                      \"integration bridge.\"))"},{"line_number":76,"context_line":"        return upgradecheck.Result("},{"line_number":77,"context_line":"            upgradecheck.Code.SUCCESS,"},{"line_number":78,"context_line":"            _(\"L3 agents use integration bridge to connect external gateways\"))"}],"source_content_type":"text/x-python","patch_set":6,"id":"5fc1f717_6226dcbd","line":75,"range":{"start_line":70,"start_character":20,"end_line":75,"end_character":42},"in_reply_to":"5fc1f717_0ddb914f","updated":"2019-03-19 09:30:11.000000000","message":"Very good point Ryan. I changed that. Thx","commit_id":"f698ffe83498c1ee0b8d2a3fc007d80dfc999b46"}]}
