)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"d679c1cb35e46686c9f360e0d9a52e5e5c50e849","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"2822144c_7a6efa0a","updated":"2024-03-26 17:42:56.000000000","message":"Held node shows that our plugin for headers and code etc formatting is not working. WIP until we understand that better.","commit_id":"9f9e3c5065371df0dd4b9d72c87ffb8c99e30229"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"8620e34483a5d06477dcc20af209382ad43d51b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"17e6ae19_3ddc821f","updated":"2024-04-30 23:46:36.000000000","message":"Looks good to me.","commit_id":"b2607979ab25ceaf1353c34149cec9e35af57329"}],"docker/etherpad/Dockerfile":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"537f087594d2f3a6df2f894ec1c60967341c21ca","unresolved":false,"context_lines":[{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"# Copy the configuration file."},{"line_number":173,"context_line":"COPY --chown\u003detherpad:etherpad ${SETTINGS} \"${EP_DIR}\"/settings.json"},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"# Fix group permissions"},{"line_number":176,"context_line":"RUN chmod -R g\u003du ."}],"source_content_type":"text/x-dockerfile","patch_set":16,"id":"ba72b8c6_8efe73fd","line":173,"updated":"2024-04-30 19:09:05.000000000","message":"We should probably chmod this 0600 as well, but can do it in a separate change. The file already contains the password to the database backend, so these loose perms aren\u0027t a regression, just something we need to fix at some point. Same probably goes for the old APIKEY.txt file inside the container (just looked out of curiosity).","commit_id":"b2607979ab25ceaf1353c34149cec9e35af57329"}],"zuul.d/docker-images/etherpad.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"882c2aa6b52cfd3eb8cf5d9b43af2a457aa9eb58","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    vars: \u0026etherpad_vars"},{"line_number":8,"context_line":"      docker_images:"},{"line_number":9,"context_line":"        - context: docker/etherpad"},{"line_number":10,"context_line":"          target: production"},{"line_number":11,"context_line":"          repository: opendevorg/etherpad"},{"line_number":12,"context_line":"    files: \u0026etherpad_files"},{"line_number":13,"context_line":"      - docker/etherpad/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"579eed50_11d57bc0","line":10,"updated":"2024-03-25 16:43:55.000000000","message":"We should double check that this does what we want on the registry side. Basically we should build the thing that the Dockerfile knows as \"production\" but then push it to the registry as opendevorg/etherpad:latest.","commit_id":"284ff01fe49a34c02a50de59f1e39fce7039be2a"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"8620e34483a5d06477dcc20af209382ad43d51b3","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    vars: \u0026etherpad_vars"},{"line_number":8,"context_line":"      docker_images:"},{"line_number":9,"context_line":"        - context: docker/etherpad"},{"line_number":10,"context_line":"          target: production"},{"line_number":11,"context_line":"          repository: opendevorg/etherpad"},{"line_number":12,"context_line":"    files: \u0026etherpad_files"},{"line_number":13,"context_line":"      - docker/etherpad/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6685c281_7f2efd5a","line":10,"in_reply_to":"579eed50_11d57bc0","updated":"2024-04-30 23:46:36.000000000","message":"AFAICT that\u0027s what will happen.  Do you want to do more testing locally, or see what happens as this merges","commit_id":"284ff01fe49a34c02a50de59f1e39fce7039be2a"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"8fd2191594f390a6fd33f62551921c03df57a8ee","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    vars: \u0026etherpad_vars"},{"line_number":8,"context_line":"      docker_images:"},{"line_number":9,"context_line":"        - context: docker/etherpad"},{"line_number":10,"context_line":"          target: production"},{"line_number":11,"context_line":"          repository: opendevorg/etherpad"},{"line_number":12,"context_line":"    files: \u0026etherpad_files"},{"line_number":13,"context_line":"      - docker/etherpad/"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9c453c36_2bd39fe7","line":10,"in_reply_to":"6685c281_7f2efd5a","updated":"2024-05-01 15:20:03.000000000","message":"I think the existing testing has covered this reasonably well since we know we\u0027re pulling :latest in the docker-compose file and then end up with something that has new code and no apikey so we know we\u0027re actually updating with the new code.","commit_id":"284ff01fe49a34c02a50de59f1e39fce7039be2a"}]}