Re: Uprade to latest Johnzon+OpenJPA+OWB - New TomEE release?

2022-02-09 Thread Jean-Louis Monteiro
I'll see if I can do the release tomorrow.
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com


On Wed, Feb 9, 2022 at 10:33 AM Zowalla, Richard <
richard.zowa...@hs-heilbronn.de> wrote:

> It was a toggle ;) - I restarted the build yesterday night and all is
> green now:
> https://ci-builds.apache.org/job/Tomee/job/master-build-full/476/
>
> Am Mittwoch, dem 09.02.2022 um 10:21 +0100 schrieb Thomas Andraschko:
> > one EAR test fails, everything else looks good
> > how should we proceed?
> >
> > Am Di., 8. Feb. 2022 um 16:56 Uhr schrieb Jean-Louis Monteiro <
> > jlmonte...@tomitribe.com>:
> >
> > > I still have a weird JAX-WS issue I'd like to pin down and fix. But
> > > I can
> > > backport easily on the 8.x maintenance branch
> > >
> > > --
> > > Jean-Louis Monteiro
> > > http://twitter.com/jlouismonteiro
> > > http://www.tomitribe.com
> > >
> > >
> > > On Tue, Feb 8, 2022 at 4:44 PM Wiesner, Martin <
> > > martin.wies...@hs-heilbronn.de> wrote:
> > >
> > > > Hi all,
> > > >
> > > > release of version 8.0.10 would be a fine, as brought up by
> > > > Thomas.
> > > > Maintenance branch for 8.x could be opened once these dep updates
> > > > are
> > > > included.
> > > >
> > > > Is there anything else pending for inclusion into 8.0.10?
> > > >
> > > > Best
> > > > Martin
> > > > —
> > > > https://twitter.com/mawiesne
> > > >
> > > > Am 08.02.2022 um 16:02 schrieb Thomas Andraschko <
> > > > andraschko.tho...@gmail.com>:
> > > >
> > > > great, thanks!
> > > >
> > > > Am Di., 8. Feb. 2022 um 15:56 Uhr schrieb Zowalla, Richard <
> > > > richard.zowa...@hs-heilbronn.de > > > richard.zowa...@hs-heilbronn.de
> > > > > :
> > > >
> > > > Sounds great to have a new release of TomEE soon.
> > > >
> > > > Am Dienstag, dem 08.02.2022 um 15:51 +0100 schrieb Jean-Louis
> > > > Monteiro:
> > > > I have done all the updates this morning already.
> > > >
> > > > The short version of the build is green
> > > > https://ci-builds.apache.org/job/Tomee/job/master-build-quick/228/
> > > > The complete version is still pending
> > > > https://ci-builds.apache.org/job/Tomee/job/master-build-full/475/
> > > >
> > > > After that I don't see why we would not be able to release and
> > > > switch
> > > > after
> > > > that to 9.x
> > > >
> > > > --
> > > > Jean-Louis Monteiro
> > > > http://twitter.com/jlouismonteiro
> > > > http://www.tomitribe.com
> > > >
> > > >
> > > > On Tue, Feb 8, 2022 at 3:44 PM Thomas Andraschko <
> > > > andraschko.tho...@gmail.com> wrote:
> > > >
> > > > Hi,
> > > >
> > > > Romain released new versions of those 3 libs - and i would need
> > > > the
> > > > upgrade
> > > > of Johnzon (polymorphic issues) and OpenJPA (@AttributeConverter
> > > > support).
> > > > Would it be possible to upgrade those and release a new version?
> > > > Would be
> > > > very great.
> > > >
> > > > I can provide a PR for those updates.
> > > >
> > > > Best regards,
> > > > Thomas
> > > >
> > > >
> > > >
> > > >
>


[GitHub] [tomee] rzo1 merged pull request #812: Regenerated BOMs after dependency upgrades

2022-02-09 Thread GitBox


rzo1 merged pull request #812:
URL: https://github.com/apache/tomee/pull/812


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [tomee] github-actions[bot] opened a new pull request #812: Regenerated BOMs after dependency upgrades

2022-02-09 Thread GitBox


github-actions[bot] opened a new pull request #812:
URL: https://github.com/apache/tomee/pull/812


   Found some uncommited changes (BOM regeneration) after running build on 
TomEE master


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [tomee] rzo1 merged pull request #811: Regenerated BOMs

2022-02-09 Thread GitBox


rzo1 merged pull request #811:
URL: https://github.com/apache/tomee/pull/811


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




Re: Generated BOM files

2022-02-09 Thread Zowalla, Richard
Thanks for the pointer, David.

I just had a look and pushed an additional step to our config, which is
based on the setup provided by Camel.

Seems to work nicely: https://github.com/apache/tomee/pull/811

Gruß
Richard



Am Mittwoch, dem 09.02.2022 um 07:47 -0800 schrieb David Jencks:
> While GitHub actions shouldn’t be able to commit anything, they can
> create a PR that can then be reviewed and merged by a committer.
> Apache camel uses this to good effect: they have far more complicated
> things than BOMs generated.
> 
> David Jencks 
> 
> Sent from my iPhone
> 
> > On Feb 9, 2022, at 6:40 AM, Zowalla, Richard <
> > richard.zowa...@hs-heilbronn.de> wrote:
> > 
> > I think, that the inconvience originates from the fact, that one
> > has to
> > conduct a quick build w/o tests after changing a simple version
> > string
> > - even if one does not expect any compile issues and just want to
> > see
> > the impact in a CI/CD environment
> > 
> > The current "generate in the build" is tied to "package".
> > 
> > 
> > > Am Mittwoch, dem 09.02.2022 um 15:33 +0100 schrieb Jean-Louis
> > > Monteiro:
> > > Thanks David for the extra infra information. That makes the
> > > automation a
> > > bit harder indeed.
> > > 
> > > Let's keep the "generate in the build" as it is today and see if
> > > we
> > > can
> > > remember to push and when I say we, I mean "I" 
> > > 
> > > --
> > > Jean-Louis Monteiro
> > > http://twitter.com/jlouismonteiro
> > > http://www.tomitribe.com
> > > 
> > > 
> > > On Wed, Feb 9, 2022 at 3:15 PM David Blevins <
> > > dblev...@tomitribe.com>
> > > wrote:
> > > 
> > > > The trick is that Apache doesn’t have any bot accounts that we
> > > > could use to
> > > > do commits to master.  So there really isn’t any way to use
> > > > Jenkins, GitHub
> > > > Actions, etc.
> > > > 
> > > > The limited functionality Apache does have for committing files
> > > > is
> > > > for
> > > > website generation, but it is setup to only work for branches
> > > > called
> > > > “asf-site” (or something like that) and works from only one
> > > > specific
> > > > Jenkins node.
> > > > 
> > > > The best we could do is create a bot that made a PR one of us
> > > > had
> > > > to merge
> > > > and/or get this to be done in the build and we ensure we
> > > > remember
> > > > to commit
> > > > it. (We could potentially do both so there is a convenient
> > > > backup
> > > > if we
> > > > forget to do the commit ourselves before we push).
> > > > 
> > > > -David
> > > > 
> > > > On Wed, Feb 9, 2022 at 8:51 AM Jean-Louis Monteiro <
> > > > jlmonte...@tomitribe.com>
> > > > wrote:
> > > > 
> > > > > Hi all,
> > > > > 
> > > > > We have discussed many times with Richard on Slack mainly
> > > > > around
> > > > > this
> > > > > topic. But I wanted to discuss it over here and have some
> > > > > brainstorming.
> > > > > 
> > > > > We have had BOM files for quite a while. To avoid the pain to
> > > > > update and
> > > > > maintain them, David created a script to generate them. All
> > > > > good.
> > > > > 
> > > > > The problem comes when one is updating or adding or removing
> > > > > a
> > > > dependency.
> > > > > And I must apologize because it happened to me pretty much
> > > > > every
> > > > > single
> > > > > time. Richard has been looking after the build and fixing my
> > > > > garbage by
> > > > > generating again the BOM files to commit them. Thanks for
> > > > > that.
> > > > > 
> > > > > We discussed an approach to generate them in the build so
> > > > > Jenkins
> > > > > is
> > > > always
> > > > > happy. It works but it has bigger side effects in my opinion.
> > > > > 
> > > > > 1/ Jenkins does not commit and then it does not fix my
> > > > > garbage
> > > > > 2/ the snapshots the user uses don't reflect what the CI is
> > > > > testing which
> > > > > deserves the purpose.
> > > > > 3/ the mess is hidden and when cutting the release there is a
> > > > > risk for
> > > > the
> > > > > BOM files to not be up to date
> > > > > 
> > > > > I think we should revert this and at least let the build to
> > > > > fail
> > > > > so we
> > > > can
> > > > > fix it and maintain the BOM files.
> > > > > 
> > > > > I have also investigated Github actions. We could also create
> > > > > a
> > > > > couple of
> > > > > Github actions
> > > > > 
> > > > > - to generate the BOM files AND commit them to git if they
> > > > > changed. So
> > > > they
> > > > > are always up to date and the CI system runs on what the user
> > > > > is
> > > > > using
> > > > > 
> > > > > - check file headers to make sure they have ASLv2 header.
> > > > > This is
> > > > > a
> > > > common
> > > > > error and CI will fail with RAT/checkstyle/PMD in the sanity
> > > > > checks build
> > > > > 
> > > > > - do some updates on the website if needed
> > > > > 
> > > > > We could start with the BOM and look at the headers. They
> > > > > should
> > > > > be
> > > > fairly
> > > > > easy to handle and bring some immediate value.
> > > > > 
> > > > > 

[GitHub] [tomee] github-actions[bot] opened a new pull request #811: Regenerated BOMs

2022-02-09 Thread GitBox


github-actions[bot] opened a new pull request #811:
URL: https://github.com/apache/tomee/pull/811


   Found some uncommited changes after running build on master


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




Re: Generated BOM files

2022-02-09 Thread David Jencks
While GitHub actions shouldn’t be able to commit anything, they can create a PR 
that can then be reviewed and merged by a committer. Apache camel uses this to 
good effect: they have far more complicated things than BOMs generated.

David Jencks 

Sent from my iPhone

> On Feb 9, 2022, at 6:40 AM, Zowalla, Richard 
>  wrote:
> 
> I think, that the inconvience originates from the fact, that one has to
> conduct a quick build w/o tests after changing a simple version string
> - even if one does not expect any compile issues and just want to see
> the impact in a CI/CD environment
> 
> The current "generate in the build" is tied to "package".
> 
> 
>> Am Mittwoch, dem 09.02.2022 um 15:33 +0100 schrieb Jean-Louis Monteiro:
>> Thanks David for the extra infra information. That makes the
>> automation a
>> bit harder indeed.
>> 
>> Let's keep the "generate in the build" as it is today and see if we
>> can
>> remember to push and when I say we, I mean "I" 
>> 
>> --
>> Jean-Louis Monteiro
>> http://twitter.com/jlouismonteiro
>> http://www.tomitribe.com
>> 
>> 
>> On Wed, Feb 9, 2022 at 3:15 PM David Blevins 
>> wrote:
>> 
>>> The trick is that Apache doesn’t have any bot accounts that we
>>> could use to
>>> do commits to master.  So there really isn’t any way to use
>>> Jenkins, GitHub
>>> Actions, etc.
>>> 
>>> The limited functionality Apache does have for committing files is
>>> for
>>> website generation, but it is setup to only work for branches
>>> called
>>> “asf-site” (or something like that) and works from only one
>>> specific
>>> Jenkins node.
>>> 
>>> The best we could do is create a bot that made a PR one of us had
>>> to merge
>>> and/or get this to be done in the build and we ensure we remember
>>> to commit
>>> it. (We could potentially do both so there is a convenient backup
>>> if we
>>> forget to do the commit ourselves before we push).
>>> 
>>> -David
>>> 
>>> On Wed, Feb 9, 2022 at 8:51 AM Jean-Louis Monteiro <
>>> jlmonte...@tomitribe.com>
>>> wrote:
>>> 
 Hi all,
 
 We have discussed many times with Richard on Slack mainly around
 this
 topic. But I wanted to discuss it over here and have some
 brainstorming.
 
 We have had BOM files for quite a while. To avoid the pain to
 update and
 maintain them, David created a script to generate them. All good.
 
 The problem comes when one is updating or adding or removing a
>>> dependency.
 And I must apologize because it happened to me pretty much every
 single
 time. Richard has been looking after the build and fixing my
 garbage by
 generating again the BOM files to commit them. Thanks for that.
 
 We discussed an approach to generate them in the build so Jenkins
 is
>>> always
 happy. It works but it has bigger side effects in my opinion.
 
 1/ Jenkins does not commit and then it does not fix my garbage
 2/ the snapshots the user uses don't reflect what the CI is
 testing which
 deserves the purpose.
 3/ the mess is hidden and when cutting the release there is a
 risk for
>>> the
 BOM files to not be up to date
 
 I think we should revert this and at least let the build to fail
 so we
>>> can
 fix it and maintain the BOM files.
 
 I have also investigated Github actions. We could also create a
 couple of
 Github actions
 
 - to generate the BOM files AND commit them to git if they
 changed. So
>>> they
 are always up to date and the CI system runs on what the user is
 using
 
 - check file headers to make sure they have ASLv2 header. This is
 a
>>> common
 error and CI will fail with RAT/checkstyle/PMD in the sanity
 checks build
 
 - do some updates on the website if needed
 
 We could start with the BOM and look at the headers. They should
 be
>>> fairly
 easy to handle and bring some immediate value.
 
 What do you think?
 
 --
 Jean-Louis Monteiro
 http://twitter.com/jlouismonteiro
 http://www.tomitribe.com
 
>>> --
>>> Sent from Gmail Mobile
>>> 


Re: Generated BOM files

2022-02-09 Thread Zowalla, Richard
I think, that the inconvience originates from the fact, that one has to
conduct a quick build w/o tests after changing a simple version string
- even if one does not expect any compile issues and just want to see
the impact in a CI/CD environment

The current "generate in the build" is tied to "package".


Am Mittwoch, dem 09.02.2022 um 15:33 +0100 schrieb Jean-Louis Monteiro:
> Thanks David for the extra infra information. That makes the
> automation a
> bit harder indeed.
> 
> Let's keep the "generate in the build" as it is today and see if we
> can
> remember to push and when I say we, I mean "I" 
> 
> --
> Jean-Louis Monteiro
> http://twitter.com/jlouismonteiro
> http://www.tomitribe.com
> 
> 
> On Wed, Feb 9, 2022 at 3:15 PM David Blevins 
> wrote:
> 
> > The trick is that Apache doesn’t have any bot accounts that we
> > could use to
> > do commits to master.  So there really isn’t any way to use
> > Jenkins, GitHub
> > Actions, etc.
> > 
> > The limited functionality Apache does have for committing files is
> > for
> > website generation, but it is setup to only work for branches
> > called
> > “asf-site” (or something like that) and works from only one
> > specific
> > Jenkins node.
> > 
> > The best we could do is create a bot that made a PR one of us had
> > to merge
> > and/or get this to be done in the build and we ensure we remember
> > to commit
> > it. (We could potentially do both so there is a convenient backup
> > if we
> > forget to do the commit ourselves before we push).
> > 
> > -David
> > 
> > On Wed, Feb 9, 2022 at 8:51 AM Jean-Louis Monteiro <
> > jlmonte...@tomitribe.com>
> > wrote:
> > 
> > > Hi all,
> > > 
> > > We have discussed many times with Richard on Slack mainly around
> > > this
> > > topic. But I wanted to discuss it over here and have some
> > > brainstorming.
> > > 
> > > We have had BOM files for quite a while. To avoid the pain to
> > > update and
> > > maintain them, David created a script to generate them. All good.
> > > 
> > > The problem comes when one is updating or adding or removing a
> > dependency.
> > > And I must apologize because it happened to me pretty much every
> > > single
> > > time. Richard has been looking after the build and fixing my
> > > garbage by
> > > generating again the BOM files to commit them. Thanks for that.
> > > 
> > > We discussed an approach to generate them in the build so Jenkins
> > > is
> > always
> > > happy. It works but it has bigger side effects in my opinion.
> > > 
> > > 1/ Jenkins does not commit and then it does not fix my garbage
> > > 2/ the snapshots the user uses don't reflect what the CI is
> > > testing which
> > > deserves the purpose.
> > > 3/ the mess is hidden and when cutting the release there is a
> > > risk for
> > the
> > > BOM files to not be up to date
> > > 
> > > I think we should revert this and at least let the build to fail
> > > so we
> > can
> > > fix it and maintain the BOM files.
> > > 
> > > I have also investigated Github actions. We could also create a
> > > couple of
> > > Github actions
> > > 
> > > - to generate the BOM files AND commit them to git if they
> > > changed. So
> > they
> > > are always up to date and the CI system runs on what the user is
> > > using
> > > 
> > > - check file headers to make sure they have ASLv2 header. This is
> > > a
> > common
> > > error and CI will fail with RAT/checkstyle/PMD in the sanity
> > > checks build
> > > 
> > > - do some updates on the website if needed
> > > 
> > > We could start with the BOM and look at the headers. They should
> > > be
> > fairly
> > > easy to handle and bring some immediate value.
> > > 
> > > What do you think?
> > > 
> > > --
> > > Jean-Louis Monteiro
> > > http://twitter.com/jlouismonteiro
> > > http://www.tomitribe.com
> > > 
> > --
> > Sent from Gmail Mobile
> > 


smime.p7s
Description: S/MIME cryptographic signature


Re: Generated BOM files

2022-02-09 Thread Zowalla, Richard
Oh... i did miss the asf-site thing, David mentioned. 
So my proposal is not useful ;)

