Tests for multiple client access to the Infinispan cluster

[mirostary]

Changes:

• For better readability created an abstract test class that separated test deployment settings and used methods from the actual tests
• Increased Infinispan cluster allocated resources for better stability during restarts
• Second client is copied from the openshift.yml file and deployed with a different name by another YAML file
• Updated readme

3 more tests for multiple access:

• simple connect to the second client
• updating counters
• connecting back after Datagrid restart

[mirostary]

Hi @mjurc, this PR is now ready for review.
I'm just investigating one more issue which sometimes appears in the cache counter.

[mjurc]

The code seems to be okay, the utility classes and deployments do what they are supposed to do.

@mirostary Now that we have multitude of non-trivial scenarios in this test case, would you kindly add some simple summary to each test what the test does and where the assertions come from?

[mirostary]

@mjurc I wrote a simple description of all tests and methods and I also disable the 7th test for now because of the issue (quarkusio/quarkus#15816).

[mjurc]

@mirostary Thanks for the fixes!

There is one more thing however, I realised. We need to have the switch to run the unit tests for those modules in GitHub actions. Please add those and once the new actions pass, this will be ready for merge.

[Sgitario]

run tests

[mirostary]

@mjurc
oh, you mean to add -Dinclude.datagrid switch to the https://github.com/quarkus-qe/quarkus-openshift-test-suite/blob/main/.github/workflows/ci.yml. Am I right?

[Sgitario]

run tests

[mjurc]

@mjurc
oh, you mean to add -Dinclude.datagrid switch to the https://github.com/quarkus-qe/quarkus-openshift-test-suite/blob/main/.github/workflows/ci.yml. Am I right?

@mirostary Yes!

[rsvoboda]

Update the README to include info about include.datagrid flag!

Shouldn't be the RHDG tests using @OnlyIfConfigured("ts.authenticated-registry") ?

Do we have tests for upstream ISPN?

[rsvoboda]

also pls rebase to pick #246 changes

[mirostary]

Update the README to include info about include.datagrid flag!

Shouldn't be the RHDG tests using @OnlyIfConfigured("ts.authenticated-registry") ?

done

Do we have tests for upstream ISPN?

No, only tests related to Openshift Datagrid Operator

Still, some failed checks here... I don't know why JVM build action is failing. According to this https://main-jenkins-csb-quarkusqe.cloud.paas.psi.redhat.com/job/openshift-test-suite-ci/433/, at least the "Run OpenShift Tests Suite in JVM" should be ok.

[rsvoboda]

run tests

[mirostary]

@mjurc @rsvoboda
According to the comment in QUARKUS-876, I removed the "@ Disabled" annotation and created the hotrod-client.properties for reducing the time when an application is trying to connect to the failed server. This PR is ready to review/merge.

[rsvoboda]

@mjurc what's the eta for review of this PR?

[mjurc]

Requested changes implemented.

CI passed:
https://main-jenkins-csb-quarkusqe.cloud.paas.psi.redhat.com/job/openshift-test-suite-ci/483/
https://main-jenkins-csb-quarkusqe.cloud.paas.psi.redhat.com/job/openshift-test-suite-native-ci/344/

@mirostary Thanks for implementation. Kindly backport this coverage to 1.11 branch as well, thanks.