Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fine-tuning Flux configuration#196

Merged
kvaps merged 1 commit intocozystack:maincozystack/cozystack:mainfrom
kingdonb:fix-195kingdonb/cozystack:fix-195Copy head branch name to clipboard
Jul 4, 2024
Merged

Fine-tuning Flux configuration#196
kvaps merged 1 commit intocozystack:maincozystack/cozystack:mainfrom
kingdonb:fix-195kingdonb/cozystack:fix-195Copy head branch name to clipboard

Conversation

@kingdonb
Copy link
Member

@kingdonb kingdonb commented Jul 3, 2024

Fix #195

Don't set the interval so short on HelmReleases, with this many HelmReleases that really hamstrings the control plane.

Also, copy the install/upgrade remediation config from system packages to the Kubernetes templates for addon packages (cilium, flux, ingress) - in my testing the ingress-nginx chart fails every time the first time. Maybe that should be filed as a separate issue, I haven't looked into detail, it is some issue related to a secret not being created, I think it said something related to an admission controller.

Looks as though it's a conflict with being installed at the same time as the cert-manager addon.

Don't set the `interval` so short on HelmReleases, with this many
HelmReleases that really hamstrings the control plane.

Also, copy the install/upgrade remediation config from system packages
to the Kubernetes templates for addon packages (cilium, flux, ingress) -
in my testing the ingress-nginx chart fails every time the first time.
Maybe that should be filed as a separate issue, I haven't looked into
detail, it is some issue related to a secret not being created.

I assume it's a conflict with being installed at the same time as the
cert-manager addon (but I'm not sure and I can't look into it now!)

Signed-off-by: Kingdon Barrett <kingdon+github@tuesdaystudios.com>
Copy link
Member

@kvaps kvaps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@kvaps kvaps merged commit 2ab9a38 into cozystack:main Jul 4, 2024
@kingdonb kingdonb deleted the fix-195 branch July 4, 2024 00:38
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.

(pre-0.8) add-ons, cilium should have retries configuration

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.