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] Updatings values.yaml to enable monitoring and readme instructions #4

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

grebois
Copy link
Collaborator

@grebois grebois commented Jul 17, 2018

No description provided.

@grebois grebois changed the title Updatings values.yaml to enable monitoring and readme instructions [MongoDB] Updatings values.yaml to enable monitoring and readme instructions Jul 19, 2018
@@ -333,3 +354,727 @@ rs0:PRIMARY> rs.conf()
}
rs0:PRIMARY>
```

# Updating from 3.4.0
Copy link
Owner

@sekka1 sekka1 Jul 20, 2018

Choose a reason for hiding this comment

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

Can we put everything below this in another .md file named updating_from_3.4.0.md and either link to it or just leave it as another file, because this chart will be updated again and the question is, do you keep the 3.4.0 update here or do you delete it or what? I think moving it out to another file now makes the main README.md clean and approachable.

@sekka1
Copy link
Owner

sekka1 commented Jul 25, 2018

Didnt you have to edit the helm/stable chart and had a PR out? Where does that stand?

We should include that information with this type of PRs b/c it really depends on that to work.

This goes back to being able to consume this PR also, you should anticipate the clients questions and answer them before they ask. Because now im concerned that hasnt been merged and if i try this it would not work. Then that leads me to not trying it out right now (which i have set aside time for).

@sekka1
Copy link
Owner

sekka1 commented Jul 25, 2018

@grebois there are some authentication issues when I upgraded:

  Warning  Unhealthy              39s   kubelet, ip-10-120-36-40.ec2.internal  Liveness probe failed: time="2018-07-25T19:32:37Z" level=error msg="Cannot connect to server using url mongodb://****:****@localhost:27017: server returned error on SASL authentication step: Authentication failed." source="connection.go:84"
time="2018-07-25T19:32:37Z" level=error msg="Can't connect to MongoDB: Cannot connect using uri: mongodb://metrics:password@localhost:27017" source="mongodb_exporter.go:104"

@sekka1
Copy link
Owner

sekka1 commented Jul 25, 2018

PR for the helm/stable changes: helm/charts#6282

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.

2 participants