From 3f7d1a1c1e68002fb8b1be27db63b74c4fb456a5 Mon Sep 17 00:00:00 2001 From: Valeriano Manassero <14011549+valeriano-manassero@users.noreply.github.com> Date: Wed, 19 Jul 2023 11:28:26 +0200 Subject: [PATCH] 240 fix init container waits forever pinging a redis in production config (#241) * Fixed: unused leftover * Fixed: init container fail * Changed: bump up version --- charts/clearml/Chart.yaml | 6 +++--- charts/clearml/README.md | 6 ++---- charts/clearml/values-production.yaml | 30 ++++++++++----------------- charts/clearml/values.yaml | 4 ---- 4 files changed, 16 insertions(+), 30 deletions(-) diff --git a/charts/clearml/Chart.yaml b/charts/clearml/Chart.yaml index 129ad96..1ab30a9 100644 --- a/charts/clearml/Chart.yaml +++ b/charts/clearml/Chart.yaml @@ -2,7 +2,7 @@ apiVersion: v2 name: clearml description: MLOps platform type: application -version: "7.2.2" +version: "7.2.3" appVersion: "1.11.0" kubeVersion: ">= 1.21.0-0 < 1.28.0-0" home: https://clear.ml @@ -32,5 +32,5 @@ dependencies: condition: elasticsearch.enabled annotations: artifacthub.io/changes: | - - kind: added - description: defaulr externalServices value + - kind: fixed + description: production values.yaml proposal diff --git a/charts/clearml/README.md b/charts/clearml/README.md index 0904c13..66563d4 100644 --- a/charts/clearml/README.md +++ b/charts/clearml/README.md @@ -1,6 +1,6 @@ # ClearML Ecosystem for Kubernetes -![Version: 7.2.2](https://img.shields.io/badge/Version-7.2.2-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.11.0](https://img.shields.io/badge/AppVersion-1.11.0-informational?style=flat-square) +![Version: 7.2.3](https://img.shields.io/badge/Version-7.2.3-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.11.0](https://img.shields.io/badge/AppVersion-1.11.0-informational?style=flat-square) MLOps platform @@ -145,7 +145,7 @@ Kubernetes: `>= 1.21.0-0 < 1.28.0-0` | Key | Type | Default | Description | |-----|------|---------|-------------| -| apiserver | object | `{"additionalConfigs":{},"affinity":{},"containerSecurityContext":{},"enabled":true,"existingAdditionalConfigsConfigMap":"","existingAdditionalConfigsSecret":"","extraEnvs":[],"image":{"pullPolicy":"IfNotPresent","registry":"","repository":"allegroai/clearml","tag":"1.11.0-373"},"indexReplicas":0,"indexShards":1,"ingress":{"annotations":{},"enabled":false,"hostName":"api.clearml.127-0-0-1.nip.io","ingressClassName":"","path":"/","tlsSecretName":""},"initContainers":{"resources":{"limits":{"cpu":"10m","memory":"64Mi"},"requests":{"cpu":"10m","memory":"64Mi"}}},"nodeSelector":{},"podAnnotations":{},"podSecurityContext":{},"prepopulateEnabled":true,"processes":{"count":8,"maxRequests":1000,"maxRequestsJitter":300,"timeout":24000},"replicaCount":1,"resources":{"limits":{"cpu":"2000m","memory":"1Gi"},"requests":{"cpu":"100m","memory":"256Mi"}},"service":{"annotations":{},"nodePort":30008,"port":8008,"type":"NodePort"},"tolerations":[]}` | Api Server configurations | +| apiserver | object | `{"additionalConfigs":{},"affinity":{},"containerSecurityContext":{},"enabled":true,"existingAdditionalConfigsConfigMap":"","existingAdditionalConfigsSecret":"","extraEnvs":[],"image":{"pullPolicy":"IfNotPresent","registry":"","repository":"allegroai/clearml","tag":"1.11.0-373"},"ingress":{"annotations":{},"enabled":false,"hostName":"api.clearml.127-0-0-1.nip.io","ingressClassName":"","path":"/","tlsSecretName":""},"initContainers":{"resources":{"limits":{"cpu":"10m","memory":"64Mi"},"requests":{"cpu":"10m","memory":"64Mi"}}},"nodeSelector":{},"podAnnotations":{},"podSecurityContext":{},"prepopulateEnabled":true,"processes":{"count":8,"maxRequests":1000,"maxRequestsJitter":300,"timeout":24000},"replicaCount":1,"resources":{"limits":{"cpu":"2000m","memory":"1Gi"},"requests":{"cpu":"100m","memory":"256Mi"}},"service":{"annotations":{},"nodePort":30008,"port":8008,"type":"NodePort"},"tolerations":[]}` | Api Server configurations | | apiserver.additionalConfigs | object | `{}` | files declared in this parameter will be mounted and read by apiserver (examples in values.yaml) if not overridden by existingAdditionalConfigsSecret | | apiserver.affinity | object | `{}` | Api Server affinity setup | | apiserver.containerSecurityContext | object | `{}` | Api Server containers security context | @@ -154,8 +154,6 @@ Kubernetes: `>= 1.21.0-0 < 1.28.0-0` | apiserver.existingAdditionalConfigsSecret | string | `""` | reference for files declared in existing Secret will be mounted and read by apiserver (examples in values.yaml) if not overridden by existingAdditionalConfigsConfigMap | | apiserver.extraEnvs | list | `[]` | Api Server extra envrinoment variables | | apiserver.image | object | `{"pullPolicy":"IfNotPresent","registry":"","repository":"allegroai/clearml","tag":"1.11.0-373"}` | Api Server image configuration | -| apiserver.indexReplicas | int | `0` | Number of additional replicas in Elasticsearch indexes | -| apiserver.indexShards | int | `1` | Number of shards in Elasticsearch indexes | | apiserver.ingress | object | `{"annotations":{},"enabled":false,"hostName":"api.clearml.127-0-0-1.nip.io","ingressClassName":"","path":"/","tlsSecretName":""}` | Ingress configuration for Api Server component | | apiserver.ingress.annotations | object | `{}` | Ingress annotations | | apiserver.ingress.enabled | bool | `false` | Enable/Disable ingress | diff --git a/charts/clearml/values-production.yaml b/charts/clearml/values-production.yaml index 98bec80..0f9d9ae 100755 --- a/charts/clearml/values-production.yaml +++ b/charts/clearml/values-production.yaml @@ -17,29 +17,21 @@ webserver: enabled: true hostName: "app.clearml.127-0-0-1.nip.io" redis: - master: - name: "{{ .Release.Name }}-redis" - persistence: - enabled: true - accessModes: - - ReadWriteOnce - size: 5Gi - ## If undefined (the default) or set to null, no storageClassName spec is set, choosing the default provisioner - storageClass: null - slave: - persistence: - enabled: true - accessModes: - - ReadWriteOnce - size: 5Gi - ## If undefined (the default) or set to null, no storageClassName spec is set, choosing the default provisioner - storageClass: null architecture: replication - + master: + persistence: + enabled: true + accessModes: + - ReadWriteOnce + size: 5Gi + ## If undefined (the default) or set to null, no storageClassName spec is set, choosing the default provisioner + storageClass: null + replica: + replicaCount: 2 mongodb: enabled: true architecture: replicaset - replicaCount: 3 + replicaCount: 2 arbiter: enabled: false pdb: diff --git a/charts/clearml/values.yaml b/charts/clearml/values.yaml index ef2ba7c..e1a7f1d 100644 --- a/charts/clearml/values.yaml +++ b/charts/clearml/values.yaml @@ -110,10 +110,6 @@ apiserver: maxRequestsJitter: 300 # -- Api Server extra envrinoment variables extraEnvs: [] - # -- Number of additional replicas in Elasticsearch indexes - indexReplicas: 0 - # -- Number of shards in Elasticsearch indexes - indexShards: 1 # -- specific annotation for Api Server pods podAnnotations: {} # -- Api Server resources per pod; these are minimal requirements, it's suggested to increase