chef-server, chef-server-api, and chef-server-webui do not load /etc/chef/server.rb if run as a Rack app without their wrapper script.

Details

  • Type: Bug Bug
  • Status: Fix Provided Fix Provided
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 0.8.10
  • Fix Version/s: 0.8.14
  • Component/s: Chef Server, WebUI
  • Labels:
    None

Description

When running chef-server and chef-server-webui under Unicorn (or presumably other Rack containers) without use of the bundled scripts, the default configuration file in /etc/chef/server.rb is not loaded. The config/init.rb script in each application should call Chef::Config.from_file("/etc/chef/server.rb").

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Mark Imbriaco added a comment -

Fixed in: http://github.com/imbriaco/chef/tree/CHEF-1088

Show
Mark Imbriaco added a comment - Fixed in: http://github.com/imbriaco/chef/tree/CHEF-1088
Hide
Permalink
Matthew Kent added a comment -

Making sure it gets seen

Show
Matthew Kent added a comment - Making sure it gets seen
Hide
Permalink
Daniel DeLeo added a comment -

Mark, this is definitely cool, we use unicorn and love it, and we want you to be able to do the same. Looks like your patch will break support for specifying an alternate config file on the command line, though.

Seems like merb might devour the command line options, making it difficult to detect if we need to read the config file in init.rb or not. Maybe we can address the issue by improving the config.ru we ship with chef-server? As always, ping me in #chef-hacking if you want to discuss.

  • Dan
Show
Daniel DeLeo added a comment - Mark, this is definitely cool, we use unicorn and love it, and we want you to be able to do the same. Looks like your patch will break support for specifying an alternate config file on the command line, though. Seems like merb might devour the command line options, making it difficult to detect if we need to read the config file in init.rb or not. Maybe we can address the issue by improving the config.ru we ship with chef-server? As always, ping me in #chef-hacking if you want to discuss. Dan
Hide
Permalink
Daniel DeLeo added a comment -

Bulk re-assigning the fix version to 0.8.14

Show
Daniel DeLeo added a comment - Bulk re-assigning the fix version to 0.8.14
Hide
Permalink
Joseph Anthony Pasquale Holsten added a comment -

pretty sure this is won't fix

Show
Joseph Anthony Pasquale Holsten added a comment - pretty sure this is won't fix

People

  • Assignee:
    Mark Imbriaco
    Reporter:
    Mark Imbriaco
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: