Project

General

Profile

Actions

Bug #14272

closed

hammer activation-key still mentions 'content-host'

Added by Zach Huntington-Meath over 8 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Hammer
Target version:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1317682
Description of problem:

Current version of hammer shows that the subcommand 'activation-key' still shows the term 'content-host'. We should not be using 'content-host' anymore, and instead use 'host'.

hammer> activation-key create -h
Usage:
activation-key create [OPTIONS]

Options:
--content-view CONTENT_VIEW_NAME Content view name to search by
--content-view-id CONTENT_VIEW_ID content view numeric identifier
--description DESCRIPTION description
--lifecycle-environment LIFECYCLE_ENVIRONMENT_NAME Name to search by
--lifecycle-environment-id LIFECYCLE_ENVIRONMENT_ID
--max-content-hosts MAX_CONTENT_HOSTS maximum number of registered content hosts
--name NAME name
--organization ORGANIZATION_NAME Organization name to search by
--organization-id ORGANIZATION_ID organization ID
--organization-label ORGANIZATION_LABEL Organization label to search by
--unlimited-content-hosts Set content hosts max to unlimited
-h, --help print help

So the following changes will be required:

  • max-content-hosts > max-hosts
  • unlimited-content-hosts > unlimited-hosts

It is possible that other subcommands are still referring to 'content-host', so a thorough investigation and fix will be required.

Version-Release number of selected component (if applicable):

  • Satellite 6.2.0 SNAP 3.1

How reproducible:

Steps to Reproduce:
1. Check the output of running the activation-key hammer command and check for 'content-host' in its output
2.
3.

Actual results:

Expected results:

Additional info:

Actions #1

Updated by The Foreman Bot over 8 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/5899 added
Actions #2

Updated by Zach Huntington-Meath over 8 years ago

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

Updated by Eric Helms over 8 years ago

  • Translation missing: en.field_release set to 86
Actions #4

Updated by The Foreman Bot over 8 years ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/369 added
Actions

Also available in: Atom PDF