Bi-weekly summary of handbook changes

Hi team,

A gitlab pipeline will post a summary of the modifications to the public handbook repo every two weeks.

This is used to streamline the process of informing team members of changes to the handbook or processes that affect them.

Related Jira ticket: BB-6109

6 Likes

@team

Summary of merge requests for opencraft/documentation/public from 2022-04-19

Merge request: feat: [BB-6109] announce handbook summary of changes
Description: This MR implements a script to automatically summarize merge requests in handbook repo and posts it in announcements forum

Testing instructions

HANDBOOK_SUMMARY_NO_OF_DAYS=24 make post_summary

It should print summary in terminal

How change summary will be displayed in forum

Author: Navin Karkera

For more details visit: #440.


2 Likes

@team

Summary of merge requests for opencraft/documentation/public from 2022-05-03

Merge request: Adding Dat to cell list
Description: Adding Dat to cell list

Author: Saksham Agrawal

For more details visit: #444.


Merge request: Update security notification process [MNG-2952]
Description: I created a new email address for security notifications. It will trigger the creation of an OpsGenie alert, so that the on-call team can triage it. This email will be used for security notifications from associated software projects as well as internally to escalate a security issue.

Author: Braden MacDonald

For more details visit: #442.


4 Likes

@team

Summary of merge requests for opencraft/documentation/public from 2022-05-17

Merge request: Show revision date for each page [MNG-2203]
Description: This shows a revision date on each page, using the git-revision-date plugin. Conveniently, our theme already includes support for the plugin and displays the data once the plugin is installed.

Author: Braden MacDonald

For more details visit: #446.


Merge request: [FAL-49] Move Saksham from Deathstar to Falcon.
Description: Ref.

Testing

  • Check that list of cells renders correctly (with Saksham being listed as a member of Falcon).

Reviewers

  • [x] @saksham115

Author: Tim Krones

For more details visit: #445.


Merge request: [STAR-2294] Refine documentation on Community Relations and corresponding roles
Description: Part of cell support initiative to improve documentation on cell and company-wide roles.

  • Separates responsibilities of Community Liaison role from responsibilities of OSPR Liaison role (to reflect the fact that we’ve been treating these two as separate roles internally for a while, and to make it possible for future assignees of the roles to understand what is expected of them).
  • Applies various additional updates to make descriptions of roles that fall under Community Relations match their current scope.

Tickets

Author notes

Regarding scope of the Community Liaison role:

  • [x] Do we want to keep the constraint that the Community Liaison role should own the Core Contributor Program epic? In reality, this has never been the case – the epic has always been owned by Braden.
  • [x] Do we have plans to take back responsibility for leading the bi-weekly contributor’s meetup? Jorge Londoño took over from Nizar in February and it looks like he’s planning on keeping the role (and other participants are happy with his work).
  • [x] Are monthly community hangouts still happening? According to the Confluence page that the text currently links to, the latest event happened over a year ago, in March 2021.
    • They are still happening but not on a specific schedule: ref.

Testing

Read the updated text and check the rendered version for formatting issues.

Reviewers

  • [x] @FoxyFoxie

Author: Tim Krones

For more details visit: #441.


3 Likes

@bebop Just a heads-up that I created a ticket with subtasks for each one of you for checking out these changes; it’s scheduled for next sprint.

We’ll need to find a new assignee for the Community Liaison role (as well as the Community Relations epic), so I wanted to make sure everyone would be able to block out time to read the updated role definitions from this PR in detail.

Hi! To find out what I can do, say @crafty display help.

@team

Summary of merge requests for opencraft/documentation/public from 2022-05-31

Merge request: [STAR-2522] Update new hire process doc
Description: I’ve made a few changes to the recruitment doc, to fix a few errors and to include the new process around certificates of residency, which are now required for every contractor.

Author: Gabriel D’Amours

For more details visit: #449.


Merge request: docs: [BB-6259] remove mention of manual scheduling of social chats
Description: Description

As part of eliminating confusion around scheduling social chats, this MR removes mention of manual scheduling as this process is now automated.

Related tickets

Jira: BB-6259

Author: Navin Karkera

For more details visit: #448.


1 Like

@team

Summary of merge requests for opencraft/documentation/public from 2022-06-16

Merge request: [BB-6322] Assign Community Relations roles
Description: - Officially assigns OSPR Liaison role (which had already had an owner prior to BB-6322 but we hadn’t made that explicit in the handbook).

Screenshots

Tickets

Author: Tim Krones

For more details visit: #451.


Merge request: fix: formatting issue
Description: The Note was overshadowing the list of criteria to consider a ticket Done.

Jira ticket

FAL-211

Screenshot

Broken (currently on handbook):

Fixed:

Testing instructions

  1. In a virtualenv, run:

    make install_prereqs
    make preview
    
  2. View preview/task_workflows.html and check that the above issue is fixed as shown.

Author: Jillian Vogel

For more details visit: #450.


1 Like

@team

Summary of merge requests for opencraft/documentation/public from 2022-06-28

