Skip to content

service account support for drone server #76

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jimsheldon merged 2 commits into from
Jun 24, 2022
Merged

service account support for drone server #76

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9ae4f9e
service account support for drone server
bkk-bcd Jun 16, 2022
a5bd967
vscode unformat
bkk-bcd Jun 16, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion charts/drone/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ name: drone
description: Drone is a self-service Continuous Delivery platform for busy development teams
# TODO: Un-comment once we move back to apiVersion: v2.
# type: application
version: 0.2.5
version: 0.3.0
appVersion: 2.12.0
kubeVersion: "^1.13.0-0"
home: https://drone.io/
Expand Down
1 change: 1 addition & 0 deletions charts/drone/templates/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ spec:
{{- toYaml . | nindent 8 }}
{{- end }}
automountServiceAccountToken: {{ .Values.automountServiceAccountToken }}
serviceAccountName: {{ include "drone.serviceAccountName" . }}
securityContext:
{{- toYaml .Values.podSecurityContext | nindent 8 }}
containers:
Expand Down
12 changes: 12 additions & 0 deletions charts/drone/templates/serviceaccount.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
{{- if .Values.serviceAccount.create -}}
apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ include "drone.serviceAccountName" . }}
labels:
{{- include "drone.labels" . | nindent 4 }}
{{- with .Values.serviceAccount.annotations }}
annotations:
{{- toYaml . | nindent 4 }}
{{- end }}
{{- end }}
4 changes: 4 additions & 0 deletions charts/drone/values.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,10 @@
"$id": "#/properties/nodeSelector",
"type": "object"
},
"serviceAccount": {
"$id": "#/properties/serviceAccount",
"type": "object"
},
"tolerations": {
"$id": "#/properties/tolerations",
"type": "array"
Expand Down
10 changes: 9 additions & 1 deletion charts/drone/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,15 @@ imagePullSecrets: []
nameOverride: ""
fullnameOverride: ""

serviceAccount:
# Specifies whether a service account should be created
create: true
# Annotations to add to the service account
annotations: {}
# The name of the service account to use.
# If not set and create is true, a name is generated using the fullname template
name: ""

# Drone server does not interact with the Kubernetes API server
automountServiceAccountToken: false

Expand Down Expand Up @@ -40,7 +49,6 @@ service:
type: ClusterIP
port: 80


## If you'd like to create an ingress in front of the Drone server, you can enable it
## here. Please refer to your service provider's documenatation for any configuration
## that is specific to their ingress implementation.
Expand Down