-
Notifications
You must be signed in to change notification settings - Fork 141
Remove observability from alpha1 #4215
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
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4215 +/- ##
=======================================
Coverage 86.09% 86.09%
=======================================
Files 131 131
Lines 14171 14171
Branches 35 35
=======================================
Hits 12200 12200
Misses 1767 1767
Partials 204 204 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@tataruty We should add a release note too that we removed this deprecated API |
df68e1c to
81b7e7a
Compare
|
Can we update the description on what happens when we use that API version? Could you check the upgrade process? For example if a user is using this version and we delete it -- what does the user see? or does it get automatically upgraded? Looks good to me otherwise |
@salonichf5 it seems automatically upgraded without any error, you can see screenshot attached to the description |
salonichf5
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.
great job
Proposed changes
Problem:
We need to remove v1alpha1 version of the Observability API in the future. We are supporting a newer version v1alpha2 for Observability API now.
#2986 to support the newer version.
Solution: Remove observability from v1alpha1.
Testing: Functional and unit tests

tested upgrade:
Closes #2767
Checklist
Before creating a PR, run through this checklist and mark each as complete.
Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.