karpenter-provider-aws
karpenter-provider-aws copied to clipboard
aws
chore: add configurable ttl to post install hook
Fixes https://github.com/aws/karpenter-provider-aws/issues/6765
Description
Added configuration option for the ttlSecondsAfterFinished field on the post install job so that Argo CD based installs can complete (as Argo CD needs to reap the job rather than the internal controller).
How was this change tested?
Forked the chart, installed it with Argo CD with a value of 30 for .Values.postInstallHook.ttlSecondsAfterFinished.
Does this change impact docs?
- [ ] Yes, PR includes docs updates
- [ ] Yes, issue opened: #
- [x] No
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Deploy Preview for karpenter-docs-prod ready!
| Name | Link |
|---|---|
| Latest commit | f50ffb62da01d874960620528fcc2048257bf078 |
| Latest deploy log | https://app.netlify.com/sites/karpenter-docs-prod/deploys/66c481e4678c420008eab373 |
| Deploy Preview | https://deploy-preview-6766--karpenter-docs-prod.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
![netlify[bot] avatar](https://avatars.githubusercontent.com/in/13473?v=4 "Published by a pub.dev verified publisher"){kind=small}
Pull Request Test Coverage Report for Build 10400404075
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
- For more information on this, see Tracking coverage changes with pull request builds.
- To avoid this issue with future PRs, see these Recommended CI Configurations.
- For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
Details
- 0 of 0 changed or added relevant lines in 0 files are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage remained the same at 79.327%
| Totals | |
|---|---|
| Change from base Build 10396139252: | 0.0% |
| Covered Lines: | 5921 |
| Relevant Lines: | 7464 |
