Skip to content

Commit

Permalink
Merge pull request #94 from warpstreamlabs/ra/validate-config
Browse files Browse the repository at this point in the history
Validate required options on apply
  • Loading branch information
richardartoul authored Aug 27, 2024
2 parents f6b32db + 8c8d54d commit fb6515e
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 1 deletion.
6 changes: 6 additions & 0 deletions charts/warpstream-agent/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [0.13.16] - 2024-08-27

### Added

- Throw error if any required configuration options are empty.

## [0.13.15] - 2024-08-25

### Added
Expand Down
2 changes: 1 addition & 1 deletion charts/warpstream-agent/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: warpstream-agent
description: WarpStream Agent for Kubernetes.
type: application
version: 0.13.16
version: 0.13.17
appVersion: v577
icon: https://avatars.githubusercontent.com/u/132156278
home: https://docs.warpstream.com/warpstream/
Expand Down
9 changes: 9 additions & 0 deletions charts/warpstream-agent/templates/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,15 @@ metadata:
labels:
{{- include "warpstream-agent.labels" . | nindent 4 }}
data:
{{- if not .Values.config.bucketURL }}
{{- fail "bucketURL configuration value must be set" }}
{{- end }}
WARPSTREAM_BUCKET_URL: {{ .Values.config.bucketURL | quote }}
{{- if not .Values.config.virtualClusterID }}
{{- fail "virtualClusterID configuration value must be set" }}
{{- end }}
WARPSTREAM_DEFAULT_VIRTUAL_CLUSTER_ID: {{ .Values.config.virtualClusterID | quote }}
{{- if not .Values.config.region }}
{{- fail "region configuration value must be set" }}
{{- end }}
WARPSTREAM_REGION: {{ .Values.config.region | quote }}

0 comments on commit fb6515e

Please sign in to comment.