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

Allow multiple depedency graphs for download-and-submit #468

Open
ChipWolf opened this issue Dec 5, 2024 · 1 comment
Open

Allow multiple depedency graphs for download-and-submit #468

ChipWolf opened this issue Dec 5, 2024 · 1 comment
Labels
investigation Something to investigate

Comments

@ChipWolf
Copy link

ChipWolf commented Dec 5, 2024

Currently download-and-submit will take the latest dependency graph from the artifacts.

In the case of monorepo matrices, where many dependency graph artifacts are generated, there is a need to download all of them and submit them together only if the entire matrix succeeds & a release is generated.

The code is already here to upload multiple dependency graphs, so it should just be a case of parameterising latest:

let dependencyGraphArtifacts = (
await artifactClient.listArtifacts({
latest: true,
findBy
})
).artifacts.filter(artifact => artifact.name.startsWith(DEPENDENCY_GRAPH_PREFIX))

@bigdaz
Copy link
Member

bigdaz commented Dec 12, 2024

Thanks for the report @ChipWolf . Can you elaborate on the use case where you have multiple dependency graph files being generated with the same name in a single workflow execution?

Please describe the workflow structure and provide concrete examples of the file names involved?
Thanks!

@bigdaz bigdaz added investigation Something to investigate and removed investigation Something to investigate labels Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigation Something to investigate
Projects
None yet
Development

No branches or pull requests

2 participants