)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6eeb428bda22bfb406699a6171b1e50157c653dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"89fde5bf_072dbfdd","updated":"2025-07-08 07:02:30.000000000","message":"recheck unstable stream, maybe we need to make it n-v?","commit_id":"df21d7166acfff87215569350e775c2533c3d055"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"3bef076868540d3584d547a23ccfb5afdeaca394","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"38d26bba_00986a5d","updated":"2025-07-08 15:56:09.000000000","message":"Holding off on a +2 since I wrote a big chunk of this change.","commit_id":"f288b3fbc466090b4374460dff7af0ca5e5d605b"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"65437782cf639349e98761ead0b166f46acf55db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"72109c5d_59307289","updated":"2025-07-08 16:00:54.000000000","message":"+1 simply because I wrote a large chunk of this change.","commit_id":"d2b9b66a295d549d4a379a7fbbf5859aca240645"}],"dib-elements/infra-package-needs/post-install.d/80-enable-infra-services":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6eeb428bda22bfb406699a6171b1e50157c653dd","unresolved":true,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" \u0026\u0026 $DIB_RELEASE \u003e 7 ) || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"c2dbd1fa_10acef19","line":20,"range":{"start_line":20,"start_character":46,"end_line":20,"end_character":63},"updated":"2025-07-08 07:02:30.000000000","message":"nit: this also looks obsolete?","commit_id":"df21d7166acfff87215569350e775c2533c3d055"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6a72be0e51029d52542c8e4f6b5165ebd86fb89e","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" \u0026\u0026 $DIB_RELEASE \u003e 7 ) || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"d99d50a8_90bcd2a9","line":20,"range":{"start_line":20,"start_character":46,"end_line":20,"end_character":63},"in_reply_to":"c2dbd1fa_10acef19","updated":"2025-07-08 15:13:30.000000000","message":"Done","commit_id":"df21d7166acfff87215569350e775c2533c3d055"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"33f86fd1f88d41d739c5f5c5bbb776090433d153","unresolved":true,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" \u0026\u0026 ${DIB_RELEASE//[^0-9]/} -gt 7 ) || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"048aa3c4_a0aaa3c0","line":20,"range":{"start_line":20,"start_character":46,"end_line":20,"end_character":78},"updated":"2025-07-08 09:29:16.000000000","message":"I was thinking that this isn\u0027t needed at all, i.e.\n```suggestion\n        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then\n```\nor something more fancyful like in L15","commit_id":"f257dc26a2a026d65d5b842233c927f1c44683ef"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"aa944febbd0a8ae794529f83efeaac6a0cbf1acd","unresolved":true,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" \u0026\u0026 ${DIB_RELEASE//[^0-9]/} -gt 7 ) || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"8c78f673_f43eadf6","line":20,"range":{"start_line":20,"start_character":46,"end_line":20,"end_character":78},"in_reply_to":"0291bd59_c356751e","updated":"2025-07-08 14:45:20.000000000","message":"Yes I think that is correct. We could simplify this further. Maybe in a followup since this stack is now green?","commit_id":"f257dc26a2a026d65d5b842233c927f1c44683ef"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"4f0b3f523015811391c43aa9af933cd8e64da53d","unresolved":true,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" \u0026\u0026 ${DIB_RELEASE//[^0-9]/} -gt 7 ) || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"0291bd59_c356751e","line":20,"range":{"start_line":20,"start_character":46,"end_line":20,"end_character":78},"in_reply_to":"048aa3c4_a0aaa3c0","updated":"2025-07-08 12:44:30.000000000","message":"Well, I think we don\u0027t have centos7 images, so it might be ok - but I\u0027ll wait for @cboylan@sapwetik.org to decide ;-)","commit_id":"f257dc26a2a026d65d5b842233c927f1c44683ef"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6a72be0e51029d52542c8e4f6b5165ebd86fb89e","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" \u0026\u0026 ${DIB_RELEASE//[^0-9]/} -gt 7 ) || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"6d800e5c_cdd26151","line":20,"range":{"start_line":20,"start_character":46,"end_line":20,"end_character":78},"in_reply_to":"8c78f673_f43eadf6","updated":"2025-07-08 15:13:30.000000000","message":"Fix applied.","commit_id":"f257dc26a2a026d65d5b842233c927f1c44683ef"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"b979fdf2ad2a3f31734ed77a335f24fab5272d1c","unresolved":true,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"6f1a70cb_836071f4","line":20,"range":{"start_line":20,"start_character":16,"end_line":20,"end_character":17},"updated":"2025-07-08 15:58:20.000000000","message":"This is a stray (","commit_id":"f288b3fbc466090b4374460dff7af0ca5e5d605b"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"65437782cf639349e98761ead0b166f46acf55db","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        elif [[ $DISTRO_NAME \u003d \"ubuntu\" || $DISTRO_NAME \u003d \"debian\" ]]; then"},{"line_number":18,"context_line":"            # stick with default systemd timesyncd on bookworm, focal and beyond"},{"line_number":19,"context_line":"            exit 0"},{"line_number":20,"context_line":"        elif [[ ( $DISTRO_NAME \u003d\u003d \"centos\" || $DISTRO_NAME \u003d\u003d \"fedora\" || $DISTRO_NAME \u003d\u003d \"rocky\" ]]; then"},{"line_number":21,"context_line":"            systemctl enable chronyd"},{"line_number":22,"context_line":"        else"},{"line_number":23,"context_line":"            systemctl enable ntpd.service"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"27424dc4_a6a57efc","line":20,"range":{"start_line":20,"start_character":16,"end_line":20,"end_character":17},"in_reply_to":"6f1a70cb_836071f4","updated":"2025-07-08 16:00:54.000000000","message":"Done","commit_id":"f288b3fbc466090b4374460dff7af0ca5e5d605b"}]}