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.