You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by anshul1886 <gi...@git.apache.org> on 2016/06/24 04:27:52 UTC
[GitHub] cloudstack pull request #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migrati...
GitHub user anshul1886 opened a pull request:
https://github.com/apache/cloudstack/pull/1596
CLOUDSTACK-9353: [XenServer] Fixed VM migration with storage
In turn this also fixes VM migration with local storage
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/anshul1886/cloudstack-1 CLOUDSTACK-9353
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1596.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1596
----
commit 458264aac843d4d21500a5f0ab49f8e6a00410cd
Author: Anshul Gangwar <an...@accelerite.com>
Date: 2016-06-15T06:08:03Z
CLOUDSTACK-9353: [XenServer] Fixed VM migration with storage
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by anshul1886 <gi...@git.apache.org>.
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, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by remibergsma <gi...@git.apache.org>.
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. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by swill <gi...@git.apache.org>.
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 the 4.9 RC yet.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by swill <gi...@git.apache.org>.
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**
**`/tmp/MarvinLogs/DeployDataCenter__Jun_27_2016_22_57_23_SVYGMK:`**
* [dc_entries.obj](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/DeployDataCenter__Jun_27_2016_22_57_23_SVYGMK/dc_entries.obj)
* [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/DeployDataCenter__Jun_27_2016_22_57_23_SVYGMK/failed_plus_exceptions.txt)
* [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/DeployDataCenter__Jun_27_2016_22_57_23_SVYGMK/runinfo.txt)
**`/tmp/MarvinLogs/test_network_346OKD:`**
* [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/test_network_346OKD/failed_plus_exceptions.txt)
* [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/test_network_346OKD/results.txt)
* [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/test_network_346OKD/runinfo.txt)
**`/tmp/MarvinLogs/test_vpc_routers_YO70GO:`**
* [failed_plus_exceptions.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/test_vpc_routers_YO70GO/failed_plus_exceptions.txt)
* [results.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/test_vpc_routers_YO70GO/results.txt)
* [runinfo.txt](https://objects-east.cloud.ca/v1/e465abe2f9ae4478b9fff416eab61bd9/PR1596/tmp/MarvinLogs/test_vpc_routers_YO70GO/runinfo.txt)
Uploads will be available until `2016-08-28 02:00:00 +0200 CEST`
*Comment created by [`upr comment`](https://github.com/cloudops/upr).*
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by anshul1886 <gi...@git.apache.org>.
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 and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack pull request #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migrati...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1596
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by remibergsma <gi...@git.apache.org>.
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 well, although we cannot merge both without some tweaks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1596: CLOUDSTACK-9353: [XenServer] Fixed VM migration with...
Posted by swill <gi...@git.apache.org>.
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
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---