Skip to content

Add soft Pod anti-affinity to IMC dispatchers#4640

Merged
knative-prow-robot merged 1 commit into
knative:masterfrom
antoineco:imc-dispatcher-affinity
Dec 9, 2020
Merged

Add soft Pod anti-affinity to IMC dispatchers#4640
knative-prow-robot merged 1 commit into
knative:masterfrom
antoineco:imc-dispatcher-affinity

Conversation

@antoineco
Copy link
Copy Markdown
Contributor

Proposed Changes

  • Add soft Pod anti-affinity to IMC dispatchers

Dispatchers consume a fair amount of CPU and may starve on CPU time under load, as observed in the graph below (load-test at 24K then 27K events/sec against 5 replicas). Defining requests and limits would be too opinionated because we don't know where Knative is being deployed, but we can at least try to ensure dispatchers are spread across nodes.

multi-dispatcher

Release Note

Docs

@google-cla google-cla Bot added the cla: yes Indicates the PR's author has signed the CLA. label Dec 9, 2020
@knative-prow-robot knative-prow-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 9, 2020
@knative-prow-robot knative-prow-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Dec 9, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 9, 2020

Codecov Report

Merging #4640 (ed32d34) into master (ca2def5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4640   +/-   ##
=======================================
  Coverage   81.31%   81.31%           
=======================================
  Files         290      290           
  Lines        8176     8176           
=======================================
  Hits         6648     6648           
  Misses       1128     1128           
  Partials      400      400           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca2def5...a40a0c9. Read the comment docs.

@devguyio
Copy link
Copy Markdown
Contributor

devguyio commented Dec 9, 2020

That sounds great, thanks for the great PR description!

/assign
/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Dec 9, 2020
@zhongduo
Copy link
Copy Markdown
Contributor

zhongduo commented Dec 9, 2020

I like the idea. Also the picture is amazing. Thanks for the great work.

/lgtm

@tayarani
Copy link
Copy Markdown
Contributor

tayarani commented Dec 9, 2020

/lgtm

@knative-prow-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: antoineco, pierDipi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 9, 2020
@knative-prow-robot knative-prow-robot merged commit 4f49cc7 into knative:master Dec 9, 2020
@antoineco antoineco deleted the imc-dispatcher-affinity branch December 9, 2020 15:43
matzew pushed a commit to matzew/eventing that referenced this pull request Dec 9, 2020
matzew pushed a commit to matzew/eventing that referenced this pull request Dec 9, 2020
openshift-merge-robot pushed a commit to openshift/knative-eventing that referenced this pull request Dec 9, 2020
* Add soft Pod anti-affinity to IMC dispatchers (knative#4640)

* make RELEASE=v0.19.2 generate-release

Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>

Co-authored-by: Antoine Cotten <hello@acotten.com>
openshift-merge-robot pushed a commit to openshift/knative-eventing that referenced this pull request Dec 9, 2020
* Add soft Pod anti-affinity to IMC dispatchers (knative#4640)

* make RELEASE=v0.19.2 generate-release

Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>

Co-authored-by: Antoine Cotten <hello@acotten.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants