[issues-94] New WildFly Application to be used in Apache Kafka related Tests

[tommaso-borgato]

Resolves #94

Coupled with Intersmash/intersmash-tests#86

Description

New Application to be used in Apache Kafka related Tests:

• MP Reactive Messaging with per-connector secured connection test
• MP Reactive Messaging Bootable JAR with per-connector secured connection test
• MP Reactive Messaging Bootable JAR with globally secured connection test

Ci Runs:

• eap-8.x-cross-product-face#209
• eap-8.x-cross-product-face#214
• eap-8.x-cross-product-face#243
• eap-8.x-cross-product-face#246
• eap-8.x-cross-product-face#250
• eap-8.x-cross-product-face#258

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change
• Chore (non-breaking change which doesn't affect codebase;
  test, version modification, documentation, etc.)

Checklist

• Code contributed by me aligns with current project license: Apache 2.0
• Pull Request contains a description of the changes
• Pull Request does not include fixes for multiple issues/topics
• Code is self-descriptive and/or documented
• I have implemented unit tests to cover my changes
• I tested my code in OpenShift

[fabiobrz]

Thanks for this PR @tommaso-borgato - I've dropped some comments because it looks like we need to check and add back some tests.

[fabiobrz]

kafka-application is not bootable, so why do we add this configuration? I recall we discussed it already, and it seems to me we said it is required for bootable jar applications only, or am I missing something?

@jfdenise - do you have any hints about adding this specific configuration?

[jfdenise]

@fabiobrz , I can see the kafka application being built as bootable JAR:

intersmash-applications/wildfly/kafka-application/pom.xml

Line 94 in 38d17b0

<bootable-jar>true</bootable-jar>

I suspect that the usage of this script is due to wildfly/wildfly-cloud-galleon-pack#65 that has been fixed recently in the cloud feature-pack.
I plan to do a new release today of the cloud FP before WildFly 40.Final, this script will become useless.

[jfdenise]

@fabiobrz @tommaso-borgato I have just released cloud feature-pack 9.2.2.Final

[tommaso-borgato]

9.2.2.Final

Thank you @jfdenise; I suppose we should wait deleting management.cli until EAP versions we test include these changes, right?

[jfdenise]

Are you already testing EAP 8.2 with bootable JAR? The PROD produced cloud FP is not ready for Bootable JAR support, so such tests shouldn't be runnable yet.

Bootable JAR support for Openshift for EAP 8.2 is currently a WIP RFE, QE is testing a wip branch that includes this fix already: https://github.com/jfdenise/eap-cloud-galleon-pack/tree/bootable-in-cloud-fp

So theoretically, you should delete this management.cli script, not needed when Bootable JAR for Openshift will be ready.

[fabiobrz]

Thanks @jfdenise !

[tommaso-borgato]

@jfdenise Thank you, we are refining these tests using wildfly ... we can get rid of scripts once https://github.com/jfdenise/eap-cloud-galleon-pack/tree/bootable-in-cloud-fp is merged and released right?

[jfdenise]

@tommaso-borgato , we just got the pre-check, I will merge soon.

[fabiobrz]

LGTM, thanks for the great work here @tommaso-borgato, merging!