Skip to content

OSGi features cleanup#42

Merged
asfgit merged 2 commits into
apache:masterfrom
neykov:osgi/features-cleanup
Aug 22, 2016
Merged

OSGi features cleanup#42
asfgit merged 2 commits into
apache:masterfrom
neykov:osgi/features-cleanup

Conversation

@neykov

@neykov neykov commented Aug 19, 2016

Copy link
Copy Markdown
Member

neykov added 2 commits August 19, 2016 13:09
Split loading in chunks by using prerequisite flag.
</feature>

<feature name="brooklyn-boot" version="${project.version}" description="Bundles to start on boot">
<feature prerequisite="true">brooklyn-with-ui</feature>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I tried to build this I get:

[ERROR] Failed to execute goal org.apache.karaf.tooling:karaf-maven-plugin:4.0.4:assembly (default-assembly) on project apache-brooklyn: Unable to build assembly: Could not find matching feature for brooklyn-with-ui -> [Help 1]

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry about that, didn't push the latest changes. Should be all good now.

@geomacy

geomacy commented Aug 22, 2016

Copy link
Copy Markdown
Contributor

Looks good to me from review point of view and AMP starts successfully. Haven't tested a large number of restarts using the script though.

@Graeme-Miller

Copy link
Copy Markdown
Contributor

successfully tested

@asfgit asfgit merged commit afb6fbe into apache:master Aug 22, 2016
asfgit pushed a commit that referenced this pull request Aug 22, 2016
OSGi features cleanup

Depends on apache/brooklyn-server#303
@neykov neykov deleted the osgi/features-cleanup branch August 22, 2016 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants