[GitHub] anuragaw commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance

2018-12-24 Thread GitBox
anuragaw commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance URL: https://github.com/apache/cloudstack/pull/3101#issuecomment-449790458 Trillian test results seem fine. This is an

[GitHub] blueorangutan commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance

2018-12-24 Thread GitBox
blueorangutan commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance URL: https://github.com/apache/cloudstack/pull/3101#issuecomment-449777316 Trillian test result (tid-3274) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rhtyd opened a new issue #3111: Allow extending and shrinking of public IP address range

2018-12-24 Thread GitBox
rhtyd opened a new issue #3111: Allow extending and shrinking of public IP address range URL: https://github.com/apache/cloudstack/issues/3111 Currently, there is not API to update or shrink/expand the public IP range for an advanced zone. Add an API to allow that. # ISSUE TYPE

[GitHub] rafaelweingartner commented on a change in pull request #3099: Add resource UUID instead of resource id in resourcedetails api (#2527)

2018-12-24 Thread GitBox
rafaelweingartner commented on a change in pull request #3099: Add resource UUID instead of resource id in resourcedetails api (#2527) URL: https://github.com/apache/cloudstack/pull/3099#discussion_r243851497 ## File path:

[GitHub] rafaelweingartner commented on a change in pull request #3099: Add resource UUID instead of resource id in resourcedetails api (#2527)

2018-12-24 Thread GitBox
rafaelweingartner commented on a change in pull request #3099: Add resource UUID instead of resource id in resourcedetails api (#2527) URL: https://github.com/apache/cloudstack/pull/3099#discussion_r243851497 ## File path:

[GitHub] blueorangutan commented on issue #3107: Add api arg validator for create project api (#3097)

2018-12-24 Thread GitBox
blueorangutan commented on issue #3107: Add api arg validator for create project api (#3097) URL: https://github.com/apache/cloudstack/pull/3107#issuecomment-449761525 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #3107: Add api arg validator for create project api (#3097)

2018-12-24 Thread GitBox
rhtyd commented on issue #3107: Add api arg validator for create project api (#3097) URL: https://github.com/apache/cloudstack/pull/3107#issuecomment-449761488 @blueorangutan package This is an automated message from the

[cloudstack] branch master updated (2699586 -> 3279707)

