Skip to content

Update the WSL guideline #5462

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

Merged
merged 2 commits into from
Aug 16, 2021
Merged

Update the WSL guideline #5462

merged 2 commits into from
Aug 16, 2021

Conversation

dend
Copy link
Contributor

@dend dend commented Jun 22, 2021

MongoDB in WSL is supported - linked to the official documentation as well. I just updated the admonition with a link to the document above.

@dend
Copy link
Contributor Author

dend commented Jun 22, 2021

@andf-mongodb - can you please review and see if I need to tweak anything to merge this in? Want to make sure that folks are able to use the product in WSL.

@andf-mongodb
Copy link
Contributor

andf-mongodb commented Jun 24, 2021

Thanks @dend. I can't direct users to your linked resource until it is updated to MDB v4.4 (or v5.0 in two weeks). Currently your instructions install v3.6. Background: mongodb package is hosted by Ubuntu and is wildly out of date. mongodb-org (MDB Community) or mongodb-enterprise (MDB Enterprise) are the MDB-hosted and latest version metapackages. See this section of our docs.

Are you able to update your doc? Do you need a hand / peer reviewer? Thank you.

@dend
Copy link
Contributor Author

dend commented Jun 25, 2021

Thanks @andf-mongodb - will work on this with @mattwojo. Will keep this PR open and keep you posted on our progress.

@mattwojo
Copy link

@andf-mongodb / @dend - I updated our docs for WSL to reflect the install steps for MDB v5.0: https://docs.microsoft.com/windows/wsl/tutorials/wsl-database#install-mongodb

Works great on my machine, but please take a look and let me know if you have any questions or concerns.
Thanks!!

@dend
Copy link
Contributor Author

dend commented Aug 3, 2021

Thanks for the ping @mattwojo - I followed the instructions and got MongoDB running in WSL.

@andf-mongodb - let us know if the changes look good, and whether we need to add any other details to make this PR ready for merging.

@andf-mongodb
Copy link
Contributor

Thank you -- the linked instructions now look good to me, thank you for the update!

I am leaving MDB shortly, so let me hand off this PR to the team to handle the change. I will recommend at least a removal of the "do not use" messaging, but am not sure of company stance on linking out to third party instructions.

Thanks again for your work on this!

@dend
Copy link
Contributor Author

dend commented Aug 9, 2021

Sounds good @andf-mongodb. If it helps (and the team doesn't want to link to external resources), I am happy to create an explicit PR here with instructions for WSL.

@dend
Copy link
Contributor Author

dend commented Aug 13, 2021

Tagging @ianf-mongodb and @jeff-allen-mongo to see if there is anything we can do to help move this forward.

Copy link
Collaborator

@jeff-allen-mongo jeff-allen-mongo left a comment

Choose a reason for hiding this comment

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

Thanks for the ping, @dend. I left a comment regarding formatting, and once that is addressed I can merge this in. Thank you.

@dend
Copy link
Contributor Author

dend commented Aug 16, 2021

@jeff-allen-mongo - done, thank you so much for the suggestion!

@jeff-allen-mongo jeff-allen-mongo merged commit dbec6ba into mongodb:master Aug 16, 2021
@jeff-allen-mongo
Copy link
Collaborator

Thanks for updating @dend. I've merged this change in. Cheers!

mongo-cr-bot pushed a commit that referenced this pull request Dec 1, 2023
* DOCSP-33214 Self-Managed Backups for Sharded Clusters

* Backup procedure prep

* Backups

* Fixes build issue

* Tests tutorial

* Adds before you begin content

* minor text edits

* Fixes per Jeff

* Fixes rendering error

* Fixes rendering error

* Fixes rendering error

* Fixes per Jeff

* Fixes per Jeff

* Fixes per Jeff



* Fixes per Jeff

* Fixes per Jeff

* Fixes per Nandinin

* Fixes per Nandinin

* Fixes per Nandini

* Fixes per Maria

* fixes per Maria

* Fixes per Ashley

* Fixes per Nandini

* Fixes per Nandini

* Docsp-34017-bucket-rounding-seconds additional updates saved file (#5326)

* DOCSP-34017-bucket-rounding-seconds

* DOCSP-34017-bucket-rounding-seconds

* DOCSP-34017-bucket-rounding-seconds

* DOCSP-34017-bucket-rounding-seconds

* DOCSP-34017-bucket-rounding-seconds

* DOCSP-34017-bucket-rounding-seconds

---------



* Removes artifact from another branch

* Removes artifact from another branch

* Fixes per Tim

* Fixes per Tim

* Fixes per Tim

* Fixes per Tim

* Fixes per Tim

* Fixes per Jason

* Fixes per Jason

* fixes per Jason

* Fixes per Jason

* Fixes per Jason

---------

Co-authored-by: Jeff Allen <[email protected]>
Co-authored-by: jason-price-mongodb <[email protected]>
Co-authored-by: jason-price-mongodb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants