<{^_^}>
ofborg/infrastructure#20 (by cole-h, 11 seconds ago, open): No more warnings
<cole-h>
Since `phpfpm.pools.main.listen` gets removed in this, some extra caution is warranted (e.g. the secrets/encrypted stuff should be checked for references to that as well)
<cole-h>
Also, if we were relying on the silent concatenation of `types.string`
<LnL>
hopefully that'll make it easier to find next time
<cole-h>
"Task Assignment based on Guessing Size" interesting
* cole-h
files it away with the growing pile of academic papers to read eventually
<LnL>
so some context for the stdenv check
<LnL>
the first two steps are just to get the store path of the stdenv (for the current pr)
<LnL>
a bit awkward but there's not really a nice way with the nix-* commands
<LnL>
then the last queries that store path, but instead of on the local machine it asks the cache
<LnL>
--size is just because that's one the simplest query commands exposed
<cole-h>
Huh
<LnL>
eg. nix path-info /nix/store/8z47adw4zaw9q8vybw8r1wp2rg9r0r7r-hello-2.10 most likely fails with path is not valid since you don't have that in your local store
<cole-h>
So first command gets you /nix/store/stdenv.drv, second command gets you /nix/store/stdenv, and third command checks if that stdenv if cached? Did I get the gist of it?
<LnL>
but nix path-info /nix/store/8z47adw4zaw9q8vybw8r1wp2rg9r0r7r-hello-2.10 --store https://cache.nixos.org will return because it's a valid path in the cache
<LnL>
yep
<cole-h>
I think that should be documented in-line, if possible :^)
<cole-h>
Or as a doc comment for the function
<LnL>
yeah, good point
<LnL>
with the new commands this would be so much nicer: nix path-info -f. hello --store https://cache.nixos.org
<cole-h>
Hopefully they stabilize soon-ish :^)
<LnL>
it's like using git-hash-object & friends to make a commit :p
<cole-h>
lol :D
<LnL>
anyway, trying it out so hopefully I won't wake up to a giant gueue again
<cole-h>
That thing was monstrous
<cole-h>
Looked at the dash this morning, saw it reached about -60 at one point :D
<LnL>
yeah and unlike before they won't magically disappear with a deploy anymore
<cole-h>
Looks like those 2 tests fail because it skips the stuff that doesn't exist in the cache lmao
<cole-h>
and it doesn't output what the test wants
<LnL>
yeah needs a bit more work
<cole-h>
I always get bamboozled in my ofborg checkout. I try to `git checkout master`, which checks out master from *your* fork (since I added you as a remote)
<cole-h>
and then I remember our master is actually "released"
<gchristensen>
hah
<gchristensen>
yeah
<cole-h>
It happens roughly once every time I touch ofborg lol
<cole-h>
gchristensen: I'm thinking of updating the README to remove mention of "known users" and similar old stuff. I was thinking of expanding it to also note that `@ofborg` works the same as `@GrahamcOfBorg` -- yay/nay?
<gchristensen>
doit! :)
* cole-h
also changes the headline from `# grahamcofborg` to `# ofborg` :^)
<gchristensen>
yay
* cole-h
also references @ofborg before @GrahamcOfBorg in all applicable instances
<cole-h>
;^)
<gchristensen>
<3
<cole-h>
gchristensen: Should `config.{extra-,}known-users.json` be removed?
<gchristensen>
sure!
<gchristensen>
might need to be some fixups in the infra repo
<cole-h>
Oh, good point.
<cole-h>
Seems to only be referenced in `config_private` -- simple removal there