Skip to content

Add bookkeeper ledger commands - #83

Closed
zymap wants to merge 11 commits into
masterfrom
bookie-command
Closed

Add bookkeeper ledger commands#83
zymap wants to merge 11 commits into
masterfrom
bookie-command

Conversation

@zymap

@zymap zymap commented Oct 14, 2019

Copy link
Copy Markdown
Member

Motivation

Introduce a admin client for bookKeeper.
Add ledger commands to the admin client

Modifications

  • adjust the admin to provide the bookKeeper service
  • Add ledger commands

Ouput:

➜  pulsarctl git:(bookie-command) ./pulsarctl bookies ledger delete -h
USED FOR:
    This command is used for deleting a ledger.

REQUIRED PERMISSION:
    none

OUTPUT:
    #normal output
    Successfully delete the ledger (ledger-id)

Usage: pulsarctl bookies ledger delete [flags]

Examples:
    #Delete the specified ledger
    pulsarctl bookies ledger delete --ledger-id (ledger-id)



Ledger flags:
      --ledger-id int   the delete ledger id (default -1)

Common flags:
  -s, --admin-service-url string    The admin web service url that pulsarctl connects to. (default "http://localhost:8080")
      --auth-params string          Authentication parameters are used to configure the public and private key files required by tls
                                     For example: "tlsCertFile:val1,tlsKeyFile:val2"
      --bookie-service-url string   The bookie web service url that pulsarctl connects to. (default "http://localhost:8081")
  -C, --color string                toggle colorized logs (true,false,fabulous) (default "true")
  -h, --help                        help for this command
      --tls-allow-insecure          Allow TLS insecure connection
      --tls-trust-cert-pat string   Allow TLS trust cert file path
  -v, --verbose int                 set log level, use 0 to silence, 4 for debugging (default 3)

Use 'pulsarctl bookies ledger delete [command] --help' for more information about a command.

---

*Motivation*

Introduce a admin client for bookKeeper.
Add ledger commands to the admin client

*Modifications*

- adjust the `admin` to provide the bookKeeper service
- Add ledger commands
@zymap zymap changed the title Add bookie ledger commands [WIP] Add bookie ledger commands Oct 14, 2019
@zymap

zymap commented Oct 14, 2019

Copy link
Copy Markdown
Member Author

@sijie I add the bookie client into the pulsarctl. Can you take a quick look to see if I'm on the right track? Thanks!

Comment thread pkg/pulsar/api_version.go
@sijie sijie mentioned this pull request Oct 15, 2019
29 tasks
@zymap zymap changed the title [WIP] Add bookie ledger commands Add bookie ledger commands Oct 16, 2019
@zymap

zymap commented Oct 16, 2019

Copy link
Copy Markdown
Member Author

@sijie PTAL

@zymap zymap changed the title Add bookie ledger commands Add bookkeeper ledger commands Oct 16, 2019
@zymap zymap self-assigned this Oct 17, 2019
@sijie sijie added this to the 0.2.0 milestone Oct 24, 2019
@zymap zymap mentioned this pull request Oct 31, 2019
5 tasks
@zymap zymap closed this Nov 7, 2019
@wolfstudy
wolfstudy deleted the bookie-command branch November 8, 2019 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants