{"action":"created","issue":{"url":"https://api.github.com/repos/apache/cloudstack/issues/1711","repository_url":"https://api.github.com/repos/apache/cloudstack","labels_url":"https://api.github.com/repos/apache/cloudstack/issues/1711/labels{/name}","comments_url":"https://api.github.com/repos/apache/cloudstack/issues/1711/comments","events_url":"https://api.github.com/repos/apache/cloudstack/issues/1711/events","html_url":"https://github.com/apache/cloudstack/pull/1711","id":183512248,"number":1711,"title":"XenServer 7 Support","user":{"login":"syed","id":55854,"avatar_url":"https://avatars.githubusercontent.com/u/55854?v=3","gravatar_id":"","url":"https://api.github.com/users/syed","html_url":"https://github.com/syed","followers_url":"https://api.github.com/users/syed/followers","following_url":"https://api.github.com/users/syed/following{/other_user}","gists_url":"https://api.github.com/users/syed/gists{/gist_id}","starred_url":"https://api.github.com/users/syed/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/syed/subscriptions","organizations_url":"https://api.github.com/users/syed/orgs","repos_url":"https://api.github.com/users/syed/repos","events_url":"https://api.github.com/users/syed/events{/privacy}","received_events_url":"https://api.github.com/users/syed/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":49,"created_at":"2016-10-17T19:59:35Z","updated_at":"2016-12-09T05:20:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/cloudstack/pulls/1711","html_url":"https://github.com/apache/cloudstack/pull/1711","diff_url":"https://github.com/apache/cloudstack/pull/1711.diff","patch_url":"https://github.com/apache/cloudstack/pull/1711.patch"},"body":"This PR adds support for XenServer 7. I have manually done the following tests\n- Create a new cluster with XenServer7\n- Add Primary storage: Should create an SR on XS7\n- Add another XS7 host to the Pool \n- Add host2 to Cloudstack\n- Create VM1 from template \n- Create VM2 from template \n- Ping/SSH VM1 to VM2 and vice-versa\n- Stop/Delete/Expunge VM2\n- Create Data disk \n- Attach it to VM1\n- Create VM snaphsot of VM1\n- Restore VM snapshot of VM1\n- Delete VM snapshot of VM1\n- Create Volume snapshot of Datadisk\n- Create volume snapshot of Root disk\n- Create new template from snapshot of root disk\n- Create volume from snapshot of datadisk\n- Detach datadisk volume\n- Delete datadisk volume\n- Aquire a public IP\n- Create a static nat to VM1\n- Live migrate VM1 while traffic on VM\n- Delete VM1\n"},"comment":{"url":"https://api.github.com/repos/apache/cloudstack/issues/comments/265936126","html_url":"https://github.com/apache/cloudstack/pull/1711#issuecomment-265936126","issue_url":"https://api.github.com/repos/apache/cloudstack/issues/1711","id":265936126,"user":{"login":"karuturi","id":186833,"avatar_url":"https://avatars.githubusercontent.com/u/186833?v=3","gravatar_id":"","url":"https://api.github.com/users/karuturi","html_url":"https://github.com/karuturi","followers_url":"https://api.github.com/users/karuturi/followers","following_url":"https://api.github.com/users/karuturi/following{/other_user}","gists_url":"https://api.github.com/users/karuturi/gists{/gist_id}","starred_url":"https://api.github.com/users/karuturi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karuturi/subscriptions","organizations_url":"https://api.github.com/users/karuturi/orgs","repos_url":"https://api.github.com/users/karuturi/repos","events_url":"https://api.github.com/users/karuturi/events{/privacy}","received_events_url":"https://api.github.com/users/karuturi/received_events","type":"User","site_admin":false},"created_at":"2016-12-09T05:20:13Z","updated_at":"2016-12-09T05:20:13Z","body":"Yes. I think its ready code wise. since the last test run, lot of code changes happened. If we can run one more round of tests, that would be good.\r\n\r\nregarding squashing, \r\nPlease don't do that. The three commits are different and cannot be merged to a single commit. probably, we can get rid of the merge commit."}}
{"actor":{"display_login":"karuturi"}}