Skip to content

tt cartridge #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
LeonidVas opened this issue Mar 31, 2022 · 0 comments · Fixed by #72
Closed

tt cartridge #34

LeonidVas opened this issue Mar 31, 2022 · 0 comments · Fixed by #72
Assignees

Comments

@LeonidVas
Copy link
Contributor

LeonidVas commented Mar 31, 2022

To work with the "cartridge" we want to reuse the https://github.com/tarantool/cartridge-cli module, except the following commands:

  • start
  • stop
  • status
  • connect

and probably:

  • build
  • pack
  • create

The following implementation is suggested:
Add "cartridge-cli" to "tt" as a submodule with some wrapper that will mask excluded commands and pass others. Exceptions must also be taken into account for the "help" command.

Commands:

  • start
  • stop
  • status
  • connect
    must be updated in this task to support "cartridge" functionality.
@LeonidVas LeonidVas added 8sp and removed 8sp labels Mar 31, 2022
@Totktonada Totktonada added teamE and removed teamE labels Mar 31, 2022
0x501D added a commit that referenced this issue Jun 19, 2022
This patchset adds cartridge-cli commands to tt:
*  admin
*  bench
*  build
*  clean
*  create
*  failover
*  pack
*  repair
*  replicasets

Closes #34
0x501D added a commit that referenced this issue Jun 19, 2022
Cartridge-cli must be patched in order to connect to tt cobra commands
tree. Also before build tt we must generate all cartridge-cli codegen
files.

Follow up #34
0x501D added a commit that referenced this issue Jun 19, 2022
Some long commands from cartridge-cli does not fit in tt help logic and tt
raisies nil pointer dereference error.

Follow up #34
0x501D added a commit that referenced this issue Jun 19, 2022
Cartridge-cli needs golang version >= 1.18

Follow up #34
@0x501D 0x501D mentioned this issue Jun 19, 2022
0x501D added a commit that referenced this issue Jun 21, 2022
Cartridge-cli needs golang version >= 1.18

Part of #34
0x501D added a commit that referenced this issue Jun 21, 2022
0x501D added a commit that referenced this issue Jun 21, 2022
This patchset adds cartridge-cli commands to tt:
*  admin
*  bench
*  build
*  create
*  failover
*  pack
*  repair
*  replicasets

Cartridge-cli must be patched in order to connect to tt cobra commands
tree. Also before build tt we must generate all cartridge-cli codegen
files.

Closes #34
0x501D added a commit that referenced this issue Jun 21, 2022
Some long commands from cartridge-cli does not fit in tt help logic and tt
raisies nil pointer dereference error.

Follow up #34
0x501D added a commit that referenced this issue Jun 21, 2022
This patchset adds cartridge-cli commands to tt:
*  admin
*  bench
*  build
*  create
*  failover
*  pack
*  repair
*  replicasets

Cartridge-cli must be patched in order to connect to tt cobra commands
tree. Also before build tt we must generate all cartridge-cli codegen
files.

Closes #34
0x501D added a commit that referenced this issue Jun 21, 2022
Some long commands from cartridge-cli does not fit in tt help logic and tt
raisies nil pointer dereference error.

Follow up #34
LeonidVas pushed a commit that referenced this issue Jun 21, 2022
Cartridge-cli needs golang version >= 1.18

Part of #34
LeonidVas pushed a commit that referenced this issue Jun 21, 2022
LeonidVas pushed a commit that referenced this issue Jun 21, 2022
This patchset adds cartridge-cli commands to tt:
*  admin
*  bench
*  build
*  create
*  failover
*  pack
*  repair
*  replicasets

Cartridge-cli must be patched in order to connect to tt cobra commands
tree. Also before build tt we must generate all cartridge-cli codegen
files.

Closes #34
LeonidVas pushed a commit that referenced this issue Jun 21, 2022
Some long commands from cartridge-cli does not fit in tt help logic and tt
raisies nil pointer dereference error.

Follow up #34
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 a pull request may close this issue.

3 participants