You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by neykov <gi...@git.apache.org> on 2016/03/09 17:46:00 UTC
[GitHub] brooklyn-server pull request: [OSGI] brooklyn.properties & CAMP pl...
GitHub user neykov opened a pull request:
https://github.com/apache/brooklyn-server/pull/52
[OSGI] brooklyn.properties & CAMP platform init
Also share the code with classic launcher.
Getting an exception in the logs, which is coming from the `cxf-async-client` jar, used by winrm. Seems like a bundle init ordering issue. Can be ignored for the time being, the service being registered there is not used.
```
2016-03-09 18:41:49,298 | ERROR | ort.http.async]) | configadmin | 3 - org.apache.felix.configadmin - 1.8.8 | [org.osgi.service.cm.ManagedService, id=146, bundle=84/mvn:org.apache.cxf/cxf-rt-transports-http-hc/3.1.4]: Unexpected problem updating configuration org.apache.cxf.transport.http.async
java.lang.NullPointerException
at org.apache.cxf.transport.http.asyncclient.AsyncHTTPConduitFactory.addListener(AsyncHTTPConduitFactory.java:285)[84:org.apache.cxf.cxf-rt-transports-http-hc:3.1.4]
at org.apache.cxf.transport.http.asyncclient.AsyncHTTPConduitFactory.<init>(AsyncHTTPConduitFactory.java:144)[84:org.apache.cxf.cxf-rt-transports-http-hc:3.1.4]
at org.apache.cxf.transport.http.asyncclient.Activator$ConduitConfigurer.updated(Activator.java:79)[84:org.apache.cxf.cxf-rt-transports-http-hc:3.1.4]
at org.apache.felix.cm.impl.helper.ManagedServiceTracker.updated(ManagedServiceTracker.java:189)[3:org.apache.felix.configadmin:1.8.8]
at org.apache.felix.cm.impl.helper.ManagedServiceTracker.updateService(ManagedServiceTracker.java:152)[3:org.apache.felix.configadmin:1.8.8]
at org.apache.felix.cm.impl.helper.ManagedServiceTracker.provideConfiguration(ManagedServiceTracker.java:85)[3:org.apache.felix.configadmin:1.8.8]
at org.apache.felix.cm.impl.ConfigurationManager$ManagedServiceUpdate.provide(ConfigurationManager.java:1444)[3:org.apache.felix.configadmin:1.8.8]
at org.apache.felix.cm.impl.ConfigurationManager$ManagedServiceUpdate.run(ConfigurationManager.java:1400)[3:org.apache.felix.configadmin:1.8.8]
at org.apache.felix.cm.impl.UpdateThread.run0(UpdateThread.java:143)[3:org.apache.felix.configadmin:1.8.8]
at org.apache.felix.cm.impl.UpdateThread.run(UpdateThread.java:110)[3:org.apache.felix.configadmin:1.8.8]
at java.lang.Thread.run(Thread.java:745)[:1.7.0_80]
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/neykov/brooklyn-server osgi/brooklyn.properties
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/brooklyn-server/pull/52.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #52
----
commit e6b3ad900228d206f671f6e8b0350e95441e8ed1
Author: Svetoslav Neykov <sv...@cloudsoftcorp.com>
Date: 2016-03-08T09:56:10Z
[BROOKLYN-183] Load brooklyn.properties on OSGi init
commit ec3cc790a13363f07108725f1d810e9a244f35d5
Author: Svetoslav Neykov <sv...@cloudsoftcorp.com>
Date: 2016-03-08T10:06:01Z
[BROOKLYN-183] Reuse classic init code, loading CAMP platform
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-server pull request: [OSGI] brooklyn.properties & CAMP pl...
Posted by neykov <gi...@git.apache.org>.
Github user neykov commented on the pull request:
https://github.com/apache/brooklyn-server/pull/52#issuecomment-194962725
Decided to stop troubleshooting before investing too much time - at the point where I figured out it's not really required for everything to work. But agree it needs to be addressed sooner or later.
Re missing items:
* locations - due to service loader
* catalog - there's something more going on there, will troubleshoot once this + the service loader fix lands. It's a non-critical problem - brooklyn still can be used without it, so better address the immediate problems first.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-server pull request: [OSGI] brooklyn.properties & CAMP pl...
Posted by neykov <gi...@git.apache.org>.
Github user neykov commented on the pull request:
https://github.com/apache/brooklyn-server/pull/52#issuecomment-208857690
NPE fixed in https://github.com/apache/cxf/pull/128 and https://github.com/apache/cxf/pull/129. Confirming it's harmless and can be ignored, but can't be suppressed from Brooklyn's side.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-server pull request: [OSGI] brooklyn.properties & CAMP pl...
Posted by ahgittin <gi...@git.apache.org>.
Github user ahgittin commented on the pull request:
https://github.com/apache/brooklyn-server/pull/52#issuecomment-194958381
code changes look good and i've confirmed it builds and runs fine. fewer items (apps, locations, etc) are still being pulled in by default when in osgi mode, but i think that's the `ServiceLoader` issue.
worth spending a few minutes to see if that error can be eliminated because it will frighten people but good to merge. (i'll leave it to you in case the answer to the above question is different.)
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-server pull request: [OSGI] brooklyn.properties & CAMP pl...
Posted by ahgittin <gi...@git.apache.org>.
Github user ahgittin commented on the pull request:
https://github.com/apache/brooklyn-server/pull/52#issuecomment-194927141
nice
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] brooklyn-server pull request: [OSGI] brooklyn.properties & CAMP pl...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/brooklyn-server/pull/52
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---