Hi Aruna/ Azeez, Please see my inline comment.
On Tue, Apr 29, 2014 at 7:14 PM, Aruna Karunarathna <[email protected]> wrote: > Hi all, > > After having a offline discussion with Azeez and Kishanthan, Few changes > was suggested for the previous model. > > 1. Remove Deployer based approach for applying patches in C5 Kernel. > > In the previous model, patching solution was dependent on the > DeploymentEngine directly. Patch Module is going to be shipped as a > individual component > and can be invoked when the server is running or in shutdown mode. > Providing a tool to apply patches. > e.g > > ./applypatches.sh PatchZipLocation > +1 for using decoupling this feature from DeploymentEngine. But my concern here is Isn't it better we use a repository listener to patches directory and hot deploy patches instead of using a tool for this. I feel in this way its more user friendly. > > 2. Adding a patch version for a OSGI bundle. > > In the current approach we can't discretely identify a patch by it's > bundle name. Patch name for a particular bundle was suggested as follows > > e.g *org.wso2.carbon.clustering_5.0.0.jar* in the first patch add a new > patched bundle as *org.wso2.carbon.clustering_5.0.0.0001.jar* > > 3. When applying a patch > > unzip the applying zip to the * /c5-home/repository/patches* folder (zip > file structure would remain the same) > > > - for config files patch apply would remain as the way in the previous > mail. > - for OSGi bundles > > there will be an entry update in the *bundles.info <http://bundles.info>*file > to persist the changes in a server restart. > > for e.g > org.wso2.carbon.clustering,5.0.0,../../plugins/org.wso2.carbon.clustering_5.0.0.jar,4,true > will change to > > > org.wso2.carbon.clustering,5.0.0.0001,../../../patches/patch00001/org.wso2.carbon.clustering_5.0.0.0001.jar,4,true > > 4. Use the /c5-home/repository/patches/patch0000 folder for backup > purposes. > > > I will continue the POC for patching c5 servers with the above changes. > > Thanks, Manoj
_______________________________________________ Architecture mailing list [email protected] https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
