cdisc-org / cdisc-rules-engine

Open source offering of the cdisc rules engine
MIT License
49 stars 13 forks source link

With Joe Ben and IT, Sign up for the service where we can build the Azure container instance #765

Open drewcdisc opened 3 months ago

drewcdisc commented 2 months ago

@joebenclark This comment is only to let you know that we were talking about you. @SFJohnson24 will reach out to you with specifics when this PBI is being delivered. FYI: Sam is currently envisioning using a GitHub action to build a temporary Azure Container Instance every time we run the testing environment. Sam will have those details when that time comes.

joebenclark commented 2 months ago

@SFJohnson24 does this sound right: https://learn.microsoft.com/en-us/azure/container-instances/container-instances-github-action

SFJohnson24 commented 2 months ago

@joebenclark yes this is what I had in mind. We may need to persist the report after the container spins down so we may also need Azure Files mounted to the container. https://learn.microsoft.com/en-us/azure/container-instances/container-instances-volume-azure-files I have not looked into whether we can get the report after the run before the container spins down, in which case this may not be necessary.

SFJohnson24 commented 2 months ago

@joebenclark I met with @gerrycampion yesterday and discussed the test suite. We agreed on an architecture that would only need git actions so we may not need to provision any azure resources for this. I will keep this ticket up in case we reach a block with git actions implementation but using only git actions would have the benefit of not just simplicity but also cost savings. We will likely still want to provision these resources to test large dataset with engine hosted but not for the current test suite--more performance testing.