Cross Site - Gossip Router tunnel support

[pruivo]

Adds support to use JGroups' TUNNEL protocol with GossipRouter pods.
It uses the JGroups JAR inside the server image to avoid depending on another image.

PR is open to get feedback sooner than later. No rush to integrate yet.

TODO:

• Create an entry point in the server image (avoid doing "java -jar ...")
• Handle server image upgrades (?)
• Add Gossip Router pod condition to Infinispan CR
• Add readiness & liveness probes

[pruivo]

depends on

• Update server image to 13.0 #1238
• Increase Pod memory for XSite tests #1252

[dmvolod]

Also we need to watch Deployment resources to catch unpredictable resource remove/delete/update

[pruivo]

Also we need to watch Deployment resources to catch unpredictable resource remove/delete/update

I wasn't aware of that. Is there any example in the code?

[dmvolod]

I wasn't aware of that. Is there any example in the code?

You should add appsv1.Deployment to the secondaryResourceTypes list

infinispan-operator/controllers/infinispan_controller.go

Line 126 in 6381d8b

secondaryResourceTypes := []client.Object{&appsv1.StatefulSet{}, &corev1.ConfigMap{}, &corev1.Secret{}}

[pruivo]

updated the code to use controllerutil.CreateOrUpdate().

Also, just crossed my mind, I should add some readiness and liveness probes to GossipRouter pod.

[pruivo]

code updated.
Added probes
Added condition "GossipRouterReady" example

Status:
  Conditions:
    Status:   True
    Type:     PreliminaryChecksPassed
    Status:   True
    Type:     GossipRouterReady
    Message:  View: test-cross-site-view-internal-xsite2-0-63738
    Status:   True
    Type:     WellFormed
    Message:  Cross-Site view: xsite1,xsite2
    Status:   True
    Type:     CrossSiteViewFormed

[pruivo]

updated.
Added: shutting down the GossipRouter pod (set Deployment.Spec.Replicas=0) if Infinispan.Spec.Replicas == 0

[ryanemerson]

If you're happy with the changes @dmvolod I will merge

[dmvolod]

If you're happy with the changes @dmvolod I will merge

Yes, @ryanemerson looks good, thanks
Also Rolling Upgrade PR need to be adopted for GossipRouter deployment if it will be supported

[ryanemerson]

Thanks @pruivo

[ryanemerson]

Also Rolling Upgrade PR need to be adopted for GossipRouter deployment if it will be supported

Good point @dmvolod. FYI @gustavonalle