[GitHub] [bookkeeper] dlg99 opened a new pull request, #3713: [FEATURE] Added api/v1/bookie/cluster_info REST API

[GitBox <gi...@apache.org>]

dlg99 opened a new pull request, #3713:
URL: https://github.com/apache/bookkeeper/pull/3713

   cherry-pick of PR 3710, with minor conflicts resolved
   
   Descriptions of the changes in this PR:
   
   Information provided by current REST API is not enough (and cumbersome to combine) to answer such question as "is any data in danger if I shut down one more bookie". E.g. getting list of underreplicated ledgers can get some info but it is either fast (no ledgers) or can be super slow on large cluster with some bookies lost (it retrieves full list of ledgers). Even if there are no UR ledgers it still possible that the problem is that Auditor is down etc.
   
    Added api/v1/bookie/cluster_info REST API
   
   ```
   curl -s 127.0.0.1:8080/api/v1/bookie/cluster_info
   {
     "auditorElected" : false,
     "auditorId" : "",
     "clusterUnderReplicated" : false,
     "ledgerReplicationEnabled" : true,
     "totalBookiesCount" : 1,
     "writableBookiesCount" : 1,
     "readonlyBookiesCount" : 0,
     "unavailableBookiesCount" : 0
   }%
   ```
   
   Side-fix:
   `org.apache.bookkeeper.stream.cluster.StandaloneStarter` (used by bookie standalone) did not pass `LedgerManagerFactory` to the http server thus REST calls that needed it didn't work.
   
   Reviewers: Nicolò Boschi <bo...@gmail.com>, Enrico Olivelli <eo...@gmail.com>
   
   This closes #3710 from dlg99/rest-cluster-info
   
   (cherry picked from commit 032aef7e75f6b3487a6611a2b8af8306b9a3f7bb)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [bookkeeper] dlg99 merged pull request #3713: [FEATURE] Added api/v1/bookie/cluster_info REST API

[GitBox <gi...@apache.org>]

dlg99 merged PR #3713:
URL: https://github.com/apache/bookkeeper/pull/3713


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org