Project

General

Profile

Bug #28097

"foreman-maintain packages update" fails to update all packages

Added by Martin Bacovsky 6 months ago. Updated about 2 months ago.

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

Description

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

Description of problem:
"foreman-maintain packages update" fails to update all packages with:

ERROR: parameter 'PACKAGES ...': no value provided

It should work similar way as "yum update" works. No parameter means all packages.

Yet worse, there is explicit all packages for yum as "\*"
But for foreman-maintain it needs to be escaped twice "\\*"

Not obvious nor documented.

Version-Release number of selected component (if applicable):
@Satellite 6.6.0 Snap22
rubygem-foreman_maintain-0.4.9-1.el7sat.noarch

How reproducible:
deterministic

Steps to Reproduce:
1. # foreman-maintain packages update
ERROR: parameter 'PACKAGES ...': no value provided

See: 'foreman-maintain packages update --help'

Actual results:
"foreman-maintain packages update" dourly expects a parameter

Expected results:
"foreman-maintain packages update" treats no parameter the same way as "yum update" does

Additional info:
Undocumented workaround is
  1. foreman-maintain packages update \\*

Associated revisions

Revision ca79a376 (diff)
Added by Martin Bacovsky about 2 months ago

Fixes #28097 - Allow packages update with no packages

This patch lets packages update command to be invoked
without any args to update all packages.

Revision bad91540 (diff)
Added by Martin Bacovsky about 2 months ago

Refs #28097 - Confirm update all

Revision 6ad88575 (diff)
Added by Martin Bacovsky about 2 months ago

Refs #28097 - updated formatting of the message

Revision 0b1b8ae6 (diff)
Added by Martin Bacovsky about 2 months ago

Refs #28097 - Disable --assumeyes

Revision b6b2581e (diff)
Added by Martin Bacovsky about 2 months ago

Refs #28097 - fixed typo

History

#1 Updated by The Foreman Bot 6 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_maintain/pull/286 added

#2 Updated by Martin Bacovsky about 2 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF