Skip to content

Fix component overrides #1289

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

Conversation

Tomasz-Smelcerz-SAP
Copy link
Member

Description

  • All logic moved out of config-reader to be able to unit-test without K8s dependencies
  • Logic for joining component overrides from multiple sources fixed
  • Tests for joining override values from multiple sources added (for both common and component overrides)

Related issue(s)
Fixes #1276

@Tomasz-Smelcerz-SAP Tomasz-Smelcerz-SAP added kind/bug Categorizes issue or PR as related to a bug. area/installation Issues or PRs related to installation labels Oct 16, 2018
@Tomasz-Smelcerz-SAP Tomasz-Smelcerz-SAP added this to the Sprint_Toad_3 milestone Oct 16, 2018
@@ -111,8 +112,8 @@ func concatLabels(labels ...string) string {
return strings.Join(labels, ", ")
}

func toStringStringMap(input map[string][]byte) map[string]string {
var output = make(map[string]string)
func toStringStringMap(input map[string][]byte) inputMap {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename

@Tomasz-Smelcerz-SAP Tomasz-Smelcerz-SAP force-pushed the fix-component-overrides-join branch from 5e1469d to 6783e65 Compare October 22, 2018 07:37
@Tomasz-Smelcerz-SAP Tomasz-Smelcerz-SAP merged commit d55a219 into kyma-project:master Oct 22, 2018
grischperl pushed a commit to grischperl/kyma that referenced this pull request Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/installation Issues or PRs related to installation kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid overrides creation when multiple objects exists for the same component.
3 participants