Skip to content

Commit

Permalink
make sure the env var is injected
Browse files Browse the repository at this point in the history
  • Loading branch information
maximlt committed Oct 2, 2024
1 parent 42ea3d1 commit 9d4b94d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ jobs:
conda activate test
conda install -c ae5-admin anaconda-project ae5-tools pyyaml
- name: Inject env vars
env:
HOLOVIZ_DISCORD_TAGONLY_WEBHOOK: ${{ secrets.HOLOVIZ_DISCORD_TAGONLY_WEBHOOK }}
run: |
conda activate test
python inject_env_vars.py
Expand Down
2 changes: 2 additions & 0 deletions inject_env_vars.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@
val = os.getenv(env_var)
if val:
variables[env_var] = val
else:
raise LookupError(f"Missing env var {env_var}")

spec['VARIABLES'] = variables

Expand Down

0 comments on commit 9d4b94d

Please sign in to comment.