Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1263
---
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 swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-209095939
This one looks like it is ready to go now. I will add it to my next 'ready
to merge' batch. Thanks again for all the hard work @GabrielBrascher. :+1:
---
If your
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-209084883
I am sorry, but I forgot to update the com.cloud.api.ApiServletTest class,
now that those tests are running, all checks have passed.
@bhaisaab @DaanHoogl
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-209009495
@GabrielBrascher thanks, Travis failed again. I'll test this on my env
tomorrow (night for me here). Meanwhile, can you build with -T4 -Dnoredist
-Dsimulator flags
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208969926
@bhaisaab force pushed.
---
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 user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208956680
@GabrielBrascher can you do a rebase or push -f again so that Travis build
can be re-kicked
---
If your project is set up for it, you can reply to this email and
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208535036
@GabrielBrascher yes, this is something that we are trying to work through.
We have updated the source to give more verbose logging so we can better
understand the s
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208533273
Travis keeps with the issue *missing file/directory
'/home/travis/build/apache/cloudstack/tools/marvin/dist/Marvin-***.tar.gz*'.
---
If your project is set
Github user GabrielBrascher closed the pull request at:
https://github.com/apache/cloudstack/pull/1263
---
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 f
GitHub user GabrielBrascher reopened a pull request:
https://github.com/apache/cloudstack/pull/1263
Removed unused code from com.cloud.api.ApiServer
**Removed â\_â from variables names**: private variables with â\_â
at the beginning is common in C++ but not in Java.
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208339528
@GabrielBrascher could you please close and re-open this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208332183
can be a polution of the workspace (I'll clean it) or a jenkins having a
different git loaded then is on disk. I've seen the last one once before on an
interna
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-208330435
@DaanHoogland do you know what is going on with jenkins in this case?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207989355
@DaanHoogland thanks for the tip.
---
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 pr
GitHub user GabrielBrascher reopened a pull request:
https://github.com/apache/cloudstack/pull/1263
Removed unused code from com.cloud.api.ApiServer
**Removed â\_â from variables names**: private variables with â\_â
at the beginning is common in C++ but not in Java.
Github user GabrielBrascher closed the pull request at:
https://github.com/apache/cloudstack/pull/1263
---
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 f
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207956609
@GabrielBrascher you could try to close the PR and reopen it. It matters if
the queue at builds.apache.org has a full queue or not.
---
If your project is set
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207914050
Seems that `push -f` is not making the Jenkins and Travis to rerun their
tests. Is there any workaround for that?
---
If your project is set up for it, you
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207490720
Built the package LGTM after review
[INFO]
[INFO] Reactor Summary:
[INFO]
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207488773
@pdube removed those uderscores, also I modified the
"com.cloud.api.ApiServer.setEncodeApiResponse(boolean)" method to static again.
Thanks.
---
If your p
Github user GabrielBrascher commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59043684
--- Diff: server/src/com/cloud/api/ApiServlet.java ---
@@ -156,7 +158,7 @@ void processRequestInContext(final HttpServletRequest
req, final HttpS
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59026874
--- Diff: server/src/com/cloud/api/ApiServlet.java ---
@@ -156,7 +158,7 @@ void processRequestInContext(final HttpServletRequest
req, final HttpServletResp
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59026830
--- Diff: server/src/com/cloud/api/ApiServer.java ---
@@ -1340,37 +1340,31 @@ public String getSerializedApiError(final
ServerApiException ex, final Map ge
Github user pdube commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1263#discussion_r59026583
--- Diff: server/src/com/cloud/api/ApiServer.java ---
@@ -223,10 +226,7 @@
private static ExecutorService s_executor = new ThreadPoolExecutor(10,
Github user swill commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207112861
Yes, I think this one is ready. I am trying to get two LGTM code reviews
on every PR, so can one of @DaanHoogland or @pdube give me a LGTM, I have one
from @rafaelwe
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-207084266
@swill what about this one?
The tests were already executed and everything passed.
---
If your project is set up for it, you can reply to this email a
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-204043787
The code seems ok now. 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 projec
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203947763
@DaanHoogland @rafaelweingartner @pdube removed those ".class" files, also
all checks have passed.
Thanks.
---
If your project is set up for it, you ca
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203492750
@DaanHoogland @rafaelweingartner well noted, I have to remove those
".class" files;
@pdube I am going to verify this checkstyle error.
Thanks.
---
Github user pdube commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203434250
Hi @GabrielBrascher
Travis noted:
Failed during checkstyle execution: There are 1 checkstyle errors.
Have you run *mvn validate* on your code?
Github user rafaelweingartner commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203406148
@GabrielBrascher, it seems that everything is ok with this PR (giving CI),
but to push it forward with reviews, can you remove the .class from your commit?
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203396908
@GabrielBrascher you have .class files in your commit according to github.
Can you check?
And please repush anyway? thanks
---
If your project is set
Github user bvbharatk commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-203392098
### ACS CI BVT Run
**Sumarry:**
Build Number 147
Hypervisor xenserver
NetworkType Advanced
Passed=106
Failed=0
Skipped=4
Github user GabrielBrascher commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-172386387
@rodrigo93 This problem is not from the Jenkins. I am aware with the real
problem (APIChecker bean) just haven't forced to commit again with the
corrections
Github user rodrigo93 commented on the pull request:
https://github.com/apache/cloudstack/pull/1263#issuecomment-172383741
@GabrielBrascher Nice change. It is always good to keep the good
programming practices, especially in a project of this size.
Can you commit it again? It seem
GitHub user GabrielBrascher opened a pull request:
https://github.com/apache/cloudstack/pull/1263
Removed unused code from com.cloud.api.ApiServer
**Removed â\_â from variables names**: private variables with â\_â
at the beginning is common in C++ but not in Java.
36 matches
Mail list logo
