Skip to content

Add early access CDR config and GAR for AoU data collections#396

Merged
yonghaoy merged 3 commits into
masterfrom
add-preprod-cdr-config
May 7, 2026
Merged

Add early access CDR config and GAR for AoU data collections#396
yonghaoy merged 3 commits into
masterfrom
add-preprod-cdr-config

Conversation

@yonghaoy
Copy link
Copy Markdown
Contributor

@yonghaoy yonghaoy commented May 7, 2026

Summary

  • Adds cdr_config_early_access.json with registered tier (wb-crisp-citrus-2555, R2025Q4R4) and controlled tier (wb-bright-olive-4015, C2025Q4R3) entries
  • Maps two new data collection UUIDs to the early_access cdrEnv in datacollection_mapping.json
  • Wires up the new config in configs.go (embed) and main.go (switch case)

Test plan

  • Verify go build ./... passes in src/aou-common/load-envs
  • Verify a workspace linked to the early access data collections resolves the correct CDR config
  • Confirm env vars are correctly exported for both registered and controlled tiers

🤖 Generated with Claude Code

PHP-148755

Adds cdr_config_early_access.json with registered (wb-crisp-citrus-2555)
and controlled (wb-bright-olive-4015) tier entries, and maps the two new
data collection UUIDs to the early_access cdrEnv.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@yonghaoy yonghaoy requested review from a team as code owners May 7, 2026 13:17
Matches the prod cdrv8 controlled tier paths (WGS, microarray, clinvar,
CMRG, etc.) so early access workspaces expose the same genomic env vars.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@yonghaoy
Copy link
Copy Markdown
Contributor Author

yonghaoy commented May 7, 2026

@tarekmamdouh I would need your help to take pass those env vars (those are copied from prod ct so those might be wrong)?

@pantherman594 I added our use app cache gar as env var, it should be okay?

@yonghaoy yonghaoy changed the title Add early access CDR config for AoU data collections Add early access CDR config and GAR for AoU data collections May 7, 2026
@pantherman594
Copy link
Copy Markdown
Contributor

pantherman594 commented May 7, 2026

I added our use app cache gar as env var, it should be okay?

That should be fine, but I don't think I see it in this PR?

Copy link
Copy Markdown
Contributor

@pantherman594 pantherman594 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go changes LGTM

@tarekmamdouh
Copy link
Copy Markdown
Contributor

@tarekmamdouh I would need your help to take pass those env vars (those are copied from prod ct so those might be wrong)?

@pantherman594 I added our use app cache gar as env var, it should be okay?

I update the config and pushed the change

@yonghaoy yonghaoy merged commit 0d5de19 into master May 7, 2026
4 of 6 checks passed
@yonghaoy yonghaoy deleted the add-preprod-cdr-config branch May 7, 2026 19:14
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