search

logstash-plugins / logstash-output-s3

Apache License 2.0
58 stars 151 forks source link

feat: support ONEZONE_IA #213

Closed max-wittig closed 4 years ago

max-wittig commented 4 years ago

@karenzone @colinsurprenant Could you please have a look?

colinsurprenant commented 4 years ago

Thanks for your contribution @max-wittig - this seems like a no-brainer to add! LGTM. To be able to push a new plugin version we also need to bump the version (to 4.2.0) in logstash-output-s3.gemspec and add to the CHANGELOG.md. Would you be able to also add this to the PR?

Once @karenzone has reviewed docs related changes we'll merge and push the new plugin version.

max-wittig commented 4 years ago

@colinsurprenant Thanks for the review 👍. I've pushed a new commit with the version bump.

colinsurprenant commented 4 years ago

v4.2.0 published! thanks @max-wittig @karenzone !

max-wittig commented 4 years ago

Thanks @colinsurprenant :+1:

Any idea, when this new version will be used by ELK?

karenzone commented 4 years ago

It will be picked up in our next minor release. You can update it already. Thanks again for contributing the updates!