Am Mittwoch, dem 09.02.2022 um 14:32 + schrieb Zowalla, Richard:
> Hi,
> 
> Our GitHub Actions already do the Style/RAT checks.
> 
> ASF Infra also have some concerns regarding GitHub actions [1] and
> discussed it on users@infra.a.o multiple times. 
> 
> However, Jenkins is capable of committing changes (we do it for the
> generated tomee site) - so perhaps it could also be done for the main
> repo?
> 
> We could set up an additional job, which is triggered after a commit,
> which
> 
> (1) checks out the project
> (2) do a quick build w/o tests
> (3) generate the boms
> (4) commit and push if there is a change (we do sth similar with the
> tomee site every 12h)
> 
> Then we setup the subsequent jobs (quick, full) to trigger only after
> a
> successful "bom generation job".
> 
> Wouldn't that work?
> 
> Gruß
> Richard
> 
> [1] 
> https://cwiki.apache.org/confluence/display/BUILDS/GitHub+Actions+status
> 
> Am Mittwoch, dem 09.02.2022 um 09:15 -0500 schrieb David Blevins:
> > The trick is that Apache doesn’t have any bot accounts that we
> > could
> > use to
> > do commits to master.  So there really isn’t any way to use
> > Jenkins,
> > GitHub
> > Actions, etc.
> > 
> > The limited functionality Apache does have for committing files is
> > for
> > website generation, but it is setup to only work for branches
> > called
> > “asf-site” (or something like that) and works from only one
> > specific
> > Jenkins node.
> > 
> > The best we could do is create a bot that made a PR one of us had
> > to
> > merge
> > and/or get this to be done in the build and we ensure we remember
> > to
> > commit
> > it. (We could potentially do both so there is a convenient backup
> > if
> > we
> > forget to do the commit ourselves before we push).
> > 
> > -David
> > 
> > On Wed, Feb 9, 2022 at 8:51 AM Jean-Louis Monteiro <
> > jlmonte...@tomitribe.com>
> > wrote:
> > 
> > > Hi all,
> > > 
> > > We have discussed many times with Richard on Slack mainly around
> > > this
> > > topic. But I wanted to discuss it over here and have some
> > > brainstorming.
> > > 
> > > We have had BOM files for quite a while. To avoid the pain to
> > > update and
> > > maintain them, David created a script to generate them. All good.
> > > 
> > > The problem comes when one is updating or adding or removing a
> > > dependency.
> > > And I must apologize because it happened to me pretty much every
> > > single
> > > time. Richard has been looking after the build and fixing my
> > > garbage by
> > > generating again the BOM files to commit them. Thanks for that.
> > > 
> > > We discussed an approach to generate them in the build so Jenkins
> > > is always
> > > happy. It works but it has bigger side effects in my opinion.
> > > 
> > > 1/ Jenkins does not commit and then it does not fix my garbage
> > > 2/ the snapshots the user uses don't reflect what the CI is
> > > testing
> > > which
> > > deserves the purpose.
> > > 3/ the mess is hidden and when cutting the release there is a
> > > risk
> > > for the
> > > BOM files to not be up to date
> > > 
> > > I think we should revert this and at least let the build to fail
> > > so
> > > we can
> > > fix it and maintain the BOM files.
> > > 
> > > I have also investigated Github actions. We could also create a
> > > couple of
> > > Github actions
> > > 
> > > - to generate the BOM files AND commit them to git if they
> > > changed.
> > > So they
> > > are always up to date and the CI system runs on what the user is
> > > using
> > > 
> > > - check file headers to make sure they have ASLv2 header. This is
> > > a
> > > common
> > > error and CI will fail with RAT/checkstyle/PMD in the sanity
> > > checks
> > > build
> > > 
> > > - do some updates on the website if needed
> > > 
> > > We could start with the BOM and look at the headers. They should
> > > be
> > > fairly
> > > easy to handle and bring some immediate value.
> > > 
> > > What do you think?
> > > 
> > > --
> > > Jean-Louis Monteiro
> > > http://twitter.com/jlouismonteiro
> > > http://www.tomitribe.com
> > > 
-- 
Richard Zowalla, M.Sc.
Research Associate, PhD Student | Medical Informatics

Hochschule Heilbronn – University of Applied Sciences
Max-Planck-Str. 39 
D-74081 Heilbronn 
phone: +49 7131 504 6791 (zur Zeit nicht via Telefon erreichbar)
mail: richard.zowa...@hs-heilbronn.de
web: https://www.mi.hs-heilbronn.de/ 


smime.p7s
Description: S/MIME cryptographic signature


Re: Generated BOM files

2022-02-09 Thread Zowalla, Richard
Hi,

Our GitHub Actions already do the Style/RAT checks.

ASF Infra also have some concerns regarding GitHub actions [1] and
discussed it on users@infra.a.o multiple times. 

However, Jenkins is capable of committing changes (we do it for the
generated tomee site) - so perhaps it could also be done for the main
repo?

We could set up an additional job, which is triggered after a commit,
which

(1) checks out the project
(2) do a quick build w/o tests
(3) generate the boms
(4) commit and push if there is a change (we do sth similar with the
tomee site every 12h)

Then we setup the subsequent jobs (quick, full) to trigger only after a
successful "bom generation job".

Wouldn't that work?

Gruß
Richard

[1] 
https://cwiki.apache.org/confluence/display/BUILDS/GitHub+Actions+status

