F5Networks / f5-appsvcs-extension

F5 BIG-IP Application Services 3 Extension
Apache License 2.0
163 stars 52 forks source link

Minimum Monitors for Members in Class Pool does not support "all" #827

Open amolari opened 2 months ago

amolari commented 2 months ago

Environment

Summary

Similar to issue #106 but for pool members (member-specific) monitor, it's not possible to achieve "Availability Requirement = All".

Steps To Reproduce

Steps to reproduce the behavior:

  1. Submit the following declaration:
    
      "mainICAP_pool": {
        "members": [
          {
            "addressDiscovery": "fqdn",
            "autoPopulate": true,
            "hostname": "example.com",
            "servicePort": 1344,
            "priorityGroup": 5,
            "minimumMonitors": "all",
            "monitors": [
              {
                "use": "mainICAP_stby_http"
              },
              {
                "use": "icap-STBY-MD-CORE-2_https"
              }
            ]
          }

2. Observe the following error response:
```json
{
  "code": 422,
  "errors": [
    "/Common/Shared/mainICAP_pool/members/3/minimumMonitors: should be integer"
  ],
  "declarationFullId": "",
  "message": "declaration is invalid",

Expected Behavior

I would expect that the configuration property "minimumMonitors" to be exactly the same for pool monitor and pool member monitor. Also, I think a string "all" is OK (probably added to have a wording parity with tmsh), but integer value of 0 should also provide the same result as "all".

Actual Behavior

Not possible to configure the pool member monitor as required.

sunitharonan commented 4 weeks ago

We have a service request for the same, I have created a backlog item AUTOTOOL-4394 and added it to our upcoming sprint. Thanks

mdditt2000 commented 2 weeks ago

Planing for AS3-53