[GitHub] cloudstack issue #1786: CLOUDSTACK-9618: Load Balancer configuration page do...

2017-02-15 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1786 LGTM based on code review @karuturi Since this is a network plugin related fix and requires Netscaler, I don't think there is a need to run the BVTs. --- If your project is set up for

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1913 @borisstoyanov @rhtyd @DaanHoogland Can you start vmware tests on this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1818 @ustcweizhou agreed. I am merging this PR. I added your comment to the description of the bug(so that the bug is not closed and can be taken up later) I am merging this for now. Thanks.

Re: Can't create a zone using master

2017-02-15 Thread Rajani Karuturi
This is the order in which they are merged. * 7233ac3 N Merge pull request #977 from ustcweizhou/vm-snapshot 2 weeks ago [Rajani Karuturi] * f10c8bf N Merge pull request #1700 from wido/ipv6-basic-networking 3 weeks ago [Rajani Karuturi] * 4721c53 N Merge pull request #1749 from

[GitHub] cloudstack issue #1792: CLOUDSTACK-9623: Deploying virtual machine fails due...

2017-02-15 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1792 Code changes LGTM @nitin-maharana Can you check on the travis failure? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
For the CloudStack networking person who might investigate this, here is the code I changed per Wei’s comments to get this to work: https://github.com/mike-tutkowski/cloudstack/commit/fdb4fd1e4b4fbb61d8e30006fceca86b8664bb5e On 2/15/17, 11:22 PM, "Tutkowski, Mike"

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1913 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1913 @blueorangutan package --- 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

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1727 @nvazquez Can you resolve conflicts? It might be due to the merge of your other PR #1747 :) Thanks. --- If your project is set up for it, you can reply to this email and have your reply

Re: Attaching more than 14 data volumes to an instance

2017-02-15 Thread Koushik Das
The hardcoded value of 15 needs to be fixed, it can be replaced with getMaxDataVolumesSupported() just above it. Please file a bug and if possible raise a PR as well. -Koushik On 15/02/17, 11:21 PM, "Voloshanenko Igor" wrote: On VM we try to emulate real

[GitHub] cloudstack pull request #1833: BUG-ID:CLOUDSTACK-9678

2017-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1833 --- 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

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Hi Rajani, It looks like a combination of actions has fixed this for me: 1) From Wei: You may remove the "else" part which does not exist in previous versions in ConfigurationManagerImpl.java. 2) I made use of the new system VM template. It looks like we have a bug in ConfigurationManagerImpl.

[GitHub] cloudstack issue #1758: CLOUDSTACK-9588: Add Load Balancer functionality in ...

2017-02-15 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1758 @karuturi This is ready for merge. UI change only, 2 LGTMs, UI screenshot --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request #1913: CLOUDSTACK-9752: [Vmware] Optimization of vol...

2017-02-15 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1913#discussion_r101452884 --- Diff: plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java --- @@ -1577,11 +1577,15 @@ public Answer

[GitHub] cloudstack issue #1834: CLOUDSTACK-9679:Allow master user to manage subordin...

2017-02-15 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1834 Code changes LGTM --- 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

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 Thanks @priyankparihar @serg38 It takes about 6h to execute, can you guys look into resolving the issue with root resize tests and I'll rebuild then. --- If your project is set up

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1913 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-491 --- 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

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1913 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1913 @blueorangutan test centos7 vmware-60u2 --- 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

[GitHub] cloudstack issue #1910: CLOUDSTACK-9748:VPN Users search functionality broke...

2017-02-15 Thread SudharmaJain
Github user SudharmaJain commented on the issue: https://github.com/apache/cloudstack/pull/1910 Code changes looks good. LGTM --- 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

[GitHub] cloudstack issue #1889: CLOUDSTACK-9718: Revamp the dropdown showing lists o...

2017-02-15 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1889 LGTM based on code review --- 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

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Actually Rajani, I just saw the email you sent out about a new system VM template for master. I can try to make use of that new template and see if that fixes my agent issues. > On Feb 15, 2017, at 10:21 PM, Rajani Karuturi wrote: > > unfortunately, running one set of

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1941 @anshul1886 can you check and comment on template related failure? --- 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

[GitHub] cloudstack issue #1834: CLOUDSTACK-9679:Allow master user to manage subordin...

2017-02-15 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1834 ### ACS CI BVT Run **Sumarry:** Build Number 345 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=3 Skipped=7 _Link to logs Folder

[GitHub] cloudstack pull request #1818: CLOUDSTACK-9655 The template which is registe...

2017-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1818 --- 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

