Skip to content

Commit

Permalink
Merge pull request #100 from ahreehong/enable-relay
Browse files Browse the repository at this point in the history
Allow tink-stack relay to be disbaled

## Description

<!--- Please describe what this PR is going to change -->

## Why is this needed

Fixes: #98

## How Has This Been Tested?
Tested locally with EKS-Anywhere and verified that the tink-stack relay container does not start when `enabled: false`

## How are existing users impacted? What migration steps/scripts do we need?

This fixes a bug where `stack.relay.enabled: false` gets ignored
  • Loading branch information
jacobweinstock authored Jun 4, 2024
2 parents a6f0454 + 7ef45cf commit 4975f50
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion tinkerbell/stack/templates/nginx.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@ spec:
- mountPath: /usr/share/nginx/html
name: hook-artifacts
{{- end }}
{{- if .Values.stack.relay.enabled }}
- name: {{ .Values.stack.relay.name }}
image: {{ .Values.stack.relay.image }}
args: ["-m", "{{ .Values.stack.relay.presentGiaddrAction }}", "-c", "{{ .Values.stack.relay.maxHopCount }}", "-id", "{{ $dhcpInterfaceName }}", "-iu", "eth0", "-U", "eth0", "smee.{{ .Release.Namespace }}.svc.{{ .Values.stack.clusterDomain }}."]
Expand All @@ -102,6 +103,7 @@ spec:
capabilities:
add:
- NET_RAW
{{- end }}
volumes:
- name: nginx-conf
configMap:
Expand All @@ -115,7 +117,7 @@ spec:
path: {{ .Values.stack.hook.downloadsDest }}
type: DirectoryOrCreate
{{- end }}
{{- if $listenBroadcast }}
{{- if and .Values.stack.relay.enabled $listenBroadcast }}
initContainers:
- name: relay-broadcast-interface
command:
Expand Down

0 comments on commit 4975f50

Please sign in to comment.