-
Notifications
You must be signed in to change notification settings - Fork 532
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
Remove tempo serverless #4599
Remove tempo serverless #4599
Conversation
5b7e7c1
to
bf80b11
Compare
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.
👋 Good bye serverless. Nice cleanup
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.
o7 serverless
I'm good on this as well. Let's just wait for Kim to weigh in on removing the docs pages and then we can merge.
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.
Thank you for updating the doc! Changes look good! We should add an alias for any deleted pages. For an example of an alias: #4598 (comment)
Note that aliases use relative locations. For detailed instructions: https://grafana.com/docs/writers-toolkit/write/front-matter/#aliases
What this PR does:
serverless was deprecated in tempo 2.7 release.
Tempo serverless related code, config options, and docs is being removed.
This PR Removes:
2.serverless doc pages and mention from the docs
following config options are no longer valid:
following serverless related metrics are being removed:
Which issue(s) this PR fixes:
Fixes #4014
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]