-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Conversation
edc8a12
to
9db9cec
Compare
The bitnami images (e.g. bitnami/ghost ) from the flux-get-started repo have too many tags and we get rate-limited. This makes the test timeout. I am considering to create and use a branch in flux-get-started guide, pointing to images from a registry local to the e2e tests. However, we would still need to load the redis/ghost/mongodb images in the local registry, which will take time. Another option is to create a new example altogether, with tiny images. I wouldn't want to maintain that though. @stefanprodan @hiddeco any other ideas? |
We can build empty |
Ah, it seems I can use an insecure registry mirror kubernetes-sigs/kind#110 (comment) This will allow us to use the flux-started-guide unchanged. However this implies changing the k8s node configuration, which I can do for I think I can l live with that. |
It's not ideal though, because I don't want to use Kind when working on the tests locally |
Here is an intermediate solution. Only make Flux use a local registry without changing the k8s configuration. Kubernetes will pull images in whatever way it used to but Flux will query the local registry. When spawning Kind we can also use a mirror. In fact, if we don't change the get-started example we also need to modify Flux's configuration. |
efe2cb1
to
131a6a9
Compare
I bit the bullet and I added a local registry to the test. Note that the registry is only used by Flux to scan images (and not by the cluster). Adding an |
We may also want to change |
ba20a51
to
03f8c52
Compare
Test both updating images of a Deployment and a HelmRelease, manually and automatically.
088b131
to
74295a8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work Fons, thanks 🥇
e2c529f
to
332b8cb
Compare
All tests until now run without using the registry scanner, so I thought it was about time to have an end-to-end test for image releases (which is a fundamental Flux feature)
We probably shouldn't depend on external registries (particularly on(I have added a local registry for Flux)stefanprodan/podinfo
since it is often updated) for tests. I plan to add an internal registry to the tests, but that will come later. If this test becomes too flakey I am happy to disable it until then.Addresses part of #2593 (adds a test covering
fluxctl release
).TODO:
HelmRelease
image updates