helm-kubernetes-services icon indicating copy to clipboard operation
helm-kubernetes-services copied to clipboard

Add version compatibility to the HPA API

Open paul-pop opened this issue 2 years ago • 2 comments

Description

Fixes #145.

TODOs

Read the Gruntwork contribution guidelines.

  • [x] Update the docs.
  • [x] Run the relevant tests successfully, including pre-commit checks.
  • [x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
  • [x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Use autoscaling/v2 API for Kubernetes versions >= 1.23.

Migration Guide

paul-pop avatar Sep 02 '22 20:09 paul-pop

@yorinasub17 👋

Any chance for a quick look and this a potential release so k8s 1.23 gets the support it needs?

paul-pop avatar Sep 08 '22 17:09 paul-pop

@autero1 Can you please re-check? Hopefully this gives other contributors some examples on how to test capabilities and server versions during the render.

I've also fixed the pre-commit hook for helmlint and updated it to the latest version as the previous one was returning a warning:

➜  helm-kubernetes-services git:(fix/145/hpa-api-version-compatibility) pre-commit run --all-files                   
[WARNING] Unexpected key(s) present on https://github.com/gruntwork-io/pre-commit: sha
helmlint.................................................................Passed

paul-pop avatar Sep 29 '22 16:09 paul-pop

Hi, sorry for the delay. I've kicked off the tests again.

autero1 avatar Oct 16 '22 06:10 autero1

All the tests failed on:

Running Go tests in  with command: go test -v -timeout 60m -parallel 128  -tags integration . -short
# sigs.k8s.io/json/internal/golang/encoding/json
../../.go_workspace/pkg/mod/sigs.k8s.io/[email protected]/internal/golang/encoding/json/encode.go:1249:12: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
../../.go_workspace/pkg/mod/sigs.k8s.io/[email protected]/internal/golang/encoding/json/encode.go:1255:18: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
FAIL	github.com/gruntwork-io/helm-kubernetes-services/test [build failed]

autero1 avatar Oct 16 '22 06:10 autero1

Closing as #151 was merged with the changes.

paul-pop avatar Jan 24 '23 16:01 paul-pop