Validate Tests #29

Open
chuckbutler opened this Issue Oct 22, 2017 · 7 comments

Comments

Projects
None yet
3 participants
Collaborator

chuckbutler commented Oct 22, 2017

The tests to validate layer-filebeat are -ancient- and I don't think they've been used basically at all. We need some level of automated test suite in here so I can go into "satisfy the tests" mode to accept PR's with my limited time.

I'm calling all hands who care about the future of these charms and their longevity to lend a hand in contributing tests or test-harness setup.

Acceptance Criteria:

  • Has automated unit tests (where applicable)
  • Passes flake8 checks
  • Has at least a single/simple integration test to validate log shipping happens through
    • Directly to ES
    • Routed through Logstash

With those two primary use cases sorted, plus the project meta I feel like we'll have a good enough path of coverage that I can ship features sent in by the community.

Collaborator

chuckbutler commented Oct 22, 2017

@jamesbeedy @tbaumann can I lean on you to submit validation runs using the python juju framework to drive an automation test? (specifically wrt kafka) and/or a hand in setting up a test harness job using jenkins pipelines?

I didn't have much success running charm tests. But that is mostly to inadequacy of tools and documentation. With some pointers what to actually do, I'm sure I can get some stuff done.

Collaborator

chuckbutler commented Oct 22, 2017

I'll spec out the elasticsearch / filebeat tests sometime this week and follow up with what I come up with. I'm sure i'll find dragons too :)

Collaborator

chuckbutler commented Oct 28, 2017

Note to self: Take a look at this over the weekend. @jamesbeedy i saw some new elasticsearch work dropped in #juju this week. Are those in the store/available? Should I begin basing work off of that stack of charms?

@chuckbutler not in the store yet. I’ll cut a beta of the new elasticsearch charm this weekend. It uses unreleased charms.reactive, charmhelpers, and juju, so still a bit fresh. I think as long as you are running 2.3beta the built charm should work for you though. I’ll make note of this.

Collaborator

chuckbutler commented Oct 28, 2017

Solid, I'll give it a go on both then once i've got the tests sorted + charm details. No rush/pressure I just wanted to make sure I wasn't targeting something deprecated since i've been out of the loop for a good while.

“submit validation runs using the python juju framework to drive an automation test? (specifically wrt kafka) and/or a hand in setting up a test harness job using jenkins pipelines?” - trying to hook this all up for my org right now, possibly I can roll the beats charms into the mix when things are up and running.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment