Skip to content

[MLOB] add crewai integration tile #20053

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 7 commits into from
May 22, 2025
Merged

[MLOB] add crewai integration tile #20053

merged 7 commits into from
May 22, 2025

Conversation

Yun-Kim
Copy link
Contributor

@Yun-Kim Yun-Kim commented Apr 11, 2025

What does this PR do?

Adds the CrewAI integration tile: https://datadoghq.atlassian.net/browse/MLOB-2281?atlOrigin=eyJpIjoiZWY4MjhjMWIwM2MyNGRmNGFkNDI4YWQyZTJiNmMxYWIiLCJwIjoiaiJ9

Motivation

The ML Observability team is adding instrumentation for CrewAI, and would like to include this as part of Datadog's public integration tiles.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@Yun-Kim Yun-Kim requested review from a team as code owners April 11, 2025 00:04
@cswatt cswatt added the editorial review Waiting on a more in-depth review from a docs team editor label Apr 11, 2025
@cswatt
Copy link
Contributor

cswatt commented Apr 11, 2025

added DOCS-10602 to track documentation team review

cswatt
cswatt requested changes Apr 16, 2025
View reviewed changes
Copy link
Contributor

@cswatt cswatt left a comment

Choose a reason for hiding this comment

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

Left some feedback and questions. I realize this follows the general format of the other LLM integrations (i.e. the extraneous information I'd like to remove), but I'm working on editing those other integrations as well.

@Yun-Kim
Copy link
Contributor Author

Yun-Kim commented Apr 17, 2025

Thanks for the suggestions and feedback @cswatt! I've hopefully addressed them all, we are however waiting on the CrewAI logo from the DRUIDS team (ticket). I'm not sure how the flow works - I assume this integration tile is blocked until then, is that right?

@Yun-Kim Yun-Kim requested a review from cswatt April 17, 2025 17:47
cswatt
cswatt previously approved these changes Apr 21, 2025
View reviewed changes
apiazza-dd
apiazza-dd previously approved these changes May 6, 2025
View reviewed changes
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed cswatt’s stale review May 6, 2025 20:56

Review from cswatt is dismissed. Related teams and files:

  • documentation
    • crewai/manifest.json
@Yun-Kim Yun-Kim requested a review from cswatt May 6, 2025 20:59
@Yun-Kim
Copy link
Contributor Author

Yun-Kim commented May 6, 2025

Sorry for the multiple re-review requests @cswatt, this should be the last one 😓 Thank you!

@Yun-Kim
Copy link
Contributor Author

Yun-Kim commented May 21, 2025

/merge

@dd-devflow
Copy link

dd-devflow bot commented May 21, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-05-21 23:34:19 UTC ℹ️ Start processing command /merge

2025-05-21 23:34:24 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-05-22 03:35:04 UTC ⚠️ MergeQueue: This merge request was unqueued

devflow unqueued this merge request: It did not become mergeable within the expected time

@temporal-github-worker-1 temporal-github-worker-1 bot dismissed apiazza-dd’s stale review May 22, 2025 01:33

Review from apiazza-dd is dismissed. Related teams and files:

  • saas-integrations
    • .github/CODEOWNERS
    • .github/workflows/config/labeler.yml
@Yun-Kim
Copy link
Contributor Author

Yun-Kim commented May 22, 2025

/merge

@dd-devflow
Copy link

dd-devflow bot commented May 22, 2025 •
edited
Loading

View all feedbacks in Devflow UI.

2025-05-22 17:05:01 UTC ℹ️ Start processing command /merge

2025-05-22 17:05:09 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

2025-05-22 17:50:50 UTC ℹ️ MergeQueue: This merge request was already merged

This pull request was merged directly.

@steveny91 steveny91 added this pull request to the merge queue May 22, 2025
Merged via the queue into master with commit 9e8d782 May 22, 2025
47 of 49 checks passed
@steveny91 steveny91 deleted the yunkim/crewai-tile branch May 22, 2025 17:50
github-actions bot pushed a commit that referenced this pull request May 22, 2025
@github-merge-queue
[MLOB] add crewai integration tile (#20053)
5881eca 
* Add crewai integration tile

* address suggestions

* Add python disclaimer

* add source type ID

* Display on public website 9e8d782
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cswatt cswatt cswatt approved these changes

@steveny91 steveny91 steveny91 approved these changes

@apiazza-dd apiazza-dd apiazza-dd left review comments

Assignees
No one assigned
Labels
agent/approved dev/testing dev/tooling docs/approved documentation ecosystems/review-requested editorial review Waiting on a more in-depth review from a docs team editor integration/crewai mergequeue-status: done product/review-requested qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations team/documentation team/saas-integrations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Yun-Kim @cswatt @steveny91 @apiazza-dd