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(pubsub): prevent NPE when retrying a pipeline execution with #8172

Merged
merged 1 commit into from
Apr 15, 2020

Conversation

danielpeach
Copy link
Contributor

a pubsub trigger

Retrying a pipeline with a pubsub trigger + any other trigger doesn't work because the pubsub trigger type doesn't implement a manualExecutionComponent. Wrote out a larger description of the problem here: spinnaker/spinnaker#5682

This PR "solves" the problem by preventing the retry modal from blowing up, but eventually would love to see the attached issue solved.

@maggieneterval maggieneterval self-requested a review April 15, 2020 13:47
@maggieneterval maggieneterval added the ready to merge Reviewed and ready for merge label Apr 15, 2020
@mergify mergify bot merged commit 5d95271 into spinnaker:master Apr 15, 2020
@danielpeach
Copy link
Contributor Author

@Mergifyio backport release-1.18.x

@danielpeach
Copy link
Contributor Author

@Mergifyio backport release-1.19.x

@mergify
Copy link
Contributor

mergify bot commented Apr 15, 2020

@danielpeach is not allowed to run commands

1 similar comment
@mergify
Copy link
Contributor

mergify bot commented Apr 15, 2020

@danielpeach is not allowed to run commands

@danielpeach
Copy link
Contributor Author

@spinnakerbot cherry-pick 1.18

@danielpeach
Copy link
Contributor Author

@spinnakerbot cherry-pick 1.19

@spinnakerbot
Copy link
Contributor

Support for automating cherry picks is being removed from spinnakerbot in favor of the mergify backport command. To help with the transition, I'll run the required command for you now, but in the future please run the mergify command directly.

@spinnakerbot
Copy link
Contributor

@Mergifyio backport release-1.18.x

@spinnakerbot
Copy link
Contributor

Support for automating cherry picks is being removed from spinnakerbot in favor of the mergify backport command. To help with the transition, I'll run the required command for you now, but in the future please run the mergify command directly.

@spinnakerbot
Copy link
Contributor

@Mergifyio backport release-1.19.x

mergify bot pushed a commit that referenced this pull request Apr 15, 2020
@mergify
Copy link
Contributor

mergify bot commented Apr 15, 2020

Command backport release-1.18.x: success

Backports have been created

@mergify
Copy link
Contributor

mergify bot commented Apr 15, 2020

Command backport release-1.19.x: success

Backports have been created

mergify bot added a commit that referenced this pull request Apr 15, 2020
…) (#8174)

a pubsub trigger

(cherry picked from commit 5d95271)

Co-authored-by: Daniel Peach <daniel.peach@armory.io>
mergify bot added a commit that referenced this pull request Apr 15, 2020
…) (#8173)

a pubsub trigger

(cherry picked from commit 5d95271)

Co-authored-by: Daniel Peach <daniel.peach@armory.io>
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Reviewed and ready for merge target-release/1.20
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants