Project

General

Profile

Bug #28097

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

Added by Martin Bacovsky about 1 year ago. Updated 10 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 10 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 10 months ago

Refs #28097 - Confirm update all

Revision a53b27f2 (diff)
Added by Martin Bacovsky 10 months ago

Refs #28097 - rubocop

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

Refs #28097 - updated formatting of the message

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

Refs #28097 - Disable --assumeyes

Revision b6b2581e (diff)
Added by Martin Bacovsky 10 months ago

Refs #28097 - fixed typo

History

#1 Updated by The Foreman Bot about 1 year 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 10 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF