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 gitlab_url trait to the chart #1569

Closed
rprimet opened this issue Feb 11, 2020 · 4 comments
Closed

Add gitlab_url trait to the chart #1569

rprimet opened this issue Feb 11, 2020 · 4 comments

Comments

@rprimet
Copy link

rprimet commented Feb 11, 2020

Looking at the gitlab-specific configuration section in the chart (

elif auth_type == 'gitlab':
c.JupyterHub.authenticator_class = 'oauthenticator.gitlab.GitLabOAuthenticator'
for trait, cfg_key in common_oauth_traits + (
('gitlab_group_whitelist', None),
('gitlab_project_id_whitelist', None),
):
if cfg_key is None:
cfg_key = camelCaseify(trait)
set_config_if_not_none(c.GitLabOAuthenticator, trait, 'auth.gitlab.' + cfg_key)
), it seems that the gitlab_url trait is missing from the list of configurable gitlab-specific traits.

Adding it would allow configuring a custom gitlab instance without resorting to python snippets.

@metonymic-smokey
Copy link
Contributor

@rprimet I'll make an attempt to figure this out.
From what I understood, you want the gitlab_url trait to be added to the existing traits like gitlab_group_whitelist and gitlab_project_id_whitelist right?
Is there anything I've missed?

@rprimet
Copy link
Author

rprimet commented Feb 13, 2020

@metonymic-smokey Yes that's it!

@metonymic-smokey
Copy link
Contributor

@rprimet I have a query about this issue.
In this WIP PR, I have used None as the cfg_key for the gitlab_url trait. I am not exactly sure if that's right/what value I should be using there.
Would love your inputs on the same!

Thanks!

@consideRatio
Copy link
Member

Thanks @rprimet and @metonymic-smokey for reporting and fixing this!

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

No branches or pull requests

3 participants