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

Error out in instance controller when a parameter has an undefined plan trigger #1336

Merged
merged 5 commits into from
Feb 11, 2020

Conversation

ANeumann82
Copy link
Member

Verify plans are referenced and referenced plans are defined

Signed-off-by: Andreas Neumann aneumann@mesosphere.com

Fixes #1268

error out in instance controller when a parameter has an undefined plan trigger

Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Copy link
Member

@porridge porridge left a comment

Choose a reason for hiding this comment

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

LGTM, other than the comments inline.

pkg/kudoctl/packages/verifier/plan/verify_references.go Outdated Show resolved Hide resolved
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
@ANeumann82 ANeumann82 merged commit 62fc580 into master Feb 11, 2020
@ANeumann82 ANeumann82 deleted the an/verify-trigger-plans branch February 11, 2020 14:55
zen-dog added a commit that referenced this pull request Feb 13, 2020
…trigger

Summary:
This is a followup to #1336 which verified that triggered plans are actually defined in the OV. This PR adds this logic to the instance webhook.

Fixes #1268

Signed-off-by: Aleksey Dukhovniy <alex.dukhovniy@googlemail.com>
ANeumann82 added a commit that referenced this pull request Feb 13, 2020
…an trigger (#1336)

* Verify plans are referenced and referenced plans are defined
* error out in instance controller when a parameter has an undefined plan trigger

Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
zen-dog pushed a commit that referenced this pull request Feb 17, 2020
…trigger (#1345)

Summary:
This is a followup to #1336 which verified that triggered plans are actually defined in the OV. This PR adds this logic to the instance webhook.

Fixes #1268

Signed-off-by: Aleksey Dukhovniy <alex.dukhovniy@googlemail.com>
runyontr pushed a commit that referenced this pull request Mar 11, 2020
…an trigger (#1336)

* Verify plans are referenced and referenced plans are defined
* error out in instance controller when a parameter has an undefined plan trigger

Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
Signed-off-by: Thomas Runyon <runyontr@gmail.com>
runyontr pushed a commit that referenced this pull request Mar 11, 2020
…trigger (#1345)

Summary:
This is a followup to #1336 which verified that triggered plans are actually defined in the OV. This PR adds this logic to the instance webhook.

Fixes #1268

Signed-off-by: Aleksey Dukhovniy <alex.dukhovniy@googlemail.com>
Signed-off-by: Thomas Runyon <runyontr@gmail.com>
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.

Explicitly specifying nonexistent trigger plan is silently treated as deploy
2 participants