Project

General

Profile

Actions

Bug #31536

closed

'hammer host puppetrun' (deprecated) not marked as deprecated in hammer

Added by Dominik Matoulek almost 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Category:
Other commands
Target version:
-
Difficulty:
Triaged:
Yes
Team Backlog:
Found in Releases:
In Kanboard:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1907807

Description of problem:
The puppetrun functionality was deprecated. However, hammer still shows original help for the puppetrun subcommand and 'not implemented' for invocation

How reproducible:
always

Steps to Reproduce:
1. # hammer host puppetrun --help
Usage:
hammer host puppetrun [OPTIONS]

Options:
--id ID
--location[-id|-title] Set the current location context for the request. Name/Title/Id can be used
--name NAME Name to search by
--organization[-id|-title] Set the current organization context for the request. Name/Title/Id can be used
-h, --help Print help

2. # hammer host puppetrun --organization-id 1 --location-id 1 --id 2
Error: 501 Not Implemented

Actual results:
1. shows original help
2. shows 'Not implemented'

Actions #1

Updated by Dominik Matoulek almost 4 years ago

  • Subject changed from 'hammer host puppetrun' (deprecated) not marked as deprecated in hammer to 'hammer host puppetrun' (deprecated) not marked as deprecated in hammer
Actions #2

Updated by Dominik Matoulek almost 4 years ago

  • Triaged changed from No to Yes
Actions #3

Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Dominik Matoulek
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/554 added
Actions #4

Updated by The Foreman Bot almost 4 years ago

  • Fixed in Releases hammer-cli-foreman-2.4.0 added
Actions #5

Updated by Anonymous almost 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #6

Updated by Shira Maximov almost 4 years ago

  • Fixed in Releases hammer-cli-foreman-2.3.1 added
Actions

Also available in: Atom PDF