Upstart provider erroneously requires start_command, stop_command, and restart_command to be manually set

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 10.14.0
  • Fix Version/s: 10.14.0
  • Component/s: None
  • Labels:
    None

Description

The upstart service provider calls super in its define_resource_requirements method. Thus, the define_resource_requirements method in the simple provide is called and erroneously requires `start_command`, `stop_command`, and other attributes to be set. For example the following recipe code succeeds in 10.12.0 but fails in 10.14.0:

service "cron" do                                                                                                      
  provider Chef::Provider::Service::Upstart                                                                              
  action :start
end

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. CHEF-3320 Chef::Provider::Service::Upstart does not set start_command

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Steven Danna added a comment -

This fixes the case above and the actual cookbook failure (logstash) in which I originally found the bug. Also somehow origin was pointed at the opscode repo when I pushed:

https://github.com/opscode/chef/tree/CHEF-3410

Show
Steven Danna added a comment - This fixes the case above and the actual cookbook failure (logstash) in which I originally found the bug. Also somehow origin was pointed at the opscode repo when I pushed: https://github.com/opscode/chef/tree/CHEF-3410
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

merged to 10-stable.

Show
Bryan McLellan [Opscode] added a comment - merged to 10-stable.

People

  • Assignee:
    Unassigned
    Reporter:
    Steven Danna
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: