-
Notifications
You must be signed in to change notification settings - Fork 1.3k
CLOUDSTACK-9681: Make the migration of volume to go through allocator… #1835
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
Conversation
|
@anshul1886 Please resolve the conflicts. Code changes LGTM |
0485765 to
814f8a7
Compare
|
Failures seems to be transitional in nature. |
814f8a7 to
2c74396
Compare
|
@anshul-gangwar : There are some tests failing from PrimaryDataStoreDaoImplTest due to extra null value, I guess. Once you fix that, we can review it. Thanks. |
|
@koushik-das @nitin-maharana PR #1994 has the tests which is kind of forcing the SQL statements to be used in implementation. So I am thinking of way how to go about it. |
2c74396 to
5467be2
Compare
… so that appropraite checks are made Also, make detached disk to go through findStoragePoolsForMigration API from UI
5467be2 to
1191117
Compare
|
Failures reported are due to network related issues which are unrelated to this PR. |
ACS CI BVT RunSumarry: Link to logs Folder (search by build_no): https://www.dropbox.com/sh/r2si930m8xxzavs/AAAzNrnoF1fC3auFrvsKo_8-a?dl=0 Failed tests:
Skipped tests: Passed test suits: |
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✖centos6 ✔centos7 ✖debian. JID-1291 |
|
@anshul1886 please check/fix build failures |
|
@anshul1886 ping, can you fix the build failures? |
|
Ping, any updates? I'll remove this from 4.11 milestone due to lack of engagement and discussions. |
|
@anshul1886 please rebase and re-open if still relevant |
… so that appropraite checks are made
Also, make detached disk to go through findStoragePoolsForMigration API from UI