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

Universal Profiling cleanups for 9.0.0 #106592

Closed
4 tasks done
danielmitterdorfer opened this issue Mar 21, 2024 · 3 comments
Closed
4 tasks done

Universal Profiling cleanups for 9.0.0 #106592

danielmitterdorfer opened this issue Mar 21, 2024 · 3 comments
Labels
Meta Team:obs-knowledge Meta label for Observability Knowledge team :UniversalProfiling/Application Elastic Universal Profiling REST APIs and infrastructure

Comments

@danielmitterdorfer
Copy link
Member

danielmitterdorfer commented Mar 21, 2024 •
edited by rockdaboot
Loading

This is a meta-issue to collect all cleanups to be done for the next major version.

Tasks
Beta Give feedback
  1. [Profiling, 9.0.0] Remove unused fields from the stacktraces and flamegraph API #106593
    :UniversalProfiling/Application Team:obs-knowledge
  2. [Profiling, 9.0.0] Remove backwards compatibility code in CO2 calculator #106596
    :UniversalProfiling/Application Team:obs-knowledge
  3. [Profiling, 9.0.0] Remove backwards compatibility code for host.arch #106597
    :UniversalProfiling/Application Team:obs-knowledge
  4. [Profiling, 9.0.0] Remove backwards compatibility for missing cloud.provider, cloud.region, host.type #106681
    :UniversalProfiling/Application Team:obs-knowledge
@danielmitterdorfer danielmitterdorfer added Meta :UniversalProfiling/Application Elastic Universal Profiling REST APIs and infrastructure labels Mar 21, 2024
@elasticsearchmachine elasticsearchmachine added the Team:obs-knowledge Meta label for Observability Knowledge team label Mar 21, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/obs-knowledge-team (Team:obs-knowledge)

@danielmitterdorfer danielmitterdorfer mentioned this issue Mar 21, 2024
Merged
@DaveCTurner
Copy link
Contributor

I'd suggest adding the @UpdateForV9 annotation to relevant points in the code to keep track of these things today. When we decide to start working on v9, we'll go through all the places where this annotation is used to make sure we don't miss anything.

@rockdaboot rockdaboot mentioned this issue Mar 22, 2024
Closed
danielmitterdorfer added a commit to danielmitterdorfer/elasticsearch that referenced this issue Apr 5, 2024
@danielmitterdorfer
[Profiling] Annotate TODOs for 9.0.0 upgrade
dccec19 
As suggested by David in elastic#106592, we're adding the annotation
`@UpdateForV9` to all places in the Universal Profiling plugin that
can be removed once we move to 9.0.0.

Relates elastic#106592
@danielmitterdorfer danielmitterdorfer mentioned this issue Apr 5, 2024
Merged
@danielmitterdorfer
Copy link
Member Author

I'd suggest adding the @UpdateForV9 annotation to relevant points in the code to keep track of these things [...]

Thanks. I've pushed #107150.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Meta Team:obs-knowledge Meta label for Observability Knowledge team :UniversalProfiling/Application Elastic Universal Profiling REST APIs and infrastructure
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@danielmitterdorfer @DaveCTurner @elasticsearchmachine