)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e6c353659e08be4dc3ebdba9535d1495fc2e1a26","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1ef871f7_bc9e4f83","updated":"2024-04-18 00:03:55.000000000","message":"How sure are we this works?  I\u0027m remembering back to I596d4e129b1617b9d4e52c0d9bc969db906ea4ff and I think the package compression changed.  \n\nI don\u0027t really have an issue, this has a nice big comment that explains what\u0027s going on which is the important bit 😊","commit_id":"5ff7dd8289e0171d627f9f6e4b27c51df21671d8"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4ff490a0f01d42540a7f8c67a643b131fbf3b956","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4de88885_3c55dd44","updated":"2024-04-18 05:40:14.000000000","message":"See https://review.opendev.org/c/openstack/diskimage-builder/+/915915 for verification, I also tested the build locally.","commit_id":"5ff7dd8289e0171d627f9f6e4b27c51df21671d8"}],"Dockerfile":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"fb400b8d46993cf0aa44a16c5efa41f05467a227","unresolved":true,"context_lines":[{"line_number":84,"context_line":"# symlink ourselves which is the only functional change that is"},{"line_number":85,"context_line":"# actually needed."},{"line_number":86,"context_line":"# https://salsa.debian.org/installer-team/debootstrap/-/commit/1223abb5e0c02f9145002e5838f909fe69bbb403"},{"line_number":87,"context_line":"RUN ln -s gutsy /usr/share/debootstrap/scripts/noble"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"# Podman install mainly for the \"containerfile\" elements of dib that"},{"line_number":90,"context_line":"# build images from extracts of upstream containers."}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"5196df4d_de8febe2","line":87,"range":{"start_line":87,"start_character":10,"end_line":87,"end_character":15},"updated":"2024-04-17 15:26:51.000000000","message":"Nit, should this be a fully rooted path to make it more clear what the symlink target is relative to on the filesystem?","commit_id":"5ff7dd8289e0171d627f9f6e4b27c51df21671d8"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a296afb797b0b6ee3604c8fb53d0118972e69b39","unresolved":false,"context_lines":[{"line_number":84,"context_line":"# symlink ourselves which is the only functional change that is"},{"line_number":85,"context_line":"# actually needed."},{"line_number":86,"context_line":"# https://salsa.debian.org/installer-team/debootstrap/-/commit/1223abb5e0c02f9145002e5838f909fe69bbb403"},{"line_number":87,"context_line":"RUN ln -s gutsy /usr/share/debootstrap/scripts/noble"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"# Podman install mainly for the \"containerfile\" elements of dib that"},{"line_number":90,"context_line":"# build images from extracts of upstream containers."}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"726b532a_2b847c69","line":87,"range":{"start_line":87,"start_character":10,"end_line":87,"end_character":15},"in_reply_to":"5196df4d_de8febe2","updated":"2024-04-17 17:55:47.000000000","message":"This is how all the existing symlinks look like, see also the referenced upstream commit:\n\n```\nroot@3a64208f93fa:/# ls -l /usr/share/debootstrap/scripts/jammy\nlrwxrwxrwx 1 root root 5 Dec 20 19:25 /usr/share/debootstrap/scripts/jammy -\u003e gutsy\n```\n\nThese are simply relative to the target basename.","commit_id":"5ff7dd8289e0171d627f9f6e4b27c51df21671d8"}]}