00:00
<
cole-h >
hm, but do we really want it to auto-deploy when we push to infra?
00:00
<
cole-h >
(e.g. PRs, broken commits, etc)
00:01
<
cole-h >
Like when I made a PR that accidentally depended on another PR recently and subsequently broke the enter-env script...
*whistles*
00:01
<
gchristensen >
it'll pause before actually deploying though
00:01
<
gchristensen >
let's see
00:01
<
cole-h >
that's true
00:04
<
gchristensen >
retry doesn't work though
00:04
<
gchristensen >
it retries the same commit
00:04
<
cole-h >
oh does it
00:04
<
gchristensen >
yeah
00:06
<
cole-h >
OK, then auto-deploy it is :D
00:07
<
gchristensen >
yeah, okay
00:07
<
gchristensen >
"Triggered from Webhook"
00:07
<
gchristensen >
so merging a PR will automatically run the buildkite pipeline
00:08
<
cole-h >
Would also be cool to have the blocked deploys time out after a day (maybe), if possible?
00:08
<
gchristensen >
yeah something like that maybe
00:08
<
gchristensen >
I wonder if that is possible
00:09
<
{^_^} >
buildkite/feedback#474 (by sayboras, 2 years ago, open): How to configure timeout for block step
00:09
<
cole-h >
Oh well, not a big deal
00:09
<
gchristensen >
so, yeah
00:10
<
gchristensen >
server intake should only update more steps if there are no changes
00:11
<
gchristensen >
I should add more agents for the GC steps....
00:12
<
cole-h >
Would be nicer if morph had async `morph exec` though... ;d
00:12
<
cole-h >
Would also be nice if morph was maintained...................
00:12
<
gchristensen >
I understand they're waiting on major changes until flakes are stabilized
00:13
<
{^_^} >
DBCDK/morph#141 (by cole-h, 3 weeks ago, open): nix: add config file to NIX_SSHOPTS
00:13
<
gchristensen >
good point
00:15
<
cole-h >
(Of course, not to hate; I understand how little free time we all have right now..)
00:17
<
gchristensen >
oh interesting
00:17
<
gchristensen >
you see it too?
00:18
<
cole-h >
interesting indeed
00:18
<
cole-h >
adding agents immediately adds activity to them
00:18
<
gchristensen >
I guess the concurrency and ordering rules aren't pick yenough
00:18
<
gchristensen >
I mean, not sure it is a problem really
00:19
<
cole-h >
I think it's because the gc step is part of the infra-gc concurrency group, while dry activation is part of -deploy
00:19
<
gchristensen >
lol at dry activation finishing before GC
00:19
<
gchristensen >
I think maybe just a "wait" step before dry activation ...
00:19
<
gchristensen >
btw I could do this unless you're keen on it :)
00:20
<
cole-h >
Up to you ;)
00:20
<
gchristensen >
I'll send a PR ...
00:21
<
cole-h >
oh, nice, wait waits for ALL previous steps to finish
00:21
<
cole-h >
I thought it would just wait for the one directly prior to it
00:21
<
cole-h >
that's a nice idea then, yeah :)
00:21
<
gchristensen >
yeah I'm actually not sure
00:22
<
gchristensen >
I was just wondering the same thing
00:22
<
cole-h >
A wait step waits for all previous steps to have successfully completed before allowing following jobs to continue.
00:22
<
cole-h >
(and also: I concur in the "lol at dry activation finishing before GC")
00:23
<
gchristensen >
so, we'll be able to see what it actually does
00:23
<
cole-h >
or is that a side effect of commiting to "origin"
00:23
<
cole-h >
we can test that...
00:24
<
gchristensen >
it is a checkbox I have checked
00:24
<
cole-h >
Did you manually cancel the other build? Or did the PR build cancel it automagically?
00:24
<
gchristensen >
it actually failed
00:25
<
gchristensen >
because a GC step was interrupted when i added more agents
00:25
<
gchristensen >
ah ha, that looks like it is doing what we wanted!
00:25
<
gchristensen >
yea?
00:25
<
cole-h >
Yeah, except.... I only saw 5/6 machines running GC
00:25
* cole-h
wonders if my NRHOSTS logic is busted
00:25
<
gchristensen >
there is actually only 5
00:26
<
gchristensen >
but on restart it noticed 1 was recently taken, so it skipped 1
00:26
<
gchristensen >
cool, I think we can merge that PR ... want to approve?
00:27
<
cole-h >
(jk, of course)
00:28
<
gchristensen >
I invited you to the org so you can send PRs and see what the pipeline does
00:29
<
cole-h >
I assume the "autodeploy" is locked behind being a member of the org, then
00:29
<
cole-h >
Smart, was going to suggest some kind of ACL like that
00:29
<
gchristensen >
PRs have to come from within the org
00:30
<
gchristensen >
there are only 5 agents :)
00:30
<
cole-h >
Cool stuff
00:31
<
cole-h >
By "within the org" do you mean I have to push to a branch on "origin" and PR from there?
00:35
<
cole-h >
assuming we should still `exit 1` in terraform-import if the targets change? Or is that automagically handled in buildkite?
00:37
<
gchristensen >
so we could exit 1, or move the rest of terraform-input.yml in to a new .yml that is uploaded at the end of terraform-import if there are no changes
00:38
<
cole-h >
One is a +1/-1 diff, one is larger... but it probably makes more sense to make a new yml
00:38
<
gchristensen >
agree
00:41
supersandro2000 has quit [Disconnected by services]
00:41
supersandro2000 has joined #nixos-borg
00:41
* cole-h
wonders if `- wait` works inside of a "buildkite-agent pipeline upload"ed pipeline
00:42
<
cole-h >
only one way to find out ig
00:59
<
{^_^} >
ofborg/infrastructure#50 (by cole-h, 7 minutes ago, open): terraform-import: deploy only if targets unchanged
00:59
<
cole-h >
looks like it works
01:00
<
gchristensen >
okay now a real test!
01:01
<
{^_^} >
ofborg/infrastructure#51 (by grahamc, 7 seconds ago, open): Delete default.nix
01:03
<
gchristensen >
w000t
01:05
<
gchristensen >
it is a bit messy, but I'm thinking I'd like to relocate the ircbot system {^_^} to use the ofborg rabbitmq server
01:06
<
gchristensen >
though I suppose I could just make a new rabbitmq server on the free tier and not worry abou tit
01:07
<
cole-h >
Semi-related: is the ofborg queue paid for out of pocket, by the foundation, or "other"?
01:08
<
gchristensen >
right now out of pocket but I could expense it to the foundation if I wasn't lazy about expensing it ...
01:10
<
gchristensen >
all is not lost, though, I put it down as a business expense for tax purposes
01:12
<
gchristensen >
eh I'll make a new rabbitmq server for this on the free tier
01:12
<
gchristensen >
unlikely to go over the 1M/mo limit on messages
01:13
<
cole-h >
Until someone decides to spam (re)open-close PRs :D
01:32
<
gchristensen >
grahamvault?
01:32
<
gchristensen >
cole-h: reload?
01:36
<
cole-h >
gchristensen: It's the GitHub app for authenticated push with enter-env :P
01:36
<
gchristensen >
yeah but ... reload ... I changed it ...
01:37
<
gchristensen >
okay i changed it again
01:37
<
cole-h >
Nice little petunia
01:37
<
gchristensen >
thanks
01:38
<
cole-h >
rounded square for apps vs circle for users is kinda funny
01:38
<
gchristensen >
yeah..
01:39
<
cole-h >
if it were me, I'd add the `[bot]` tag that bots get when commenting to the app name (and s/bot/app/) instead, and keep the circle icon
01:39
<
cole-h >
(or use all rounded squares)
02:13
supersandro2000 has joined #nixos-borg
02:40
orivej_ has quit [Ping timeout: 240 seconds]
03:02
orivej has joined #nixos-borg
03:25
orivej has quit [Ping timeout: 240 seconds]
05:35
supersandro2000 has quit [Ping timeout: 265 seconds]
05:52
supersandro2000 has joined #nixos-borg
05:59
evanjs has joined #nixos-borg
05:59
evanjs has quit [Remote host closed the connection]
05:59
evanjs has joined #nixos-borg
06:19
evanjs has joined #nixos-borg
06:29
evanjs has joined #nixos-borg
06:45
cole-h has quit [Ping timeout: 256 seconds]
08:40
orivej has joined #nixos-borg
10:45
orivej has quit [Ping timeout: 256 seconds]
11:02
evanjs has joined #nixos-borg
14:33
orivej has joined #nixos-borg
15:48
{^_^} has quit [Remote host closed the connection]
15:49
{^_^} has joined #nixos-borg
15:53
cole-h has joined #nixos-borg
16:03
orivej has quit [Ping timeout: 240 seconds]
16:06
{^_^} has quit [Remote host closed the connection]
16:06
{^_^} has joined #nixos-borg
16:46
cole-h has quit [Ping timeout: 240 seconds]
17:47
orivej has joined #nixos-borg
23:00
cole-h has joined #nixos-borg
23:57
{^_^} has quit [Remote host closed the connection]
23:57
{^_^} has joined #nixos-borg