2018-12-24 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 2699586 network: Send userdata to Virtual Router if IPv6 is enabled (#3100) add 3279707 api: Throw

[GitHub] rhtyd closed pull request #3108: Throw InvalidParameterValueException for failing ApiArgValidator

2018-12-24 Thread GitBox
rhtyd closed pull request #3108: Throw InvalidParameterValueException for failing ApiArgValidator URL: https://github.com/apache/cloudstack/pull/3108 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

[GitHub] rhtyd commented on a change in pull request #3095: Prevent corner case for infinite PrepareForMaintenance

2018-12-24 Thread GitBox
rhtyd commented on a change in pull request #3095: Prevent corner case for infinite PrepareForMaintenance URL: https://github.com/apache/cloudstack/pull/3095#discussion_r243851027 ## File path: server/src/com/cloud/resource/ResourceManagerImpl.java ## @@ -271,6 +272,8 @@

[GitHub] rhtyd commented on a change in pull request #3095: Prevent corner case for infinite PrepareForMaintenance

2018-12-24 Thread GitBox
rhtyd commented on a change in pull request #3095: Prevent corner case for infinite PrepareForMaintenance URL: https://github.com/apache/cloudstack/pull/3095#discussion_r243851027 ## File path: server/src/com/cloud/resource/ResourceManagerImpl.java ## @@ -271,6 +272,8 @@

[GitHub] rhtyd commented on issue #3105: VmwareStorageLayoutHelper throws StackOverflowError fix

2018-12-24 Thread GitBox
rhtyd commented on issue #3105: VmwareStorageLayoutHelper throws StackOverflowError fix URL: https://github.com/apache/cloudstack/pull/3105#issuecomment-449760881 LGTM, @dhlaluku since this is a bugfix do you think 4.11 can benefit from it?

[GitHub] rhtyd closed pull request #3106: marvin: add missing test data for test_migration smoketest

2018-12-24 Thread GitBox
rhtyd closed pull request #3106: marvin: add missing test data for test_migration smoketest URL: https://github.com/apache/cloudstack/pull/3106 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

[cloudstack] branch 4.11 updated: marvin: add missing test data for test_migration smoketest (#3106)

2018-12-24 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 68b4b84 marvin: add missing test data for

[GitHub] rhtyd closed pull request #3100: ipv6: Send userdata to Virtual Router if IPv6 is enabled

2018-12-24 Thread GitBox
rhtyd closed pull request #3100: ipv6: Send userdata to Virtual Router if IPv6 is enabled URL: https://github.com/apache/cloudstack/pull/3100 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

[cloudstack] branch master updated: network: Send userdata to Virtual Router if IPv6 is enabled (#3100)

2018-12-24 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 The following commit(s) were added to refs/heads/master by this push: new 2699586 network: Send userdata to Virtual

[GitHub] anuragaw commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance

2018-12-24 Thread GitBox
anuragaw commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance URL: https://github.com/apache/cloudstack/pull/3101#issuecomment-449751709 Thanks for the clarification @kiwiflyer !

[GitHub] kiwiflyer commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance

2018-12-24 Thread GitBox
kiwiflyer commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance URL: https://github.com/apache/cloudstack/pull/3101#issuecomment-449751491 @anuragaw The queues feature was introduced into libvirt back in 2013, so it's been available for a while. We

[GitHub] anuragaw commented on issue #3100: ipv6: Send userdata to Virtual Router if IPv6 is enabled

2018-12-24 Thread GitBox
anuragaw commented on issue #3100: ipv6: Send userdata to Virtual Router if IPv6 is enabled URL: https://github.com/apache/cloudstack/pull/3100#issuecomment-449748163 Code LGTM This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance

2018-12-24 Thread GitBox
blueorangutan commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance URL: https://github.com/apache/cloudstack/pull/3101#issuecomment-449747455 @anuragaw a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] anuragaw commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance

2018-12-24 Thread GitBox
anuragaw commented on issue #3101: kvm: Set amount of queues for Virtio SCSI driver to vCPU of Instance URL: https://github.com/apache/cloudstack/pull/3101#issuecomment-449747371 @blueorangutan test This is an automated

[GitHub] anuragaw commented on issue #3106: marvin: add missing test data for test_migration smoketest

2018-12-24 Thread GitBox
anuragaw commented on issue #3106: marvin: add missing test data for test_migration smoketest URL: https://github.com/apache/cloudstack/pull/3106#issuecomment-449746448 Code LGTM. This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #3095: Prevent corner case for infinite PrepareForMaintenance

2018-12-24 Thread GitBox
blueorangutan commented on issue #3095: Prevent corner case for infinite PrepareForMaintenance URL: https://github.com/apache/cloudstack/pull/3095#issuecomment-449741647 Trillian test result (tid-3273) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] rhtyd opened a new issue #3110: Root volume resize performed on scaling service offering

2018-12-24 Thread GitBox
rhtyd opened a new issue #3110: Root volume resize performed on scaling service offering URL: https://github.com/apache/cloudstack/issues/3110 When service offering of a (stopped) VM is changed using the `scaleVirtualMachine` API, its root disk get resized. In case the primary storage

[GitHub] brankomijuskovic opened a new issue #3109: Account removal triggers event of unrelated user (from the different account) removal which has the same ID as account which is being deleted

2018-12-24 Thread GitBox
brankomijuskovic opened a new issue #3109: Account removal triggers event of unrelated user (from the different account) removal which has the same ID as account which is being deleted URL: https://github.com/apache/cloudstack/issues/3109 # ISSUE TYPE * Bug Report

[GitHub] rafaelweingartner commented on issue #351: This branch implements the CSV and native HA support for Hyper-V

2018-12-24 Thread GitBox
rafaelweingartner commented on issue #351: This branch implements the CSV and native HA support for Hyper-V URL: https://github.com/apache/cloudstack/pull/351#issuecomment-449715347 Hello Haodeon, Maybe this feature that you have read about depends on this PR. However, I am not 100%