Skip to content

Add PHP support #1436

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
Open
mikkeschiren opened this issue Aug 9, 2024 · 13 comments
Open

Add PHP support #1436

mikkeschiren opened this issue Aug 9, 2024 · 13 comments
Labels
php

Comments

@mikkeschiren
Copy link

Are there any plans to add support for PHP?
@RonFed
Copy link
Collaborator

RonFed commented Aug 12, 2024

Hey, it is in our roadmap. Currently with low priority

@mikkeschiren
Copy link
Author

As PHP is the most common language for web applications it would make sense to higher the priority :)

@github-actions GitHub Actions
Copy link
Contributor

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label Oct 14, 2024
@mikkeschiren
Copy link
Author

mikkeschiren commented Oct 14, 2024 •
edited
Loading

I still think that this should be a prio :)

@RonFed RonFed removed the stale label Oct 15, 2024
@github-actions GitHub Actions
Copy link
Contributor

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label Dec 16, 2024
@mikkeschiren
Copy link
Author

Still think the same :)

@Fgruntjes
Copy link

Same here, "PHP is used by 75.2% of all the websites whose server-side programming language we know." (source: https://w3techs.com/technologies/details/pl-php)

@github-actions github-actions bot removed the stale label Dec 23, 2024
@gourav-rcrm
Copy link

Is there any release schedule for PHP?

@github-actions GitHub Actions
Copy link
Contributor

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label Apr 14, 2025
@BenElferink
Copy link
Contributor

Working on it, 1st PR is open: #2727

@BenElferink BenElferink removed the stale label Apr 17, 2025
@BenElferink
Copy link
Contributor

Following up with the 2nd PR: #2732
OTel injection succeeded, telemetry data not ready yet.

@BenElferink
Copy link
Contributor

Updated the same PR: #2732
Telemetry data is available 🚀

@BenElferink BenElferink mentioned this issue Apr 21, 2025
Merged
1 task
BenElferink added a commit that referenced this issue Apr 22, 2025
@BenElferink @Copilot
feat: auto-instrument PHP versions 8+ (#2732)
ab61892 
## Description

Resolves issue #1436

This PR injects and enables the [OpenTelemetry PHP
extension](https://github.com/open-telemetry/opentelemetry-php-instrumentation)
into PHP-detected pods.

- The `opentelemetry.so` file is compiled for each PHP version (see [PHP
agents here](https://github.com/odigos-io/opentelemetry-php))
- Each PHP version is assigned it's own agent

###  Proof of injection (from `kubectl exec` into the PHP-detected pod):

<img width="1173" alt="Screenshot 2025-04-17 at 9 38 32"
src="https://github.com/user-attachments/assets/cdf4b274-7718-4fdd-b47b-1ff76298b273"
/>

### Proof of trace:

<img width="1718" alt="Screenshot 2025-04-18 at 22 30 43"
src="https://github.com/user-attachments/assets/faabbbac-34fd-4aad-9dc7-792e449d3abd"
/>
<img width="1718" alt="Screenshot 2025-04-20 at 23 41 59"
src="https://github.com/user-attachments/assets/0e5cec18-525d-48d4-b920-4e07d7b4d4f9"
/>
<img width="1289" alt="Screenshot 2025-04-21 at 17 55 00"
src="https://github.com/user-attachments/assets/623efca1-42fb-4efc-8056-d3625954d7d6"
/>

## How Has This Been Tested?

- [x] Manual Testing

---

TODO
- Write demo PHP app, and include it in E2E tests
- Update docs

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@BenElferink
Copy link
Contributor

All merged and released 🎉
Please install Odigos v1.0.175 or higher to instrument PHP applications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
php
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mikkeschiren @Fgruntjes @BenElferink @RonFed @gourav-rcrm