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

Easier environment setup with Fabric #34

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

JackWink
Copy link

I've been working on adding fabric support for a quicker dev environment setup. It's not quite ready, but it's far enough along that I'd like to get feedback before I continue too far down this road.

Some things that I'm still working on:

  • fab run and fab run:development don't run start ngrok
  • fab run:production doesn't touch iptables or init the models

@JackWink
Copy link
Author

Fabric isn't really designed to start two blocking operations, so I don't think it'd be a good idea to have it run ngrok for fab run:development, also not sure people want to automatically open a tunnel to their localhost.

Starting to reconsider even including fab run:production or having a production environment as far as fabric/config is concerned. I'd guess most people already have their own system in place for deploying it / exporting their prod environment variables.

@sinak
Copy link
Member

sinak commented Oct 15, 2014

Hey Jack, thanks for the PR. I actually don't have too much experience with Fabric, did you end up giving up?

@JackWink
Copy link
Author

Not entirely, just got busy with other stuff while waiting for feedback.

I use fabric as a Makefile replacement, it accomplishes the same thing, but you're able to script in python and run shell commands both locally and remotely. Completely understand if you're not sold on it, but I think there are a couple commits in here that you might want to cherry pick if that's the case.

catachresis pushed a commit to 18mr/call-congress that referenced this pull request Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants