dd010101 / vyos-jenkins

Scripts for building custom VyOS stream (1.5 circinus) packages/images. Also legacy scripts for building frozen 1.3 equuleus/1.4 sagitta packages/images.
98 stars 32 forks source link

Move branch filter logic #30

Closed GurliGebis closed 4 months ago

GurliGebis commented 4 months ago

By moving it up here, we make sure only jobs that should be built is added to the JOBS array in the first place. This is a better way, since the skipped jobs won't occupy the build slots (since we take X number of jobs from the JOBS array for each batch, for a CONCURRENT_JOBS_COUNT of 10 - if 6 is skipped, only 4 jobs will be started).

Now, they are not added to the JOBS array, so the rest works as intended.

Before:

Building jobs in Jenkins...
Total number of jobs: 91
Concurrent jobs: 10 (Can be overridden by setting the CONCURRENT_JOBS_COUNT environment variable)

[   Running   ] Package: aws-gateway-load-balancer-tunnel-handler - Branch: sagitta
[   Running   ] Package: ddclient - Branch: sagitta
[   Skipped   ] Package: dropbear - Branch: equuleus (excluded)
[   Running   ] Package: dropbear - Branch: sagitta
[   Running   ] Package: ethtool - Branch: sagitta
[   Skipped   ] Package: frr - Branch: equuleus (excluded)
[   Running   ] Package: frr - Branch: sagitta
[   Skipped   ] Package: hostap - Branch: equuleus (excluded)
[   Running   ] Package: hostap - Branch: sagitta
[   Running   ] Package: hsflowd - Branch: sagitta

After:

Building jobs in Jenkins...
Total number of jobs: 91
Concurrent jobs: 10 (Can be overridden by setting the CONCURRENT_JOBS_COUNT environment variable)

[   Running   ] Package: aws-gateway-load-balancer-tunnel-handler - Branch: sagitta
[   Running   ] Package: ddclient - Branch: sagitta
[   Running   ] Package: dropbear - Branch: sagitta
[   Running   ] Package: ethtool - Branch: sagitta
[   Running   ] Package: frr - Branch: sagitta
[   Running   ] Package: hostap - Branch: sagitta
[   Running   ] Package: hsflowd - Branch: sagitta
[   Running   ] Package: hvinfo - Branch: sagitta
[   Running   ] Package: ipaddrcheck - Branch: sagitta
[   Running   ] Package: isc-dhcp - Branch: sagitta
dd010101 commented 4 months ago

Sure.

GurliGebis commented 4 months ago

Seems like I created it as a draft by mistake - it can be merged now 😊

dd010101 commented 4 months ago

👍