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

Improve coalesce detection #5379

Closed
3 tasks done
olivierlambert opened this issue Nov 16, 2020 · 7 comments
Closed
3 tasks done

Improve coalesce detection #5379

olivierlambert opened this issue Nov 16, 2020 · 7 comments

Comments

@olivierlambert
Copy link
Member

olivierlambert commented Nov 16, 2020

Context

GC/coalesce process in XS/XCP-ng isn't exposed outside. But since xapi-project/sm#524 this will change: a task will be created for each coalesce process. So now we'll have a way to know if a coalesce is active.

Usage

Coupled with our knowledge of chains that must be coalesced, we have a way to show it's in progress, and the progress of this (it's a task with a %).

TODO

@KittyKatt
Copy link

Very interested in this feature. Not asking for an ETA, but expressing a need and interest. 👍

@olivierlambert
Copy link
Member Author

For now, this upstream code isn't inside 8.2 but will probably land in 8.3 :) As soon Citrix Hypervisor 8.3 is out (or/if the patch backported to 8.2), we'll expose that into XO 👍

@olivierlambert
Copy link
Member Author

The task is visible in the task list, but it should be simple to add it to the advanced SR view for the related SR.

@olivierlambert
Copy link
Member Author

@marcungeschikts target before the end of the year

@julien-f
Copy link
Member

#3081 should be done at the same time.

@olivierlambert
Copy link
Member Author

Also, the task text is providing the SR UUID with it, so it should be trivial to also translate it toward the SR name with a link to the SR. See:

Capture d’écran du 2023-10-16 21-47-51

@olivierlambert
Copy link
Member Author

It's done since XO 5.90

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment