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

[GitHub] cloudstack pull request: L10n update master 20160127

2016-03-19 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-197977522 ### ACS CI BVT Run **Sumarry:** Build Number 107 Hypervisor xenserver NetworkType Advanced Passed=103 Failed=15 Skipped=4

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-19 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-211859221 The changes are only limited to UI, so there is no need to run smoke/integration tests. Cursory look at the changes looks fine. @swill I think these shou

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-211917846 Thanks koushik. I tend to agree with you on this one. I will add it to my merge list. Thanks... --- If your project is set up for it, you can reply to this email

[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-212324488 @swill I've just rebase from master, update the L10N files and push --force this PR. cc @koushik-das --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-212530190 Jenkins failed with a timeout (not related to your code). Can you squash your commits and do a force push again? Sorry for the runaround.. --- If your project is s

[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 your proje

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-20 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-212585886 @DaanHoogland do you know why jenkins is failing here? --- 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: L10n update master 20160127

2016-04-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-213099792 @swill, @milamberspace sorry, the build is gone, I didn't react quick enough --- If your project is set up for it, you can reply to this email and have your re

[GitHub] cloudstack pull request: L10n update master 20160127

2016-04-21 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-213100441 That is fine. @milamberspace would you mind just doing a force push again to kick off jenkins. I have seen other jenkins runs passing so I think jenkins is just not

[GitHub] cloudstack pull request: L10n update master 20160127

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

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

[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-22 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1376#discussion_r60733737 --- Diff: tools/transifex/.tx/config --- @@ -89,3 +89,41 @@ trans.pt_BR = work-dir/messages_pt_BR.properties trans.ru_RU = work-dir/messages_ru_RU.prop

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-02 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-216221653 LGTM tag:easypr --- 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 master 20160127

2016-05-04 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-216889967 This PR currently has merge conflicts, but #1515 is next in line, so you may want to wait till it is merged before you fix these conflicts. --- If your project is

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-10 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-218367016 #1515 has now been merged (sorry for the delay). Once you have a chance to fix the merge conflicts, I can get this merged. Thx... --- If your project is set up for

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-10 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-218375832 @swill conflicts are fixed. Ready to merge. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-10 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-218376280 @swill @milamberspace weird, we've lost our Travis integration with PRs here. https://travis-ci.org/apache/cloudstack/ says "The repository at apache/cloudstack was n

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-12 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-218808022 @milamberspace can you do a force push again. I have pushed fixes to Jenkins and Travis this morning, so with a new push we should be able to get this green. Thx...

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-12 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-218958767 @swill Done. Jenkins and Travis are green now. Ready to merge. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack pull request: L10n update master 20160127

2016-05-12 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-218959204 LGTM tag:mergeready --- 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: L10n update master 20160127

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1376 --- 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: L10n update master 20160127

2016-05-26 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1376#issuecomment-221846787 ### ACS CI BVT Run **Sumarry:** Build Number 56 Hypervisor xenserver NetworkType Advanced Passed=72 Failed=1 Skipped=3 _