Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Info

Quick Facts:

  • 2 week sprint-aligned rotation within standard business hours.

  • @openedx/

...

Contents

Table of Contents

...

Schedule

...

Tentative schedule. Please adjust as appropriate:

...

3/7–3/18: Edward Zarecor

...

3/21-4/15: Sarina Canelake

On-Call Order

For exact dates, see the “Axim On-Call” Google calendar (Axim-internal).

(New Axim employees, you’ll need access to that calendar if you don’t have it already)

Past On-Calls

Expand
titlePast On-Calls

2022

  • 4/18-4/

...

...

2023

Details

🔔 What is

...

axim-oncall?

  • A lightweight rotation of tCRIL Axim engineers, ensuring that someone is always responsible for responding to administrative requests from the community.

❌ What isn’t

...

axim-oncall?

  • Something that should be done outside of normal working hours.

  • Something that should affect weekends, holidays, or PTO.

  • Pager duty.

  • A designation that you need to handle ALL incoming community requests yourself.

📒 What are the responsibilities?

  • Respond to GitHub Requests within one business day. See inline instructions for “Common Requests”.

    Ensure
    • Requests should automatically ping the @openedx/tcrilaxim-oncall GitHub group.

    • Assign yourself to the issue.

    • If you can’t resolve the request within a day, that is fine. Just give an “Ack” and a general timeline estimate.

  • Review and merge CLA PRs, delete related branches. Currently a PR is created daily, many of which are empty – we’ll work on improving the Zap to account for this.

Expand
titleBuild failure for CLA PRs: AssertionError: Account Name is...

If you get a pull that has a new organization’s name in it, you’ll get a failure on the build. You’ll need to add the new org’s info (found in Salesforce; attached to the account record in the “related” tab under contracts) to orgs.yaml.

Reference PR: https://github.com/edx/repo-tools-data/pull/171

    • Please note that users must also be added to the CLA check database. The specifics are documented here.

  • By the end of the sprint, ensure each GitHub Request is either:

    • closed

    within the sprint
    • , or

    • explicitly reviewed & handed off to the next on-call engineer at the beginning of the following sprint.

    • If you are swamped during the sprint, delegate to other team members!

  • Respond to other pings to the @openedx/tcrilaxim-oncall group if they happen.

  • Keep an eye on the #openedx-transition and #tcril-2u-openedx channels. If tCRIL #ask-axim Slack channel. If Axim-oriented questions come up that nobody is responding to, bring them up in daily standup so we don’t forget about them.

⁉️ How do I handle requests?

🌀 How does the rotation work?

  • The Rotation order of the rotation is defined by at the list of people on the tCRIL Engineering homepage, excluding Ed and Jennatop of this page.

  • At the semi-weekly Planning Meeting, the on-call engineer switches to the next person in the order. Switching process:

    • Update the @openedx/tcrilaxim-oncall GitHub group.

    • The on-call engineer should audit their personal list of assigned issues. For issues that are related to on-call, the engineer can decide to “keep” the issue or “hand off” the issue to the next on-call engineer.

    • Make sure to update the assignee when handing off an issue.

  • If the on-call engineer is on PTO during their turn in the rotation, they should trade days with another engineer or trade the entire sprint. Don’t do on-call from vacation (smile)

🧠 Anything else to know?

  • Record any access changes or decisions you make in GitHub issues. This will help us keep track of what changes we make and why.

  • To be on the rotation, you will need admin (“owner”) rights on the openedx GitHub organization, which will confer you admin-level rights to all repositories in the organization. Be judicious with these. In the majority of cases we should be abiding by all normal branch protection rules.

    • Appropriate uses of admin rights include:

      • Handling tCRIL Axim GitHub Requests.

      • Fixing minutiae like out-of-date documentation links or typos in repository descriptions.

      • Merging without review or CI validation in order to fix CI itself or update repository metadata (openedx.yml).

    • Inappropriate uses include:

      • Merging app code without appropriate review and CI validation.

      • Adding or removing access rights when there is no corresponding tCRIL Axim GitHub Request.