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(artifacts): separate resolve artifacts for trigger of type pipeline #4570

Conversation

nemesisOsorio
Copy link
Member

@nemesisOsorio nemesisOsorio commented Oct 24, 2023

Follow up to

Triggering a pipeline from Pipeline A using the pipeline stage where Pipeline B uses BakeManifestStage or DeployManifestStage which rely on expectedArtifacts is broken. ArtifactUtils is filtering the expectedArtifacts needed in stages.

In 1.28, you were able to define expectedArtifacts in Pipeline B, and when you triggered from Pipeline A, Pipeline B was able to resolve the artifacts from Pipeline A. Passing artifacts between pipelines

We are partially reverting some changes from:

because they are not needed anymore.

Fixes spinnaker/spinnaker#6897

@nemesisOsorio
Copy link
Member Author

Closing in favor of #4579

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.

Automated triggers with artifact constraints are broken if you have 2 or more of the same type
2 participants