[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-25 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-407849628
 
 
   @rafaelweingartner corresponding issues have been created.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-25 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-407836513
 
 
   I was going to create those issues after this got merged and those actually 
became _issues_.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-25 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-407833744
 
 
   @rhtyd @rafaelweingartner @marcaurele @borisstoyanov @DaanHoogland this PR 
should be good to be merged, I also added `TODO and followups` section in the 
description for followup PRs.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-25 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-407791115
 
 
   @marcaurele done, let's see how much Jenkins likes this change!


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-24 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-407469314
 
 
   @rafaelweingartner according to 
[this](https://issues.jenkins-ci.org/browse/JENKINS-46641), the crash happened 
most probably because of running out of memory and not timeout.
   This is happening now, as I mentioned, because this PR enables PMD on all 
projects and there are huge number of problems it can find.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-19 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-406393616
 
 
   @rafaelweingartner it might be because of timeout issue of some sorts. 
previously PMD wasn't running for all the projects which this PR changes that, 
and it breaks at the end of the job while preparing PMD reports:
   
   ```
   [PMD] Successfully parsed file 
/home/jenkins/jenkins-slave/workspace/cloudstack-pr-analysis/utils/target/pmd.xml
 with 137579 unique warnings and 127 duplicates.
Using GitBlamer to create author and commit information for all 
warnings.
GIT_COMMIT=40e8287b158cab4422df81946d570d5a4f74feea, 
workspace=/home/jenkins/jenkins-slave/workspace/cloudstack-pr-analysis
> git rev-parse 40e8287b158cab4422df81946d570d5a4f74feea^{commit} # 
timeout=10
No blame results for request 
.
No blame results for request 
.
No blame results for request 
.
No blame results for request 
.
No blame results for request 
.
No blame results for request 
.
No blame results for request 
.
No blame results for request 
.
   Agent went offline during the build
   ERROR: Connection was broken: java.io.EOFException
   at 
java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2679)
   at 
java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:3154)
   at 
java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:862)
   at java.io.ObjectInputStream.(ObjectInputStream.java:358)
   at 
hudson.remoting.ObjectInputStreamEx.(ObjectInputStreamEx.java:48)
   at 
hudson.remoting.AbstractSynchronousByteArrayCommandTransport.read(AbstractSynchronousByteArrayCommandTransport.java:36)
   at 
hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:63)
   Caused: java.io.IOException: Unexpected termination of the channel
   at 
hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:77)
   Build step 'Publish PMD analysis results' marked build as failure
   ERROR: Step ?Publish duplicate code analysis results? failed: no workspace 
for cloudstack-pr-analysis #7418
   ERROR: Step ?Archive the artifacts? failed: no workspace for 
cloudstack-pr-analysis #7418
   ERROR: Step ?Publish Cobertura Coverage Report? failed: no workspace for 
cloudstack-pr-analysis #7418
   ERROR: Step ?Publish OpenClover coverage report? failed: no workspace for 
cloudstack-pr-analysis #7418
   Putting comment on the pull request
   ERROR: H31 is offline; cannot locate JDK 1.8 (latest)
   ERROR: H31 is offline; cannot locate JDK 1.8 (latest)
   Finished: FAILURE
   ```
   
   The last `` took a long time (I didn't pay close attention, but 
something like 5 minutes I guess).


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-19 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-406344674
 
 
   @rafaelweingartner hmm, not sure, let me just push an empty commit to kick 
it off again.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-13 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-404858588
 
 
   @rhtyd rebased.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-07-03 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-402212403
 
 
   @borisroman was the test interrupted or still running?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-06-26 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-400421427
 
 
   @marcaurele @rhtyd @rafaelweingartner @DaanHoogland @borisroman rebased on 
latest master and fixed the conflicts, should be good to merge after Travis, 
Jenkins, BO passed.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-17 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-389886169
 
 
   @rhtyd is this test still running or has got interrupted?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-11 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-388419213
 
 
   @rhtyd that sounds like a plan! :+1: 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-11 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-388380772
 
 
   I agree with @rhtyd , I don't want to proceed until we would understand 
what's wrong with BO packaging. Specially that almost that's the only system we 
depend on for packaging/testing PRs and the fact that packaging manually works 
and not on BO.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-11 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-388380772
 
 
   I agree with @rhtyd , I don't want to proceed until we would understand 
what's wrong with BO packaging. Specially that almost that's the only system we 
depend on for packaging/testing PRs and the fact that packaging manually works 
and not on BO. /cc @rafaelweingartner 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-10 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-388118485
 
 
   @rhtyd I agree, figuring out `VERSION` in `deb` packaging process is weird 
and totally depends on code formatting, but at the same time it's fixed within 
this PR (33b020319b3930ecc8cdf430789bb2fa36447de7) and generally executing 
`build-deb.sh` works, that's why I suspect there's something wrong with BO 
packaging deb (as @rafaelweingartner successfully managed to build the same 
artifacts himself based on the changes in this PR)


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-08 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-387511224
 
 
   @rhtyd thanks for the explanation, but that would be a surprising 
coincidence and I would say unlikely, since both packaging job - after the 
debian fix - failed.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-08 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-387498388
 
 
   Thanks @rafaelweingartner for validating the packaging, that's why I'm 
guessing the way BO does the packaging is somehow broken/different/hacked, or 
simply does something which I haven't foreseen?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-08 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-387478524
 
 
   Can any one who has access to packaging job, provide me some log to see why 
debian packaging fails? And also the command/script is used to trigger the job.
   I cannot reproduce this myself.
   /cc @DaanHoogland @rhtyd 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-04 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386690495
 
 
   This PR should be good now. Call for reviewing: @rhtyd @rafaelweingartner 
@marcaurele @DaanHoogland @GabrielBrascher @wido @borisroman @swill 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-04 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386684256
 
 
   @DaanHoogland it's still failing. Are you running customized command for 
building deb packages? I cannot reproduce this.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-04 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386639382
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-04 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386618114
 
 
   Thanks @DaanHoogland, it is fixed already, I haven't pushed yet.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-03 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-386329926
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] khos2ow commented on issue #2613: Cleanup POMs

2018-05-01 Thread GitBox
khos2ow commented on issue #2613: Cleanup POMs
URL: https://github.com/apache/cloudstack/pull/2613#issuecomment-385786660
 
 
   @blueorangutan package


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services