Skip to content

Commit

Permalink
feat(helm): add config for revisionHistoryLimit and reuse resources i…
Browse files Browse the repository at this point in the history
…n initContainer

Signed-off-by: t3mi <[email protected]>
  • Loading branch information
t3mi committed Aug 5, 2024
1 parent ecfdd57 commit 813a820
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 0 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Added

- [#538](https://github.com/spegel-org/spegel/pull/538) Replace mock OCI client with in memory client.
- [#553](https://github.com/spegel-org/spegel/pull/553) Add configuration for revisionHistoryLimit in the Helm Chart.

### Changed

Expand All @@ -18,6 +19,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- [#520](https://github.com/spegel-org/spegel/pull/520) Add tests for metrics.
- [#536](https://github.com/spegel-org/spegel/pull/536) Update Go version to 1.22.5.
- [#547](https://github.com/spegel-org/spegel/pull/547) Set blob content type to disable detection.
- [#553](https://github.com/spegel-org/spegel/pull/553) Re-use resources value for initContainer in the Helm Chart.

### Deprecated

Expand Down
1 change: 1 addition & 0 deletions charts/spegel/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ spec:
| podSecurityContext | object | `{}` | Security context for the pod. |
| priorityClassName | string | `"system-node-critical"` | Priority class name to use for the pod. |
| resources | object | `{}` | Resource requests and limits for the Spegel container. |
| revisionHistoryLimit | int | `10` | The number of old history to retain to allow rollback. |
| securityContext | object | `{}` | Security context for the Spegel container. |
| service.metrics.port | int | `9090` | Port to expose the metrics via the service. |
| service.registry.hostPort | int | `30020` | Local host port to expose the registry. |
Expand Down
3 changes: 3 additions & 0 deletions charts/spegel/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ metadata:
labels:
{{- include "spegel.labels" . | nindent 4 }}
spec:
revisionHistoryLimit: {{ .Values.revisionHistoryLimit }}
updateStrategy:
{{- toYaml .Values.updateStrategy | nindent 4 }}
selector:
Expand Down Expand Up @@ -60,6 +61,8 @@ spec:
valueFrom:
fieldRef:
fieldPath: status.hostIP
resources:
{{- toYaml .Values.resources | nindent 10 }}
volumeMounts:
- name: containerd-config
mountPath: {{ .Values.spegel.containerdRegistryConfigPath }}
Expand Down
3 changes: 3 additions & 0 deletions charts/spegel/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@ podAnnotations: {}
podSecurityContext: {}
# fsGroup: 2000

# -- The number of old history to retain to allow rollback.
revisionHistoryLimit: 10

# -- Security context for the Spegel container.
securityContext: {}
# capabilities:
Expand Down

0 comments on commit 813a820

Please sign in to comment.