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

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

Closed
kastl-ars opened this issue Jun 18, 2024 · 6 comments

Comments

@kastl-ars
Copy link

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
Copy link
Collaborator

pat-s commented Jun 18, 2024

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
Copy link
Author

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
Copy link
Collaborator

pat-s commented Jun 18, 2024

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
Copy link
Member

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.

woodpecker-ci/woodpecker#3803

@kastl-ars
Copy link
Author

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.

@pat-s
Copy link
Collaborator

pat-s commented Sep 13, 2024

Closing here as the upstream PR has been merged.

@pat-s pat-s closed this as completed Sep 13, 2024
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

3 participants