phunt / zookeeper_dashboard

Django based dashboard for an Apache ZooKeeper cluster.
http://hadoop.apache.org/zookeeper/
Apache License 2.0
166 stars 47 forks source link

Update the repo? #9

Open paetling opened 8 years ago

paetling commented 8 years ago

@phunt I stumbled upon this repo last week and decided I wanted to use it to help monitor our zookeeper cluster. I forked the repo and did a bunch of cleanup:

  1. Got rid of compiled binary file and switched to using kazoo
  2. Upgraded to use django 1.9
  3. Added a requirements.txt file
  4. Made it really easy to run gunicorn in front of the server.

I have some debug statements I need to take out, remove the Dockerfile, and add back in some files, but I was wondering if you were interested in merging these updates back into this repo.

phunt commented 8 years ago

Hi @paetling . Sometime back the upstream Apache project (ZooKeeper) took on continued development of the dashboard. See https://issues.apache.org/jira/browse/ZOOKEEPER-701 (notice it kinda morphed into something else...)

As a result I haven't been actively developing this project, however I'm happy to pull changes if people are still finding it interesting.

I notice immediately that you've removed some important files. The license, readme, etc... I would not feel comfortable merging that change, e.g. removing Apache licensing from the project. Would you be able to review/address and submit a squashed patch?

I notice docker support was added, which is quite cool. Adding details on that to the README would be nice. :-)

Regards!