Repository navigation
docs: add adr for the sched historical import script - #5205
Conversation
Refs linuxfoundation/lfx-cdp-insights-ops#944 Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
PR SummaryLow Risk Overview The ADR records the chosen approach: a one-time, re-runnable crowd.dev script that calls the Sched API per event and emits activities through Reviewed by Cursor Bugbot for commit 66dad56. Bugbot is set up for automated code reviews on this repo. Configure here. |
[Low impact] The documentation appears safe to merge as a proposed decision. SummaryAdds proposed ADR-0037 for a one-time, re-runnable Sched historical import and adds it to the ADR index.
Reviews (6) · Last reviewed commit: "docs: qualify the service account altern..." · Reviewed by Greptile |
There was a problem hiding this comment.
🔵 Needs a closer look
CDP’s first OCI Vault integration requires CloudOps/security validation of the unspecified workload-identity and access-policy design, secret provisioning, and credential deletion before the ADR can be accepted.
No open issues 🎉
What changed in this PR
Documents a proposed approach for importing LF’s historical Sched events into CDP before the move to Sessionize in 2027. This PR adds documentation only; implementation and product decisions remain pending.
Changes:
- Proposes a one-time, re-runnable script using the existing data-sink ingestion pipeline.
- Defines event-to-project mapping, per-event Oracle Vault secrets, and safeguards for repeat runs and API limits.
- Records alternatives, operational risks, and open product questions.
| File | Description |
|---|---|
docs/adr/README.md |
Adds the index entry for proposed ADR-0037. |
docs/adr/0037-sched-historical-import-script.md |
Documents the historical import approach, alternatives, constraints, and unresolved decisions. |
Review effort: Balanced
…import-adr Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org> # Conflicts: # docs/adr/README.md
mbani01
left a comment
There was a problem hiding this comment.
LGTM - as discussed, we can skip the vault since this will be a one-time use and there's no need to store the keys
There was a problem hiding this comment.
🔵 Needs a closer look
Teammate validation is needed for the Sched API assumptions, username stability underlying deduplication, and initial OCI Vault/workload-identity setup, as this docs-only PR provides no implementation or POC to verify them.
No open issues 🎉
Review effort: Balanced
Refs linuxfoundation/lfx-cdp-insights-ops#944 Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
There was a problem hiding this comment.
🔵 Needs a closer look
CDP/product and security review is needed to resolve credential permissions and replay identity rules and validate Sched-specific backfill assumptions not established by repository evidence.
No open issues 🎉
Review effort: Balanced
Refs linuxfoundation/lfx-cdp-insights-ops#944 Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
Refs linuxfoundation/lfx-cdp-insights-ops#944 Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
There was a problem hiding this comment.
🔵 Needs a closer look
A CDP reviewer with Sched API/POC access needs to validate credential, rate-limit, and identifier-stability assumptions and resolve timestamp/project choices, since the linked requirements were inaccessible.
No open issues 🎉
1 resolved since last review
Review effort: Balanced

Summary
Adds ADR-0037, the technical approach for importing past Sched events into CDP, for review by a teammate before any code is written. LF events leave Sched in 2027, so instead of an ongoing connector we import the history of LF's past Sched events (658 that LF staff administer, 2016–2026) once, with code that can be re-run later. The product spec is here.
Changes
docs/adr/0037-sched-historical-import-script.md(status: proposed):docs/adr/README.md: index row for ADR-0037.Numbered 0037 because #5185 already uses 0036.
Organizations and the product sign-off (key type, activity types and scores, registration time, project per event) are still open with product; the ADR notes them and covers the technical approach only. On approval, the reviewer goes into Deciders and the status moves to accepted.
Type of change
Related issue
Closes linuxfoundation/lfx-cdp-insights-ops#944