Project

General

Profile

Bug #16629

[hammer] discovery_rule info command should show the associated org and locations

Added by Sachin Ghai about 3 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1377990
Description of problem:
In sat6.3 , when we create a discovery rule, use needs to provide the org/location. But when the rule is created, info command doesn't show the associated info:

~]# hammer -u admin -p changeme discovery_rule create --name test --hostgroup=test --search='cpu_count=1' --organization-ids 1

~]# hammer -u admin -p changeme discovery_rule list

---|------|----------|-------------|------------|-------------|--------
ID | NAME | PRIORITY | SEARCH | HOST GROUP | HOSTS LIMIT | ENABLED
---|------|----------|-------------|------------|-------------|--------
1 | test | 0 | cpu_count=1 | test | 0 | true
---|------|----------|-------------|------------|-------------|--------

~]# hammer -u admin -p changeme discovery_rule info --id 1

ID: 1
Name: test
Priority: 0
Search: cpu_count=1
Host Group: test
Hosts Limit: 0
Enabled: true
Hostname template:
Hosts:

Version-Release number of selected component (if applicable):
satellite-6.3.0-1.0.git.7.fb12bf2.el7sat.noarch
tfm-rubygem-hammer_cli_foreman_discovery-0.0.3-2.el7.noarch

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual results:
missing org and location from info command

Expected results:

discovery_rule info command should show the associated org and locations

Additional info:

Associated revisions

Revision 7fae5f08 (diff)
Added by Ori Rabin almost 3 years ago

Fixes #16629 - discovery_rule info command should show the associated org and locations

History

#1 Updated by Lukas Zapletal almost 3 years ago

  • Assignee deleted (Lukas Zapletal)

#2 Updated by The Foreman Bot almost 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ori Rabin
  • Pull request https://github.com/theforeman/hammer-cli-foreman-discovery/pull/31 added

#3 Updated by Ivan Necas almost 3 years ago

  • Target version set to 1.15.3

#4 Updated by Ori Rabin almost 3 years ago

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

Also available in: Atom PDF