-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Helm path update #1061
Helm path update #1061
Conversation
So this basically needs to sit here for a while until we release Loki v0.4.0 with the new path. I completely missed this in #1049. |
This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
0ffdf41
to
a3cf4d9
Compare
a3cf4d9
to
ca21682
Compare
Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott <[email protected]>
ca21682
to
b3886f8
Compare
Signed-off-by: Joe Elliott <[email protected]>
@cyriltovena |
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
…-test Fix TestChunksBasic() to check all chunks returned
* Updated helm chart push paths Signed-off-by: Joe Elliott <[email protected]> * Updated helm chart versions Signed-off-by: Joe Elliott <[email protected]> * Updated test to use new format when pushing Signed-off-by: Joe Elliott <[email protected]>
Signed-off-by: Joe Elliott [email protected]
What this PR does / why we need it:
Helm chart paths were not up to date with the latest Loki paths due to this issue:
#852
This PR moves them forward to the v1 paths and increments the path version of all 3 helm charts.