Skip to content

Use alpine docker images #130

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
Merged
merged 1 commit into from
Jul 4, 2019
Merged

Use alpine docker images #130

merged 1 commit into from
Jul 4, 2019

Conversation

smacker
Copy link
Contributor

@smacker smacker commented Jul 3, 2019

I don't see an issue for this but somebody suggested this change in empathy session.

Change images of redis and postgres to alpine to reduce disk and memory
footprint.
Also updated redis to newer version just in case.

I tested it with previously initialized workdir, all data is preserved.

@smacker
Use alpine docker images
63e5ae2 
Change images of redis and postgres to alpine to reduce disk and memory
footprint.
Also updated redis to newer version just in case.

I tested it with previously initialized workdir, all data is preserved.

Signed-off-by: Maxim Sukharev <max@smacker.ru>
@smacker smacker requested a review from a team July 3, 2019 16:27
@smacker smacker merged commit a433b5b into src-d:master Jul 4, 2019
@dpordomingo dpordomingo mentioned this pull request Nov 5, 2019
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@carlosms carlosms carlosms approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@smacker @carlosms