nozawat/resource-agents — pull request #11 by nozawat (2013-06-17 08:24:08+00:00)
Title: Review JBoss-RA for JBoss6 (second)

前回のレビュー結果は以下です。

上記レビューを踏まえて修正しました。
パッチの分け方は以下の通りです。

１）既存の動作(JBoss5)に関係するが動作に影響のない軽微な修正は以下の３つです。
* Low: jboss: remove a bogus option in the default of run_opts …
* Low: jboss: Unification of the use shell.
* Low: jboss: correctly pass the environment variables with space chara… …

２）JBoss 6 に対応する一連のパッチの集まりは以下の６つです。
* Medium: jboss: Addition of the jboss_version parameter (Support for J… …
* Medium: jboss: Addition of start method for JBoss6 (Support for JBoss… …
* Medium: jboss: The start of JBoss6 is a background method (Support fo… …
* Medium: jboss: Addition of stop processing for JBoss6 (Support for JB… …
* Medium: jboss: Addition of the jboss_base_dir parameter (Support for … …
* Low: jboss: jboss_home is no longer unique in the multi-instances con… …

パッチについては指摘内容を反映させたものになっていますが、先週の打ち合わせ時点からは以下の点が変わっています。
* JBoss5のPIDファイルによる停止方式への変更についてはJBoss5.1.0がrun.shの中でPIDファイルを作成しない為、停止が出来なくなることが分かりました。その為、JBoss5は既存処理を変えないような作りでパッチを作成しました。その為、「JBoss6対応を行うため、 JBoss5 の既存の動作を大きく変更する修正」のパッチ分類については無いです。
* ocf_is_verチェック処理の変更のお話をしましたが、ocf_is_verチェック処理は１桁のバージョンだとエラーとなってしまう為、ocf_is_verでチェックする為には内部で形式を変換する必要がありました。その為、ocf_is_verチェックは利用せずにocf_is_decimalのチェック方式のままとなっています。
* JBoss6対応については要約に(Support for JBoss6 1/5)のように対応に必要なパッチ番号を記入しました。