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

Adds memcached #23

Merged
merged 1 commit into from Feb 10, 2015
Merged

Adds memcached #23

merged 1 commit into from Feb 10, 2015

Conversation

@leonelgalan
Copy link
Contributor

@leonelgalan leonelgalan commented Jan 12, 2015

Allows to use memcached to enable caching using dalli.

@BrandonMathis
Copy link
Contributor

@BrandonMathis BrandonMathis commented Jan 13, 2015

Have you tried building a fresh ansible box and standing up a simple rails app from this branch?

I have been using this repo to test smashing_boxer since we currently don't have any automated testing

https://github.com/BrandonMathis/vanilla-rails-app

@leonelgalan
Copy link
Contributor Author

@leonelgalan leonelgalan commented Jan 13, 2015

No, but I did use it on malartu by running everything

@BrandonMathis
Copy link
Contributor

@BrandonMathis BrandonMathis commented Jan 26, 2015

@leonelgalan ansible-galaxy this dude 👍

@BrandonMathis
Copy link
Contributor

@BrandonMathis BrandonMathis commented Jan 29, 2015

@leonelgalan what is the status of this pr. Do we still want to merge?

@leonelgalan
Copy link
Contributor Author

@leonelgalan leonelgalan commented Jan 29, 2015

Needs rebase.

@leonelgalan leonelgalan force-pushed the feature/memcached branch from ce02768 to daa062f Jan 29, 2015
@leonelgalan
Copy link
Contributor Author

@leonelgalan leonelgalan commented Jan 29, 2015

I just installed weaveup on my server (weaveups new dev) using this, works like a charm! :neckbeard:

@BrandonMathis
Copy link
Contributor

@BrandonMathis BrandonMathis commented Feb 10, 2015

doesn't memcached need to be restarted after every deploy?

@leonelgalan
Copy link
Contributor Author

@leonelgalan leonelgalan commented Feb 10, 2015

No, you will destroy the cache that way. It might need, but that's up to the developer.

BrandonMathis added a commit that referenced this pull request Feb 10, 2015
@BrandonMathis BrandonMathis merged commit bed6fd3 into master Feb 10, 2015
@BrandonMathis BrandonMathis deleted the feature/memcached branch Feb 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants