Skip to content

Remove LazyRecoverable and dalli (and memcached) #517

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

Merged
merged 1 commit into from
Dec 10, 2019
Merged

Conversation

markets
Copy link
Collaborator

@markets markets commented Oct 3, 2019

Closes #358
Closes #327

Added data: { disable_with: '...' } to the button to disable it while the request is running (should avoid fast double clicks).

Copy link
Collaborator

@sauloperez sauloperez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@markets markets changed the title Remove LazyRecoverable and dalli Remove LazyRecoverable and dalli (and memcached) Oct 9, 2019
@sseerrggii
Copy link
Contributor

Hi @markets we are back with new infra 🎉
Tested 💚

@sauloperez
Copy link
Collaborator

I created coopdevs/timeoverflow-provisioning#144 out of this.

@sauloperez sauloperez merged commit 3f318cc into develop Dec 10, 2019
@sauloperez sauloperez deleted the remove_cache branch December 10, 2019 15:40
@markets
Copy link
Collaborator Author

markets commented Dec 10, 2019

cool 🤘🤘:metal:

we are back with new infra

maybe now we can revert this revert #513? would be amazing to get rid off Elastic 👌

@sauloperez
Copy link
Collaborator

sauloperez commented Dec 11, 2019

Yep, that's the next one on the list. Probably after Xmas though. @danypr92 and I need to settle things up on the provisioning side asap but we can already deploy releases as usual 🎉

@sauloperez sauloperez mentioned this pull request Jan 21, 2020
@sauloperez
Copy link
Collaborator

sauloperez commented Mar 20, 2020

I can confirm we are now memcached-free in production, thanks to @mperezv . One thing less to run 🎉

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.

Can we remove Memcached? Is this optimization really needed on password reset?
4 participants