)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"54ae51857aa56dd6e82392a252723aad9974b73b","unresolved":true,"context_lines":[{"line_number":25,"context_line":"After that it just loops."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"There is no rebasing logic yet for when it \"runs out\" of submittable"},{"line_number":28,"context_line":"changes, but it will not be difficult to add."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Relates to b/333."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1617c2a0_74869e21","line":28,"updated":"2023-11-27 14:06:54.000000000","message":"This does mean that it can (also) break canon, though.","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"cd2a351a9bc399f6f88f498dbc82d11c81e1baa8","unresolved":false,"context_lines":[{"line_number":25,"context_line":"After that it just loops."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"There is no rebasing logic yet for when it \"runs out\" of submittable"},{"line_number":28,"context_line":"changes, but it will not be difficult to add."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Relates to b/333."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1d0c5492_e5cc04e8","line":28,"in_reply_to":"1617c2a0_74869e21","updated":"2023-11-27 15:51:17.000000000","message":"It submits everything Gerrit considers submittable, as clbot also tried to do. The rebasing is a separate step afterwards. This isn\u0027t quite the same as a merge linearisation bot (a la bors), it\u0027s a replacement for gerrit-queue that does the same job but actually works.","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"3f3a9aea274a33fe9bcfcd8bfbbce2c7f0f114e8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b9e1141e_f91b6364","updated":"2023-11-27 14:04:26.000000000","message":"Some of the background information given in the commit message is necessary to understand some of the details, so it\u0027d be nice to have them in a comment as well.","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"cd2a351a9bc399f6f88f498dbc82d11c81e1baa8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82089c33_34c8cff7","in_reply_to":"b9e1141e_f91b6364","updated":"2023-11-27 15:51:17.000000000","message":"Done","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"}],"ops/gerrit-autosubmit/default.nix":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"3f3a9aea274a33fe9bcfcd8bfbbce2c7f0f114e8","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  src \u003d ./.;"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  buildInputs \u003d with pkgs; ["},{"line_number":7,"context_line":"    pkg-config"},{"line_number":8,"context_line":"    openssl"},{"line_number":9,"context_line":"  ];"},{"line_number":10,"context_line":"}"}],"source_content_type":"text/x-nix","patch_set":1,"id":"e581fdf0_157e38ab","line":7,"range":{"start_line":7,"start_character":4,"end_line":7,"end_character":14},"updated":"2023-11-27 14:04:26.000000000","message":"nativeBuildInputs \u003d [\n  pkgs.buildPackages.pkg-config\n]","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"cd2a351a9bc399f6f88f498dbc82d11c81e1baa8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  src \u003d ./.;"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"  buildInputs \u003d with pkgs; ["},{"line_number":7,"context_line":"    pkg-config"},{"line_number":8,"context_line":"    openssl"},{"line_number":9,"context_line":"  ];"},{"line_number":10,"context_line":"}"}],"source_content_type":"text/x-nix","patch_set":1,"id":"3235793d_84246e81","line":7,"range":{"start_line":7,"start_character":4,"end_line":7,"end_character":14},"in_reply_to":"e581fdf0_157e38ab","updated":"2023-11-27 15:51:17.000000000","message":"Done","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"}],"ops/gerrit-autosubmit/src/main.rs":[{"author":{"_account_id":1000034,"name":"sterni","email":"sternenseemann@systemli.org","username":"sterni"},"change_message_id":"3f3a9aea274a33fe9bcfcd8bfbbce2c7f0f114e8","unresolved":true,"context_lines":[{"line_number":25,"context_line":"                    .context(\"Gerrit base URL (no trailing slash) must be set in GERRIT_URL\")?,"},{"line_number":26,"context_line":"                username: env::var(\"GERRIT_USERNAME\")"},{"line_number":27,"context_line":"                    .context(\"Gerrit username must be set in GERRIT_USERNAME\")?,"},{"line_number":28,"context_line":"                password: env::var(\"GERRIT_PASSWORD\")"},{"line_number":29,"context_line":"                    .context(\"Gerrit password must be set in GERRIT_PASSWORD\")?,"},{"line_number":30,"context_line":"            })"},{"line_number":31,"context_line":"        }"}],"source_content_type":"text/x-rustsrc","patch_set":1,"id":"92e1cb23_407b81b0","line":28,"range":{"start_line":28,"start_character":36,"end_line":28,"end_character":51},"updated":"2023-11-27 14:04:26.000000000","message":"Wouldn\u0027t using a file be nicer w.r.t. systemd?","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"cd2a351a9bc399f6f88f498dbc82d11c81e1baa8","unresolved":false,"context_lines":[{"line_number":25,"context_line":"                    .context(\"Gerrit base URL (no trailing slash) must be set in GERRIT_URL\")?,"},{"line_number":26,"context_line":"                username: env::var(\"GERRIT_USERNAME\")"},{"line_number":27,"context_line":"                    .context(\"Gerrit username must be set in GERRIT_USERNAME\")?,"},{"line_number":28,"context_line":"                password: env::var(\"GERRIT_PASSWORD\")"},{"line_number":29,"context_line":"                    .context(\"Gerrit password must be set in GERRIT_PASSWORD\")?,"},{"line_number":30,"context_line":"            })"},{"line_number":31,"context_line":"        }"}],"source_content_type":"text/x-rustsrc","patch_set":1,"id":"aa6a04e0_3ce001d7","line":28,"range":{"start_line":28,"start_character":36,"end_line":28,"end_character":51},"in_reply_to":"92e1cb23_407b81b0","updated":"2023-11-27 15:51:17.000000000","message":"No, our secret setup works well with EnvironmentFile, and in this case it means I can just directly use the renamed file from gerrit-queue :)","commit_id":"45ca5e5218fb4780453ecb48a34d0d922275a205"},{"author":{"_account_id":1000036,"name":"flokli","email":"flokli@flokli.de","username":"flokli"},"change_message_id":"b78e68dbf4d9d45da23f1433ae35fa294bbe219f","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    use std::collections::HashMap;"},{"line_number":29,"context_line":"    use std::env;"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    pub struct Config {"},{"line_number":32,"context_line":"        gerrit_url: String,"},{"line_number":33,"context_line":"        username: String,"},{"line_number":34,"context_line":"        password: String,"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"5d767ef5_8683c8ae","line":31,"updated":"2023-11-27 21:21:39.000000000","message":"I personally prefer clap in Derive - it gives nicer help, rather than trying to execute and run from one panic to another.","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"1e9e20f74ba8074fcfe7e37757dbbde863f19ac8","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    use std::collections::HashMap;"},{"line_number":29,"context_line":"    use std::env;"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    pub struct Config {"},{"line_number":32,"context_line":"        gerrit_url: String,"},{"line_number":33,"context_line":"        username: String,"},{"line_number":34,"context_line":"        password: String,"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"6a0c231a_0b531baa","line":31,"in_reply_to":"5d767ef5_8683c8ae","updated":"2023-11-27 21:40:00.000000000","message":"I don\u0027t want to drag in an absolutely humongous library just to fetch a few envvars. I think this is a sufficiently simple program that it doesn\u0027t need complicated configuration handling.","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"},{"author":{"_account_id":1000036,"name":"flokli","email":"flokli@flokli.de","username":"flokli"},"change_message_id":"b78e68dbf4d9d45da23f1433ae35fa294bbe219f","unresolved":true,"context_lines":[{"line_number":41,"context_line":"                    .context(\"Gerrit base URL (no trailing slash) must be set in GERRIT_URL\")?,"},{"line_number":42,"context_line":"                username: env::var(\"GERRIT_USERNAME\")"},{"line_number":43,"context_line":"                    .context(\"Gerrit username must be set in GERRIT_USERNAME\")?,"},{"line_number":44,"context_line":"                password: env::var(\"GERRIT_PASSWORD\")"},{"line_number":45,"context_line":"                    .context(\"Gerrit password must be set in GERRIT_PASSWORD\")?,"},{"line_number":46,"context_line":"            })"},{"line_number":47,"context_line":"        }"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"3aa4753b_1dc66ce7","line":44,"updated":"2023-11-27 21:21:39.000000000","message":"Can this be GERRIT_PASSWORD_FILE? That\u0027ll allow keeping all of the config visible, and the password to not live in the env.","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"142391ea14082235ff7ce297e618066a5db9735a","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                    .context(\"Gerrit base URL (no trailing slash) must be set in GERRIT_URL\")?,"},{"line_number":42,"context_line":"                username: env::var(\"GERRIT_USERNAME\")"},{"line_number":43,"context_line":"                    .context(\"Gerrit username must be set in GERRIT_USERNAME\")?,"},{"line_number":44,"context_line":"                password: env::var(\"GERRIT_PASSWORD\")"},{"line_number":45,"context_line":"                    .context(\"Gerrit password must be set in GERRIT_PASSWORD\")?,"},{"line_number":46,"context_line":"            })"},{"line_number":47,"context_line":"        }"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"b00a719c_18ac79d3","line":44,"in_reply_to":"04a7c10e_4910ef77","updated":"2023-11-27 21:50:23.000000000","message":"Opened b/334, I don\u0027t wanna deal with redoing the secret right now, can refactor later.","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"1e9e20f74ba8074fcfe7e37757dbbde863f19ac8","unresolved":true,"context_lines":[{"line_number":41,"context_line":"                    .context(\"Gerrit base URL (no trailing slash) must be set in GERRIT_URL\")?,"},{"line_number":42,"context_line":"                username: env::var(\"GERRIT_USERNAME\")"},{"line_number":43,"context_line":"                    .context(\"Gerrit username must be set in GERRIT_USERNAME\")?,"},{"line_number":44,"context_line":"                password: env::var(\"GERRIT_PASSWORD\")"},{"line_number":45,"context_line":"                    .context(\"Gerrit password must be set in GERRIT_PASSWORD\")?,"},{"line_number":46,"context_line":"            })"},{"line_number":47,"context_line":"        }"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"04a7c10e_4910ef77","line":44,"in_reply_to":"3aa4753b_1dc66ce7","updated":"2023-11-27 21:40:00.000000000","message":"I decided to copy the configuration structure of gerrit-queue, so that the secrets and module we have don\u0027t need to change much. I\u0027d rather keep it that way until this is submitted and whitby is canonicalised, the config locations and envvar naming etc. is not hard to change later.","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"},{"author":{"_account_id":1000036,"name":"flokli","email":"flokli@flokli.de","username":"flokli"},"change_message_id":"b78e68dbf4d9d45da23f1433ae35fa294bbe219f","unresolved":true,"context_lines":[{"line_number":95,"context_line":"fn list_submittable(cfg: \u0026gerrit::Config) -\u003e Result\u003cVec\u003cSubmittableChange\u003e\u003e {"},{"line_number":96,"context_line":"    let mut out \u003d Vec::new();"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    let result: Vec\u003cgerrit::ChangeInfo\u003e \u003d gerrit::get("},{"line_number":99,"context_line":"        \u0026cfg,"},{"line_number":100,"context_line":"        \"/changes/?q\u003dis:submittable+label:Autosubmit+-is:wip+is:open\u0026o\u003dSKIP_DIFFSTAT\u0026o\u003dCURRENT_REVISION\","},{"line_number":101,"context_line":"    )"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"e451e910_bb0aa344","line":98,"updated":"2023-11-27 21:21:39.000000000","message":"changes","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"},{"author":{"_account_id":1000001,"name":"tazjin","email":"tazjin@tvl.su","username":"tazjin"},"change_message_id":"1e9e20f74ba8074fcfe7e37757dbbde863f19ac8","unresolved":false,"context_lines":[{"line_number":95,"context_line":"fn list_submittable(cfg: \u0026gerrit::Config) -\u003e Result\u003cVec\u003cSubmittableChange\u003e\u003e {"},{"line_number":96,"context_line":"    let mut out \u003d Vec::new();"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"    let result: Vec\u003cgerrit::ChangeInfo\u003e \u003d gerrit::get("},{"line_number":99,"context_line":"        \u0026cfg,"},{"line_number":100,"context_line":"        \"/changes/?q\u003dis:submittable+label:Autosubmit+-is:wip+is:open\u0026o\u003dSKIP_DIFFSTAT\u0026o\u003dCURRENT_REVISION\","},{"line_number":101,"context_line":"    )"}],"source_content_type":"text/x-rustsrc","patch_set":2,"id":"6c1a8d70_38ce2e2f","line":98,"in_reply_to":"e451e910_bb0aa344","updated":"2023-11-27 21:40:00.000000000","message":"Done","commit_id":"a6a7042c6493f592b9c736211d01729ec5cdb089"}]}