-
Notifications
You must be signed in to change notification settings - Fork 23
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
Refactor Ansible playbook to extract a separate Jupyterhub role #76
Conversation
asmacdo
commented
May 10, 2023
•
edited by yarikoptic
Loading
edited by yarikoptic
- Ensure that no local paths are used (see comment below)
- Get the load balancer (metallb) working: https://kind.sigs.k8s.io/docs/user/loadbalancer/
- Add tests to the molecule
- Add CI workflow to trigger tests
- Add basic documentation on how to use this locally: initiate section Testing with the details in README.md
- Figure out deployment aspects before merging PR:
- Would it break existing hub if re-deployed?
(From roles/ dir) === Do not change lines below === { "chain": [], "cmd": "molecule init role dandi.jupyterhub --driver-name podman", "exit": 0, "extra_inputs": [], "inputs": [], "outputs": [], "pwd": "roles" } ^^^ Do not change lines above ^^^
kubeconfig: "{{ lookup('env', 'KUBECONFIG') }}" | ||
env: | ||
KUBECONFIG: /home/austin/.kube/config-molecule-test | ||
K8S_AUTH_KUBECONFIG: /home/austin/.kube/config-molecule-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems to have absolute paths and austin specific -- could be avoided ? may be at least via ~/
or $HOME
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops missed that one thanks
License | ||
------- | ||
|
||
BSD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BSD | |
BSD-3 |
?
@@ -0,0 +1,15 @@ | |||
--- | |||
# TODO FIX THIS HACK |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# TODO FIX THIS HACK | |
# TODO FIX THIS HACK: create proper vars definitions | |
# They come from https://github.com/dandi/dandi-hub/blob/dandi/group_vars/all but only the ones | |
# used in this playbook. |
Paused for now |
This is likely not necessary if we move forward with #90 |