Include/Exclude filters do not split comma-separated values

Aberta
#375 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
48/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
rust
Domínio
cli

Direção de pesquisa

Comece pelo comando stackablectl release install e rastreie como as opções --include e --exclude são analisadas. Reproduza o problema com stackablectl release install 25.3 -e airflow,druid; está concluído quando valores separados por vírgulas excluírem ambos os operadores ou o comando informar claramente que o valor é inválido.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

type/bug

@xeniape found this bug

Affected version

25.3.0

Current and expected behavior

Current Behaviour:

stackablectl release install 25.3 -e airflow,druid

This installs all operators. None are excluded, because there is no operator named airflow,druid.

Expected behaviour:

The help text suggest you can specify multiple values (INCLUDED_PRODUCTS and EXCLUDED_PRODUCTS):

Options:
  -i, --include <INCLUDED_PRODUCTS>
          Whitelist of product operators to install

  -e, --exclude <EXCLUDED_PRODUCTS>
          Blacklist of product operators to install

stackablectl release install 25.3 -e airflow,druid

This should install all operators for the release, except airflow and druid.

Possible solution

Try to unmarshall the comma-separated list of operators (or at least error out).

Additional context

No response

Environment

No response

Would you like to work on fixing this bug?

maybe

Linguagem predominante
Rust
Estrelas
10
Forks
5
Merge médio
4h 41min
PRs com merge (30d)
4

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de stackabletech/stackablectl

Todas as issues de stackabletech/stackablectl

Issues semelhantes

Mais issues de Rust

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.