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

update serviceName to use headless service #88

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

update serviceName to use headless service #88

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
acc5478
update serviceName to use headless service
chris-vest Nov 15, 2024
8cf3650
update serviceName to use headless service
chris-vest Nov 15, 2024
8c76417
add annotations to headless serviec
chris-vest Nov 20, 2024
c001857
Update README and values.schema.json for modified charts
chris-vest Nov 20, 2024
f3a0f63
add annotations to headless serviec
chris-vest Nov 20, 2024
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/nitro/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ maintainers:

type: application

version: 0.6.20
version: 0.6.21

appVersion: "v3.2.1-d81324d"
5 changes: 3 additions & 2 deletions charts/nitro/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,13 @@ helm install xai offchainlabs/nitro -f values.yaml
| `serviceMonitor.relabelings` | Add relabelings for the metrics being scraped | `[]` |
| `perReplicaService.enabled` | Enable a service for each sts replica | `false` |
| `perReplicaService.publishNotReadyAddresses` | Publish not ready addresses | `true` |
| `perReplicaService.annotations` | Annotations for the service | `{}` |
| `perReplicaService.annotations` | Annotations for the per replica service | `{}` |
| `perReplicaHeadlessService.enabled` | Enable a headless service for each sts replica | `false` |
| `perReplicaHeadlessService.publishNotReadyAddresses` | Publish not ready addresses | `true` |
| `perReplicaHeadlessService.annotations` | Annotations for the headless service | `{}` |
| `perReplicaHeadlessService.annotations` | Annotations for the per replica headless service | `{}` |
| `headlessservice.enabled` | Enable headless service | `true` |
| `headlessservice.publishNotReadyAddresses` | Publish not ready addresses | `true` |
| `headlessservice.annotations` | Annotations for the headless service | `{}` |
| `jwtSecret.enabled` | Enable a jwt secret for use with the stateless validator | `false` |
| `jwtSecret.value` | Value of the jwt secret for use with the stateless validator | `""` |
| `auth.enabled` | Enable auth for the stateless validator | `false` |
Expand Down
4 changes: 4 additions & 0 deletions charts/nitro/templates/headlessservice.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ metadata:
name: {{ include "nitro.fullname" . }}-headless
labels:
{{- include "nitro.labels" . | nindent 4 }}
annotations:
{{- if $.Values.headlessservice.annotations }}
{{- toYaml $.Values.headlessservice.annotations | nindent 4 }}
{{- end }}
spec:
type: ClusterIP
clusterIP: None
Expand Down
6 changes: 3 additions & 3 deletions charts/nitro/templates/statefulset.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,15 @@ metadata:
annotations:
nitro.arbitrum.io/desiredReplicas: {{ $.Values.replicaCount | quote }}
spec:
serviceName: "{{ include "nitro.fullname" . }}"
serviceName: {{ include "nitro.fullname" . }}-headless
replicas: {{ .Values.replicaCount }}
selector:
matchLabels:
{{- include "nitro.selectorLabels" . | nindent 6 }}
podManagementPolicy: Parallel
{{- if .Values.updateStrategy }}
{{- with .Values.updateStrategy }}
updateStrategy:
updateStrategy:
{{- toYaml . | nindent 4 }}
{{- end }}
{{- end }}
Expand Down Expand Up @@ -276,7 +276,7 @@ spec:
app: {{ template "nitro.name" . }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
spec:
spec:
accessModes:
{{- toYaml .Values.blobPersistence.accessModes | nindent 8 }}
{{- if .Values.blobPersistence.storageClassName }}
Expand Down
6 changes: 4 additions & 2 deletions charts/nitro/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,25 +100,27 @@ serviceMonitor:

## @param perReplicaService.enabled Enable a service for each sts replica
## @param perReplicaService.publishNotReadyAddresses Publish not ready addresses
## @param perReplicaService.annotations Annotations for the service
## @param perReplicaService.annotations Annotations for the per replica service
perReplicaService:
enabled: false
publishNotReadyAddresses: true
annotations: {}

## @param perReplicaHeadlessService.enabled Enable a headless service for each sts replica
## @param perReplicaHeadlessService.publishNotReadyAddresses Publish not ready addresses
## @param perReplicaHeadlessService.annotations Annotations for the headless service
## @param perReplicaHeadlessService.annotations Annotations for the per replica headless service
perReplicaHeadlessService:
enabled: false
publishNotReadyAddresses: true
annotations: {}

## @param headlessservice.enabled Enable headless service
## @param headlessservice.publishNotReadyAddresses Publish not ready addresses
## @param headlessservice.annotations Annotations for the headless service
headlessservice:
enabled: true
publishNotReadyAddresses: true
annotations: {}

## @param jwtSecret.enabled Enable a jwt secret for use with the stateless validator
## @param jwtSecret.value Value of the jwt secret for use with the stateless validator
Expand Down
Loading