Project

General

Profile

Bug #17304

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

Added by Swapnil Abnave almost 2 years ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Category:
Other commands
Target version:
Difficulty:
medium
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

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

Related to Foreman - Bug #18933: OS: unify description vs title - replace description with title on UIPending2017-03-16

History

#1 Updated by Swapnil Abnave almost 2 years ago

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

#2 Updated by Tomáš Strachota almost 2 years ago

  • Category changed from Foreman commands (obsolete) to Other commands

#3 Updated by Swapnil Abnave almost 2 years ago

  • Status changed from New to Need more information

#4 Updated by Swapnil Abnave over 1 year ago

  • Target version changed from 1.15.6 to 177

#5 Updated by Swapnil Abnave over 1 year ago

  • Status changed from Need more information to Assigned

#6 Updated by Swapnil Abnave over 1 year ago

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

#7 Updated by Swapnil Abnave over 1 year ago

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

#8 Updated by Swapnil Abnave over 1 year ago

  • Description updated (diff)
  • Assignee set to Swapnil Abnave

#9 Updated by Swapnil Abnave over 1 year ago

  • Status changed from Assigned to Pending

#10 Updated by Swapnil Abnave over 1 year ago

  • Target version changed from 177 to 1.17.0-RC1

#11 Updated by Swapnil Abnave about 1 year ago

  • Target version changed from 1.17.0-RC1 to 1.15.6

#12 Updated by Amit Karsale about 1 month 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?

#13 Updated by Tomáš Strachota about 1 month 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.

#14 Updated by Tomáš Strachota about 1 month ago

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