Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/image_licenses.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,6 @@ licenses:
docker.io/grafana/tempo:
license: AGPL-3.0-only
licenseLink: https://raw.githubusercontent.com/grafana/tempo/refs/heads/main/LICENSE
docker.io/hjacobs/kube-janitor:
license: GPL-3.0
licenseLink: https://github.com/hjacobs/kube-janitor/blob/main/LICENSE
docker.io/memcached:
license: BSD-3
licenseLink: https://raw.githubusercontent.com/memcached/memcached/refs/heads/master/LICENSE
Expand Down Expand Up @@ -128,6 +125,9 @@ licenses:
ghcr.io/teutonet/oci-images/solr-ckan:
license: MIT
licenseLink: https://github.com/teutonet/oci-images/blob/main/LICENSE
ghcr.io/twin/k8s-ttl-controller:
license: MIT
licenseLink: https://github.com/TwiN/k8s-ttl-controller/blob/master/LICENSE
mirror.gcr.io/aquasec/trivy-operator:
license: Apache-2.0
licenseLink: https://github.com/aquasecurity/trivy-operator/blob/main/LICENSE
Expand Down
4 changes: 2 additions & 2 deletions .github/trusted_registries.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,6 @@ docker.io:
kubernetes-reflector: ALL_TAGS
fluxcd: ALL_IMAGES
grafana: ALL_IMAGES
hjacobs:
kube-janitor: ALL_TAGS
memcached: ALL_TAGS
otel:
opentelemetry-collector-contrib: ALL_TAGS
Expand All @@ -41,6 +39,8 @@ ghcr.io:
kyverno-cli: ALL_TAGS
readiness-checker: ALL_TAGS
teutonet: ALL_IMAGES
twin:
k8s-ttl-controller: ALL_TAGS
mirror.gcr.io:
aquasec: ALL_IMAGES
quay.io:
Expand Down
15 changes: 15 additions & 0 deletions charts/base-cluster/README.md.gotmpl
Original file line number Diff line number Diff line change
Expand Up @@ -415,4 +415,19 @@ For this we switched from the distributed deployment to the single binary deploy
Because of this you need to migrate to the `.monitoring.tracing.<field>` instead
of `.monitoring.tracing.ingester.<field>`

### 11.x.x -> 12.0.0

`kube-janitor` is replaced by `ttl-controller` ([k8s-ttl-controller](https://github.com/TwiN/k8s-ttl-controller)).

If you had `kube-janitor.enabled: true`, change it to:
Comment thread
cwrau marked this conversation as resolved.

```yaml
ttl-controller:
enabled: true
```

Resources annotated with `janitor/ttl` must be re-annotated to `k8s-ttl-controller.twin.sh/ttl`.
Duration values (e.g. `1h`, `7d`) transfer as-is. However, `janitor/expires` absolute timestamps and
the `forever` value are not supported by `k8s-ttl-controller` and must be removed or replaced.

Comment thread
cwrau marked this conversation as resolved.
{{ .Files.Get "values.md" }}
2 changes: 1 addition & 1 deletion charts/base-cluster/ci/artifacthub-values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ monitoring:
enabled: true
securityScanning:
enabled: true
kube-janitor:
ttl-controller:
enabled: true
descheduler:
enabled: true
Expand Down
25 changes: 0 additions & 25 deletions charts/base-cluster/templates/global/kube-janitor.yaml

This file was deleted.

30 changes: 30 additions & 0 deletions charts/base-cluster/templates/global/ttl-controller.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
{{- if dig "ttl-controller" "enabled" false .Values.AsMap -}}
apiVersion: helm.toolkit.fluxcd.io/v2
kind: HelmRelease
metadata:
name: ttl-controller
namespace: kube-system
labels: {{- include "common.labels.standard" $ | nindent 4 }}
app.kubernetes.io/component: ttl-controller
spec:
chart:
spec: {{- include "base-cluster.helm.chartSpec" (dict "repo" "twin" "chart" "k8s-ttl-controller" "context" $) | nindent 6 }}
interval: 1h
driftDetection:
mode: enabled
values:
priorityClassName: cluster-components
image:
repository: {{ printf "%s/twin/k8s-ttl-controller" ($.Values.global.imageRegistry | default "ghcr.io") }}
Comment thread
cwrau marked this conversation as resolved.
Comment thread
cwrau marked this conversation as resolved.
resources:
limits:
memory: 1Gi
postRenderers:
- kustomize:
patches:
- target:
kind: Deployment
patch: |
- op: remove
path: /spec/template/spec/dnsPolicy
{{- end -}}
2 changes: 1 addition & 1 deletion charts/base-cluster/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -1632,7 +1632,7 @@
},
"additionalProperties": false
},
"kube-janitor": {
"ttl-controller": {
"type": "object",
"properties": {
"enabled": {
Expand Down
13 changes: 5 additions & 8 deletions charts/base-cluster/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -198,14 +198,11 @@ global:
charts:
velero: 7.2.2
condition: '{{ ne (.Values.backup.provider).velero nil }}'
kube-janitor:
url: https://github.com/teutonet/kube-janitor
twin:
url: https://twin.github.io/helm-charts
charts:
kube-janitor:
path: unsupported/helm
tag: 23.7.0
type: git
condition: '{{ dig "kube-janitor" "enabled" false .Values.AsMap }}'
k8s-ttl-controller: 0.4.0
condition: '{{ dig "ttl-controller" "enabled" false .Values.AsMap }}'
authentication:
grafana:
authenticationPath: /protocol/openid-connect/auth
Expand Down Expand Up @@ -493,7 +490,7 @@ backup:
cpu: 500m
memory: 500Mi

kube-janitor:
ttl-controller:
enabled: false

tetragon:
Expand Down
Loading