Project

General

Profile

Bug #7060

puppet gem is required even when puppetca module is disabled

Added by Dmitri Dolguikh over 6 years ago. Updated over 2 years ago.

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

Related issues

Related to Smart Proxy - Bug #7080: Couldn't enable plugin puppet: undefined method `<=>' for :puppet_provider:SymbolClosed2014-08-14
Related to Smart Proxy - Bug #7240: When loading using bundler smart-proxy fails to load puppetca module if puppet module is disabledClosed2014-08-22

Associated revisions

Revision ed3332ae (diff)
Added by Dmitri Dolguikh over 6 years ago

fixes #7060: puppet gem is no longer required if puppetca is disabled.

History

#1 Updated by The Foreman Bot over 6 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/198 added
  • Pull request deleted ()

#2 Updated by Dominic Cleal over 6 years ago

  • Project changed from Foreman to Smart Proxy
  • Category changed from Smart Proxy to Puppet
  • Legacy Backlogs Release (now unused) set to 10

#3 Updated by Lukas Zapletal over 6 years ago

  • Bugzilla link set to 1129439

#4 Updated by Anonymous over 6 years ago

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

#5 Updated by Jason Montleon over 6 years ago

This seems to be breaking foreman-proxy on RHEL6 unless :puppet_provider: puppetrun is explicitly set in /etc/foreman-proxy/settings.d/puppet.yml

E, [2014-08-13T17:26:31.882793 #44503] ERROR -- : Couldn't enable plugin puppet: undefined method `<=>' for :puppetdir:Symbol:/usr/share/foreman-proxy/lib/proxy/plugin.rb:139:in `<=>'/n/usr/share/foreman-proxy/lib/proxy/plugin.rb:139:in `sort'/n/usr/share/foreman-proxy/lib/proxy/plugin.rb:139:in `log_used_default_settings'/n/usr/share/foreman-proxy/lib/proxy/plugin.rb:149:in `configure_plugin'/n/usr/share/foreman-proxy/lib/proxy/plugin.rb:22:in `configure_loaded_plugins'/n/usr/share/foreman-proxy/lib/proxy/plugin.rb:22:in `each'/n/usr/share/foreman-proxy/lib/proxy/plugin.rb:22:in `configure_loaded_plugins'/n/usr/share/foreman-proxy/lib/smart_proxy.rb:116:in `launch'/n/usr/share/foreman-proxy/bin/smart-proxy:6

<jsherrill> jmontleon: in ruby 1.8.7 it doens't look like you can sort symbols?
<jsherrill> irb(main):002:0> [:a, :b, :c].sort
<jsherrill> NoMethodError: undefined method `<=>' for :a:Symbol
<jsherrill> so i'm guessing its parsing those config files and using symbols

<jsherrill> jmontleo: but the proper fix would be to not rely on sortable symbols IMHO

jsherrill> or else explicitly include the Comparable monkey patch

#6 Updated by Dominic Cleal over 6 years ago

  • Related to Bug #7080: Couldn't enable plugin puppet: undefined method `<=>' for :puppet_provider:Symbol added

#7 Updated by Lukas Zapletal over 6 years ago

I tested this only on RHEL7 which has Ruby 2.0...

#8 Updated by Dominic Cleal over 6 years ago

  • Related to Bug #7240: When loading using bundler smart-proxy fails to load puppetca module if puppet module is disabled added

Also available in: Atom PDF