-
Notifications
You must be signed in to change notification settings - Fork 468
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for Image Cleaner #343
Conversation
@microsoft-github-policy-service agree |
4f3d07c
to
1c22fa4
Compare
1c22fa4
to
23f1d29
Compare
23f1d29
to
f14ea2f
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.
Thanks @skolobov for contributing this pr to us and apology for this crazy late response. I'd like to merge this pr as part of v7.5.0
, but only one review comment to solve.
@@ -23,6 +23,8 @@ resource "azurerm_kubernetes_cluster" "main" { | |||
disk_encryption_set_id = var.disk_encryption_set_id | |||
dns_prefix = var.prefix | |||
http_application_routing_enabled = var.http_application_routing_enabled | |||
image_cleaner_enabled = var.image_cleaner_enabled | |||
image_cleaner_interval_hours = var.image_cleaner_enabled && var.image_cleaner_interval_hours != null ? var.image_cleaner_interval_hours : null |
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.
Could we use image_cleaner_interval_hours = var.image_cleaner_interval_hours
here?
Potential Breaking Changes in 86e038e: |
Describe your changes
Add support for optional Image Cleaner feature in AKS.
Issue number
Fixes #342
Checklist before requesting a review
CHANGELOG.md
file