Mellanox / network-operator / 22036250267

Build # 22036250267

Build Type

push

github

Committed by web-flow

Commit Message

fix: relax kubeVersion constraint to support pre-release suffixes (#2167)

## Summary
- Append `-0` to the Helm chart `kubeVersion` constraint (`>= 1.21.0` →
`>= 1.21.0-0`) so it accepts Kubernetes versions with pre-release
suffixes

## Problem
Kubernetes distributions like EKS append pre-release metadata to the
version string (e.g. `v1.34.3-eks-ac2d5a0`). Per the [semver
spec](https://semver.org/#spec-item-11), pre-release versions have lower
precedence than the release version, so Helm's `>= 1.21.0` constraint
rejects `1.34.3-eks-ac2d5a0` even though the cluster version clearly
satisfies the requirement.

```
Error: chart requires kubeVersion: >= 1.21.0 which is incompatible with Kubernetes v1.34.3-eks-ac2d5a0
```

## Fix
Appending `-0` to the constraint includes pre-release versions in the
comparison. This is the standard Helm convention — the bundled
`sriov-network-operator` sub-chart already uses `>= 1.16.0-0`.

Reference: https://helm.sh/docs/topics/charts/#the-apiversion-field

## Test plan
- [x] Verified `helm install` succeeds on EKS v1.34.3-eks-ac2d5a0 with
patched chart
- [x] No change in behavior for standard Kubernetes versions (e.g.
v1.32.0)

Run Details

4168 of 6749 relevant lines covered (61.76%)

48.56 hits per line

Source File

Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available