Bug #16629

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

Added by Sachin Ghai 7 months ago. Updated 4 months ago.

Status:Closed
Priority:Normal
Assigned To:Ori Rabin
Category:-
Target version:Foreman - Team Ivan Iteration 7
Difficulty: Pull request:https://github.com/theforeman/hammer-cli-foreman-discovery/pull/31
Bugzilla link:1377990
Story points-
Velocity based estimate-

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
Added by Ori Rabin 4 months ago

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

History

#1 Updated by Lukas Zapletal 5 months ago

  • Assigned To deleted (Lukas Zapletal)

#2 Updated by The Foreman Bot 5 months ago

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

#3 Updated by Ivan Necas 5 months ago

  • Target version set to Team Ivan Iteration 7

#4 Updated by Ori Rabin 4 months ago

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

Also available in: Atom PDF