Project

General

Profile

Bug #10628

Some parameter in Sat 6.1 CLI were renamed

Added by Adam Price over 6 years ago. Updated about 3 years ago.

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

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1223086
Description of problem:
In Satellite 6.1 some CLI parameters were renamed which can break customer scripts.

How reproducible:
Always

Steps to Reproduce:
I patched hammer-cli-console and produced diff of hammer options between Satellite 6.0 and 6.1.

Steps to get the list:
- Clean Sat6.x install
- gem install hammer-cli-console (with https://github.com/komidore64/hammer-cli-console/pull/4 merged)
- cat > /etc/hammer/cli.commands.d/console.yml <<EOF
:console:
:enable_module: true
EOF
- hammer -u admin -p <password> command-tree --with-options > hammer_sat6.0.txt

The resulting files are attached.

The main differences are caused partly by aliased options e.g. puppetclass -> puppet-class, ptable->partition-table, etc and partly by various changes in content-view options.

Actual results:
e.g. hammer os add-ptable --id 1 --ptable-id 1
Fails with unrecognised option

Expected results:
e.g. hammer os add-ptable --id 1 --ptable-id 1
Os updated

Additional info:
Patch for aliased options is ready. For Katello stuff the changes need review as some are intentional.

Associated revisions

Revision f87bc6ec (diff)
Added by Adam Price over 6 years ago

refs #10628 - adding back content-view[-id], BZ 1223086

Revision 2488fd79 (diff)
Added by Adam Price over 6 years ago

refs #10628 - adding back organization for puppet-module info, bz
1223086

Revision a65cc8ec (diff)
Added by Adam Price over 6 years ago

refs #10628 - add orgs and CVs back to puppet_module list, bz 1223086

Revision 298d1817 (diff)
Added by Adam Price over 6 years ago

refs #10628 - add org back to content-view version commands, bz 1223086

History

#1 Updated by The Foreman Bot over 6 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/297 added
  • Pull request deleted ()

#2 Updated by Eric Helms over 6 years ago

  • Legacy Backlogs Release (now unused) set to 31
  • Triaged changed from No to Yes

#3 Updated by Eric Helms about 6 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF