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

Fix #348 for mongodb installation #349

Merged
merged 2 commits into from
Nov 28, 2017

Conversation

raju249
Copy link
Contributor

@raju249 raju249 commented Nov 26, 2017

Fixes #348

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream master branch.
  • I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

  • Add correct instructions for smooth installation of mongodb.

@vaibhavsingh97
Copy link
Member

@raju249 Please add changes in PR template and also please update your branch

@raju249
Copy link
Contributor Author

raju249 commented Nov 27, 2017

Sure @vaibhavsingh97
Doing it now.
Thanks for review.

@fossasia-bot
Copy link

fossasia-bot bot commented Nov 27, 2017

Hi @raju249!

Looks like your PR has some conflicts. 😟
Could you resolve them and rebase on top of the latest upstream code ?

@raju249
Copy link
Contributor Author

raju249 commented Nov 27, 2017

@vaibhavsingh97 Updated the branch with latest master and updated PR template as well.
Thanks

Copy link
Contributor

@AnshulMalik AnshulMalik left a comment

Choose a reason for hiding this comment

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

👍

@raju249
Copy link
Contributor Author

raju249 commented Nov 27, 2017

Is @fossasia-bot not working as expected ?
Seems like its giving PR conflicts for every PR

@cclauss
Copy link
Contributor

cclauss commented Nov 28, 2017

I am not a fan of @fossasia-bot... It is VERY chatty with little helpful info and lots of false negative messages.

@raju249
Copy link
Contributor Author

raju249 commented Nov 28, 2017

Yeah right 😅.
Why don't we fix it or just remove it ?
@cclauss


```bash
sudo apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv 7F0CEB10
echo "deb http://repo.mongodb.org/apt/ubuntu "$(lsb_release -sc)"/mongodb-org/3.0 multiverse" | sudo tee /etc/apt/sources.list.d/mongodb-org-3.0.list
sudo apt-get update
sudo apt-get install -y mongodb-org
sudo apt-get install -y mongodb
Copy link
Member

Choose a reason for hiding this comment

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

I can find that with
sudo apt-get install -y mongodb-org

E: There were unauthenticated packages and -y was used without --allow-unauthenticated
comes have you tried this thing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't get this error. What I got with old version of command is the package was not located . The I checked the link mentioned on the issue page where it says to remove the org. And when I tried that while installing, it worked (removing org works )

@bhaveshAn bhaveshAn merged commit e596bdf into fossasia:master Nov 28, 2017
@bhaveshAn
Copy link
Member

Thanks @raju249

@raju249 raju249 deleted the 348-update-install-guide branch November 28, 2017 14:16
@raju249
Copy link
Contributor Author

raju249 commented Nov 28, 2017

👍 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix installation guide for mongodb installation steps
5 participants