acceptance-test-harness icon indicating copy to clipboard operation
acceptance-test-harness copied to clipboard

Conditional build: do we need to rebuild the full ATH for all Renovabot / Dependabot PRs?

Open dduportal opened this issue 1 year ago • 5 comments

As per https://github.com/jenkins-infra/helpdesk/issues/3551#issuecomment-1544478322, the Infra team would like to control the costs on ci.jenkins.io.

We saw a lot of builds (using a LOT of highmem VM agents) for PR changing Dockerfile dependencies such as https://github.com/jenkinsci/acceptance-test-harness/pull/1164 or https://github.com/jenkinsci/acceptance-test-harness/pull/1149.

At first sight, it seems a waste to execute the whole ATH for such small changes. Is there any means today to conditionnaly build?

Also, it does not look a good idea to run the ATH on ci.jenkins.io if a PR changes the file src/main/resources/ath-container/Dockerfile: it's built by infra.ci and pushed to the DockerHub.

dduportal avatar May 11 '23 18:05 dduportal