Re: git tags on cloudstack repo

2015-11-05 Thread Erik Weber
On Thu, Nov 5, 2015 at 9:34 AM, Rene Moser wrote: > Hi > > I have kind of a question about the tagging policy in cloudstack git repo: > > We now have tag 4.6.0 but we call it an RC. This raises 2 problems: > > Unless I'm seeing wrong, we have a branch, and it's called

[GitHub] cloudstack pull request: Fix some small UI bugs

2015-11-05 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1034 Fix some small UI bugs You can merge this pull request into a Git repository by running: $ git pull https://github.com/ustcweizhou/cloudstack ui-changes Alternatively you can review

Re: User vm password gets update only the master VR.

2015-11-05 Thread Bharat Kumar
Hi Wilder, If we think that it is ok to have the work around for now fix this later, we can bring the priority down to critical. Thanks, Bharat. On 05-Nov-2015, at 2:49 pm, Wilder Rodrigues wrote: > Hi Bharat, > > Please check if what you just mention will

Re: git tags on cloudstack repo

2015-11-05 Thread Rene Moser
On 11/05/2015 09:45 AM, Erik Weber wrote: > Unless I'm seeing wrong, we have a branch, and it's called I am not seeing it on github yet, but on here https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=tags

Re: git tags on cloudstack repo

2015-11-05 Thread Daan Hoogland
I don't like the rc1 tag. The date tag is more descriptive. and easier to automate. As for the tag being or not in git doesn't really matter. I used to rename the released branch to GA-4.4.x after the vote and leave the tag on there as well. In the end it is the commit-id that is significnant. On

Re: User vm password gets update only the master VR.

2015-11-05 Thread Bharat Kumar
I have’t checked, but form the code it looks like it will work if we reissue the reset command after the slave becomes master. —Bharat. On 05-Nov-2015, at 12:59 pm, Erik Weber > wrote: On Thu, Nov 5, 2015 at 7:15 AM, Bharat Kumar

[GitHub] cloudstack pull request: CLOUDSTACK-9037 patterns can be more elab...

2015-11-05 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1035 CLOUDSTACK-9037 patterns can be more elaborate then prefixes. little fix to make sure for instance "eth" is not regarded as interface when it is part of "methamfetamine" You can merge

Re: User vm password gets update only the master VR.

2015-11-05 Thread Bharat Kumar
Hi, Wilder has already reduced the priority to critical, I found it while manually testing some of my changes. I am not aware of a test which checks this. —Bharat. On 05-Nov-2015, at 3:03 pm, Remi Bergsma wrote: > Hi Bharat, > > First of all: thanks for

Re: User vm password gets update only the master VR.

2015-11-05 Thread Remi Bergsma
Hi Bharat, First of all: thanks for reporting the issue! Would it be possible to write a Marvin test for it, so we can test this automatically and prevent regression? Or did you find this by an existing Marvin test that I missed? Please downgrade to critical. We should fix this, and we will,

git tags on cloudstack repo

2015-11-05 Thread Rene Moser
Hi I have kind of a question about the tagging policy in cloudstack git repo: We now have tag 4.6.0 but we call it an RC. This raises 2 problems: 1st: from the public perspective, this is a released version. Or how can anyone not related to the project see, if this is an RC or not? 2nd: tags

Re: git tags on cloudstack repo

2015-11-05 Thread Wei ZHOU
+1 with 4.6.0-rc1 2015-11-05 9:34 GMT+01:00 Rene Moser : > Hi > > I have kind of a question about the tagging policy in cloudstack git repo: > > We now have tag 4.6.0 but we call it an RC. This raises 2 problems: > > 1st: from the public perspective, this is a released

Re: [VOTE] Apache CloudStack 4.6.0

2015-11-05 Thread Nux!
Thanks Remi! -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Remi Bergsma" > To: dev@cloudstack.apache.org > Sent: Wednesday, 4 November, 2015 20:45:59 > Subject: Re: [VOTE] Apache CloudStack 4.6.0 >

Re: git tags on cloudstack repo

2015-11-05 Thread Boris Schrijver
Good point! Hadn't looked there. I don't think we should tag an RC as it's still under active development. --  Met vriendelijke groet / Kind regards, Boris Schrijver PCextreme B.V. http://www.pcextreme.nl/contact Tel direct: +31 6 33784542 > On November 5, 2015 at 9:47 AM Rene Moser

Re: git tags on cloudstack repo

2015-11-05 Thread Remi Bergsma
Guys, I accidentally pushed the 4.6.0 tag and I shouldn't have done that before it is final. Sorry! Let's see if we can get rid of it until the vote passes. Regards, Remi Sent from my iPhone > On 05 Nov 2015, at 09:35, Rene Moser wrote: > > Hi > > I have kind of a

Re: User vm password gets update only the master VR.

2015-11-05 Thread Wilder Rodrigues
Hi Bharat, Please check if what you just mention will work. If it does, then the issue is not a blocker. Cheers, Wilder > On 05 Nov 2015, at 10:09, Bharat Kumar wrote: > > I have’t checked, but form the code it looks like it will work if we reissue > the reset

Re: git tags on cloudstack repo

2015-11-05 Thread Remi Bergsma
The 4.6.0 tag is now removed. To https://git-wip-us.apache.org/repos/asf/cloudstack.git - [deleted] 4.6.0 Didn’t take it like that, René, you had a valid point. No problem. Regards, Remi On 05/11/15 09:55, "Rene Moser" wrote: >Hi > >On 11/05/2015 09:52 AM,

Re: User vm password gets update only the master VR.

2015-11-05 Thread Wilder Rodrigues
Cool that you went the extra mile with the manual test. :) Once fixed, I will also write a test to cover the scenario you described in the issue. It will make it future proof. Thanks again, Bharat! Cheers, Wilder > On 05 Nov 2015, at 10:54, Bharat Kumar wrote: > >

Re: git tags on cloudstack repo

2015-11-05 Thread Boris Schrijver
I don't see a tag either. When the RC passes its vote, then a tag and corresponding release will be created. Until then, we just have the rc branch. --  Met vriendelijke groet / Kind regards, Boris Schrijver PCextreme B.V. http://www.pcextreme.nl/contact Tel direct: +31 6 33784542 > On

Re: git tags on cloudstack repo

2015-11-05 Thread Daan Hoogland
On Thu, Nov 5, 2015 at 9:54 AM, Daan Hoogland wrote: > > On Thu, Nov 5, 2015 at 9:52 AM, Remi Bergsma > wrote: > >> I accidentally pushed the 4.6.0 tag and I shouldn't have done that before >> it is final. Sorry! >> >> Let's see if we can

[GitHub] cloudstack pull request: Fix some small UI bugs

2015-11-05 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1034#discussion_r44015311 --- Diff: ui/scripts/vpc.js --- @@ -4184,7 +4184,7 @@ // Get VPN gateways $.ajax({ -

[GitHub] cloudstack pull request: Fix some small UI bugs

2015-11-05 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/1034#issuecomment-154048544 +1 LGTM, just looked at the code, did not check the actual UI rendering. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CID 1338016: multi catch instead of pokem...

2015-11-05 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/1018#issuecomment-154050065 simple enough, but why is Jenkins failing, the link even goes 404. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: squashed commit for dockerfiles part#2 in...

2015-11-05 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/999#issuecomment-154050476 +1 LGTM , does not touch core, totally independent code, only affects the behavior of built docker images. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1021#discussion_r43996820 --- Diff: server/src/com/cloud/api/ApiResponseGsonHelper.java --- @@ -27,30 +29,40 @@ import com.google.gson.GsonBuilder; /** - *

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-154025365 @DaanHoogland About the issue you pointed out in com.cloud.vm.VmWorkStart serialization. Nice find but these are already existing issues. They surely needs

[GitHub] cloudstack pull request: CLOUDSTACK-9038 - Infrastructure tab is s...

2015-11-05 Thread rags22489664
GitHub user rags22489664 opened a pull request: https://github.com/apache/cloudstack/pull/1036 CLOUDSTACK-9038 - Infrastructure tab is slow because of synchronous API calls Making parallel asynchronous calls to speed up the infrastructure page. You can merge this pull request into

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-05 Thread atrbgithub
GitHub user atrbgithub opened a pull request: https://github.com/apache/cloudstack/pull/1037 Fix for CLOUDSTACK-8937 - XenServer migrations with storage failing i… …n clustered management server environment This pull request relates to the following Jira bug report:

Re: Compiling master - build error

2015-11-05 Thread cs user
Hi Daan, Thanks for getting back to me. I just had chance to try this again today and it appeared to go through fine. Very strange! Cheers On Mon, Nov 2, 2015 at 3:12 PM, Daan Hoogland wrote: > hey some guy, > > Our jenkins apidoc builders have been failing regularly

[GitHub] cloudstack pull request: CID 1338016: multi catch instead of pokem...

2015-11-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1018#issuecomment-154050616 @runseb because this is to old you get a 404. My guess about the failure is that it ran on any of the machines mentioned in INFRA-10703. these have regular

Debian packages 4.6 RC1

2015-11-05 Thread Boris Schrijver
Hi all! I've made the DEB packages of the 4.6.0-RC20151104T1522 publicly available here: https://cloudstack.o.auroraobjects.eu/precise/cloudstack-agent_4.6.0_all.deb https://cloudstack.o.auroraobjects.eu/precise/cloudstack-cli_4.6.0_all.deb

[GitHub] cloudstack pull request: CLOUDSTACK-9037 patterns can be more elab...

2015-11-05 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1035#issuecomment-154019836 tested. 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

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1021#discussion_r43996196 --- Diff: api/src/org/apache/cloudstack/api/response/SslCertResponse.java --- @@ -38,6 +40,7 @@

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1021#discussion_r43996309 --- Diff: api/src/org/apache/cloudstack/api/response/UserVmResponse.java --- @@ -256,6 +259,7 @@

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-154030678 @DaanHoogland That's right. I had already mentioned about the agent commands in one of my previous comments --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-154032129 @koushik-das you said 'LogLevel is used in the latter one only.'. That is not true, I think. see com.cloud.agent.transport.RequestTest --- If your project is

[GitHub] cloudstack pull request: Fix for CLOUDSTACK-8937 - XenServer migra...

2015-11-05 Thread atrbgithub
Github user atrbgithub commented on the pull request: https://github.com/apache/cloudstack/pull/925#issuecomment-154042688 Fix for master branch has been proposed here https://github.com/apache/cloudstack/pull/1037 --- If your project is set up for it, you can reply to this email

Re: [VOTE] Apache CloudStack 4.6.0

2015-11-05 Thread Wilder Rodrigues
Hi all, I have been busy testing the 4.6.0 RC since last night. The results below are still partial (I still have to test XenServer 6.5), but it looks very promising! My vote will be given once I’m done with Xen65 and upgrade tests. Cheers, Wilder :: Environment 1 :: * Hardware required:

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-154035894 @DaanHoogland If you look at ApiResponseGsonHelper.java in this PR there are GSon builders private static final GsonBuilder s_gBuilder; private

[GitHub] cloudstack pull request: CLOUDSTACK-9038 - Infrastructure tab is s...

2015-11-05 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1036#issuecomment-154035977 Thanks @rags22489664 sounds good! Will try to test it soon. --- 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-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1021#discussion_r43996031 --- Diff: api/src/org/apache/cloudstack/api/response/SSHKeyPairResponse.java --- @@ -40,6 +42,7 @@ @SerializedName("fingerprint")

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1021#discussion_r43997043 --- Diff: server/src/com/cloud/api/ApiResponseGsonHelper.java --- @@ -71,4 +83,19 @@ public boolean shouldSkipField(FieldAttributes f) {

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-154027199 @koushik-das I don't think you are right, command serialisation uses LogLevel as well. Look at com.cloud.agent.transport.RequestTest and tell me if you agree.

Re: [RFC] Metrics views for CloudStack UI

2015-11-05 Thread Nux!
Great work Rohit, What I'd like to see: - vCPU list/count for instance metrics (GHz is meaningless to me) - can we make the whole thing wider so we can fit more columns there without that ugly horizontal scroll bar? So much wasted screen space -- Sent from the Delta quadrant using Borg

Re: [VOTE] Apache CloudStack 4.6.0

2015-11-05 Thread Nux!
Installation on CentOS 6 mgmt and HVs worked great, added some templates, deployed some instances, no issues. I'll get back if I hit problems. Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Nux!" > To:

[GitHub] cloudstack pull request: CLOUDSTACK-9040: Use Tomcat6 for Debian p...

2015-11-05 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1040#issuecomment-154161203 LGTM We can deal with other Tomcat versions later. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: [4.6] VPN works?

2015-11-05 Thread Remi Bergsma
Hi Lucian, As far as I know it works. The Marvin tests also passes. When I test manual and connect two VPCs together, I can ping the hosts on each side. I can send you a small script that sets it up, if you want. Regards, Remi On 05/11/15 19:47, "Nux!" wrote: >Hi guys, >

[4.6] VPN works?

2015-11-05 Thread Nux!
Hi guys, I am testing 4.6 briefly and basic operations work. I am also trying now VPN functionality on a SNAT public IP, however I don't seem to get it working. Can someone confirm it works for them so I know it's me (as always)? I am not usually using this feature, but thought to try it

Re: [VOTE] Apache CloudStack 4.6.0

2015-11-05 Thread Mike Tutkowski
Hi, I don't know if it's something in my environment, but I am running a basic zone with one pod and one XenServer 6.5 cluster and my VMs don't seem to get IP addresses from the range I specified. The VM's hostname is not set properly nor does it produce the correct IP address when I run

[GitHub] cloudstack pull request: CLOUDSTACK-9040: Use Tomcat6 for Debian p...

2015-11-05 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1040 CLOUDSTACK-9040: Use Tomcat6 for Debian packages. Use Tomcat6 for Debian packages. How to test: Package debian packages and install them. Will depend specifically on tomcat6

Re: [4.6] VPN works?

2015-11-05 Thread Nux!
Remi, This was a normal server-client VPN, not site2site. The client is Win7, but the network was a bit dodgy there, hence my doubts the VPN was actually broken. :) Lucian PS: script welcome for possible future uses, thanks! -- Sent from the Delta quadrant using Borg technology! Nux!

[GitHub] cloudstack pull request: CLOUDSTACK-9040: Use Tomcat6 for Debian p...

2015-11-05 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1040#issuecomment-154239490 @ustcweizhou We can! Though in the current structure, if we change to support Tomcat7 we can't support Tomcat6... In my opinion, we leave it to Tomcat6 for the

[GitHub] cloudstack pull request: CLOUDSTACK-9040: Use Tomcat6 for Debian p...

2015-11-05 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1040#issuecomment-154233552 cannot we fix the issue in tomcat7 suppport ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-05 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-154200225 :+1: LGTM @nlivens excellent work -- much appreciate the hard work. --- If your project is set up for it, you can reply to this email and have your reply

Re: [4.6] VPN works?

2015-11-05 Thread Jayapal Reddy Uradi
When I test last time remote access vpn and s2s vpn are working. I think there are no changes done on vpn after that. @Nux If you still see problem observe /var/log/auth.log in the VR. Thanks, Jayapal > On 06-Nov-2015, at 5:22 am, Nux! wrote: > > Remi, > > This was a normal

[GitHub] cloudstack pull request: CLOUDSTACK-9039: Fix paths for logging Ub...

2015-11-05 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1039#discussion_r44112143 --- Diff: python/lib/cloudutils/serviceConfigServer.py --- @@ -107,7 +107,8 @@ def checkHostName(): bash("chown cloud.cloud

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-11-05 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-154335646 ok so bottom line, it seems reviews are +1 and we will merge this in 4.7. ASAP after 4.6.0 is out, taking. the only thing will be to keep an eye on it and

[GitHub] cloudstack pull request: CLOUDSTACK-9039: Fix paths for logging Ub...

2015-11-05 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/1039 CLOUDSTACK-9039: Fix paths for logging Ubuntu Management Fix paths for logging Ubuntu Management. How to test: Install via DEB packages... You can merge this pull request into

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

2015-11-05 Thread bhaisaab
GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1038 Metrics views for CloudStack UI FS: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Metrics+Views+for+CloudStack+UI JIRA: https://issues.apache.org/jira/browse/CLOUDSTACK-9020 You

[RFC] Metrics views for CloudStack UI

2015-11-05 Thread Rohit Yadav
Hi all, The present CloudStack UI hides most of the metrics data such as cpu, memory, disk, network usage in inner detail views. Such information is critical to find issues in one’s cloud, for example finding clusters where hosts are failing, or finding storage pools where disk space has

[GitHub] cloudstack pull request: Metrics views for CloudStack UI

2015-11-05 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-154067974 For screenshots, please see the FS: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Metrics+Views+for+CloudStack+UI --- If your project is set up for it,

[GitHub] cloudstack pull request: CLOUDSTACK-8485: listAPIs are taking too ...

2015-11-05 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-154062799 I am not fine with this change but must be honest and report regressions tests from the bubble: they all passed. ``` Test router internal advanced

Getting Full Volume Snapshots every day

2015-11-05 Thread mvs babu
Hi Team, We have scheduled daily volume snapshots in ACS 4.3.1 and XenServer 6.2 SP1 as Hypervisor. ​In secondary storage, we are getting full backups every day instead of incremental backups. After restarting management service and SQL service, we found that the next backup was incremental