[GitHub] cloudstack pull request #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1747 --- 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

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1747 merging --- 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

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1941 @rhtyd That test is failing during cleanup to delete the template but there is no such change in PR which will come into picture while deletion of template. --- If your project is set up

[GitHub] cloudstack issue #1833: BUG-ID:CLOUDSTACK-9678

2017-02-15 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1833 ### ACS CI BVT Run **Sumarry:** Build Number 344 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=1 Skipped=7 _Link to logs Folder

new systemvm template for master

2017-02-15 Thread Rajani Karuturi
FYI, With strongswan[1] and java8[2] changes, a new systemvm template is required on master. Latest systemvm template is available at https://builds.cloudstack.org/job/build-master-systemvm/lastSuccessfulBuild/ Thank you Pierre-Luc Dion for creating the build :) [1]

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Hi Rajani, Just to make sure I'm running with the right code, can you provide me the SHA that you'd like me to test with? Thanks! Mike > On Feb 15, 2017, at 10:21 PM, Rajani Karuturi wrote: > > unfortunately, running one set of tests itself is very time >

Re: Can't create a zone using master

2017-02-15 Thread Rajani Karuturi
unfortunately, running one set of tests itself is very time consuming(multiplied by hypervisors, network modes etc. its impossible to run all) If I remember correctly, wido tested it with PR 1700 (IPV6 in basic networking) Can you try a commit before and a commit after that PR? Thanks, ~

[GitHub] cloudstack pull request #1771: CLOUDSTACK-9611: Dedicating a Guest VLAN rang...

2017-02-15 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1771#discussion_r101451791 --- Diff: server/src/com/cloud/network/NetworkServiceImpl.java --- @@ -3085,9 +3085,10 @@ public GuestVlan

[GitHub] cloudstack issue #1834: CLOUDSTACK-9679:Allow master user to manage subordin...

2017-02-15 Thread pdumbre
Github user pdumbre commented on the issue: https://github.com/apache/cloudstack/pull/1834 Issue Description: Domain admin can not view templates uploaded by his domain users.Only root admin can view templates uploaded by all domain users. Fix: Code changes are done to make

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1941 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #672: Allow VM snapshots and volume snapshots to exist toge...

2017-02-15 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/672 @anshul1886 yes given you've created a new PR, you may close this one. --- 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

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1943 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-488 --- 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

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1943 @blueorangutan test --- 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

[GitHub] cloudstack issue #1900: CLOUDSTACK-8862: Introduced new state attaching for ...

2017-02-15 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1900 Rebased with current master. --- 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

[GitHub] cloudstack issue #1927: ipv6: Set IPv6 CIDR and Gateway in 'nic' profile

2017-02-15 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1927 LGTM, let's merge this blocker @karuturi --- 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

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1943 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-489 --- 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

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1941 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-490 --- 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

[GitHub] cloudstack issue #1834: CLOUDSTACK-9679:Allow master user to manage subordin...

2017-02-15 Thread pdumbre
Github user pdumbre commented on the issue: https://github.com/apache/cloudstack/pull/1834 Test Results Passed: https://travis-ci.org/apache/cloudstack/builds/184533080 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1833: BUG-ID:CLOUDSTACK-9678

2017-02-15 Thread pdumbre
Github user pdumbre commented on the issue: https://github.com/apache/cloudstack/pull/1833 Test Results Passed: https://travis-ci.org/apache/cloudstack/builds/201069131 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread priyankparihar
Github user priyankparihar commented on the issue: https://github.com/apache/cloudstack/pull/1813 Hi @pdion891, @borisstoyanov, @anshul1886 and @pdube, Thanks for your valuable suggestion and help. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #859: Bug-ID: CLOUDSTACK-8882: calculate network offering u...

2017-02-15 Thread kishankavala
Github user kishankavala commented on the issue: https://github.com/apache/cloudstack/pull/859 @DaanHoogland , Test creates two UsageNetworkOfferings, both with same network offering but different nicIds. Test case is verified by checking that 2 usage records are created instead of

Attaching more than 14 data volumes to an instance

2017-02-15 Thread Friðvin Logi Oddbjörnsson
CloudStack is currently limiting the number of data volumes, that can be attached to an instance, to 14. More specifically, this limitation relates to the device ids that CloudStack considers valid for data volumes. In method VolumeApiServiceImpl.getDeviceId(long, Long), only device ids 1, 2, and

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1941 @blueorangutan help --- 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

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1941 @rhtyd I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test [mgmt os] [hypervisor] [additional tests] Mgmt OS options: ['centos6',

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1941 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 @blueorangutan test centos7 vmware-60u2 --- 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

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-62sp1) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 @blueorangutan test centos7 xenserver-62sp1 --- 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

[GitHub] cloudstack issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-15 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1818 @ustcweizhou mm.. I am confused now :) a new button definitely makes sense but, a better error message doesn't worsen the current situation. So, until we have new button, we can have this

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1941 @blueorangutan test centos7 xenserver-65sp1 --- 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

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 @blueorangutan test centos7 vmware-65 --- 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

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 @borisstoyanov unsupported parameters provided. Supported mgmt server os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, kvm-centos7, kvm-ubuntu,

[GitHub] cloudstack pull request #1942: CLOUDSTACK-9784 : GPU detail not displayed in...

2017-02-15 Thread niteshsarda
GitHub user niteshsarda opened a pull request: https://github.com/apache/cloudstack/pull/1942 CLOUDSTACK-9784 : GPU detail not displayed in GPU tab of management server UI. ISSUE == When GPU tab of the host is selected on the management server UI, no

[GitHub] cloudstack pull request #1379: CLOUDSTACK-8324: config drive data set/get sc...

2017-02-15 Thread jayapalu
Github user jayapalu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1379#discussion_r101223426 --- Diff: setup/bindir/cloud-set-guest-sshkey-configdrive.in --- @@ -0,0 +1,107 @@ +#!/bin/bash +# +# Init file for SSH Public Keys

[GitHub] cloudstack issue #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

2017-02-15 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1379 These scripts CI tests are not needed --- 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

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1918 @blueorangutan package --- 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

[GitHub] cloudstack pull request #1943: CLOUDSTACK-9339: Send correct network type to...

2017-02-15 Thread ProjectMoon
GitHub user ProjectMoon opened a pull request: https://github.com/apache/cloudstack/pull/1943 CLOUDSTACK-9339: Send correct network type to router config This forces the network type (nw_type) of the IP address being sent into router configuration scripts to be lowercase. All of

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-15 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1847 @nvazquez What would you like to do? --- 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

[GitHub] cloudstack pull request #669: Made the adding new keyboard language support ...

2017-02-15 Thread anshul1886
GitHub user anshul1886 reopened a pull request: https://github.com/apache/cloudstack/pull/669 Made the adding new keyboard language support easier https://issues.apache.org/jira/browse/CLOUDSTACK-8665 This branch has implemented following improvements in console proxy

[GitHub] cloudstack pull request #1379: CLOUDSTACK-8324: config drive data set/get sc...

2017-02-15 Thread jayapalu
Github user jayapalu commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1379#discussion_r101223451 --- Diff: setup/bindir/cloud-set-guest-sshkey-configdrive.in --- @@ -0,0 +1,107 @@ +#!/bin/bash +# +# Init file for SSH Public Keys

[GitHub] cloudstack issue #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

2017-02-15 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1379 These scripts are for config drive feature. Deploy vm in shared network without services then vm with config drive will be deployed. Place these scripts in vm template to set VM password

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1918 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1943 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1943 @blueorangutan package --- 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

[GitHub] cloudstack pull request #1813: CLOUDSTACK-9604: Root disk resize support for...

2017-02-15 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1813#discussion_r101231917 --- Diff: plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixRequestWrapperTest.java --- @@ -436,7

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 @blueorangutan package --- 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

[GitHub] cloudstack pull request #669: Made the adding new keyboard language support ...

2017-02-15 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/669 --- 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

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1918 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-487 --- 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

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1918 @blueorangutan test --- 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

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1918 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request #1813: CLOUDSTACK-9604: Root disk resize support for...

2017-02-15 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1813#discussion_r101232184 --- Diff: server/src/com/cloud/vm/UserVmManagerImpl.java --- @@ -3520,27 +3520,17 @@ public UserVmVO doInTransaction(TransactionStatus status)

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1941 Thanks @anshul1886 @blueorangutan package --- 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

[GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...

2017-02-15 Thread rashmidixit
Github user rashmidixit commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1889#discussion_r101275471 --- Diff: api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java --- @@ -206,7 +206,7 @@ public void execute() { }

[GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...

2017-02-15 Thread rashmidixit
Github user rashmidixit commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1889#discussion_r101275818 --- Diff: api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java --- @@ -206,7 +206,7 @@ public void execute() { }

[GitHub] cloudstack-www pull request #35: Update "who" page to reflect the new additi...

2017-02-15 Thread rafaelweingartner
GitHub user rafaelweingartner opened a pull request: https://github.com/apache/cloudstack-www/pull/35 Update "who" page to reflect the new additions to PMC and committers Self descriptive PR title ;) You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack-www issue #35: Update "who" page to reflect the new additions to ...

2017-02-15 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack-www/pull/35 @swill, could you take a look in this PR? --- 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

[GitHub] cloudstack issue #1941: CLOUDSTACK-8663: Fixed various issues to allow VM sn...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1941 Trillian test result (tid-829) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 42936 seconds Marvin logs:

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Thanks, Wei. I will give that a shot and report back. On 2/15/17, 1:26 PM, "Wei ZHOU" wrote: Mike, You may remove the "else" part which does not exist in previous versions in ConfigurationManagerImpl.java. -Wei 2017-02-15 20:55 GMT+01:00

Re: Can't create a zone using master

2017-02-15 Thread Wei ZHOU
Mike, You may remove the "else" part which does not exist in previous versions in ConfigurationManagerImpl.java. -Wei 2017-02-15 20:55 GMT+01:00 Tutkowski, Mike : > OK, so, what I plan to do is jump back a bunch of commits and see if it > works. If it does, I can

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1918 Trillian test result (tid-824) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34138 seconds Marvin logs:

[GitHub] cloudstack issue #1836: [4.10/master] Smoketest Health

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1836 Trillian test result (tid-823) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 31872 seconds Marvin logs:

[GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...

2017-02-15 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1889#discussion_r101313545 --- Diff: ui/scripts/ui-custom/migrate.js --- @@ -0,0 +1,127 @@ +// Licensed to the Apache Software Foundation (ASF) under one --- End diff

Re: Attaching more than 14 data volumes to an instance

2017-02-15 Thread Voloshanenko Igor
I think explanation very easy. PCI itself can handle up to 32 devices. If you run lspci inside empty (fresh created) VM - you will see, that 8 slots already occupied [root@test-virtio-blk ~]# lspci 00:00.0 Host bridge: Intel Corporation 440FX - 82441FX PMC [Natoma] (rev 02) 00:01.0 ISA bridge:

Re: Attaching more than 14 data volumes to an instance

2017-02-15 Thread Rafael Weingärtner
I thought that on a VM we would not be bound by PCI limitations. Interesting explanations, thanks. On Wed, Feb 15, 2017 at 12:19 PM, Voloshanenko Igor < igor.voloshane...@gmail.com> wrote: > I think explanation very easy. > PCI itself can handle up to 32 devices. > > If you run lspci inside

Re: Attaching more than 14 data volumes to an instance

2017-02-15 Thread Voloshanenko Igor
On VM we try to emulate real hardware ))) So any device honor specification In this case PCI :) To be honest we can increase limits by adding multifunctional devices or migrate to virtio-iscsi-blk But as for me - 14 disks more than enough now About 3 for cdrom. I will check . I think CDROM

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1943 Trillian test result (tid-825) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28120 seconds Marvin logs:

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-15 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1943 LGTM based on code review. --- 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

Re: Attaching more than 14 data volumes to an instance

2017-02-15 Thread Rafael Weingärtner
I hate to say this, but probably no one knows why. I looked at the history and this method has always being like this. The device ID 3 seems to be something reserved, probably for Xen tools (big guess here)? Also, regarding the limit; I could speculate two explanations for the limit. A developer

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-15 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1918 @nvazquez no problem. I think it is ok to use everything that is available because the service offering is not restricting the CPU usage. @nvazquez would it be possible for

[GitHub] cloudstack issue #1818: CLOUDSTACK-9655 The template which is registered in ...

2017-02-15 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1818 @karuturi I think it should work like (1) list template status in all zones (now only one is shown) (2) delete template from a zone will pass zoneid and remove template from the

Re: Attaching more than 14 data volumes to an instance

2017-02-15 Thread Syed Ahmed
Know that 0 is reserved for the ROOT disk and 3 is for the CD-ROM for attaching ISOs On Wed, Feb 15, 2017 at 12:20 Voloshanenko Igor wrote: > I think explanation very easy. > PCI itself can handle up to 32 devices. > > If you run lspci inside empty (fresh created) VM

Re: Can't create a zone using master

2017-02-15 Thread Tutkowski, Mike
Hi Wei, OK, the situation is partially improved now: I can create a zone/pod/cluster. However, my SSVM and CPVM don’t report the agent as Running. From cloud.log in SSVM: Wed Feb 15 21:12:59 UTC 2017 Incompleted parameters STORAGE_IP:, STORAGE_NETMASK:, STORAGE_CIDR:. Cannot setup storage

  1   2   >