Merge request: [BB-6453] Bump pip-tools
Description: The CI process was broken due to pip and pip-tools being incompatible.

It seems the docker image for python:3.8 just got upgraded to 3.8.13 and pip to 22.0.4 which is incompatible with pip-tools 6.4.0

This MR bumps pip-tools to 6.7.0 which is compatible.

Author: Paulo Viadanna

For more details visit: #453.


Merge request: Update Navin as core member and removing Sandeep
Description: Updating Navin as the core member and removing Sandeep.

Author: Saksham Agrawal

For more details visit: #452.


1 Like

@team

Summary of merge requests for opencraft/documentation/public from 2022-07-12

Merge request: docs: add documentation about firefighting task creation automation
Description:

Author: Kaustav Banerjee

For more details visit: #459.


Merge request: [STAR-2504] fix: Broken navigation links
Description: Quick fixes for a few broken navigation links as a follow-up to this MR.

Jira Ticket: Log in - OpenCraft

Author: Fox Piacenti

For more details visit: #458.


Merge request: [STAR-1746] Update recruitment documentation
Description: This MR improves and updates documentation on how we proceed with recruitment steps.

Author: Gabriel D’Amours

For more details visit: #457.


Merge request: [STAR-2622] Update candidate selection steps
Description: Clarify role and steps for the Admin Specialist in the selection steps.

Author: Gabriel D’Amours

For more details visit: #455.


Merge request: feat: auto migrate zoom recordings
Description: This MR implements a script to automatically migrate zoom meeting recordings to google drive.

The plan is to have the script run automatically every week (tentatively) using Gitlab schedules feature.

Following features are included:

  • Migrate meetings from multiple zoom accounts based on provided config (environment variables).
  • Supports filtering meetings based on topic regex pattern, start and end date.
  • Email notification about the process result i.e. uploaded/failed videos for each account.
  • Uploads videos to designated folder in drive for each account.
  • Added a section about automation in handbook with instructions about running the script

Supporting information

Testing instructions:

Please follow the instructions provided here to test the script locally.

Rendered instructions can be found in automation/migrate_zoom_recordings.html section of the preview in MR

Post merge checklist:

  • [x] Update current scheduled jobs to include PROCESS_NAME variable
  • [ ] Add/update gitlab ci/cd variables
  • [x] Create a new schedule for this process

Author: Navin Karkera

For more details visit: #454.


Merge request: [STAR-2302, STAR-2504] Revise Business Development, DD, add Value Proposition.
Description: This MR revises Business Development Processes, specifically the sales process. The revisions involve working with the Epic Planning and Sustainability Manager, processes for Client Owners to become more involved in sales, and some significant changes to the Discovery Duty role.

This MR also introduces the Value Proposition Template and processes surrounding its creation and maintenance, and several smaller tweaks.

JIRA tickets: Log in - OpenCraft Log in - OpenCraft

Author: Fox Piacenti

For more details visit: #447.


2 Likes

– Removed – This automated post was made in error, but I can’t figure out how to delete it (yet)

@team

Summary of merge requests for opencraft/documentation/public from 2022-07-26

Merge request: docs: Add executive summary to sales section.
Description: This MR adds an example executive summary to the handbook.

Author: Fox Piacenti

For more details visit: #466.


Merge request: feat: auto migrate zoom recordings
Description: Fix accidental removal of commit a6ef28637da29c5cddfcf934fb3b5a7c5f7b4af6 in 5b4334c306f0f2f6bb9108c7e99c9bdec3e22422.

Author: Navin Karkera

For more details visit: #463.


Merge request: docs: add section to include relevant repositories in task description
Description: Includes a section in the standard task description template in our handbook, for the task reporter to include the repository/branch details where-ever relevant/possible during task creation.

Jira: BB-6472

Author: Navin Karkera

For more details visit: #461.


Merge request: [STAR-2578, STAR-2636] Adjust firefighting rules to reduce stress and improve coverage.
Description: - Add recommendations for handling scenarios where pre-allocated FF hours aren’t sufficient.

  • Make firefighting rules more flexible by allowing cell-specific defaults for total volume of firefighting hours and number of firefighters per sprint.
  • Record cell-specific rules for Bebop.

Also, refactor end extend existing documentation on firefighting:

Tickets

Related PRs

Test instructions

  • Read the following sections in the rendered version of the changes and make sure that the formatting is appropriate and that links work correctly:
    • Cells > Cell Budgets > Firefighting and cell budgets
    • Cells > Cell-Specific Rules > Bebop
    • Processes > Firefighting (new page)
    • Roles and Expectations > List of roles > Firefighter

Reviewers

Author: Tim Krones

For more details visit: #456.


1 Like

@team

Summary of merge requests for opencraft/documentation/public from 2022-08-09

Merge request: fix: remove dat from handbook list
Description:

Author: Arunmozhi

For more details visit: #471.


Merge request: fix: set test environment prefix
Description: Set env prefix for test environment to avoid clashing with production env variables.

Author: Navin Karkera

For more details visit: #470.


