Skip to content
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

Add opentelementry extensions #98

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add opentelementry extensions #98

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
20f7d1c
Add opentelementry extensions
eduard-melnytskyi Aug 13, 2024
ade81e3
Add opentelementry extensions to the all php images
eduard-melnytskyi Aug 16, 2024
dc17bdd
Add ini files for opentelementry, grcp and protobuf extensions
eduard-melnytskyi Aug 18, 2024
8dad686
Merge master.
eduard-melnytskyi Dec 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions alpine/3.18/8.2/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,9 @@ ARG PHP_EXTENSIONS="\

ARG PHP_PECL_EXTENSIONS="\
apcu \
opentelemetry \
Copy link

Choose a reason for hiding this comment

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

please move those to ADDITIONAL_PHP_PECL_EXTENSIONS section

Also we need to build images and test them locally.

Afterwards we need to get an approve from @alexanderM91 and after merge build will be triggered for docker hub.

Also please do the same for 8.1 version

Copy link
Author

Choose a reason for hiding this comment

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

updated Dockerfile for all PHP version

protobuf \
grpc \
redis"

ARG ADDITIONAL_PHP_PECL_EXTENSIONS="\
Expand Down
Loading