Enterprise create kafka interactive #1819
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jackdelahunt
reviewed
Mar 1, 2023
| } | ||
|
|
||
| var idx int | ||
| survey.AskOne(promptForCluster, &idx) |
There was a problem hiding this comment.
No error handling here
jackdelahunt
reviewed
Mar 1, 2023
| //create a map of cluster ids to clusters | ||
| clusterMap := make(map[string]*kafkamgmtclient.EnterpriseClusterListItem) | ||
| for _, cluster := range opts.kfmClusterList.Items { | ||
| clusterMap[*cluster.ClusterId] = &cluster |
There was a problem hiding this comment.
clusterMap[cluster.GetId()] = &cluster
dimakis
changed the title
jackdelahunt
pushed a commit
that referenced
this pull request
Mar 3, 2023
* refactor: moving list enterprise clusters to util package * feat: start of interactive enterprise create
jackdelahunt
pushed a commit
that referenced
this pull request
Mar 7, 2023
* refactor: moving list enterprise clusters to util package * feat: start of interactive enterprise create
jackdelahunt
added a commit
that referenced
this pull request
Mar 7, 2023
* Enterprise crete kafka interactive (#1819) * refactor: moving list enterprise clusters to util package * feat: start of interactive enterprise create * refactor: refactor of create kafka ams checks for enterprise quota (#1822) * Enterprise create kafka interactive refactor2 (#1824) * feat: when using dedicated with kafka create, we now check the capacity of the cluster * fix: now correct error on size greater then capacity * fix: correct error when no quota exists in RH infra * refactor: refactor of the interactive create kafka cmd * refactor: extract marketplace prompt to its own function * refactor: refactoring dedicated list to use refactored search string func * refactor: refactor and addition of name + id of cluster in prompt * refactor: input correct quota tag * refactor: refactor to functional create enterprise flow with some updates based on reviews * fix: more edge case checking in kafka create enterprise --------- Co-authored-by: Jack Delahunt <[email protected]> --------- Co-authored-by: Dimitri Saridakis <[email protected]>
jackdelahunt
pushed a commit
that referenced
this pull request
Mar 7, 2023
* refactor: moving list enterprise clusters to util package * feat: start of interactive enterprise create
dimakis
added a commit
that referenced
this pull request
Mar 8, 2023
* Enterprise crete kafka interactive (#1819) * refactor: moving list enterprise clusters to util package * feat: start of interactive enterprise create * refactor: refactor of create kafka ams checks for enterprise quota (#1822) * feat: when using dedicated with kafka create, we now check the capacity of the cluster * refactor: refactor of the interactive create kafka cmd * refactor: refactoring dedicated list to use refactored search string func * refactor: refactor and addition of name + id of cluster in prompt * refactor: refactor to functional create enterprise flow with some updates based on reviews * Add machine pool logic (#1795) * feat: addition of the new dedicated command to select ocm cluster * feat: addition of getting machine pools and hitting the register cluster endpoint in KFM * refactor: addition of close ocm connection * fix: addition of autoscaling check logic * refactor: addresses commments made in PR * refactor: addresses comments in review * feat: addition of the addon installation flow * feat: addition of cluster-id flag * refactor: addresses some further comments in the review * feat: addition of localization * refactor: addition of error when user has no valid clusters * chore(vendor): update vendoring (#1798) * Add cluster id to create kafka (#1804) * feat: addition of descriptions and examples to dedicated * feat: Allows the CLI to install stage env addons when terraforming customer-cloud dataplane clusters * refactor: the default api url is now set when creating a connection instead of in the registercluster cmd file * refactor: update some func and var names * refactor: add cluster-id flag to create kafka cmd to allow kafka creation on CC cluster * refactor: doc and small refactor * refactor: Refactor ocm methods out of register cmd file (#1807) * feat: addition of descriptions and examples to dedicated * feat: Allows the CLI to install stage env addons when terraforming customer-cloud dataplane clusters * refactor: extraction of clustermgmt funcs from register to own dir * refactor: addresses cluster pagination and refactor of ocm functions * feat: added deregister command * refactor: imrpoved quality of deregister command * chore: added localisation to deregister command * feat: deregister now waits for all kafkas to be deleted * fix: better logging when deleting kafka * refactor: getting cluster list now ask kfm then ocm * chore: remove unused fields * fix: edge case handling in deregister * docs: update the docs (#1826) * refactor: print the httpresponse (#1827) * fix: fixing broken lints * chore: rebase changes --------- Co-authored-by: Dimitri Saridakis <[email protected]> Co-authored-by: Ramakrishna Pattnaik <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #
Verification Steps
Type of change