search

vmware / fluent-plugin-vmware-loginsight

Fluentd plugin to forward logs to VMware Aria Operations for Logs
Other
10 stars 13 forks source link

fix: remove class variable inside plugin #23

Closed Cryptophobia closed 3 years ago

Cryptophobia commented 3 years ago

Signed-off-by: Anton Ouzounov aouzounov@vmware.com

@alex-vmw @slimm609

From logs from https://github.com/vmware/kube-fluentd-operator:

2021-04-03 00:26:13 +0000 [info]: Reloading new config
2021-04-03 00:26:13 +0000 [error]: Failed to reload config file: Unreloadable plugin plugin: vmware_loginsight, plugin_id: object:c2c4, class_name: Fluent::VmwareLoginsightOutput)
2021-04-03 00:26:32 +0000 [info]: Reloading new config
2021-04-03 00:26:32 +0000 [error]: Failed to reload config file: Unreloadable plugin plugin: vmware_loginsight, plugin_id: object:c378, class_name: Fluent::VmwareLoginsightOutput)

The output plugin does not allow reloading via the fluentd gracefulreload api endpoint via signal SIGUSR2

vmwclabot commented 3 years ago

@Cryptophobia, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <john.doe@email.org> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

mohitevishal commented 3 years ago

This change is cancelled in favor of https://github.com/vmware/fluent-plugin-vmware-loginsight/pull/24