-
Notifications
You must be signed in to change notification settings - Fork 30
add rcon-rest-api container #57
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
Conversation
gonna add new livenessprobes later. |
same as in my other pr...
i dont even know what this is talking about. ^^ |
oi. |
as known:
|
@Kariton there are some conflicts due to the port fixer merge, do you want to update this and I'll begin QA on this as well? |
hm. github merge resolver merges main onto PR branch. |
i think disabling tls per default is better because it requires further configuration anyways. |
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.
Left a comment about making the values rconAPI
Also left comment for chart version |
nekomeowww/factorio-rcon-api
This is a fairly new project i discovered by chance.
hope you dont mind adding a additional (optional) container which adds so much value (or has the potential to do so).
some quick examples: