Project

General

Profile

Bug #2714

foreman-proxy: I receives Error when initiates Puppet runs via the MCollective

Added by Peter Gustafsson over 7 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Puppet
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

Hi,

When i initiates Puppet runs via the MCollective i get ["failed to execute puppetrun: 500 Internal Server Error"] in the foreman webUI and i can see this in the proxy log:

E, [2013-06-26T10:02:15.005985 #19889] ERROR -- : Exception 'can't convert Array into String' when executing '/usr/bin/sudo/usr/bin/mcopuppetrunonce-Itest01.nix.example.com'
E, [2013-06-26T10:02:15.006767 #19889] ERROR -- : Failed puppet run: Check Log files

Br, Peter

Associated revisions

Revision 61394852 (diff)
Added by Dmitri Dolguikh over 7 years ago

Fixes #2714 - corrected IO#popen parameter on ruby 1.8.7

Revision b1db26c0 (diff)
Added by Dmitri Dolguikh over 7 years ago

Fixes #2714 - corrected IO#popen parameter on ruby 1.8.7
(cherry picked from commit 61394852e191756fc5eea0a92d581cf1dca39f21)

History

#1 Updated by Dominic Cleal over 7 years ago

  • Project changed from Foreman to Smart Proxy
  • Status changed from New to Assigned
  • Assignee set to Dmitri Dolguikh

#2 Updated by Peter Gustafsson over 7 years ago

i'm using foreman-proxy 1.2 RC3
  1. rpm -q foreman-proxy
    foreman-proxy-1.2.0-0.3.RC3.el6.noarch

#3 Updated by Dominic Cleal over 7 years ago

Dmitri, would you mind having a look again? Maybe something wrong with the IO.popen in shell_command? I suspect this might be a Ruby 1.8 issue as the proxy isn't SCL'd.

#4 Updated by Dominic Cleal over 7 years ago

  • Target version set to 1.2.0
1.9.3p392 :002 > IO.popen(["echo", "test"])
 => #<IO:fd 5> 

1.8.7 :001 > IO.popen(["echo", "test"])
TypeError: can't convert Array into String
    from (irb):1:in `popen'
    from (irb):1

sigh.

#5 Updated by Dominic Cleal over 7 years ago

  • Category set to Puppet

#6 Updated by Dmitri Dolguikh over 7 years ago

  • Status changed from Assigned to Ready For Testing

PR:https://github.com/theforeman/smart-proxy/pull/99

#7 Updated by Peter Gustafsson over 7 years ago

Above PR works for me.

Br, Peter

#8 Updated by Anonymous over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#9 Updated by Greg Sutcliffe over 2 years ago

  • Target version deleted (1.2.0)

Also available in: Atom PDF