[HDDS-6382] EC: ListPipelines command should consider EC Containers

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: EC-Branch
• Component/s: SCM
• Labels: pull-request-available
• Assignee: Stephen O'Donnell
• Reporter: Stephen O'Donnell
• Created: 28/Feb/22 15:14
• Updated: 08/Mar/22 17:34
• Resolved: 08/Mar/22 17:34

Description

In ListPipeline.java we have:

@Override
public void execute(ScmClient scmClient) throws IOException {
  Stream<Pipeline> stream = scmClient.listPipelines().stream();
  if (!Strings.isNullOrEmpty(factor)) {
    stream = stream.filter(
        p -> ReplicationConfig.getLegacyFactor(p.getReplicationConfig())
            .toString().compareToIgnoreCase(factor) == 0);
  }
  if (!Strings.isNullOrEmpty(state)) {
    stream = stream.filter(p -> p.getPipelineState().toString()
        .compareToIgnoreCase(state) == 0);
  }
  stream.forEach(System.out::println);
} 

This code will fail if the pipeline is an EC pipeline. We should also allow for filtering on EC pipelines.

Issue Links

• links to: GitHub Pull Request #3161