Skip to content
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

Jump to bottom

Feat: Deploy only operator with external connect url #166

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Feat: Deploy only operator with external connect url #166

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
caef167
Added an option for providing the url to the connect server when the …
Jul 16, 2023
b6e30ed
Added description to readme.
Jul 16, 2023
eb7af18
Merge branch 'main' into feature/custom-operator-url
Aug 19, 2023
165a909
Readded documentation.
Aug 19, 2023
cfe99e0
Added proper if statement.
Aug 19, 2023
16ef34d
It no works with () around and statement.
Aug 19, 2023
a6720bd
Added connectUrl documentation.
Aug 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 3 additions & 1 deletion charts/connect/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ helm install --set connect.applicationName=connect connect ./connect
| operator.nodeSelector | object | `{}` | [Node selector](https://kubernetes.io/docs/concepts/scheduling-eviction/assign-pod-node/#nodeselector) stanza for the operator pod |
| operator.annotations | object | `{}` | Additional annotations to be added to the Operator deployment resource. |
| operator.labels | object | `{}` | Additional labels to be added to the Operator deployment resource. |
| operator.logLevel | string | `info` | Log level of the Operator container. Valid options are: debug, info and error. |
| operator.logLevel | string | `info` | Log level of the Operator container. Valid options are: debug, info and error. |
| operator.podAnnotations | object | `{}` | Additional annotations to be added to the Operator pods. |
| operator.podLabels | object | `{}` | Additional labels to be added to the Operator pods. |
| operator.pollingInterval | integer | `600` | How often the 1Password Operator will poll for secrets updates. |
Expand All @@ -126,6 +126,8 @@ helm install --set connect.applicationName=connect connect ./connect
| operator.token.value | string | `"onepassword-token"` | An API token generated for 1Password Connect to be used by the Connect Operator |
| operator.watchNamespace | list | `[]` | A list of namespaces for the 1Password Operator to watch and manage. Use the empty list to watch all namespaces. |
| operator.resources | object | `{}` | The resources requests/limits for the 1Password Operator pod |
| operator.connectUrl | string | | The url to the connect server with a schema, when a connect server is not deployed by this chart |


### CRD

Expand Down
4 changes: 4 additions & 0 deletions charts/connect/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,12 +51,16 @@ app.kubernetes.io/instance: {{ .Release.Name }}
{{- end }}

{{- define "onepassword-connect.url" -}}
{{- if and (.Values.operator.connectUrl) (not .Values.connect.create) -}}
{{ .Values.operator.connectUrl }}
{{- else -}}
{{- if .Values.connect.tls.enabled -}}
https://{{ .Values.connect.host }}:{{ .Values.connect.api.httpsPort }}
{{- else -}}
http://{{ .Values.connect.host }}:{{ .Values.connect.api.httpPort }}
{{- end }}
{{- end }}
{{- end }}

{{/*
loadBalancer configuration for the the 1Password API and Sync service.
Expand Down