Bug #18663

hammer subnet create/update shouldn't contain option --subnet-parameters-attributes

Added by Tomáš Strachota 9 months ago. Updated 8 months ago.

Status:Closed
Priority:Low
Assigned To:Dominik Hlavac Duran
Category:Networking
Target version:Foreman - Team Marek backlog
Difficulty: Bugzilla link:1426612
Found in release: Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/292
Story points-
Velocity based estimate-
Releasehammer-cli-foreman-0.11.0Release relationshipAuto

Description

Description of problem:

Commands hammer subnet create/update contain option --subnet-parameters-attributes that is generated automatically from apidoc. The option should be removed and replaced with set-parameter and delete-parameter commands as how it's done for other resources that have parameters (eg. domains).

Version-Release number of selected component (if applicable):
foreman 1.14
hammer-cli-foreman 0.9.0

Steps to Reproduce:
1. run: hammer subnet create -h
2. run: hammer subnet update -h

Actual results:
Help output for the both commands contain option --subnet-parameters-attributes

Expected results:
The option should be removed and there should be two new commands:
hammer subnet set-parameter
hammer subnet delete-parameter

Associated revisions

Revision 8b12dea5
Added by Dominik Hlavac 8 months ago

Fixes #18663 - replaces --subnet-parameters-attributes with parameter commands

History

#1 Updated by Tomáš Strachota 9 months ago

  • Project changed from Foreman to Hammer CLI
  • Subject changed from hammer subnet create/update shouldn't contain option --subnet-parameters-attributes to hammer subnet create/update shouldn't contain option --subnet-parameters-attributes
  • Category changed from Network to Networking
  • Target version set to Team Marek backlog

#2 Updated by Dominik Hlavac Duran 8 months ago

  • Status changed from New to Assigned
  • Assigned To set to Dominik Hlavac Duran
  • Priority changed from Normal to Low

#3 Updated by The Foreman Bot 8 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/292 added

#4 Updated by Tomáš Strachota 8 months ago

  • Release set to hammer-cli-foreman-0.11.0

#5 Updated by Anonymous 8 months ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF