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

Confusing error message about successful image scan #356

Open
tun0 opened this issue Mar 10, 2023 · 3 comments
Open

Confusing error message about successful image scan #356

tun0 opened this issue Mar 10, 2023 · 3 comments

Comments

@tun0
Copy link

tun0 commented Mar 10, 2023

This is tad confusing:

jsonPayload: {
  ImageRepository: {2}
  controller: "imagerepository"
  controllerGroup: "image.toolkit.fluxcd.io"
  controllerKind: "ImageRepository"
  error: "Succeeded"
  level: "error"
  msg: "successful scan: found 8 tags"
  name: "huurwoningen-referrer-revenue"
  namespace: "flux-system"
  reconcileID: "a999f75e-e3de-40e6-a80b-f68095301532"
  ts: "2023-03-10T09:18:27.127Z"
}

The fact that it succeeded is an error? 🤷

It was followed by this:

jsonPayload: {
  ImageRepository: {2}
  controller: "imagerepository"
  controllerGroup: "image.toolkit.fluxcd.io"
  controllerKind: "ImageRepository"
  error: "timed out waiting for the condition"
  errorCauses: [1]
  level: "error"
  msg: "Reconciler error"
  name: "huurwoningen-referrer-revenue"
  namespace: "flux-system"
  reconcileID: "a999f75e-e3de-40e6-a80b-f68095301532"
  ts: "2023-03-10T09:18:27.316Z"
}
@tun0
Copy link
Author

tun0 commented Mar 10, 2023

See also this thread on Slack.

@darkowlzz
Copy link
Contributor

darkowlzz commented Mar 14, 2023

A fix for this is in #357.
We have an RC build based on v0.26.0/latest main to try this

ghcr.io/fluxcd/image-reflector-controller:rc-b89a023e@sha256:377a7bc1cf45b4245a82c21be149d8784c8cd74c5c1bc65188e68c28fcbd5dc0

Build logs https://github.com/fluxcd/image-reflector-controller/actions/runs/4417823595.
It should be compatible with v0.25.0 as well.
Please give it a try and let us know if the issue still persists.

@tun0
Copy link
Author

tun0 commented Mar 15, 2023

Deployed the RC build yesterday. Given the issue hasn't resurfaced in the last few days, and I don't know how to trigger it, validating the fix might prove to be a tad tricky 😉

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

No branches or pull requests

2 participants