woodpecker-ci / helm

This repo contains the helm charts of the Woodpecker project.
Apache License 2.0
20 stars 15 forks source link

Server pod errors out on fresh installation: could not setup service manager: forge not configured #202

Open kastl-ars opened 3 weeks ago

kastl-ars commented 3 weeks ago

I just started playing around with woodpecker on Kubernetes (k3s in my case).

However even without any user-provided values the current chart version leads to failing pods in my tests:

$ helm install woodpecker woodpecker/woodpecker
[...]
$ kubectl logs woodpecker-server-0
{"level":"info","time":"2024-06-18T08:37:45Z","message":"log level: info"}
{"level":"fatal","error":"can't setup globals: could not setup service manager: forge not configured","time":"2024-06-18T08:37:45Z","message":"error running server"}
$

Is there something that I need to prepare before installation? If so, that step is missing in the README (or I have completely missed it)...

Kind Regards, Johannes

pat-s commented 3 weeks ago

As the error message tells you: it seems like you did not configure a forge (= Git Provider). WP won't start without it and it can't infer your git backend.

kastl-ars commented 3 weeks ago

Thanks for the fast reply!

If setting a forge is required it should be documented in the README and the values.yaml. (If I find time next week I can come up with a PR).

Also, unless I am missing something, neither the README.md nor the values.yaml contain any mention of the word forge. Which makes it extra difficult to find what there needs to configured.

(This might be obvious to someone running woodpecker in- and outside kubernetes for a long time, but for someone like me, just starting out with the helm chart, this is tough...)

pat-s commented 3 weeks ago

Yeah, I understand. It is the "classical" discussion of whether to provide non-meaningful defaults to avoid a "default" startup error or to require user input upfront.

This page page could be a good start point to link the minimum config requirements to get going without any prior knowledge. It could reference Forges and then be linked into the README of the helm chart?

Happy to take your suggestions which places might be best suited to place backrefs!

anbraten commented 3 weeks ago

I think a good way to start is to improve the guide "How to setup my own instance" in the docs and link this in the helm readme / comments.

https://github.com/woodpecker-ci/woodpecker/pull/3803

kastl-ars commented 2 weeks ago

Yeah, I understand. It is the "classical" discussion of whether to provide non-meaningful defaults to avoid a "default" startup error or to require user input upfront.

This page page could be a good start point to link the minimum config requirements to get going without any prior knowledge. It could reference Forges and then be linked into the README of the helm chart?

Happy to take your suggestions which places might be best suited to place backrefs!

Thanks, I'll try to find some time to dig into this next week and report back. Maybe I can come up with a PR to add some things to the README or the values.yaml.