chore(install): merge plugin config for include files #3143
+19
−16
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Plugins defined in include files are overwriting plugin configuration as a whole. This changes the behaviour to instead merge, only overwriting the specific keys they define while retaining the keys defined in previous include files, as is being done for plugins under plugins key.
This is needed for Extensions plugin to be able to use pluginConfig from default plugins file when enabling preinstalled plugins.
Which issue(s) does this PR fix
Demo
marketplace-include-config.mov
How to test on rhdh-local
Create
dynamic-plugins.marketplace.yaml
underconfigs/app-config
with this contentadd to dynamic-plugins.override.yaml:
Bulk import should be successfully mounted although it doesnt have pluginConfig specified in
dynamic-plugins.marketplace.yaml
PR acceptance criteria
Please make sure that the following steps are complete:
How to test changes / Special notes to the reviewer