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

Fix performance glitch while sorting image locations #88

Open
wants to merge 3 commits into
base: stackhpc/2024.1
Choose a base branch
from

Conversation

bbezak
Copy link
Member

@bbezak bbezak commented Feb 6, 2025

Some of the available glance stores like file, cinder etc has capability to reuse already initiated driver (DRIVER_REUSABLE = 0b01000000). In Caracal we have added a feature to sort image locations based on store weight. As RBD driver of glance does not have this reuse capability, during image list API call it initializes the RBD driver for each of the available image which is causing noticable delay in list call.

To avoid this, using new interface added in glance_store which will directly get the weight of the store from memory and return it back to user.

Depends-On: stackhpc/glance_store#3
Closes-Bug: #2086675
Change-Id: I662ba19697e03917ca999920ea7be93a0b2a8296 (cherry picked from commit ce537e073b2e7efb332d719bb45bf20c403c140a)

Some of the available glance stores like file, cinder etc has
capability to reuse already initiated driver (DRIVER_REUSABLE = 0b01000000). In
Caracal we have added a feature to sort image locations based on store
weight. As RBD driver of glance does not have this reuse capability, during
image list API call it initializes the RBD driver for each of the available
image which is causing noticable delay in list call.

To avoid this, using new interface added in glance_store which will directly
get the weight of the store from memory and return it back to user.

Depends-On: https://review.opendev.org/c/openstack/glance_store/+/940531
Closes-Bug: #2086675
Change-Id: I662ba19697e03917ca999920ea7be93a0b2a8296
(cherry picked from commit ce537e073b2e7efb332d719bb45bf20c403c140a)
@bbezak bbezak requested a review from a team February 6, 2025 12:53
@bbezak bbezak closed this Feb 11, 2025
@bbezak bbezak reopened this Feb 11, 2025
@seunghun1ee seunghun1ee reopened this Feb 11, 2025
@seunghun1ee seunghun1ee reopened this Feb 11, 2025
@seunghun1ee seunghun1ee reopened this Feb 11, 2025
@seunghun1ee seunghun1ee reopened this Feb 11, 2025
@seunghun1ee seunghun1ee force-pushed the glance_rbd_slow branch 4 times, most recently from f90cf22 to 5e77257 Compare February 11, 2025 12:22
@seunghun1ee
Copy link
Member

This PR depends on stackhpc/requirements#70

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants