Bug 1132642

Summary:	hammer-cli-import all - lost --macro-mapping at some point
Product:	Red Hat Satellite	Reporter:	Grant Gainey <ggainey>
Component:	Transitions	Assignee:	Grant Gainey <ggainey>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Lukas Pramuk <lpramuk>
Severity:	high	Docs Contact:
Priority:	high
Version:	6.0.4	CC:	bbuckingham, cperry, ggainey, jmontleo, mkollar, xdmoon
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2014-09-11 12:24:48 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1115190, 1125243

Description Grant Gainey 2014-08-21 17:20:13 UTC

Error: undefined local variable or method `option_macro_mapping' for #<HammerCLIImport::ImportCommand::AllCommand:0x7ff0b9375f80>

Put it back!

Comment 1 RHEL Program Management 2014-08-21 17:23:08 UTC

Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.

Comment 3 Grant Gainey 2014-08-21 17:25:20 UTC

To reproduce:

hammer import all --help

Expected behavior:

One of the options is "--macro_mapping FILE Mapping of Satellite-5 config-file-macros to puppet facts"

Current behavior:

No such option.

Comment 4 Grant Gainey 2014-08-21 17:34:31 UTC

master commit 012acb610f50ac8c78c70f1eff82613ec83da425

Comment 9 Matej Kollar 2014-09-05 11:59:08 UTC

verified with rubygem-hammer_cli_import-0.10.4-1

Comment 10 Bryan Kearney 2014-09-11 12:24:48 UTC

This was delivered with Satellite 6.0 which was released on 10 September 2014.