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

Feat(CR-27391): add new option to app-proxy #403

Merged
merged 15 commits into from
Feb 20, 2025
Merged
Changes from 13 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
1 change: 1 addition & 0 deletions charts/gitops-runtime/README.md
Original file line number Diff line number Diff line change
@@ -90,6 +90,7 @@ sealed-secrets:
| app-proxy.config.env | string | `"production"` | |
| app-proxy.config.logLevel | string | `"info"` | Log Level |
| app-proxy.config.skipGitPermissionValidation | string | `"false"` | Skit git permissions validation |
| app-proxy.config.clusterChunkSize | int | `0` | Number of clusters per request. App-proxy will split and refresh the cluster list with chunks with size `clusterChunkSize`. When the default value is used the cluster list will be refreshed with one chunk (no split; not recommended when cluster list is too large). |
| app-proxy.env | object | `{}` | |
| app-proxy.extraVolumeMounts | list | `[]` | Extra volume mounts for main container |
| app-proxy.extraVolumes | list | `[]` | extra volumes |
Original file line number Diff line number Diff line change
@@ -4,6 +4,9 @@ argoCdUsername: {{ .Values.config.argoCdUsername }}
argoWorkflowsInsecure: {{ .Values.config.argoWorkflowsInsecure | quote }}
argoWorkflowsUrl: {{ default "" .Values.config.argoWorkflowsUrl }}
cors: {{ .Values.global.codefresh.url }}
{{- if ((.Values.config).clusterChunkSize) }}
clusterChunkSize: {{ .Values.config.clusterChunkSize | quote }}
{{- end }}
env: {{ .Values.config.env | quote}}
isConfigurationRuntime: {{ .Values.global.runtime.isConfigurationRuntime | quote }}
runtimeName: {{ required "global.runtime.name is required" .Values.global.runtime.name | quote}}
Original file line number Diff line number Diff line change
@@ -36,6 +36,12 @@ CF_HOST:
key: base-url
optional: true
CLUSTER: https://kubernetes.default.svc
CLUSTER_CHUNK_SIZE:
valueFrom:
configMapKeyRef:
name: cap-app-proxy-cm
key: clusterChunkSize
optional: true
CORS:
valueFrom:
configMapKeyRef:
5 changes: 5 additions & 0 deletions charts/gitops-runtime/values.yaml
Original file line number Diff line number Diff line change
@@ -481,6 +481,11 @@ app-proxy:
skipGitPermissionValidation: "false"
# -- Log Level
logLevel: "info"
# -- define cluster list size per request to report the cluster state to platform, e.g.
# if you have 90 clusters and set clusterChunkSize: 40, it means cron job will report cluster state to platform in 3 iterations (40,40,10)
# - reduce this value if you have a lot of clusters and the cron job is failing with payload too large error
# - use 0 to sync all clusters at once
clusterChunkSize: 50

env: {}

5 changes: 3 additions & 2 deletions installer-image/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
FROM debian:bullseye-slim
#bookworm-slim
FROM debian:12.9-slim

ARG CF_CLI_VERSION=v0.1.70
ARG TARGETARCH
@@ -9,4 +10,4 @@ COPY --from=bitnami/kubectl:1.32.1 /opt/bitnami/kubectl/bin/kubectl /usr/local/b

RUN adduser --shell /bin/bash codefresh
USER codefresh
WORKDIR /home/codefresh
WORKDIR /home/codefresh