Skip to content

Commit

Permalink
254 clearml server helper function wrong readiness probes fail becaus…
Browse files Browse the repository at this point in the history
…e wrong redis service is referenced (#257)

* Fixed: redis name generation

* Changed: bump up version
  • Loading branch information
valeriano-manassero authored Oct 25, 2023
1 parent 0892882 commit 5ed9673
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 8 deletions.
6 changes: 3 additions & 3 deletions charts/clearml/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: clearml
description: MLOps platform
type: application
version: "7.4.0"
version: "7.4.1"
appVersion: "1.12.0"
kubeVersion: ">= 1.21.0-0 < 1.29.0-0"
home: https://clear.ml
Expand Down Expand Up @@ -32,5 +32,5 @@ dependencies:
condition: elasticsearch.enabled
annotations:
artifacthub.io/changes: |
- kind: added
description: service accounts for core clearml components
- kind: fixed
description: redis service name reference generation
2 changes: 1 addition & 1 deletion charts/clearml/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# ClearML Ecosystem for Kubernetes

![Version: 7.4.0](https://img.shields.io/badge/Version-7.4.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.12.0](https://img.shields.io/badge/AppVersion-1.12.0-informational?style=flat-square)
![Version: 7.4.1](https://img.shields.io/badge/Version-7.4.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.12.0](https://img.shields.io/badge/AppVersion-1.12.0-informational?style=flat-square)

MLOps platform

Expand Down
4 changes: 0 additions & 4 deletions charts/clearml/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -216,12 +216,8 @@ Redis Service name
*/}}
{{- define "redis.servicename" -}}
{{- if .Values.redis.enabled }}
{{- if eq .Values.redis.architecture "standalone" }}
{{- tpl .Values.redis.master.name . }}
{{- else }}
{{- printf "%s-headless" (tpl .Values.redis.master.name . ) }}
{{- end }}
{{- else }}
{{- .Values.externalServices.redisHost }}
{{- end }}
{{- end }}
Expand Down

0 comments on commit 5ed9673

Please sign in to comment.