[GitHub] cloudstack pull request: UI localization - hu

2015-05-02 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/224#issuecomment-98329583 Hello, Thanks for your PR. The file plugins/hypervisors/kvm/pom.xml don't need to be modify to add the Hungarian language (mistake

[GitHub] cloudstack pull request: CLOUDSTACK-6181 Specify GB for the value ...

2015-05-23 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/294 CLOUDSTACK-6181 Specify GB for the value of rootdisksize parameter. Add some Bytes/GB for log or exception messages. Fix Gb-GB. The new behavior (since 4.4) rootdisksize for KVM

[GitHub] cloudstack pull request: CLOUDSTACK-8744 Add missing localization ...

2015-08-18 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/712 CLOUDSTACK-8744 Add missing localization (l10n) for several parts in … …the UI - l10n for the SSH Key Pairs behavior - l10n for Autoscaling / LB sections - l10n

[GitHub] cloudstack pull request: CLOUDSTACK-8649: Fixed unnecessary double...

2015-08-18 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/615#issuecomment-132091017 The PR #709 don't fix the issue for me. I can't add a SSH key pair without revert the #615. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-8649: Fixed unnecessary double...

2015-08-18 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/615#issuecomment-132097141 Sorry, I clean up my browser cache, now works with the #709 fix. Thanks @borisroman --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Removed double encoding of Public Key fro...

2015-08-18 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/709#issuecomment-132097225 LGTM +1 --- 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: CLOUDSTACK-8649: Fixed unnecessary double...

2015-08-17 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/615#issuecomment-131899397 Seems this PR breaks the addition of a public key on the Web UI. The reponse is : Public key is invalid (works successfully with Cloudmonkey

[GitHub] cloudstack pull request: Fixed typo

2015-08-17 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/708#issuecomment-131907084 Good catch! 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

[GitHub] cloudstack pull request: CLOUDSTACK-8749: KVM - cleanup linklocal ...

2015-08-19 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/718#issuecomment-132642477 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

[GitHub] cloudstack pull request: Update L10N resource files on 4.5 branch ...

2015-08-19 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/697#issuecomment-132515960 Thanks for the merge. yes not necessary, the PR #699 make the works for master branch. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: Update L10N resource files on master bran...

2015-08-19 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/699#discussion_r37389129 --- Diff: client/WEB-INF/classes/resources/messages_es.properties --- @@ -808,7 +808,7 @@ label.skip.guide=He utilizado CloudStack anteriormente

[GitHub] cloudstack pull request: CLOUDSTACK-8740: add custom.css to make c...

2015-08-18 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/704#issuecomment-132252887 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

[GitHub] cloudstack pull request: Update L10N resource files on master bran...

2015-08-18 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/699#discussion_r37315501 --- Diff: client/WEB-INF/classes/resources/messages_es.properties --- @@ -808,7 +808,7 @@ label.skip.guide=He utilizado CloudStack anteriormente

[GitHub] cloudstack pull request: Update L10N resource files on master bran...

2015-07-30 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/640#issuecomment-126315563 Sorry, I already merged on apache repo. I will better the next time. Howto find the rules before merging ? (2 LGTM, How many asfbot/cloudstack-pull

[GitHub] cloudstack pull request: Update L10N resource files on master bran...

2015-07-30 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/640 Update L10N resource files on master branch with 4.6 translation strings from Transifex (20150730) You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: Update L10N resource files on 4.5 branch ...

2015-07-30 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/641 Update L10N resource files on 4.5 branch with 4.5 translation strings… … from Transifex (20150730) You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: Update L10N resource files on 4.4 branch ...

2015-07-30 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/642 Update L10N resource files on 4.4 branch with 4.4 translation strings… … from Transifex (20150730) You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: Improve debug message when the scheduler ...

2015-08-11 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/676 Improve debug message when the scheduler prepare to snapshot (volume … …'id' and add volume name) Signed-off-by: Milamber milam...@apache.org You can merge this pull request

[GitHub] cloudstack pull request: [ACS4.5] Improve debug message when the s...

2015-08-11 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/677 [ACS4.5] Improve debug message when the scheduler prepare to snapshot (volume … …'id' and add volume name) Signed-off-by: Milamber milam...@apache.org You can merge

