Skip to content

pathogen-repo-build: Support artifact-paths with AWS Batch#164

Merged
victorlin merged 4 commits intomasterfrom
victorlin/artifact-paths
Mar 23, 2026
Merged

pathogen-repo-build: Support artifact-paths with AWS Batch#164
victorlin merged 4 commits intomasterfrom
victorlin/artifact-paths

Conversation

@victorlin
Copy link
Member

@victorlin victorlin commented Mar 18, 2026

This is now possible with the wait-N jobs.

Should improve debugging experience for workflows using the AWS Batch runtime.

Checklist

@victorlin victorlin self-assigned this Mar 18, 2026
Without this, the workflow execution happens in the run-build job
instead of the wait-N jobs.
@victorlin victorlin force-pushed the victorlin/artifact-paths branch from 6941e1a to bfa71f3 Compare March 18, 2026 20:26
This is now possible with the wait-N jobs.
@victorlin victorlin force-pushed the victorlin/artifact-paths branch from bfa71f3 to e716626 Compare March 18, 2026 20:42
@victorlin victorlin changed the title Support artifact-paths with AWS Batch pathogen-repo-build: Support artifact-paths with AWS Batch Mar 18, 2026
This allows the full build log to be uploaded upon completion, matching
behavior in run-build with local runtimes.
@victorlin victorlin force-pushed the victorlin/artifact-paths branch 2 times, most recently from 8e106ec to 8281273 Compare March 20, 2026 17:25
joverlee521 added a commit to nextstrain/seasonal-flu that referenced this pull request Mar 20, 2026
… builds

Ensuring we don't include private builds in public GH Action artifacts
in anticipation of nextstrain/.github#164
joverlee521 added a commit to nextstrain/seasonal-flu that referenced this pull request Mar 20, 2026
… builds

Ensuring we don't include private builds in public GH Action artifacts
in anticipation of nextstrain/.github#164
Unlike local runtimes, where uploaded artifacts are the only copy of
generated files, AWS Batch instances temporarily persist files after the
run finishes. This means it's not important to upload the auspice/ and
results/ folders, which could contain private or large data.
@victorlin victorlin force-pushed the victorlin/artifact-paths branch from 8281273 to 337b4a9 Compare March 23, 2026 18:16
@victorlin victorlin merged commit f90c23b into master Mar 23, 2026
38 checks passed
@victorlin victorlin deleted the victorlin/artifact-paths branch March 23, 2026 22:11
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.

2 participants