Skip to content
This repository has been archived by the owner on Apr 12, 2021. It is now read-only.

Adds redis-server to snap #902

Merged
merged 1 commit into from May 31, 2017
Merged

Adds redis-server to snap #902

merged 1 commit into from May 31, 2017

Conversation

adam-stokes
Copy link
Contributor

@adam-stokes adam-stokes commented May 31, 2017 •
edited

Fixes #901

To test:

make snap
snap install ./conjure-up* --dangerous --classic
/snap/conjure-up/current/bin/redis-cli

It should connect to redis-server on our custom port 6380

Signed-off-by: Adam Stokes battlemidget@users.noreply.github.com

Adds redis-server to snap
069d74e 
Fixes #901

Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
@adam-stokes adam-stokes requested a review from johnsca May 31, 2017 01:01
johnsca
johnsca approved these changes May 31, 2017
View reviewed changes
Copy link
Contributor

@johnsca johnsca left a comment

Choose a reason for hiding this comment

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

Due to fetch issues, I haven't been able to build the snap to test this, but the changes look fine so I'm +1 for merging this.

@adam-stokes adam-stokes merged commit b19e3a0 into master May 31, 2017
@adam-stokes adam-stokes deleted the feature/901-add-redis branch May 31, 2017 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@johnsca johnsca johnsca approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@adam-stokes @johnsca