-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[xCluster] Deadlock during Setup from BG thread AutoFlag validation #22376
Labels
area/docdb
YugabyteDB core features
kind/bug
This issue is a bug
priority/highest
Highest priority issue
Comments
yugabyte-ci
added
kind/bug
This issue is a bug
priority/medium
Medium priority issue
priority/highest
Highest priority issue
and removed
priority/medium
Medium priority issue
labels
May 13, 2024
hari90
added a commit
that referenced
this issue
May 15, 2024
…lagsConfigChange Summary: `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion` take the ClusterConfig write lock before ReplicationInfo write lock. Whereas `AddCDCStreamToUniverseAndInitConsumer` take the locks in the other order causing a deadlock of setup replication. Fixed the lock order in `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion`. Jira: DB-11277 Test Plan: Jenkins Reviewers: slingam, jhe, xCluster, asrivastava Reviewed By: asrivastava Subscribers: ybase Differential Revision: https://phorge.dev.yugabyte.com/D35072
hari90
added a commit
that referenced
this issue
May 15, 2024
… handling AutoFlagsConfigChange Summary: Original commit: d9deed4 / D35072 `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion` take the ClusterConfig write lock before ReplicationInfo write lock. Whereas `AddCDCStreamToUniverseAndInitConsumer` take the locks in the other order causing a deadlock of setup replication. Fixed the lock order in `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion`. Jira: DB-11277 Test Plan: Jenkins Reviewers: slingam, jhe, xCluster, asrivastava Reviewed By: asrivastava Subscribers: ybase Tags: #jenkins-ready Differential Revision: https://phorge.dev.yugabyte.com/D35077
hari90
added a commit
that referenced
this issue
May 15, 2024
…en handling AutoFlagsConfigChange Summary: Original commit: d9deed4 / D35072 `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion` take the ClusterConfig write lock before ReplicationInfo write lock. Whereas `AddCDCStreamToUniverseAndInitConsumer` take the locks in the other order causing a deadlock of setup replication. Fixed the lock order in `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion`. Jira: DB-11277 Test Plan: Jenkins Reviewers: slingam, jhe, xCluster, asrivastava Reviewed By: asrivastava Subscribers: ybase Tags: #jenkins-ready Differential Revision: https://phorge.dev.yugabyte.com/D35076
svarnau
pushed a commit
that referenced
this issue
May 25, 2024
…lagsConfigChange Summary: `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion` take the ClusterConfig write lock before ReplicationInfo write lock. Whereas `AddCDCStreamToUniverseAndInitConsumer` take the locks in the other order causing a deadlock of setup replication. Fixed the lock order in `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion`. Jira: DB-11277 Test Plan: Jenkins Reviewers: slingam, jhe, xCluster, asrivastava Reviewed By: asrivastava Subscribers: ybase Differential Revision: https://phorge.dev.yugabyte.com/D35072
svarnau
pushed a commit
that referenced
this issue
May 29, 2024
…en handling AutoFlagsConfigChange Summary: Original commit: d9deed4 / D35072 `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion` take the ClusterConfig write lock before ReplicationInfo write lock. Whereas `AddCDCStreamToUniverseAndInitConsumer` take the locks in the other order causing a deadlock of setup replication. Fixed the lock order in `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion`. Jira: DB-11277 Test Plan: Jenkins Reviewers: slingam, jhe, xCluster, asrivastava Reviewed By: asrivastava Subscribers: ybase Tags: #jenkins-ready Differential Revision: https://phorge.dev.yugabyte.com/D35076
svarnau
pushed a commit
that referenced
this issue
May 30, 2024
… handling AutoFlagsConfigChange Summary: Original commit: d9deed4 / D35072 `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion` take the ClusterConfig write lock before ReplicationInfo write lock. Whereas `AddCDCStreamToUniverseAndInitConsumer` take the locks in the other order causing a deadlock of setup replication. Fixed the lock order in `HandleLocalAutoFlagsConfigChange` and `RefreshAutoFlagConfigVersion`. Jira: DB-11277 Test Plan: Jenkins Reviewers: slingam, jhe, xCluster, asrivastava Reviewed By: asrivastava Subscribers: ybase Tags: #jenkins-ready Differential Revision: https://phorge.dev.yugabyte.com/D35077
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/docdb
YugabyteDB core features
kind/bug
This issue is a bug
priority/highest
Highest priority issue
Jira Link: DB-11277
Description
HandleLocalAutoFlagsConfigChange
andRefreshAutoFlagConfigVersion
take the ClusterConfig write lock before ReplicationInfo write lock. WhereasAddCDCStreamToUniverseAndInitConsumer
take the locks in the other order.Issue Type
kind/bug
Warning: Please confirm that this issue does not contain any sensitive information
The text was updated successfully, but these errors were encountered: