Skip to content

fix(signal-processing): Correctly template namespace in notebook#437

Merged
NickLarsenNZ merged 9 commits into
mainfrom
fix/signal-processing-jupyterhub-chart
Jul 13, 2026
Merged

fix(signal-processing): Correctly template namespace in notebook#437
NickLarsenNZ merged 9 commits into
mainfrom
fix/signal-processing-jupyterhub-chart

Conversation

@Techassi

@Techassi Techassi commented Jul 13, 2026

Copy link
Copy Markdown
Member

The namespace templating is broken in a couple of places and this PR fixes that.

Comment thread stacks/signal-processing/jupyterhub.yaml Outdated
The iptables call ing the It fails for me. I am on kernel 7.0. It could be that.
Or maybe oci.stackable.tech/demos/jupyter-pyspark-with-alibi-detect needs some update?
Add extraEnv for providing the templated namespace.
We would use the downward api, but that breaks because the value of extraEnv is also passed to `hub` which expects a dict.
@NickLarsenNZ

Copy link
Copy Markdown
Member

Tested the demo and it now works. Just need to update the branches back to main.

Comment thread demos/signal-processing/create-nifi-ingestion-job.yaml Outdated
Comment thread stacks/signal-processing/jupyterhub.yaml Outdated
Co-authored-by: Nick <10092581+NickLarsenNZ@users.noreply.github.com>
@Techassi Techassi marked this pull request as ready for review July 13, 2026 15:03

@NickLarsenNZ NickLarsenNZ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@NickLarsenNZ NickLarsenNZ merged commit 9de95ee into main Jul 13, 2026
2 checks passed
@NickLarsenNZ NickLarsenNZ deleted the fix/signal-processing-jupyterhub-chart branch July 13, 2026 15:43
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