[GitHub] cloudstack pull request: Sorting of security groups

2015-10-16 Thread atrbgithub
GitHub user atrbgithub opened a pull request: https://github.com/apache/cloudstack/pull/937 Sorting of security groups Simple change to sort the security groups in alphabetical order within the instance creation wizard. This makes it much easier to find a security group when th

[GitHub] cloudstack pull request: [4.6][BLOCKER] CLOUDSTACK-8923: Create st...

2015-10-16 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/934#issuecomment-148724604 So, 1. we did not test enough 2. we have an issue that sneaked into master 3. you have a test env issue I don't like --- If your project is set

gson upgrade test required

2015-10-16 Thread Daan Hoogland
H, PR932 contains a 'working' patch for gson. this touches every command and answer serialization however, so I would ask everybody to include it in their next test run. Both CIs report success in github. please, please, very please, -- Daan

[GitHub] cloudstack pull request: CLOUDSTACK-8708: gson update and ArrayTyp...

2015-10-16 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/932#issuecomment-148722212 So far so good, all test passed in CI. This touches every command and answer that is serialized, however. please all test --- If your project is set up for it,

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-148715124 Hi @miguelaferreira @remibergsma Thanks a lot for your help. I will try to make a better description of the problem and work on commits and code as you sug

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

2015-10-16 Thread jenkins
See

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-148704543 Hi @nvazquez Please reorganise your commits, as we need them to be atomic. It cannot go in like this. I'm also with @miguelaferreira, please let us know

[GitHub] cloudstack pull request: [4.6][BLOCKER] CLOUDSTACK-8923: Create st...

2015-10-16 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/934#issuecomment-148703718 Run the following tests: ``` Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS === ok Test

Re: [4.6] Instance passwords broken? "serve_password: requested password not found"

2015-10-16 Thread Rohit Yadav
Hi guys, I made the change to get the password saved in specific interface files (password- file) as earlier there was an bug/security issue with using simply the password file. I’ll look into it and get back on this. Thanks for reporting. On 16-Oct-2015, at 2:36 pm, Nux! mailto:n...@li.nux.ro

[GitHub] cloudstack pull request: [4.6][BLOCKER] CLOUDSTACK-8923: Create st...

2015-10-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/934 --- 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: [4.6][BLOCKER] CLOUDSTACK-8923: Create st...

2015-10-16 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/934#issuecomment-148696476 Thanks @DaanHoogland and @NuxRo . I will merge this in sometime. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cloudstack pull request: [4.6][BLOCKER] CLOUDSTACK-8923: Create st...

2015-10-16 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/934#issuecomment-148689735 Code LGTM. I couldn't test but did a check in the source and found the recurring pattern for this was applied in several cases. It should not have been applied

[GitHub] cloudstack pull request: [4.6][BLOCKER] CLOUDSTACK-8923: Create st...

2015-10-16 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/934#issuecomment-148684942 I ran BVT tests on a setup with two xenserver 6.5. I didnt see any new failures. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request: Add agent name in received response

2015-10-16 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/936#issuecomment-148669695 Code LGTM, as testing is a drag at this moment I think we should put this in a batch and test a combination of small PRs to report on. This one is candidate!

[GitHub] cloudstack pull request: Add agent name in received response

2015-10-16 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/936 Add agent name in received response Before change: 2015-10-15 12:55:34,268 DEBUG [c.c.a.t.Request] (RouterStatusMonitor-1:ctx-d2b917a4) Seq 20-2693152577167557361: Sending { Cmd ,

RE: Problems with first steps of CS plugin development

2015-10-16 Thread Schubert, Sven
Hi, I have been trying to do the example as well. When trying to build I get the following error: [ERROR] Failed to execute goal on project cloud-server: Could not resolve dependencies for project org.apache.cloudstack:cloud-server:jar:4.5.2: Failure to find com.cloud.test:cloud-plugin-api-tim

[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-16 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/884#issuecomment-148661785 @remibergsma This PR is marked as closed, but it was actually reverted. Although it's not a bugfix, it is a fix for the Unit Tests on Windows. Do we want to mer

Re: [4.6] Instance passwords broken? "serve_password: requested password not found"

2015-10-16 Thread Nux!
Ok, opened the following with some more info as well. https://issues.apache.org/jira/browse/CLOUDSTACK-8957 -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Nux!" > To: dev@cloudstack.apache.org > Sent: Friday, 16 October, 2015 09:53:

Re: [4.6] Instance passwords broken? "serve_password: requested password not found"

2015-10-16 Thread Nux!
Will do shortly -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Remi Bergsma" > To: dev@cloudstack.apache.org > Sent: Friday, 16 October, 2015 09:52:24 > Subject: Re: [4.6] Instance passwords broken? "serve_password: requested > pass

Re: [4.6] Instance passwords broken? "serve_password: requested password not found"

2015-10-16 Thread Remi Bergsma
Do we already have an Jira issue for this? If not, please create one so we can look into it. Thanks, Remi On 16/10/15 06:13, "Suresh Sadhu" wrote: >Password saved in the right file but it's not applying on vm. its bug. > >You can check the generated passwords in below file in the router

Re: [DISCUSS] Log shipping from system vms

2015-10-16 Thread Wido den Hollander
On 16-10-15 09:17, Erik Weber wrote: > I'm working a case where it would be required to ship logs from system vms > (in particular VRs). > > I can fix that by modifying rsyslog.conf inside systemvm.iso, but that > means I have to remember it every time we update, which is most likely to > be for

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-148639781 Hi @nvazquez Thanks for your contribution. I'm not quite sure I understand the problem you are trying to solve, but that likely my own lack of NXS k

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42216002 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java --- @@ -1079,8 +1088,10 @@ private static void createNvpPortGro

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42216007 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java --- @@ -1101,6 +1112,12 @@ private static void createNvpPortGro

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42215862 --- Diff: services/console-proxy-rdp/rdpconsole/src/test/java/rdpclient/MockServerTest.java --- @@ -161,7 +161,8 @@ public void testIsMockServerC

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42215794 --- Diff: plugins/network-elements/nicira-nvp/src/com/cloud/network/resource/NiciraNvpResource.java --- @@ -191,6 +204,19 @@ public PingCommand g

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42215621 --- Diff: plugins/network-elements/nicira-nvp/src/com/cloud/network/resource/NiciraNvpResource.java --- @@ -191,6 +204,19 @@ public PingCommand g

[GitHub] cloudstack pull request: From4.5.1: NSX/Nicira Plugin does not sup...

2015-10-16 Thread miguelaferreira
Github user miguelaferreira commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42215505 --- Diff: plugins/network-elements/nicira-nvp/src/com/cloud/network/resource/NiciraNvpResource.java --- @@ -180,6 +182,17 @@ public Type getType(

[DISCUSS] Log shipping from system vms

2015-10-16 Thread Erik Weber
I'm working a case where it would be required to ship logs from system vms (in particular VRs). I can fix that by modifying rsyslog.conf inside systemvm.iso, but that means I have to remember it every time we update, which is most likely to be forgotten at some point in time. This it to discuss i