Skip to content

Conversation

Kidswiss
Copy link
Contributor

@Kidswiss Kidswiss commented Oct 13, 2025

Due to ArgoCD being ArgoCD the very first sync with "server-side-apply" will hang.

This generates a race condition with kindev non-converged provisioning as we need some service account token secrets to generate kube-configs.

By applying secrets early we can ensure that they actually get deployed even if ArgoCD gets stuck.

Relates to: vshn/kindev#43

Checklist

  • The PR has a meaningful title. It will be used to auto generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • I run make e2e-test against local kindev and all checks passed
  • Link this PR to related issues or PRs.

@Kidswiss Kidswiss added the patch label Oct 13, 2025
@Kidswiss Kidswiss requested review from a team, TheBigLee, mdnix, mikeshootzz and zugao and removed request for a team October 13, 2025 11:32
Due to ArgoCD being ArgoCD the very first sync with "server-side-apply"
will hang.

This generates a race condition with kindev non-converged provisioning
as we need some service account token secrets to generate kube-configs.

By applying secrets early we can ensure that they actually get deployed
even if ArgoCD gets stuck.
@Kidswiss Kidswiss force-pushed the fix/wrong_vcluster_config branch from 15bc61c to b0eb468 Compare October 13, 2025 11:40
Copy link
Contributor

@zugao zugao left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants