Project

General

Profile

Actions

Bug #17304

closed

hammer os: unify --description vs. --title for its subcommands

Added by Swapnil Abnave over 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Other commands
Target version:
Difficulty:
medium
Triaged:
No
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

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:
  1. Keep --title everywhere, no --description
  2. Update command would need to have --new-title

Related issues 1 (1 open0 closed)

Related to Foreman - Bug #18933: OS: unify description vs title - replace description with title on UIPendingSwapnil Abnave03/16/2017Actions
Actions #1

Updated by Swapnil Abnave over 7 years ago

  • Project changed from Foreman to Hammer CLI
  • Category set to Foreman commands (obsolete)
Actions #2

Updated by Tomáš Strachota over 7 years ago

  • Category changed from Foreman commands (obsolete) to Other commands
Actions #3

Updated by Swapnil Abnave over 7 years ago

  • Status changed from New to Need more information
Actions #4

Updated by Swapnil Abnave about 7 years ago

  • Target version changed from 1.15.6 to 177
Actions #5

Updated by Swapnil Abnave about 7 years ago

  • Status changed from Need more information to Assigned
Actions #6

Updated by Swapnil Abnave about 7 years ago

ToDo:
  1. Keep --title everywhere, no --description
  2. Update command would need to have --new-title
Actions #7

Updated by Swapnil Abnave about 7 years ago

  • Related to Bug #18933: OS: unify description vs title - replace description with title on UI added
Actions #8

Updated by Swapnil Abnave about 7 years ago

  • Description updated (diff)
  • Assignee set to Swapnil Abnave
Actions #9

Updated by Swapnil Abnave about 7 years ago

  • Status changed from Assigned to Pending
Actions #10

Updated by Swapnil Abnave almost 7 years ago

  • Target version changed from 177 to 1.17.0-RC1
Actions #11

Updated by Swapnil Abnave over 6 years ago

  • Target version changed from 1.17.0-RC1 to 1.15.6
Actions #12

Updated by Amit Karsale over 5 years ago

  • Triaged set to No

The downstream of this bug(1393524) is closed saying WONTFIX, is this issue still holds valid at upstream and need a fix?

Actions #13

Updated by Tomáš Strachota over 5 years ago

  • 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.

Actions #14

Updated by Tomáš Strachota over 5 years ago

I closed the issue. Feel free to re-open if it causes troubles and you feel it should be fixed.

Actions

Also available in: Atom PDF