Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merger options #53

Merged
merged 2 commits into from
Oct 21, 2021
Merged

Merger options #53

merged 2 commits into from
Oct 21, 2021

Conversation

tongueroo
Copy link
Collaborator

This is a πŸ™‹β€β™‚οΈ feature or enhancement.
This is a 🧐 documentation change.

  • I've added tests (if it's a bug, feature or enhancement)
  • I've adjusted the documentation (if it's a feature or enhancement)
  • The test suite passes (run bundle exec rspec to verify this)

Summary

Allow user to change core merging behavior.

.kubes/config.rb

Kubes.configure do |config|
  # ...
  config.merger.options = {overwrite_arrays: true} # default
end

Context

Some users prefer the Array be added to during an array. IE: base/deployment.yaml sidecar containers.

How to Test

Configure the option and run kubes apply and confirm that merged behavior is what you want.

Version Changes

Patch

@tongueroo tongueroo merged commit ddb3dc6 into master Oct 21, 2021
@tongueroo tongueroo deleted the merger-options branch October 21, 2021 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant