fix review comments

Author: rahulait

cloud/linode/loadbalancers.go

@@ -61,7 +61,7 @@ var (
 		string(linodego.AlgorithmRingHash):   true,
 		string(linodego.AlgorithmLeastConn):  true,
 	}
-	// validTCPStickiness is a map of valid HTTP stickiness options
+	// validHTTPStickiness is a map of valid HTTP stickiness options
 	validHTTPStickiness = map[string]bool{
 		string(linodego.StickinessNone):       true,
 		string(linodego.StickinessHTTPCookie): true,

docs/configuration/annotations.md

@@ -21,8 +21,8 @@ The keys and the values in [annotations must be strings](https://kubernetes.io/d
 | `throttle` | `0`-`20` (`0` to disable) | `0` | Client Connection Throttle, which limits the number of subsequent new connections per second from the same client IP |
 | `default-protocol` | `tcp`, `udp`, `http`, `https` | `tcp` | This annotation is used to specify the default protocol for Linode NodeBalancer |
 | `default-proxy-protocol` | `none`, `v1`, `v2` | `none` | Specifies whether to use a version of Proxy Protocol on the underlying NodeBalancer |
-| `default-algorithm` | `roundrobin`, `leastconn`, `source`, `ring_hash` | `roundrobin` | This annotation is used to specify the default alogrithm for Linode NodeBalancer |
-| `default-stickiness` | `none`, `session`, `table`, `http_cookie`, `source_ip` | `session` UDP, `table` HTTP/HTTPs | This annotation is used to specify the default stickiness for Linode NodeBalancer |
+| `default-algorithm` | `roundrobin`, `leastconn`, `source`, `ring_hash` | `roundrobin` | This annotation is used to specify the default algorithm for Linode NodeBalancer |
+| `default-stickiness` | `none`, `session`, `table`, `http_cookie`, `source_ip` | `session` (for UDP), `table` (for HTTP/HTTPs) | This annotation is used to specify the default stickiness for Linode NodeBalancer |
 | `port-*` | json object | | Specifies port specific NodeBalancer configuration. See [Port Configuration](#port-specific-configuration) |
 | `check-type` | `none`, `connection`, `http`, `http_body` | `none` for UDP, else `connection` | The type of health check to perform against back-ends. See [Health Checks](loadbalancer.md#health-checks) |
 | `check-path` | string | | The URL path to check on each back-end during health checks |

e2e/test/lb-with-udp-ports/chainsaw-test.yaml

@@ -55,8 +55,8 @@ spec:
                   port_7070_timeout=$(echo $nbconfig | jq '.check_timeout == 3')
                   port_7070_attempts=$(echo $nbconfig | jq '.check_attempts == 2')
                   port_7070_protocol=$(echo $nbconfig | jq '.protocol == "udp"')
-                  #port_7070_up_nodes=$(echo $nbconfig | jq '(.nodes_status.up)|tonumber >= 2')
-                  # Placeholder for the actual check until we have the support
+                  # TODO: Implement the actual check for UDP node health when support is added
+                  # port_7070_up_nodes=$(echo $nbconfig | jq '(.nodes_status.up)|tonumber >= 2')
                   port_7070_up_nodes="true"
 
                   if [[ $port_7070_check == "true" && $port_7070_interval == "true" && $port_7070_timeout == "true" && $port_7070_attempts == "true" && $port_7070_protocol == "true" && $port_7070_up_nodes == "true" ]]; then