Skip to content

Commit

Permalink
fix: optimize mirror node deployment and fix resources for root-conta…
Browse files Browse the repository at this point in the history
…iner (#813)

Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
  • Loading branch information
leninmehedy committed Apr 4, 2024
1 parent 5f886e7 commit dbffd87
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 23 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
{{- $balanceUploader := ($node.sidecars).accountBalanceUploader | default $.Values.defaults.sidecars.accountBalanceUploader -}}
{{- $backupUploader := ($node.sidecars).backupUploader | default $.Values.defaults.sidecars.backupUploader -}}
{{- $otelCollector := ($node.sidecars).otelCollector | default $.Values.defaults.sidecars.otelCollector -}}
{{- $resources := $node.resources | default $.Values.defaults.resources }}
{{- $cloud := $.Values.cloud }}
{{- $defaults := $.Values.defaults }}
{{- $minioserver := (index $.Values "minio-server") }}
Expand Down Expand Up @@ -90,7 +89,7 @@ spec:
imagePullPolicy: {{ include "fullstack.images.pullPolicy" (dict "image" $root.image "defaults" $root) }}
securityContext: # need to run as root with privileged mode
{{- include "fullstack.root.security.context.privileged" . | nindent 10 }}
{{- with $resources }}
{{- with default $defaults.root.resources $node.resources }}
resources:
{{- toYaml . | nindent 10 }}
{{- end }}
Expand Down
24 changes: 3 additions & 21 deletions charts/fullstack-deployment/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,13 +63,6 @@ defaults:
dataStats: "50Gi"
dataUpgrade: "5Gi"
output: "5Gi"
resources:
requests:
cpu: 100m
memory: 100Mi
limits:
cpu: 1
memory: 2G
root: # root container
image:
registry: "ghcr.io"
Expand Down Expand Up @@ -140,7 +133,7 @@ defaults:
prioritize: true
resources: {}
accountBalanceUploader:
enabled: true
enabled: false
nameOverride: "account-balance-uploader"
image:
registry: gcr.io
Expand Down Expand Up @@ -251,10 +244,10 @@ hedera-mirror-node:
enabled: false
rosetta: # not needed for default FST use case
enabled: false
monitor: # not needed for default FST use case
enabled: false
redis:
enabled: false # not needed for default FST use case
web3:
enabled: false
global:
namespaceOverride: "{{ tpl (.Values.global.namespaceOverride | toString) }}"

Expand Down Expand Up @@ -354,17 +347,6 @@ hedera-mirror-node:
effect: "NoSchedule"
monitor:
enabled: false
web3:
nodeSelector: {}
tolerations:
- key: "fullstack-scheduling.io/os"
operator: "Equal"
value: "linux"
effect: "NoSchedule"
- key: "fullstack-scheduling.io/role"
operator: "Equal"
value: "network"
effect: "NoSchedule"

haproxy-ingress:
controller:
Expand Down

0 comments on commit dbffd87

Please sign in to comment.