-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added support for imagePullSecrets in Loki Helm chart #1632
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1632 +/- ##
==========================================
- Coverage 61.74% 61.72% -0.03%
==========================================
Files 109 109
Lines 8308 8308
==========================================
- Hits 5130 5128 -2
- Misses 2783 2784 +1
- Partials 395 396 +1
|
@@ -1,6 +1,6 @@ | |||
apiVersion: "v1" | |||
name: loki-stack | |||
version: 0.30.0 | |||
version: 0.31.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version: 0.31.0 | |
version: 0.31.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have a point there, will update accordingly.
production/helm/loki/Chart.yaml
Outdated
@@ -1,6 +1,6 @@ | |||
apiVersion: "v1" | |||
name: loki | |||
version: 0.25.0 | |||
version: 0.26.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version: 0.26.0 | |
version: 0.26.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have a point there, will update accordingly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Added support for imagePullSecrets in Loki Helm chart * Updating only path version for loki charts
What this PR does / why we need it:
This PR adds support for imagePullSecrets in the Loki Helm chart. This is required if the extra container image is hosted in a local registry.
Code is taken from stable/grafana chart.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Checklist