It is INFRA, see INFRA-17091 for example.

On October 12, 2018 at 20:47:24, zeo...@gmail.com (zeo...@gmail.com) wrote:

So it seems that the last commit before the 0.2 release of
metron-bro-plugin-kafka broke the one basic unit test that we had. Since
metron 0.6.0 pins to 0.1
<
https://github.com/apache/metron/blob/Metron_0.6.0/metron-deployment/ansible/roles/bro/vars/main.yml#L30>

this wouldn't cause an obvious issue if you spun up the sensors in
full-dev, and even if we were to point metron to 0.2 of the plugin we
wouldn't have necessarily seen the issue because we do a --force
<
https://github.com/apache/metron/blob/Metron_0.6.0/metron-deployment/ansible/roles/bro/tasks/metron-bro-plugin-kafka.yml#L35>

to remove interaction.

After we get a chance to review/merge a recent PR
<https://github.com/apache/metron-bro-plugin-kafka/pull/2>, we should have
a few more 'actual' unit tests, and I would like to get travis setup for
the repo <https://travis-ci.org/apache/metron-bro-plugin-kafka> to actually
check those (PR incoming) so it's less likely this sort of thing would
happen again. However, it doesn't appear I have the right permissions to
be able to get travis moving on my own. Does anybody else have the right
access, or is this an asf infrastructure ticket?

I'd also like us to consider doing a 0.3 release for the plugin so we
aren't distributing something that has broken tests. Thoughts on that?

Jon
-- 

Jon

Reply via email to