[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-22 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123661436 The https://builds.apache.org/job/cloudstack-pull-requests fail after adding the Default Charset test. I think the localization of the Jenkins server needs to be

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/612 --- 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-8658: make initializer static ...

2015-07-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123609068 LGTM :+1: merging... Thanks, @DaanHoogland !!! Tested with: KVM + Qemu on Centos7 Management Server on Centos7 Agent 4.6.0

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123579225 Keys are being created: [root@kvm1 ~]# ls -lart ~/.ssh/ total 16 dr-xr-x---. 3 root root 4096 Jul 22 02:37 .. drwx--. 2 root root 4096

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123571169 Will test it against KVM + Agent. --- 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 pro

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123526216 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 enab

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123471733 Hey @DaanHoogland Just the initialiser stuff and the return of the getPreferredCharset() method. Cheers, Wilder --- If your project i

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123460301 @wilderrodrigues of course, sure you want to wait for that? And to what extend? we want tests for this initializer but also for the agent and it's functionin

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/612#issuecomment-123387872 Hey @DaanHoogland Could you write a test for that, please? :) Cheers, Wilder --- If your project is set up for it, you can reply to th

[GitHub] cloudstack pull request: CLOUDSTACK-8658: make initializer static ...

2015-07-21 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/612 CLOUDSTACK-8658: make initializer static instead of default bad bug: ommision of the static keyword on initializer leaves data uninitialized :{ You can merge this pull request into a Git r