Feature #7660

Missing init script for puppet queue daemon.

Added by Jason Smith 12 months ago. Updated 14 days ago.

Status:Merged - Pending Release Start date:05/25/2011
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:-
Target version:3.0.0
Affected Puppet version:2.6.8 Branch:
Keywords:
Votes: 0

Description

The puppet package is missing an init script for the queue daemon. It may not be as common as the client or server, but it would still be nice to include. I just copied the client init script and modified it to manage the queue daemon.

History

#1
Updated by James Turnbull 12 months ago

  • Status changed from Unreviewed to Needs Decision
  • Assignee set to Nigel Kersten

#2
Updated by Nigel Kersten 8 months ago

  • Status changed from Needs Decision to Accepted
  • Target version set to 3.x

Jason, any chance you could provide your modified script in a pull request?

#3
Updated by Jason Smith 8 months ago

I had only used github read-only till now so I created an account, forked puppet and made a pull request. Let me know if I did anything wrong. Since we only use RedHat here, I am not able to do the same for any of the other OSes in the conf directory.

#4
Updated by Nigel Kersten 8 months ago

That sounds great Jason. If you could add a link to the branch and pull request to this ticket, we can make sure it doesn’t slip through the cracks.

#5
Updated by Jason Smith 8 months ago

Pull request made: https://github.com/puppetlabs/puppet/pull/159

For branch: https://github.com/smithj4/puppet/tree/feature-7660

#6
Updated by Deepak Giridharagopal 6 months ago

  • Status changed from Accepted to In Topic Branch Pending Review

I’ve rebased this onto 2.7.x branch and made a new pull req:

https://github.com/puppetlabs/puppet/pull/222

#7
Updated by Deepak Giridharagopal 6 months ago

Jason, have you signed a CLA? We’ll need to do that prior to merging in your code.

#8
Updated by Jason Smith 6 months ago

Deepak Giridharagopal wrote:

Jason, have you signed a CLA? We’ll need to do that prior to merging in your code.

I hadn’t before, but I just did it online. Do I still need to submit a paper or fax copy?

#9
Updated by Daniel Pittman 4 months ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release
  • Assignee deleted (Nigel Kersten)

Jason Smith wrote:

Deepak Giridharagopal wrote:

Jason, have you signed a CLA? We’ll need to do that prior to merging in your code.

I hadn’t before, but I just did it online. Do I still need to submit a paper or fax copy?

Hey. Sorry this slipped below the radar. The online version is sufficient, and the content looks good, so I am going to go ahead and merge this into master; that will make the change part of the Telly release, when that rolls around.

#10
Updated by Daniel Pittman 14 days ago

  • Target version changed from 3.x to 3.0.0

Also available in: Atom PDF