Skip to content

refactor(providers): remove profile telemetry buckets - #3914

Draft
johntmyers wants to merge 1 commit into
mainfrom
refactor/3906-provider-telemetry-cleanup/johntmyers
Draft

johntmyers wants to merge 1 commit into
mainfrom
refactor/3906-provider-telemetry-cleanup/johntmyers

Conversation

@johntmyers

Copy link
Copy Markdown
Collaborator

🏗️ build-from-issue-agent

Summary

Remove the static inference provider table and retire profile identity from provider lifecycle telemetry. Provider events continue to report operation and outcome, so adding an example YAML profile requires no telemetry registration or table row.

Related Issue

Refs #3906. This is a draft until the telemetry collector/report owner confirms that events without providerProfile are accepted alongside historical events with it.

Changes

  • Remove the provider-profile telemetry enum and providerProfile parameter from the core event payload.
  • Simplify provider create, update, and delete event emission without profile lookups or ID mapping. Keep existing success/failure outcomes.
  • Remove the inference provider table, use openshell profile list for the imported catalog, and update the published telemetry description.

Deviations from Plan

The current main branch has no architecture/ directory, so the documentation update is limited to the relevant published docs. The plan comment was revised to reflect that.

Testing

  • cargo test -p openshell-core --lib telemetry (11 passed)
  • cargo test -p openshell-core --lib --no-default-features telemetry (2 passed)
  • cargo test -p openshell-server --lib grpc::provider::tests (156 passed)
  • mise run pre-commit
  • mise run ci
  • External collector/report compatibility confirmed by telemetry owner

The first mise run test attempt lacked local mke2fs; after installing it, a supervisor-network test timed out, including when run alone. The same test passed in the subsequent full mise run ci run. No e2e files or sandbox infrastructure changed.

Checklist

  • Conventional Commit with DCO sign-off
  • Published providers and telemetry docs updated
  • Confirm downstream collector accepts the omitted field before merging

Remove the static inference provider table and stop emitting profile identity in provider lifecycle events. Keep operation and outcome for aggregate reporting.

Refs #3906

Signed-off-by: John Myers <9696606+johntmyers@users.noreply.github.com>
@copy-pr-bot

copy-pr-bot Bot commented Sep 29, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant