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.

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

mongodb-docker: provide better error message when docker isn't available #2066

Closed
carolynvs opened this issue May 16, 2022 · 4 comments · Fixed by #2217
Closed

mongodb-docker: provide better error message when docker isn't available #2066

carolynvs opened this issue May 16, 2022 · 4 comments · Fixed by #2217
Labels
2 - 🍕 Pizza should be eaten daily user experience 🌈💖 Make it easier for everyone to use Porter

Comments

@carolynvs
Copy link
Member

I just ran porter list when docker wasn't running and the error message isn't really helpful. It would be great if the mongodb-docker plugin can detect that docker isn't responding and then tell people to start it.

porter list
2022-05-16T11:33:47.714-0500	error	could not configure a tracer: context deadline exceeded
2022-05-16T11:33:48.719-0500	error	could not configure a tracer: context deadline exceeded
2022-05-16T11:33:51.810-0500	error	rpc error: code = Unknown desc = error creating porter-mongodb-docker-plugin-data docker volume:
2022-05-16T11:33:51.810-0500	error	could not list installations: could not read storage schema document: rpc error: code = Unknown desc = error creating porter-mongodb-docker-plugin-data docker volume:
@joshuabezaleel
Copy link
Contributor

joshuabezaleel commented May 19, 2022

Is this similar to #1926 ?

@carolynvs
Copy link
Member Author

They are both user experience improvements but are detecting different error cases. This one is for when docker isn't running, and the mongodb-docker plugin should provide a better error message when docker isn't running. The other is for the mongodb storage plugin to check for common cases and provide a better message (for example when mongodb is down).

@joshuabezaleel
Copy link
Contributor

Ah got it, thank you!

@carolynvs carolynvs added user experience 🌈💖 Make it easier for everyone to use Porter 2 - 🍕 Pizza should be eaten daily labels May 26, 2022
@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Closed by #2217.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 - 🍕 Pizza should be eaten daily user experience 🌈💖 Make it easier for everyone to use Porter
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants