about summary refs log tree commit diff
AgeCommit message (Collapse)AuthorFilesLines
2020-11-22 r/1902 fix(users/multi/home): re-add overridden htop package to home-manager ↵multi1-0/+2
configuration. I previously removed my local package set from my HM config while the latter was being made readTree compatible. Now that both the HM config and the local package set can be built with readTree, I can re-enable the locally-overridden htop package. Change-Id: I77e20248c010bc7027e0b0a3164ec48d6ec29f31 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2132 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz>
2020-11-22 r/1901 refactor(user/multi/pkgs): make my local package set readTree-compatible.multi2-4/+11
This adds readTree configuration for accessing my local package set, and also adds these packages to the CI configuration. Change-Id: Icd2d16e85859343902e73a466f3c6ba8d781537f Reviewed-on: https://cl.tvl.fyi/c/depot/+/2131 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz>
2020-11-22 r/1900 fix(users/multi/home): use local depot checkout.multi1-2/+6
This reverts 1478317d149539d74fa4bad8414658fb7119ea07. Using depot.depotPath in my home-manager configuration results in my NIX_PATH and home-manager config file path being pointed at a copy of the depot in the nix store, which makes building from my local depot checkout a bit cumbersome. Change-Id: Ib687d3e8147cb32df071d3c19a5300294ea62c0c Reviewed-on: https://cl.tvl.fyi/c/depot/+/2130 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz>
2020-11-22 r/1899 refactor(users/multi): migrate whitby home-manager config to ↵multi3-69/+11
readTree-compatible version. Add a new home-manager-compatible configuration file which loads the common config attrset used by the readTree machinery into a structure which the home-manager command line tool understands. Garbage-collect the old home-manager configuration file used on whitby, and update the HOME_MANAGER_CONFIG path to point at the new shim config. Instead of having a per-environment HM configuration (not that I have more than one environment), there's now a single configuration which evaluates to an attrset of configurations, which can be loaded and built using "home-manager build -A $attr". Change-Id: Id8b35dc89aabffedf1a4dadfa0d3d4b914e4e2e7 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2129 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz>
2020-11-22 r/1898 refactor(users/multi): make my user directory CI-compatible.multi3-0/+93
My home-manager config is not currently readTree compatible, which means that it's not built by CI. This constructs a house of cards around home-manager to make this buildable in CI. Change-Id: I80480f24ff47347f46d708edbbf34d59fa76adac Reviewed-on: https://cl.tvl.fyi/c/depot/+/2123 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz>
2020-11-22 r/1897 refactor(users/multi/whitby): use the depot's own source location attribute.multi2-9/+2
The depot knows where it is, not because it knows where it isn't, but because it does an "import ./." at the top level and then makes this path value available in the attrset passed to the rest of the tree. My home-manager config on whitby previously involved manual specification of the depot checkout location on whitby, however this isn't necessary when the depot can already magically tell us where it is. Change-Id: I68577c8ef2cda6ba5bc46cf8f4821aac021c8066 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2122 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz>
2020-11-22 r/1896 feat(panettone): Bring back + fix irccat issue creation announcementGriffin Smith4-6/+43
This reverts commit e1067b1497b3dea0c37ae51bba21f42f1e7d35b5. The original issue here was misusing ISSUE-ID instead of ID, but also the associated username for the message should've been CN instead of DN Change-Id: I1629c0cb7597ff2ee2867f27870378eecdafe126 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2125 Tested-by: BuildkiteCI Reviewed-by: eta <eta@theta.eu.org>
2020-11-21 r/1895 fix(users/multi/whitby): fix home-manager configuration on whitby.multi1-1/+1
"let pkgs = import <nixpkgs> {}; in pkgs.home-manager.src" evaluates to the source derivation for home-manager, however home-manager's configuration machinery expects to be passed the store path of this derivation instead of the derivation object itself. Change-Id: I6b0ba3efaff9d080900349529576443192b058e9 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2121 Reviewed-by: multi <depot@in-addr.xyz> Tested-by: BuildkiteCI
2020-11-21 r/1894 chore(3p): Bump nixpkgs to nixos-unstable from 2020-11-21Vincent Ambo17-91/+71
Included fixes for random breakage: * 3p/awscli: pick from the stable channel; it is broken on unstable * 3p/googletest: bumped version & removed patches that nixpkgs applies * 3p/lisp/cffi: bumped library version for SBCL compat * 3p/nix: fix libsystemd attribute * 3p/nix: reformatted (clang-format handling of ternaries changed) * glittershark/home: Use home-manager from nixkpgs * glittershark/kernel: bumped linux-ck patch hash * glittershark/kernel: removed "patch patch" * multi/whitby: Use home-manager from nixpkgs * tazjin/frog: drop Sourcetrail (it doesn't build currently) Note that in addition to these changes, some previous CLs updated the versions of git and cgit which was necessary for this channel bump, but which could not be done in the same commit due to the nature of the subtree merges. Change-Id: If2563e8a68e2750c4b913a976ff7b93b42e8b7f3 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2110 Tested-by: BuildkiteCI Reviewed-by: multi <depot@in-addr.xyz> Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-21 r/1893 fix(glittershark/kernel): Ensure kernel changes cachebust linux-ckVincent Ambo1-0/+1
Previously changed kernel versions would not cachebust the patch download, because it would still be using the same SHA hash. Forcing a different store path (by adding the version to the name) also forces a redownload of the patch (and in turn cause the hash to mismatch), avoiding this as a silent cause of failures in channel updates. Change-Id: I81a136ee2401126795cf042b0aadf2a1e7a707b4 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2114 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-21 r/1892 fix(3p/cgit): Rename remaining 'sha1' references to 'oid'Vincent Ambo1-2/+2
This changed upstream & upstream-upstream, but was still used in our patches. Change-Id: Id33f868a4f188deb3348425276bc5b4c4b36aff9
2020-11-21 r/1891 merge(3p/cgit): subtree merge at adcc4f82Vincent Ambo26-135/+299
Change-Id: I16941629a17a2f39b9324cb85dc0cec6b104dfe4
2020-11-21 r/1890 merge(3p/git): Merge git subtree at v2.29.2Vincent Ambo1485-100645/+232762
This also bumps the stable nixpkgs to 20.09 as of 2020-11-21, because there is some breakage in the git build related to the netrc credentials helper which someone has taken care of in nixpkgs. The stable channel is not used for anything other than git, so this should be fine. Change-Id: I3575a19dab09e1e9556cf8231d717de9890484fb
2020-11-21 r/1889 merge(3p/absl): subtree merge of Abseil up to e19260fVincent Ambo854-5218/+11182
... notably, this includes Abseil's own StatusOr type, which conflicted with our implementation (that was taken from TensorFlow). Change-Id: Ie7d6764b64055caaeb8dc7b6b9d066291e6b538f
2020-11-21 r/1888 chore(tvix): Use StatusOr API available in Abseil's versionVincent Ambo4-8/+13
The Abseil version of `StatusOr` does not come with the status macros or the `Consume*` family of functions. This change modifies the existing code to use the common denominator of the API that is available between Abseil's own implementation of `StatusOr` and the one from Tensorflow that we are currently using. Change-Id: I5c37f68636a1fd54d153f95d7303ab8644abb774
2020-11-21 r/1887 chore(tazjin/emacs): Minor cleanup of unused modes and configVincent Ambo4-6/+0
Change-Id: I38c6a32339bf7606b733a938b4e55df2506a13a7 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2107 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2020-11-19 r/1886 fix(tvix): Use copy constructor to add strings to protosGriffin Smith2-1/+29
Passing a string directly to add_paths like this causes the proto class to take ownership over the string, meaning when it is destructed it will *explicitly* free the string. When the string's actual owner (the derivation struct) then goes out of scope it'll get freed again, causing a double-free. This fixes that to instead use the copy constructor to assign to a pointer to a new path, and covers the whole to_proto method with a rapidcheck test. Fixes: b/64 Change-Id: I84235bed9104ff430a0acf686d4a96f1e2e9a897 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2106 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2020-11-19 r/1885 fix(tvix): Use mutable field for assigning proto fieldsGriffin Smith1-2/+2
This was accidentally using the proto arena API to assign the derivation field of a BuildDerivationRequest. We *thought* this was causing a double free, but even with this change that's still happening. That said, this change is probably still a good idea since it's using the proto API as intended. References: b/64 Change-Id: I950a4eafb214e9113639ea54d2dfd4659b7be931 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2104 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2020-11-19 r/1884 revert(panettone): announce newly created issues using irccatglittershark4-38/+4
This reverts commit 2e2bdf9c6ce1cd66ba5cfe1a42786a6f486b7969. Reason for revert: this is not working, and is resulting in newly created issues just showing a blank page (b/74) Change-Id: I3f06afc52d6c5289269402fc75bb32ad9c376bf4 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2082 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2020-11-18 r/1883 chore(users/multi): Update htop derivation for new upstream.multi13-3609/+10
- htop has moved upstreams, which has been producing new releases, so update the derivation to pull from the new repository on GitHub. - All of the patches I have locally have been merged upstream, so drop them from the depot. - Pull from a reasonably recent git commit instead of from a numbered release, as the ZFS ARC stats and CPU meter columnation patches haven't made it into a release yet. Change-Id: I66ad4c035df07709abf4f75a9d4e1486920091d0 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2105 Reviewed-by: multi <depot@in-addr.xyz> Tested-by: BuildkiteCI
2020-11-17 r/1882 feat(ops/pipelines): Check in the static pipelineVincent Ambo1-0/+15
This file represents the static pipeline which is configured in the Buildkite web UI. Updates to this file should be applied in the admin interface. These steps are responsible for launching the dynamic pipeline evaluation, or falling back to the fallback pipeline if evaluation fails. Change-Id: I6d7dd623cde65e8c69faea729f737c9bba00c2fb Reviewed-on: https://cl.tvl.fyi/c/depot/+/2103 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-17 r/1881 feat(ops/pipelines): Add a fallback Buildkite configurationVincent Ambo1-0/+8
This adds a simple fallback Buildkite pipeline configuration which always fails the pipeline, but correctly reports back the failure status. Note that this also requires changes in the Buildkite configuration that is not in version-control. Relates to b/66. Change-Id: I6802a6f76448c3893798a06d514e6ccba0f50dd2 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2102 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-17 r/1880 feat(ops/panettone): Add configuration for irccatVincent Ambo2-0/+21
Adds configuration options for the (inconsistently named) environment variables that configure irccat integration with Panettone. The defaults match the irccat setup on whitby. Change-Id: I6857512a2e3f29f16777493eb981cc69ce3c045f Reviewed-on: https://cl.tvl.fyi/c/depot/+/2080 Tested-by: BuildkiteCI Reviewed-by: kanepyork <rikingcoding@gmail.com>
2020-11-16 r/1879 feat(glittershark): Add resume + deploy it to gws.fyiGriffin Smith24-0/+2503
Change-Id: If9cf64c97621313af86074752d0959506059888a Reviewed-on: https://cl.tvl.fyi/c/depot/+/2079 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-11-16 r/1878 feat(gs/emacs): Add some config for vtermGriffin Smith3-5/+34
for some reason installing it directly via nix doesn't work atm, so I have this hack here Change-Id: I45093633c35e756988078eb136c6e7bc3c532eea Reviewed-on: https://cl.tvl.fyi/c/depot/+/2078 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-11-08 r/1877 feat(panettone): announce newly created issues using irccateta4-4/+38
- The new PANETTONE.IRC package contains the SEND-IRC-NOTIFICATION function, which opens a new TCP socket to irccat (if it's running and configured) in order to announce the creation of new issues. - The IRCCATHOST and IRCCATPORT environment variables must be set for this to work. - Additionally, the ISSUECHANNEL environment variable may be used to direct announcements at a given channel (otherwise it'll just use the first one). Change-Id: I429a66f24d0f80ed10db173d6af7105fb1d3d023 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2077 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-08 r/1876 feat(whitby): Enable irccat moduleVincent Ambo1-0/+19
Enables irccat, running as 'tvlbot' on ##tvl and ##tvl-dev and listening on TCP 4722. Change-Id: Ia1eb533d0aacb0c15d6b3fa1cfd854ffbce27d23 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2075 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi> Reviewed-by: lukegb <lukegb@tvl.fyi>
2020-11-08 r/1875 feat(ops/irccat): Add a NixOS module for launching irccatVincent Ambo1-0/+49
This module configures irccat by creating a JSON configuration file from a user-supplied Nix struct (this is not checked for correctness), and merging it recursively with secrets from `/etc/secrets/irccat.json` at service launch time. This way we get the ability to configure (most) options declaratively via Nix, while providing the secrets outside of Nix. Side note: We need to figure out a secrets distribution mechanism. Tested: Wrote a dummy config in whitby/default.nix locally and checked that this builds, but I have not actually run the service yet. I expect that some minor tweaks will end up being necessary. Change-Id: I02a2e8dc40a7f8417fd77afcf8a12ac3df117988 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2074 Tested-by: BuildkiteCI Reviewed-by: lukegb <lukegb@tvl.fyi> Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-08 r/1874 feat(3p/irccat): init at '06a985y4'Vincent Ambo1-0/+16
This is to be used for forwarding messages to our IRC channels. Change-Id: I6362c6f50a22f504588a7161d41a170f4e7a6edc Reviewed-on: https://cl.tvl.fyi/c/depot/+/2073 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi> Reviewed-by: lukegb <lukegb@tvl.fyi>
2020-11-08 r/1873 fix(buildLisp): Override LANG in sbclWith for misconfigured machinesVincent Ambo1-1/+2
Machines on which LANG is misconfigured have trouble with SBCL loading files that contain characters in certain encodings. This overrides whichever local LANG (if any) is set. Change-Id: Ic4341a01c4393e7f697de6cecc58dea4f2d85987 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2076 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-08 r/1872 chore(3p/exwm): Update derivation to build local sourcesVincent Ambo2-13/+7
... this should also update my system EXWM. Change-Id: Idfbbda67613ac678dc2d5f82533e1c6176ab4a28 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2072 Tested-by: BuildkiteCI Reviewed-by: glittershark <grfn@gws.fyi>
2020-11-08 r/1871 chore(3p/emacs/exwm): Import from commit ↵Vincent Ambo16-0/+8567
'0368127976bda29d35eed788edfe74644ecd3845' Imported from https://github.com/ch11ng/exwm/commit/0368127976bda29d35eed788edfe74644ecd3845 git-subtree-dir: third_party/emacs/exwm git-subtree-mainline: e84f9ef0adb48db610fd06d6ff75edd073712a90 git-subtree-split: 0368127976bda29d35eed788edfe74644ecd3845 Change-Id: Id3af5610254180f42947d71265aad89def7c6a3d
2020-11-05 r/1870 fix(whitby): Use new IRC bouncer location for clbotVincent Ambo1-1/+1
... I found this location in the logs, because the certs are now valid for this, but I'm not actually sure if it's right. Change-Id: I5ac88073e3bf6a95fead4c1d34515622c4416c6a Reviewed-on: https://cl.tvl.fyi/c/depot/+/2070 Tested-by: BuildkiteCI Reviewed-by: lukegb <lukegb@tvl.fyi>
2020-11-05 r/1869 feat(ops/nixos/paroxysm): Set Restart = "always"Griffin Smith1-0/+1
Sometimes (like today) paroxysm crashes. We'd like it to restart if that happens. Change-Id: I98841096bcd6605c4279744ae5c65a9c92092a21 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2069 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2020-11-04 r/1868 feat(gs/emacs): Disable truncate-lines in org modeGriffin Smith2-6/+13
Change-Id: Ie1a539d894ae6223196f9e5b912317a00716ba72 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2067 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-11-04 r/1867 fix(tazjin/emacs): Do not assume '.txt' files are MarkdownVincent Ambo1-1/+0
Change-Id: I5cab039938e415facf81492e7e3775672ee45db9 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2068 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2020-10-27 r/1866 feat(web/tvl): graph implrBartosz Stebel1-2/+2
Considered adding hswaw, but after q3k left it'd be a single edge, so perhaps no point. Change-Id: Ifd8609a5227e5c3bee1d5726bb5cf70ebb2cefdf Reviewed-on: https://cl.tvl.fyi/c/depot/+/2053 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2020-10-27 r/1865 refactor(gs/system): Prefer dobharchu over cerberusGriffin Smith1-18/+16
This machine is significantly faster. Also, drop nixbuild, since the transfer speed is too slow to make it worth it. Change-Id: Ic14ef96e03a81dc429e4b4fec961c891dbb4b2b9 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2066 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-27 r/1864 feat(gs/home): Add ssh host alias for cerberusGriffin Smith1-0/+7
My old macbook, which I still use sometimes Change-Id: I3ad080083b67b6cfab4cd31a4dce0a80a7227bd5 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2065 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-27 r/1863 feat(gs/emacs): Add bindings for smerge-keep-{lower,upper}Griffin Smith1-1/+3
Change-Id: I83a251b551e0d6c9ca6994b8f9111ffee33653fc Reviewed-on: https://cl.tvl.fyi/c/depot/+/2064 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-27 r/1862 feat(gs/emacs): Include repo in PR linksGriffin Smith1-1/+13
When posting a link to a PR created during an org headline, include the name of the repository the PR was opened to in the link text. Change-Id: I6c564aee3b098d3c6f96c7d7d609aa2638bc98e1 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2063 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-27 r/1861 fix(web/tvl): Fix TVL graphV1-1/+1
Change-Id: I6ce3be85a0248841837d18fdd4d251729061bfc8 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2062 Reviewed-by: lukegb <lukegb@tvl.fyi> Reviewed-by: firefly <firefly@firefly.nu> Tested-by: BuildkiteCI
2020-10-23 r/1860 fix(gs/mugwump): Add missing scheme to grafana datasourceGriffin Smith1-1/+1
Change-Id: I954fcca422f2e1325c2455cb1c4d77d53673901f Reviewed-on: https://cl.tvl.fyi/c/depot/+/2061 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-23 r/1859 feat(gs/mugwump): Only run nix GC monthlyGriffin Smith1-0/+2
Change-Id: I3b81fe5a76c26e42fb6d2937ce980e12964d70b9 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2060 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-23 r/1858 refactor(gs/mugwump): Only one buildkite builderGriffin Smith1-1/+2
There's just not enough juice in this machine to run more than one. Change-Id: I6e6afc86337ca023e718023e4789fc29b6d8e175 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2059 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-23 r/1857 feat(gs/system): Only enable remote builders on chupacabraGriffin Smith2-25/+25
Don't enable whitby+nixbuild as remote builders on every machine (eg not mugwump), only chupacabra Change-Id: I8aa8f20d76da4ec0d8caa64ef04697b7e76cbc03 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2058 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-23 r/1856 feat(gs/home): Add script to rebuild mugwumpGriffin Smith1-0/+10
Eventually this should be generalized, but for now this is good enough + nice to have Change-Id: Icca815b651cfb6f8f0cd2d6a1f64e56c63d2fef5 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2057 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-23 r/1855 feat(gs/emacs): Enable luaGriffin Smith1-1/+1
Change-Id: Iff1eed338323a49de504a02f870e17eeedde57d3 Reviewed-on: https://cl.tvl.fyi/c/depot/+/2056 Reviewed-by: glittershark <grfn@gws.fyi> Tested-by: BuildkiteCI
2020-10-22 r/1854 feat(tvl-slapd): add etu to slapdElis Hirwing1-0/+5
Change-Id: I39ecf2167fd65f305853bf0e48c6208d94a5bf1f Reviewed-on: https://cl.tvl.fyi/c/depot/+/2055 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2020-10-20 r/1853 fix(tvl-ebooks): Update code to bring up to date with realityBen Cartwright-Cox1-16/+66
Fixes include: 1) users can now opt out of being mkov'd, instead their messages will be ascii blocked out. 2) highlights are avoided, by learning names from the main tvl channel and adding a dot after the first char, for example: > 21:31:35 <•eta-eb> m.ulti: bas1l we quickly connect controller is mostly agreed 3) highlight avoidance is now stored in redis, to avoid restarts from destroying the map and causing a bunch of highlights upon restart Change-Id: I1055992aab3a06aa1f4ba937fc3ef45f2f78cedc Reviewed-on: https://cl.tvl.fyi/c/depot/+/2054 Tested-by: BuildkiteCI Reviewed-by: cynthia <cynthia@tvl.fyi> Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: ben <tvl@benjojo.co.uk>