532228 – app-admin/puppet-3.7 init script spurious puppetmaster dependency

Bug 532228 - app-admin/puppet-3.7 init script spurious puppetmaster dependency

Summary: app-admin/puppet-3.7 init script spurious puppetmaster dependency

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Matthew Thode ( prometheanfire )

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2014-12-11 05:35 UTC by Eric Gisse
Modified:	2017-07-23 02:09 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Eric Gisse 2014-12-11 05:35:31 UTC

The puppet init script (/etc/init.d/puppet) has a dependency on puppetmaster that is higly problematic on non-master nodes:

depend() {
        need localmount
        use dns logger puppetmaster netmount nfsmount
}

For puppet nodes that ARE NOT master (ie, almost all) this is a big problem because starting puppet tries to start up the master. 

Can the depend() section please get changed to the following?

depend() {
        need localmount
        use dns logger netmount nfsmount
}

This way puppet can start without trying to invoke master.

Comment 1 Eric Gisse 2014-12-11 05:39:18 UTC

app-admin/puppet 3.7

Comment 2 Matthew Thode ( prometheanfire ) archtester

2014-12-11 05:49:51 UTC

this only depends/starts it if it's enabled, not simply installed.  might still be a good change though

Comment 3 Matthew Thode ( prometheanfire ) archtester

2017-07-23 02:09:27 UTC

gonna close this as it's only started if enabled