search

itzg / minecraft-server-charts

MIT License
267 stars 142 forks source link

How to configure velocity #165

Closed arno4000 closed 1 year ago

arno4000 commented 1 year ago

According to https://github.com/itzg/minecraft-server-charts/blob/master/charts/minecraft-proxy/values.yaml#L127 the minecraft-proxy chart currently only supports config files, which are yaml. But velocity uses toml for its config file and velocity is supported by the chart.

So my question is, how can I specify the config for velocity in the values.yaml of the minecraft-proxy chart?

itzg commented 1 year ago

Yeah it seems to do extraneous processing to take the text block and turn it into yaml

https://github.com/itzg/minecraft-server-charts/blob/2406a8c2c6c075cf11bde390e8ac8ea006b49e4a/charts/minecraft-proxy/templates/configmap.yaml#L12

Removing the toYaml would probably fix it.

Since I'm not personally working on this repo anymore, then a PR would be great.

arno4000 commented 1 year ago

It actually seems that the toYaml is not the problem. Instead, the itzg/bungeecord image seems to have a bug when mounting the velocity.toml to /server. I created a issue for this problem at https://github.com/itzg/docker-bungeecord/issues/133.