)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"847a9cd0847f74d93f3b130e54b46b792dd0f07a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2d6cce01_90d1f19e","updated":"2021-05-23 17:56:04.000000000","message":"Can you split this into two changes (DNS change first)? That needs to be deployed before the nginx changes, otherwise the ACME units will be throwing errors.","commit_id":"75cec238002c6435af45aa6334e056bcb5fb8f89"},{"author":{"_account_id":1000010,"name":"aspen","email":"root@gws.fyi","username":"aspen"},"change_message_id":"aa55dce3a04787d7aac9fac6996b0357d272d885","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0fabbe3f_1580035b","in_reply_to":"2d6cce01_90d1f19e","updated":"2021-05-23 19:56:09.000000000","message":"CL/3149","commit_id":"75cec238002c6435af45aa6334e056bcb5fb8f89"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"f1916344be54549bc5798ea99abd57e452a8dd6c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"466723ff_91f6d80c","updated":"2021-09-03 11:54:49.000000000","message":"The folder should also be added to the Restic backup directories I think, they\u0027re configured in the main whitby config.","commit_id":"289cdc83426905cd861a6568fc169bb5c715cd84"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"8cf2ebf566ec0e91378fd763a7cd6a2e66d244ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"36f73b38_1e9248b0","in_reply_to":"466723ff_91f6d80c","updated":"2021-09-10 11:47:36.000000000","message":"Done","commit_id":"289cdc83426905cd861a6568fc169bb5c715cd84"}],"ops/modules/www/deploys.tvl.fyi.nix":[{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"278888839111f6f1e949d784919608736148353f","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    systemd.services.ensure-deploys-dir \u003d {"},{"line_number":10,"context_line":"      description \u003d \"Ensure the html directory for deploys.tvl.fyi exists\";"},{"line_number":11,"context_line":"      wantedBy \u003d [ \"multi-user.target\" ];"},{"line_number":12,"context_line":"      script \u003d \"mkdir -p /srv/www/deploys.tvl.fyi\";"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      serviceConfig \u003d {"},{"line_number":15,"context_line":"        User \u003d \"nginx\";"}],"source_content_type":"text/x-nix","patch_set":6,"id":"fc894624_55693155","line":12,"range":{"start_line":12,"start_character":25,"end_line":12,"end_character":49},"updated":"2021-06-01 14:48:14.000000000","message":"unsure about this service; /srv is not a folder that exists by default and `nginx` doesn\u0027t have permission to create things at /\n\nIt seems like we should anchor this somewhere that nginx has access to (something in /var?), or see if we can use something like the tmpfiles mechanism for the folder creation instead (as it does it as root and then chowns).\n\nAlso, since this folder contains mutable data which we might want to keep, it should probably be added to the `restic` setup in whitby/default.nix","commit_id":"22facd311c848efa6ba3e48c1c7e7e123d61d615"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"8b6c579def66be58a3df8f4c5b8dd5468bc84929","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    systemd.services.ensure-deploys-dir \u003d {"},{"line_number":10,"context_line":"      description \u003d \"Ensure the html directory for deploys.tvl.fyi exists\";"},{"line_number":11,"context_line":"      wantedBy \u003d [ \"multi-user.target\" ];"},{"line_number":12,"context_line":"      script \u003d \"mkdir -p /srv/www/deploys.tvl.fyi\";"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      serviceConfig \u003d {"},{"line_number":15,"context_line":"        User \u003d \"nginx\";"}],"source_content_type":"text/x-nix","patch_set":6,"id":"7ce0ddc8_82b45d1a","line":12,"range":{"start_line":12,"start_character":25,"end_line":12,"end_character":49},"in_reply_to":"36bac1d7_fd5e9eda","updated":"2021-08-28 11:08:52.000000000","message":"Fine.","commit_id":"22facd311c848efa6ba3e48c1c7e7e123d61d615"},{"author":{"_account_id":1000010,"name":"aspen","email":"root@gws.fyi","username":"aspen"},"change_message_id":"5aef4631fdec12e352f816319223e5c7fe92d6fc","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    systemd.services.ensure-deploys-dir \u003d {"},{"line_number":10,"context_line":"      description \u003d \"Ensure the html directory for deploys.tvl.fyi exists\";"},{"line_number":11,"context_line":"      wantedBy \u003d [ \"multi-user.target\" ];"},{"line_number":12,"context_line":"      script \u003d \"mkdir -p /srv/www/deploys.tvl.fyi\";"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      serviceConfig \u003d {"},{"line_number":15,"context_line":"        User \u003d \"nginx\";"}],"source_content_type":"text/x-nix","patch_set":6,"id":"723155a5_08e7e9d0","line":12,"range":{"start_line":12,"start_character":25,"end_line":12,"end_character":49},"in_reply_to":"7ce0ddc8_82b45d1a","updated":"2021-09-01 22:18:09.000000000","message":"Done","commit_id":"22facd311c848efa6ba3e48c1c7e7e123d61d615"},{"author":{"_account_id":1000010,"name":"aspen","email":"root@gws.fyi","username":"aspen"},"change_message_id":"cba677565d0df903c1ce9f545f9560656cb4c25d","unresolved":true,"context_lines":[{"line_number":9,"context_line":"    systemd.services.ensure-deploys-dir \u003d {"},{"line_number":10,"context_line":"      description \u003d \"Ensure the html directory for deploys.tvl.fyi exists\";"},{"line_number":11,"context_line":"      wantedBy \u003d [ \"multi-user.target\" ];"},{"line_number":12,"context_line":"      script \u003d \"mkdir -p /srv/www/deploys.tvl.fyi\";"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      serviceConfig \u003d {"},{"line_number":15,"context_line":"        User \u003d \"nginx\";"}],"source_content_type":"text/x-nix","patch_set":6,"id":"36bac1d7_fd5e9eda","line":12,"range":{"start_line":12,"start_character":25,"end_line":12,"end_character":49},"in_reply_to":"fc894624_55693155","updated":"2021-08-27 14:27:14.000000000","message":"so, how do you feel about just putting this at `/var/html` for now and calling it a day","commit_id":"22facd311c848efa6ba3e48c1c7e7e123d61d615"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"f1916344be54549bc5798ea99abd57e452a8dd6c","unresolved":true,"context_lines":[{"line_number":12,"context_line":"      script \u003d \"mkdir -p /var/html/deploys.tvl.fyi\";"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      serviceConfig \u003d {"},{"line_number":15,"context_line":"        User \u003d \"nginx\";"},{"line_number":16,"context_line":"        Group \u003d \"nginx\";"},{"line_number":17,"context_line":"        Type \u003d \"oneshot\";"},{"line_number":18,"context_line":"      };"}],"source_content_type":"text/x-nix","patch_set":7,"id":"1e068ee8_0ab0d4a3","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":23},"updated":"2021-09-03 11:54:49.000000000","message":"this will fail, nginx doesn\u0027t have permission to create things in /var\n\nI still think this should use systemd\u0027s tmpfiles mechanism","commit_id":"289cdc83426905cd861a6568fc169bb5c715cd84"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"8cf2ebf566ec0e91378fd763a7cd6a2e66d244ce","unresolved":false,"context_lines":[{"line_number":12,"context_line":"      script \u003d \"mkdir -p /var/html/deploys.tvl.fyi\";"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"      serviceConfig \u003d {"},{"line_number":15,"context_line":"        User \u003d \"nginx\";"},{"line_number":16,"context_line":"        Group \u003d \"nginx\";"},{"line_number":17,"context_line":"        Type \u003d \"oneshot\";"},{"line_number":18,"context_line":"      };"}],"source_content_type":"text/x-nix","patch_set":7,"id":"c02ed7b3_7ddd8a10","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":23},"in_reply_to":"1e068ee8_0ab0d4a3","updated":"2021-09-10 11:47:36.000000000","message":"Done","commit_id":"289cdc83426905cd861a6568fc169bb5c715cd84"}]}