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
pcr: delete physical_replication.enabled setting #119149
Conversation
.filter(option => canViewKvGraphs || !option.isKvDashboard) | ||
// Don't show the replication dashboard if not enabled. | ||
.filter( | ||
option => | ||
this.props.crossClusterReplicationEnabled || | ||
option.label !== "Physical Cluster Replication", | ||
); |
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.
I think this is fine to always show this dashboard. If people don't like it, we can follow up with something that just sets a new feature flag if you happen to have > 1 tenant.
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.
I think hiding it was a private-preview thing; with it coming out of preview I don't think there is any precedent for keeping it optional/hidden.
TFTR! bors r+ |
Merge conflict. |
Release note: none. Epic: none.
Release note (enterprise change): Physical Cluster Replication is now always enabled, and the setting physical_replication.enabled has been removed.
bors r+ |
Build succeeded: |
Release note (enterprise change): Physical Cluster Replication no longer requires toggling the setting
physical_replication.enabled
to be used, and the setting has been removed.