StackStorm/st2

'st2 <thing> delete' should accept multiple positional args for removal

Open

#4,729 建立於 2019年6月28日

在 GitHub 查看
 (0 留言) (2 反應) (0 負責人)Python (5,794 star) (740 fork)batch import
CLIchatopscomponent:st2clientenhancementgood first issue

描述

Initially comes from a release testing when specific testing pack installs 100+ action aliases and we have no way to delete them in a bulky way.

The following construct fails because st2 action-alias delete can only accept single positional arg as id:

st2 action-alias delete pack.alias1 pack.alias2

Better example that ideally should work:

st2 action-alias delete $(st2 action-alias list -p chatops_tests -a id | awk '{print $2}')

The general CLI improvement here and helpful from user's perspective would be allowing multiple IDs for removal. From what I remember, only st2 pack delete 1 2 3 4 5 only works this way.

Affected CLI:

st2 action delete
st2 action-alias delete
st2 key delete
st2 policy delete
...
and so on

貢獻者指南

技術棧
pythonshell
領域
clibackend
議題類型
feature
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
half day
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
fresh
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
understanding of CLI designPython programming
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
40
研究方向
The issue requests adding support for multiple positional arguments in several delete CLI commands. Start by examining the existing implementation of `st2 pack delete` which already supports multiple IDs. Look at the command registration and argument parsing in the st2client package, likely in files like `st2client/commands/action.py` or `st2client/commands/resource.py`. The pattern for accepting multiple args is likely using `nargs='+'` in argparse. Identify all affected commands listed in the issue and apply the same pattern. No linked PRs or comments exist, so the change is straightforward once the pattern is understood.