[GitHub] cloudstack pull request: CLOUDSTACK-8395: vmops plugin should work...

2015-04-21 Thread xenserverarmy
Github user xenserverarmy commented on the pull request:

https://github.com/apache/cloudstack/pull/186#issuecomment-94880329
  
This looks solid, but I'd update line 457 to tick-quote the vmchain as 
you've done elsewhere


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack-docs-rn pull request: add note on XenServer: we depend ...

2015-05-23 Thread xenserverarmy
Github user xenserverarmy commented on the pull request:

https://github.com/apache/cloudstack-docs-rn/pull/21#issuecomment-104945193
  
I agree that we should add this to the 4.4 and 4.5 notes, but would suggest 
an addition relating to the config-timeout.  The default has changed with 
XenServer version, and I don't recall it ever being 180.  Text similar to the 
following would alert admins to the issue of aggressive fencing with redundant 
controllers.



Occasionally XenServer will determine that a storage heartbeat failure has 
occurred and fence a host, or worse an entire pool.  This typically occurs when 
a redundant storage solution or a storage solution with redundant controllers 
is used, and the failover timer used by the storage solution is longer than the 
default XenServer HA timeout of 30 seconds.  The value for this timeout can be 
changed, but it’s important to know that simply enabling HA from within 
XenCenter, or using the default form of the pool-enable-ha command will reset 
the timeout to the default value.  In the following example, the timeout has 
been increased from the default to two minutes.

# xe pool-enable-ha heartbeat-sr-uuids={SR-UUID} ha-config:timeout=120



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Fix/citrix resource base packages

2015-05-27 Thread xenserverarmy
Github user xenserverarmy commented on the pull request:

https://github.com/apache/cloudstack/pull/317#issuecomment-105936788
  
One suggestion.  In the future we may have pure Xen support, so if possible 
we should ensure all naming for XenServer is "XenServer" and not Xen.  So 
please change "com.cloud.hypervisor.xenserver.resource.wrapper.xenbase" to 
"com.cloud.hypervisor.xenserver.resource.wrapper.xenserverbase"


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8987 call s3xen/swiftxen plugi...

2015-10-26 Thread xenserverarmy
Github user xenserverarmy commented on the pull request:

https://github.com/apache/cloudstack/pull/970#issuecomment-151157372
  
@koushik-das One of the goals of a8212d9ef458dd7ac64b021e6fa33fcf64b3cce0 
was to ensure that we have clarity in which hypervisor we support since Xen != 
XenServer.  Reverting based on name IMO might be the simple solution, but could 
ultimately cause additional confusion as we don't currently have a pure Xen 
hypervisor implementation.

@wilderrodrigues @remibergsma, I didn't have s3 or swift storage to test 
with at the time of my changes. Would not the better solution be to rename the 
plugin s3xenserver and swiftxenserver?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: Rename xapi plugins for s3 and swift to m...

2015-10-28 Thread xenserverarmy
Github user xenserverarmy commented on the pull request:

https://github.com/apache/cloudstack/pull/982#issuecomment-151967711
  
@remibergsma How did the tests work out?



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---