Azure / karpenter-provider-azure / 14542378984

94%

Build Type

push

github

Committed by web-flow

Commit Message

feat(upgrade): adding new NodeClass status controller logic for upgrades of k8s, and node image versions (#729)

* controller at point of working status updates

* break out k8s reconcile method

* refactor to k8sVersion having seperate recomciler/provider

* refactoring and cleaniness

* change helper to just trim version suffix

* rename Images to NodeImages

* update Image -> NodeImage naming and add unit tests

* add test cases for controller, (all success cases with no MW support)

* add successful test cases for List images provider within imagefamily

* add TODOs about error sim testing

* fix CommunityImageVersionsAPI being set in azureEnv + update node versioning test to have actual CIG versioning

* Update pkg/controllers/nodeclass/status/k8sversion.go

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update pkg/controllers/nodeclass/status/nodeimage.go

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* updating k8s -> Kubernetes (without make presubmit)

* make presubmit

* follow styling of params all on seperate lines

* update controller params to be interfaces

* update k8s -> kubernetes [think I got all non-vars]

* update wording from MW ->  maintenance window

* make trimVersionSuffix private

* Update pkg/controllers/nodeclass/status/nodeimage_test.go

Co-authored-by: Matthew Christopher <matthchr@users.noreply.github.com>

* MW -> maintenance window

* remove old import comment

* make reconciler interface private

* rename var to supportedImage

* change to helper functions of getSIG and getCIG

* add examples of node image IDs (no make presubmit)

* make presubmit

* add TODO comment on why magic MaxConcurrentReconciles number

* update kubernetes logging to named

* updated nodeclass.nodeimage logger to be named

* refactor to const for new reconciler names

* add examples for trimVersionSuffix

* update logging to be cleaner

* remove alias on []NodeImage

* add comment on sort for c... (continued)

Run Details

172 of 273 new or added lines in 11 files covered. (63.0%)

4 existing lines in 2 files now uncovered.

48716 of 51146 relevant lines covered (95.25%)

0.96 hits per line