[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330711812 Trillian test result (tid-1529) Environment: kvm-centos6 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] yvsubhash commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
yvsubhash commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330439828 LGTM for code This is an automated

[GitHub] PranaliM commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
PranaliM commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330447354 tag:mergeready This is an automated

[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-19 Thread git
blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330463131 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance. URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330462365 I've just re-run the iso tests and they are

[GitHub] blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance. URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330462469 @borisstoyanov a Trillian-Jenkins test job

[GitHub] borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
borisstoyanov commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance. URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330462365 I've just re-run the iso tests and they are

[GitHub] borisstoyanov commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-19 Thread git
borisstoyanov commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330463000 I've re-run the tests and they are passing... @blueorangutan test

[GitHub] SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry?

2017-09-19 Thread git
SudharmaJain commented on issue #876: CLOUDSTACK-8865: Adding SR doesn't create Storage_pool_host_ref entry? URL: https://github.com/apache/cloudstack/pull/876#issuecomment-330660489 @rhtyd All the tests failed here, except the skipped ones. Even the listSystemVms command returned

[GitHub] kiwiflyer commented on issue #2083: Iptables speedup

2017-09-19 Thread git
kiwiflyer commented on issue #2083: Iptables speedup URL: https://github.com/apache/cloudstack/pull/2083#issuecomment-330659335 @DaanHoogland What can we do to help with this one? It looks really worth while. @dmabry

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330667959 Trillian test result (tid-1530) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt

[GitHub] blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance.

2017-09-19 Thread git
blueorangutan commented on issue #1252: CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance. URL: https://github.com/apache/cloudstack/pull/1252#issuecomment-330638199 Trillian test result (tid-1527)

[GitHub] rhtyd closed pull request #2153: CLOUDSTACK-9956 File search on the vmware datastore may select wrong file if there are multiple files with same name.

2017-09-19 Thread git
rhtyd closed pull request #2153: CLOUDSTACK-9956 File search on the vmware datastore may select wrong file if there are multiple files with same name. URL: https://github.com/apache/cloudstack/pull/2153 This is an

[cloudstack] branch master updated: CLOUDSTACK-9956: File search on the vmware datastore may select wrong file if there are multiple files with same name (#2153)

2017-09-19 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 3f69c83 CLOUDSTACK-9956: File search on

[GitHub] rhtyd commented on issue #2153: CLOUDSTACK-9956 File search on the vmware datastore may select wrong file if there are multiple files with same name.

2017-09-19 Thread git
rhtyd commented on issue #2153: CLOUDSTACK-9956 File search on the vmware datastore may select wrong file if there are multiple files with same name. URL: https://github.com/apache/cloudstack/pull/2153#issuecomment-330486621 Test LGTM, merging this.

[GitHub] rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330486375 @wido I can run another round of testing, for comparison I'm using the #2225 test results as baseline for comparison

[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330486577 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330501396 Packaging result: ?centos6 ?centos7 ?debian. JID-1105

[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330518581 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been

[GitHub] DaanHoogland commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
DaanHoogland commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330518438 @blueorangutan test

[GitHub] rhtyd commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance

2017-09-19 Thread git
rhtyd commented on issue #2260: CLOUDSTACK-10065: Optimize SQL queries in listTemplate API to improve performance URL: https://github.com/apache/cloudstack/pull/2260#issuecomment-330487064 Test LGTM (did not do code review), we'll need one more code review after that if there are no

[GitHub] rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330487165 @blueorangutan package This is an

[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330496645 Packaging result: ?centos6 ?centos7 ?debian. JID-1103

[GitHub] blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot)

2017-09-19 Thread git
blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot) URL: https://github.com/apache/cloudstack/pull/2257#issuecomment-330496618 Packaging result: ?centos6 ?centos7 ?debian. JID-1102

[GitHub] rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
rhtyd commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330501756 @blueorangutan test This is an

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330501929 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330523983 @borisstoyanov I understand these words: "help", "hello", "thanks", "package", "test" Test command

[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330524943 @blueorangutan test centos7 kvm-centos7 component/test_affinity_groups_projects.py,

[GitHub] rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse Scrolling is not working in instance VM console

2017-09-19 Thread git
rhtyd commented on a change in pull request #2240: CLOUDSTACK-10051 Mouse Scrolling is not working in instance VM console URL: https://github.com/apache/cloudstack/pull/2240#discussion_r139642434 ## File path: systemvm/js/jquery.flot.navigate.js ## @@ -0,0 +1,336 @@ +/*

[GitHub] rhtyd commented on a change in pull request #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
rhtyd commented on a change in pull request #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#discussion_r139642828 ## File path: test/integration/testpaths/Upload_Volume_test_case.py ##

[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330487639 @borisstoyanov I've rekicked one of the failing travis jobs, in general LGTM. @blueorangutan package

[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330523775 @blueorangutan help This is an

[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330525186 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] rhtyd closed pull request #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle

2017-09-19 Thread git
rhtyd closed pull request #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle URL: https://github.com/apache/cloudstack/pull/2078 This is an automated message from the Apache Git Service. To respond to the message,

[cloudstack] branch master updated: CLOUDSTACK-9902: add consoleproxy.sslEnable global setting (#2078)

2017-09-19 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 3bc2341 CLOUDSTACK-9902: add

[GitHub] blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot)

2017-09-19 Thread git
blueorangutan commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot) URL: https://github.com/apache/cloudstack/pull/2257#issuecomment-330486317 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted

[GitHub] rhtyd commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot)

2017-09-19 Thread git
rhtyd commented on issue #2257: CLOUDSTACK-10060:ListUsage API always displays the Virtual size as '0' for Usage type=9 (snapshot) URL: https://github.com/apache/cloudstack/pull/2257#issuecomment-330486146 @blueorangutan package

[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330487855 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330502525 @rhtyd the changes are in /component/.. do you think it's required to run a smoketest?

[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330508802 @borisstoyanov good point, can you run a component test run to confirm regression testings?

[GitHub] rhtyd commented on issue #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle

2017-09-19 Thread git
rhtyd commented on issue #2078: CLOUDSTACK-9902: add consoleproxy.sslEnable toggle URL: https://github.com/apache/cloudstack/pull/2078#issuecomment-330485772 LGTM, this is a schema only change and evaluation/testing with Travis is enough to qualify merging.

[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330487344 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I

[GitHub] rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
rhtyd commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330487314 @blueorangutan package

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-09-19 Thread git
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-330487359 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330497849 Packaging result: ?centos6 ?centos7 ?debian. JID-1104

[GitHub] rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
rhtyd commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330501194 @blueorangutan test centos7 kvm-centos6 This is an automated

[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330501412 Packaging result: ?centos6 ?centos7 ?debian. JID-1106

[GitHub] blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance

2017-09-19 Thread git
blueorangutan commented on issue #1707: CLOUDSTACK-9397: Add Watchdog timer to KVM Instance URL: https://github.com/apache/cloudstack/pull/1707#issuecomment-330501286 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos6) has been kicked to run smoke tests

[GitHub] jayakarteek commented on a change in pull request #2122: CLOUDSTACK-9909: Logrotate config for tomcat needs to be updated for ?

2017-09-19 Thread git
jayakarteek commented on a change in pull request #2122: CLOUDSTACK-9909: Logrotate config for tomcat needs to be updated for ? URL: https://github.com/apache/cloudstack/pull/2122#discussion_r139658445 ## File path: server/conf/cloudstack-catalina7.logrotate ## @@ -0,0

[GitHub] borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
borisstoyanov commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330512052 @rhtyd those tests pass in travis do you want explicit run on KVM env?

[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330521737 @borisstoyanov yes, since we're changing test code, let's validate them against one of the hypervisors; kvm

[GitHub] blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-09-19 Thread git
blueorangutan commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-330748617 Trillian test result (tid-1531) Environment: kvm-centos7 (x2), Advanced

[GitHub] blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-09-19 Thread git
blueorangutan commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-330747639 Trillian test result (tid-1532) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server

[GitHub] GabrielBrascher commented on issue #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL

2017-09-19 Thread git
GabrielBrascher commented on issue #2185: CLOUDSTACK-9976: Redirect saml2 failed login message to a configurable URL URL: https://github.com/apache/cloudstack/pull/2185#issuecomment-330608678 Sorry for the delay @rhtyd. I will work on that when possible.

[GitHub] sgoeminn opened a new pull request #2268: CLOUDSTACK-10081

2017-09-19 Thread git
sgoeminn opened a new pull request #2268: CLOUDSTACK-10081 URL: https://github.com/apache/cloudstack/pull/2268 CloudUtils getDevInfo function will now return "bridge" instead of "dev" when the name of a ovs bridge is passed.

[GitHub] blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type

2017-09-19 Thread git
blueorangutan commented on issue #2246: CLOUDSTACK-10046 checksum validation for any java supported Digests-type URL: https://github.com/apache/cloudstack/pull/2246#issuecomment-330626931 Trillian test result (tid-1528) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server