Skip to content
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

Change image tag from 1.4.1 to 1.4.0 in Helm chart #2015

Merged
merged 9 commits into from
Apr 30, 2020

Conversation

etashsingh
Copy link
Contributor

@etashsingh etashsingh commented Apr 30, 2020

Docker image tag changed from 1.4.1 back to 1.4.0 as it was running into an error while deploying.
Updated chart version values
Provides solution to issue: #1979
Signed-off-by: Etash Singh etashsingh29@gmail.com

Change image tag from 1.4.1 to 1.4.0 to make the chart work as the image with 1.4.1 runs into errors while deploying
@CLAassistant
Copy link

CLAassistant commented Apr 30, 2020

CLA assistant check
All committers have signed the CLA.

@codecov-io
Copy link

Codecov Report

Merging #2015 into master will increase coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2015      +/-   ##
==========================================
+ Coverage   63.70%   63.85%   +0.14%     
==========================================
  Files         133      133              
  Lines       10179    10179              
==========================================
+ Hits         6485     6500      +15     
+ Misses       3205     3188      -17     
- Partials      489      491       +2     
Impacted Files Coverage Δ
pkg/promtail/positions/positions.go 60.71% <0.00%> (+13.39%) ⬆️

@slim-bean
Copy link
Collaborator

Thanks @etashsingh ! I think however if i'm following the issue correctly we need to change the promtail chart image 1.4.0 not the loki chart, could you make this update please?

@etashsingh
Copy link
Contributor Author

@slim-bean
Thanks for clearing out this for me.
I have made the changes to the Promtail chart and undone the changes from the Loki chart.
Thanks!

@slim-bean
Copy link
Collaborator

looking better! we should also increase the loki-stack chart version like you had before.

(anytime we increase the chart version for either loki or promtail we increase the lok-stack version)

@etashsingh
Copy link
Contributor Author

@slim-bean didn't know that!
Thanks again for pointing it out!
loki-stack chart version is updated now.

Copy link
Collaborator

@slim-bean slim-bean left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! We'll make sure all the tests pass and get this merged!

Thanks for your first contribution @etashsingh 🎉

@etashsingh
Copy link
Contributor Author

@slim-bean you're welcome and thanks a lot for helping me out!

@owen-d owen-d merged commit 521999c into grafana:master Apr 30, 2020
mraboosk pushed a commit to mraboosk/loki that referenced this pull request Oct 7, 2024
* Change image tag from 1.4.1 to 1.4.0

Change image tag from 1.4.1 to 1.4.0 to make the chart work as the image with 1.4.1 runs into errors while deploying

* Update chart version value

* update loki stack chert version

* Changed image tag value from 1.4.1 to 1.4.0

* Update values.yaml

* Update Chart.yaml

* Update Chart.yaml

* Update Chart.yaml

* Update loki-stack chart version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants