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

Oncoanalyser / Sash stack and glue services #610

Merged
merged 11 commits into from
Oct 27, 2024

Conversation

alexiswl
Copy link
Member

@alexiswl alexiswl commented Oct 22, 2024

~~ Tests currently failing due to cdk-nag on AWS Step Functions~~

TODO

  • Apply Add nextflow access to data-bucket infrastructure#485 to development account
  • Test oncoanalyser SFN both successfully submitting job and retrieving batch events (and then spitting them out again as workflow run statechange events)
  • Test sash SFN both successfully submitting job and retrieving batch events (and then spitting them out again as workflow run statechange events)

Related PRs

@alexiswl alexiswl added feature New feature epic labels Oct 22, 2024
@alexiswl alexiswl self-assigned this Oct 22, 2024
@victorskl
Copy link
Member

Related umccr/nextflow-stack#115

@reisingerf
Copy link
Member

Yup, happy to suppress cdk-nag.

@alexiswl alexiswl marked this pull request as ready for review October 23, 2024 02:31
Alexis Lucattini added 2 commits October 23, 2024 14:28
* Updated stacky names from mock to stacky
* ICAV2_ACCESS_TOKEN_SECRET_ID an optional env var for fill placeholders lambda
* Add tagResource policy to nf stack
* Fix launch nextflow pipeline sfn
* Fix rule for default Batch event
* Fix batch parameter path link lambda
* Fix override resource requirements
* Add portal run id to engine parameters and to batch parameters
* Add orcabus: true to parameters to allow v1 portal to filter out events
* Fix url extension function
* Fix rna pipeline list flatten
@alexiswl alexiswl requested a review from scwatts October 25, 2024 01:19
Copy link
Member

@scwatts scwatts left a comment

Choose a reason for hiding this comment

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

Thanks for the reviewer request, assuming just for visibility - I don't have much context wrt OrcaBus for these changes!

@victorskl
Copy link
Member

🥁

@alexiswl alexiswl merged commit 074ac45 into main Oct 27, 2024
6 checks passed
@alexiswl alexiswl deleted the feature/oncoanalyser-nextflow branch October 27, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants