Skip to content

Commit c075d14

Browse files
jjoyce0510John Joyce
andauthored
fix(): Fix bundled venv (#14660)
Co-authored-by: John Joyce <[email protected]>
1 parent e2c5767 commit c075d14

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

docker/snippets/ingestion/build_bundled_venvs_unified.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"""
33
Self-contained script to create bundled venvs for DataHub ingestion sources.
44
This script creates virtual environments with predictable names following the pattern:
5-
<plugin-name>-default that are leveraged within acryl-executor to run ingestion jobs.
5+
<plugin-name>-bundled that are leveraged within acryl-executor to run ingestion jobs.
66
"""
77

88
import os
@@ -12,12 +12,12 @@
1212

1313

1414
def generate_venv_mappings(plugins: List[str]) -> List[Tuple[str, str]]:
15-
"""Generate simple venv name mappings using <plugin-name>-default pattern."""
15+
"""Generate simple venv name mappings using <plugin-name>-bundled pattern."""
1616
venv_mappings = []
1717

1818
for plugin in plugins:
19-
# Simple, predictable naming: <plugin-name>-default
20-
venv_name = f"{plugin}-default"
19+
# Simple, predictable naming: <plugin-name>-bundled
20+
venv_name = f"{plugin}-bundled"
2121
venv_mappings.append((plugin, venv_name))
2222

2323
return venv_mappings

docker/snippets/ingestion/build_bundled_venvs_unified.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/bin/bash
22
# Self-contained script to build bundled venvs for DataHub ingestion sources
3-
# This script creates venvs with predictable names: <plugin-name>-default
3+
# This script creates venvs with predictable names: <plugin-name>-bundled
44
# which are then leveraged within acryl-executor to run ingestion jobs.
55
set -euo pipefail
66

0 commit comments

Comments
 (0)