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 locale and other environment variables support for systemd service jupyterhub #414

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

wqh17101
Copy link

@wqh17101 wqh17101 commented Aug 7, 2019

  • Add / update documentation
  • Add tests
    no more tests are needed.

#410

@wqh17101
Copy link
Author

wqh17101 commented Aug 7, 2019

@yuvipanda @GeorgianaElena @minrk please review

@wqh17101 wqh17101 changed the title Add locale support Add locale and other environment variables support for systemd service jupyterhub Aug 7, 2019
Copy link
Member

@consideRatio consideRatio left a comment

Choose a reason for hiding this comment

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

I understand that this PR is centered about the technical change of appending the locale as reported via os.popen("locale") to environment_for_system_service.conf, and then making the jupyterhub registered service run with that file.

I think the idea sounds good, but I'm quite uncertain on how the LANG environment variable functions. Perhaps the feature can be documented more general about setting env vars for the jupyterhub service process and then providing one example that the LANG environment variable for example can be configured via this feature.

I added a lot of small syntax details to the documentation which I also think could be revised to be a bit briefer.

docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
tljh/installer.py Outdated Show resolved Hide resolved
tljh/systemd-units/jupyterhub.service Outdated Show resolved Hide resolved
docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
docs/topic/env-var-for-tljh-service.rst Outdated Show resolved Hide resolved
@consideRatio consideRatio mentioned this pull request Sep 11, 2024
31 tasks
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.

3 participants