Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-584

ZooKeeper service instance should be registered in the OSGi registry

        Details

        • Type: New Feature
        • Status: Open
        • Priority: Major
        • Resolution: Unresolved
        • Affects Version/s: None
        • Fix Version/s: None
        • Component/s: server
        • Labels:
          None

          Description

          When Zookeeper is booted in an OSGi framework by ManagedService it would be quite handy to have ManagedService register a management interface for that ZooKeeper instance into the OSGi service registry.

            Issue Links

            depends upon

            Improvement - An improvement or enhancement to an existing feature or task. ZOOKEEPER-425 Add OSGi metadata to zookeeper.jar

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              bosschaert David Bosschaert added a comment -

              Yeah, what you'd really want is an OSGi bundle that launches the Zookeeper Service from within the OSGi Framework. It should be configurable through the OSGi Configuration Admin Service and support changes in configuration after the server was launched.

              An initial attempt to provide some of this functionality has already been done in the context of ZOOKEEPER-425. Look at the Activator.java and ManagedService.java of zk_patch3.patch in that bug.

              Show
              bosschaert David Bosschaert added a comment - Yeah, what you'd really want is an OSGi bundle that launches the Zookeeper Service from within the OSGi Framework. It should be configurable through the OSGi Configuration Admin Service and support changes in configuration after the server was launched. An initial attempt to provide some of this functionality has already been done in the context of ZOOKEEPER-425 . Look at the Activator.java and ManagedService.java of zk_patch3.patch in that bug.
              Hide
              Permalink
              bosschaert David Bosschaert added a comment -

              For the moment I put the bundle that runs the zookeeper server in the CXF-DOSGi codebase (because I'm using the functionality there).
              It can configure (through the OSGi Configuration Admin Service) and run Zookeeper Server.
              Reconfiguration is also supported (just change the configuration information on the PID: org.apache.cxf.dosgi.discovery.zookeeper.server and the reconfiguration is applied by shutting down and starting up the zookeeper server in-process.

              You can find it here: http://svn.apache.org/repos/asf/cxf/dosgi/trunk/discovery/distributed/zookeeper-server

              Let me know when you think the zookeeper codebase is ready for this. It works for me, but I understand that the general feeling is that some changes need to be made to the zookeeper core before something like this can be accepted.
              I'd be happy to contribute this to zookeeper if you like to have it.

              Show
              bosschaert David Bosschaert added a comment - For the moment I put the bundle that runs the zookeeper server in the CXF-DOSGi codebase (because I'm using the functionality there). It can configure (through the OSGi Configuration Admin Service) and run Zookeeper Server. Reconfiguration is also supported (just change the configuration information on the PID: org.apache.cxf.dosgi.discovery.zookeeper.server and the reconfiguration is applied by shutting down and starting up the zookeeper server in-process. You can find it here: http://svn.apache.org/repos/asf/cxf/dosgi/trunk/discovery/distributed/zookeeper-server Let me know when you think the zookeeper codebase is ready for this. It works for me, but I understand that the general feeling is that some changes need to be made to the zookeeper core before something like this can be accepted. I'd be happy to contribute this to zookeeper if you like to have it.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  maguro Alan Cabrera
                • Votes:
                  2 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:

                    Development