Skip to content
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

[INLONG-5102][Manager] Command tools add CRUD for inlong cluster #5958

Merged
merged 6 commits into from
Sep 21, 2022

Conversation

haifxu
Copy link
Contributor

@haifxu haifxu commented Sep 20, 2022

Prepare a Pull Request

Motivation

Manager tools supports CRUD for inlong cluster.

Modifications

1.Add CRUD for inlong cluster.
2.Add cluster type parameter validation.
2.Add cluster id validation in org.apache.inlong.manager.client.api.inner.client.InlongClusterClient#update.

Verifying this change

  1. managerctl list cluster
cluster      Get cluster summary information
      Usage: cluster [options]
        Options:
          --tag
            cluster tag
          --type
            cluster type
  1. managerctl describe cluster
cluster      Get cluster details
      Usage: cluster [options]
        Options:
        * -id, --id
            cluster id
  1. managerctl create cluster
cluster      Create cluster by json file
      Usage: cluster [options]
        Options:
          -f, --file
            json file
  1. managerctl update cluster
cluster      Update cluster by json file
      Usage: cluster [options]
        Options:
          -f, --file
            json file
  1. managerctl delete cluster
cluster      Delete cluster by cluster id
      Usage: cluster [options]
        Options:
        * -id, --id
            cluster id

@dockerzhang dockerzhang changed the title [INLONG-5102][Manager] Add CRUD for inlong cluster [INLONG-5102][Manager] Command tools add CRUD for inlong cluster Sep 20, 2022
@haifxu haifxu requested review from healchow and removed request for dockerzhang and yunqingmoswu September 21, 2022 06:55
@dockerzhang dockerzhang merged commit 767a0ed into apache:master Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature][Manager] Support CRUD for inlong cluster
3 participants