You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/09/23 07:58:25 UTC
[GitHub] [inlong] haifxu opened a new pull request, #6007: [INLONG-6004][Manager] Command tools add CRUD for inlong cluster node
haifxu opened a new pull request, #6007:
URL: https://github.com/apache/inlong/pull/6007
### Prepare a Pull Request
- Fixes #6004
### Motivation
Command tools add CRUD for inlong cluster node.
### Modifications
1..Add CRUD for inlong cluster tag.
### Verifying this change
1. managerctl list cluster-node
```
cluster-node Get cluster node summary information
Usage: cluster-node [options]
Options:
--tag
cluster tag
--type
cluster type
```
2. managerctl describe cluster-node
```
cluster-node Get cluster node details
Usage: cluster-node [options]
Options:
* -id, --id
cluster node id
```
3. managerctl create cluster-node
```
cluster-node Create cluster node by json file
Usage: cluster-node [options]
Options:
-f, --file
json file
```
4. managerctl update cluster-node
```
cluster-node Update cluster node by json file
Usage: cluster-node [options]
Options:
-f, --file
json file
```
5. managerctl delete cluster-node
```
cluster-node Delete cluster node by node id
Usage: cluster-node [options]
Options:
* -id, --id
cluster node id
```
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] haifxu commented on pull request #6007: [INLONG-6004][Manager] Command tools add CRUD for inlong cluster node
Posted by GitBox <gi...@apache.org>.
haifxu commented on PR #6007:
URL: https://github.com/apache/inlong/pull/6007#issuecomment-1257379896
> It is recommended to abstract CRUD as an interface or abstract class, and implement corresponding CRUD methods for various resources, so that the scalability is better.
Thanks for the suggestion, i will try this in the next pr.
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [inlong] healchow merged pull request #6007: [INLONG-6004][Manager] Command tools add CRUD for inlong cluster node
Posted by GitBox <gi...@apache.org>.
healchow merged PR #6007:
URL: https://github.com/apache/inlong/pull/6007
--
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@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org