Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 4 Next »

\uD83D\uDDD3 Date

\uD83D\uDC65 Participants

⏪ Previous TODOs

DescriptionAssigneeTask appears on
  • Jeremy Ristau Will have someone ask about forum performance testing on the DEPR ticket
Jeremy Ristau2024-11-14 Meeting notes
  • Feanil Patel follow-up with Ed/Felipe about the codejail service and whether we should make it part of Openedx
Feanil Patel2024-10-24 Meeting notes
  • Feanil Patel Update elasticsearch support windows now that it’s viable again.
Feanil Patel2024-10-17 Meeting notes
  • Feanil Patel ticket enabling cron CI of master every week so we know when external changes might have broken some repos that are usually not getting updates.
Feanil Patel2024-09-12 Meeting notes
  • Jeremy Ristau ensure DEPR tickets are created for any frontend that can be deleted as a result of the new course-authoring MFE.
Jeremy Ristau2024-05-30 Meeting notes

\uD83D\uDDE3 Discussion topics

Topic

Presenter

Notes

https://github.com/openedx/edx-platform/pull/34746

Feanil Patel

  • The change creates a maintainer team, wg-maintenance-edx-platform, with Feanil and Kyle as members

  • maintainers OEP: https://open-edx-proposals.readthedocs.io/en/latest/processes/oep-0055-proc-project-maintainers.html#removing-a-maintainer

  • Robert: what does this change mean? should we have an ADR for it?

    • why change it? to have a different contact team

      • Feanil: Yes

      • Feanil: with DD work, it seems like arch-bom doesn’t have time to be the contact

      • Feanil has been doing a lot of decision making around edx-platform, would be good to represent that

  • Robert: edx-platform is in a state of evolution

    • not perfect architecturally

    • in the past, we have let things land imperfectly, eg things that should be plugins landing in the core

    • seems like Axim’s stance is to block these sort of changes going forward

    • question: is Axim investing in plugin points/improvements so that people are preemtively unblocked?

      • Feanil: willing to negotiate, but our drive to make those plugin points is higher

        • It is hard to remove things later

          • eg, edx-enterprise

  • Discussion around hopes/fears of having Axim be the “decider” on edx-platform changes

    • Feanil: Looking forward to having a framework in place for negotiating architectural tradeoffs

  • Jeremy & Robert will discuss and comment on PR

DD for server-side templates

Robert Raposa

(tabled)

Redwood bug triage for edx-platform

Kyle McCormick

(tabled)

Settings Files

  • Settings get set in yaml files and override in multiple settings.py files should we re-think this wholly.

    • Good to have this discussion on the obeseletion of OEP-45

New label for OSPRs needing reviewers?

Michelle Philbrick

  • Context:

    • Lots of repos still need maintainers or maintainers are out of date.

      • Can we update the auto tagging of reviewers on repos that are no longer maintained?

    • Needs reviewer filtering not working because PRs have people assigned that no longer makes sense.

      • Can we fix this or have a label that is independent?

    • Michelle and Tim not able to remove reviewers in certain cases which makes it hard to manage PRs that need review.

      • You need write access to a repo to remove reviewers on an issue.

      • We need to be able to identify PRs where the assigned reviewer is no longer reviewing or has otherwise stalled.

      • It would be great for other reviewers to quickly find these items and get them reviewed.

✅ Action items

  • Feanil Patel reach out to DKH to see who at 2U should be included in the Node 20 Retro, maybe an Arbibom Person? (Also optional, Sarina C, Robert R)
  • Michelle Philbrick will put in an Axim Request to add a new label to all repos.

⤴ Decisions

    ⏺️ Recording and Transcripts

  • No labels