~sircmpwn/sr.ht-discuss

Re: builds.sr.ht: Allow worker settle timeout to be configured

Details
Message ID
<WH7H6S.R0DHM517165N2@ccw.icu>
DKIM signature
missing
Download raw message
I've been wondering whether it might be a good idea to expose more 
worker options in the config file.
I wrote some patches that expose the following options:

- s3-location: allows the s3 storage location to be changed
- max-artifacts: allows the maximum s3 artifact count to be changed
- max-artifact-size: allows the maximum artifact file size to be changed
- startup-timeout: allows the worker-startup timeout to be changed (as 
initially proposed)

The names aren't final yet, and I haven't had the chance to test the 
patches. (My new server is faster than the old one and able to start 
the workers within the 2-minute default, and I don't know enough about 
minio to validate the server location)

About the patches: I've designed the new configuration options to be 
optional. When they aren't set, the current hard-coded values will be 
set as a default.
It might also be a good idea to use the default when they are set to "".

I've attached the patches to this email because they aren't meant to be 
merged but to inspire discussion (and because I don't know how to use 
git send-email to reply to a thread without creating a new one)
I also have a copy on my sourcehut instance: 
https://git.hut.ccw.icu/~comcloudway/builds.sr.ht
Reply to thread Export thread (mbox)