Juju Charms Collection
logstash-forwarder package

Code review comment for lp:~woutervb/charms/trusty/logstash-forwarder/focal

  1. Trusty Tahr (14.04)
  2. focal
  3. Merge into trunk
Revision history for this message
Wouter van Bommel (woutervb) wrote :

> Some questions/typo regarding mako/jinja and casting to a list.
>
> Also, isn't filebeat what we want to use instead of logstash-forwarder now ?

Thanks for having a look, updated the comment and removed the list (the list was a 2to3 leftover).

Indeed we would like to go to filebeat, but the charm lacks some features used in the sca spec, so we now 'fixed' logstash-forwarder so we can at least rollout on focal for the services that support it.

« Back to merge proposal