Bug 482940 – sys-cluster/cinder should install an init.d script and sudo conf file

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Davide Rebeccani 2013-08-29 16:49:09 UTC]

sys-cluster/cinder is missing OpenRC init files and confd files. There are some errors in the ebuild that block cinder services to start

Reproducible: Always

Steps to Reproduce:
1.emerge sys-cluster/cinder
2./etc/init.d/cinder-[api,scheduler,volume] start
3.
Actual Results:  
/etc/init.d/cinder-[api,scheduler,volume]: No such file or directory

Expected Results:  
/etc/init.d/cinder-[api,scheduler,volume] [Ok]

[Comment 1 Davide Rebeccani 2013-08-29 16:50:33 UTC]

Created attachment 357344 [details]
sys-luster/cinder fixed ebuild

Added USE flags for every cinder services. Probably needs more work

[Comment 2 Davide Rebeccani 2013-08-29 16:51:11 UTC]

Created attachment 357346 [details]
sys-cluster/cinder initd file

[Comment 3 Davide Rebeccani 2013-08-29 16:51:35 UTC]

Created attachment 357348 [details]
sys-cluster/cinder confd file

[Comment 4 Davide Rebeccani 2013-08-29 16:52:18 UTC]

Created attachment 357350 [details]
sys-cluster/cinder sudo config file

[Comment 5 Jeroen Roovers (RETIRED) 2013-08-29 17:09:34 UTC]

Comment on attachment 357344 [details]
sys-luster/cinder fixed ebuild

--- cinder-2013.1.9999.ebuild   2013-08-20 15:37:55.340621355 +0200
+++ -   2013-08-29 19:09:20.393813088 +0200
@@ -16,7 +16,7 @@
 LICENSE="Apache-2.0"
 SLOT="0"
 KEYWORDS=""
-IUSE=""
+IUSE="api scheduler volume"
 
 DEPEND="dev-python/setuptools[${PYTHON_USEDEP}]"

@@ -45,14 +45,29 @@
                >=dev-python/python-keystoneclient-0.2.0[${PYTHON_USEDEP}]
                dev-python/python-swiftclient[${PYTHON_USEDEP}]
                >=dev-python/oslo-config-1.1.0[${PYTHON_USEDEP}]
-               virtual/python-argparse[${PYTHON_USEDEP}]"
+               virtual/python-argparse[${PYTHON_USEDEP}]
+               >=sys-block/iscsitarget-1.4.20.2_p20130821
+               sys-block/open-iscsi"
 
 PATCHES=( )

+pkg_setup() {
+    enewgroup cinder
+    enewuser cinder -1 -1 /var/lib/cinder cinder 
+}
+
 python_install() {
        distutils-r1_python_install
+
+    newconfd "${FILESDIR}/cinder.confd" cinder
+       newinitd "${FILESDIR}/cinder.initd" cinder
+
+    use api && dosym /etc/init.d/cinder /etc/init.d/cinder-api
+    use scheduler && dosym /etc/init.d/cinder /etc/init.d/cinder-scheduler
+    use volume && dosym /etc/init.d/cinder /etc/init.d/cinder-volume
+       
        keepdir /etc/cinder
-       keepdir /etc/cinder/rootwrap.d
+       dodir /etc/cinder/rootwrap.d
        insinto /etc/cinder

        newins "${S}/etc/cinder/cinder.conf.sample" "cinder.conf"
@@ -62,4 +77,9 @@
        newins "${S}/etc/cinder/rootwrap.conf" "rootwrap.conf"
        insinto /etc/cinder/rootwrap.d
        newins "${S}/etc/cinder/rootwrap.d/volume.filters" "volume.filters"
+    #add sudoers definitions for user nova
+    insinto /etc/sudoers.d/
+    doins ${FILESDIR}/cinder_sudoers
+       dodir /var/log/cinder
+       fowners cinder:cinder /var/log/cinder
 }

[Comment 6 Matthew Thode ( prometheanfire ) 2013-09-04 20:26:01 UTC]

can you re-upload the conf.d file, you seem to have uploaded the init.d file in it's place.

[Comment 7 Davide Rebeccani 2013-09-05 16:49:33 UTC]

Created attachment 357948 [details]
sys-cluster/cinder confd file

[Comment 8 Davide Rebeccani 2013-09-05 16:49:58 UTC]

(In reply to Matthew Thode ( prometheanfire ) from comment #6)
> can you re-upload the conf.d file, you seem to have uploaded the init.d file
> in it's place.

Done. Sorry for the mistake. :-)

[Comment 9 Matthew Thode ( prometheanfire ) 2013-09-05 19:41:50 UTC]

ok, it's in tree :D