-
1. Re: Consolidating release items in one place
mageshbk Sep 21, 2011 11:57 AM (in response to mageshbk)Tom, this is nothing to do with the assembly stuff that you are working on. But yes it will have a small impact.
Keith, the core dependencies are imported liek this
<dependency> <groupId>org.switchyard</groupId> <artifactId>switchyard-core-parent</artifactId> <version>${project.version}</version> <type>pom</type> <scope>import</scope> </dependency>
-
2. Re: Consolidating release items in one place
tfennelly Sep 21, 2011 12:06 PM (in response to mageshbk)Thanks Magesh.
Hmmm... not sure I like the idea of moving runtime code into the release module, even if the code in question is container (as6/as7) specific. Or am I misunderstanding something?
-
3. Re: Consolidating release items in one place
mageshbk Sep 21, 2011 1:43 PM (in response to tfennelly)This cropped up a couple of times.
http://community.jboss.org/message/609210
-
4. Re: Consolidating release items in one place
tfennelly Sep 22, 2011 4:52 AM (in response to mageshbk)Fair enough... I personally don't really think those things logically follow, but that's OK... no biggie
-
5. Re: Consolidating release items in one place
mageshbk Sep 22, 2011 8:27 AM (in response to mageshbk)Some gist on the changes:
- tests-as* moved to release/jboss-as*/tests
- release/jboss-as7/tests reused in release/jboss-as7/standalone/dist, release/jboss-as7/openshift/express/dist
- no changes in functionality just re-location of path, for commonality
-
6. Re: Consolidating release items in one place
tfennelly Sep 23, 2011 5:32 AM (in response to mageshbk)Hey magesh... I think the restructuring you've done in the release module looks great. As I said above, not sure I like the idea of moving runtime code into the release repo, but that's a different topic... the restructuring you did in release is a lot more sane than what's there atm for sure.
I'm working on SWITCHYARD-473 (https://issues.jboss.org/browse/SWITCHYARD-473) and there's major overlap with what you did and with what I'm doing there, so (as we were saying on IRC) we need to collaborate a bit. I think merging these is going to be a major pain
I've hit a point now where I think I need to create a common as7 "modules" assembly module, which does nothing more than just assemble all the as7 modules that are common to both standalone and openshift. Under your new structure, I think it would live in release/jboss-as7/modules and would be used by release/jboss-as7/standalone and release/jboss-as7/openshift.
I pushed what I currently have so you can take a peek (links below). I might make copies of my branches and see can I merge your changes... just to see what merge issues arrise.
-
7. Re: Consolidating release items in one place
tfennelly Sep 26, 2011 8:59 AM (in response to tfennelly)Here's Magesh's work merged with the work I did:
- https://github.com/tfennelly/jboss-switchyard-release/tree/SWITCHYARD-473-452
- https://github.com/tfennelly/jboss-switchyard-core/tree/SWITCHYARD-473-452
See the "modules" module in https://github.com/tfennelly/jboss-switchyard-release/tree/SWITCHYARD-473-452/jboss-as7
I't not finished yet.. need to move in more modules from the components etc, but you should be able to see the general direction.
-
8. Re: Consolidating release items in one place
kcbabo Sep 26, 2011 9:49 AM (in response to tfennelly)I haven't had a chance to review the changes yet, but some general comments on this subject:
Component and runtime modules should *not* package themselves for any target runtime. This should be done as part of the assembly/packaging configuration.
Distro-specific code does not belong in the component/core repositories. This includes deployer code for AS6/AS7. At this point, the release repository is the most appropriate place to hold this code. The release repo is really more than just a build repository - it contains platform-specific code and build support.
-
9. Re: Consolidating release items in one place
kcbabo Sep 26, 2011 12:11 PM (in response to mageshbk)Hey Magesh,
I had a look and this looks reasonable to me. What's the purpose of the standalone module inside of the AS6 and AS7 modules? Will there be another module besides standalone there at some point? Or can we simply move build and extensions up a level?
cheers,
keith
Magesh Bojan wrote:
As part of this JIRA I thought it would be better to finally put this topic to sleep. Move the core AS6, AS7 deployment stuff to release. Here are the changes, let me know your comments:
-
10. Re: Consolidating release items in one place
tfennelly Sep 26, 2011 3:38 PM (in response to tfennelly)AS7 module assembly cleaned up a bit more... https://github.com/tfennelly/jboss-switchyard-release/tree/SWITCHYARD-473-452/jboss-as7/modules
Look how the assembly.xml <includes> assembly.xml files.
-
11. Re: Consolidating release items in one place
kcbabo Sep 26, 2011 9:01 PM (in response to tfennelly)This is looking much better. Between Magesh's change and your latest update to modularize the assembly packaging, things are a lot cleaner. We should add a README which includes details on what you should check/change when adding another module assembly. I assume that anything built from SwitchYard repositories goes in src/main/resources/switchyard and everything else goes in src/main/resources/external ? We should add a big comment block to modules/pom.xml to separate out the switchyard vs. external dependencies.
One other thing to keep in mind is that we will want to start providing a stand-alone installer in the near future that allows you to overlay a set of SwitchYard modules and config on top of an existing AS7 install. This is nice for users with an existing AS7 install and it also will provide a download which is a bit more manageable in size. I imagine that the approach we are taking here with modularization will be quite consistent with this approach, but wanted to throw it out there just in case you see any issues.
Tom Fennelly wrote:
AS7 module assembly cleaned up a bit more... https://github.com/tfennelly/jboss-switchyard-release/tree/SWITCHYARD-473-452/jboss-as7/modules
Look how the assembly.xml <includes> assembly.xml files.
-
12. Re: Consolidating release items in one place
mageshbk Sep 26, 2011 10:32 PM (in response to kcbabo)Keith Babo wrote:
Hey Magesh,
I had a look and this looks reasonable to me. What's the purpose of the standalone module inside of the AS6 and AS7 modules? Will there be another module besides standalone there at some point? Or can we simply move build and extensions up a level?
My idea was maybe someday someone will add AS6 to OpenShift . I think we can move the extensions/deployer one level up. But the distribution assembly should be under standalone. I will send a pull request soon.
-
13. Re: Consolidating release items in one place
mageshbk Sep 26, 2011 10:37 PM (in response to tfennelly)Look how the assembly.xml <includes> assembly.xml files.
That is super cool .
-
14. Re: Consolidating release items in one place
kcbabo Sep 26, 2011 10:50 PM (in response to mageshbk)The question I'm asking is what is the significance of the term 'standalone'? We are not going to have a separate deployer for domain deployments vs. standalone deployments, are we? Assuming that we won't, I would prefer to change the name to remove any confusion with the standalone mode in AS7.
Magesh Bojan wrote:
My idea was maybe someday someone will add AS6 to OpenShift . I think we can move the extensions/deployer one level up. But the distribution assembly should be under standalone. I will send a pull request soon.