Skip to content
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

Reduce ReplicaSets history limit #400

Merged
merged 3 commits into from
Nov 14, 2024
Merged

Conversation

ffilippopoulos
Copy link
Member

No description provided.

@ffilippopoulos ffilippopoulos requested a review from a team as a code owner November 13, 2024 12:26
@ffilippopoulos ffilippopoulos force-pushed the deployment-revision-limit branch from da69e55 to fcc440f Compare November 13, 2024 13:26
hectorhuertas
hectorhuertas previously approved these changes Nov 13, 2024
@ffilippopoulos ffilippopoulos force-pushed the deployment-revision-limit branch from 7e70588 to dab5a9c Compare November 13, 2024 14:02
Copy link

Post kustomize build diff:

diff --git a/root-manifests/build-output b/built-manifests/build-output
index ae30615..b4be581 100644
--- a/root-manifests/build-output
+++ b/built-manifests/build-output
@@ -548,6 +548,14 @@ rules:
       - list
       - update
       - create
+  - apiGroups:
+      - apps
+    resources:
+      - deployments
+    verbs:
+      - get
+      - list
+      - update
 ---
 aggregationRule:
   clusterRoleSelectors:

=============================================
k8s objects: 0 to add, 0 to destroy
1 changed hunks

@ffilippopoulos ffilippopoulos merged commit 6135a36 into master Nov 14, 2024
3 checks passed
@ffilippopoulos ffilippopoulos deleted the deployment-revision-limit branch November 14, 2024 10:16
ffilippopoulos added a commit that referenced this pull request Nov 14, 2024
This reverts commit 6135a36.
The configuration/functionality introduced by this one was ineffective
ffilippopoulos added a commit that referenced this pull request Nov 14, 2024
This reverts commit 6135a36.
The configuration/functionality introduced by this one was ineffective
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants