Skip to content

foundation for job submission to head node#90

Merged
thecaffiend merged 16 commits into
mainfrom
75-foundation-for-job-submission-to-head-node
Aug 22, 2024
Merged

foundation for job submission to head node#90
thecaffiend merged 16 commits into
mainfrom
75-foundation-for-job-submission-to-head-node

Conversation

@thecaffiend
Copy link
Copy Markdown
Member

@thecaffiend thecaffiend commented Aug 22, 2024

TO TEST

  • visual inspection of code, pulumi preview should have no changes (assuming tested against what was deployed at time of PR)
  • This has been deployed and demonstrated to @mehalter , but feel free to try to break it more if you want. the data format for the API endpoint is:

the pipeline name here is the only one in the DDB at this time

{
"pipelineName": "bactopia 3.0.1 tutorial analysispipeline",
"pipelineVersion": "3.0.1",
"inputPath": "s3://input/blah/blah/blah",
"outputPath": "s3://output/blah/blah/blah"
}

@thecaffiend thecaffiend linked an issue Aug 22, 2024 that may be closed by this pull request
Copy link
Copy Markdown
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, only things were formatting and import sorting which I fixed in a commit to help with expedience. Seems great and all of the TODOs for later make sense.

:shipit:

@thecaffiend thecaffiend merged commit 21fe20d into main Aug 22, 2024
@thecaffiend thecaffiend deleted the 75-foundation-for-job-submission-to-head-node branch August 22, 2024 20:14
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.

foundation for job submission to head node

2 participants