Project

General

Profile

Actions

Bug #21375

open

Organization-title option is missing for hammer-cli-katello commands

Added by Stanislav Tkachenko about 7 years ago. Updated over 6 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Hammer
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

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

Description of problem:
Taxonomy titles were added in https://github.com/theforeman/hammer-cli-foreman/pull/299
However, accordint to @Tomáš Strachota hammer-cli-katello overrides setting for such organization related options across the plugin, so `location-title` is present while `organization-title` is missing. For example, `hammer host-collection hosts --help`

Version-Release number of selected component (if applicable):
Satellite 6.3 Snap 20:
  • satellite-6.3.0-20.0.beta.el7sat.noarch
  • katello-3.4.5-4.el7sat.noarch
  • foreman-1.15.4.8-1.el7sat.noarch
  • tfm-rubygem-hammer_cli_foreman-0.11.0.1-1.el7sat.noarch
  • tfm-rubygem-hammer_cli_katello-0.11.0-1.el7sat.noarch

How reproducible:
Always

Steps to Reproduce:
1. hammer host-collection hosts --help
2. Check whether both location-title and organization-title are present
3.

Actual results:
--location-title is available.
--organization-title is missing.

Expected results:
Options for both taxonomy titles are present.

Actions #1

Updated by Tomáš Strachota about 7 years ago

  • Project changed from Foreman to Katello
  • Category set to Hammer
Actions #2

Updated by Tomáš Strachota about 7 years ago

The issue is caused by hammer-cli-katello's id resolver that overrides the foreman's original searchables for organizations:
https://github.com/Katello/hammer-cli-katello/blob/master/lib/hammer_cli_katello/id_resolver.rb#L16-L19

Actions #3

Updated by Eric Helms about 7 years ago

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

Updated by John Mitsch over 6 years ago

  • Translation missing: en.field_release changed from 284 to 352
Actions

Also available in: Atom PDF