Closed engedaam closed 1 week ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
pkg/controllers/nodeclaim/lifecycle/controller.go | 4 | 7 | 57.14% | ||
pkg/controllers/controllers.go | 0 | 8 | 0.0% | ||
pkg/controllers/node/health/controller.go | 51 | 65 | 78.46% | ||
<!-- | Total: | 70 | 95 | 73.68% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
pkg/test/expectations/expectations.go | 2 | 94.73% | ||
pkg/scheduling/requirements.go | 2 | 98.01% | ||
<!-- | Total: | 4 | --> |
Totals | |
---|---|
Change from base Build 11886240983: | -0.07% |
Covered Lines: | 8734 |
Relevant Lines: | 10779 |
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: engedaam, jonathan-innis
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Fixes #N/A
Description
RepairPolicy
that will support node conditions that Karpenter will forcefully terminate nodes. The cloud provider policies will be unhealthy conditions a node can enter and the duration for Karpenter to react.How was this change tested?
make resubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.