----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18309/#review35005 -----------------------------------------------------------
systemvm/patches/debian/buildsystemvm.sh <https://reviews.apache.org/r/18309/#comment65380> this line be added from the script in cloud-tools/appliance/definitions/systemvm*template/postinstall.sh in configure_services() please remove the space at the end of the line systemvm/patches/debian/config/etc/modules <https://reviews.apache.org/r/18309/#comment65379> as discussed: this file should not be neccessary - daan Hoogland On Feb. 20, 2014, 1:39 p.m., Joris van Lieshout wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18309/ > ----------------------------------------------------------- > > (Updated Feb. 20, 2014, 1:39 p.m.) > > > Review request for cloudstack, Alex Huang, daan Hoogland, and Hugo Trippaers. > > > Repository: cloudstack-git > > > Description > ------- > > Couple changes in sysctl.conf: > 1. up the amount of connections that the svm can handle. > 2. Panic on OOM and OOPS so that ACS can act on a misbehaving SVM. > 3. nonlocal_bind to fix some situations where master to backup fails in > redundant rvms. > Also adding /etc/modules to preload modules so that setting some of the > sysctl settings during boot works. For this I also changed procps runlevels. > > > Diffs > ----- > > systemvm/patches/debian/buildsystemvm.sh 7f6188a > systemvm/patches/debian/config/etc/modules PRE-CREATION > systemvm/patches/debian/config/etc/sysctl.conf 586d5bd > > Diff: https://reviews.apache.org/r/18309/diff/ > > > Testing > ------- > > We have been running these adjustments in production on our own 4.2 and 3.0 > SVM with success for a couple months now. > > > Thanks, > > Joris van Lieshout > >
