Add setting for concurrent builds #110

New issue

Closed

opened 2022-02-27 09:22:18 +01:00 by Jef Roosens · 1 comment

Jef Roosens commented

2022-02-27 09:22:18 +01:00

Owner

I suggest an env var, VIETER_MAX_CONCURRENT_BUILDS, which sets how many builds can be running at once. Implementation should be fairly simple:

we currently already have a for loop that checks every 5s whether the build container has finished. This could be expanded to make it check for multiple containers, and at the start of each iteration, we check whether there's a new spot free. Because the check itself is synchronous, no issues should arise from changing the same variable concurrently.

I suggest an env var, `VIETER_MAX_CONCURRENT_BUILDS`, which sets how many builds can be running at once. Implementation should be fairly simple: we currently already have a for loop that checks every 5s whether the build container has finished. This could be expanded to make it check for multiple containers, and at the start of each iteration, we check whether there's a new spot free. Because the check itself is synchronous, no issues should arise from changing the same variable concurrently.