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

mlaunch init --help should not require mongod #894

Closed
stennie opened this issue Nov 8, 2022 · 0 comments
Closed

mlaunch init --help should not require mongod #894

stennie opened this issue Nov 8, 2022 · 0 comments
Assignees
@stennie
Labels
improvement mlaunch

Comments

@stennie
Copy link
Collaborator

stennie commented Nov 8, 2022

mlaunch launches mongod --help to discover supported options, as some (like --tls*) are version-dependent.

The error message could be more informative for this use case or perhaps usage of mongod --help could be avoided:

mongod wasn't found in current $PATH, please specify with --binarypath

However, mlaunch should at least warn if no mongod is found in the path as that will be a fatal error when trying to launch a new cluster.
@stennie stennie added this to the 1.8.0 milestone Nov 8, 2022
@stennie stennie self-assigned this Nov 8, 2022
@stennie stennie removed this from the 1.8.0 milestone Nov 8, 2022
@stennie stennie changed the title mlaunch init --help should print more informative message when mongod isn't found mlaunch init --help should not require mongod Nov 8, 2022
stennie added a commit to stennie/mtools that referenced this issue Nov 8, 2022
@stennie
Fix rueckstiess#894: mlaunch init --help should not require mongod
6cccc08
@stennie stennie closed this as completed in fa1787f Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stennie stennie

Labels
improvement mlaunch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stennie