Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

Lazy loading with pipelineRef#4820

Closed
edgarulg wants to merge 22 commits intospinnaker:masterfrom
edgarulg:lazy-loading-1-35
Closed

Lazy loading with pipelineRef#4820
edgarulg wants to merge 22 commits intospinnaker:masterfrom
edgarulg:lazy-loading-1-35

Conversation

@edgarulg
Copy link
Copy Markdown
Contributor

@edgarulg edgarulg commented Jan 6, 2025

pipeline triggers contains the whole execution of the parent execution. When a Spinnaker user defines multiple pipelines that are chained by pipeline triggers Spinnaker's UI is slowed down because it loads all nested executions.

Orca will return any execution to outsider services with PipelineRef trigger but for Orca itself it will use the whole execution context to evaluate SPeLs.

it will provide a new endpoint to retrieve the execution context including nested execution.

mergify Bot and others added 4 commits December 11, 2024 15:53
…naker#4814)

(cherry picked from commit 266ce73)

Co-authored-by: Christos Arvanitis <christos.arvanitis@armory.io>
…to PipelineRefTrigger (spinnaker#4816) (spinnaker#4818)

* fix(pipelineRef): add resolvedExpectedArtifacts from pipelineTrigger to PipelineRefTrigger

* fix(pipelineRef): add tests around PipelineRefTrigger

---------

Co-authored-by: Jason <jason.mcintosh@harness.io>
(cherry picked from commit 3c65fe0)

Co-authored-by: Edgar Garcia <63310723+edgarulg@users.noreply.github.com>
@spinnakerbot
Copy link
Copy Markdown
Contributor

Please delete the pull request instructions from the body of your pull request message.

The instructions start with the line:

We prefer small, well tested pull requests.

You can reopen your pull request when this has been addressed.

@edgarulg edgarulg reopened this Jan 6, 2025
@edgarulg edgarulg closed this Feb 18, 2025
@edgarulg
Copy link
Copy Markdown
Contributor Author

closed in favor of: #4842

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants