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

Update periodic-links-checks, add maxHistory for Helm runner #531

Merged
merged 3 commits into from
Oct 11, 2021

Conversation

mszostok
Copy link
Member

@mszostok mszostok commented Oct 11, 2021

Description

Currently on cluster, we have 232 Helm release secrets. Unfortunately, the Helm runner upgrades Helm release with maxHistory parameters set to 0 which means no history limit. This causes, kubed to crash as we have limit to 100mb and kubed to sync Secrets and ConfigMaps fetch all of them. Unfortunately, we cannot narrow down the fetched secrets.

Running helm upgrade cmd locally reduce the history to 3 latest releases:

 ▲ helm upgrade cert-manager ./deploy/kubernetes/charts/cert-manager -n capact-system --reuse-values --history-max 3
 ▲ helm history cert-manager
REVISION	UPDATED                 	STATUS    	CHART                     	APP VERSION	DESCRIPTION
32      	Mon Oct  4 14:20:29 2021	superseded	cert-manager-0.5.0-f6cea95	           	Upgrade complete
33      	Fri Oct  8 08:39:30 2021	superseded	cert-manager-0.5.0-f279dd9	           	Upgrade complete
34      	Mon Oct 11 12:43:03 2021	deployed  	cert-manager-0.5.0        	           	Upgrade complete

Changes proposed in this pull request:

  • Update periodic-links-checks.yaml to run only on weekdays
  • Add option to specify MaxHistory for Helm runner (default 10)

Related issue(s)

@mszostok mszostok added area/ci Relates to CI enhancement New feature or request labels Oct 11, 2021
@mszostok mszostok added this to the 0.6.0 milestone Oct 11, 2021
@mszostok mszostok changed the title Update periodic-links-checks.yaml Update periodic-links-checks, add maxHistory for Helm runner Oct 11, 2021
@mszostok mszostok merged commit 1c1917a into main Oct 11, 2021
@mszostok mszostok deleted the update-cron branch October 11, 2021 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Relates to CI enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants