stackabletech / stackabletech/stackablectl
Include/Exclude filters do not split comma-separated values
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Rust
- Estrelas
- 10
- Forks
- 5
- Merge médio
- 4h 41min
- PRs com merge (30d)
- 4
Descrição
@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
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
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.
Avaliação
- Stack de tecnologia
- rust
- Domínio
- cli
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 48/100