stackabletech / stackabletech/stackablectl
Include/Exclude filters do not split comma-separated values
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Rust
- Sterne
- 10
- Forks
- 5
- Ø Merge
- 4 Std. 41 Min.
- Gemergte PRs (30 T.)
- 4
Beschreibung
@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
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne beim stackablectl release install-Befehl und verfolge, wie die Optionen --include und --exclude geparst werden. Reproduziere das Problem mit stackablectl release install 25.3 -e airflow,druid; fertig ist die Aufgabe, wenn durch kommagetrennte Werte beide Operatoren ausgeschlossen werden oder der Befehl eindeutig einen ungültigen Wert meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- cli
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100