Project

General

Profile

Bug #1566

Puppet CA set incorrectly when only one proxy is defined

Added by Greg Sutcliffe almost 11 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Normal
Category:
Host creation
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

This is related to the way a combo-box defaults to "blank" even when there is only one valid option. Because we hide the rendering of the combobox when there is only one puppet proxy, it can't be set to the right value.

Need to investigate why the proxy isn't being set to the only valid option when there's only one proxy.

Associated revisions

Revision abc12655 (diff)
Added by Ohad Levy almost 11 years ago

fixes #1566 - Puppet CA set incorrectly when only one proxy is defined

History

#1 Updated by Greg Sutcliffe almost 11 years ago

You can work around this with this patch:

diff --git a/app/helpers/hosts_and_hostgroups_helper.rb b/app/helpers/hosts_and_hostgroups_
index d03c609..885a2e7 100644
--- a/app/helpers/hosts_and_hostgroups_helper.rb
+++ b/app/helpers/hosts_and_hostgroups_helper.rb
@@ -63,7 +63,7 @@ module HostsAndHostgroupsHelper
     ca      = SmartProxy.joins(:features).where(:features => { :name => "Puppet CA" })
     proxies = SmartProxy.joins(:features).where(:features => { :name => "Puppet" })
     # do not show the ca proxy, if we have only one of those and its the same as the puppe
-    fields =  puppet_ca(f) unless ca.count == 1 and ca.map(&:id) == proxies.map(&:id)
+    fields =  puppet_ca(f) unless ca.count == 0 and ca.map(&:id) == proxies.map(&:id)
     "#{fields} #{puppet_master(f)}".html_safe
   end


Although you'll have to edit your hosts again :(

#2 Updated by Ohad Levy almost 11 years ago

  • Category set to Host creation
  • Target version set to 1.0

#3 Updated by Ohad Levy almost 11 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF