Skip to content

refactor!: rename hydrovu source to pvacd_hydrovu for multi-tenant layout (ST2DAT-241) - #47

Open
crawson-icasa wants to merge 3 commits into
mainfrom
ST2DAT-241-refactor-bernco_hydrovu-and-pvacd_hydrovu-multi-tenant-HydroVu-source-layout
Open

refactor!: rename hydrovu source to pvacd_hydrovu for multi-tenant layout (ST2DAT-241)#47
crawson-icasa wants to merge 3 commits into
mainfrom
ST2DAT-241-refactor-bernco_hydrovu-and-pvacd_hydrovu-multi-tenant-HydroVu-source-layout

Conversation

@crawson-icasa

Copy link
Copy Markdown
Contributor

Why

  • BernCo is the first case of two agencies on one source system. The single HydroVu tenant was called hydrovu, so the platform name and the tenant name were the same string. There was no room for a second tenant beside it. docs/sources/bernco_hydrovu.md catalogued the blockers in its Open Question 1.

How

  • Tenant-scoped names become pvacd_hydrovu. Platform-scoped names stay hydrovu.
  • Moved: the config block, the registry entry, the source folder, the dataset, and the job and schedule names. Assets are now raw_pvacd_hydrovu_readings, canonical_bundles_pvacd_hydrovu, and frost_load_pvacd_hydrovu. The deploy variable SECRET_HYDROVU becomes SECRET_PVACD_HYDROVU, so BernCo's ticket adds a second export rather than reworking this one.

Notes

  • BREAKING CHANGE: raw_pvacd is superseded by raw_pvacd_hydrovu, and the transform watermark moves with it. PVACD's dlt cursors reset on the first run after deploy.
  • No objects were copied. raw_pvacd/ is left in place, orphaned.
  • BernCo still needs a SECRET_BERNCO_HYDROVU export and a secretmanager.secretAccessor grant on hydrovu_bernco. This was a rename only, so its ticket decides whether to extract the shared HydroVu client or copy it.

@crawson-icasa
crawson-icasa requested review from chasetmartin and likithabommasani21 and removed request for likithabommasani21 August 29, 2026 02:01
@github-actions

github-actions Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Your pull request is automatically being deployed to Dagster Cloud.

Location Status Link Updated
aqueduct_dagster_defs_definitions View in Cloud Aug 29, 2026 at 02:04 AM (UTC)

@likithabommasani21 likithabommasani21 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me
Note : The FROST clean-out needs to happen before the renamed pipeline's first run against production, not after — otherwise there's a window where the old data and the fresh re-post coexist and you get duplicates anyway. Just want to make sure that's sequenced correctly.

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.

2 participants