Skip to content

Commit

Permalink
fix(otellogs): set resources on Otelcol logs collector daemonset
Browse files Browse the repository at this point in the history
The resources were already defined in `values.yaml`
in the `otellogs.daemonset.resources` property,
but this property was not being used in the daemonset template.
  • Loading branch information
andrzej-stencel committed May 25, 2022
1 parent c13bce9 commit fd24a78
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 0 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

- chore(deps): upgrade fluentd to 1.14.6-sumo-3 [#2287][#2287]

### Fixed

- fix(otellogs): set resources on Otelcol logs collector daemonset [#2291]

[#2287]: https://github.com/SumoLogic/sumologic-kubernetes-collection/pull/2287
[#2291]: https://github.com/SumoLogic/sumologic-kubernetes-collection/pull/2291
[Unreleased]: https://github.com/SumoLogic/sumologic-kubernetes-collection/compare/v2.8.0...release-v2.8

## [v2.8.0]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,8 @@ spec:
httpGet:
path: /
port: 13133 # Health Check extension default port.
resources:
{{- toYaml .Values.otellogs.daemonset.resources | nindent 10 }}
volumeMounts:
- mountPath: /etc/otelcol
name: otelcol-config
Expand Down
7 changes: 7 additions & 0 deletions tests/helm/logs_otc_daemonset/static/basic.output.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,13 @@ spec:
httpGet:
path: /
port: 13133 # Health Check extension default port.
resources:
limits:
cpu: 1000m
memory: 1Gi
requests:
cpu: 100m
memory: 32Mi
volumeMounts:
- mountPath: /etc/otelcol
name: otelcol-config
Expand Down

0 comments on commit fd24a78

Please sign in to comment.