search

RackSec / desdemona

Data-backed security operations
Eclipse Public License 1.0
2 stars 7 forks source link

Swift plugin #129

Closed sirsean closed 8 years ago

sirsean commented 8 years ago

This adds an Onyx plugin for writing messages to a Swift cluster. It currently only connects to a local mimic instance, but that should be merely a configuration change to switch it over to real Cloud Files.

Because this is closer to our real data store, the temporary MySQL database has been removed. We no longer start it in docker-compose and we no longer attempt to connect to it. All SQL-related code has been removed.

codecov-io commented 8 years ago

Current coverage is 93.12%

Merging #129 into master will increase coverage by +1.12% as of 908a6ce

@@            master    #129   diff @@
======================================
  Files           17      17       
  Stmts          225     262    +37
  Branches        11      11       
  Methods          0       0       
======================================
+ Hit            207     244    +37
  Partial         11      11       
  Missed           7       7

Review entire Coverage Diff as of 908a6ce

Powered by Codecov. Updated on successful CI builds.

reaperhulk commented 8 years ago

jenkins, retest this please

lvh commented 8 years ago

The docker build appears to report the tests 4 times: https://jenkins.racksec.io/job/desdemona-docker-pr-builder/22/console

... but the new swift plugin tests don't appear to be a part of that.

sirsean commented 8 years ago

That is ... weird.

Ran 38 tests containing 141 assertions.

That's what I see locally, and the Docker build is clearly not doing that. I'll look into it.

sirsean commented 8 years ago

Still can't figure out what's going on with the Docker tests in CI. It runs all the tests for me (even inside Docker locally) but clearly isn't running the new tests in CI. The docker-compose file shows that the command it's running is just lein test, which should really work.