You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@karaf.apache.org by Miroslav Beranič <mi...@mibesis.si> on 2018/09/11 06:35:32 UTC
PR608 - Updated Scheduler feature to support persistence - Failed
Hi,
I've subscribed to dev mailing list.
I've submitted a pull request at https://github.com/apache/karaf/pull/608,
but I see Jenkins build failed do to:
Caused by: org.apache.rat.mp.RatCheckException: Too many files with
unapproved license: 4 See RAT report in:
/home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
located at :
https://builds.apache.org/job/karaf-pr/665/console
What should I update, to make this work?
Kind Regards,
Miroslav
--
Miroslav Beranič
MIBESIS
+386(0)40/814-843
miroslav.beranic@mibesis.si
http://www.mibesis.si
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Jean-Baptiste Onofré <jb...@nanthrax.net>.
Hi,
Catcha. I will provide my comments directly in the PR.
Regards
JB
On 11/09/2018 09:12, Miroslav Beranič wrote:
> Hi JB,
>
> yes, I know. I need Quartz API, so this was my main motivation. I guess not
> all changes are valid.
> Open for any suggestion, let me know.
>
> Regards,
> Miroslav
>
>
> V V tor., 11. sep. 2018 ob 08:49 je oseba Jean-Baptiste Onofré <
> jb@nanthrax.net> napisala:
>
>> Hi,
>>
>> The ASF headers are missing.
>>
>> By the way, I see a lot of incoherent changes here and not good IMHO.
>>
>> I will do a complete review.
>>
>> Regards
>> JB
>>
>> On 11/09/2018 08:35, Miroslav Beranič wrote:
>>> Hi,
>>>
>>> I've subscribed to dev mailing list.
>>>
>>> I've submitted a pull request at
>> https://github.com/apache/karaf/pull/608,
>>> but I see Jenkins build failed do to:
>>> Caused by: org.apache.rat.mp.RatCheckException: Too many files with
>>> unapproved license: 4 See RAT report in:
>>>
>> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
>>>
>>>
>>> located at :
>>> https://builds.apache.org/job/karaf-pr/665/console
>>>
>>> What should I update, to make this work?
>>>
>>> Kind Regards,
>>> Miroslav
>>>
>>>
>>
>> --
>> Jean-Baptiste Onofré
>> jbonofre@apache.org
>> http://blog.nanthrax.net
>> Talend - http://www.talend.com
>>
>
>
--
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Miroslav Beranič <mi...@mibesis.si>.
Hi JB,
yes, I know. I need Quartz API, so this was my main motivation. I guess not
all changes are valid.
Open for any suggestion, let me know.
Regards,
Miroslav
V V tor., 11. sep. 2018 ob 08:49 je oseba Jean-Baptiste Onofré <
jb@nanthrax.net> napisala:
> Hi,
>
> The ASF headers are missing.
>
> By the way, I see a lot of incoherent changes here and not good IMHO.
>
> I will do a complete review.
>
> Regards
> JB
>
> On 11/09/2018 08:35, Miroslav Beranič wrote:
> > Hi,
> >
> > I've subscribed to dev mailing list.
> >
> > I've submitted a pull request at
> https://github.com/apache/karaf/pull/608,
> > but I see Jenkins build failed do to:
> > Caused by: org.apache.rat.mp.RatCheckException: Too many files with
> > unapproved license: 4 See RAT report in:
> >
> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
> >
> >
> > located at :
> > https://builds.apache.org/job/karaf-pr/665/console
> >
> > What should I update, to make this work?
> >
> > Kind Regards,
> > Miroslav
> >
> >
>
> --
> Jean-Baptiste Onofré
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>
--
Miroslav Beranič
MIBESIS
+386(0)40/814-843
miroslav.beranic@mibesis.si
http://www.mibesis.si
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Jean-Baptiste Onofré <jb...@nanthrax.net>.
Hi,
The ASF headers are missing.
By the way, I see a lot of incoherent changes here and not good IMHO.
I will do a complete review.
Regards
JB
On 11/09/2018 08:35, Miroslav Beranič wrote:
> Hi,
>
> I've subscribed to dev mailing list.
>
> I've submitted a pull request at https://github.com/apache/karaf/pull/608,
> but I see Jenkins build failed do to:
> Caused by: org.apache.rat.mp.RatCheckException: Too many files with
> unapproved license: 4 See RAT report in:
> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
>
>
> located at :
> https://builds.apache.org/job/karaf-pr/665/console
>
> What should I update, to make this work?
>
> Kind Regards,
> Miroslav
>
>
--
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Miroslav Beranič <mi...@mibesis.si>.
Hi François,
thanks!
Regards,
Miroslav
V V tor., 11. sep. 2018 ob 08:46 je oseba Francois Papon <
francois.papon@openobject.fr> napisala:
> Hi,
>
> You are missing the Apache License header in files :
>
> 4 Unknown Licenses
>
> *****************************************************
>
> Files with unapproved licenses:
>
> scheduler/src/main/java/org/apache/karaf/scheduler/SchedulerStorage.java
>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiScheduler.java
>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/QuartzSchedulerStorage.java
>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiSchedulerFactory.java
>
> I put some comments on your PR ;)
>
> You can see the detailed files here :
>
>
> https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
>
> regards,
>
> François Papon
> fpapon@apache.org
>
> Le 11/09/2018 à 10:35, Miroslav Beranič a écrit :
> > Hi,
> >
> > I've subscribed to dev mailing list.
> >
> > I've submitted a pull request at
> https://github.com/apache/karaf/pull/608,
> > but I see Jenkins build failed do to:
> > Caused by: org.apache.rat.mp.RatCheckException: Too many files with
> > unapproved license: 4 See RAT report in:
> >
> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
> >
> >
> > located at :
> > https://builds.apache.org/job/karaf-pr/665/console
> >
> > What should I update, to make this work?
> >
> > Kind Regards,
> > Miroslav
> >
> >
>
>
>
--
Miroslav Beranič
MIBESIS
+386(0)40/814-843
miroslav.beranic@mibesis.si
http://www.mibesis.si
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Miroslav Beranič <mi...@mibesis.si>.
Ahaaaa, this is it. I was always compiling with -Pfastinstall ...
Thanks
Rgds,
Miroslav
V V tor., 11. sep. 2018 ob 11:01 je oseba Jean-Baptiste Onofré <
jb@nanthrax.net> napisala:
> Hi,
>
> Jenkins is doing the same as local.
>
> Just do a complete:
>
> mvn clean install -Prat
>
> you will have the same.
>
> Regards
> JB
>
> On 11/09/2018 10:58, Miroslav Beranič wrote:
> > Hi François,
> >
> > I have to be logged in to access
> >
> https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
> >
> > Is registration open or I need invite? I did not see a registration form.
> >
> > But more important:
> >
> > How do I compile locally in a way, to have this error before pushing to
> > Apache's Jenkins - save me embarrassment and you resources on servers.
> >
> > Regards,
> > Miroslav
> >
> >
> > V V tor., 11. sep. 2018 ob 08:46 je oseba Francois Papon <
> > francois.papon@openobject.fr> napisala:
> >
> >> Hi,
> >>
> >> You are missing the Apache License header in files :
> >>
> >> 4 Unknown Licenses
> >>
> >> *****************************************************
> >>
> >> Files with unapproved licenses:
> >>
> >>
> scheduler/src/main/java/org/apache/karaf/scheduler/SchedulerStorage.java
> >>
> >>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiScheduler.java
> >>
> >>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/QuartzSchedulerStorage.java
> >>
> >>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiSchedulerFactory.java
> >>
> >> I put some comments on your PR ;)
> >>
> >> You can see the detailed files here :
> >>
> >>
> >>
> https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
> >>
> >> regards,
> >>
> >> François Papon
> >> fpapon@apache.org
> >>
> >> Le 11/09/2018 à 10:35, Miroslav Beranič a écrit :
> >>> Hi,
> >>>
> >>> I've subscribed to dev mailing list.
> >>>
> >>> I've submitted a pull request at
> >> https://github.com/apache/karaf/pull/608,
> >>> but I see Jenkins build failed do to:
> >>> Caused by: org.apache.rat.mp.RatCheckException: Too many files with
> >>> unapproved license: 4 See RAT report in:
> >>>
> >>
> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
> >>>
> >>>
> >>> located at :
> >>> https://builds.apache.org/job/karaf-pr/665/console
> >>>
> >>> What should I update, to make this work?
> >>>
> >>> Kind Regards,
> >>> Miroslav
> >>>
> >>>
> >>
> >>
> >>
> >
>
> --
> Jean-Baptiste Onofré
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>
--
Miroslav Beranič
MIBESIS
+386(0)40/814-843
miroslav.beranic@mibesis.si
http://www.mibesis.si
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Jean-Baptiste Onofré <jb...@nanthrax.net>.
Hi,
Jenkins is doing the same as local.
Just do a complete:
mvn clean install -Prat
you will have the same.
Regards
JB
On 11/09/2018 10:58, Miroslav Beranič wrote:
> Hi François,
>
> I have to be logged in to access
> https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
>
> Is registration open or I need invite? I did not see a registration form.
>
> But more important:
>
> How do I compile locally in a way, to have this error before pushing to
> Apache's Jenkins - save me embarrassment and you resources on servers.
>
> Regards,
> Miroslav
>
>
> V V tor., 11. sep. 2018 ob 08:46 je oseba Francois Papon <
> francois.papon@openobject.fr> napisala:
>
>> Hi,
>>
>> You are missing the Apache License header in files :
>>
>> 4 Unknown Licenses
>>
>> *****************************************************
>>
>> Files with unapproved licenses:
>>
>> scheduler/src/main/java/org/apache/karaf/scheduler/SchedulerStorage.java
>>
>> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiScheduler.java
>>
>> scheduler/src/main/java/org/apache/karaf/scheduler/core/QuartzSchedulerStorage.java
>>
>> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiSchedulerFactory.java
>>
>> I put some comments on your PR ;)
>>
>> You can see the detailed files here :
>>
>>
>> https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
>>
>> regards,
>>
>> François Papon
>> fpapon@apache.org
>>
>> Le 11/09/2018 à 10:35, Miroslav Beranič a écrit :
>>> Hi,
>>>
>>> I've subscribed to dev mailing list.
>>>
>>> I've submitted a pull request at
>> https://github.com/apache/karaf/pull/608,
>>> but I see Jenkins build failed do to:
>>> Caused by: org.apache.rat.mp.RatCheckException: Too many files with
>>> unapproved license: 4 See RAT report in:
>>>
>> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
>>>
>>>
>>> located at :
>>> https://builds.apache.org/job/karaf-pr/665/console
>>>
>>> What should I update, to make this work?
>>>
>>> Kind Regards,
>>> Miroslav
>>>
>>>
>>
>>
>>
>
--
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Miroslav Beranič <mi...@mibesis.si>.
Hi François,
I have to be logged in to access
https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
Is registration open or I need invite? I did not see a registration form.
But more important:
How do I compile locally in a way, to have this error before pushing to
Apache's Jenkins - save me embarrassment and you resources on servers.
Regards,
Miroslav
V V tor., 11. sep. 2018 ob 08:46 je oseba Francois Papon <
francois.papon@openobject.fr> napisala:
> Hi,
>
> You are missing the Apache License header in files :
>
> 4 Unknown Licenses
>
> *****************************************************
>
> Files with unapproved licenses:
>
> scheduler/src/main/java/org/apache/karaf/scheduler/SchedulerStorage.java
>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiScheduler.java
>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/QuartzSchedulerStorage.java
>
> scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiSchedulerFactory.java
>
> I put some comments on your PR ;)
>
> You can see the detailed files here :
>
>
> https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
>
> regards,
>
> François Papon
> fpapon@apache.org
>
> Le 11/09/2018 à 10:35, Miroslav Beranič a écrit :
> > Hi,
> >
> > I've subscribed to dev mailing list.
> >
> > I've submitted a pull request at
> https://github.com/apache/karaf/pull/608,
> > but I see Jenkins build failed do to:
> > Caused by: org.apache.rat.mp.RatCheckException: Too many files with
> > unapproved license: 4 See RAT report in:
> >
> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
> >
> >
> > located at :
> > https://builds.apache.org/job/karaf-pr/665/console
> >
> > What should I update, to make this work?
> >
> > Kind Regards,
> > Miroslav
> >
> >
>
>
>
--
Miroslav Beranič
MIBESIS
+386(0)40/814-843
miroslav.beranic@mibesis.si
http://www.mibesis.si
Re: PR608 - Updated Scheduler feature to support persistence - Failed
Posted by Francois Papon <fr...@openobject.fr>.
Hi,
You are missing the Apache License header in files :
4 Unknown Licenses
*****************************************************
Files with unapproved licenses:
scheduler/src/main/java/org/apache/karaf/scheduler/SchedulerStorage.java
scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiScheduler.java
scheduler/src/main/java/org/apache/karaf/scheduler/core/QuartzSchedulerStorage.java
scheduler/src/main/java/org/apache/karaf/scheduler/core/StdOsgiSchedulerFactory.java
I put some comments on your PR ;)
You can see the detailed files here :
https://builds.apache.org/view/K/view/Karaf/job/karaf-pr/ws/target/karaf-4.2.2-SNAPSHOT.rat/*view*/
regards,
François Papon
fpapon@apache.org
Le 11/09/2018 à 10:35, Miroslav Beranič a écrit :
> Hi,
>
> I've subscribed to dev mailing list.
>
> I've submitted a pull request at https://github.com/apache/karaf/pull/608,
> but I see Jenkins build failed do to:
> Caused by: org.apache.rat.mp.RatCheckException: Too many files with
> unapproved license: 4 See RAT report in:
> /home/jenkins/jenkins-slave/workspace/karaf-pr/target/karaf-4.2.2-SNAPSHOT.rat
>
>
> located at :
> https://builds.apache.org/job/karaf-pr/665/console
>
> What should I update, to make this work?
>
> Kind Regards,
> Miroslav
>
>