-
Notifications
You must be signed in to change notification settings - Fork 31
Replace unmaintained loki-stack addon with separate loki and promtail addons #609
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
base: main
Are you sure you want to change the base?
Conversation
39167d9 to
9d1a1d5
Compare
|
Promtail is also deprecated, is the plan to initially switch out loki-stack for loki then switch promtail -> alloy later? |
Yes exactly, we're planning to get to latest Loki v2.X first and then follow up with a separate change to upgrade to Loki v3.X which is where the migration from Promtail to Alloy will happen. We have a few long-term deployments which make use of the existing logging stack so we're aiming to make the switch as smooth as possible and ideally avoid losing any historical data, hence the more gradual approach. |
860a995 to
fbc296a
Compare
fbc296a to
63019fb
Compare
sd109
left a comment
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.
Looks like a good start! I've left a few suggestions for improvements and noted a couple of concerns that we should consider carefully before merging this change.
e321f04 to
c3ae364
Compare
Review changes on comments Co-authored-by: Scott Davidson <[email protected]>
Co-authored-by: Scott Davidson <[email protected]>
fb8e101 to
9340778
Compare
ca917da to
6a11c29
Compare
Closes #606