Tweaked system and mongod to scale better #6838

Merged
merged 1 commit into from Jan 23, 2017

Conversation

Projects
None yet
4 participants
Contributor

perrito666 commented Jan 19, 2017

Mongod requiers some tweaks in sysctl and in its parameters to
scale properly without consumin excessive memory and resources.
The cache size has been limited and a few sysctl values have
been tweaked to better suit the controller.

There is a follow up patch to be sent after this one that makes the memory allocation policy configurable.

QA

  • Bootstrap juju.
  • ps should show you a mongod running with --wiredTigerCacheSizeGB 1 among its parameters.
  • Syslog should not show suggestions by mongod to change transparent hughe pages settings.
  • The following settings should be set in proc:
@@ -398,6 +419,11 @@ type EnsureServerParams struct {
// This method will remove old versions of the mongo init service as necessary
// before installing the new version.
func EnsureServer(args EnsureServerParams) error {
@howbazaar

howbazaar Jan 23, 2017

Owner

When is this being called? Will an updated apiserver call this method?

@perrito666

perrito666 Jan 23, 2017

Contributor

This should be called every time jujud starts

@@ -183,7 +187,10 @@ func newConf(args ConfigArgs) common.Conf {
" --smallfiles"
} else {
mongoCmd = mongoCmd +
- " --storageEngine wiredTiger"
+ " --storageEngine wiredTiger" +
@howbazaar

howbazaar Jan 23, 2017

Owner

Will this get rewritten when the agent starts? Or do we need an upgrade step?

@perrito666

perrito666 Jan 23, 2017

Contributor

It was supposed to, but for some reason its not, ill fix that since its the intended behavior

@perrito666

perrito666 Jan 23, 2017

Contributor

fixed now

Tweaked system and mongod to scale better
Mongod requiers some tweaks in sysctl and in its parameters to
scale properly without consumin excessive memory and resources.
The cache size has been limited and a few sysctl values have
been tweaked to better suit the controller.
Contributor

reedobrien commented Jan 23, 2017

$$merge$$

Contributor

jujubot commented Jan 23, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 10b7ef7 into juju:develop Jan 23, 2017

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