Am Mittwoch, dem 09.02.2022 um 09:15 -0500 schrieb David Blevins:
> The trick is that Apache doesn’t have any bot accounts that we could
> use to
> do commits to master.  So there really isn’t any way to use Jenkins,
> GitHub
> Actions, etc.
> 
> The limited functionality Apache does have for committing files is
> for
> website generation, but it is setup to only work for branches called
> “asf-site” (or something like that) and works from only one specific
> Jenkins node.
> 
> The best we could do is create a bot that made a PR one of us had to
> merge
> and/or get this to be done in the build and we ensure we remember to
> commit
> it. (We could potentially do both so there is a convenient backup if
> we
> forget to do the commit ourselves before we push).
> 
> -David
> 
> On Wed, Feb 9, 2022 at 8:51 AM Jean-Louis Monteiro <
> jlmonte...@tomitribe.com>
> wrote:
> 
> > Hi all,
> > 
> > We have discussed many times with Richard on Slack mainly around
> > this
> > topic. But I wanted to discuss it over here and have some
> > brainstorming.
> > 
> > We have had BOM files for quite a while. To avoid the pain to
> > update and
> > maintain them, David created a script to generate them. All good.
> > 
> > The problem comes when one is updating or adding or removing a
> > dependency.
> > And I must apologize because it happened to me pretty much every
> > single
> > time. Richard has been looking after the build and fixing my
> > garbage by
> > generating again the BOM files to commit them. Thanks for that.
> > 
> > We discussed an approach to generate them in the build so Jenkins
> > is always
> > happy. It works but it has bigger side effects in my opinion.
> > 
> > 1/ Jenkins does not commit and then it does not fix my garbage
> > 2/ the snapshots the user uses don't reflect what the CI is testing
> > which
> > deserves the purpose.
> > 3/ the mess is hidden and when cutting the release there is a risk
> > for the
> > BOM files to not be up to date
> > 
> > I think we should revert this and at least let the build to fail so
> > we can
> > fix it and maintain the BOM files.
> > 
> > I have also investigated Github actions. We could also create a
> > couple of
> > Github actions
> > 
> > - to generate the BOM files AND commit them to git if they changed.
> > So they
> > are always up to date and the CI system runs on what the user is
> > using
> > 
> > - check file headers to make sure they have ASLv2 header. This is a
> > common
> > error and CI will fail with RAT/checkstyle/PMD in the sanity checks
> > build
> > 
> > - do some updates on the website if needed
> > 
> > We could start with the BOM and look at the headers. They should be
> > fairly
> > easy to handle and bring some immediate value.
> > 
> > What do you think?
> > 
> > --
> > Jean-Louis Monteiro
> > http://twitter.com/jlouismonteiro
> > http://www.tomitribe.com
> > 


smime.p7s
Description: S/MIME cryptographic signature


Re: Generated BOM files

2022-02-09 Thread David Blevins
The trick is that Apache doesn’t have any bot accounts that we could use to
do commits to master.  So there really isn’t any way to use Jenkins, GitHub
Actions, etc.

The limited functionality Apache does have for committing files is for
website generation, but it is setup to only work for branches called
“asf-site” (or something like that) and works from only one specific
Jenkins node.

The best we could do is create a bot that made a PR one of us had to merge
and/or get this to be done in the build and we ensure we remember to commit
it. (We could potentially do both so there is a convenient backup if we
forget to do the commit ourselves before we push).

-David

On Wed, Feb 9, 2022 at 8:51 AM Jean-Louis Monteiro 
wrote:

> Hi all,
>
> We have discussed many times with Richard on Slack mainly around this
> topic. But I wanted to discuss it over here and have some brainstorming.
>
> We have had BOM files for quite a while. To avoid the pain to update and
> maintain them, David created a script to generate them. All good.
>
> The problem comes when one is updating or adding or removing a dependency.
> And I must apologize because it happened to me pretty much every single
> time. Richard has been looking after the build and fixing my garbage by
> generating again the BOM files to commit them. Thanks for that.
>
> We discussed an approach to generate them in the build so Jenkins is always
> happy. It works but it has bigger side effects in my opinion.
>
> 1/ Jenkins does not commit and then it does not fix my garbage
> 2/ the snapshots the user uses don't reflect what the CI is testing which
> deserves the purpose.
> 3/ the mess is hidden and when cutting the release there is a risk for the
> BOM files to not be up to date
>
> I think we should revert this and at least let the build to fail so we can
> fix it and maintain the BOM files.
>
> I have also investigated Github actions. We could also create a couple of
> Github actions
>
> - to generate the BOM files AND commit them to git if they changed. So they
> are always up to date and the CI system runs on what the user is using
>
> - check file headers to make sure they have ASLv2 header. This is a common
> error and CI will fail with RAT/checkstyle/PMD in the sanity checks build
>
> - do some updates on the website if needed
>
> We could start with the BOM and look at the headers. They should be fairly
> easy to handle and bring some immediate value.
>
> What do you think?
>
> --
> Jean-Louis Monteiro
> http://twitter.com/jlouismonteiro
> http://www.tomitribe.com
>
-- 
Sent from Gmail Mobile


Generated BOM files

2022-02-09 Thread Jean-Louis Monteiro
Hi all,

We have discussed many times with Richard on Slack mainly around this
topic. But I wanted to discuss it over here and have some brainstorming.

We have had BOM files for quite a while. To avoid the pain to update and
maintain them, David created a script to generate them. All good.

The problem comes when one is updating or adding or removing a dependency.
And I must apologize because it happened to me pretty much every single
time. Richard has been looking after the build and fixing my garbage by
generating again the BOM files to commit them. Thanks for that.

We discussed an approach to generate them in the build so Jenkins is always
happy. It works but it has bigger side effects in my opinion.

1/ Jenkins does not commit and then it does not fix my garbage
2/ the snapshots the user uses don't reflect what the CI is testing which
deserves the purpose.
3/ the mess is hidden and when cutting the release there is a risk for the
BOM files to not be up to date

I think we should revert this and at least let the build to fail so we can
fix it and maintain the BOM files.

I have also investigated Github actions. We could also create a couple of
Github actions

- to generate the BOM files AND commit them to git if they changed. So they
are always up to date and the CI system runs on what the user is using

- check file headers to make sure they have ASLv2 header. This is a common
error and CI will fail with RAT/checkstyle/PMD in the sanity checks build

- do some updates on the website if needed

We could start with the BOM and look at the headers. They should be fairly
easy to handle and bring some immediate value.

What do you think?

--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com


[GitHub] [tomee-site-generator] jgallimore merged pull request #32: Add version placeholder

2022-02-09 Thread GitBox


jgallimore merged pull request #32:
URL: https://github.com/apache/tomee-site-generator/pull/32


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




Re: Uprade to latest Johnzon+OpenJPA+OWB - New TomEE release?

2022-02-09 Thread Zowalla, Richard
It was a toggle ;) - I restarted the build yesterday night and all is
green now: 
https://ci-builds.apache.org/job/Tomee/job/master-build-full/476/

Am Mittwoch, dem 09.02.2022 um 10:21 +0100 schrieb Thomas Andraschko:
> one EAR test fails, everything else looks good
> how should we proceed?
> 
> Am Di., 8. Feb. 2022 um 16:56 Uhr schrieb Jean-Louis Monteiro <
> jlmonte...@tomitribe.com>:
> 
> > I still have a weird JAX-WS issue I'd like to pin down and fix. But
> > I can
> > backport easily on the 8.x maintenance branch
> > 
> > --
> > Jean-Louis Monteiro
> > http://twitter.com/jlouismonteiro
> > http://www.tomitribe.com
> > 
> > 
> > On Tue, Feb 8, 2022 at 4:44 PM Wiesner, Martin <
> > martin.wies...@hs-heilbronn.de> wrote:
> > 
> > > Hi all,
> > > 
> > > release of version 8.0.10 would be a fine, as brought up by
> > > Thomas.
> > > Maintenance branch for 8.x could be opened once these dep updates
> > > are
> > > included.
> > > 
> > > Is there anything else pending for inclusion into 8.0.10?
> > > 
> > > Best
> > > Martin
> > > —
> > > https://twitter.com/mawiesne
> > > 
> > > Am 08.02.2022 um 16:02 schrieb Thomas Andraschko <
> > > andraschko.tho...@gmail.com>:
> > > 
> > > great, thanks!
> > > 
> > > Am Di., 8. Feb. 2022 um 15:56 Uhr schrieb Zowalla, Richard <
> > > richard.zowa...@hs-heilbronn.de > > richard.zowa...@hs-heilbronn.de
> > > > :
> > > 
> > > Sounds great to have a new release of TomEE soon.
> > > 
> > > Am Dienstag, dem 08.02.2022 um 15:51 +0100 schrieb Jean-Louis
> > > Monteiro:
> > > I have done all the updates this morning already.
> > > 
> > > The short version of the build is green
> > > https://ci-builds.apache.org/job/Tomee/job/master-build-quick/228/
> > > The complete version is still pending
> > > https://ci-builds.apache.org/job/Tomee/job/master-build-full/475/
> > > 
> > > After that I don't see why we would not be able to release and
> > > switch
> > > after
> > > that to 9.x
> > > 
> > > --
> > > Jean-Louis Monteiro
> > > http://twitter.com/jlouismonteiro
> > > http://www.tomitribe.com
> > > 
> > > 
> > > On Tue, Feb 8, 2022 at 3:44 PM Thomas Andraschko <
> > > andraschko.tho...@gmail.com> wrote:
> > > 
> > > Hi,
> > > 
> > > Romain released new versions of those 3 libs - and i would need
> > > the
> > > upgrade
> > > of Johnzon (polymorphic issues) and OpenJPA (@AttributeConverter
> > > support).
> > > Would it be possible to upgrade those and release a new version?
> > > Would be
> > > very great.
> > > 
> > > I can provide a PR for those updates.
> > > 
> > > Best regards,
> > > Thomas
> > > 
> > > 
> > > 
> > > 


