Github user swill commented on the issue:
https://github.com/apache/cloudstack/pull/1596
Ok, this is coming back clean. Do you guys agree I should merge this into
4.9 as it fixes issues? Thanks...
---
If your project is set up for it, you can reply to this email and have your
Github user swill commented on the issue:
https://github.com/apache/cloudstack/pull/1596
### CI RESULTS
```
Tests Run: 85
Skipped: 0
Failed: 0
Errors: 0
Duration: 4h 02m 49s
```
**Associated Uploads**
Github user anshul1886 commented on the issue:
https://github.com/apache/cloudstack/pull/1596
@remibergsma Yes, we can close old one. @swill This PR is created against
4.7 so it can be forward merged to all branches.
---
If your project is set up for it, you can reply to this email
Github user swill commented on the issue:
https://github.com/apache/cloudstack/pull/1596
This should probably get int0 4.9 right? If this is broken we should try
to get the fix in as early as possible right? There are other potential issues
pending, so we have not pushed forward on
Github user remibergsma commented on the issue:
https://github.com/apache/cloudstack/pull/1596
@anshul1886 Ok, thanks. I think we should close that old one, shouldn't we?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user anshul1886 commented on the issue:
https://github.com/apache/cloudstack/pull/1596
@remibergsma #925 is no more valid after fix of #1336. This PR handles the
broken case i.e. migration with storage with cluster is currently broken.
---
If your project is set up for it,
Github user remibergsma commented on the issue:
https://github.com/apache/cloudstack/pull/1596
Same approach as proposed in #925, although both address different
problems. I'd say let's test this stuff and get it in to resolve these issues.
We may want to look to the other PR as