-
Notifications
You must be signed in to change notification settings - Fork 362
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 influxdb module #903
Add influxdb module #903
Conversation
Two more things:
|
Lock file has been removed and I changed I took the ElasticMQ as an example https://github.com/cachix/devenv/blob/main/src/modules/services/elasticmq.nix#L18 - I think, that might need some changing as well. |
Seems like the test failed:
|
Test is fixed! I forgot to set the port for the cli-command and it worked on my machine, since another influx instance was running on that port 🤦 |
At least I thought I had fixed the test. But I am unsure how to interpret these checks. InfluxDB seems to have been successful, but others failed? |
Is it supposed to work on macOS? https://github.com/cachix/devenv/actions/runs/7248487703/job/19744543883?pr=903 |
It should work on macOS (and I am using it right now). I do not see the Maybe the timeout was too short, I increased it to match the one in elasticMQ test https://github.com/cachix/devenv/blob/main/examples/elasticmq/.test.sh#L14 |
That seems to have done the trick! https://github.com/cachix/devenv/actions/runs/7250287690/job/19760416126 |
Hi everybody!
This is my first attempt at a devenv module, to add support for InfluxDB.
I'm looking forward to some feedback!