Skip to content

feat(datasets): Add autogen support for OpiktraceDataset#1295

Merged
SajidAlamQB merged 17 commits into
mainfrom
feat/add-autogen-support-for-opiktracedataset
Feb 5, 2026
Merged

feat(datasets): Add autogen support for OpiktraceDataset#1295
SajidAlamQB merged 17 commits into
mainfrom
feat/add-autogen-support-for-opiktracedataset

Conversation

@SajidAlamQB
Copy link
Copy Markdown
Contributor

@SajidAlamQB SajidAlamQB commented Jan 30, 2026

Description

Related to: #1277

To test for QA use the kedro-academy example: kedro-org/kedro-academy#105

Adds autogen mode to OpikTraceDataset, enabling OpenTelemetry based tracing for AutoGen agent pipelines via Opik's OTLP endpoint.

Development notes

Added autogen mode to OpikTraceDataset that returns a configured OpenTelemetry Tracer
_build_autogen_tracer() sets up an OTLP exporter with Opik authentication headers.
AutoGen mode skips Opik SDK configure() uses OTLP directly.

Checklist

  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Updated jsonschema/kedro-catalog-X.XX.json if necessary
  • Added a description of this change in the relevant RELEASE.md file
  • Added tests to cover my changes
  • Received approvals from at least half of the TSC (required for adding a new, non-experimental dataset)

SajidAlamQB and others added 6 commits January 28, 2026 11:23
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
@SajidAlamQB SajidAlamQB marked this pull request as ready for review February 3, 2026 15:40
Comment thread kedro-datasets/kedro_datasets_experimental/opik/opik_trace_dataset.py Outdated
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
…://github.com/kedro-org/kedro-plugins into feat/add-autogen-support-for-opiktracedataset

Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Comment thread kedro-datasets/kedro_datasets_experimental/opik/opik_trace_dataset.py Outdated
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Comment thread kedro-datasets/pyproject.toml Outdated
Copy link
Copy Markdown
Contributor

@ElenaKhaustova ElenaKhaustova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from the nit comment it looks good to me, thanks!

Tested with kedro-org/kedro-academy#105 and python3.11, python3.12

Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
Copy link
Copy Markdown
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some small comments, but generally this looks good to me 👍

Comment thread kedro-datasets/kedro_datasets_experimental/opik/opik_trace_dataset.py Outdated
Comment thread kedro-datasets/pyproject.toml
Signed-off-by: Sajid Alam <sajid_alam@mckinsey.com>
@SajidAlamQB SajidAlamQB merged commit fe0b064 into main Feb 5, 2026
28 checks passed
@SajidAlamQB SajidAlamQB deleted the feat/add-autogen-support-for-opiktracedataset branch February 5, 2026 14:37
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.

3 participants