-
Notifications
You must be signed in to change notification settings - Fork 332
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
change: [M3-7715] - Remove kubernetesDashboardAvailability
feature flag
#10121
change: [M3-7715] - Remove kubernetesDashboardAvailability
feature flag
#10121
Conversation
kubernetesDashboardAvailability
feature flagkubernetesDashboardAvailability
feature flag
Coverage Report: β
|
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.
It's always exciting to see feature flag clean up, thanks for doing it! Confirmed the Dashboard Availability link is still present and opens after the un-flagging. (And I turned the flag off in the dev environment in LD.)
Wondering if we should make a note in the release testing doc to archive the flag in LD completely post-release. The final LD clean up is easy to forget sometimes.
As tempting as it is, this PR does not attempt to fix how broken this summary section is.
I know, it's hard. π
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.
Thanks @bnussman-akamai!
Description π
Just doing some feature flag clean up. The Kubernetes Dashboard feature has been in production for at least a year or two. I think we can safely clean up this flag! π§Ή
As tempting as it is, this PR does not attempt to fix how broken this summary section is.
Preview π·
Note
This should not result in UI changes. This feature has been enabled for a long time and should remain enabled.
How to test π§ͺ
Kubernetes Dashboard
button on the Kubernetes Details page on any LKE clusterAs an Author I have considered π€