Radoslav Husar
Radoslav Husar
Done except for 3 branches that have some work in it that needs to be triaged whether still applicable.
> Hmm... adding the system property should _not_ be required, although it certainly works, so no objections. This could be done in the XML config directly: JGroups parses comma-deparated attributes...
The whole BasicRuntimeIT is a mess as well... it's being overriden (http://localhost:8080/) but in the templates this forms URIs without the context path - and since it only chcecks for...
> The whole BasicRuntimeIT is a mess as well... it's being overriden (http://localhost:8080/) but in the templates this forms URIs without the context path - and since it only chcecks...
> FYI the server host is overriden on server provisioning/bootable jar environment, and on those the app is deployed at root context, i.e. thus no context path > […](#) >...
> @pferraro @rhusar not sure who is now the responsible for this QS, my guess it's one of you, please review @emmartins It's me. I can't assign myself as reviewer...
> @rhusar Is there anything else that you need from my side? Nope - just competing priorities on my side to get to do a full review...
@emmartins Is this expected? The CI fails with > { could not load Hive kube config from path /secrets/hive-hive-credentials/kubeconfig: could not load kubeconfig: stat /secrets/hive-hive-credentials/kubeconfig: no such file or directory}
> It is unrelated has failed with a similar stack trace for other PRs. In addition, since it is a test only dependency, it is not being driven to WildFly...
@ryanemerson Sure thing! Lets catch up on Monday as I am on PTO today.