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

build: add dev-get-id, dev-enter, and dev-load-data targets #7278

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

cfm
Copy link
Member

@cfm cfm commented Oct 24, 2024

Status

Ready for review, but posting mostly for feedback. If we want this, I'll see if I can make it agnostic between Docker and Podman.

Description of Changes

Adds some helper Make targets working with a running make dev[-tor] container:

  1. dev-get-id looks up the container ID
  2. dev-enter starts a shell in the running container
  3. dev-load-data runs loaddata.py in the running container

Testing

Play around with it and let me know if this is worth polishing.

Deployment

Development-only.

"dev-enter" is intended to disambiguate this action (start a shell in
the running development container) from the "dev-shell" script (start a
shell in a new development container).
@legoktm
Copy link
Member

legoktm commented Oct 25, 2024

I have an alias of "sdshell" set to podman exec --user=root -it (podman ps --filter name=securedrop --format '{{.ID}}') bash. So +1 from me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants