Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add jinja template support to pod-name config via environment variable #1368

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

lresende
Copy link
Member

No description provided.

@lresende lresende requested a review from kevin-bates February 19, 2024 04:21
Copy link
Member

@kevin-bates kevin-bates left a comment

Choose a reason for hiding this comment

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

It would be awesome to add this to gateway provisioners as well!

@@ -75,7 +75,9 @@ There are several supported `KERNEL_` variables that the Enterprise Gateway serv
cases, Enterprise Gateway will honor that value. However, when provided,
it is the user's responsibility that KERNEL_POD_NAME is unique relative to
any pods in the target namespace. In addition, the pod must NOT exist -
unlike the case if KERNEL_NAMESPACE is provided.
unlike the case if KERNEL_NAMESPACE is provided. The KERNEL_POD_NAME can
also be provided as a jinja2 template (e.g "{{ kernel_id }}") string which
Copy link
Member

Choose a reason for hiding this comment

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

I think it would be good to show a practical example rather than just kernel-id. Could you please add a string prefix like "my-kernel" or something?

@kevin-bates kevin-bates added enhancement kubernetes gateway-provisioners Has application for gateway-provisioners (EG 4.0) labels Feb 19, 2024
@lresende lresende force-pushed the apple-pod-name-template branch from f7abefa to 2a213b6 Compare February 19, 2024 17:42
@lresende
Copy link
Member Author

@kevin-bates I added a little more complex jinja template, hopefully gives a better picture to users. As for gateway-provisioners, please let me know where to...

@lresende lresende requested a review from kevin-bates February 19, 2024 18:00
Copy link
Member

@kevin-bates kevin-bates left a comment

Choose a reason for hiding this comment

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

Excellent. Thank you @lresende!

@lresende lresende merged commit dd9e4ba into jupyter-server:main Feb 20, 2024
14 checks passed
@lresende lresende deleted the apple-pod-name-template branch February 20, 2024 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement gateway-provisioners Has application for gateway-provisioners (EG 4.0) kubernetes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants