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

Fixing default storage driver #59

Merged
merged 4 commits into from
May 11, 2022
Merged

Fixing default storage driver #59

merged 4 commits into from
May 11, 2022

Conversation

llbbl
Copy link
Contributor

@llbbl llbbl commented Jan 25, 2019

Resolves
#58

paulfantom
paulfantom approved these changes Jan 25, 2019
View reviewed changes
Copy link
Collaborator

@paulfantom paulfantom left a comment

Choose a reason for hiding this comment

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

lgtm

@llbbl
Copy link
Contributor Author

llbbl commented Jan 25, 2019

Not sure why travis failing. Is that something I did ?

@paulfantom
Copy link
Collaborator

paulfantom commented Jan 25, 2019 •
edited
Loading

It cannot install docker-ce package in version 18.09.1 on Debian Jessie and Ubuntu Xenial. Maybe try with 18.09?

@llbbl
Copy link
Contributor Author

llbbl commented Jan 26, 2019

ok

llbbl added 2 commits January 26, 2019 12:58
@llbbl
Update main.yml
6c304dc 
dropping docker version
@llbbl
Update main.yml
6ef1498 
dropping docker version
@llbbl
Copy link
Contributor Author

llbbl commented Jan 26, 2019

Trying 18.06.1 now

@llbbl
Copy link
Contributor Author

llbbl commented Jan 26, 2019

Worked that time

@llbbl
Copy link
Contributor Author

llbbl commented Apr 28, 2022

@paulfantom you ok?

@paulfantom
Copy link
Collaborator

Wow, that's a blast from a past :D Sorry for a LOOOOOONG delay.

@paulfantom paulfantom merged commit 8722be5 into mongrelion:master May 11, 2022
@llbbl
Copy link
Contributor Author

llbbl commented May 11, 2022

no worries. I get it! thanks for getting this merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulfantom paulfantom paulfantom 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.
2 participants
@llbbl @paulfantom