Bug #17304
closed
hammer os: unify --description vs. --title for its subcommands
Added by Swapnil Abnave about 8 years ago.
Updated over 6 years ago.
Description
Description of problem:
Upon os creation we refer to an entity as '--description' and then upon os update/info/deletion we refer to the very same entity as '--title'
Version-Release number of selected component (if applicable):
Sat6.3.0 Snap6
Steps to Reproduce:
1. # hammer os create --name RedHat --description 'RHEL Server 7.3' --major 7 --minor 3 --media 'Internal RHEL'
2. # hammer os update --title 'RHEL Server 7.3' --media 'Internal RHEL'
3. # hammer os info --title 'RHEL Server 7.3'
4. # hammer os delete --title 'RHEL Server 7.3'
Actual results:
create expects --description VS. update/info/delete expect --title
Expected results:
all subcommands expects the same (preferably --title as --description is only one exception to the rule)
ToDo:
- Keep --title everywhere, no --description
- Update command would need to have --new-title
Related issues
1 (1 open — 0 closed)
- Project changed from Foreman to Hammer CLI
- Category set to Foreman commands (obsolete)
- Category changed from Foreman commands (obsolete) to Other commands
- Status changed from New to Need more information
- Target version changed from 1.15.6 to 177
- Status changed from Need more information to Assigned
ToDo:
- Keep --title everywhere, no --description
- Update command would need to have --new-title
- Related to Bug #18933: OS: unify description vs title - replace description with title on UI added
- Description updated (diff)
- Assignee set to Swapnil Abnave
- Status changed from Assigned to Pending
- Target version changed from 177 to 1.17.0-RC1
- Target version changed from 1.17.0-RC1 to 1.15.6
The downstream of this bug(1393524) is closed saying WONTFIX, is this issue still holds valid at upstream and need a fix?
- Status changed from Pending to Closed
Normally I'd say keep this open to track the issue. In this case I think we can close it too. Even though it might not be the best UX hammer behaves consistently with the UI. Changing this will require redesign of the server part first.
I closed the issue. Feel free to re-open if it causes troubles and you feel it should be fixed.
Also available in: Atom
PDF