Skip to content

Refactor Helm chart sync workflows to use reusable templates#45

Open
EvantAiI wants to merge 3 commits intonetbirdio:mainfrom
EvantAiI:feature/add-chart-sync-workflows
Open

Refactor Helm chart sync workflows to use reusable templates#45
EvantAiI wants to merge 3 commits intonetbirdio:mainfrom
EvantAiI:feature/add-chart-sync-workflows

Conversation

@EvantAiI
Copy link
Copy Markdown

Description:

This PR refactors our GitHub Actions workflows for syncing Helm charts to make them more modular and DRY.

What changed:

  • Created a reusable template (.github/workflows/sync-chart-template.yml) that handles the core sync logic (version checks, upstream cloning, rsync, PR creation, and auto-merge).
  • Refactored sync-operator-charts.yml to use the new template. Instead of one monolithic script, it now runs two distinct jobs: one for kubernetes-operator and another for netbird-operator-config.
  • Added sync-netbird-chart.yml to sync the main netbird chart from upstream using the new template.

Good to know:

  • Individual PRs: Syncs will now generate separate PRs per chart instead of one massive global PR. This makes diffs much easier to review and allows for targeted rollbacks if something breaks.
  • Security permissions remain untouched (secrets: inherit is used to pass down the tokens to the template).
  • The auto-merge behavior is preserved.

girardy added 3 commits March 24, 2026 14:55
- Extract chart synchronization logic into a reusable workflow (`sync-chart-template.yml`)
- Update `sync-operator-charts.yml` to use the new template, splitting the sync into two distinct jobs
- Add new `sync-netbird-chart.yml` workflow to sync the main netbird chart
@EvantAiI
Copy link
Copy Markdown
Author

Pr it's ready

For netbird and dashboard action will fetch the docker hub registry if anyone has a better idea please be free to update my code.

For kubernetes-operator and config actions will sync from the upstream repo.

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.

1 participant