Skip to content

Commit

Permalink
Merge pull request #19 from innovatrics/feature/update-chart-to-match…
Browse files Browse the repository at this point in the history
…-release-2024-06-10

Feature/update chart to match release 2024 06 10
  • Loading branch information
egid-fertal-inno authored Jul 2, 2024
2 parents 6786823 + a402ac0 commit 2af317f
Show file tree
Hide file tree
Showing 7 changed files with 31 additions and 7 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@
- There are multiple test pods, some of which run based on what is possible to test in the selected chart configuration
- Added support for tenant management API (ingress)
- Deployment revisionHistoryLimit and updateStrategy are configurable via values
- Access Controller MQTT settings are configurable via values
- Bumped version of SmartFace Station to 1.25.1
- Bumped version of SmartFace Platform to 4.26.0
- Bumped version of Access Controller to 1.11.0

### Changed
- Bumped version of `sf-tenant-management` subchart
Expand Down
2 changes: 1 addition & 1 deletion Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ maintainers:

version: 0.8.0

appVersion: "v5_4.25.2"
appVersion: "v5_4.26.0"
9 changes: 6 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# smartface

![Version: 0.8.0](https://img.shields.io/badge/Version-0.8.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v5_4.25.2](https://img.shields.io/badge/AppVersion-v5_4.25.2-informational?style=flat-square)
![Version: 0.8.0](https://img.shields.io/badge/Version-0.8.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v5_4.26.0](https://img.shields.io/badge/AppVersion-v5_4.26.0-informational?style=flat-square)

SmartFace is a Scalable Facial Recognition Server Platform Able to Process Multiple Real-Time Video Streams. Currently the helm chart supports edge stream and Lightweight Face Identification System (LFIS) deployments

Expand Down Expand Up @@ -149,7 +149,10 @@ metadata:
| accessController.image.pullPolicy | string | `"IfNotPresent"` | Docker image pull policy |
| accessController.image.registry | string | `nil` | The Docker registry, overrides `global.image.registry` |
| accessController.image.repository | string | `"innovatrics/smartface/sf-access-controller"` | Docker image repository |
| accessController.image.tag | string | `"v5_1.9.1"` | Access Controller follows different versioning, so the chart app needs to be overridden |
| accessController.image.tag | string | `"v5_1.11.0"` | Access Controller follows different versioning, so the chart app needs to be overridden |
| accessController.mqttConfig.enabled | bool | `false` | |
| accessController.mqttConfig.sendImageData | bool | `false` | |
| accessController.mqttConfig.topic | string | `"edge-stream/{sourceId}/access-notifications/{notificationType}"` | |
| accessController.name | string | `"access-controller"` | |
| accessController.nodeSelector | object | `{}` | |
| accessController.podAnnotations | object | `{}` | Annotations for accessController pods |
Expand Down Expand Up @@ -532,7 +535,7 @@ metadata:
| station.image.pullPolicy | string | `"IfNotPresent"` | Docker image pull policy |
| station.image.registry | string | `nil` | The Docker registry, overrides `global.image.registry` |
| station.image.repository | string | `"innovatrics/smartface/sf-station"` | Docker image repository |
| station.image.tag | string | `"v5_1.24.3"` | Smartface Station follows different versioning, so the chart app needs to be overridden |
| station.image.tag | string | `"v5_1.25.1"` | Smartface Station follows different versioning, so the chart app needs to be overridden |
| station.name | string | `"station"` | |
| station.nodeSelector | object | `{}` | |
| station.podAnnotations | object | `{}` | Annotations for station pods |
Expand Down
4 changes: 4 additions & 0 deletions ci/multitenant-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,7 @@ api:
sf-tenant-management:
enabled: true
installCrd: true

dbSynchronizationLeader:
enabled: true
enableAuth: true
7 changes: 7 additions & 0 deletions templates/edge-streams/access-controller-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ spec:
{{- include "smartface.commonEnv" . | nindent 8 }}
{{- include "smartface.rmqConfig" . | nindent 8 }}
{{- include "smartface.authenticationConfig" . | nindent 8 }}
{{- include "smartface.mqttConfig" . | nindent 8 }}
- name: "Kestrel__EndPoints__Http__Url"
value: "http://0.0.0.0:{{ .Values.accessController.containerPort }}"
- name: "Kestrel__EndPoints__Http__RequireAuthentication"
Expand All @@ -67,6 +68,12 @@ spec:
value: "http://0.0.0.0:{{ .Values.accessController.authContainerPort }}"
- name: "Kestrel__EndPoints__HttpAuth__RequireAuthentication"
value: "true"
- name: "MQTT__Enabled"
value: {{ .Values.accessController.mqttConfig.enabled | quote }}
- name: "MQTT__Topic"
value: {{ .Values.accessController.mqttConfig.topic | quote }}
- name: "MQTT__SendImageData"
value: {{ .Values.accessController.mqttConfig.sendImageData | quote }}
resources:
{{- toYaml .Values.accessController.resources | nindent 10 }}
volumeMounts:
Expand Down
4 changes: 3 additions & 1 deletion templates/tests/multitenant-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,14 @@ spec:
- name: {{ $name | quote }}
image: {{ include "smartface.image" (dict "local" .Values.tests.image "global" .Values.global.image "defaultVersion" .Chart.AppVersion) }}
imagePullPolicy: {{ .Values.tests.image.pullPolicy }}
args: ['--filter', 'Category=Multitenant']
args: ['--filter', 'Category=Multitenant{{- if not .Values.dbSynchronizationLeader.enabled }}&FullyQualifiedName!=SmartFace.CloudFunctionalTests.Multitenant.CloudLeaderMultitenantWlSyncTests.Wl_Updates_For_Tenants_Smoke_Test{{- end}}']
env:
- name: "SF_TEST_API_URL"
value: {{ $apiUrl | quote }}
- name: "SF_TEST_GQL_API_URL"
value: "http://{{ include "smartface.graphqlApi.name" . }}.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.graphqlApi.servicePort }}"
- name: "DBSYNC_LEADER_TEST_API_URL"
value: "http://{{ include "smartface.dbSynchronizationLeader.name" . }}.{{ .Release.Namespace }}.svc.cluster.local:{{ .Values.dbSynchronizationLeader.servicePort }}"
- name: "SF_API_METRICS_URL"
value: {{ $apiMetricsUrl | quote }}
{{- include "smartface.testsAuthConfig" . | nindent 4 }}
Expand Down
8 changes: 6 additions & 2 deletions values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -639,11 +639,15 @@ accessController:
# -- Docker image repository
repository: innovatrics/smartface/sf-access-controller
# -- Access Controller follows different versioning, so the chart app needs to be overridden
tag: "v5_1.9.1"
tag: "v5_1.11.0"
# -- Overrides the image tag with an image digest
digest: null
# -- Docker image pull policy
pullPolicy: IfNotPresent
mqttConfig:
enabled: false
topic: "edge-stream/{sourceId}/access-notifications/{notificationType}"
sendImageData: false

station:
enabled: false
Expand Down Expand Up @@ -674,7 +678,7 @@ station:
# -- Docker image repository
repository: innovatrics/smartface/sf-station
# -- Smartface Station follows different versioning, so the chart app needs to be overridden
tag: "v5_1.24.3"
tag: "v5_1.25.1"
# -- Overrides the image tag with an image digest
digest: null
# -- Docker image pull policy
Expand Down

0 comments on commit 2af317f

Please sign in to comment.