Fix colocated_cluster test - Githubissues

ansible-middleware / wildfly

A collection of roles to help install, setup and maintain Java JEE appserver Wildfly

GNU General Public License v2.0

12 stars 22 forks source link

Fix colocated_cluster test #257

Closed istraka closed 1 month ago

istraka commented 1 month ago

There are two problems with the test:

legacy redundant configuration (overriden - it has no effect & is outdated): https://github.com/ansible-middleware/wildfly/blob/main/molecule/colocated_cluster/templates/yaml_configuration.yml.j2#L5
configure_tcp_ping.yml.j2 [1] defines only two nodes in the cluster, but there in fact three. It still works, bacause nodes seem to share topology transitively, but it should contain whole topology.

https://github.com/ansible-middleware/wildfly/blob/main/molecule/colocated_cluster/templates/configure_tcp_ping.yml.j2

I will submit the PR.