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.

Sign up for GitHub

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

Jump to bottom

docs: mark serverless as deprecated #4017

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

docs: mark serverless as deprecated #4017

wants to merge 1 commit into from

Conversation

electron0zero
Copy link
Member

@electron0zero electron0zero commented Aug 28, 2024 •
edited
Loading

What this PR does:
we are deprecating serverless, so mark it as deprecated in the tempo docs

Which issue(s) this PR fixes:
we are discussing it in #4014

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@joe-elliott
Copy link
Member

I believe we are still discussing this: #4014

and would like to allow people from the community to give feedback before we mark this deprecated

@electron0zero
Copy link
Member Author

electron0zero commented Aug 28, 2024 •
edited
Loading

will leave it as draft for time being, and give time for comments and feedback.

@knylander-grafana knylander-grafana added the type/docs Improvements or additions to documentation label Aug 28, 2024
@github-actions GitHub Actions
Copy link
Contributor

This PR has been automatically marked as stale because it has not had any activity in the past 60 days.
The next time this stale check runs, the stale label will be removed if there is new activity. This pull request will be closed in 15 days if there is no new activity.
Please apply keepalive label to exempt this Pull Request.

@github-actions github-actions bot added the stale Used for stale issues / PRs label Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@knylander-grafana knylander-grafana Awaiting requested review from knylander-grafana knylander-grafana will be requested when the pull request is marked ready for review knylander-grafana is a code owner

@joe-elliott joe-elliott Awaiting requested review from joe-elliott joe-elliott will be requested when the pull request is marked ready for review joe-elliott is a code owner

@mdisibio mdisibio Awaiting requested review from mdisibio mdisibio will be requested when the pull request is marked ready for review mdisibio is a code owner

@mapno mapno Awaiting requested review from mapno mapno will be requested when the pull request is marked ready for review mapno is a code owner

@yvrhdn yvrhdn Awaiting requested review from yvrhdn yvrhdn will be requested when the pull request is marked ready for review yvrhdn is a code owner

@zalegrala zalegrala Awaiting requested review from zalegrala zalegrala will be requested when the pull request is marked ready for review zalegrala is a code owner

@ie-pham ie-pham Awaiting requested review from ie-pham ie-pham will be requested when the pull request is marked ready for review ie-pham is a code owner

@stoewer stoewer Awaiting requested review from stoewer stoewer will be requested when the pull request is marked ready for review stoewer is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
stale Used for stale issues / PRs type/docs Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@electron0zero @joe-elliott @knylander-grafana