Github user neykov commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/106#discussion_r59538390
--- Diff:
core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java
---
@@ -74,6 +80,17 @@ public void unbind(ServiceReference<ManagementContext>
managementContext) throws
public class CatalogPopulator extends BundleTracker<Iterable<? extends
CatalogItem<?, ?>>> {
+ private final String BROOKLYN_PACKAGE_REGEX =
"org.apache.brooklyn.*";
+
+ private final ConfigKey<List<String>> WHITELIST =
+ ConfigKeys.newConfigKey(new TypeToken<List<String>>(){},
"brooklyn.catalog.osgi.application.whitelist",
+ "CSV Whitelist of patterns to match bundle symbolic id if
bundle is to be permitted to add " +
+ "applications (templates) to the catalog",
ImmutableList.of(BROOKLYN_PACKAGE_REGEX));
--- End diff --
If I am evaluating Brooklyn and load my own bundle in the container I'd
expect that its `catalog.bom` is picked up out of the box. I think Brooklyn
should accept all bundles, letting downstream distributions set their own
policy.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
