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

Change naming from jupyter to jupyterhub when referring to hub #642

Merged
merged 5 commits into from Apr 12, 2018

Conversation

willingc
Copy link
Contributor

@willingc willingc commented Apr 11, 2018

This PR should add more consistent naming (and less confusion down the road) by using jupyterhub when referencing the JupyterHub hub. Typically, the Jupyter team refers to the Jupyter notebook or the team itself when using Jupyter name. JupyterHub is used to reference all hub related components and activities.

@jlewi I think I caught all the non-doc places that this occurred.


This change is Reviewable

@willingc
Copy link
Contributor Author

/assign @jlewi

@jlewi
Copy link
Contributor

jlewi commented Apr 11, 2018

Do you also want to rename https://github.com/kubeflow/kubeflow/blob/master/kubeflow/core/jupyterhub_spawner.py?

Since we aren't actually spawning JupyterHub?

@willingc
Copy link
Contributor Author

@jlewi Hmm... I didn't think of that one before. It may be best as kubeform_spawner. I guess it depends if you think there may be swapping of spawners over time. You make the call :-)

@jlewi
Copy link
Contributor

jlewi commented Apr 11, 2018

kubeform_spawner seems good to me.

@jlewi
Copy link
Contributor

jlewi commented Apr 12, 2018

/test all

@jlewi
Copy link
Contributor

jlewi commented Apr 12, 2018

/ok-to-test

@jlewi
Copy link
Contributor

jlewi commented Apr 12, 2018

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jlewi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit c8ea44e into kubeflow:master Apr 12, 2018
@willingc willingc deleted the naming-clarify branch April 12, 2018 22:59
saffaalvi pushed a commit to StatCan/kubeflow that referenced this pull request Feb 11, 2021
…low#642)

* Change naming from jupyter to jupyterhub when referencing the hub

* Change jupyter to jupyterhub when referring to the hub

* Change references to jupyter to jupyterhub when hub is being used

* Rename jupyterhub spawner to kubeform_spawner

* update docs for name changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants