Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Added AWS DynamoDB support #166

Closed
wants to merge 2 commits into from
Closed

Conversation

mjubenot
Copy link

Added support for DynamoDB, a noSQL database managed by AWS

Heavily inspired by the etcd implementation

@Mierdin
Copy link
Member

Mierdin commented Jan 14, 2018

Hello @mjubenot - thanks for working on this! I wonder if you've had a chance to look at #161. Regarding both this PR as well as #165 I am considering massive simplifications to the whole comms and DB layers, which would cut down on external dependencies. So I'll keep this open for now since that's probably still a ways off (at least the comms part) but we may not want to introduce these new dependencies on light of those efforts.

On a related note, have you used ToDD with the extensions you've built here? Have you managed to get ToDD running on top of DynamoDB and Amazon MQ?

@mjubenot
Copy link
Author

Hello @Mierdin , and thank you for providing this tool in the first place. I've already taken a look at your vision of ToDD in the future, and after using it for a while , I think your points makes a lot of sense.

We're running ToDD with these extensions. Developping these was easier than I expected thanks to your well organized code. I was able to get started without a deep understanding of the inner working.

We also had to rewrite the factcollectors for it to be compatible with the AWS environment, but ToDD was up and running in an EC2 instance with Amazon MQ and DynamoDB once these were done.

We actually use a combination of packer, ansible and terraform to automatically deploy servers and agent in AWS. Our end goal is to include ToDD in a continous delivery pipeline, as a tool for the testing phase, but we still need to create some tools around it.

Apart from this, we're not using it for real-case tests as of now, mainly due to the lack of testlet we may want to use. It is a great testing tool nonetheless, and we may use it in the near future.

@Mierdin
Copy link
Member

Mierdin commented Oct 13, 2018

@mjubenot I'll be revisiting this soon, and I'm committed to a much simpler approach with gRPC, rather than an external messaging system. So while I definitely appreciate your work here, I'm going to close this PR to make way for that future approach.

Before I do, I noticed you mentioned a testlet that didn't exist that you needed. Can you give me an idea of what kind of testlet you were looking for?

@Mierdin
Copy link
Member

Mierdin commented Nov 25, 2018

Closing as discussed. The rebuild of ToDD will make all these external dependencies unnecessary.

@Mierdin Mierdin closed this Nov 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants