Skip to content
This repository has been archived by the owner on Jul 25, 2023. It is now read-only.

maintenance/update cli tooling #46

Merged
merged 13 commits into from
May 3, 2022
Merged

Conversation

wcarlsen
Copy link
Contributor

  • Install confluent cli tool instead of the soon to be EOL ccloud cli
  • Changed entrypoint login scripts and updated cli executable to confluent
  • Updated CcloudCluster class and changed parsing to utilize json
  • Alter list topics to utilize json output
  • Alter describe topics to utilize json output
  • Alter list service-accounts to utilize json output
  • use list service-account
  • Alter create service-accounts to utilize json output
  • Alter api-keys to utilize json output
  • Alter acls to utilize json output
  • Convert ACL UserId from long to string, to make it clear that it is not in use
  • Convert ACL serviceAccountId from long to string

@wcarlsen wcarlsen requested a review from SEQUOIIA April 29, 2022 12:20
@wcarlsen
Copy link
Contributor Author

@wcarlsen wcarlsen added the dependencies Pull requests that update a dependency file label Apr 29, 2022
@devex-sa
Copy link

CodeScene Delta Analysis Results

View detailed results in CodeScene

Recommended Review Level Detailed -- Increased risk for defects: The change involves more modules and modified lines of code than 99% of the historic high-risk change sets. The risk is higher as much of the code in this repo (99% of all commits) is written by other authors.

Review of New Files

0 new file(s) below the threshold for new code (9).
1 new file(s) where the code health is above the threshold.

Copy link
Contributor

@jdsmithit jdsmithit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is a good chance to look at adding a read me file?

Copy link
Member

@SEQUOIIA SEQUOIIA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love me some Confluent CLI updates. Weee.

@wcarlsen wcarlsen merged commit a77b92e into master May 3, 2022
@wcarlsen wcarlsen deleted the maintenance/update-cli-tooling branch May 3, 2022 08:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants