You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by Zoran Regvart <zo...@regvart.com> on 2018/10/30 15:19:12 UTC
Spring and Spring Boot upgrades
Hi Cameleers,
I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
any comments welcome:
https://github.com/apache/camel/pull/2584
zoran
--
Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by Andrea Cosentino <an...@yahoo.com.INVALID>.
spring-cloud is actually at 2.0.2.RELEASE where possible.
--
Andrea Cosentino
----------------------------------
Apache Camel PMC Chair
Apache Karaf Committer
Apache Servicemix PMC Member
Email: ancosen1985@yahoo.com
Twitter: @oscerd2
Github: oscerd
On Friday, November 2, 2018, 11:52:30 AM GMT+1, Luca Burgazzoli <lb...@gmail.com> wrote:
Is the spring-cloud dep up to date ? there may be incompatibility
---
Luca Burgazzoli
On Fri, Nov 2, 2018 at 11:31 AM Zoran Regvart <zo...@regvart.com> wrote:
>
> Hi Claus,
> both are defined in `spring-cloud-consul`[1][2], looks like they
> should both have @ConditionalOnMissingBean. What puzzles me is that it
> worked before the upgrade, perhaps the order of beans changed?
>
> I think this needs to be fixed in `spring-cloud-consul`, though it
> could point to a incompatibility between Spring Boot 2.0 and 2.1 --
> that's where auto-configuration is.
>
> zoran
>
> [1] https://github.com/spring-cloud/spring-cloud-consul/blob/e8d5b575861ff39d4ea8a7fdf0eeaff6ea8dd3b8/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulServiceRegistryAutoConfiguration.java#L60-L64
> [2] https://github.com/spring-cloud/spring-cloud-consul/blob/e8d5b575861ff39d4ea8a7fdf0eeaff6ea8dd3b8/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulDiscoveryClientConfiguration.java#L61-L65
>
> On Fri, Nov 2, 2018 at 11:26 AM, Claus Ibsen <cl...@gmail.com> wrote:
> > Hi
> >
> > This seems like a bit weird error.
> >
> > Is that heartbeatProperties something that is inherited from
> > spring-boot itself or where does it come from?
> >
> > And would a fix be to add @ConditionalOnMissingBean on both of them?
> > On Wed, Oct 31, 2018 at 3:54 PM Zoran Regvart <zo...@regvart.com> wrote:
> >>
> >> Hi Cameleers,
> >> there are test failures in the `camel-spring-cloud-consul`[1], to me
> >> it seems like an issue with Spring Boot auto-configuration:
> >> `ConsulDiscoveryClientConfiguration::heartbeatProperties` and
> >> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` try to
> >> define the same bean, where only
> >> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` is
> >> @ConditionalOnMissingBean.
> >>
> >> Not sure what changed or how we can address this in Camel.
> >>
> >> zoran
> >>
> >> [1] https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/2/testReport/
> >>
> >> On Tue, Oct 30, 2018 at 10:23 PM, Zoran Regvart <zo...@regvart.com> wrote:
> >> > Hi Cameleers,
> >> > looks like the build succeeded, but is hanging for some reason (could
> >> > be infrastructure related, not sure):
> >> >
> >> > https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/1/console
> >> >
> >> > zoran
> >> >
> >> > On Tue, Oct 30, 2018 at 4:19 PM, Zoran Regvart <zo...@regvart.com> wrote:
> >> >> Hi Cameleers,
> >> >> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> >> >> any comments welcome:
> >> >>
> >> >> https://github.com/apache/camel/pull/2584
> >> >>
> >> >> zoran
> >> >> --
> >> >> Zoran Regvart
> >> >
> >> >
> >> >
> >> > --
> >> > Zoran Regvart
> >>
> >>
> >>
> >> --
> >> Zoran Regvart
> >
> >
> >
> > --
> > Claus Ibsen
> > -----------------
> > http://davsclaus.com @davsclaus
> > Camel in Action 2: https://www.manning.com/ibsen2
>
>
>
> --
> Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by Luca Burgazzoli <lb...@gmail.com>.
Is the spring-cloud dep up to date ? there may be incompatibility
---
Luca Burgazzoli
On Fri, Nov 2, 2018 at 11:31 AM Zoran Regvart <zo...@regvart.com> wrote:
>
> Hi Claus,
> both are defined in `spring-cloud-consul`[1][2], looks like they
> should both have @ConditionalOnMissingBean. What puzzles me is that it
> worked before the upgrade, perhaps the order of beans changed?
>
> I think this needs to be fixed in `spring-cloud-consul`, though it
> could point to a incompatibility between Spring Boot 2.0 and 2.1 --
> that's where auto-configuration is.
>
> zoran
>
> [1] https://github.com/spring-cloud/spring-cloud-consul/blob/e8d5b575861ff39d4ea8a7fdf0eeaff6ea8dd3b8/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulServiceRegistryAutoConfiguration.java#L60-L64
> [2] https://github.com/spring-cloud/spring-cloud-consul/blob/e8d5b575861ff39d4ea8a7fdf0eeaff6ea8dd3b8/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulDiscoveryClientConfiguration.java#L61-L65
>
> On Fri, Nov 2, 2018 at 11:26 AM, Claus Ibsen <cl...@gmail.com> wrote:
> > Hi
> >
> > This seems like a bit weird error.
> >
> > Is that heartbeatProperties something that is inherited from
> > spring-boot itself or where does it come from?
> >
> > And would a fix be to add @ConditionalOnMissingBean on both of them?
> > On Wed, Oct 31, 2018 at 3:54 PM Zoran Regvart <zo...@regvart.com> wrote:
> >>
> >> Hi Cameleers,
> >> there are test failures in the `camel-spring-cloud-consul`[1], to me
> >> it seems like an issue with Spring Boot auto-configuration:
> >> `ConsulDiscoveryClientConfiguration::heartbeatProperties` and
> >> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` try to
> >> define the same bean, where only
> >> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` is
> >> @ConditionalOnMissingBean.
> >>
> >> Not sure what changed or how we can address this in Camel.
> >>
> >> zoran
> >>
> >> [1] https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/2/testReport/
> >>
> >> On Tue, Oct 30, 2018 at 10:23 PM, Zoran Regvart <zo...@regvart.com> wrote:
> >> > Hi Cameleers,
> >> > looks like the build succeeded, but is hanging for some reason (could
> >> > be infrastructure related, not sure):
> >> >
> >> > https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/1/console
> >> >
> >> > zoran
> >> >
> >> > On Tue, Oct 30, 2018 at 4:19 PM, Zoran Regvart <zo...@regvart.com> wrote:
> >> >> Hi Cameleers,
> >> >> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> >> >> any comments welcome:
> >> >>
> >> >> https://github.com/apache/camel/pull/2584
> >> >>
> >> >> zoran
> >> >> --
> >> >> Zoran Regvart
> >> >
> >> >
> >> >
> >> > --
> >> > Zoran Regvart
> >>
> >>
> >>
> >> --
> >> Zoran Regvart
> >
> >
> >
> > --
> > Claus Ibsen
> > -----------------
> > http://davsclaus.com @davsclaus
> > Camel in Action 2: https://www.manning.com/ibsen2
>
>
>
> --
> Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by Zoran Regvart <zo...@regvart.com>.
Hi Claus,
both are defined in `spring-cloud-consul`[1][2], looks like they
should both have @ConditionalOnMissingBean. What puzzles me is that it
worked before the upgrade, perhaps the order of beans changed?
I think this needs to be fixed in `spring-cloud-consul`, though it
could point to a incompatibility between Spring Boot 2.0 and 2.1 --
that's where auto-configuration is.
zoran
[1] https://github.com/spring-cloud/spring-cloud-consul/blob/e8d5b575861ff39d4ea8a7fdf0eeaff6ea8dd3b8/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/serviceregistry/ConsulServiceRegistryAutoConfiguration.java#L60-L64
[2] https://github.com/spring-cloud/spring-cloud-consul/blob/e8d5b575861ff39d4ea8a7fdf0eeaff6ea8dd3b8/spring-cloud-consul-discovery/src/main/java/org/springframework/cloud/consul/discovery/ConsulDiscoveryClientConfiguration.java#L61-L65
On Fri, Nov 2, 2018 at 11:26 AM, Claus Ibsen <cl...@gmail.com> wrote:
> Hi
>
> This seems like a bit weird error.
>
> Is that heartbeatProperties something that is inherited from
> spring-boot itself or where does it come from?
>
> And would a fix be to add @ConditionalOnMissingBean on both of them?
> On Wed, Oct 31, 2018 at 3:54 PM Zoran Regvart <zo...@regvart.com> wrote:
>>
>> Hi Cameleers,
>> there are test failures in the `camel-spring-cloud-consul`[1], to me
>> it seems like an issue with Spring Boot auto-configuration:
>> `ConsulDiscoveryClientConfiguration::heartbeatProperties` and
>> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` try to
>> define the same bean, where only
>> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` is
>> @ConditionalOnMissingBean.
>>
>> Not sure what changed or how we can address this in Camel.
>>
>> zoran
>>
>> [1] https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/2/testReport/
>>
>> On Tue, Oct 30, 2018 at 10:23 PM, Zoran Regvart <zo...@regvart.com> wrote:
>> > Hi Cameleers,
>> > looks like the build succeeded, but is hanging for some reason (could
>> > be infrastructure related, not sure):
>> >
>> > https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/1/console
>> >
>> > zoran
>> >
>> > On Tue, Oct 30, 2018 at 4:19 PM, Zoran Regvart <zo...@regvart.com> wrote:
>> >> Hi Cameleers,
>> >> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
>> >> any comments welcome:
>> >>
>> >> https://github.com/apache/camel/pull/2584
>> >>
>> >> zoran
>> >> --
>> >> Zoran Regvart
>> >
>> >
>> >
>> > --
>> > Zoran Regvart
>>
>>
>>
>> --
>> Zoran Regvart
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2
--
Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by Claus Ibsen <cl...@gmail.com>.
Hi
This seems like a bit weird error.
Is that heartbeatProperties something that is inherited from
spring-boot itself or where does it come from?
And would a fix be to add @ConditionalOnMissingBean on both of them?
On Wed, Oct 31, 2018 at 3:54 PM Zoran Regvart <zo...@regvart.com> wrote:
>
> Hi Cameleers,
> there are test failures in the `camel-spring-cloud-consul`[1], to me
> it seems like an issue with Spring Boot auto-configuration:
> `ConsulDiscoveryClientConfiguration::heartbeatProperties` and
> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` try to
> define the same bean, where only
> `ConsulServiceRegistryAutoConfiguration::heartbeatProperties` is
> @ConditionalOnMissingBean.
>
> Not sure what changed or how we can address this in Camel.
>
> zoran
>
> [1] https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/2/testReport/
>
> On Tue, Oct 30, 2018 at 10:23 PM, Zoran Regvart <zo...@regvart.com> wrote:
> > Hi Cameleers,
> > looks like the build succeeded, but is hanging for some reason (could
> > be infrastructure related, not sure):
> >
> > https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/1/console
> >
> > zoran
> >
> > On Tue, Oct 30, 2018 at 4:19 PM, Zoran Regvart <zo...@regvart.com> wrote:
> >> Hi Cameleers,
> >> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> >> any comments welcome:
> >>
> >> https://github.com/apache/camel/pull/2584
> >>
> >> zoran
> >> --
> >> Zoran Regvart
> >
> >
> >
> > --
> > Zoran Regvart
>
>
>
> --
> Zoran Regvart
--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
Re: Spring and Spring Boot upgrades
Posted by Zoran Regvart <zo...@regvart.com>.
Hi Cameleers,
there are test failures in the `camel-spring-cloud-consul`[1], to me
it seems like an issue with Spring Boot auto-configuration:
`ConsulDiscoveryClientConfiguration::heartbeatProperties` and
`ConsulServiceRegistryAutoConfiguration::heartbeatProperties` try to
define the same bean, where only
`ConsulServiceRegistryAutoConfiguration::heartbeatProperties` is
@ConditionalOnMissingBean.
Not sure what changed or how we can address this in Camel.
zoran
[1] https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/2/testReport/
On Tue, Oct 30, 2018 at 10:23 PM, Zoran Regvart <zo...@regvart.com> wrote:
> Hi Cameleers,
> looks like the build succeeded, but is hanging for some reason (could
> be infrastructure related, not sure):
>
> https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/1/console
>
> zoran
>
> On Tue, Oct 30, 2018 at 4:19 PM, Zoran Regvart <zo...@regvart.com> wrote:
>> Hi Cameleers,
>> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
>> any comments welcome:
>>
>> https://github.com/apache/camel/pull/2584
>>
>> zoran
>> --
>> Zoran Regvart
>
>
>
> --
> Zoran Regvart
--
Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by Zoran Regvart <zo...@regvart.com>.
Hi Cameleers,
looks like the build succeeded, but is hanging for some reason (could
be infrastructure related, not sure):
https://builds.apache.org/view/C/view/Apache%20Camel/job/Camel/job/spring-upgrades/1/console
zoran
On Tue, Oct 30, 2018 at 4:19 PM, Zoran Regvart <zo...@regvart.com> wrote:
> Hi Cameleers,
> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> any comments welcome:
>
> https://github.com/apache/camel/pull/2584
>
> zoran
> --
> Zoran Regvart
--
Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by Zoran Regvart <zo...@regvart.com>.
Hi Cameleers,
the release notes mention that the deprecated methods were removed in
2.1[1], I don't recall seeing that we use any of those. So other than the
change when loading configurations via @SpringBootTest I haven't found any
other differences,
zoran
[1]
https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.1-Release-Notes
On Mon, Nov 5, 2018 at 4:03 PM, Claus Ibsen <cl...@gmail.com> wrote:
> Hi
>
> Thanks the PR has been merged so Camel 2.23 supports Spring Boot 2.1.
> I wonder if on top of your head / anyone else knows if that upgrade
> means Spring Boot 2.0.x is not supported / incompatible.
> Just in case someone tries to use Camel 2.23 with Spring Boot 2.0.x.
>
> And if we know this ahead of time we can write it on the release notes
> that its SB 2.1+ etc.
>
> On Tue, Oct 30, 2018 at 4:19 PM Zoran Regvart <zo...@regvart.com> wrote:
> >
> > Hi Cameleers,
> > I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> > any comments welcome:
> >
> > https://github.com/apache/camel/pull/2584
> >
> > zoran
> > --
> > Zoran Regvart
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2
>
--
Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by David Karlsen <da...@gmail.com>.
FWIW I'm currently using camel 2.22.1 with boot 2.1 without any problems
Den man. 5. nov. 2018, 16:04 skrev Claus Ibsen <cl...@gmail.com>:
> Hi
>
> Thanks the PR has been merged so Camel 2.23 supports Spring Boot 2.1.
> I wonder if on top of your head / anyone else knows if that upgrade
> means Spring Boot 2.0.x is not supported / incompatible.
> Just in case someone tries to use Camel 2.23 with Spring Boot 2.0.x.
>
> And if we know this ahead of time we can write it on the release notes
> that its SB 2.1+ etc.
>
> On Tue, Oct 30, 2018 at 4:19 PM Zoran Regvart <zo...@regvart.com> wrote:
> >
> > Hi Cameleers,
> > I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> > any comments welcome:
> >
> > https://github.com/apache/camel/pull/2584
> >
> > zoran
> > --
> > Zoran Regvart
>
>
>
> --
> Claus Ibsen
> -----------------
> http://davsclaus.com @davsclaus
> Camel in Action 2: https://www.manning.com/ibsen2
>
Re: Spring and Spring Boot upgrades
Posted by Claus Ibsen <cl...@gmail.com>.
Hi
Thanks the PR has been merged so Camel 2.23 supports Spring Boot 2.1.
I wonder if on top of your head / anyone else knows if that upgrade
means Spring Boot 2.0.x is not supported / incompatible.
Just in case someone tries to use Camel 2.23 with Spring Boot 2.0.x.
And if we know this ahead of time we can write it on the release notes
that its SB 2.1+ etc.
On Tue, Oct 30, 2018 at 4:19 PM Zoran Regvart <zo...@regvart.com> wrote:
>
> Hi Cameleers,
> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> any comments welcome:
>
> https://github.com/apache/camel/pull/2584
>
> zoran
> --
> Zoran Regvart
--
Claus Ibsen
-----------------
http://davsclaus.com @davsclaus
Camel in Action 2: https://www.manning.com/ibsen2
Re: Spring and Spring Boot upgrades
Posted by Zoran Regvart <zo...@regvart.com>.
Hi David,
you can see it here:
https://gist.github.com/zregvart/0581388eb6021c0b22eaeb5e19150579
I fiddled with configuration classes loaded, but that test really made
no sense to me, it seemed to me that it was exercising
ApplicationContextRunner not something Camel related at all...
zoran
On Tue, Oct 30, 2018 at 8:55 PM, David Karlsen <da...@gmail.com> wrote:
> Do you remember which error message you got with the test?
>
> Den tir. 30. okt. 2018 kl. 16:19 skrev Zoran Regvart <zo...@regvart.com>:
>
>> Hi Cameleers,
>> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
>> any comments welcome:
>>
>> https://github.com/apache/camel/pull/2584
>>
>> zoran
>> --
>> Zoran Regvart
>>
>
>
> --
> --
> David J. M. Karlsen - http://www.linkedin.com/in/davidkarlsen
--
Zoran Regvart
Re: Spring and Spring Boot upgrades
Posted by David Karlsen <da...@gmail.com>.
Do you remember which error message you got with the test?
Den tir. 30. okt. 2018 kl. 16:19 skrev Zoran Regvart <zo...@regvart.com>:
> Hi Cameleers,
> I've tinkered with Spring (5.1.2) and Spring Boot (2.1.0) upgrades,
> any comments welcome:
>
> https://github.com/apache/camel/pull/2584
>
> zoran
> --
> Zoran Regvart
>
--
--
David J. M. Karlsen - http://www.linkedin.com/in/davidkarlsen