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

Describe minimal responsibilities of Turing mybinder Federation administrator #1785

Merged
merged 13 commits into from May 4, 2021

Conversation

sgibson91
Copy link
Member

@sgibson91 sgibson91 commented Mar 8, 2021

Summary

This PR adds a document that describes the minimal responsibilities required of the Turing mybinder Federation cluster administrator, a role that has so far been filled by @sgibson91.

List of changes proposed in this PR (pull-request)

  • Add project_management/turing_mybinder_role.md

What should a reviewer concentrate their feedback on?

  • Anything I've missed?
  • Is everything clear?
  • Everything looks ok?

Acknowledging contributors

@pr-triage pr-triage bot added the PR: draft label Mar 8, 2021
@netlify
Copy link

netlify bot commented Mar 8, 2021

Deploy preview for the-turing-way ready!

Built with commit 12aad64

https://deploy-preview-1785--the-turing-way.netlify.app

@sgibson91 sgibson91 changed the title Add description of mybinder maintainer role Describe minimal responsibilities of Turing mybinder Federation administrator Mar 8, 2021
@sgibson91 sgibson91 marked this pull request as ready for review March 8, 2021 15:23
@betatim
Copy link
Collaborator

betatim commented Mar 16, 2021

Looks good to me.

It would be good if a new person introduces themselves to the rest of the mybinder.org team. Not quite sure how to integrate that into this document because it is a one off task, not an ongoing one. Maybe it isn't needed but I thought that explicitly mentioning "introduce yourself and tell people who you are and why you are showing up" would send a clear signal that "yes please, come and talk to us and chat and become more than some semi-anonymous GitHub handle" is encouraged.

@sgibson91
Copy link
Member Author

Thanks @betatim! I had had the same thought in my head and would absolutely have recommended this in conversations with the new person - but finding a way to explicitly note this (whether it's this document or elsewhere) would be beneficial ✨

Explain that knowing each other is important in the Binder community
Link as places to get up-to-date info
Copy link
Collaborator

@Arielle-Bennett Arielle-Bennett left a comment

Choose a reason for hiding this comment

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

Have left some minor comments in, but otherwise this is looking good!


## Requirements

- An account in the Turing's Azure tenancy
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might be worth adding who can apply for these / how people apply for these (as you've done with the point below)

Copy link
Member Author

Choose a reason for hiding this comment

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

Addressed in f591545

Copy link
Member Author

Choose a reason for hiding this comment

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

Might be worth adding who can apply for these / how people apply for these (as you've done with the point below)

There are a couple of internal people I'd like to approach first - but if neither of them are interested, I'll add a more formal "how to apply" section with guidance from @malvikasharan

project_management/turing_mybinder_role.md Outdated Show resolved Hide resolved
project_management/turing_mybinder_role.md Show resolved Hide resolved
@sgibson91
Copy link
Member Author

Thank you @Arielle-Bennett! 🌺

Copy link
Collaborator

@KirstieJane KirstieJane left a comment

Choose a reason for hiding this comment

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

Really appreciate capturing all this! Thank you!

A couple of comments in line. Hopefully very easy to incorporate.

project_management/turing_mybinder_role.md Outdated Show resolved Hide resolved
project_management/turing_mybinder_role.md Outdated Show resolved Hide resolved
project_management/turing_mybinder_role.md Outdated Show resolved Hide resolved
project_management/turing_mybinder_role.md Show resolved Hide resolved
@sgibson91
Copy link
Member Author

sgibson91 commented Apr 13, 2021

TODO:

  • Add responsibility of generating stats for quarterly impact reports and links to tools to achieve this

sgibson91 and others added 3 commits April 15, 2021 12:54
Co-authored-by: Kirstie Whitaker <kwhitaker@turing.ac.uk>
Co-authored-by: Kirstie Whitaker <kwhitaker@turing.ac.uk>
@sgibson91
Copy link
Member Author

Thanks @KirstieJane! I've incorporated your feedback and restructured the responsibilities section

Copy link
Collaborator

@KirstieJane KirstieJane left a comment

Choose a reason for hiding this comment

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

This looks great @sgibson91 !! Thank you!

Fingers crossed we can find someone who wants to take the work forwards!

@sgibson91 sgibson91 merged commit 1b95de0 into the-turing-way:master May 4, 2021
@pr-triage pr-triage bot removed the PR: unreviewed label May 4, 2021
@sgibson91 sgibson91 deleted the turing-mybinder-role branch May 4, 2021 08:35
@pr-triage pr-triage bot added the PR: merged label May 4, 2021
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

5 participants