-
Notifications
You must be signed in to change notification settings - Fork 147
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 #3552
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tamirkamara
approved these changes
Jun 8, 2023
…into adjust-ohdsi-to-oss
tamirkamara
approved these changes
Jun 11, 2023
yuvalyaron
added a commit
that referenced
this pull request
Jun 26, 2023
* add ohdsi workspace service (before adjustments to the OSS) * add execute permission to scripts * Fix Postgres timeouts when install OHDSI (#3559) [ohdsi] fix postgres timeouts * Add OHDSI workspace service (#3552) * 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 * update changelog * Update CHANGELOG.md Co-authored-by: Tamir Kamara <26870601+tamirkamara@users.noreply.github.com> * add ohdsi ws service to the CI * clarified README * added name, description and overview to the template_schema * move README content to docs * change default display name * add diagram to and instructions about setting up the CDM data source * add link to ohdsi-on-azure * move ohdsi-on-azure to the top * link to OHDSIonAzure for deploying synapse * Update docs/tre-templates/workspace-services/ohdsi.md Co-authored-by: Marcus Robinson <marrobi@microsoft.com> * add Using a sample CDM data source section --------- Co-authored-by: Tamir Kamara <26870601+tamirkamara@users.noreply.github.com> Co-authored-by: Marcus Robinson <marrobi@microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is being addressed
Added OHDSI workspace service.
Currently supports Synapse only.