Merge request: docs: fix markdown formatting
Description: ## Description

Fixes a few markdown formatting issues in handbook

Supporting information

Link to other information about the change, such as Jira issues, GitHub issues, or Discourse discussions.
Be sure to check they are publicly readable, or if not, repeat the information here.

Testing instructions

Please provide detailed step-by-step instructions for testing this change.

Deadline

“None” if there’s no rush, or provide a specific date or event (and reason) if there is one.

Other information

OpenCraft Internal: BB-6536

Author: Kaustav Banerjee

For more details visit: #469.


Merge request: feat: retrospective meeting migration
Description: Migrates sprint retrospective meeting from zoom to google drive and posts the link to respective cell forum.

Test instructions

You can find successful run of this new schedule here

To test locally:

  1. Launch any retrospective meeting from opencraft calendar. Let it start recording and then leave.
  2. export environment variables locally (ignore DISCOURSE_API_*, it is handle by DISCOURSE_CONFIG), ref
  3. Make sure to set DEBUG=True
  4. run make migrate_retro_meetings
  5. Inspect drive folder for uploaded video.
  6. Check forum post printed in console.
  7. Finally delete the retrospective meeting from zoom recordings either from zoom ui or run the above script with DEBUG=False and make sure to change discourse config to something random.

Other information

Private-ref: BB-6586

Post merge checklist

  • [ ] Update env variables
  • [ ] create new schedule for retro migration

Author: Navin Karkera

For more details visit: #468.


Merge request: docs: minimize exposure of private links in public PRs and commits
Description: ## Description

It is important that all information which is valuable for context about changes from PRs/commits should be publicly available in those PRs and commits for all community members to access and not be behind private links.

As per this change to OEP-51 and this discussion, the Open edX community is trying to ensure that private links and references are not included in the subject lines of the PRs and commits. Further such private references should only be present at the footers of the commit messages with them being tagged as such.

This MR adds/tweaks revelant sections in our handbook, so that we follow the recommendations of the community for our upstream PRs and commits as well.

Supporting information

Link to other information about the change, such as Jira issues, GitHub issues, or Discourse discussions.
Be sure to check they are publicly readable, or if not, repeat the information here.

Testing instructions

Please provide detailed step-by-step instructions for testing this change.

Deadline

“None” if there’s no rush, or provide a specific date or event (and reason) if there is one.

Other information

OpenCraft internal ticket BB-6536

Author: Kaustav Banerjee

For more details visit: #465.


1 Like

@team

Summary of merge requests for opencraft/documentation/public from 2022-08-23

Merge request: docs: client strategy tickets scheduling was automated
Description: Changed wording to indicate that client tickets will be scheduled
automatically once a quarter, instead of client owners having to do it
manually.

Private-ref: BB-6257

Author: Maxim Beder

For more details visit: #474.


Merge request: Remove Pro & Teacher Plan
Description: ## Description

This MR removes the Pro & Teacher plan, leaving only an explanation as to what it was.

Supporting information

Other information

Also tweaked the sustainability section to match our decision to still offer courses.opencraft.com up for community course projects.

Author: Fox Piacenti

For more details visit: #472.


Merge request: [STAR-1746] Added link to Billing onboarding document
Description: Added link to Billing onboarding document → tutorials/howtos/billing-onboarding.md · master · opencraft / documentation / public · GitLab

Author: Gabriel D’Amours

For more details visit: #464.


@team

Summary of merge requests for opencraft/documentation/public from 2022-09-06

Merge request: [STAR-2754] Fix links for Value Proposition Template and Client Briefing Template
Description: ## Description

This fixes links in the repo to template files.

MkDocs does not render Markdown files which aren’t in the navigation. Rather than attempt to fix this bug, we’re elevating it to ‘feature’ since we don’t want to use the same rendering theme that the rest of the handbook does, and GitLab already provides one. So, included are updated links and an updated way to build new links in case other files in this repository need to be used in a similar way.

Testing instructions

Use the ‘view app’ button on this MR, and browse the tree to verify the new README runs correctly. Also you can build locally (after initial setup, see the repo README) with make run and test there.

Author: Fox Piacenti

For more details visit: #475.


@team

Summary of merge requests for opencraft/documentation/public from 2022-09-20

Merge request: fix: use resumable google uploads

Description: Using resumable=true will upload files in chunks which defaults to 104857600 bytes i.e. 104 mb.

Author: Navin Karkera

For more details visit: #476.


Merge request: [STAR-2688] Update handbook to match plans for addressing sustainability issues.

Description:

  • Apply changes from STAR-2602 and corresponding forum discussion to the handbook.
  • Complete steps for retiring DevOps Specialist role.
  • Apply small drive-by fixes.

Tickets

Test instructions

  • Changes from STAR-2602 are in 4c54b0d563eefbbba84786b1725fb42952faa6ba. Review this commit separately and confirm that the process changes triggered by STAR-2602 are reflected in the text.
  • Check preview deployment of changes for any rendering issues.

Reviewers

Author: Tim Krones

For more details visit: #467.