gchristensen changed the topic of #nixos-borg to: https://www.patreon.com/ofborg https://monitoring.nix.ci/dashboard/db/ofborg?refresh=10s&orgId=1&from=now-1h&to=now "I get to skip reviewing the PHP code and just wait until it is rewritten in something sane, like POSIX shell. || https://logs.nix.samueldr.com/nixos-borg
<gchristensen> I'm sure this image will finish building today
<gchristensen> I can never remember how I deploy to my mac
<gchristensen> maybe I won't finish this image today
<samueldr> in the end I didn't feel the idea I had really worked
<{^_^}> ofborg/infrastructure#6 (by samueldr, 20 seconds ago, open): Adds self-contained landing page + svg logo.
<samueldr> well, now I know you have seen it :)
<gchristensen> :D
<samueldr> the hexagon part of C worked better in my mind when the lambda flake was simplified to a simple hexagon
<gchristensen> ah
<gchristensen> back in 20, we can try more things :)
<gchristensen> (deploying now)
<samueldr> and had previously played with "hy" https://stuff.samueldr.com/screenshots/2018/08/20180808204534.png
<samueldr> oops!
<samueldr> never tell this to my developer colleagues... I always get mad at them for forgetting to handle the <title> https://github.com/ofborg/infrastructure/pull/7
<{^_^}> ofborg/infrastructure#7 (by samueldr, 33 seconds ago, open): Fixes <title> tag.
<gchristensen> haha
<gchristensen> samueldr: https://nix.ci/ looks a little avant garde
<samueldr> ah poo!, forgot about the font ;)
<samueldr> well, also kinda thought that inkscape would rasterize the font
<samueldr> or uh, vectorize
<samueldr> (since I locally have the font, it looks right)
<gchristensen> aah
<samueldr> and since all my computers are synchronized, fonts included... I'm not the best one to see the issue
<gchristensen> :D
<gchristensen> deploying ...
<samueldr> if only there was a simple way to build some kind of test environment, maybe virtual, with a limited set of controled packages and features to test things...
<samueldr> really makes you think
<gchristensen> :D
<gchristensen> beautiful!
<samueldr> I see absolutely nothing different :/
<samueldr> :)
<gchristensen> ok going to publish this thing
<gchristensen> some OfBorg updates! build time-outs, push-button deploys, monitoring transparency, homepages, :D https://www.patreon.com/posts/timeouts-nix-ci-20643198
<gchristensen> gleber_, globin, can both of you upgrade your ofborg builder when convenient? :)
orivej has quit [Ping timeout: 268 seconds]
<timokau[m]> Nice, thanks for your work :)
<timokau[m]> That sounds like you intend to eventuall deprecate the comments api, is that true? I like being able to interact with ofBorg without a GUI.
orivej has joined #nixos-borg
<LnL> it's really noisy and something like the checks also makes the results much clearer
<timokau[m]> But then how can I trigger a bulid via email and how will I be notified if it succeeds/fails?
<timokau[m]> Success notification is not *that* important if we have `bulid-then-merge`
<timokau[m]> My ideal interface would be just like the current one but with only one comment per command.
<gchristensen> timokau[m]: just ... thoughts ... there. good to know some people really like it :)
orivej has quit [Ping timeout: 268 seconds]
<gchristensen> it would be cool if I could kick off / ban users until they upgrade
<gchristensen> thanuk you gleber_ for upgrading! I think rbvermaa's builder will go away soon, ping globin :)
<gchristensen> hmmmm LnL I I think the BuildResult could use some adjusting to make the states make more sense
<gchristensen> for example, on a merge failure maybe we shouldn't even have a attempted_attrs / skipped_attrs
<LnL> you mean like an enum?
<gchristensen> yeah
<LnL> I didn't need it, but serde has an option to inline enums IIRC
<LnL> #[serde(untagged)]
<gchristensen> hmm that could make it an easier migration
<LnL> alternatively we could do something like tag each 'version'
<gchristensen> oh yeah
<LnL> but I'm not sure if you can define a default tag to migrate the first time
<gchristensen> I think you can!
<gchristensen> not sure
<{^_^}> [ofborg] @grahamc pushed 0 commits to next: https://git.io/fNQvx
<{^_^}> [ofborg] @grahamc closed pull request #164 → Make ofborg able to (un)label issues and PRs → https://git.io/vpC2M
<{^_^}> [ofborg] @grahamc closed pull request #203 → Disable aliases in OfBorg's outPaths → https://git.io/fNByo
<{^_^}> [ofborg] @grahamc pushed 0 commits to bump-0.1.7: https://git.io/fNQvh
<LnL> let me do some testing, maybe only versioning on the rust side is fine
<gchristensen> cool
<{^_^}> [ofborg] @grahamc pushed to bug-156-deleted-comments « Don't trigger events on deleted comments »: https://git.io/fNQfS
<{^_^}> [ofborg] @grahamc opened pull request #214 → Don't trigger events on deleted comments → https://git.io/fNQfH
<gchristensen> LnL: I wonder if it ofborg shouldn't react to edits either?
<LnL> I use edits for rebuilds
<gchristensen> aye
<gchristensen> ok I'll keep that :)
<gchristensen> what if ofborg reacted to comments based on status, "+1" comments are being handled, "confused" if not
<gchristensen> is that too obtuse?
<{^_^}> [ofborg] @grahamc pushed 2 commits to released: https://git.io/fNQUj
<{^_^}> [ofborg] @grahamc merged pull request #214 → Don't trigger events on deleted comments → https://git.io/fNQfH
<samueldr> gchristensen: 👍 when it read the comment
<{^_^}> [ofborg] @grahamc pushed 0 commits to bug-156-deleted-comments: https://git.io/fNQTf
<LnL> oh something for pending builds would be neat
<samueldr> at least for now
<gchristensen> for pending build status, I'm thinking more of a self-updating comment
<gchristensen> (for those who saw the "reasons this is broken" post, yes, yes, I'd be adding a central state service :$)
<gchristensen> or a mutex based on the "heart" reaction :')
<LnL> lol
<gchristensen> infinisil: would you mind if I tested somethin on your `sad` PR?
<infinisil> Heh sure
<gchristensen> ok, please don't merge it :)
<infinisil> Alright
<gchristensen> ok, done, thank you infinisil
<infinisil> Hmm, why is it still evaluating though
<gchristensen> it hasn't finished yet
<infinisil> Ah right, takes a while
<gchristensen> it also got restarted part way through due to a deploy
<gchristensen> that would be a nice thing, let evaluators and builders finish their task prior to restarting
<samueldr> state!
<samueldr> gchristensen: do you think it's bad to save data inside the first ofborg comment?
<samueldr> I think (have to verify) that with <!-- --> you could do some
<gchristensen> you mean
<gchristensen> like
<infinisil> Lol
<samueldr> (ab)using github as your state
<samueldr> I mean that
<infinisil> Oh god
<samueldr> (I thought about using something similar for the backports tooling)
<gchristensen> github is not suitable for storing state
<gchristensen> but I would be happy with using this as a marker to find the comment to update later :)
<infinisil> Except if you use a repo for it, then you have transactional updates even :)
<samueldr> definitely not, but a light representation where failing isn't an issue it could do
<gchristensen> so my thought is have a central store of what the comment should be and erase it / rewrite it each time
<samueldr> diffing is hard, so always rewriting is probably better
<LnL> gchristensen: :o
<gchristensen> I know I know :|
<gchristensen> elbow surgery apparently changes you
<samueldr> they removed the bit that's harshly against state
<samueldr> probably next to the funny bone
<gchristensen> lol
<gchristensen> there is another idea
<gchristensen> which is comment once with a link to an external dashboard
<gchristensen> which loads all the data it needs from the logs db
<samueldr> that's probably better, and I think github has support for linking without comments
<gchristensen> you think so?
<infinisil> Stupid idea: Have a github repo with an Issue for every nixpkgs PR which contains all kinds of info. Link to the PR, such that it gets shown in the github comment history
<infinisil> Better link to the issue from the PR. Would be a bit too uneventful though
<infinisil> (and then it's just the same as a dashboard)
<gchristensen> well the dashboard also has streaming log input :)
<gchristensen> output*
<samueldr> though I feel the comments are also a worthy thing to keep around, it's an end-user notification (in case you were thinking about moving it all away)
<samueldr> well, unless the new github thing for CI has better things
<{^_^}> [ofborg] @grahamc pushed 3 commits to tagged-comments: https://git.io/fNQIo
<gchristensen> fundamentally they're just like statuses r
<gchristensen> which makes it hard because a build failure is an "X"
<samueldr> (while a transition into failure should be an X)
<{^_^}> [ofborg] @grahamc opened pull request #215 → Tagged comments → https://git.io/fNQLJ
<gchristensen> ok well I think the dashboard thing is pretty cool
<gchristensen> it is just a bit more "ambitious" I think?
<samueldr> it is
<gchristensen> but having a list of changed attributes and a "build this one" button would be cool
<gchristensen> ok, it is dinner time.
orivej has joined #nixos-borg
jtojnar has quit [Ping timeout: 260 seconds]
<LnL> that machine specifically seems to spike to 100% from time to time
jtojnar has joined #nixos-borg
<gchristensen> it is a builder building :)
<gchristensen> LnL:
<LnL> oh, and the others are only infra/evaluators?
<gchristensen> eya
<LnL> ok, everything's fine then :D
<gchristensen> :D
<LnL> also, the untagged serialisation logic looks sane
<gchristensen> ooh!
<LnL> not sure if the behaviour of ambiguous enums is well defined, but that shouldn't really matter
<gchristensen> should we do an untagged union of the old version, vs. new version, and then a tagged union for v2 / v3?
<LnL> seems to take the first one
<gchristensen> I'm thinking I might create a spot bidder thing with buildkite which automatically adds more evaluators using spot instances on Packet, every 15 minutes
<gchristensen> the evals would be faster than the hetzner cloud machine, and it might be a bit cheaper
<LnL> don't think that's necessary but we could, makes it easier for potential other things (eg. non rust) to handle the message versions
<gchristensen> however, the obvious #1 burning thing to solve is multiple core-N machines.
<gchristensen> nixops' inability to handle multiple nixpkgs revisions stresses me out
<infinisil> gchristensen++
<{^_^}> gchristensen's karma got increased to 17
<infinisil> Yes!
<gchristensen> https://github.com/NixOS/nixops/pull/665 need to improve this I guess
<{^_^}> nixops#665 (by grahamc, 1 year ago, open): Support multiple versions of nixpkgs in one network
<gchristensen> ooohhh https://nix.ci/prometheus/graph?g0.range_input=2d&g0.expr=node_systemd_unit_state%7Bname%3D~%22.*service%24%22%2Cstate%3D%22failed%22%7D&g0.tab=0
<gchristensen> terraform is simultaneously a good and very bad tool
<gchristensen> like, `count = N` is really cute until you want to do a staged upgrade