Add local dev environment for testing without AWS or hardware#44
Open
InterstellarIntruder wants to merge 1 commit into
Open
Add local dev environment for testing without AWS or hardware#44InterstellarIntruder wants to merge 1 commit into
InterstellarIntruder wants to merge 1 commit into
Conversation
InterstellarIntruder
force-pushed
the
feature/local-dev-setup
branch
from
9a48ec3 to
bd5d616
Compare
- Fix missing dependencies: add netifaces, remove python_version guard on importlib_resources, loosen boto3 pin for modern Python compat - Add Docker mock whale tag (Alpine + SSH + sample /data/) so ceti whaletag commands can run locally without physical hardware - Add moto-based S3 mock for offline pytest (no AWS credentials needed) - Add Makefile targets: install, test, whaletag-up, whaletag-down - Update README with Local Development quickstart section Addresses Project-CETI#22, Project-CETI#29
InterstellarIntruder
force-pushed
the
feature/local-dev-setup
branch
from
bd5d616 to
6ffaf8d
Compare
Author
|
Heads up -- I noticed after submitting that #33, #34, and #35 from @claymore666 and #32 from @BlindMuadDib cover a lot of the same ground. This PR uses moto instead of LocalStack and a simpler Docker setup, but I'm fine closing this if you'd rather go with those. |
|
Yeah I needed something to test against, might be outdated and/or overly complicated. I'd say its worth reducing complexity so one less failure point. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Addresses #22, fixes #29
Note: There's overlap here with #33 (LocalStack), #34 (netifaces dep), #35 (whale tag mockup), and #32 (importlib_resources). This PR takes a lighter approach -- moto instead of LocalStack, a minimal Docker mock -- but happy to close this in favor of those if the maintainers prefer.
Currently there's no way to work on this project without AWS credentials and a physical whale tag. This adds a local dev setup so anyone can clone, install, and run tests.
What's included:
/data/)netifaces, removedpython_version < "3.9"guard onimportlib_resources, loosenedboto3pin for modern Pythonmake install,make test,make whaletag-up,make whaletag-downQuick start: