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

Add parallel signing of images to main pipeline #82

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

joinemm
Copy link
Collaborator

@joinemm joinemm commented Nov 1, 2024

New function created in utils as the existing functions make too many assumptions and do too many things (and as such would not work in a parallel environment)

@joinemm joinemm requested a review from a team November 1, 2024 17:09
utils.groovy Outdated Show resolved Hide resolved
@joinemm joinemm force-pushed the pr-parallel-signing branch from 00d19dc to 70edd59 Compare November 4, 2024 08:49
@joinemm joinemm requested review from henrirosten and a team November 4, 2024 08:59
@henrirosten
Copy link
Collaborator

LGTM, assuming this has been tested properly.

@joinemm joinemm merged commit f939740 into tiiuae:main Nov 4, 2024
1 check passed
@joinemm joinemm deleted the pr-parallel-signing branch November 4, 2024 09:24
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.

3 participants