jenkins-infra / helpdesk

Open your Infrastructure related issues here for the Jenkins project
https://github.com/jenkins-infra/helpdesk/issues/new/choose
17 stars 11 forks source link

Branch strategy on ci.jenkins.io diverges from prior configuration #3650

Closed NotMyFault closed 1 year ago

NotMyFault commented 1 year ago

Service(s)

ci.jenkins.io

Summary

The previous configuration aborted branch builds, once a PR has been created (e.g., dependabot/renovate, or when working directly on the repository). That's no longer the case. Currently, if you create an upstream branch and a PR, both are built, doubling the prior workload.

Reproduction steps

No response

dduportal commented 1 year ago

Caused by the operation in https://github.com/jenkins-infra/helpdesk/issues/3535

dduportal commented 1 year ago

@NotMyFault for which job(s) do you see this behavior?

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Core/ organization scanning, applying the following change (will trigger and indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Core/ organization scanning, applying the following change (will trigger and indexation):

* Exclude archived repositories

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)

Applied recursively, the "Abort build" option is now present on each.

NotMyFault commented 1 year ago

NotMyFault for which job(s) do you see this behavior?

I did a quick demo earlier with https://github.com/jenkinsci/console-column-plugin/pull/123: Screenshot 2023-07-05 at 15 53 55 The couple of checks weren't there before the update, coming from https://ci.jenkins.io/job/Plugins/job/console-column-plugin/job/yoho/2/, which built the branch separately to the PR.

For the https://ci.jenkins.io/job/Core/ organization scanning, applying the following change (will trigger and indexation):

  • Exclude archived repositories

Shouldn't that be applied to every folder? We don't really need to index archived repositories anywhere.

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Plugins/ organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

I did a quick demo earlier with jenkinsci/console-column-plugin#123: Screenshot 2023-07-05 at 15 53 55 The couple of checks weren't there before the update, coming from https://ci.jenkins.io/job/Plugins/job/console-column-plugin/job/yoho/2/, which built the branch separately to the PR.

Thanks! Applying to the "Plugins" org scanning (https://github.com/jenkins-infra/helpdesk/issues/3650#issuecomment-1621825160). Brace youselves.

  • Exclude archived repositories

Shouldn't that be applied to every folder? We don't really need to index archived repositories anywhere.

100% agree with you. Worth working on https://github.com/jenkins-infra/helpdesk/issues/3071

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Plugins/ organization scanning, applying the following change (will trigger an indexation):

* Exclude archived repositories

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)

Ran with success (and only 1 rate limit). Result:

Evaluating orphaned items in Plugins
Will remove testproject-plugin
Will remove icon-shim-plugin
Will remove azure-commons-plugin
Will remove analysis-test-plugin
Will remove android-lint-plugin
Will remove custom-view-tabs-plugin
Will remove suppress-stack-trace-plugin
Will remove aribot-plugin
Will remove blueocean-executor-info-plugin
Will remove osf-builder-suite-xml-linter-plugin
Will remove analysis-core-plugin
Will remove pipeline-leo-deploy-step-plugin
Will remove dry-plugin
Will remove warnings-plugin
Will remove evergreen-plugin
Will remove stride-notification-plugin
Will remove findbugs-plugin
Will remove analysis-collector-plugin
Will remove fabric-beta-publisher-plugin
Will remove checkstyle-plugin
Will remove lambda-test-runner-plugin
Will remove tasks-plugin
Will remove pmd-plugin
Will remove remoting-security-workaround-plugin
Will remove headspin-plugin
Will remove ruby-runtime-plugin
Will remove google-container-security-plugin
Will remove coding-webhook-plugin
Will remove lightstep-incident-response-plugin
Will remove maven-hudson-dev-plugin
Will remove qualys-api-security-plugin
Will remove fortify-cloudscan-plugin
Will remove popper-api-plugin
Finished: SUCCESS
dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Tools/ organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Tools/ organization scanning, applying the following change (will trigger an indexation):

* Exclude archived repositories

* Remove `maven-license-plugin` (as per https://matrix.to/#/!JLUOInpEYmxJIYXlzs:matrix.org/$Zj8TB1Cx0YyhsaeVFkt41zGjehMgEk2kQ2XUHogodjk?via=libera.chat&via=matrix.org&via=gitter.im) to avoid duplication with https://ci.jenkins.io/job/Pklugins/

* Scan Organization Triggers  set to 1day (instead of 1 week)

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)
[Wed Jul 05 16:24:47 UTC 2023] Finished organization scan. Scan took 57 sec
Evaluating orphaned items in Tools
Will remove maven-license-plugin
Will remove custom-distribution-service
Finished: SUCCESS
dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Infra organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Infra organization scanning, applying the following change (will trigger an indexation):

* Remove `awestruct-ibeams`

* Scan Organization Triggers  set to 1day (instead of 1 week)

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)
[Wed Jul 05 16:26:44 UTC 2023] Finished organization scan. Scan took 19 sec
Evaluating orphaned items in Infra
Will remove awestruct-ibeams
Finished: SUCCESS
dduportal commented 1 year ago

For the https://ci.jenkins.io/job/jenkinsfile-runner organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/jenkinsfile-runner organization scanning, applying the following change (will trigger an indexation):

* Exclude archived repositories

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)
[Wed Jul 05 16:32:12 UTC 2023] Finished organization scan. Scan took 53 sec
Finished: SUCCESS
dduportal commented 1 year ago

For the https://ci.jenkins.io/job/jenkinsci-libraries organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/jenkinsci-libraries organization scanning, applying the following change (will trigger an indexation):

* Exclude archived repositories

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)
[Wed Jul 05 17:00:17 UTC 2023] Finished organization scan. Scan took 2 min 25 sec
Evaluating orphaned items in Libraries
Will remove bytecode-compatibility-transformer
Will remove extras-executable-war
Finished: SUCCESS
dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Packaging/ organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Packaging/ organization scanning, applying the following change (will trigger an indexation):

* Exclude archived repositories

* Orphaned Item Strategy
  -> Enable Abort builds
  -> Discard old items (no max day, no max items)
[Wed Jul 05 17:05:04 UTC 2023] Finished organization scan. Scan took 1 min 53 sec
Evaluating orphaned items in Packaging
Will remove jnlp-agents
Will remove docker-ssh-slave
Will remove docker-jnlp-slave
Will remove docker-slave
Finished: SUCCESS
dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Stapler/ organization scanning, applying the following change (will trigger an indexation):

dduportal commented 1 year ago

For the https://ci.jenkins.io/job/Stapler/ organization scanning, applying the following change (will trigger an indexation):

* Exclude archived repositories

* Orphaned Item Strategy
  -> Enable Abort builds
[Wed Jul 05 17:09:12 UTC 2023] Finished organization scan. Scan took 11 sec
Finished: SUCCESS
dduportal commented 1 year ago

Setup applied everywhere. I'm closing, thanks @NotMyFault !

Feel free to reopen if I missed anything!