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

fix(core/pipeline): Fix revert button regression for templated pipelines #7427

Merged
merged 2 commits into from
Sep 24, 2019

Conversation

jervi
Copy link
Contributor

@jervi jervi commented Sep 24, 2019

When pressing the revert button for a templated pipeline, the original pipeline got merged with the rendered pipeline, resulting in a strange frankenstein pipeline that you couldn't save.

When pressing the revert button for a templated pipeline, the original pipeline got merged with the rendered pipeline, resulting in a strange frankenstein pipeline that you couldn't save.
@jervi
Copy link
Contributor Author

jervi commented Sep 24, 2019

Regression in #7318
FYI @Jammy-Louie

Copy link

@Jammy-Louie Jammy-Louie left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks

@Jammy-Louie Jammy-Louie merged commit b942b5b into spinnaker:master Sep 24, 2019
@Jammy-Louie
Copy link

@spinnakerbot cherry-pick 1.16

@spinnakerbot
Copy link
Contributor

Cherry pick successful: #7429

christopherthielen added a commit to christopherthielen/deck that referenced this pull request Sep 24, 2019
91e4c42 fix(tasks): (re)-enable durations on canceled tasks (spinnaker#7433)
4d3454c fix(core/pipeline): migrate more manual execution field layouts
15de4a8 fix(core/forms): MapEditorInput: Add validation for empty keys and empty values. Remove `errors` prop in favor of using `validation.validationMessage` which is passed from FormField/FormikFormField
70dee3f fix(core/pipeline): Don't break templated pipelines when updating config (spinnaker#7428)
b942b5b fix(core/pipeline): Fix revert button regression for templated pipelines (spinnaker#7427)
d7490f9 feat(core): rerun child pipelines with artifacts (spinnaker#7422)
christopherthielen added a commit that referenced this pull request Sep 24, 2019
91e4c42 fix(tasks): (re)-enable durations on canceled tasks (#7433)
4d3454c fix(core/pipeline): migrate more manual execution field layouts
15de4a8 fix(core/forms): MapEditorInput: Add validation for empty keys and empty values. Remove `errors` prop in favor of using `validation.validationMessage` which is passed from FormField/FormikFormField
70dee3f fix(core/pipeline): Don't break templated pipelines when updating config (#7428)
b942b5b fix(core/pipeline): Fix revert button regression for templated pipelines (#7427)
d7490f9 feat(core): rerun child pipelines with artifacts (#7422)
jervi added a commit to jervi/deck that referenced this pull request Sep 26, 2019
I accidentally broke this in spinnaker#7427. Adding the removed line back, but only for non-templated pipelines.
louisjimenez pushed a commit that referenced this pull request Sep 26, 2019
)

I accidentally broke this in #7427. Adding the removed line back, but only for non-templated pipelines.
spinnakerbot pushed a commit that referenced this pull request Sep 26, 2019
)

I accidentally broke this in #7427. Adding the removed line back, but only for non-templated pipelines.
louisjimenez pushed a commit that referenced this pull request Sep 26, 2019
) (#7442)

I accidentally broke this in #7427. Adding the removed line back, but only for non-templated pipelines.
@jervi jervi deleted the revert_for_templated_pipelines branch September 27, 2019 12:29
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
91e4c42 fix(tasks): (re)-enable durations on canceled tasks (spinnaker#7433)
4d3454c fix(core/pipeline): migrate more manual execution field layouts
15de4a8 fix(core/forms): MapEditorInput: Add validation for empty keys and empty values. Remove `errors` prop in favor of using `validation.validationMessage` which is passed from FormField/FormikFormField
70dee3f fix(core/pipeline): Don't break templated pipelines when updating config (spinnaker#7428)
b942b5b fix(core/pipeline): Fix revert button regression for templated pipelines (spinnaker#7427)
d7490f9 feat(core): rerun child pipelines with artifacts (spinnaker#7422)
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
…innaker#7440)

I accidentally broke this in spinnaker#7427. Adding the removed line back, but only for non-templated pipelines.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants