Project

General

Profile

Bug #29097

Installer prints out puppet-agent-oauth on most runs

Added by Eric Helms 6 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Related issues

Related to Installer - Bug #30214: No such file or directory - foreman-maintain on DebianResolved

Associated revisions

Revision 44591b97 (diff)
Added by Eric Helms 6 months ago

Fixes #29097: Move puppet-agent-oauth reinstall to Foreman hooks

Revision f35ae08d (diff)
Added by Ewoud Kohl van Wijngaarden about 2 months ago

Refs #29097 - Handle command not found

44591b97fe11bcb8c55af88309166aab0fa4ac2c started to execute this command
using execute_command() which uses open3 under the hood. When a command
doesn't exist, Errno::ENOENT is raised; unlike system() which returns
false. This catches it and handles the output.

This surfaced on Debian with the Puppet hook:

/usr/lib/ruby/2.5.0/open3.rb:199:in `spawn': No such file or directory - rpm (Errno::ENOENT)
from /usr/lib/ruby/2.5.0/open3.rb:199:in `popen_run'
from /usr/lib/ruby/2.5.0/open3.rb:190:in `popen2e'
from /usr/lib/ruby/2.5.0/open3.rb:349:in `capture2e'
from /usr/share/foreman-installer/hooks/boot/01-kafo-hook-extensions.rb:110:in `execute_command'
from /usr/share/foreman-installer/hooks/pre/31-puppet_agent_oauth.rb:2:in `block (4 levels) in load'

History

#1 Updated by The Foreman Bot 6 months ago

  • Assignee set to Eric Helms
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-installer/pull/472 added

#2 Updated by The Foreman Bot 6 months ago

  • Fixed in Releases 2.1.0 added

#3 Updated by Eric Helms 6 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman-installer/pull/532 added

#5 Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Related to Bug #30214: No such file or directory - foreman-maintain on Debian added

Also available in: Atom PDF