Skip to content

Add remaining resource types to remote workspace collection#317

Open
dbx-shauver wants to merge 1 commit intorelease/0.8.0from
feature/pr4-remaining-remote-resources
Open

Add remaining resource types to remote workspace collection#317
dbx-shauver wants to merge 1 commit intorelease/0.8.0from
feature/pr4-remaining-remote-resources

Conversation

@dbx-shauver
Copy link
Copy Markdown

Summary

  • Adds Instance Pools, MLflow Experiments, Alerts, and Vector Search Endpoints to the remote workspace (non-current) collection loop
  • Instance Pools and Experiments include permission edges via permissions.get()
  • Alerts and Vector Search Endpoints produce vertices only (no ACL API available — same behavior as local collection)
  • All four types use ws_{ws_id}_ prefixed vertex IDs, consistent with remote collection convention from PRs 2/3
  • All blocks gated by existing COLLECTION_CONFIG flags

Context

This is PR 4 of 5 in the cross-workspace collection gap closure plan. Builds on the structural refactor from PR #313.

Test plan

  • Run collection against a multi-workspace account
  • Verify brickhound_vertices contains InstancePool, Experiment, Alert, VectorSearchEndpoint rows with ws_{ws_id}_ prefixed IDs for remote workspaces
  • Verify brickhound_edges contains permission edges for Instance Pools and Experiments
  • Verify setting collect_instance_pools=False skips Instance Pool collection
  • Confirm post-collection validation passes (no duplicate vertex IDs)

This pull request was AI-assisted by Isaac.

…mote collection

Adds four resource types to the remote workspace (non-current) collection loop
in BrickHound's data collection notebook:
- Instance Pools: vertices + permissions.get("instance-pools")
- MLflow Experiments: vertices + permissions.get("experiments")
- Alerts: vertices only (no ACL API available)
- Vector Search Endpoints: vertices only (no ACL API available)

All use ws_{ws_id}_ prefixed vertex IDs. Each block gated by existing
COLLECTION_CONFIG flags and wrapped in try/except.

Co-authored-by: Isaac
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.

1 participant