puppetlabs / puppetlabs/puppetlabs-docker

Add option to add ExecReload in the systemd service file

Open
#969 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Ruby
Stars
94
Forks
337
Avg merge
11d 15h
Merged PRs (30d)
2

Description

Use Case

The ExecReload statement in the service unit file is useful and is supposed by some dockerized services such as puppetserver.

Describe the Solution You Would Like

A way to support that option

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the code that generates the systemd service unit file and the existing service options. Trace how options are defined and rendered, then identify the relevant tests; the work is done when users can configure an ExecReload statement and the generated unit file is covered by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, ruby
Domain
devops, infrastructure
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.