)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"e9447f24372f4ceb64e67005ecc476e53cfbede8","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Imports the current state of the tvl.fyi zone and configures simple CI"},{"line_number":10,"context_line":"checks on the file format."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"No deployment automation exists for this (yet?)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia7d72e02b9f6d3adef994c5dc1898cc0df9dfcfb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c2ae9dd5_bd0c1798","line":12,"updated":"2021-03-14 13:17:53.000000000","message":"Didn\u0027t you have the gcloud interaction already figured out for your dns blog? :p","commit_id":"08333ebc60b68add597b7b8626eb19d3a20575e6"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"984b1aa246ed924f4b5f6a8fd66881d6ed04b77c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Imports the current state of the tvl.fyi zone and configures simple CI"},{"line_number":10,"context_line":"checks on the file format."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"No deployment automation exists for this (yet?)."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia7d72e02b9f6d3adef994c5dc1898cc0df9dfcfb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1c3757f4_3ff87736","line":12,"in_reply_to":"c2ae9dd5_bd0c1798","updated":"2021-03-14 13:21:49.000000000","message":"That wasn\u0027t deployed automatically though - I had a wrapper script for editing posts which un/rechunked them.","commit_id":"08333ebc60b68add597b7b8626eb19d3a20575e6"}],"ops/dns/default.nix":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"e9447f24372f4ceb64e67005ecc476e53cfbede8","unresolved":false,"context_lines":[{"line_number":2,"context_line":"{ pkgs, ... }:"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"let checkZone \u003d zone: file: pkgs.runCommandNoCC \"${zone}-check\" {} \u0027\u0027"},{"line_number":5,"context_line":"  ${pkgs.bind}/bin/named-checkzone -i local ${zone} ${file} | tee $out"},{"line_number":6,"context_line":"\u0027\u0027;"},{"line_number":7,"context_line":"in {"},{"line_number":8,"context_line":"  tvl-fyi \u003d checkZone \"tvl.fyi\" ./tvl.fyi.zone;"}],"source_content_type":"text/x-nix","patch_set":1,"id":"cbf8e6af_9adf9adc","line":5,"updated":"2021-03-14 13:17:53.000000000","message":"Don\u0027t know too much about bind or DNS — from skimming the man page it seems we might want to use named-compilezone for deployment purposes or stricter checking maybe? We would lose the output probably and might  not be necessary for gcloud.","commit_id":"08333ebc60b68add597b7b8626eb19d3a20575e6"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"984b1aa246ed924f4b5f6a8fd66881d6ed04b77c","unresolved":false,"context_lines":[{"line_number":2,"context_line":"{ pkgs, ... }:"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"let checkZone \u003d zone: file: pkgs.runCommandNoCC \"${zone}-check\" {} \u0027\u0027"},{"line_number":5,"context_line":"  ${pkgs.bind}/bin/named-checkzone -i local ${zone} ${file} | tee $out"},{"line_number":6,"context_line":"\u0027\u0027;"},{"line_number":7,"context_line":"in {"},{"line_number":8,"context_line":"  tvl-fyi \u003d checkZone \"tvl.fyi\" ./tvl.fyi.zone;"}],"source_content_type":"text/x-nix","patch_set":1,"id":"31894ea5_38dcda59","line":5,"in_reply_to":"cbf8e6af_9adf9adc","updated":"2021-03-14 13:21:49.000000000","message":"`gcloud dns` consumes the actual zone file format (which is checked in here), not the compiled representation used for bind.\n\nThe thing here is that named-checkzone is a tool from the bind server utilities, but our actual server isn\u0027t bind so we\u0027re only using it for validating the file.","commit_id":"08333ebc60b68add597b7b8626eb19d3a20575e6"},{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"e9447f24372f4ceb64e67005ecc476e53cfbede8","unresolved":true,"context_lines":[{"line_number":6,"context_line":"\u0027\u0027;"},{"line_number":7,"context_line":"in {"},{"line_number":8,"context_line":"  tvl-fyi \u003d checkZone \"tvl.fyi\" ./tvl.fyi.zone;"},{"line_number":9,"context_line":"  meta.targets \u003d [ \"tvl-fyi\" ];"},{"line_number":10,"context_line":"}"}],"source_content_type":"text/x-nix","patch_set":1,"id":"c9169a21_66b87d93","line":9,"updated":"2021-03-14 13:17:53.000000000","message":"we could of course automatically generate the targets for when we have more zones:\n\n```\nlet\n  …\n\n  zones \u003d {\n    tvl-fyi \u003d …\n  };\n\nin zones // {\n  meta.targets \u003d builtins.attrNames zones;\n}\n```","commit_id":"08333ebc60b68add597b7b8626eb19d3a20575e6"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"3af0362d09a881b2d657fa3b976218366f046337","unresolved":false,"context_lines":[{"line_number":6,"context_line":"\u0027\u0027;"},{"line_number":7,"context_line":"in {"},{"line_number":8,"context_line":"  tvl-fyi \u003d checkZone \"tvl.fyi\" ./tvl.fyi.zone;"},{"line_number":9,"context_line":"  meta.targets \u003d [ \"tvl-fyi\" ];"},{"line_number":10,"context_line":"}"}],"source_content_type":"text/x-nix","patch_set":1,"id":"03082bfa_0244c67b","line":9,"in_reply_to":"c9169a21_66b87d93","updated":"2021-03-14 13:22:05.000000000","message":"Done","commit_id":"08333ebc60b68add597b7b8626eb19d3a20575e6"}]}