)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a8c389b8c393d649d842d15d73555210e37e7881","unresolved":true,"context_lines":[{"line_number":9,"context_line":"By default our mariadb database for gitea nodes limits itself to a"},{"line_number":10,"context_line":"maximum of 100 connections. We\u0027ve seen errors like this:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" ...eb/routing/logger.go:102:func1() [I] router: completed POST /openstack/requirements/git-upload-pack for 127.0.0.1:50562, 500 Internal Server Error in 2.6ms @ context/user.go:17(web.gitHTTPRouters.UserAssignmentWeb)"},{"line_number":13,"context_line":" ...ules/context/repo.go:467:RepoAssignment() [E] GetUserByName: Error 1040: Too many connections"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"And after reading gitea\u0027s source code this appears to be related to user"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8844bc55_113058f9","line":12,"updated":"2024-02-02 12:59:49.000000000","message":"This commit message appears on the opendev.org/opendev/system-config page during the gitea test and thus triggers https://opendev.org/opendev/system-config/src/branch/master/testinfra/test_gitea.py#L133\n\nI think it is still good to have the explicit error here in the commit message, we can either temporarily disable the check or test a different repo. Or come up with some clever idea to make the check more specific?","commit_id":"d66bfbfd7b21543aa7ace0c22b604c7c521a64d7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"949eb5215766931ea120d2e35a45fac9fec2e966","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"13b41a36_7d11be57","updated":"2024-02-01 23:05:55.000000000","message":"Thanks.\n\nNot approving so we can bundle with other gitea changes.","commit_id":"d66bfbfd7b21543aa7ace0c22b604c7c521a64d7"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"dd9351b885fce6bec15e19a1e435e3568adb4df7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f46c4880_1f4531d4","updated":"2024-02-02 10:07:01.000000000","message":"recheck holding a build for debugging","commit_id":"d66bfbfd7b21543aa7ace0c22b604c7c521a64d7"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"27bfeda8680d0cd64bce1783ba1c5f94414fa1fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3a553890_7e339d4e","updated":"2024-02-05 21:48:08.000000000","message":"Approving here since only the file name changed. This was done to make the alphabetical sorting of config application less ambiguous. There are no other configs added in so this latest patchset should have the same behavior as the previous patchset.","commit_id":"dbe477b2052c248eb0c0b27e9366e91eb774fe29"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6c7bfbcaa69d50c0fadaea6134902ad1e2de4cf9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bab94b1d_c0b33150","updated":"2024-02-05 21:09:16.000000000","message":"recheck indications are that ppas should be working again.","commit_id":"dbe477b2052c248eb0c0b27e9366e91eb774fe29"}],"testinfra/test_gitea.py":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"54a6c424ac417a6ebc9076262f6f78170f35addd","unresolved":true,"context_lines":[{"line_number":128,"context_line":"    for path in paths_to_check:"},{"line_number":129,"context_line":"        cmd \u003d host.run(\u0027curl --insecure \u0027"},{"line_number":130,"context_line":"                       \u0027--resolve gitea99.opendev.org:3081:127.0.0.1 \u0027"},{"line_number":131,"context_line":"                       \u0027https://gitea99.opendev.org:3081\u0027 + path)"},{"line_number":132,"context_line":"        assert \u0027status-page-500\u0027 not in cmd.stdout"},{"line_number":133,"context_line":"        assert \u0027\u003ctitle\u003eInternal Server Error\u0027 not in cmd.stdout"},{"line_number":134,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"0cfbcb07_bdc4624d","line":131,"updated":"2024-02-04 21:49:58.000000000","message":"We should have a hosts override for gitea99 and trusted self-signed certs setup between the nodes now.  i.e. we could replace this with a requests.get() or similar that would give us the status code more easily","commit_id":"e569a07c76895a407d25ccbe05d3a58a50623d23"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"93b20ed355c4f524a75dddaecf986e91f67059ca","unresolved":true,"context_lines":[{"line_number":128,"context_line":"    for path in paths_to_check:"},{"line_number":129,"context_line":"        cmd \u003d host.run(\u0027curl --insecure \u0027"},{"line_number":130,"context_line":"                       \u0027--resolve gitea99.opendev.org:3081:127.0.0.1 \u0027"},{"line_number":131,"context_line":"                       \u0027https://gitea99.opendev.org:3081\u0027 + path)"},{"line_number":132,"context_line":"        assert \u0027status-page-500\u0027 not in cmd.stdout"},{"line_number":133,"context_line":"        assert \u0027\u003ctitle\u003eInternal Server Error\u0027 not in cmd.stdout"},{"line_number":134,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"6684df43_24ce4d7e","line":131,"in_reply_to":"0cfbcb07_bdc4624d","updated":"2024-02-05 16:28:36.000000000","message":"The issue is that the status code is 200. It is a gitea bug when rendering templates that errors are return success with an embedded http 500 document.","commit_id":"e569a07c76895a407d25ccbe05d3a58a50623d23"}]}