Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Ceph backend support #1

Open
wants to merge 47 commits into
base: master
Choose a base branch
from

Conversation

cholcombe973
Copy link

I realize this PR is very different but I wanted to at least give you the chance to merge it back up with your upstream repo. I added a Ceph backend to preserve. I ran into an issue with sqlite where it kept consistently segfaulting. See bug: https://github.com/jgallagher/rusqlite/issues/189 I tried to figure out what the root cause was but couldn't so I decided to remove the caching feature from preserve. I didn't encounter any segfault issues with the file backend so it's possible my Ceph backend has introduced some bad behavior.

@cholcombe973
Copy link
Author

So I figured out the issue was with my ceph crate that caused the segfaults. I'll add the sqlite support back into the code.

cholcombe973 and others added 9 commits November 29, 2016 16:56
reduce build matrix on travis
* setup ceph / gluster / vault only if they are needed

We should only install / setup / start the external dependencies
if they are needed to test the current test features. Additionally,
we should use the desired backend in the testing to excercise all
of the functionality

* Use $HOME/.config for the config files

* update to setup config files correctly

* use sudo for install

* remove gluster start job

* make nightly / beta test differing backends

* fix syntax

* fix syntax...again

* update for vault

* Remove duplicate build step

* fix vault json syntax

* fix up vault integration

* remove println

* make the tests more resilient
…add an explicit loglevel flag to help with setting the logging level
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants