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.

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

storcon: Attached(0) tenants that do graceful migrations can get stuck #10935

Open
jcsp opened this issue Feb 21, 2025 · 0 comments
Open

storcon: Attached(0) tenants that do graceful migrations can get stuck #10935

jcsp opened this issue Feb 21, 2025 · 0 comments
Assignees
Labels
c/storage/controller Component: Storage Controller t/bug Issue Type: Bug

Comments

@jcsp
Copy link
Contributor

jcsp commented Feb 21, 2025

This is not an issue in the field where we always used Attached(1) (i.e. 1 secondary location).

If we use Attached(0), and the scheduling optimisation logic tries to move a location by creating a secondary and waiting for it to warm up, then this can get stuck because the attached location never uploads a heatmap for the secondary to use.

The attached location only uses the PlacementPolicy to decide whether to enable heatmaps. We should either do it always, or respect the number of secondaries in the intent as well as the placement policy.

@jcsp jcsp self-assigned this Feb 21, 2025
@jcsp jcsp added t/bug Issue Type: Bug c/storage/controller Component: Storage Controller labels Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/storage/controller Component: Storage Controller t/bug Issue Type: Bug
Projects
None yet
Development

No branches or pull requests

1 participant