8000 Helm chart by jacobweinstock · Pull Request #35 · tinkerbell/tinkerbell · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Helm chart #35

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Mar 6, 2025
Merged

Helm chart #35

merged 8 commits into from
Mar 6, 2025

Conversation

jacobweinstock
Copy link
Member
@jacobweinstock jacobweinstock commented Feb 22, 2025 •
edited
Loading

Description

This is the first iteration of a working Helm chart. Because of the single binary Tinkerbell stack, we only need a single Helm chart with no other Helm chart dependencies. Example helm install command:

trusted_proxies=$(kubectl get nodes -o jsonpath='{.items[*].spec.podCIDR}' | tr ' ' ',')
LB_IP=192.168.2.116
HOOK_LB_IP=192.168.2.117
helm install tinkerbell tinkerbell/ --create-namespace --namespace tinkerbell --wait --set "trustedProxies={${trusted_proxies}}" --set "publicIP=$LB_IP" --set "deployment.envs.smee.ipxeHttpScriptOsieURL=http://$HOOK_LB_IP:8080" --set "hookos.service.loadBalancerIP=$HOOK_LB_IP"

Why is this needed

Fixes: #

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

@jacobweinstock jacobweinstock force-pushed the helm-chart branch 3 times, most recently from 76c8b59 to 3b949fc Compare March 3, 2025 23:10
@jacobweinstock jacobweinstock marked this pull request as ready for review March 4, 2025 19:49
@jacobweinstock
Add Helm gitignores:
8b40ba3 
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock
Add initial Helm chart:
62af933 
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock
Kube apply CRDs:
389b8dc 
Instead of Kube create. This should add
some resilience.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock
Remove CRDs Helm chart:
1e0f378 
The CLI applies CRDs so we don't need
a chart for them.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock
Add manifest generation to ci checking
cfef94d 
Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock
Add ci check for new files:
b4102e1 
We shouldn't have any untracked files
in CI after a build or a test run, etc.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock
Don't generate CRDs into the Helm dir:
97b6ea0 
CRDs are installed via the CLI now.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock force-pushed the helm-chart branch 2 times, most recently from 4971a92 to 1193d1f Compare March 6, 2025 21:19
@jacobweinstock
MVP working Helm chart:
e145100 
The architecture is different from the
existing microservice helm chart. Nginx
doesn't front any Tinkerbell services.
A file server for HookOS artifacts and kube-vip
are optional services now.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>
@jacobweinstock jacobweinstock merged commit c53ee62 into tinkerbell:main Mar 6, 2025
5 checks passed
@jacobweinstock jacobweinstock deleted the helm-chart branch March 6, 2025 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
helm-chart
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jacobweinstock
0