----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6658/#review10428 -----------------------------------------------------------
cloud.spec <https://reviews.apache.org/r/6658/#comment22225> I think we need config(noreplace) for this configuration files: db.properties/log4j-cloud.xml/tomcat6.conf. Admin usually will change these configuration files to suite their needs, without noreplace, these files will be replaced with a new cloudstack RPM upgrade. - edison su On Aug. 16, 2012, 7:26 p.m., Marcus Sorensen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6658/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2012, 7:26 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Update cloud.spec for current state of 4.0, minor waf and init script fixes > for RHEL/CENTOS/Fedora > > > Diffs > ----- > > agent/distro/centos/SYSCONFDIR/rc.d/init.d/cloud-agent.in 488ced2 > agent/distro/fedora/SYSCONFDIR/rc.d/init.d/cloud-agent.in 488ced2 > agent/distro/rhel/SYSCONFDIR/rc.d/init.d/cloud-agent.in 488ced2 > cloud.spec 47a457a > wscript 9e9acbd > > Diff: https://reviews.apache.org/r/6658/diff/ > > > Testing > ------- > > Tested on Centos 6.2 x64 > Build RPMS via './waf rpm' : passed > Installed RPMS to management server : passed > Installed RPMs to hypervisor host : passed > Started/stopped via init scripts : passed > Create zone, deploy instance : passed > > > Thanks, > > Marcus Sorensen > >
