[GitHub] cloudstack issue #1960: [4.11/Future] CLOUDSTACK-9782: Host HA and KVM HA pr...

2017-04-12 Thread serverchief
Github user serverchief commented on the issue: https://github.com/apache/cloudstack/pull/1960 Hi @koushik-das I believe you missed a discussion on this a while back - when this was initially proposed and we were gathering community feedback. please read this

[GitHub] cloudstack pull request: Latest changes on feature/systemvm persis...

2015-03-03 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/79#issuecomment-77063219 Wilder, Any luck with this request? Thanks. ilya --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request:

2015-03-20 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/commit/d70937aeb92e89efa8fcc39d7f89bdd8c885b2a2#commitcomment-10319442 thanks for this fix! --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: Refactor/libvirt resource

2015-05-11 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/233#issuecomment-101157780 @mlsorensen "I got the server xml worked out but it couldn't connect to the db, 'access denied for cloud@localhost'" I&#x

[GitHub] cloudstack pull request: Embedded Tomcat & Jetty

2015-06-08 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/372#issuecomment-110242990 I'm with Wilder on this. Thank you for being active and submitting really useful patches - its much appreciated. However, it would be best if you can spli

[GitHub] cloudstack pull request: CLOUDSTACK-8566: Filter sensitive details...

2015-06-26 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/536#issuecomment-115915514 Hi Rohit, Both username and passwords are needed, and we don't want to remove them and we would like to know what they are, however, it would be

[GitHub] cloudstack pull request: CLOUDSTACK-8566: Filter sensitive details...

2015-06-30 Thread serverchief
Github user serverchief commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/536#discussion_r33648388 --- Diff: api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java --- @@ -203,9 +201,22 @@ public void execute

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

2015-11-19 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158289806 +1, please merge On 11/18/15 11:03 PM, Rohit Yadav wrote: > Guys can we get some +1s on this, so let's at least merge on ma

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

2015-11-19 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-158291609 The metrics UI is not really a major feature in my opinion. We are not changing the core and any apis. All we are doing is re-presenting already

[GitHub] cloudstack pull request: CLOUDSTACK-9348: Use non-blocking SSL han...

2016-05-02 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/1493#issuecomment-216382896 @kiwiflyer, My testing with this patch - if you have at least several hundred KVM nodes connected to 2 MS via VIP and take 1 MS down, you will notice

[GitHub] cloudstack pull request: CLOUDSTACK-9040: Use Tomcat6 for Debian p...

2016-05-03 Thread serverchief
Github user serverchief commented on the pull request: https://github.com/apache/cloudstack/pull/1040#issuecomment-216645498 I know you can get it to work with ease on CentOS6 and 7.. Cant really comment on Debian as i don't have any instances. --- If your project is s