Remove duplicate Jenkins push trigger - #6166
Merged
Merged
Conversation
JP0P
marked this pull request as draft
August 19, 2026 02:27
JP0P
force-pushed
the
codex/remove-duplicate-jenkins-trigger
branch
from
August 19, 2026 21:07
b7f1ba3 to
5cc6634
Compare
JP0P
marked this pull request as ready for review
August 19, 2026 21:07
j0ntz
approved these changes
Sep 4, 2026
j0ntz
force-pushed
the
codex/remove-duplicate-jenkins-trigger
branch
from
September 4, 2026 22:21
5cc6634 to
6e300f6
Compare
j0ntz
enabled auto-merge
September 4, 2026 22:22
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes the redundant Declarative Pipeline
githubPush()trigger from theEdge React GUI multibranch
Jenkinsfile.GitHub Branch Source remains responsible for normal branch-update delivery.
Why
The multibranch job already schedules builds from Branch Source events. The
additional
githubPush()handler could inspect multiple branches for the samerepository event and schedule a revision that was already building.
This produced extra same-revision builds both for isolated updates and when
another branch was updated while the first remained active.
Live validation results
Validation ran in the live
EdgeApp/edge-react-guimultibranch job using thededicated
test-parmandtest-colbybranches.Across all 15 post-change trials:
Method and controls
githubPush()block.blank line.
without running the application build.
Source, queue-entry, and build-completion timestamps.
replaced, preserving ten verified overlap trials.
configuration state, and trial totals.
Risk
The change affects build scheduling only; it does not modify application code.
The main risk was that removing the explicit trigger might prevent expected
builds. The live results directly checked this: every one of the 25 expected
branch builds was delivered and completed successfully through Branch Source.
The findings are scoped to the scheduling paths exercised in these controlled
Edge React GUI trials.
Evidence
The report links the Jenkins builds directly. A verified SHA-256 manifest
covers all 627 retained evidence files. The raw archive and manifest are
available on request.
Note
Low Risk
CI scheduling-only change with no application code touched; live validation showed expected builds still fire via Branch Source after removing the duplicate trigger.
Overview
Removes the redundant
triggers { githubPush() }block from the multibranchJenkinsfile, so branch builds are scheduled only through GitHub Branch Source instead of a second push handler that could queue duplicate same-revision builds (including when another branch was updating while a build was already running).Adds
docs/JENKINS_TRIGGER_VALIDATION.mdanddocs/JENKINS_TRIGGER_VALIDATION_TRIALS.csv, documenting live before/after trials on dedicated Jenkins branches: extra duplicate Parm/Colby builds dropped to zero while all expected post-change builds still completed successfully.Reviewed by Cursor Bugbot for commit 33074a5. Bugbot is set up for automated code reviews on this repo. Configure here.