jenkinsci / helm-charts

Jenkins helm charts
https://artifacthub.io/packages/helm/jenkinsci/jenkins
Apache License 2.0
561 stars 889 forks source link

feat: Configure inherit yaml merge strategy #1128

Closed james1miller93 closed 3 months ago

james1miller93 commented 3 months ago

What does this PR do?

This pr exposes the configuration of inheritYamlMergeStrategy on the default agent via the chart values. The default value for this is set to false to match the default behaviour of the JCasC plugin.

The changes have been tested against a live jenkins deployment and the configuration is applied and behaves as expected. Tested with both the default value and overridden to true.

If you modified files in the ./charts/jenkins/ directory, please also include the following:

### Submitter checklist
- [x] I bumped the "version" key in `./charts/jenkins/Chart.yaml`.
- [x] I added a new changelog entry to `./charts/jenkins/CHANGELOG.md`.
- [x] I followed the [technical requirements](https://github.com/jenkinsci/helm-charts/blob/main/CONTRIBUTING.md#technical-requirements).
- [x] I ran `.github/helm-docs.sh` from the project root.

Special notes for your reviewer