[GitHub] cloudstack pull request: Update L10N resource files on 4.5 branch ...

2015-08-14 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/697 Update L10N resource files on 4.5 branch with 4.5 translation strings… … from Transifex (20150814) You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: [CS45] Add missing localization for some ...

2015-08-14 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/687#issuecomment-131043213 @bhaisaab Thanks for LGTM (Daan too) The PR for fix the master is here: (#686) https://github.com/apache/cloudstack/pull/686 Need a second

[GitHub] cloudstack pull request: Update L10N resource files on 4.5 branch ...

2015-08-14 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/697#issuecomment-131126574 Yes it's right, the original string has been changed, so the (old) translation has been removed on Transifer. Long version: * Cloudstack has now 2

[GitHub] cloudstack pull request: Update L10N resource files on master bran...

2015-08-14 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/699 Update L10N resource files on master branch with 4.6 translation stri… …ngs from Transifex (20150814) You can merge this pull request into a Git repository by running: $ git pull

[GitHub] cloudstack pull request: [CS45] Add missing localization for some ...

2015-08-14 Thread milamberspace
Github user milamberspace closed the pull request at: https://github.com/apache/cloudstack/pull/687 --- 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

[GitHub] cloudstack pull request: [ACS4.5] Improve debug message when the s...

2015-08-12 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/677#issuecomment-130187847 Ok thanks for your comment. I close this PR. I will cherry-pick the PR #676 on the 4.5 branch. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: [ACS4.5] Improve debug message when the s...

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

[GitHub] cloudstack pull request: [CLOUDSTACK-8427] Add missing localizatio...

2015-08-12 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/686 [CLOUDSTACK-8427] Add missing localization for some labels in Web UI - on new functionnality: upload volume/template from Local - on the zone configuration wizard for the Next button

[GitHub] cloudstack pull request: [CS45] Add missing localization for some ...

2015-08-12 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/687 [CS45] Add missing localization for some labels in Web UI - on the zone configuration wizard for the Next button - on templates, Shared, extratable and Dynamically Scalable

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-10-23 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/974 Update L10N resource files with 4.6 strings from Transifex (20151023) This is a "open" PR until the RC1 for Apache CloudStack. I will updated this PR next week

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-10-27 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/974#issuecomment-151403745 Mishandling git commands. I've recreated this open PR for the localization files, see #984 Closes this PR. --- If your project is set up

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-10-27 Thread milamberspace
Github user milamberspace closed the pull request at: https://github.com/apache/cloudstack/pull/974 --- 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

[GitHub] cloudstack pull request: Update L10N files for Next 4.6.

2015-10-27 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/984 Update L10N files for Next 4.6. Update the localization files for the next CloudStack version. This PR is open until the start of the release candidate 1 for CS 4.6. This PR

[GitHub] cloudstack pull request: CLOUDSTACK-8961: Changes related to the U...

2015-10-29 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/941#discussion_r43433493 --- Diff: ui/scripts/network.js --- @@ -6504,7 +6432,123

[GitHub] cloudstack pull request: CLOUDSTACK-8961: Changes related to the U...

2015-10-29 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/941#discussion_r43433516 --- Diff: ui/scripts/network.js --- @@ -6504,7 +6432,123

[GitHub] cloudstack pull request: CLOUDSTACK-8961: Changes related to the U...

2015-10-29 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/941#discussion_r43433480 --- Diff: ui/scripts/network.js --- @@ -4495,99 +4511,11 @@ $('').addClass('psk').html(_l

[GitHub] cloudstack pull request: Update L10N files for Next 4.6.

2015-10-29 Thread milamberspace
Github user milamberspace closed the pull request at: https://github.com/apache/cloudstack/pull/984 --- 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

[GitHub] cloudstack pull request: Update L10N Web UI resource files before ...

2015-10-29 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1001 Update L10N Web UI resource files before 4.6 RC1 (v2) Update the localization files for the next CloudStack 4.6. This PR cancel and replace the #984. cc @remibergsma

[GitHub] cloudstack pull request: Update L10N files for Next 4.6.

