Skip to content

Knative-Eventing: going to production #6139

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
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
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,19 @@ spec:
values:
sourceRoot: components/knative-eventing
environment: staging
clusterDir: ""
clusterDir: base
- list:
elements: []
elements:
- nameNormalized: kflux-ocp-p01
values.clusterDir: kflux-ocp-p01
- nameNormalized: stone-stage-p01
values.clusterDir: stone-stage-p01
- nameNormalized: stone-prod-p01
values.clusterDir: stone-prod-p01
- nameNormalized: stone-prod-p02
values.clusterDir: stone-prod-p02
- nameNormalized: kflux-prd-rh02
values.clusterDir: kflux-prd-rh02
template:
metadata:
name: knative-eventing-{{nameNormalized}}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,6 @@ metadata:
name: kubearchive
$patch: delete
---
# KubeArchive not yet ready to go to production
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
name: knative-eventing
$patch: delete
---
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,6 @@ metadata:
name: kubearchive
$patch: delete
---
# KubeArchive not yet ready to go to production
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
name: knative-eventing
$patch: delete
---
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,6 @@ metadata:
name: kubearchive
$patch: delete
---
# KubeArchive is not yet installed here
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
name: knative-eventing
$patch: delete
---
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
Expand Down
132 changes: 132 additions & 0 deletions components/knative-eventing/production/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,132 @@
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- https://github.com/knative/eventing/releases/download/knative-v1.15.3/eventing.yaml?timeout=90s

patches:
# kube-lint fixes
- patch: |-
apiVersion: apps/v1
kind: Deployment
metadata:
name: eventing-controller
namespace: knative-eventing
spec:
template:
spec:
containers:
- name: eventing-controller
resources:
requests:
cpu: 100m
memory: 100Mi
limits:
cpu: 150m
memory: 200Mi

- patch: |-
apiVersion: apps/v1
kind: Deployment
metadata:
name: imc-controller
namespace: knative-eventing
spec:
template:
spec:
containers:
- name: controller
resources:
requests:
cpu: 100m
memory: 100Mi
limits:
cpu: 150m
memory: 200Mi

- patch: |-
apiVersion: apps/v1
kind: Deployment
metadata:
name: imc-dispatcher
namespace: knative-eventing
spec:
template:
spec:
containers:
- name: dispatcher
resources:
requests:
cpu: 100m
memory: 100Mi
limits:
cpu: 150m
memory: 200Mi

# Requests are cpu 100m and memory 100Mi by default
# This is QoS Guaranteed (150m, 200Mi) because it was OOMKilled on Burstable (100m, 100Mi)
- patch: |-
apiVersion: apps/v1
kind: Deployment
metadata:
name: mt-broker-controller
namespace: knative-eventing
annotations:
ignore-check.kube-linter.io/unset-memory-requirements: >
"Startup traverses all the cluster. Its a known problem, being solved on upstream.
See https://github.com/knative/eventing/pull/8418"
spec:
template:
spec:
containers:
- name: mt-broker-controller
resources:
requests:
cpu: 150m
memory: 400Mi
limits:
cpu: 150m

# Requests are cpu 100m and memory 100Mi by default
- patch: |-
apiVersion: apps/v1
kind: Deployment
metadata:
name: mt-broker-filter
namespace: knative-eventing
spec:
template:
spec:
containers:
- name: filter
resources:
limits:
cpu: 150m
memory: 200Mi

# Requests are cpu 100m and memory 100Mi by default
- patch: |-
apiVersion: apps/v1
kind: Deployment
metadata:
name: mt-broker-ingress
namespace: knative-eventing
spec:
template:
spec:
containers:
- name: ingress
resources:
limits:
cpu: 150m
memory: 200Mi

# This was causing issues with kube-linter and I didn't want
# to increase its replicas to 2, so I deleted it instead
- patch: |-
$patch: delete
apiVersion: policy/v1
kind: PodDisruptionBudget
metadata:
name: eventing-webhook
namespace: knative-eventing
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- ../base
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- ../base
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- ../base
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
- ../../base