• Matt Farina's avatar
    Fix specifying of Kubernetes version from build scripts · 7838fb76
    Matt Farina authored
    
    
    This bug came about because of three issues that this change
    corrects:
    - The CI scripts run on a pull request did not test building
      Helm. This means that a failure to set a variable using LDFLAGS
      had no opportunity to be caught.
    - #8608 provided a means to match the k8s version used in linting
      and chartutil with the version of the package we pull in. With
      one problem. It attempts to set a const as if it were a string.
      This is ignored and everyone missed it.
    - #10325 moved those constants to vars so it could be set. This
      looked good and passed tests but missed that you can't set an
      int as if it were a string. See first bullet.
    
    This change fixes this by moved the internal representation to
    be a string. These are internal variables not exposed in the public
    API which makes this change non-breaking to the API.
    
    Closes #10367
    Signed-off-by: default avatarMatt Farina <matt.farina@suse.com>
    7838fb76