Hello folks,
Issues[1] were created for integrating the wildfly-vertx-feature-pack[2]
to WildFly Preview, and the PR[3] was opened as well. Any comments are
welcome :)
Thanks
[1]
On Wed, Aug 28, 2024 at 12:09 AM Brian Stansberry <
brian.stansberry(a)redhat.com> wrote:
Thanks, for this, Lin!
It's possible to maintain things in an external FP code base but still
integrate them into std WF. This is how the mvc-krazo extension works.
I'm not going to pretend that's particularly comfortable though, and it
would need some thought whether there's a sufficient benefit to it. The
basic benefit would be that the external FP could do more releases
independent of WF. We do pretty frequent releases though!
The other benefit would be if the extension wanted to experiment with
changes that can't be handled using the stability level approach. But a lot
of things can be modeled using stability levels.
A third benefit would be that in general I think it would be good if some
extensions were maintained externally. But I don't think something like
this that's meant to be used by other extensions is a good candidate for
that.
Speaking of stability levels....
I think this subsystem would come into WF as at most Preview stability,
but we have existing default stability uses of Vert.x. So the integration
of this must bridge that gap. Perhaps via hooks in the existing uses such
that if a capability provided by this subsystem is available, use it,
otherwise fall back to the existing way of integrating vert.x.
Re the 3 "Feature decision" questions, I'll comment on the doc.
On Tue, Aug 27, 2024 at 10:11 AM Kabir Khan <kkhan(a)redhat.com> wrote:
> I added my comments to the document.
>
> IMO the main use case that **I** am after is the ability to configure
> vert.x options, and I think it makes sense to put that part into WildFly
> since it will be consumed by reactive messaging and Otel.
>
> That said, it would be great to be able to continue having the
> extra functionality that I am not bothered about coming in via the feature
> pack. This depends a bit on if it is possible to augment the subsystem I
> would like to see in WildFly with this extra stuff. Some alternatives might
> be a new vertx-extras subsystem providing additional information, or
> perhaps we could simply make the additional features a lower stability
> level so that users have to opt in somehow.
>
> Thanks,
>
> Kabir
>
>
>
> On Tue, 27 Aug 2024 at 16:09, Lin Gao <lgao(a)redhat.com> wrote:
>
>> Thanks Brian.
>>
>> The extension provides more features than needed to address the issue
>> above[1], so I think it would be great to have a public discussion on which
>> one is needed and how to get it integrated into WildFly first, so I created
>> a doc at:
>>
https://docs.google.com/document/d/1gvVB3_Uh_urPZdc3eej9i7yotLDRJWBkapybC...
>> for the discussion.
>>
>> Any comment is more than welcome :)
>>
>> [1]
>>
https://github.com/smallrye/smallrye-reactive-messaging/discussions/2725
>>
>> Best regards
>>
>> On Mon, Aug 26, 2024 at 9:48 PM Brian Stansberry <
>> brian.stansberry(a)redhat.com> wrote:
>>
>>>
>>>
>>> On Sat, Aug 24, 2024 at 11:24 PM Lin Gao <lgao(a)redhat.com> wrote:
>>>
>>>> Hello,
>>>>
>>>> Sorry for the late response, I was away last week.
>>>>
>>>> Yes, I did some research on developing a Vertx subsystem for WildFly,
>>>> which has the capabilities to configure Vertx options.
>>>> Now only one Vertx instance can be configured with the subsystem, and
>>>> the Vertx instance can be referenced by other subsystems via the
capability
>>>> name.
>>>>
>>>> I think one Vertx instance should be good enough to cover most cases
>>>> unless there are demands for multiple instances.
>>>>
>>>> @Brian, what should I do to move the wildfly-vertx-extension to
>>>> wildfly-extras ? I am willing to do that shortly if applicable :) Thank
you
>>>> very much !
>>>>
>>>
>>> When you're ready, find me on Zulip when we're both online and we
can
>>> work through the process. It only takes a few minutes. I'm online in the
>>> evening often. Or, feel free to put something in my calendar and we'll
meet
>>> then. It's generally no problem at all for you to put something on my
>>> calendar between 19:00 and 21:30 US Central Daylight Time.
>>>
>>>
>>>> Best regards
>>>>
>>>> On Tue, Aug 20, 2024 at 11:27 PM Brian Stansberry <
>>>> brian.stansberry(a)redhat.com> wrote:
>>>>
>>>>> Lin Gao has done a lot in this area, so be sure to discuss with him:
>>>>>
>>>>>
https://github.com/gaol/wildfly-vertx-extension
>>>>>
>>>>> This was something we'd discussed moving to wildfly-extras last
year.
>>>>>
>>>>> On Tue, Aug 20, 2024 at 8:05 AM Jason Lee
<jasondlee(a)redhat.com>
>>>>> wrote:
>>>>>
>>>>>> (SmallRye) OpenTelemetry does use vert.x for its exporters, so
there
>>>>>> is definitely some overlap. I'm not sure what, if any,
configuration
>>>>>> options might be of interest there or what the implications of a
shared
>>>>>> vert.x instance might be off hand, but it certainly sounds like
something
>>>>>> worth investigating.
>>>>>>
>>>>>> Jason Lee
>>>>>>
>>>>>> Principal Software Engineer
>>>>>>
>>>>>> Red Hat JBoss EAP
>>>>>>
>>>>>> Java Champion
>>>>>>
>>>>>>
>>>>>> On Tue, Aug 20, 2024 at 3:02 AM Kabir Khan
<kkhan(a)redhat.com> wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> At the moment the SmallRye/MP Reactive Messaging integration
just
>>>>>>> uses a default created vert.x.
>>>>>>>
>>>>>>> I think otel uses vert.x too, probably in much the same way?
Then
>>>>>>> there might be other subsystems in various feature packs.
>>>>>>>
>>>>>>> It seems there is some desire to be able to configure the
vert.x
>>>>>>> instance
>>>>>>>
https://github.com/smallrye/smallrye-reactive-messaging/discussions/2725.
>>>>>>> Being able to configure vert.x is TBH something I've
totally ignored.
>>>>>>>
>>>>>>> I guess this would mean a Vert.X subsystem, which can
optionally
>>>>>>> create a vert.x instance with the configured parameters. Does
it make sense
>>>>>>> to create more than one instance in case subsystems have
slightly different
>>>>>>> needs?
>>>>>>>
>>>>>>> Then the reactive messaging subsystem can use that, if
present (or
>>>>>>> perhaps this should be a configuration parameter in the
reactive messaging
>>>>>>> subsystem). If not (or not configured to do so), it will use
the default
>>>>>>> like it does today.
>>>>>>>
>>>>>>> Thanks,
>>>>>>>
>>>>>>> Kabir
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> wildfly-dev mailing list -- wildfly-dev(a)lists.jboss.org
>>>>>>> To unsubscribe send an email to
wildfly-dev-leave(a)lists.jboss.org
>>>>>>> Privacy Statement:
https://www.redhat.com/en/about/privacy-policy
>>>>>>> List Archives:
>>>>>>>
https://lists.jboss.org/archives/list/wildfly-dev@lists.jboss.org/message...
>>>>>>>
>>>>>> _______________________________________________
>>>>>> wildfly-dev mailing list -- wildfly-dev(a)lists.jboss.org
>>>>>> To unsubscribe send an email to
wildfly-dev-leave(a)lists.jboss.org
>>>>>> Privacy Statement:
https://www.redhat.com/en/about/privacy-policy
>>>>>> List Archives:
>>>>>>
https://lists.jboss.org/archives/list/wildfly-dev@lists.jboss.org/message...
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Brian Stansberry
>>>>> Principal Architect, Red Hat JBoss EAP
>>>>> WildFly Project Lead
>>>>> He/Him/His
>>>>>
>>>>
>>>>
>>>> --
>>>> Lin Gao
>>>> Senior Software Engineer
>>>> JBoss Sustaining Engineering Team
>>>>
>>>>
>>>
>>> --
>>> Brian Stansberry
>>> Principal Architect, Red Hat JBoss EAP
>>> WildFly Project Lead
>>> He/Him/His
>>>
>>
>>
>> --
>> Lin Gao
>> Senior Software Engineer
>> JBoss Sustaining Engineering Team
>>
>>
--
Brian Stansberry
Principal Architect, Red Hat JBoss EAP
WildFly Project Lead
He/Him/His