-
Notifications
You must be signed in to change notification settings - Fork 1.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
4.19 merge #10303
base: 4.20
Are you sure you want to change the base?
4.19 merge #10303
Conversation
* server: fix attach uploaded volume Fixes #10120 When an uploaded volume is attached to a VM for which no existing volume can be found it was resulting in error. For such volumes, server needs to find a suitable pool first and copy them to the pool from secondary store. Signed-off-by: Abhishek Kumar <[email protected]> * fix Signed-off-by: Abhishek Kumar <[email protected]> * fix Signed-off-by: Abhishek Kumar <[email protected]> * add unit tests Signed-off-by: Abhishek Kumar <[email protected]> --------- Signed-off-by: Abhishek Kumar <[email protected]> Co-authored-by: Boris Stoyanov - a.k.a Bobby <[email protected]>
Signed-off-by: Abhishek Kumar <[email protected]>
…t_ref for the removed pools (#9852)
Signed-off-by: Abhishek Kumar <[email protected]>
return String.format("UserAccount %s.", ReflectionToStringBuilderUtils.reflectOnlySelectedFields | ||
(this, "id", "uuid", "username", "accountName")); | ||
} | ||
|
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.
Not sure if we have to retain this: https://github.com/apache/cloudstack/pull/9873/files#diff-ad04db4978745b82495c1971e127ef6d1a89fd05ff28530b775bd5ae0c0a848fR134-R138
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.
Should we "squash and merge" this pr ?
Description
This PR intends to deal with a set of merge conflicts
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?