voxpupuli / voxpupuli/puppet-php

Merging broken for php::settings and php::fpm::settings

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

Nobody has claimed this yet.

Dominant language
Puppet
Stars
86
Forks
264
Avg merge
29m
Merged PRs (30d)
1

Description

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: PE 2019.8.5
  • Ruby: 2.5.1
  • Distribution: Debian 10
  • Module version: 7.1.0

How to reproduce (e.g Puppet code you use)

Put content in php::settings and in php::fpm::settings

What are you seeing

php::settings are not merged to /etc/php/8.0/fpm/php.ini

What behaviour did you expect instead

Content of php::settings and php::fpm::settings are merged into the fpm php.ini file

Output log

Any additional information you'd like to impart

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 reproducing the issue with content in php::settings and php::fpm::settings on the stated Puppet module setup, then trace how those settings are combined for /etc/php/8.0/fpm/php.ini. Done means both settings sources are merged into the FPM php.ini configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
devops, infrastructure
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.