-
Notifications
You must be signed in to change notification settings - Fork 148
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
Add OHDSI workspace service #3562
Conversation
[ohdsi] fix postgres timeouts
* remove todos * link core vnet to postgres private dns zone when deploying core * remove synapse references, and add other data sources to the options * remove postgres_core_dns_link references * revert synapse reference deletions * remove non supported dialects * add execute permission to scripts * make some of the daimons required * add required zone field to postgres: hashicorp/terraform-provider-azurerm#16888 * add fw rule to allow open id authentication in atlas * fix firewall step * add README * update README * fix linting errors * fix linting errors
…into feature/ohdsi
Co-authored-by: Tamir Kamara <26870601+tamirkamara@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few docs related comments.
Is there a way I can test this out? What can I do about data? Can we add details to the docs?
…into feature/ohdsi
Co-authored-by: Marcus Robinson <marrobi@microsoft.com>
/test |
🤖 pr-bot 🤖 🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/5380031116 (with refid (in response to this comment from @yuvalyaron) |
Resolves #1505
What is being addressed
Added OHDSI workspace service
How is this addressed
README.md
file.