balancerHealthMonitor doesn't work with 7.2.1 when balancer is defined with chain component

[paulhyo]

balancerHealthMonitor is NOT working when balancer is defined als chain:

components:
  bhm:
    balancerHealthMonitor:
      interval: 30000
      httpClientConfig:
        connection:
          timeout: 2000
          soTimeout: 2000
  cluster1:
    chain:
      - balancer:
          priorityStrategy: {}
          clusters:
            - name: cluster1
              nodes:
                - host: localhost
                  port: 8001
                  healthUrl: http://localhost:8001/healthh
                  priority: 1
                - host: localhost
                  port: 8002
                  healthUrl: http://localhost:8002/health
                  priority: 2

balancerHealthMonitor is working when balancer is defined directly in an api:

components:
  bhm:
    balancerHealthMonitor:
      interval: 30000
      httpClientConfig:
        connection:
          timeout: 2000
          soTimeout: 2000

---

api:
. ..
  flow:
   ...
    - balancer:
        priorityStrategy: { }
        clusters:
          - name: cluster1
            nodes:
              - host: localhost
                port: 8001
                healthUrl: http://localhost:8001/healthh
                priority: 1
              - host: localhost
                port: 8002
                healthUrl: http://localhost:8002/health
                priority: 2

[predic8]

Hi @paulhyo,
thanks for reporting.

What exactly is not working? Do you get an error message or does it not behave as intended?

Feel free to send me your apis.yaml as PM.

Cheers
Thomas

[paulhyo]

Hi @predic8
The healthUrl is not checked at all when used within a chain, so no test is done. The priority 1 node get the request, even if not accessible.

When defined directly under "api", I can see the failed health checked in the log. When defined within a chain, nothing in the log (no check is done).

if I remember well, we had the same problem with 6.x versions (proxies.xml configuration) and a fix has been made.

Cheers
Stéphane

[predic8]

Hi @paulhyo,
yes we fixed that in 6.3.16. We will apply this fix also to the next 7.2.2.
Cheers
Thomas

[paulhyo]

Hi @predic8
thank you! When will version 7.2.2 be available?

Cheers
Stéphane

[predic8]

It is more than just applying the changes from 6.3.16 as a patch. With YAML a lot has changed and we need to rewrite the util that searches the balancers in the chains. So I guess 7.2.2 will be available in about a week. Maybe earlier.