[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread ArmedGuy
Github user ArmedGuy commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87756104 --- Diff: scripts/vm/hypervisor/xenserver/setup_heartbeat_sr.sh --- @@ -72,7 +72,7 @@ else fi rm $link -f fi

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-28 Thread ArmedGuy
Github user ArmedGuy commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus --config is not persistent, but helpfully overrides metadata_read_only = 1 which is set in /etc/lvm/lvm.conf on XS7. But I second that `--config global{metadata_read_only=0}` is

[GitHub] cloudstack pull request #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread ArmedGuy
Github user ArmedGuy closed the pull request at: https://github.com/apache/cloudstack/pull/1739 --- 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

[GitHub] cloudstack issue #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread ArmedGuy
Github user ArmedGuy commented on the issue: https://github.com/apache/cloudstack/pull/1739 Ah, my bad then. Thought I had searched for it. --- 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

[GitHub] cloudstack pull request #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread ArmedGuy
GitHub user ArmedGuy opened a pull request: https://github.com/apache/cloudstack/pull/1739 Fix xapi pluginlib for XenServer 7 This makes cloudstack_pluginlib.py compatible with XenServer 7 dom0, as (from what I gathered) simplejson was included by default as "json" in th