Configure deployments to have no own revisionHistoryLimit

[SaschaSchwarze0]

Proposed Changes

Continuation of #15640

The revision history limit of a deployment are its replicasets. As Knative has its own concept of revisions and the way a deployment looks like for revision is defined by the revision, there is no reason to maintain a history of replicasets. Multiple replicasets can exist if Knative configuration is updated (config-deployment) or if Knative itself is updated (and the queue-proxy image changes). Having no revisionHistoryLimit on the Deployment saves a lot of space in the Kubernetes database.

This PR changes the deployments that Knative creates to always have revisionHistoryLimit set to 0.

Release Note

Kubernetes Deployments created for a Knative Revision are not configured with revisionHistoryLimit=0 which saves a lot of space in your Kubernetes database if you have a lot of Knative Services that are not updated while you regularly update Knative itself and pick up new queue-proxy images.

[knative-prow]

Hi @SaschaSchwarze0. Thanks for your PR.

I'm waiting for a knative member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.83%. Comparing base (c3f2bfe) to head (280b048).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #15698      +/-   ##
==========================================
+ Coverage   80.76%   80.83%   +0.06%     
==========================================
  Files         222      222              
  Lines       18025    18026       +1     
==========================================
+ Hits        14558    14571      +13     
+ Misses       3094     3084      -10     
+ Partials      373      371       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dprotaso]

/lgtm
/approve

@SaschaSchwarze0 thanks for the quick turn around

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, SaschaSchwarze0

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [dprotaso]

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

[dprotaso]

/ok-to-test