stackabletech / stackabletech/stackablectl
Include/Exclude filters do not split comma-separated values
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Rust
- Estrellas
- 10
- Forks
- 5
- Merge medio
- 4 h 41 min
- PR fusionados (30 d)
- 4
Descripción
@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
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con el comando stackablectl release install y sigue cómo se analizan las opciones --include y --exclude. Reproduce el problema con stackablectl release install 25.3 -e airflow,druid; se considera hecho cuando los valores separados por comas excluyen ambos operadores o el comando informa claramente de que el valor no es válido.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust
- Área
- cli
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100