Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/462
---
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 user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/462#issuecomment-112313034
convinced, the router is a debian box (for now and will remain linux in the
near future.
LGTM
---
If your project is set up for it, you can reply to
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/460
---
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 user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/460#issuecomment-112339045
please make it a seperate fix, not an update. I jumped the gun.
On Tue, Jun 16, 2015 at 10:36 AM, Rafael da Fonseca
notificati...@github.com wrote:
Github user rsafonseca commented on the pull request:
https://github.com/apache/cloudstack/pull/460#issuecomment-112344047
@DaanHoogland done!
https://github.com/apache/cloudstack/pull/464
I should have checked travis for this one, still my bad :(
---
If your project is set
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/460#issuecomment-112333921
I shouldn't have:( exception uncought. Really curious why the pull request
builder accepted this.
---
If your project is set up for it, you can reply to this
Github user rsafonseca commented on the pull request:
https://github.com/apache/cloudstack/pull/460#issuecomment-112338005
my bad there.. will surround in try/catch block and update
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
I think it's caught and also that the pull request builder doesn't
build -Dnoredist. I will check and fix if so
On Tue, Jun 16, 2015 at 10:36 AM, rsafonseca g...@git.apache.org wrote:
Github user rsafonseca commented on the pull request:
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/402
---
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 user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/394
---
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 user rsafonseca opened a pull request:
https://github.com/apache/cloudstack/pull/460
Fix findbugs encoding issue in VmwareStorageProcessor.java
Any encoding would do fine as it's just used to generate a UUID. Sticking
with UTF-8 for consistency
You can merge this pull
GitHub user rsafonseca opened a pull request:
https://github.com/apache/cloudstack/pull/462
Fix findbugs encoding issue in VmwareResource.java
Key file should be UTF-8 encoded in VR
You can merge this pull request into a Git repository by running:
$ git pull
Github user rsafonseca commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/462#discussion_r32472663
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -705,7 +705,7 @@ public ExecutionResult
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/460#discussion_r32465788
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java
---
@@ -2265,7 +2265,7 @@ public Answer
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/462#discussion_r32466095
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java
---
@@ -705,7 +705,7 @@ public ExecutionResult
Github user rsafonseca commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/460#discussion_r32469324
--- Diff:
plugins/hypervisors/vmware/src/com/cloud/storage/resource/VmwareStorageProcessor.java
---
@@ -2265,7 +2265,7 @@ public Answer
Github user rsafonseca commented on the pull request:
https://github.com/apache/cloudstack/pull/460#issuecomment-112232241
Here's an example of an instance where i also removed a
Charset.defaultCharset() that was placed in the reading of a socket, where
cloudstack controls both ends
GitHub user rsafonseca opened a pull request:
https://github.com/apache/cloudstack/pull/402
Fix findbugs encoding issue in MemStat.java
Was safe either way as this piece of code should only run in linux and only
contains safe characters, this just gets rid of the findbugs warning
GitHub user rsafonseca opened a pull request:
https://github.com/apache/cloudstack/pull/394
Fix findbugs encoding issue
This is done by calling HttpMethodBase's getResponseBodyAsString() which
properly looks up the specified encoding in the request's Content-Type header
This
19 matches
Mail list logo