microsoft / AzureTRE-Deployment

MIT License
11 stars 22 forks source link

variable name missmatch #61

Open chboudry opened 1 year ago

chboudry commented 1 year ago

Github action is expecting :

LizaShak commented 1 year ago

Hi @chboudry , thank you for raising the issue. We will look into that.

SvenAelterman commented 1 year ago

Same experience here.

@chboudry Did you attempt a workaround?

LizaShak commented 1 year ago

@SvenAelterman If you are using an automation account, the nTEST_APP_ID is redundant. Same for TEST_WORKSPACE_APP_ID and TEST_WORKSPACE_APP_SECRET. We have an open issue to make it a non required params. For now you can put any dummy value inside (non empty value). Thank you and sorry for inconvenience.

marrobi commented 1 year ago

@LizaShak was an issue created for this? It has come up a couple of times recently. Thanks.