-
Notifications
You must be signed in to change notification settings - Fork 49
Update cluster-autoscaler to v2 version 1.0.2 #859
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@knrt10 did you test that updated component actually works on Packet? I see a lot of removal in the chart related to Packet, has this been added someplace else? If not, perhaps we should create an issue and submit our changes upstream.
Didn't test it, let me do that |
fe00dcf
to
660367a
Compare
835dfeb
to
dfcd74d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. We've tested the autoscaler together with @knrt10 and it deploys and runs correctly, but it's functionality is still broken 😞
Opened an issue upstrem and awaiting fix from Packet regarding 503 internal server error. |
Release info: kubernetes/autoscaler/releases/tag/cluster-autoscaler-chart-1.0.2 Fixes #858 Signed-off-by: knrt10 <kautilya@kinvolk.io>
Signed-off-by: knrt10 <kautilya@kinvolk.io>
dfcd74d
to
c4907ee
Compare
@knrt10 the override needs to be removed :) |
Aaah, my bad, doing it |
cluster-autoscaler chart now deploys v.18.1 by default. Signed-off-by: knrt10 <kautilya@kinvolk.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Release info:
kubernetes/autoscaler/releases/tag/cluster-autoscaler-chart-1.0.2
Fixes #858
Signed-off-by: knrt10 kautilya@kinvolk.io