smime.p7s
Description: S/MIME cryptographic signature


Re: Uprade to latest Johnzon+OpenJPA+OWB - New TomEE release?

2022-02-09 Thread Thomas Andraschko
one EAR test fails, everything else looks good
how should we proceed?

Am Di., 8. Feb. 2022 um 16:56 Uhr schrieb Jean-Louis Monteiro <
jlmonte...@tomitribe.com>:

> I still have a weird JAX-WS issue I'd like to pin down and fix. But I can
> backport easily on the 8.x maintenance branch
>
> --
> Jean-Louis Monteiro
> http://twitter.com/jlouismonteiro
> http://www.tomitribe.com
>
>
> On Tue, Feb 8, 2022 at 4:44 PM Wiesner, Martin <
> martin.wies...@hs-heilbronn.de> wrote:
>
> > Hi all,
> >
> > release of version 8.0.10 would be a fine, as brought up by Thomas.
> > Maintenance branch for 8.x could be opened once these dep updates are
> > included.
> >
> > Is there anything else pending for inclusion into 8.0.10?
> >
> > Best
> > Martin
> > —
> > https://twitter.com/mawiesne
> >
> > Am 08.02.2022 um 16:02 schrieb Thomas Andraschko <
> > andraschko.tho...@gmail.com>:
> >
> > great, thanks!
> >
> > Am Di., 8. Feb. 2022 um 15:56 Uhr schrieb Zowalla, Richard <
> > richard.zowa...@hs-heilbronn.de >>:
> >
> > Sounds great to have a new release of TomEE soon.
> >
> > Am Dienstag, dem 08.02.2022 um 15:51 +0100 schrieb Jean-Louis Monteiro:
> > I have done all the updates this morning already.
> >
> > The short version of the build is green
> > https://ci-builds.apache.org/job/Tomee/job/master-build-quick/228/
> > The complete version is still pending
> > https://ci-builds.apache.org/job/Tomee/job/master-build-full/475/
> >
> > After that I don't see why we would not be able to release and switch
> > after
> > that to 9.x
> >
> > --
> > Jean-Louis Monteiro
> > http://twitter.com/jlouismonteiro
> > http://www.tomitribe.com
> >
> >
> > On Tue, Feb 8, 2022 at 3:44 PM Thomas Andraschko <
> > andraschko.tho...@gmail.com> wrote:
> >
> > Hi,
> >
> > Romain released new versions of those 3 libs - and i would need the
> > upgrade
> > of Johnzon (polymorphic issues) and OpenJPA (@AttributeConverter
> > support).
> > Would it be possible to upgrade those and release a new version?
> > Would be
> > very great.
> >
> > I can provide a PR for those updates.
> >
> > Best regards,
> > Thomas
> >
> >
> >
> >
>


Re: Version placeholder on the website

2022-02-09 Thread Jonathan Gallimore
Thanks for the feedback and +1s. I was just thinking that there may be some
legitimate uses of ${tomee.version} that we don't want to replace - e.g.
where we're showing a pom.xml file, and tomee.version is actually defined
as a property. I'm thinking I'll simply switch this to ${TOMEE_VERSION},
merge this in, and set about updating some of the documentation to
reference it.

Let me know if you have any objections to that.

Thanks

Jon

On Tue, Feb 8, 2022 at 2:31 PM Jonathan Gallimore <
jonathan.gallim...@gmail.com> wrote:

> Hi All,
>
> I've created a PR for the site generator project:
> https://github.com/apache/tomee-site-generator/pull/32.
>
> The idea here is to allow ${tomee.version} to be specified in the
> documentation or example files, and it to be swapped for the specific
> version string (which can be specified in the Configuration class). The
> rationale here for pages to show an up-to-date version, rather than
> referencing really old versions.
>
> If folks are ok with this. I'll merge it in, and start working through the
> relevant files to add in the ${tomee.version} placeholder in the right
> places.
>
> Thanks
>
> Jon
>