[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-02-10 Thread jlk
Github user jlk commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-182427828 Cool - 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 feature

[GitHub] cloudstack pull request: CLOUDSTACK-9283: add pid to java argument...

2016-02-10 Thread davidamorimfaria
GitHub user davidamorimfaria opened a pull request: https://github.com/apache/cloudstack/pull/1409 CLOUDSTACK-9283: add pid to java arguments in cloudstack-usage.service cloudstack-usage fails to start throwing Integer exception during PID retrieval, and the service keeps

Re: Spannish translation mangled

2016-02-10 Thread Marco Sinhoreli
Hi folks, Any issue in transifex? It’s showing 5% complete. Before Brazilian Portuguese was 100%. Many others languages are only 5% complete. Best regards, Marco Sinhoreli Consultant Manager marco.sinhor...@shapeblue.com mobile: +55 21 98276 3636 Praia de Botafogo 501, bloco 1 - sala 101 –

[GitHub] cloudstack pull request: CLOUDSTACK-9283: add pid to java argument...

2016-02-10 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1409#issuecomment-182446008 proof that this works in centos 7.2 running cloudstack 4.7.1: ``` # cat /etc/redhat-release CentOS Linux release 7.2.1511 (Core)

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-02-10 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-182444102 To update, we ran more tests on our production platform spread out over 60 hypervisors we saw these lock message pop up in various occasions. Internally we

[GitHub] cloudstack pull request: CLOUDSTACK-9280: Allow system VM volumes ...

2016-02-10 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1406#issuecomment-182452345 Tests were performed using a running environment. Deploy a zone, then delete the hosts in the zone and make sure there are no system VMs remaining. Wait for

[GitHub] cloudstack pull request: [4.7] VMware boot order fix

2016-02-10 Thread bhaisaab
Github user bhaisaab closed the pull request at: https://github.com/apache/cloudstack/pull/1395 --- 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

[GitHub] cloudstack pull request: [4.7] VMware boot order fix

2016-02-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1395#issuecomment-182489462 Figured it is not needed, closing for now --- 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

Re: Contributing to the Documentation

2016-02-10 Thread Pierre-Luc Dion
We also have multiple repos: release notes: https://github.com/apache/cloudstack-docs-rn install guide: https://github.com/apache/cloudstack-docs-install admin guide: https://github.com/apache/cloudstack-docs-admin general doc: https://github.com/apache/cloudstack-docs theses project use

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-02-10 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1410 CLOUDSTACK-6928: fix issue disk I/O throttling not applied Disk IO throttling (for KVM) is not applied in the merge of 4.2. Tests passed: (1) start vm (2) attach volume

Re: Spannish translation mangled

2016-02-10 Thread Milamber
Hello, Issue fixed, I re-upload the localization files on transifex from the 4.8 branch. The issue is become because the /tools/transifex/.tx/config don't include the 4.8 config (the 4.8.0 release has been release without the updates from transifex). To fix them, I had just re-upload the

Re: Spannish translation mangled

2016-02-10 Thread Laszlo Hornyak
Looks good now, thank you! On Wed, Feb 10, 2016 at 9:37 PM, Milamber wrote: > Hello, > > Issue fixed, I re-upload the localization files on transifex from the 4.8 > branch. > > The issue is become because the /tools/transifex/.tx/config don't include > the 4.8 config (the

Re: Spannish translation mangled

2016-02-10 Thread Laszlo Hornyak
Hi, Might be a related issue, but looks like now I am translating either from Spanish or from Portuguese, but anyway it is a nice challenge... Can you give me back the original English? [image: Inline image 1] Thank you, Laszlo On Wed, Feb 10, 2016 at 2:45 PM, Marco Sinhoreli <

Re: Spannish translation mangled

2016-02-10 Thread Milamber
Hello, On transifex website, you can click on "Join Team" button to start to help in this project. https://www.transifex.com/ke4qqq/CloudStack_UI/ Milamber On 10/02/2016 21:13, Marco Sinhoreli wrote: I can help with Spanish and Portuguese (Portugal) translation too. Somebody can add me in

RE: Contributing to the Documentation

2016-02-10 Thread GoGoZoom PDX
Thanks, Paul! On Feb 9, 2016 9:55 AM, "Paul Angus" wrote: > Hi Eric, > > The docs are written in reStructuredText. http://rst.ninjs.org/ is a > handy editor > > > > > [image: ShapeBlue] > Paul Angus > VP Technology , ShapeBlue > d: *+44 203

[GitHub] cloudstack pull request: CLOUDSTACK-9283: add pid to java argument...

2016-02-10 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1409#discussion_r52551673 --- Diff: packaging/systemd/cloudstack-usage.service --- @@ -27,10 +27,11 @@ Environment=JAVA_HOME=/usr/lib/jvm/jre

[GitHub] cloudstack pull request: CLOUDSTACK-8728: Testcase to Verify if VR...

2016-02-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/684 --- 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-8728: Testcase to Verify if VR...

2016-02-10 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/684#issuecomment-182751801 LGTM !! Test router internal basic zone ... SKIP: Marvin configuration has no host credentials tocheck router services

[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2016-02-10 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1011#issuecomment-182276612 @runseb Totally agree, reverted merge. --- 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: automated 9277 9276 9275 9274 9273 9179 9...

2016-02-10 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1401#issuecomment-182276957 @runseb Keep an eye on this PR, it has the same ip address problem and I commented last week already. --- If your project is set up for it, you can reply to

RE: Spannish translation mangled

2016-02-10 Thread Nuno Tavares
Hi Milamber, Thanks, and sorry for the extra-trouble... I did the push with: tx push -s -t .. after linking the source, resource was 48xmessagesproperties. I'm not sure if I did something wrong.. didn't check that branch before, but it's showing only 5% of strings in "translated state", while

[GitHub] cloudstack pull request: 4.8.0 fix i18n es

2016-02-10 Thread ntavares
Github user ntavares commented on the pull request: https://github.com/apache/cloudstack/pull/1407#issuecomment-182282200 @Daan, yes the capitalizations are wrong in several places I've fixed the most visible (home page), but that's another matter. As for the double

[GitHub] cloudstack pull request: CLOUDSTACK-9128: Testcase to verify physi...

2016-02-10 Thread pritisarap12
Github user pritisarap12 commented on the pull request: https://github.com/apache/cloudstack/pull/1199#issuecomment-182284142 Rebased with current 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

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-02-10 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-182246389 @jlk What I've seen is that it all happens in a 2 to 3 second window. Usually the security group script doesn't run for a very long time, so that made me think 10

[GitHub] cloudstack pull request: Fixes regarding VOLUME_DELETE events resu...

2016-02-10 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1373#issuecomment-182301032 Even for forward-merged bug fixes? --- 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: CLOUDSTACK-9280: Allow system VM volumes ...

2016-02-10 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1406#issuecomment-182247435 How have you been able to test this one? --- 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:

2016-02-10 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/commit/56859c99041c66d417bf0af9a93b1d6e1e79005b#commitcomment-15999267 In tools/marvin/marvin/config/test_data.py: In tools/marvin/marvin/config/test_data.py on line 798: why on earth are

[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2016-02-10 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/1011#issuecomment-182255888 Please revert asap, there is an IP in there that should not be there. -1 --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2016-02-10 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1011#discussion_r52429775 --- Diff: tools/marvin/marvin/config/test_data.py --- @@ -784,6 +784,18 @@ "format": "OVA", "ispublic": "true" },

Important Pending Items

2016-02-10 Thread Sebastien Goasguen
Morning folks, We have several crucial pending items, that we need to resolve before moving on: 1- We need an RM for master ( just saw some commits there that should be reverted or merged properly in other branches). 2- We need to automate writing release notes, pushing/tagging new docs when

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-02-10 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1408#issuecomment-182265889 I updated the code and tested it. We came across this mainly when Terraform was being used. Just checked the logs of two hypervisors: 2016-02-10

[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2016-02-10 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1011#discussion_r52429712 --- Diff: tools/marvin/marvin/config/test_data.py --- @@ -784,6 +784,18 @@ "format": "OVA", "ispublic": "true" },

[GitHub] cloudstack pull request: Fixes regarding VOLUME_DELETE events resu...

2016-02-10 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1373#issuecomment-182267349 @ProjectMoon Currently waiting for a new Release Manager for version 4.9 to pick this up --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: 4.8.0 fix i18n es

2016-02-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1407#issuecomment-182267230 skimmed the code to check that indeed double encodings are gone. Not being spanish I can not judge the capitalisations. These would look silly in dutch. LGTM

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-02-10 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1408#discussion_r5255 --- Diff: scripts/vm/network/security_group.py --- @@ -1029,6 +1044,14 @@ def addFWFramework(brname): sys.exit(1) cmd = args[0]

[GitHub] cloudstack pull request: kvm: Aqcuire lock when running security g...

2016-02-10 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1408#discussion_r52552153 --- Diff: scripts/vm/network/security_group.py --- @@ -36,6 +39,18 @@ hyper = cfo.getEntry("hypervisor.type") if hyper == "lxc":