Demo app deploy failing on reconciliation

[jfmatth]

Describe the bug

• Deployed to MicroK8s per https://fluxcd.io/docs/guides/image-update
• Installed Flux as instructed by above page to support automation components
• After pulling down the podinfo-deployment.yaml, the reconcile fails

Steps to reproduce

• Install flux per https://fluxcd.io/docs/guides/image-update/#install-flux

flux bootstrap github \
  --components-extra=image-reflector-controller,image-automation-controller \
  --owner=$GITHUB_USER \
  --repository=flux-image-updates \
  --branch=main \
  --path=clusters/my-cluster \
  --read-write-key \
  --personal

• Deploy demo app https://fluxcd.io/docs/guides/image-update/#deploy-a-demo-app
• on Flux reconcile step I get a failure

 jfmatth@IICL00149  ~/development/flux-image-updates   main  flux reconcile kustomization flux-system --with-source

► annotating GitRepository flux-system in flux-system namespace
✔ GitRepository annotated
◎ waiting for GitRepository reconciliation
✔ fetched revision main/6fd69b58750084d37e87a02f7c58e7f65c946610
► annotating Kustomization flux-system in flux-system namespace
✔ Kustomization annotated
◎ waiting for Kustomization reconciliation
✗ Kustomization reconciliation failed: Deployment/podinfo namespace not specified, error: the server could not find the requested resource

Expected behavior

I would think it would deploy the podinfo app into my cluster

Screenshots and recordings

No response

OS / Distro

WSL2 attaching to MicroK8s on Ubuntu via kubectl

Flux version

flux: v0.18.3

Flux check

 jfmatth@IICL00149  ~/development/flux-image-updates   main  flux check
► checking prerequisites
✔ Kubernetes 1.21.5-3+83e2bb7ee39726 >=1.19.0-0
► checking controllers
✔ helm-controller: deployment ready
► ghcr.io/fluxcd/helm-controller:v0.12.0
✔ image-automation-controller: deployment ready
► ghcr.io/fluxcd/image-automation-controller:v0.15.0
✔ image-reflector-controller: deployment ready
► ghcr.io/fluxcd/image-reflector-controller:v0.12.0
✔ kustomize-controller: deployment ready
► ghcr.io/fluxcd/kustomize-controller:v0.15.5
✔ notification-controller: deployment ready
► ghcr.io/fluxcd/notification-controller:v0.17.1
✔ source-controller: deployment ready
► ghcr.io/fluxcd/source-controller:v0.16.0
✔ all checks passed
 jfmatth@IICL00149  ~/development/flux-image-updates   main 

Git provider

n/a

Container Registry provider

n/a

Additional context

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[somtochiama]

Hey @jfmatth , This is a breaking change from a new release of kustomize-controller fluxcd/kustomize-controller#426.
You can resolve this by adding a namespace to the podinfo-deployment.yaml file.
Thanks for pointing out, we will update the docs and examples to reflect this change

[jfmatth]

Thanks for the quick reply, I'll add the namespace information and see if that improves it, and do a PR against the docs to help everybody

…

On Wed, Oct 13, 2021, 2:18 PM Somtochiama ***@***.***> wrote: Hey @jfmatth <https://github.com/jfmatth> , This is a breaking change from a new release of kustomize-controller fluxcd/kustomize-controller#426 <fluxcd/kustomize-controller#426>. You can resolve this by adding a namespace to the podinfo-deployment.yaml file. Thanks for pointing out, we will update the docs and examples to reflect this change — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1939 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACGTMWAFWQBZR5JEW4LDFDUGXZTVANCNFSM5F57DP5A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[jfmatth]

Yea that did it :)

I added namespace to the metadata section and it worked as expected. Weird that it requires that, seems like a bad coupling, but oh well.

apiVersion: apps/v1
kind: Deployment
metadata:
  name: podinfo
  namespace: default
 ...

Created a PR against the Podinfo repo stefanprodan/podinfo#154