Skip to content

Conversation

@sverhoeven
Copy link
Member

@sverhoeven sverhoeven commented Sep 11, 2024

Fixes #166

TODO

  • merge main and grader
  • merge grader-samba, leaving dcache stuff intact
  • name for variable that is either dcache or samba: some suggestion data_lake_format, readonly_data_type, storage_type, data_source, data_mount_type, shared_data_source
  • add shared_data_source variable to pick either samba or dcache (or none??)
  • remove worker/jumphost cluster stuff, as its no longer needed as we will have single file server and a multiple jupyter hub machines started as separate workspaces.
  • Move dcache macaroon to org Store macaroon in Collaborative organisations secret #168
  • test with vagrant
    • dcache
    • samba
  • create component and catalog item on SRC
  • test on SRC
    • dcache without students
    • dcache with students
    • samba without students
    • samba with students
  • Remove SRC component and catalog items that are not used
  • Add deprecated to name of SRC component and catalog items that are used
  • just before merge rename occurrences of dcache-or-samba to main in repo and in SRC.
  • after merge mail Rolf that he should use eWaterCycle dcache or eWaterCycle samba + Samba server catalog items.

To test

  1. Follow instructions to create a workspace, either
    2. making the component/catalog item your self or
    3. use eWaterCycle dcache and eWaterCycle samba catalog items I made.

In ewatercycle-nlesc collab org I already created required secrets and private network.

sverhoeven and others added 30 commits February 13, 2024 13:02
…terhub_config.py in `systemctl status jupyterhub`
ERROR! [DEPRECATED]: ansible.builtin.include has been removed. Use include_tasks or import_tasks instead. This feature was removed from ansible-core in a release after 2023-05-16. Please update your playbooks.
Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.
Use value from defaults
@sverhoeven sverhoeven marked this pull request as ready for review October 14, 2024 11:39
Copy link
Member

Choose a reason for hiding this comment

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

Very good to have this readme file now 👍

'admin': True,
'api_token': '{{ lookup('password', 'jupyterhub.launcher.token length=32 chars=ascii_letters,digits') }}',
},
# {
Copy link
Member

Choose a reason for hiding this comment

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

this is now commented out. want to add a line explaining what it does if you uncomment it?

@sverhoeven sverhoeven merged commit a41b7e9 into main Jan 7, 2025
1 check passed
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.

one catalog item

3 participants