)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b64ead76b8ac9a184e8ff7ff08491945a26c04a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"636ce14f_8044ee54","updated":"2023-09-08 15:17:33.000000000","message":"In comparing this to the old change I had for java 17 I noticed a couple more things that should be updated:\n\nhttps://review.opendev.org/c/opendev/system-config/+/870877/3/playbooks/roles/gerrit/templates/gerrit.config.j2\n\nand\n\nhttps://review.opendev.org/c/opendev/system-config/+/870877/3/playbooks/zuul/gerrit/repos.yaml\n\nI think these are both minor but worth keeping in sync. I think the first one is only applicable when starting Gerrit with their init script which we don\u0027t do in our containers and may be a candidate for future cleanup. The second is the java we use to build Gerrit and I think we should build Gerrit with java 17 if we plan to run it with java 17.","commit_id":"cb8c4d4e555177913264fb1f96fdd73260df35a0"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"477205e2443e2a25b7eba992ac8b761362e382d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3d100456_60984393","updated":"2023-08-25 20:41:55.000000000","message":"We should consider if it makes sense to split the java upgrade and the OS upgrade. We can switch to java 17 on either bullseye or bookworm and bookworm also has java 11. This means the order doesn\u0027t matter too much if we decide to split things up.\n\nThat said the main distro interactions are going to be with git (I think we use container git to run git gc) and python for jeepyb. And neither of those are going to interact with java things. That means it is probably pretty safe to bundle them up and we may just end up debugging independent issues if anything goes wrong.\n\nFinally, if we are really concerned we can hold a node and inspect it a bit more closely. That said our testing is pretty good for Gerrit.","commit_id":"cb8c4d4e555177913264fb1f96fdd73260df35a0"}]}