Skip to content

Add support for config namespaces and fix lastserialparent issue#96

Merged
schrodit merged 1 commit intogardener:masterfrom
schrodit:config-namespaces
Jun 17, 2019
Merged

Add support for config namespaces and fix lastserialparent issue#96
schrodit merged 1 commit intogardener:masterfrom
schrodit:config-namespaces

Conversation

@schrodit
Copy link
Copy Markdown

What this PR does / why we need it:
Adds support for inheriting config.
This means that now step config is shared with subsequent steps.
Whereas new levels are introduced to track which config overwrites the other one.

Config will be shared by default unless the new introduced parameter spec.testflow.0.definition.private is set to false
Which issue(s) this PR fixes:
Fixes #94

Special notes for your reviewer:

Release note:

Added support for shared configuration with subsequent steps

@schrodit schrodit requested a review from a team as a code owner June 14, 2019 13:53
@gardener-robot-ci-1 gardener-robot-ci-1 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@schrodit schrodit force-pushed the config-namespaces branch 2 times, most recently from f4ffca4 to 06a6802 Compare June 14, 2019 14:32
@schrodit schrodit added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@schrodit schrodit force-pushed the config-namespaces branch from 06a6802 to 6953e39 Compare June 14, 2019 14:36
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@schrodit schrodit added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@schrodit schrodit force-pushed the config-namespaces branch from 6953e39 to 5141d64 Compare June 14, 2019 18:45
@schrodit schrodit added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 14, 2019
@schrodit schrodit force-pushed the config-namespaces branch 2 times, most recently from 51972bf to 46a9e22 Compare June 17, 2019 06:37
@schrodit schrodit force-pushed the config-namespaces branch from 46a9e22 to f7cedb2 Compare June 17, 2019 06:52
@schrodit schrodit merged commit 49025ff into gardener:master Jun 17, 2019
@schrodit schrodit deleted the config-namespaces branch June 17, 2019 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configuration Namespace

2 participants