2015-10-29 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/984#issuecomment-152099956 Closed. Replace by #1001 --- 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

[GitHub] cloudstack pull request: implement upgrade path from 4.6.0 to 4.6....

2015-11-15 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1067#issuecomment-156846865 LGTM Works on my test deployment. 2015-11-15 19:52:41,711 DEBUG [c.c.u.DatabaseUpgradeChecker] (main:null) Cleanup script /usr/share/cloudstack

[GitHub] cloudstack pull request: Make master 4.7.0-SNAPSHOT and implement ...

2015-11-15 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1068#issuecomment-156859431 LGTM Tested on a 4.6.1 to 4.70 with success 2015-11-15 22:00:04,726 DEBUG [c.c.u.DatabaseUpgradeChecker] (main:null) Cleanup script /usr

[GitHub] cloudstack pull request: Fix the new line bug (invalid char on UI)...

2015-11-09 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1053 Fix the new line bug (invalid char on UI), add some Spanish translati… …on from Transifex Some strings with invalid encoding has been fixed too (Thanks Nicolas Vazquez

[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 /va

[GitHub] cloudstack pull request: CLOUDSTACK-9043 Remove \n (new line) in G...

2015-11-06 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1043 CLOUDSTACK-9043 Remove \n (new line) in German and Chinese translation The new line isn't in the original strings and are invalid character in the javascript files/ui

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

2015-11-06 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1038#issuecomment-154384773 ![selection_172](https://cloud.githubusercontent.com/assets/3995882/10995941/902c0abe-8478-11e5-922a-b3b742e3b7ef.png) @DaanHoogland see the capture

[GitHub] cloudstack pull request: Update de_DE Localization. Now Web UI 4.6...

2015-11-04 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1032 Update de_DE Localization. Now Web UI 4.6 is fully translated in German Last update to have 100% on German in Web UI Please 2 LGTM before the RC1! Thanks @resmo

[GitHub] cloudstack pull request: CLOUDSTACK-8930: Showing blank screen whe...

2015-10-07 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/906#issuecomment-146096049 @nitin-maharana #906 and #912 combined into #915 You can close this PR. Thanks. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: Pr 906 912 bug zone wizard and adv search...

2015-10-07 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/915 Pr 906 912 bug zone wizard and adv search popup You can merge this pull request into a Git repository by running: $ git pull https://github.com/milamberspace/cloudstack PR-906-912

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

2015-10-07 Thread milamberspace
Github user milamberspace closed the pull request at: https://github.com/apache/cloudstack/pull/912 --- 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

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

2015-10-07 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-146095698 Closed. New PR #915 --- 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

[GitHub] cloudstack pull request: Pr 906 912 bug zone wizard and adv search...

2015-10-08 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/915#issuecomment-146580968 Thanks @DaanHoogland for the testing. --- 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: Dockerfile part2

2015-10-09 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/910#discussion_r41616136 --- Diff: tools/docker/Dockerfile.centos6 --- @@ -23,15 +23,19 @@ LABEL Vendor="Apache.org" License="ApacheV2" Version="

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

2015-10-06 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145985072 @DaanHoogland you need to double-check that the old zoneWizard.js file is changed on you browser (clean cache) and on the tomcat installation (if not a new

[GitHub] cloudstack pull request: messages: fix string in pt_BR translation...

2015-08-27 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/749#issuecomment-135543206 No LTGM. This isn't the good way to fix this. --- 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: messages: fix string in pt_BR translation...

2015-08-27 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/749#issuecomment-135552489 The root cause of this issue is the missing comma after the backslash in the pt_BR translation. See this key/value on source messages.properties

[GitHub] cloudstack pull request: CLOUDSTACK-5863: revert volume snapshot f...

2015-08-25 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/732#issuecomment-134553590 LGTM to me too. Tested on a master test deployment (Ubuntu 14.04 / KVM / NFS) Success with a manual snapshot from cloudmonkey and web ui

[GitHub] cloudstack pull request: L10n update master from Transifex 2015090...

2015-09-01 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/769 L10n update master from Transifex 20150901 Update L10N resource files on master branch with 4.6 translation strings from Transifex (20150901) Fix an issue with the key

[GitHub] cloudstack pull request: Update L10N resource files on 4.5 branch ...

2015-09-01 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/770 Update L10N resource files on 4.5 branch with 4.5 translation strings… … from Transifex (20150901) Fix an issue with the key message.admin.guide.read on pt_BR L10N in web UI

[GitHub] cloudstack pull request: config: doc: add default value 10 in desc...

2015-09-04 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/777#issuecomment-137753131 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

[GitHub] cloudstack pull request: sysctl: don't modify /etc/sysctl.conf

2015-09-04 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/776#issuecomment-137753484 Use a cloud(stack).conf file under /etc/sysctl.d/ seems better and follow the distro philosophy. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: debian: Updated dependencies for Packages

2015-09-08 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/783#issuecomment-138449931 Check on packages.ubuntu.com for the requirements of version/packages. All is good for 12.04/14.04/14.10/15.04. LGTM (Perhaps remove the tomcat6

[GitHub] cloudstack pull request: CLOUDSTACK-8821: UI Change

2015-09-08 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/786#discussion_r38911089 --- Diff: ui/scripts/network.js --- @@ -1612,6 +1613,36

[GitHub] cloudstack pull request: messages: fix string in pt_BR translation...

2015-09-01 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/749#issuecomment-136713014 @bhaisaab I've send 2 PR (one for master/4.6 and one for 4.5 branch) to fix this issue (and update the L10N files) from Transifex. See #769 and #770

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

2015-10-05 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/912 Fix bugs on last step on Wizard zone and Adv Search box Javascript error: Uncaught ReferenceError: _ is not defined => Missing 'l' (to call the _l() function) Bug introd

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

2015-10-05 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145455703 @remibergsma Probably this is a 'minor' blocker bug for the 4.6 release. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Fix bugs on last step on Wizard zone and ...

2015-10-05 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/912#issuecomment-145668698 @runseb Good catch! Thanks @nitin-maharana to fix part of this bugs. @remibergsma my PR add a bug on Adv search label. --- If your project is set up

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-11-29 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1138 Update L10N resource files with 4.6 strings from Transifex (20151129) Small update of L10N files before the 4.6.1 release candidate cc @remibergsma You can merge this pull

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

2015-12-04 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1098#issuecomment-162067669 @remibergsma Yes I want! The update of localization files is done. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: CLOUDSTACK-9125: Brazilian translations

2015-12-09 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1194#issuecomment-163158429 Hello, The best way to translate the Web UI of CloudStack is to use the Transifex project [1] and use the script inside the cloudstack project

[GitHub] cloudstack pull request: L10N update before 4.7.0 RC1

2015-12-11 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1217 L10N update before 4.7.0 RC1 Some sentences (mainly for the new quota plugin) needs to be translated before the 4.7.0 release candidate (planned for 14th december) https

[GitHub] cloudstack pull request: L10N update before 4.7.0 RC1

2015-12-11 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1217#issuecomment-163881611 Thanks @terbolous and @agneya2001 The PR has been updated --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: README: happy holidays!

2015-12-23 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1281#issuecomment-166833276 :+1: Happy holidays! --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9092: L10n fix in "Add LDAP Ac...

2015-11-30 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1139#issuecomment-160591699 Not tested, but code is good. LGTM --- 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: 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

[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

[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

[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

[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

[GitHub] cloudstack pull request #1583: Update L10N resource files with 4.9 strings f...

2016-06-07 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1583 Update L10N resource files with 4.9 strings from Transifex (20160607) cc @swill before the 4.9 release. Just only the latest FR translation. You can merge this pull request into a Git

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2016-01-19 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1319#discussion_r50086554 --- Diff: ui/scripts/docs.js --- @@ -1008,7 +1008,7 @@ cloudStack.docs = { }, // Add volume helpVolumeName

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

2016-01-18 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1319#discussion_r50079832 --- Diff: ui/scripts/docs.js --- @@ -1008,7 +1008,7 @@ cloudStack.docs = { }, // Add volume helpVolumeName

[GitHub] cloudstack pull request: 4.8.0 fix i18n es

2016-02-09 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1407#issuecomment-181870469 Tested on 4.8 snapshot branch. The Web UI with Spanish language works well. LGTM --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: L10n update master 20160127

2016-01-27 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1376 L10n update master 20160127 cc @remibergsma @bhaisaab @DaanHoogland You can merge this pull request into a Git repository by running: $ git pull https://github.com/milamberspace

[GitHub] cloudstack pull request: L10n update 4.8 20160127

2016-01-27 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1375 L10n update 4.8 20160127 cc @remibergsma @bhaisaab @DaanHoogland You can merge this pull request into a Git repository by running: $ git pull https://github.com/milamberspace

[GitHub] cloudstack pull request: Update L10N resource files with 4.7 strin...

2016-01-27 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1374 Update L10N resource files with 4.7 strings from Transifex (20160127) cc @remibergsma @bhaisaab @DaanHoogland You can merge this pull request into a Git repository by running

[GitHub] cloudstack pull request: Add47xmessagesproperties transifex

2016-01-27 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1098#issuecomment-175740240 @bhaisaab see PR #1374 #1375 and #1376 --- 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

2016-01-27 Thread milamberspace
Github user milamberspace closed the pull request at: https://github.com/apache/cloudstack/pull/1098 --- 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

[GitHub] cloudstack pull request: More VR performance!

2016-03-21 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1356#issuecomment-199519708 @swill Already merge to 4.8 (for next 4.8.1) and on master (4.9) same commit id: 3eceb60f5ca8b86b3308243ae2905bbfbd54bc39 --- If your project is set up

[GitHub] cloudstack pull request: speedup iptables setup

2016-03-23 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1449#issuecomment-200296178 Good job. Thanks. Works fine in my 4.8 test installation (real platform) Several tests on RV and RVR routers with network restart with/without clean

[GitHub] cloudstack pull request: More VR performance!

2016-03-20 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1356#issuecomment-198946677 FYI This PR fixes the bug CLOUDSTACK-9255 https://issues.apache.org/jira/browse/CLOUDSTACK-9255 (perhaps need to backport this PR on 4.6

[GitHub] cloudstack pull request: More VR performance!

2016-03-21 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1356#issuecomment-199448860 @DaanHoogland @remibergsma I don't want backport this fix, my (new) current installation of CS is under 4.8, but this RV bug is coming with the new python

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-22 Thread milamberspace
Github user milamberspace commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1376#discussion_r60708800 --- Diff: tools/transifex/.tx/config --- @@ -89,3 +89,41 @@ trans.pt_BR = work-dir/messages_pt_BR.properties trans.ru_RU = work-dir

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-22 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-213336322 @swill Re-push-force is done. --- 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: L10n update master 20160127

2016-04-20 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-212572821 @swill squash & force push done. --- 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: L10n update 4.8 20160422

2016-04-22 Thread milamberspace
GitHub user milamberspace opened a pull request: https://github.com/apache/cloudstack/pull/1515 L10n update 4.8 20160422 @swill the good PR for 4.8 branch. You can merge this pull request into a Git repository by running: $ git pull https://github.com/milamberspace/cloudstack

[GitHub] cloudstack pull request: L10n update 4.8 20160127

2016-04-22 Thread milamberspace
Github user milamberspace closed the pull request at: https://github.com/apache/cloudstack/pull/1375 --- 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

[GitHub] cloudstack pull request: L10n update 4.8 20160127

2016-04-22 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1375#issuecomment-213496639 Close, replace by #1515 --- 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

[GitHub] cloudstack pull request: L10n update 4.8 20160422

2016-04-29 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1515#issuecomment-215645435 @swill Now the Jenkins build works, but the Travis failed with: No output has been received in the last 10m0s, this potentially indicates a stalled build

[GitHub] cloudstack pull request: L10n update 4.8 20160422

2016-04-29 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1515#issuecomment-215671781 @swill after a new push to start a new Travis build, Travis and Jenkins are now green! You can merge the PR! :-) --- If your project is set up for it, you

[GitHub] cloudstack pull request: L10n update 4.8 20160422

2016-04-27 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1515#issuecomment-215215852 @swill re-done. --- 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: L10n update 4.8 20160422

2016-04-26 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1515#issuecomment-214791549 @swill re-push force done. --- 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

  1   2   >