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

Fixes system kustomization reconciling if no profiles installed #1396

Merged
merged 8 commits into from
Feb 8, 2022

Conversation

foot
Copy link
Contributor

@foot foot commented Feb 7, 2022

  • Kustomization controller complains about a missing profiles.yaml

What changed?

Add an empty profiles.yaml file so that the system kustomzation can reconcile.

{
  "level": "error",
  "ts": "2022-02-07T11:10:21.828Z",
  "logger": "controller.kustomization",
  "msg": "Reconciliation failed after 31.707953ms, next try in 1m0s",
  "reconciler group": "kustomize.toolkit.fluxcd.io",
  "reconciler kind": "Kustomization",
  "name": "kind-test-upgrade-system",
  "namespace": "wego-system",
  "revision": "main/ae43a6151d0f4c368a3abc352f6349837b91561e",
  "error": "kustomize build failed: accumulating resources: accumulation err='accumulating resources from 'profiles.yaml': open /tmp/kind-test-upgrade-system4030199120/.weave-gitops/clusters/kind-test-upgrade/system/profiles.yaml: no such file or directory': evalsymlink failure on '/tmp/kind-test-upgrade-system4030199120/.weave-gitops/clusters/kind-test-upgrade/system/profiles.yaml' : lstat /tmp/kind-test-upgrade-system4030199120/.weave-gitops/clusters/kind-test-upgrade/system/profiles.yaml: no such file or directory",
}

Why?

Easier to add a profiles.yaml than to add it into the kustomization.yaml dynamically.

How did you test it?

Locally.

- Kustomization controller complains about a missing profiles.yaml
@foot foot merged commit 83ef33c into main Feb 8, 2022
@foot foot deleted the adds-dummy-profiles-yaml branch February 8, 2022 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants