Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

[meta] use same staging job for all branches #728

Merged
merged 1 commit into from
Jul 17, 2020

Conversation

jmlrt
Copy link
Member

@jmlrt jmlrt commented Jul 15, 2020

We currently have staging jobs for every branch (6.8, 7.8, 7.x & master). However, we should only test staging from the minor release branches (ie: 6.8 & 7.8). In addition, staging jobs are only triggered manually after a BC, so staging job for a specific branch should rarely have more than 10 builds. This commit will rationalize and help having a cleaner Helm charts Jenkins view by using the same staging jobs for every branches.

Note that staging jobs templates should be removed from the other branches.

We currently have staging jobs for every branch (6.8, 7.8, 7.x & master).
However, we should only test staging from the minor release branches
(ie: 6.8 & 7.8). In addition, staging jobs are only triggered manually
after a BC, so staging job for a specific branch should rarely have
more than 10 builds. This commit will rationalize and help having a
cleaner Helm charts Jenkins view by using the same staging jobs for
every branches.

Note that staging jobs templates should be removed from the other
branches.
@jmlrt jmlrt added the meta label Jul 15, 2020
@jmlrt jmlrt requested a review from a team July 15, 2020 13:24
@jmlrt
Copy link
Member Author

jmlrt commented Jul 15, 2020

Current list of staging jobs in Jenkins Helm charts view:
Screenshot 2020-07-15 at 15 24 49

Copy link
Contributor

@Conky5 Conky5 left a comment

Choose a reason for hiding this comment

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

LGTM

@jmlrt jmlrt merged commit 47a45cf into elastic:master Jul 17, 2020
@jmlrt jmlrt deleted the one-job-to-rule-all-staging-builds branch July 17, 2020 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants