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.

Sign up for GitHub

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

Jump to bottom

remove memcache entirely, add in some django redis to cover it's absence #7240

Merged
merged 6 commits into from
Jun 9, 2020
Merged

remove memcache entirely, add in some django redis to cover it's absence #7240

merged 6 commits into from
Jun 9, 2020

Conversation

rebeccahhh
Copy link
Member

SUMMARY

pertains to #6932
removing the remainders of memcache and sub in django-redis to work with Redis

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • API
AWX VERSION
awx: 11.2.0

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@ryanpetrello ryanpetrello self-requested a review June 4, 2020 04:27
@rebeccahhh rebeccahhh force-pushed the memcache-memories branch from fcf1d27 to 0333425 Compare June 4, 2020 14:59
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@wenottingham wenottingham mentioned this pull request Jun 4, 2020
Closed
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@ryanpetrello ryanpetrello mentioned this pull request Jun 4, 2020
Closed
@rebeccahhh rebeccahhh force-pushed the memcache-memories branch 2 times, most recently from 309eb9c to 821c78e Compare June 4, 2020 20:40
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@rebeccahhh rebeccahhh force-pushed the memcache-memories branch from 821c78e to c6beb6a Compare June 4, 2020 22:12
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ryanpetrello
ryanpetrello reviewed Jun 4, 2020
View reviewed changes
Copy link
Contributor

@ryanpetrello ryanpetrello left a comment

Choose a reason for hiding this comment

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

Let's add a note here:

https://github.com/ansible/awx/blob/devel/CHANGELOG.md#1200-tbd

@softwarefactory-project-zuul
Copy link
Contributor

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.

@rebeccahhh rebeccahhh force-pushed the memcache-memories branch 4 times, most recently from fbe1736 to 79a9d8f Compare June 8, 2020 21:43
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@rebeccahhh rebeccahhh changed the title [WIP] remove memcache entirely, add in some django redis to cover it's absence remove memcache entirely, add in some django redis to cover it's absence Jun 8, 2020
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ryanpetrello
ryanpetrello reviewed Jun 8, 2020
View reviewed changes
@rebeccahhh rebeccahhh force-pushed the memcache-memories branch 3 times, most recently from 827bc24 to f67d098 Compare June 9, 2020 14:33
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@rebeccahhh rebeccahhh force-pushed the memcache-memories branch from f67d098 to a9b5c6e Compare June 9, 2020 14:57
@rebeccahhh rebeccahhh force-pushed the memcache-memories branch from a9b5c6e to d7f9e66 Compare June 9, 2020 15:09
@rebeccahhh rebeccahhh self-assigned this Jun 9, 2020
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 277b689 into ansible:devel Jun 9, 2020
@Klaas- Klaas- mentioned this pull request Jul 5, 2020
Open
@rebeccahhh rebeccahhh deleted the memcache-memories branch September 28, 2020 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ryanpetrello ryanpetrello ryanpetrello approved these changes

@chrismeyersfsu chrismeyersfsu chrismeyersfsu left review comments

@shanemcd shanemcd shanemcd approved these changes

Assignees

@rebeccahhh rebeccahhh

Labels
component:api
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rebeccahhh @ryanpetrello @chrismeyersfsu @shanemcd