[GitHub] rhtyd commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rhtyd commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372242182 @nvazquez `test_01_service_offering_cpu_limit_use` looks like a regression, can you check?

[GitHub] rafaelweingartner commented on issue #2387: CLOUDSTACK-8855 Improve Error Message for Host Alert State and reconnect host API.

2018-03-12 Thread GitBox
rafaelweingartner commented on issue #2387: CLOUDSTACK-8855 Improve Error Message for Host Alert State and reconnect host API. URL: https://github.com/apache/cloudstack/pull/2387#issuecomment-372255160 @DaanHoogland since your approval the code did not change. I only isolated a smoke test

[cloudstack] branch master updated (68251e8 -> 82bcc74)

2018-03-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 68251e8 Merge branch '4.11' add da8cf8c CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1 (#2480)

[cloudstack] 01/01: Merge branch '4.11'

2018-03-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 82bcc746799aa2845f469d081af47586d1f0f778 Merge: 68251e8 da8cf8c Author: Rohit Yadav

[GitHub] nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173787248 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru() {

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173787464 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru()

[GitHub] smeetsr commented on issue #2483: CLOUDSTACK-10303 : test data to nuage_test_data.py + run all tests against simulator

2018-03-12 Thread GitBox
smeetsr commented on issue #2483: CLOUDSTACK-10303 : test data to nuage_test_data.py + run all tests against simulator URL: https://github.com/apache/cloudstack/pull/2483#issuecomment-372304718 All tests in test/integration/plugins/nuagevsp have been executed for simulator See

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173746885 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru()

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173748358 ## File path: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173747369 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru()

[GitHub] marcaurele commented on a change in pull request #2481: CLOUDSTACK-10320 - Invalid pair for response object breaking response parsing

2018-03-12 Thread GitBox
marcaurele commented on a change in pull request #2481: CLOUDSTACK-10320 - Invalid pair for response object breaking response parsing URL: https://github.com/apache/cloudstack/pull/2481#discussion_r173764470 ## File path:

[GitHub] smeetsr opened a new pull request #2483: CLOUDSTACK-10303 : test data to nuage_test_data.py + run all tests against simulator

2018-03-12 Thread GitBox
smeetsr opened a new pull request #2483: CLOUDSTACK-10303 : test data to nuage_test_data.py + run all tests against simulator URL: https://github.com/apache/cloudstack/pull/2483 This is an automated message from the Apache

[GitHub] rhtyd commented on issue #2480: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1

2018-03-12 Thread GitBox
rhtyd commented on issue #2480: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1 URL: https://github.com/apache/cloudstack/pull/2480#issuecomment-372265703 Test LGTM, no failures seen. The only affected public facing services are browser related - (1) apache2 on systemvms and (2)

[GitHub] rhtyd closed pull request #2480: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1

2018-03-12 Thread GitBox
rhtyd closed pull request #2480: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1 URL: https://github.com/apache/cloudstack/pull/2480 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] rhtyd commented on issue #2480: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1

2018-03-12 Thread GitBox
rhtyd commented on issue #2480: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0,1.1 URL: https://github.com/apache/cloudstack/pull/2480#issuecomment-372265783 Merging this PR based on test results and code reviews, thanks.

[cloudstack] branch 4.11 updated: CLOUDSTACK-10319: Prefer TLSv1.2, deprecate TLSv1.0, 1.1 (#2480)

2018-03-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new da8cf8c CLOUDSTACK-10319: Prefer TLSv1.2,

[GitHub] nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173787059 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru() {

[GitHub] nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173787084 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru() {

[GitHub] nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173787025 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru() {

[GitHub] nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173787155 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru() {

[GitHub] nvazquez commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372309550 @blueorangutan package This is an automated message from the Apache Git Service.

[GitHub] greut opened a new pull request #2484: createNetworkACL: number has the wrong doc

2018-03-12 Thread GitBox
greut opened a new pull request #2484: createNetworkACL: number has the wrong doc URL: https://github.com/apache/cloudstack/pull/2484 `number` contains the doc from another field. ![screenshot from 2018-03-12

[GitHub] blueorangutan commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
blueorangutan commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372309660 @nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
blueorangutan commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372322207 Packaging result: ?centos6 ?centos7 ?debian. JID-1768 This is an automated

[GitHub] nvazquez commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372326334 @blueorangutan test This is an automated message from the Apache Git Service. To

[GitHub] nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
nvazquez commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173813069 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru() {

[GitHub] blueorangutan commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
blueorangutan commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372326470 @nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173746941 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru()

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173748753 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru()

svn commit: r25657 - in /dev/cloudstack/4.9.3.1: ./ apache-cloudstack-4.9.3.1-src.tar.bz2 apache-cloudstack-4.9.3.1-src.tar.bz2.asc apache-cloudstack-4.9.3.1-src.tar.bz2.md5 apache-cloudstack-4.9.3.1-

2018-03-12 Thread rohit
Author: rohit Date: Mon Mar 12 10:27:45 2018 New Revision: 25657 Log: Committing release candidate artifacts for 4.9.3.1 to dist/dev/cloudstack in preparation for release vote Added: dev/cloudstack/4.9.3.1/ dev/cloudstack/4.9.3.1/apache-cloudstack-4.9.3.1-src.tar.bz2 (with props)

[GitHub] rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#discussion_r173747190 ## File path: server/src/com/cloud/hypervisor/KVMGuru.java ## @@ -52,10 +59,49 @@ protected KVMGuru()

[GitHub] rafaelweingartner commented on a change in pull request #2481: CLOUDSTACK-10320 - Invalid pair for response object breaking response parsing

2018-03-12 Thread GitBox
rafaelweingartner commented on a change in pull request #2481: CLOUDSTACK-10320 - Invalid pair for response object breaking response parsing URL: https://github.com/apache/cloudstack/pull/2481#discussion_r173751610 ## File path:

[GitHub] khos2ow opened a new pull request #2485: Bump the version of Debian net-installer to 9.4.0

2018-03-12 Thread GitBox
khos2ow opened a new pull request #2485: Bump the version of Debian net-installer to 9.4.0 URL: https://github.com/apache/cloudstack/pull/2485 The new Debian 9.4.0 just released (March 10th). This is an automated message

[GitHub] khos2ow commented on issue #2485: Bump the version of Debian net-installer to 9.4.0

2018-03-12 Thread GitBox
khos2ow commented on issue #2485: Bump the version of Debian net-installer to 9.4.0 URL: https://github.com/apache/cloudstack/pull/2485#issuecomment-372447408 @rhtyd @rafaelweingartner @DaanHoogland @wido unless we still want to remain on 9.3.0, which needs to be pointed to one of the

[GitHub] borisstoyanov commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM

2018-03-12 Thread GitBox
borisstoyanov commented on issue #2482: CLOUDSTACK-10321: CPU Cap for KVM URL: https://github.com/apache/cloudstack/pull/2482#issuecomment-372555727 @blueorangutan test This is an automated message from the Apache Git

[GitHub] rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script

2018-03-12 Thread GitBox
rafaelweingartner commented on issue #2433: CLOUDSTACK-10268: Fix and enhance package script URL: https://github.com/apache/cloudstack/pull/2433#issuecomment-372515754 @borisstoyanov do you have the tests results for this PR?

[GitHub] rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule

2018-03-12 Thread GitBox
rafaelweingartner commented on issue #2475: [CLOUDSTACK-10314] Add Text-Field to each ACL Rule URL: https://github.com/apache/cloudstack/pull/2475#issuecomment-372515401 @khos2ow and @marcaurele can you guys give your opinions here?