Skip to content

(MODULES-8886) Revert removal of deepmerge function#1181

Merged
david22swan merged 2 commits intopuppetlabs:masterfrom
eimlav:modules-8886
Apr 12, 2019
Merged

(MODULES-8886) Revert removal of deepmerge function#1181
david22swan merged 2 commits intopuppetlabs:masterfrom
eimlav:modules-8886

Conversation

@eimlav
Copy link
Copy Markdown
Contributor

@eimlav eimlav commented Apr 9, 2019

Reverting this change as the deepmerge function in stdlib does not work the same as the mysql deepmerge. As a result, the mysql config is generated with incorrect duplicate entries with '-' and '_' mixed up.

Original PR #1145 @david22swan

@eimlav eimlav added the bugfix label Apr 9, 2019
@eimlav eimlav changed the title (MODULES-8886) Revert removal of deepmerge function [WIP] (MODULES-8886) Revert removal of deepmerge function Apr 9, 2019
Reverting this change as the deepmerge function in stdlib does not work
the same as the mysql function. The mysql config is generated with
incorrect duplicate entries in which '-' and '_' are mixed up when
using the stdlib function.
@eimlav eimlav force-pushed the modules-8886 branch 3 times, most recently from caf4b13 to a54ba60 Compare April 11, 2019 12:20
@eimlav eimlav changed the title [WIP] (MODULES-8886) Revert removal of deepmerge function (MODULES-8886) Revert removal of deepmerge function Apr 12, 2019
@eimlav
Copy link
Copy Markdown
Contributor Author

eimlav commented Apr 12, 2019

Adhoc ran successfully:

Results Screen Shot 2019-04-12 at 13 24 12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants