Skip to content

Commit

Permalink
Remove unused port, network policy from K8s specs
Browse files Browse the repository at this point in the history
Although I'm not 100% sure, the exposed port 3000 probably is a left-over from the past, possibly when convert-document was still part of ingest-file. The network policy prevented Prometheus from scraping ingest-file metrics (and as the metrics port is now the only port exposed by ingest-file, should be otherwise unnecessary).
  • Loading branch information
tillprochaska committed Nov 22, 2023
1 parent e1e09be commit bfad22f
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions helm/charts/aleph/templates/ingest-file.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,6 @@ spec:
command:
- ingestors
- process
ports:
- containerPort: 3000
name: http
volumeMounts:
{{ if .Values.global.google }}
- mountPath: /var/secrets/google
Expand Down Expand Up @@ -108,17 +105,6 @@ spec:
maxReplicas: {{.Values.ingestfile.hpa.maxReplicas}}
metrics: {{- toYaml .Values.ingestfile.hpa.scalingMetrics | nindent 4}}
---
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
name: {{ .Values.global.namingPrefix }}-ingest-file-policy
spec:
podSelector:
matchLabels:
app: {{ .Values.global.namingPrefix }}-ingest-file
policyTypes:
- Ingress
---
{{ if .Values.global.prometheus.enabled }}
apiVersion: v1
kind: Service
Expand Down

0 comments on commit bfad22f

Please sign in to comment.