Radoslav Husar

Results 97 comments of 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.