Re: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount FileSR. Errno is 32]

2015-11-21 Thread B Prakash
Thanks Devdeep, I had missed seeding secondary storage, it is working fine after doing the same. regards, Prakash From: Devdeep Sent: Saturday, November 21, 2015 3:22 AM To: dev@cloudstack.apache.org Subject: Re: SR_BACKEND_FAILURE_61File SR creation error

[GitHub] cloudstack pull request: Removal of DefaultUserAuthenticator empty...

2015-11-21 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1100#issuecomment-158688747 Sure ;) He is one of my team mates of that project we talked about. Today was one of the days that we meet and try to clean some code. We normally

[GitHub] cloudstack pull request: Removal of DefaultUserAuthenticator empty...

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1100#issuecomment-158688687 ah @rafaelweingartner you did the check for @alexandrelimassantana , ok ;) --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack pull request: Removal of DefaultUserAuthenticator empty...

2015-11-21 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1100#issuecomment-158688620 Hi @DaanHoogland, That class was created 5 years ago by “Manuel Amador”, at that time the class had some logic. Then, someone called “Darren She

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

2015-11-21 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1098#issuecomment-158688518 @DaanHoogland I've just update this wiki page: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Update+L10N+files+from+Transifex+to+git+repo

Re: [DISCUSS] ACS 4.5.3 release

2015-11-21 Thread Milamber
Hello Rohit, I think that the PR 626 [1] on the debian/rules must be apply on the 4.5 branch to release 4.5.3 version to allow the good building of CS 4.53 on Ubuntu. (snapshot suffix issue on cloud-agent package) $ git diff diff --git a/debian/rules b/debian/rules index 45461bb..78dd487 100

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1098#issuecomment-158686667 Tnx Do we have a page on cwiki on this already? --- 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: Add47xmessagesproperties transifex

2015-11-21 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1098#issuecomment-158686367 @DaanHoogland I don' known the best way to explain the transifex process. For all versions of CloudStack, we have a version of resources files on Tran

[GitHub] cloudstack pull request: CLOUDSTACK-9068: Listing Port Forwarding ...

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1081#issuecomment-158685974 less is more, if it works it's fine. How does it behave on resizing? --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack pull request: CLOUDSTACK-8751 minimise downtime of netw...

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/866#issuecomment-158685221 good that you found the time to implement this feature @bvbharatk , thanks. an integrations test for this is needed as we will judge future PRs against it. As i

[GitHub] cloudstack pull request: CLOUDSTACK-8751 minimise downtime of netw...

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/866#discussion_r45551366 --- Diff: server/test/com/cloud/network/element/VirtualRouterElementTest.java --- @@ -225,6 +235,62 @@ public void testPrepare() { }

[GitHub] cloudstack pull request: [FUTURE]CLOUDSTACK-8907: POD Gateway opti...

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/886#issuecomment-158684576 code looks good but I don't see the use case; in an isolated network without internet access there would still be a gateway, would there? --- If your project i

[GitHub] cloudstack pull request: Removal of DefaultUserAuthenticator empty...

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1100#issuecomment-158684056 @alexandrelimassantana Did you check who created the DefaultUserAuthenticator? Your change makes sense to me but I would like to know why it was created before

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

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/944#issuecomment-158683799 @pedro-martins like the code one question about CitrixResourceBaseTest. And also: can you rebase on a commit beyond 4.6 and not merge, please? --- If your proj

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

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/944#discussion_r45550989 --- Diff: plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/CitrixResourceBaseTest.java --- @@ -0,0 +1,37 @@ +package c

[GitHub] cloudstack pull request: Removed unnecessary @Local annotations an...

2015-11-21 Thread cristofolini
GitHub user cristofolini opened a pull request: https://github.com/apache/cloudstack/pull/1102 Removed unnecessary @Local annotations and their respective imports Following @rafaelweingartner 's findings in PR #714 that many spring beans contained an @Local annotation, we've decided

[GitHub] cloudstack pull request: Add select template dropdown when reinsta...

2015-11-21 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1101 Add select template dropdown when reinstall VM You can merge this pull request into a Git repository by running: $ git pull https://github.com/ustcweizhou/cloudstack add-select-templ

[GitHub] cloudstack pull request: CLOUDSTACK-9068: Listing Port Forwarding ...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1081#issuecomment-158678341 Thanks for the fix @nitin-maharana. Tried it and it works great! LGTM https://cloud.githubusercontent.com/assets/1630096/11320287/64aad05a-9094-11e5-90b8

[GitHub] cloudstack pull request: CLOUDSTACK-8751 minimise downtime of netw...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/866#issuecomment-158676663 Pinging @wilderrodrigues to have a look. @bvbharatk Is there any (integration) test to test this feature? --- If your project is set up for it, you can

[GitHub] cloudstack pull request: [FUTURE]CLOUDSTACK-8907: POD Gateway opti...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/886#issuecomment-158676481 @borisroman Can you have a look and resolve the conflict? When we find another reviewer we can merge this in 4.7. Pinging @DaanHoogland @bhaisaab --- If your p

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-7984: Collect network st...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/926#issuecomment-158676063 @wido I see some comments above. Not sure if they are all addressed. Can you please review this and let us know if you are OK with merging? I'll run soms tests a

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

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-158675017 Hi @rodrigo93 in the comments above it was suggested to change the commit title, can you do that please? The commit title is the first line of the commit message

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

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/944#issuecomment-158673838 Hi @pedro-martins I can run some tests but please remove the merge commit. Thanks! --- If your project is set up for it, you can reply to this email and have yo

[GitHub] cloudstack pull request: CLOUDSTACK-8969 VPN customer gateway can'...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/955#issuecomment-158673310 @SudharmaJain Any update on this? Please respond to the comments, thanks! --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-158673227 Hi @rodrigo93 Thanks for rebasing, but please make sure you have one commit (and get rid of the merge commit). Thanks! --- If your project is set up for it, you

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

2015-11-21 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-158671944 PR merged with master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not hav

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2015-11-21 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-158671871 Just to update, I merged the commit with master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: Removal of DefaultUserAuthenticator empty...

2015-11-21 Thread alexandrelimassantana
GitHub user alexandrelimassantana opened a pull request: https://github.com/apache/cloudstack/pull/1100 Removal of DefaultUserAuthenticator empty class. The DefaultUserAuthenticator is an empty class, extending from the AdapterBase and implementing the UserAuthenticator interface

[GitHub] cloudstack pull request: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1075 --- 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: CLOUDSTACK-9065: Packaging RPM, add optio...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1075#issuecomment-158668313 LGTM, based on a set of tests that I run on this branch. Screenshot is from an experimental Jenkins job (that runs the same tests I did manually until now). The

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1098#issuecomment-158667887 @milamberspace looks ok, but ii know not enough of the transifex mechs to be sure. Can you add some explanation? --- If your project is set up for it, you can

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

2015-11-21 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/944#issuecomment-158667466 merged with master top version, someone can check this PR? =) thx guys. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Cwe 190

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158664606 @remibergsma for me test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL fails. That is not the one you meant is it? I also saw the failure you had. --- If y

Re: Getting re-started

2015-11-21 Thread Daan Hoogland
Wolf, I am not sure if access to analysis is default for apache account holders. Ask infra ;) On Sat, Nov 21, 2015 at 3:55 PM, Wolf Halton wrote: > I am a little embarrassed, but I cannot log in to the analysis site with my > apache username. > 1. I could have just forgotten the password, so I r

Getting re-started

2015-11-21 Thread Wolf Halton
I am a little embarrassed, but I cannot log in to the analysis site with my apache username. 1. I could have just forgotten the password, so I reset it. 2. My account may have been approved to get access to a different apache project, and I missed a step to get set up here. 3. Could be something to

[GitHub] cloudstack pull request: Update 4.5 branch with L10N resources fil...

2015-11-21 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1099#issuecomment-158642997 LGTM, thanks @milamberspace --- 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 n

[GitHub] cloudstack pull request: Update 4.5 branch with L10N resources fil...

2015-11-21 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1099 Update 4.5 branch with L10N resources files from Transifex 20151120 Open PR before the 4.5.3 release to update the L10N resources files from Transifex to the 4.5 branch. Mer

[GitHub] cloudstack pull request: L10N : Updated locale files with latest t...

2015-11-21 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1097#issuecomment-158636704 @barunhalder1234 I thinks you don't use the Transifex website to update the translations? For example on Transifex (for 4.6/4.7 resources) the Japanes

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

2015-11-21 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1098 Add47xmessagesproperties transifex Now master branch is the next 4.7 release. Open the 4.7 resources files on Transifex. Update the transifex client config file. Small P

[GitHub] cloudstack pull request: Cwe 190

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158625241 :-) --- 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 e

[GitHub] cloudstack pull request: Cwe 190

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158624430 I was busy starting ;) --- 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: Cwe 190

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158624231 @DaanHoogland It could be a result of another failure. On master this test passes, so maybe you can run just this test yourself and look at the details? --- If

[GitHub] cloudstack pull request: Cwe 190

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158620306 thanks @remibergsma ```cidr is not formatted correctly: cloud``` sound like a valid error message. not sure if it needs fixing in the test or in the code. ---

Re: Read/Write splitting on database

2015-11-21 Thread Remi Bergsma
No, I don't think that can be done or works well. The best approach is to use MariaDB with Galera. There was a talk in Dublin about this, you might want to check the video. Regards, Remi Sent from my iPhone > On 18 Nov 2015, at 10:01, Alireza Eskandari > wrote: > > Hi all, > According to

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

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158617888 @bhaisaab @jlk want to review give a second lgtm? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

Re: 4.6 release

2015-11-21 Thread Sebastien Goasguen
Talking about 4.6 There seems to be an issue when people upgrade from 4.5.2 and the awsapi package is missing. Can someone (ping @pdion) upgrade the release notes to mention the fact that awsapi is removed and add a step in the upgrade process ? > On Nov 20, 2015, at 2:39 PM, Wido den Hollande

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

2015-11-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1021#issuecomment-158617403 [1021.network.results.txt](https://github.com/apache/cloudstack/files/40781/1021.network.results.txt) [1021.vpc.results.txt](https://github.com/apache/clou

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-8746: VM Snapshotting im...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/977#issuecomment-158615300 Hi @wido can you review this again please? Thanks! --- 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-9076: Changed ownership of dir...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1095#issuecomment-158613584 @borisroman Did you build the packages from this branch? Can you share some details to show it works properly now? --- If your project is set up for it, you ca

[GitHub] cloudstack pull request: L10N : Updated locale files with latest t...

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1097#issuecomment-158613518 Pinging @milamberspace 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 p

[GitHub] cloudstack pull request: Cwe 190

2015-11-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1057#issuecomment-158613419 @DaanHoogland @rafaelweingartner Much better now, just one failed test to resolve: Failed test: `TestVPCRouterOneNetwork` ``` Execute cmd:

Re: SR_BACKEND_FAILURE_61File SR creation error [opterr=fail to mount FileSR. Errno is 32]

2015-11-21 Thread Devdeep
Have you seeded the secondary storage with the default systemvm template? Regards, Devdeep On Sat, Nov 21, 2015 at 8:33 AM, B Prakash wrote: > Thanks Jayapal, Abhinandan. > > log SMlog doesnt giveout much, I am able to mount and edit file on mount > from both server and agent. I get the same er