Skip to content

PHOENIX-7850: Unblock Phoenix PR validation builds#2471

Merged
virajjasani merged 3 commits into
apache:masterfrom
sanjeet006py:jenkins-pr-build-retention
May 12, 2026
Merged

PHOENIX-7850: Unblock Phoenix PR validation builds#2471
virajjasani merged 3 commits into
apache:masterfrom
sanjeet006py:jenkins-pr-build-retention

Conversation

@sanjeet006py
Copy link
Copy Markdown
Contributor

@sanjeet006py sanjeet006py commented May 12, 2026

  • buildDiscarder: daysToKeep 15->3, artifactDaysToKeep 5->3, add numToKeep=2 and artifactNumToKeep=2.
  • Add rateLimitBuilds: 2 build per rolling 24h window per PR branch, userBoost=true so manual triggers can override the cap.

What changes were proposed in this pull request?

Tune PR builds:

  • Retain builds for 3 days (including archived artifacts)
  • Retain last 2 builds only (including archived artifacts)
  • Limit to one build per day.

Why are the changes needed?

To re-enable PR validation builds.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Not tested but created using Jenkins pipeline snippet generator.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Opus 4.7

- buildDiscarder: daysToKeep 15->3, artifactDaysToKeep 5->3,
  add numToKeep=2 and artifactNumToKeep=2.
- Add rateLimitBuilds: 1 build per rolling 24h window per PR branch,
  userBoost=false so manual triggers also respect the cap.
Copy link
Copy Markdown
Contributor

@virajjasani virajjasani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 for the changes, just one point which is still under discussion

Comment thread Jenkinsfile.github Outdated
numToKeepStr: '2',
artifactNumToKeepStr: '2'
))
rateLimitBuilds(throttle: [count: 1, durationName: 'day', userBoost: false])
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about count 2 and userBoost true?

@sanjeet006py sanjeet006py requested a review from virajjasani May 12, 2026 19:15
Copy link
Copy Markdown
Contributor

@virajjasani virajjasani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@virajjasani virajjasani merged commit 78d91e0 into apache:master May 12, 2026
sanjeet006py added a commit that referenced this pull request May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants