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
fluentd
as Docker log driver for Wings
#5008
Comments
Can confirm. What is happening is when wings starts up, it starts with an empty Configuration struct that is then filled with the defaults and THEN we overlay the data from the This overwrites any values in the struct that exist in the A possible fix might be to re-order the operations. Load the |
@promarcel If I provided you a source code patch that does what I described above, would you be willing to test, and do you have the ability to apply the patch and compile ? |
@danny6167 Thanks for your reply, and sorry for my late reply. |
Hey @promarcel - I have this branch here that has the required changes. |
Note: If you're not passing any options, you will need to supply an empty value to log_config:
type: fluentd
config: {} |
Current Behavior
Hey everyone, I am currently running into a more specific problem with the Wings Control Plane.
I am trying to change the log driver of Wings from
/etc/pterodactyl/config.yml
. The default value local works without problems, as doesjson-file
.However, I would like to use
fluentd
at this point (already configured system-wide via the daemon.json of Docker). However, the control plane overwrites this setting from the system with its options from the configuration for containers that it creates. I can set the type, but Wings keeps throwing the default options from the source code (compress
,max-file
,max-size
andmode
) into the config value. Passing an empty object{}
is ignored and adds the above values. Exactly the same if I manually set atag
(this is merged with the default values).Example from
/etc/pterodactyl/config.yml
:Once, servers are started on this node, they are failing due to the environment has options, the
fluentd
log driver doesn't know because the default options are being merged into theconfig:
array.Does anyone of you have an idea about this?
Expected Behavior
Using the Docker log driver
fluentd
to allow logging tools like td-agent, fluentbit and other to catch logs to a log storage solution.Steps to Reproduce
Adjust the configuration of Wings to the example provided at "Current Behavior".
Panel Version
1.11.5
Wings Version
1.11.8
Games and/or Eggs Affected
No response
Docker Image
No response
Error Logs
No response
Is there an existing issue for this?
The text was updated successfully, but these errors were encountered: