[
https://issues.apache.org/jira/browse/KNOX-3071?focusedWorklogId=939118&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-939118
]
ASF GitHub Bot logged work on KNOX-3071:
----------------------------------------
Author: ASF GitHub Bot
Created on: 21/Oct/24 08:31
Start Date: 21/Oct/24 08:31
Worklog Time Spent: 10m
Work Description: hanicz commented on code in PR #940:
URL: https://github.com/apache/knox/pull/940#discussion_r1808330808
##########
gateway-server/src/main/java/org/apache/knox/gateway/util/KnoxCLI.java:
##########
@@ -701,31 +717,34 @@ protected RemoteConfigurationRegistryClientService
getRemoteConfigRegistryClient
private class AliasListCommand extends Command {
- public static final String USAGE = "list-alias [--cluster clustername]";
+ public static final String USAGE = "list-alias [--cluster
cluster1:clusterN]";
public static final String DESC = "The list-alias command lists all of the
aliases\n" +
- "for the given hadoop --cluster. The
default\n" +
+ "for the given hadoop --cluster(s). The
default\n" +
"--cluster being the gateway itself.";
@Override
public void execute() throws Exception {
AliasService as = getAliasService();
- KeystoreService keystoreService = getKeystoreService();
+ KeystoreService keystoreService = getKeystoreService();
if (cluster == null) {
cluster = "__gateway";
}
- boolean credentialStoreForClusterAvailable =
- keystoreService.isCredentialStoreForClusterAvailable(cluster);
- if (credentialStoreForClusterAvailable) {
- out.println("Listing aliases for: " + cluster);
- List<String> aliases = as.getAliasesForCluster(cluster);
- for (String alias : aliases) {
- out.println(alias);
- }
- out.println("\n" + aliases.size() + " items.");
- } else {
- out.println("Invalid cluster name provided: " + cluster);
- }
+ String[] clusters = cluster.split(":");
Review Comment:
Yes, it makes more sense to use `,`. I also added it as a constant.
Issue Time Tracking
-------------------
Worklog Id: (was: 939118)
Time Spent: 0.5h (was: 20m)
> Batch list-alias and new create-list-aliases KnoxCLI commands
> -------------------------------------------------------------
>
> Key: KNOX-3071
> URL: https://issues.apache.org/jira/browse/KNOX-3071
> Project: Apache Knox
> Issue Type: New Feature
> Components: KnoxCLI
> Affects Versions: 2.1.0
> Reporter: Tamás Hanicz
> Priority: Minor
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> New feature for list-alias KnoxCLI command. The user should be able to
> provide multiple clusters concatenated by ':' and list the aliases for all of
> them.
> list-alias --cluster cluster1:cluster2:clusterN
> New create-list-aliases command that creates aliases on multiple clusters and
> list them at the end.
> create-list-aliases --alias alias1 --value value1 --cluster cluster1 --alias
> alias2 --value value2 --cluster2 --alias aliasN --value valueN --cluster
> clusterN
> When there is a --cluster args every alias before it will be added to that
> cluster.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
