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.

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

refactor: rename submariner-engine to submariner-gateway #122

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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 Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ preload-images:
source $(SCRIPTS_DIR)/lib/debug_functions; \
source $(SCRIPTS_DIR)/lib/deploy_funcs; \
set -e; \
for image in submariner submariner-route-agent submariner-operator lighthouse-agent submariner-globalnet lighthouse-coredns; do \
for image in submariner-gateway submariner-route-agent submariner-operator lighthouse-agent submariner-globalnet lighthouse-coredns; do \
import_image quay.io/submariner/$${image}; \
done

Expand Down
2 changes: 1 addition & 1 deletion submariner-operator/crds/crd.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ spec:
type: string
colorCodes:
type: string
engineDaemonSetStatus:
gatewayDaemonSetStatus:
properties:
lastResourceVersion:
type: string
Expand Down
2 changes: 1 addition & 1 deletion submariner-operator/templates/NOTES.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Submariner is now installed.

{{- if .Values.engine.nodeSelectorEnabled }}
{{- if .Values.gateway.nodeSelectorEnabled }}
If you haven't done so yet, please label a node as `submariner.io/gateway=true` to elect it for running Submariner.
{{- end }}

Expand Down
10 changes: 5 additions & 5 deletions submariner-operator/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,13 @@ Create the name of the submariner-operator service account to use
{{- end -}}

{{/*
Create the name of the submariner-engine service account to use
Create the name of the submariner-gateway service account to use
*/}}
{{- define "submariner.engineServiceAccountName" -}}
{{- if .Values.serviceAccounts.engine.create -}}
{{ default "submariner-engine" .Values.serviceAccounts.engine.name }}
{{- define "submariner.gatewayServiceAccountName" -}}
{{- if .Values.serviceAccounts.gateway.create -}}
{{ default "submariner-gateway" .Values.serviceAccounts.gateway.name }}
{{- else -}}
{{ default "default" .Values.serviceAccounts.engine.name }}
{{ default "default" .Values.serviceAccounts.gateway.name }}
{{- end -}}
{{- end -}}

Expand Down
2 changes: 1 addition & 1 deletion submariner-operator/templates/operator-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ metadata:
release: {{ .Release.Name | quote }}
chart: {{ template "submariner.chart" . }}
app: {{ template "submariner.fullname" . }}
component: engine
component: gateway
name: {{ template "submariner.fullname" . }}
spec:
progressDeadlineSeconds: 600
Expand Down
16 changes: 8 additions & 8 deletions submariner-operator/templates/rbac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ roleRef:
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand Down Expand Up @@ -166,7 +166,7 @@ rules:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand All @@ -175,10 +175,10 @@ metadata:
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
subjects:
- kind: ServiceAccount
name: {{ template "submariner.engineServiceAccountName" . }}
name: {{ template "submariner.gatewayServiceAccountName" . }}
namespace: {{ .Release.Namespace }}
---
apiVersion: rbac.authorization.k8s.io/v1
Expand Down Expand Up @@ -458,7 +458,7 @@ roleRef:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand Down Expand Up @@ -525,7 +525,7 @@ rules:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand All @@ -534,10 +534,10 @@ metadata:
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
subjects:
- kind: ServiceAccount
name: {{ template "submariner.engineServiceAccountName" . }}
name: {{ template "submariner.gatewayServiceAccountName" . }}
namespace: {{ .Release.Namespace }}
---
apiVersion: rbac.authorization.k8s.io/v1
Expand Down
4 changes: 2 additions & 2 deletions submariner-operator/templates/svc-acct.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@ metadata:
app: {{ template "submariner.name" . }}
{{- end }}
---
{{- if .Values.serviceAccounts.engine.create }}
{{- if .Values.serviceAccounts.gateway.create }}
apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "submariner.engineServiceAccountName" . }}
name: {{ template "submariner.gatewayServiceAccountName" . }}
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand Down
6 changes: 3 additions & 3 deletions submariner-operator/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,15 +37,15 @@ operator:
resources: {}
tolerations: []
affinity: {}
engine:
gateway:
image:
repository: quay.io/submariner/submariner
repository: quay.io/submariner/submariner-gateway
tag: "0.7.0"
serviceAccounts:
operator:
create: true
name: ""
engine:
gateway:
create: true
name: ""
routeAgent:
Expand Down
22 changes: 11 additions & 11 deletions submariner/questions.yml
Original file line number Diff line number Diff line change
@@ -1,22 +1,22 @@
questions:
- variable: defaultEngineImage
- variable: defaultGatewayImage
default: true
description: "Use default Submariner Engine image or specify a custom one"
label: Use default submariner engine image
description: "Use default Submariner Gateway image or specify a custom one"
label: Use default submariner gateway image
type: boolean
show_subquestion_if: false
group: "Container Images"
subquestions:
- variable: engine.image.repository
default: "quay.io/submariner/submariner"
description: "Submariner Engine Image Repository"
- variable: gateway.image.repository
default: "quay.io/submariner/submariner-gateway"
description: "Submariner Gateway Image Repository"
type: string
label: Submariner Engine Image Repository
- variable: engine.image.tag
label: Submariner Gateway Image Repository
- variable: gateway.image.tag
default: "0.6.0"
description: "Submariner Engine Image Tag"
description: "Submariner Gateway Image Tag"
type: string
label: Submariner Engine Image Tag
label: Submariner Gateway Image Tag
- variable: defaultRouteAgentImage
default: true
description: "Use default Submariner Route Agent image or specify a custom one"
Expand All @@ -35,7 +35,7 @@ questions:
description: "Submariner Route Agent Image Tag"
type: string
label: Submariner Route Agent Image Tag
- variable: engine.nodeSelectorEnabled
- variable: gateway.nodeSelectorEnabled
default: true
description: "Restrict submariner to nodes labeled with submariner.io/gateway=true"
label: Restrict gateway deployments to specific nodes
Expand Down
2 changes: 1 addition & 1 deletion submariner/templates/NOTES.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Submariner is now installed.

{{- if .Values.engine.nodeSelectorEnabled }}
{{- if .Values.gateway.nodeSelectorEnabled }}
If you haven't done so yet, please label a node as `submariner.io/gateway=true` to elect it for running Submariner.
{{- end }}

Expand Down
10 changes: 5 additions & 5 deletions submariner/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -32,13 +32,13 @@ Create chart name and version as used by the chart label.
{{- end -}}

{{/*
Create the name of the submariner-engine service account to use
Create the name of the submariner-gateway service account to use
*/}}
{{- define "submariner.engineServiceAccountName" -}}
{{- if .Values.serviceAccounts.engine.create -}}
{{ default "submariner-engine" .Values.serviceAccounts.engine.name }}
{{- define "submariner.gatewayServiceAccountName" -}}
{{- if .Values.serviceAccounts.gateway.create -}}
{{ default "submariner-gateway" .Values.serviceAccounts.gateway.name }}
{{- else -}}
{{ default "default" .Values.serviceAccounts.engine.name }}
{{ default "default" .Values.serviceAccounts.gateway.name }}
{{- end -}}
{{- end -}}

Expand Down
26 changes: 13 additions & 13 deletions submariner/templates/engine-deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,14 @@ metadata:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
chart: {{ template "submariner.chart" . }}
app: {{ template "submariner.fullname" . }}-engine
component: engine
app: {{ template "submariner.fullname" . }}-gateway
component: gateway
name: {{ template "submariner.fullname" . }}-gateway
spec:
revisionHistoryLimit: 5
selector:
matchLabels:
app: {{ template "submariner.fullname" . }}-engine
app: {{ template "submariner.fullname" . }}-gateway
updateStrategy:
rollingUpdate:
maxUnavailable: 1
Expand All @@ -21,7 +21,7 @@ spec:
metadata:
creationTimestamp: null
labels:
app: {{ template "submariner.fullname" . }}-engine
app: {{ template "submariner.fullname" . }}-gateway
spec:
affinity:
podAntiAffinity:
Expand All @@ -31,19 +31,19 @@ spec:
- key: app
operator: In
values:
- {{ template "submariner.fullname" . }}-engine
- {{ template "submariner.fullname" . }}-gateway
topologyKey: "kubernetes.io/hostname"
{{- with .Values.engine.affinity }}
{{- with .Values.gateway.affinity }}
{{ toYaml . | indent 8 }}
{{- end }}
nodeSelector:
{{- if .Values.engine.nodeSelectorEnabled }}
{{- if .Values.gateway.nodeSelectorEnabled }}
submariner.io/gateway: "true"
{{- end }}
{{- with .Values.engine.nodeSelector }}
{{- with .Values.gateway.nodeSelector }}
{{ toYaml . | indent 8 }}
{{- end }}
{{- with .Values.engine.tolerations }}
{{- with .Values.gateway.tolerations }}
tolerations:
{{ toYaml . | indent 8 }}
{{- end }}
Expand Down Expand Up @@ -112,11 +112,11 @@ spec:
valueFrom:
fieldRef:
fieldPath: "spec.nodeName"
image: {{ .Values.engine.image.repository }}:{{ default .Chart.AppVersion .Values.engine.image.tag }}
imagePullPolicy: {{ .Values.engine.image.pullPolicy }}
image: {{ .Values.gateway.image.repository }}:{{ default .Chart.AppVersion .Values.gateway.image.tag }}
imagePullPolicy: {{ .Values.gateway.image.pullPolicy }}
name: submariner
resources:
{{ toYaml .Values.engine.resources | indent 10 }}
{{ toYaml .Values.gateway.resources | indent 10 }}
securityContext:
allowPrivilegeEscalation: true
capabilities:
Expand All @@ -135,4 +135,4 @@ spec:
schedulerName: default-scheduler
securityContext: {}
terminationGracePeriodSeconds: 1
serviceAccountName: {{ template "submariner.engineServiceAccountName" . }}
serviceAccountName: {{ template "submariner.gatewayServiceAccountName" . }}
16 changes: 8 additions & 8 deletions submariner/templates/rbac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand Down Expand Up @@ -82,7 +82,7 @@ rules:
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand All @@ -91,10 +91,10 @@ metadata:
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: Role
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
subjects:
- kind: ServiceAccount
name: {{ template "submariner.engineServiceAccountName" . }}
name: {{ template "submariner.gatewayServiceAccountName" . }}
namespace: {{ .Release.Namespace }}
---
apiVersion: rbac.authorization.k8s.io/v1
Expand Down Expand Up @@ -299,7 +299,7 @@ subjects:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand Down Expand Up @@ -366,7 +366,7 @@ rules:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand All @@ -375,10 +375,10 @@ metadata:
roleRef:
apiGroup: rbac.authorization.k8s.io
kind: ClusterRole
name: {{ template "submariner.fullname" . }}:engine
name: {{ template "submariner.fullname" . }}:gateway
subjects:
- kind: ServiceAccount
name: {{ template "submariner.engineServiceAccountName" . }}
name: {{ template "submariner.gatewayServiceAccountName" . }}
namespace: {{ .Release.Namespace }}
---
apiVersion: rbac.authorization.k8s.io/v1
Expand Down
4 changes: 2 additions & 2 deletions submariner/templates/svc-acct.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
{{- if .Values.serviceAccounts.engine.create }}
{{- if .Values.serviceAccounts.gateway.create }}
apiVersion: v1
kind: ServiceAccount
metadata:
name: {{ template "submariner.engineServiceAccountName" . }}
name: {{ template "submariner.gatewayServiceAccountName" . }}
labels:
heritage: {{ .Release.Service | quote }}
release: {{ .Release.Name | quote }}
Expand Down
6 changes: 3 additions & 3 deletions submariner/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,9 @@ leadership:
leaseDuration: 10
renewDeadline: 5
retryPeriod: 2
engine:
gateway:
image:
repository: quay.io/submariner/submariner
repository: quay.io/submariner/submariner-gateway
tag: ""
pullPolicy: IfNotPresent
resources: {}
Expand Down Expand Up @@ -65,7 +65,7 @@ lighthouseCoredns:
tag: ""
pullPolicy: IfNotPresent
serviceAccounts:
engine:
gateway:
create: true
name: ""
routeAgent:
Expand Down