#117 Define resource limits for the database container
Merged by jskladan. Opened by csomh.
taskotron/ csomh/resultsdb resource-limits  into  develop

Download 117.patch

Otherwise PostgreSQL will run out of memory in projects where
the default limits are lower than this.

Signed-off-by: Hunor Csomortáni csomh@redhat.com

@mprahl or @ralph: please take a look.

@csomh looks good to me. Any reason you picked 0.4 instead of a nice round 0.5 CPU limit? :)

rebased onto a05a35ee1cfe07affc3aa53736b5272b3d18b893

rebased onto a718f9ab3baf54674b0239e09c946925001e2c00

That's a good question @mprahl! I have no clue :p

Right now I would guess that it came from the default CPU limit for
UpShift projects (300 millicores), increased with 100 millicores,
which made things 'feel' better.

I've quickly read through these
docs
again, but could not find anything about recommended strategies for
setting these limits.

FYI: I've noticed that the resultsdb DeployementConfigs were missing
CPU limits, so I've added those too. Better to be explicit about these
things, they say :)

Okay, still :thumbsup:

If this is fine by you, I'll happily merge this after it's rebased to current HEAD. Let me know, from the comments, this seemed to be under development/discussion still :) Thanks!

@csomh is away for a few days. Let's wait for him to come back to see what he thinks.

rebased onto ae3cb8ef4de15cbf1c3eaecb795596f8eaa78306

This was rebased, it's ready to merge, @jskladan. Thanks!

Pull-Request has been merged by jskladan

Metadata