add ability to add additional nuocollector scripts.

[butson]

It would be useful to be able to add additional configuration to nuocollector besides the insights output plugin. We want to integrate with sumologic remotely and influx/insights locally.

[sivanov-nuodb]

I'm a bit confused about why we need to support arbitrary nuocollector plugins in the insights chart.
This is already supported by the admin and database charts. Please see nuocollector.plugins.admin and nuocollector.plugins.database variables. Is there a reason not to declare the Sumologic output plugin when deploying the admin and the database?

[butson]

If you put it in nuocollector then you can’t deploy and undeploy without upgrading the chart which then will stop and restart the engine. The design as is, is to allow changes of collector without restarting engine as detection of new plugins only restart the collector container. This was the whole reason for using the sidecar to watch for changes to configmaps.

[sivanov-nuodb]

If you put it in nuocollector then you can’t deploy and undeploy without upgrading the chart which then will stop and restart the engine.

AFIAK the admin and database charts also just register ConfigMap resources for the customer plugins which doesn't require a restart of the engines. A rolling upgrade is only required if you enable the previously disabled nuocollector.