-
Notifications
You must be signed in to change notification settings - Fork 9
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 kinesis integration testing #75
base: main
Are you sure you want to change the base?
Conversation
qhool
commented
Jul 7, 2023
- docker compose.yml creates a testing environment: * Use kinesalite and dynalite (github.com/mhart) to provide kinesis and dynamodb work-alikes * Create ca-certificate and server cert signed with this ca, so that kinesalite and dynalite can pose as the real deals * Insert the ca-cert into trusted certificats for the test container * Create dns aliases for standard aws endpoints
- New consume_SUITE.erl and load_from_dir script enable testing against real AWS or using the docker compose env
- Add Makefile to hold the required commands to start tests under docker
* docker compose.yml creates a testing environment: * Use kinesalite and dynalite (github.com/mhart) to provide kinesis and dynamodb work-alikes * Create ca-certificate and server cert signed with this ca, so that kinesalite and dynalite can pose as the real deals * Insert the ca-cert into trusted certificats for the test container * Create dns aliases for standard aws endpoints * New consume_SUITE.erl and load_from_dir script enable testing against real AWS or using the docker compose env * Add Makefile to hold the required commands to start tests under docker
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm on a running it in my mac and I get:
error: failed switching to "work_user": unable to find user work_user: no matching entries in passwd file
I highly recommend documenting the setup in the README.
openssl genrsa -des3 -passout pass:foobar -out ca/ca.key $KEY_SIZE || exit 1 | ||
|
||
openssl req -new -x509 -passin pass:foobar \ | ||
-subj "/C=US/ST=CA/O=ACME/CN=acme.fake/[email protected]" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What command did you use exactly?
I'll add something once I understand what's going on.. for me the setup was |
Docker desktop uses a utility vm to host the containers. In addition to making things slower, this has other advantages: for instance, by running as root and mapping file ownership to root, it can break workflows that assume the docker user is unpriviledged.
Almost working, but I still got this error while I was running the last commit.
|
depends_on: | ||
ca: | ||
condition: service_healthy | ||
domainname: kinesis.us-east-1.amazaonaws.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo here
domainname: kinesis.us-east-1.amazaonaws.com | |
domainname: kinesis.us-east-1.amazonaws.com |
Please do not merge this PR before it has been tested in mac arm and mac x86. As soon as we get it working on x86 I'll test it in arm. Thaaanks! |