[GitHub] cloudstack issue #1689: Switched to the official SolidFire SDK for Python

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1689 @mike-tutkowski is the SolidFire Python SDK available via pip? If so, could you please add it to `requirements.txt` in the project root directory? --- If your project is set up for it, you

[GitHub] cloudstack issue #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1686 @rhytd Current implementation has a bug as well that returns GuestOsId in place of GuestOsTypeId. I am for not only switching to UUID but also for fixing it and returning correct output. ---

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1542 @rafaelweingartner @jburwell @karuturi Looks like this PR has everything to get merged. Can we do it by the release freeze? --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1615 @jburwell 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 pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80700964 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80702827 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80701833 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80704271 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80701499 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1677 @jburwell It would be nice to include this PR to the upcoming release. Are there anything else needed ? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1593: CLOUDSTACK-9417: Usage module refactoring

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1593 @rafaelweingartner @rhtyd Do we make any decision on this? The scope of this PR was very limited initially. It would be much easier if we don't expand it too far. --- If your project is set up

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1615 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-33 --- 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 #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-27 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1684 @abhinandanprateek did you ran a fresh install of 4.9.x or did an upgrade from example 4.4.4 ? The way I'm testing is as follow, in case I do something wrong: 1) ``git

[GitHub] cloudstack issue #1593: CLOUDSTACK-9417: Usage module refactoring

2016-09-27 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1593 @serg38 giving your explanations I am ok with your proposal. Just one addendum, I think we should document it as much as possible. --- If your project is set up for it, you can reply

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1677 @serg38 I just kicked a smoke test run on VMware. Assuming that passes, it will be good for 4.9.1.0. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80701705 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80702685 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -1197,3 +1205,148 @@ def test_10_destroy_cpvm(self): # Call to verify cloud process is

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1615 @rhtyd can you add the configuration data to blueorangutan for the changes to `test_ssvm.py`? @serg38 @nvazquez once my review comments are addressed and blueorganutan has been

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1615 @jburwell @karuturi @rafaelweingartner Can you check if this PR can be merged by the upcoming release? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1677: CLOUDSTACK-8830 - [Vmware] VM snapshot fails for 12 ...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1677 @blueorangutan test centos7 vmware55u3 --- 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 #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1615 @koushik-das are you LGTM 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 your project does not have this feature

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1542 @serg38 @nvazquez I apologize that I am late to the review. I would like to see a Marvin test added that verifies that nested virtualization works when the global setting is enabled (i.e.

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1615 @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 and

[GitHub] cloudstack pull request #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Ma...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1615#discussion_r80702139 --- Diff: test/integration/smoke/test_ssvm.py --- @@ -42,12 +43,19 @@ class TestSSVMs(cloudstackTestCase): def setUp(self): +

[GitHub] cloudstack issue #1615: CLOUDSTACK-9438: Fix for CLOUDSTACK-9252 - Make NFS ...

2016-09-27 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1615 Yes LGTM from my side. --- 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 #1669: Make CloudStack JSP-free

2016-09-27 Thread milamberspace
Github user milamberspace commented on the issue: https://github.com/apache/cloudstack/pull/1669 @rhtyd in the normal case, if the key label.affinity.groups doesn't exists, the default language is display ("Affinity Groups" in this case). The PR is a regression for this case (vs the

[GitHub] cloudstack issue #1673: CLOUDSTACK-9071: Properly parse stats.output.uri in ...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1673 @wido sorry for the delayed response -- I lost track of this browser tab. The quickest way may be to create a new branch from 4.8, cherry pick this commit into it, push the new branch, and

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-31 --- 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 #1689: Switched to the official SolidFire SDK for Python

2016-09-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1689 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 wishes so, or if the

[GitHub] cloudstack pull request #1685: IGNORE this PR

2016-09-27 Thread rhtyd
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1685 --- 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 is

[GitHub] cloudstack issue #1685: IGNORE this PR

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1685 @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 #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1659 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-32 --- 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 #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1686 @rhtyd @serg38 while I agree that this change makes logical sense, it breaks the API compatibility which we

[GitHub] cloudstack issue #1678: CLOUDSTACK-9503: Increased the VR script timeout. Mo...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1678 @abhinandanprateek it appears that there is a Findbugs issue that needs to be addressed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-09-27 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1642 @blueorangutan @jburwell any update on the tests? --- 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 #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-27 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1684 @pdion891 I am unable to reproduce the issue. All the authenticators configured are tried. I find it strange that there are no logs. Will it be possible to take a second look. I

[GitHub] cloudstack issue #1674: CLOUDSTACK-9460: For long running transactions, if t...

2016-09-27 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1674 ### ACS CI BVT Run **Sumarry:** Build Number 104 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=2 Skipped=4 _Link to logs Folder

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1669 @milamberspace I agree because it appears to remove the need to maintain the Python compilation script which reduces complexity and doesn't add any more build time. --- If your project is

[GitHub] cloudstack issue #1693: CLOUDSTACK-9505: Fix test_deploy_vgpu_enabled tests ...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1693 LGTM on the testing ``` test DeployVM in anti-affinity groups for project ... === TestName: test_DeployVmAntiAffinityGroup_in_project | Status : SUCCESS === test DeployVM in

[GitHub] cloudstack issue #1659: CLOUDSTACK-9339 Virtual Routers don't handle Multipl...

2016-09-27 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1659 Did a code walk-thru and review with @murali-reddy, 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

Re: [GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-27 Thread roarain...@126.com
thank you for replying。 But CloudStack4.6 mysql database cloud table configuration has no this name. mysql> use cloud Reading table information for completion of table and column names You can turn off this feature to get a quicker startup with -A Database changed mysql> select

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1542 @jburwell The only BUILTIN Centos template for Vmware is 5.3 that doesn't have KVM support to my knowledge. Are you proposing to introduce another BUILTIN template so that it is available for

[GitHub] cloudstack issue #1678: CLOUDSTACK-9503: Increased the VR script timeout. Mo...

2016-09-27 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1678 @jburwell there was some issue with jenkins, force pushing PR resolved it. --- 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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1542 @rhytd can you help @serg38 craft a template with KVM installed and the virsh commands to create a nested VM? --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1542 @jburwell @rhtyd Thanks in advance --- 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 #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1542 @serg38 no problem --- 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 #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed #1692 is testing now. It addresses ping test related failures in the smoke test suite. I am hoping the tests pass, and we can merge it first thing tomorrow. Once it has, we can rebase

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1542 @serg38 the goal is not only verify that functionality works now, but in the future to protect against regressions. My thinking is that we could deploy a CentOS VM, install KVM, and verify

RE: [DISCUSS] Replacing the VR

2016-09-27 Thread Marty Godsey
Matthew, From an Engineers viewpoint and someone who has a production deployment of CloudStack now offering VPC services to providers, I think the approach of a plugin is a wise move. As you mentioned, a full blown replacement of the VR, since it is such a vital piece of ACS, would require

[GitHub] cloudstack issue #1642: CLOUDSTACK-9504: System VMs on Managed Storage

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1642 @karuturi awaiting feature enhancement to blueorangutan which was delayed due to an medical emergency. Also, I want to wait to perform further testing until #1692 is tested and merged as it

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-27 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1684 @pdion891 The root of this issue is that 4.9 uses dynamic api checker, which by default is not enabled. So even when the login succeed it does not go anywhere as the access to any api

Re: [DISCUSS] Replacing the VR

2016-09-27 Thread Matthew Smart
Will, I think that would be very helpful to me at least and for posterity for sure. I am in the process of rolling out my first production deployment of Cloudstack so I have been busier than expected (plus I have been jumping back and forth between different offices). What I intended to

[GitHub] cloudstack issue #1593: CLOUDSTACK-9417: Usage module refactoring

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1593 @serg38 I have this PR on a list to re-review. In my view, using a global setting for this value is whole inappropriate. I have been busy with other items, and haven't had a chance to get

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1694#discussion_r80714655 --- Diff: server/test/com/cloud/storage/listener/StoragePoolMonitorTest.java --- @@ -0,0 +1,80 @@ +// Licensed to the Apache Software Foundation

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

2016-09-27 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1694#discussion_r80714972 --- Diff: server/src/com/cloud/storage/listener/StoragePoolMonitor.java --- @@ -99,12 +99,12 @@ public void processConnect(Host host, StartupCommand

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed could you please squash your commits? Once that is done, the following items will be need to be completed: * [ ] Re-execute functional test since the code has changed since they

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1669 @rhtyd @milamberspace I have started a conversation on dev@ to discuss transitioning to the JSON format. Is acceptable to hold this PR until we come to consensus on that thread? --- If your

[GitHub] cloudstack issue #1689: Switched to the official SolidFire SDK for Python

2016-09-27 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1689 @jburwell I just added that to requirements.txt. I also put a link to solidfire-sdk-python 1.1.0.92 in the description of this PR. --- If your project is set up for it, you can reply to

L10N Translation File Format

2016-09-27 Thread John Burwell
All, As part of the effort to remove the requirement for JSPs and streamline the development process, Milamber has raised the option of replacing the message*.properties files with a JSON key/value format [1]. As I understand it, Transifex supports this file format, and it appears that it

Re: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-09-27 Thread John Burwell
Will, Have we had an official 4.8.1 release? Thanks, -John > john.burw...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK @shapeblue On Aug 15, 2016, at 11:48 AM, Will Stevens wrote: > > Thanks Milamber. I will add my own

[GitHub] cloudstack issue #1689: Switched to the official SolidFire SDK for Python

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1689 @mike-tutkowski excellent -- thanks for adding that configuration bit. I am LGTM once the Jenkins and Travis complete successfully. @karuturi @rhtyd are you able to review for a

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1681 @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-27 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @blueorangutan test centos7 vmware55u3 --- 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 #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-27 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1681 @jburwell function 'findRouterEthDeviceIndex' which is modified, is invoked when ever a new tier to VPC is added (virtual NIC is hot-plugged). Many VPC tests run againest the VMware

Re: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-09-27 Thread Will Stevens
We did have an official release in the sense that it was voted on and released, but I did not make a big scene about. You guys were working on 4.8.2 and I just wanted to get it released before you guys modified the branch so the release branching didn't get messed up. *Will STEVENS* Lead

[GitHub] cloudstack issue #1692: fix ping tests to properly recognise successful ping...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1692 Looks like the blueorangutan build didn't complete properly. I am re-kicking it now. @blueorangutan test matrix --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1645 @nathanejohnson @kiwiflyer do you have test results to share for KVM + Ceph? If not, we need to rollback this PR until we get test confirmation of the functionality. --- If your project is

[GitHub] cloudstack issue #1542: CLOUDSTACK-9379: Support nested virtualization at VM...

2016-09-27 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1542 @jburwell Creating Marvin test for this PR will be a huge undertaking. It would require building and configuring a hypervisor in a deployed VM and then creating VM in VM. Considering storage,

[GitHub] cloudstack issue #1692: fix ping tests to properly recognise successful ping...

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1692 @jburwell a Trillian-Jenkins matrix job (centos6 mgmt + xs56sp1, centos7 mgmt + vmware55u3, ubuntu mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for

Re: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-09-27 Thread Will Stevens
There is a different thread (take two) which has the official votes and such. I did not do an announcement for the release though. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Tue, Sep

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-27 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1645 @jburwell , test results were posted earlier in the thread. --- 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 #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1600 @jburwell Squashed the commits. You can run the smoke tests on this. This change is mostly on the managed storage side so running any volume related tests would be helpful apart from the tests

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1694 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-30 --- 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 #1678: CLOUDSTACK-9503: Increased the VR script timeout. Mo...

2016-09-27 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue: https://github.com/apache/cloudstack/pull/1678 @karuturi Jenkins fail on find-bug: [ERROR] Failed to execute goal org.codehaus.mojo:findbugs-maven-plugin:3.0.3:findbugs (findbugs) on project cloud-plugin-network-bigswitch:

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 Thanks @milamberspace I'll fix them, the issues come up due to differences in how java/jsp urlencoded the translation strings and how the python based messages.properties->l10n js file converter

[GitHub] cloudstack issue #1669: Make CloudStack JSP-free

2016-09-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1669 @milamberspace I've fixed the issues that were raised and few other unescaped characters I found from the generated l10n js files. The `label.affinity.groups` translation does not exist in korean

[GitHub] cloudstack pull request #1694: CLOUDSTACK-9509: Host Connects Without Storag...

2016-09-27 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1694 CLOUDSTACK-9509: Host Connects Without Storage KVM hosts on shared storage failure was accepted by mgmt server with the host state as Up, even though there was no primary/shared storage

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1694 @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

[DISCUSS] Fixing API response to return uuid instead of integer for listVMs

2016-09-27 Thread Rohit Yadav
All, The listVirtualMachines API response contains a guest os_type_id field which is an integer, and therefore not useful or easily consumable [1]. All new/modern APIs for example should only return UUIDs for client/users to consume, returning integers (as in database id) may not be

[GitHub] cloudstack issue #1686: CLOUDSTACK-9507: ListVM response's guest_os id shoul...

2016-09-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1686 @serg38 I mean the os_type_id, which was previously returned as an integer but should be returned as a uuid. The uuid would be consumeable by the listGuestOsMapping API. --- If your project is

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1645 @nathanejohnson to confirm, the test results in your [26 Aug comment](https://github.com/apache/cloudstack/pull/1645#issuecomment-242715452) comment were from a run on KVM + Ceph? --- If

[GitHub] cloudstack issue #1645: On snapshot backup, this converts the rbd raw format...

2016-09-27 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1645 @jburwell that is correct, the lab I tested on is running KVM + ceph. --- 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 #1645: On snapshot backup, this converts the rbd raw format...

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1645 @nathanejohnson thank you for your confirmation --- 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 #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1681 Trillian test result (tid-58) Environment: vmware-55u3 (x2), Advanced Networking Total time taken: 33200 seconds Marvin logs:

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1600 @jburwell 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 #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1600 @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 and

Re: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-09-27 Thread John Burwell
Will, We make a announcement for 4.8.2.0. I just wanted to make sure that there was a 4.8.1 since we are working on 4.8.2.0 … Thanks, -John > john.burw...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London VA WC2N 4HSUK @shapeblue On Sep 27, 2016, at 1:09 PM, Will

[GitHub] cloudstack issue #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1600 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-34 --- 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 #1600: Support Backup of Snapshots for Managed Storage

2016-09-27 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1600 @syed @jburwell I should be able to run the tests toward the end of the week or during the weekend. At the moment, my resources in the lab are devoted to other CloudStack dev activities.

Re: [RESULT][VOTE] Apache CloudStack 4.8.1

2016-09-27 Thread Will Stevens
Yes. That is why I did the vote and release, even though we did not advertise it, it makes things easier for dev going forward. :) On Sep 27, 2016 5:05 PM, "John Burwell" wrote: > Will, > > We make a announcement for 4.8.2.0. I just wanted to make sure that there >

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-27 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1694 @rhtyd a Trillian-Jenkins test job (centos6 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 reply

[GitHub] cloudstack issue #1694: CLOUDSTACK-9509: Host Connects Without Storage

2016-09-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1694 @blueorangutan test centos6 kvm-centos7 --- 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 #1678: CLOUDSTACK-9503: Increased the VR script timeout. Mo...

2016-09-27 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1678 I think the build timed out. > [INFO] --- findbugs-maven-plugin:3.0.3:findbugs (findbugs) @ cloud-plugin-network-bigswitch --- > [INFO] Fork Value is true > Build timed out

Re: ACS 4.9 + VMware: Failed to deploy VPC VR due to IPAssoc failure

2016-09-27 Thread Prashanth Manthena
Hi Murali, Thank you for the quick response. We will try out your PR, and will keep you updated. I have couple of questions for you: Do you know of any other issues in ACS 4.9/master with VMware ? Is ACS 4.9/master with VMware being supported and qualified by the community ? Thanking you in