[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161337904 Okay... no panic! I might have been some expunge global settings that is deployed on all of our test environments. It has been reverted and I will run the

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-161320788 Sure, I couldn't find you mail address, mine is in my profile --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/943#discussion_r46436064 --- Diff: server/test/async-job-component.xml --- @@ -108,11 +108,7 @@ --- End diff -- Not sure about this,

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161310807 Ping @remibergsma @DaanHoogland @ustcweizhou @bhaisaab The test fails on latest 4.6 branch! ``` === TestName: test_01_stop_vm | Status

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-161315034 Thanks @miguelaferreira I really appreciate that! Thanks a lot for your help! I'll try to make it before the deadline. I'm not that into Python but I'll do my

[GitHub] cloudstack pull request: Removal of class AgentBasedStandaloneCons...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/855#issuecomment-161352903 LGTM (only code review), but we need to test it and verify console proxy functionality --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1108#issuecomment-161353281 We've used _var for global or class variables throughout the codebase, should we keep it? LGTM (only code review) otherwise --- If your project is set up for

[GitHub] cloudstack pull request: Create test cases to getPatchFilePath met...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/944#issuecomment-161352035 LGTM (just 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

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-161317452 I'm working on adding a new test case to the test file we already have, but unfortunately I don't know how to cover your change. What I do know is how

[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/942#issuecomment-161352291 LGTM (only 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

[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1122#issuecomment-161362360 ``` === TestName: test_24_public_IP_tag | Status : SUCCESS === Cleanup resources used ``` LGTM --- If your project is set up for it,

Re: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Daan Hoogland
I am having a problem, When trying to add a vpc from the UI the zone dropdown is empty. when trying to add an isolated network the dropdown does contain the wanted choice. Any body else seeing this? On Mon, Nov 30, 2015 at 11:12 PM, Remi Bergsma wrote: > Hi all, >

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

2015-12-02 Thread jharshman
Github user jharshman commented on the pull request: https://github.com/apache/cloudstack/pull/1013#issuecomment-161367693 @DaanHoogland from reading the mailing lists. Looks like there is a freeze for 4.7 features starting Dec 7. Can we get this change merged in before then? The

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-12-02 Thread rafaelweingartner
Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/943#discussion_r46455242 --- Diff: server/test/async-job-component.xml --- @@ -108,11 +108,7 @@ --- End diff -- @bhaisaab ,

[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

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

Build failed in Jenkins: build-master-jdk18 #418

2015-12-02 Thread jenkins
See Changes: [sanjeev] Add marvin test for bug CS-38356 Bug-Id: CS-38356 Reviewed-By: Self [sanjeev] Incorporated review comments from the PR -- [...truncated 398 lines...] Tests run: 2,

[GitHub] cloudstack pull request: Removal of class AgentBasedStandaloneCons...

2015-12-02 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/855#issuecomment-161389994 @bhaisaab, we definitely should test that. Since there were none references of that class, we suggested removing it. ACS uses spring, and since there is

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

2015-12-02 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1108#issuecomment-161388418 @bhaisaab, I assume the use of _ was introduced probably by someone that comes from a C/C++ background. IMHO that does not make much sense when coding in

Build failed in Jenkins: build-4.5 #545

2015-12-02 Thread jenkins
See Changes: [Rohit Yadav] debian: Partially pick change from #626 for getting version -- [...truncated 2951 lines...] [INFO] Using 'UTF-8' encoding to copy filtered resources. [INFO] Copying 3

[4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Mike Tutkowski
Hi, Did I miss an e-mail about manually acquiring new VMware dependencies for 4.7 or something? [INFO] [INFO] Building Apache CloudStack VMware Base 4.7.0-SNAPSHOT [INFO]

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161430026 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161431929 @pdube will run some integration tests against this branch. --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: [DISCUSS] ACS 4.5.3 release

2015-12-02 Thread Rohit Yadav
On 03-Dec-2015, at 12:19 AM, Rohit Yadav > wrote: So we got some things done on 4.5 lately; Just to be clear - I meant cherry-picking, backporting commits, or reviewing/merging 4.5 PRs lately :) Rohit Yadav Software Architect

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161428530 @wilderrodrigues Will start some 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

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1029#issuecomment-161431453 @ustcweizhou PR #1145 is merged to 4.6, please close this PR. Thanks! --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161429236 Makes sense, LGTM also based on the successful job above. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161429404 @ustcweizhou Took me a while, but the issue is not in this PR. Will post test results soon. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1145 --- 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: [DISCUSS] ACS 4.5.3 release

2015-12-02 Thread Rohit Yadav
Hi all, So we got some things done on 4.5 lately; - L10N updates - Fix missing DHCP entries - Allow users to create multiple Guest Shared Networks in same Vlan ID - Ensure power state is up to date when handling missing VMs in powerReport - We got UI enhancements - Security enhancements -

[GitHub] cloudstack pull request: debian: allow rules to pick ACS_BUILD_OPT...

2015-12-02 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1149 debian: allow rules to pick ACS_BUILD_OPTS from env Only now debian builds can be noredist etc. cc @remibergsma @wido - I think the apt-get.eu 4.6.0 deb repo was oss and not

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161223389 @ustcweizhou @bhaisaab @wido Did you actually test this PR? Cheers, Wilder --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1122#issuecomment-161225032 Thanks for writing this test, @sanju1010. It's much appreciated! @remibergsma and you executed the test and I went through the code. So, it LGTM

[GitHub] cloudstack pull request: CLOUDSTACK-9051: update nic IP address of...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1086#issuecomment-161226000 @ustcweizhou @DaanHoogland I was on holidays last week... just got back to the PR-Pool. I saw that @DaanHoogland already tested, so I will do a

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161216838 Test jenkins build job: http://sb.bhaisaab.org:/job/deb-shapeblue-build-patch-noredist/103/console --- If your project is set up for it, you can reply to

Re: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Rohit Yadav
+1 (binding) Tested with debian build against Ubuntu based KVM host, and fresh installation: - Basic and Adv zone deployment (without SG) - SystemVMs came up, with 4.6.0 systemvm template (used) - Normal VM life cycle in both isolated and VPC networks - PF, Firewall rules tested - Console proxy,

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-161222526 @remibergsma no promises but I'll try to meet that made up deadline :) --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161224861 @wilderrodrigues no, just 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

[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1122#discussion_r46391214 --- Diff: test/integration/component/test_tags.py --- @@ -2401,3 +2403,143 @@ def test_23_list_untagged_host_for_vm_migration(self):

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46391389 --- Diff: plugins/network-elements/nicira-nvp/src/main/java/com/cloud/network/element/NiciraNvpElement.java --- @@ -239,8 +244,51 @@ public

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46392407 --- Diff:

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161434770 Ping @DaanHoogland @remibergsma @borisroman @bhaisaab More tests! * Tests executed ``` nosetests --with-marvin

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161449429 Had a first look at the feature and have some questions / remarks: The quota icon is the only one in color and doesn't really fit the rest. Is that done

[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/942#issuecomment-161433986 @DaanHoogland That expunge issue is now resolved in the integration tests, my bad :-s --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161451821 @bhaisaab The SQL changes required are implemented in `schema-461to470.sql`. When I build from scratch, I hit `schema-460to470.sql` and end up with a db without

[GitHub] cloudstack pull request: Removed unused code from the EngineHostDa...

2015-12-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/942 --- 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 pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161436020 Sounds great guys! --- 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 pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1029#issuecomment-161439739 @remibergsma #1145 has conflict with master branch. you can merge this PR instead. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161437830 Ping @borisroman to have a look --- 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 pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161484253 given the number of tests done I am fine with this. I also reviewed the code, lgtm, I am not confident to put it in capitals but am floating on my experience

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1029#issuecomment-161484620 @ustcweizhou this PR is not needed as the code from 4.6 will be merged into master. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1013#issuecomment-161480457 sure, that is the goal. @borisroman @remibergsma @wilderrodrigues @wido @ustcweizhou Can you review? --- If your project is set up for it, you can reply

CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

2015-12-02 Thread Kenny Ho
Hi, I am trying out CloudStack and trying to do some BareMetal deployment. >From Jira, looks like the web UI is not ready for that so I turn to command line option. I found cloudmonkey, but on fresh install on first run, I got this error: $ sudo apt-get install python-pip $ sudo pip install

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou closed the pull request at: https://github.com/apache/cloudstack/pull/1029 --- 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: [4.7] CLOUDSTACK-9022: keep Destroyed vol...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1029#issuecomment-161535590 @DaanHoogland @remibergsma OK, I close this PR, You need to fix the conflict manually when you merge 4.6 to master. --- If your project is set up for it, you

Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Mike Tutkowski
Got it. It worked great - thanks! On Wed, Dec 2, 2015 at 10:36 PM, Rohit Yadav wrote: > > On 03-Dec-2015, at 10:23 AM, Mike Tutkowski > wrote: > > > Do you think I just need to copy vim25_55.jar >

RE: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Paul Angus
@Daan, the zone dropdown is populated for me. @Glenn, can you confirm whether is ok for you pls. Regards, Paul Angus VP Technology/Cloud Architect ShapeBlue Ltd S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue ShapeBlue

Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Rohit Yadav
You may use this — https://github.com/bhaisaab/cloudstack-nonoss git clone <> ./install-non-oss.sh Regards. On 03-Dec-2015, at 12:43 AM, Mike Tutkowski > wrote: Hi, Did I miss an e-mail about manually acquiring new VMware

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161530260 Let's ask @bhaisaab, we probably need to have the sql in both files to support both upgrade paths. Is there a better way? --- If your project is set up for it,

Re: CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

2015-12-02 Thread Rohit Yadav
Hi Kenny, On 03-Dec-2015, at 5:25 AM, Kenny Ho > wrote: I am trying out CloudStack and trying to do some BareMetal deployment. >From Jira, looks like the web UI is not ready for that so I turn to command line option. I found cloudmonkey, but on fresh

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161539545 @remibergsma I'll test the upgrade/sql issue today to check for any issues and fix if needed. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161502434 @remibergsma since the feature was going in 4.7 the schema changes were put in 4.6.1. Moving them to 460-470 path will cater to any upgrade from 4.6.* to

[GitHub] cloudstack pull request: CLOUDSTACK-9087:adding projectid paramete...

2015-12-02 Thread nitt10prashant
GitHub user nitt10prashant opened a pull request: https://github.com/apache/cloudstack/pull/1153 CLOUDSTACK-9087:adding projectid parameter to create method of class VmSnapshot Adding projectid parameter to class VmSnapshot of base.py . You can merge this pull request into a Git

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread abhinandanprateek
Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161534340 If the upgrade to 4.7, includes an upgrade to 4.6.1 then it can only be at one place and that seems to be a better option as it seems there are schema

[GitHub] cloudstack pull request: Quota

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161539427 @remibergsma we should not have the alter statement both in sql file and the Java upgrade path. Since the putting it in SQL might throw exceptions causing the

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161528070 Ping @remibergsma @DaanHoogland @borisroman Before testing, please note that the smoke/test_privategw_acl.py now requires hardware! *

Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Mike Tutkowski
Thanks, Rohit Do you think I just need to copy vim25_55.jar to deps and then run ./install-non-oss.sh? On Wed, Dec 2, 2015 at 9:49 PM, Rohit Yadav wrote: > You may use this —

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1134#discussion_r46519549 --- Diff: server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java --- @@ -229,59 +205,99 @@ public AffinityGroupVO

Re: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Daan Hoogland
Ok, thanks guys, I'll keep looking at my installation. It is a 4.6.0 upgraded to 4.6.1 for those who asked. On Thu, Dec 3, 2015 at 8:23 AM, Paul Angus wrote: > @Daan, the zone dropdown is populated for me. > @Glenn, can you confirm whether is ok for you pls. > >

Re: [4.7] Compiling with -Dnoredist fails for me

2015-12-02 Thread Rohit Yadav
On 03-Dec-2015, at 10:23 AM, Mike Tutkowski > wrote: Do you think I just need to copy vim25_55.jar to deps and then run ./install-non-oss.sh? Whatever

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161537378 LGTM. tested (1) create affinity group in project (2) deploy vm with affinity group (3) update vm affinity group (4) delete affinity group.

CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

2015-12-02 Thread Kenny Ho
Hi, I am trying out CloudStack and trying to do some BareMetal deployment. >From Jira, looks like the web UI is not ready for that so I turn to command line option. I found cloudmonkey, but on fresh install on first run, I got this error: $ sudo apt-get install python-pip $ sudo pip install

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-12-02 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-161235643 @nvazquez I have one more request for you, and (if you agree, of course) that should be the end of it on a code level. With the marvin test then this would

[GitHub] cloudstack pull request: CLOUDSTACK-9055: fix NPE in updating Redu...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1073#issuecomment-161242475 today one of my nodes was rebooted unexpectedly, I got the same error 2015-12-02 09:59:53,224 ERROR [c.c.n.r.VirtualNetworkApplianceManagerImpl]

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161243997 @wilderrodrigues I just tested. I set the storage.cleanup.interval to 60s, the storage.cleanup.delay to 120s. I restored a VM, then the old

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161249455 @remibergsma @wilderrodrigues checked the code (java and test_vm_life_cycle.py), I think this PR is not related to the failure in integration test. Can you

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161250671 test/integration/component/test_browse_volumes.py this test might be impacted. --- If your project is set up for it, you can reply to this email and have

Re: [GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread Sanjeev N
@daanhoogland I have updated the validation steps. On Wed, Dec 2, 2015 at 2:46 PM, DaanHoogland wrote: > Github user DaanHoogland commented on a diff in the pull request: > > https://github.com/apache/cloudstack/pull/1122#discussion_r46391214 > > --- Diff:

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-02 Thread nitin-maharana
GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1150 CLOUDSTACK-8968: UI icon over VM snapshot to deploy user instance Added a new Icon in Instance page to launch the VM from the snapshot. A new icon over VM snapshot object, which

[GitHub] cloudstack pull request: CLOUDSTACK-9069: Newly added project is n...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1082#issuecomment-161256224 I checked in Internet Explorer and Chromium browser also. It is working fine. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-12-02 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1122#issuecomment-161260733 thanks, i'll retest after lunch and merge --- 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: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/953#issuecomment-161262674 Made a pull request to merge in 4.6. So closing this PR. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/953 --- 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: CLOUDSTACK-9069: Newly added project is n...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/1077 --- 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

Build failed in Jenkins: build-master-slowbuild #2770

2015-12-02 Thread jenkins
See -- [...truncated 797 lines...] 2015-12-02 08:12:07,164 DEBUG [utils.script.Script] (main:) System resource: null 2015-12-02 08:12:07,165 DEBUG [utils.script.Script] (main:) Classpath

[GitHub] cloudstack pull request: CLOUDSTACK-9069: Newly added project is n...

2015-12-02 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1077#issuecomment-161263377 Closing this PR. As #1082 is there to merge in 4.6. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161270573 @wilderrodrigues I'm approaching EOD today, but can help test master tomorrow. I tested vm lifecycle manually with 4.6.1 RC1 and KVM, so I suppose 4.6 should

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/1151 [4.6] CLOUDSTACK-9075 - As a Developer I want the Private GW feature fixed on single VPCs This PR fixes the issue we faced with Private Gateways on single VPC when using ACS 4.6.0 and

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161266056 @remibergsma @DaanHoogland @wido @bhaisaab @ustcweizhou @karuturi @runseb @borisroman @jburwell If test_vm_life_cycle.py is failing because the

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1145#issuecomment-161272059 Thanks, @bhaisaab! I will test it and and also see if I have time tomorrow to run the same tests agains this PR. We need to be 100% sure that it

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161274671 Ping @remibergsma @DaanHoogland @borisroman You can start testing now... just a matter of running the following: ``` cd

[GitHub] cloudstack pull request: CLOUDSTACK-9099: SecretKey is returned fr...

2015-12-02 Thread kansal
GitHub user kansal opened a pull request: https://github.com/apache/cloudstack/pull/1152 CLOUDSTACK-9099: SecretKey is returned from the APIs - Fixed The current implementation of User and account management API (in general) return the secret key as a user or account response.

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161290470 Ping @remibergsma @DaanHoogland @borisroman Since I touched the router code, I also executed the rVPC tests to make sure those are working fine! I

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

2015-12-02 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161291156 Jenkins error is related to this: ![image](https://cloud.githubusercontent.com/assets/5129209/11532121/fa1f7274-9901-11e5-8e80-23c199c96a71.png)

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-12-02 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161286193 Code LGTM, not yet tested though. --- 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

Jenkins build is back to normal : build-master-jdk18 #419

2015-12-02 Thread jenkins
See

Re: [VOTE] Apache CloudStack 4.6.1

2015-12-02 Thread Pierre-Luc Dion
@daan, I don't see the problem on 2 env, could it be possible the zone is disabled or in basic zone ? on my side I'm having a glitch where creating a VM that support password in a VPC does not get is password updated. but the same template in a Isolated network does have is password set. not sure