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

POD-2374: Add optional Version to Staging Workspace #198

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

sahakiann
Copy link
Collaborator

@sahakiann sahakiann commented Jan 21, 2025

POD-2374

Summary of Changes
The SetUpStagingWorkspaceAndDataset WDL and related Python script were updated to take in an optional "workspace version". This can be used in situations where more data needs to be imported into an existing dataset, but the dataset is not ready to be "published" yet. Versions are not "validated" in anyway (i.e. if version 2 was already we used, we don't check first to see if that's the case, but the script will fail if the continue_if_exists flag is not used). The auth domain will not change (the same existing auth domain is used) and same for the dataset. The dataset data, however, will be updated to indicate all new workspaces that are created. See screenshots and links below.

Staging Workspace V1

Staging Workspace V2

Copy link

Thank you for contributing to the ops-terra-utils repo!
A test suite will now begin running to ensure that no breaking changes are made.
If these tests fail you can have them re-run after making changes by commenting /run_tests

@sahakiann sahakiann changed the title Ns pod 2374 add version to staging workspace POD-2374: Add optional Version to Staging Workspace Jan 21, 2025
@sahakiann sahakiann marked this pull request as ready for review January 21, 2025 18:34
@sahakiann sahakiann merged commit 9cba2a5 into main Jan 21, 2025
9 checks passed
@sahakiann sahakiann deleted the NS_POD-2374_add_version_to_staging_workspace branch January 21, 2025 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants