Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/jenkins] deprecate chart #23602

Merged
merged 3 commits into from Sep 1, 2020
Merged

[stable/jenkins] deprecate chart #23602

merged 3 commits into from Sep 1, 2020

Conversation

torstenwalter
Copy link
Collaborator

@torstenwalter torstenwalter commented Aug 24, 2020

This chart was moved to a new location: https://github.com/jenkinsci/helm-charts

Part-of: #21103
Part-of: #23562
Part-of: jenkinsci/helm-charts#4

Should be merged after jenkinsci/helm-charts#1

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

This chart was moved to a new location: https://github.com/jenkinsci/helm-charts

Signed-off-by: Torsten Walter <[email protected]>
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Aug 24, 2020
@helm-bot helm-bot added Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 24, 2020
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 24, 2020
@k8s-ci-robot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@torstenwalter
Copy link
Collaborator Author

/assign @wmcdona89

Signed-off-by: Torsten Walter <[email protected]>
@torstenwalter torstenwalter changed the title [stable/jenkins] deprectae chart [stable/jenkins] deprecate chart Aug 24, 2020
Signed-off-by: Torsten Walter <[email protected]>
@torstenwalter torstenwalter marked this pull request as ready for review September 1, 2020 07:25
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 1, 2020
@torstenwalter
Copy link
Collaborator Author

This is good to go:

$ helm repo add jenkinsci https://charts.jenkins.io/
$ helm search repo jenkinsci
"jenkinsci" has been added to your repositories
NAME             	CHART VERSION	APP VERSION	DESCRIPTION                                       
jenkinsci/jenkins	2.6.0        	lts        	Open source continuous integration server. It s...

Copy link
Member

@scottrigby scottrigby left a comment

Choose a reason for hiding this comment

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

🏆

@scottrigby
Copy link
Member

/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: scottrigby, torstenwalter

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 1, 2020
@k8s-ci-robot k8s-ci-robot merged commit 871a1be into helm:master Sep 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants