opentelemetry-demo icon indicating copy to clipboard operation
opentelemetry-demo copied to clipboard

Create an 'admin service' in PHP

Open austinlparker opened this issue 2 years ago • 6 comments

Objective

Create an 'admin service' written in PHP for the web store.

The following features should be supported:

  • Authentication for 'admin users'
  • UI for CRUD operations on items in the product catalog.
  • API for CRUD operations on items in the product catalog.

austinlparker avatar May 23 '22 20:05 austinlparker

Linking to the issue in the php repo: https://github.com/open-telemetry/opentelemetry-php/issues/658

@bobstrecansky should I assign the current issue owner to this as well?

cartersocha avatar May 27 '22 05:05 cartersocha

Sounds good!

bobstrecansky avatar May 27 '22 08:05 bobstrecansky

As discussed in the #otel-community-demo slack channel, we can also use Apache WebServer to host 'Admin Service' in PHP through mod_php. Apache instrumentation is already available to generate spans and traces.

DebajitDas avatar Jun 02 '22 07:06 DebajitDas

@DebajitDas Thank you. However we most likely would prefer not to use Apache, as that's a bit outdated, or in fact any web-sever (Nginx,, Caddy, etc.) as a runtime, but a modern faster runtime as Swoole or Roadrunner.

tidal avatar Jun 02 '22 09:06 tidal

Hey @tidal, @bobstrecansky, is there an update on this? We merged the rust service today, so PHP is the remaining outstanding language not present.

cartersocha avatar Jul 13 '22 19:07 cartersocha

As discussed with @cartersocha I will create a PR with basic service setup as a draft. This service will feature roadrunner with OpenTelemetry middleware as an app server. API Platform will make the implementation of auth and API easy. It also allows to generate an admin ui from the API.

tidal avatar Jul 25 '22 20:07 tidal

@austinlparker can we have this ticket closed? I believe we do not need an admin service anymore, as PHP was covered in the Quote Service.

julianocosta89 avatar Oct 06 '22 20:10 julianocosta89

Closing this!

cartersocha avatar Oct 06 '22 20:10 cartersocha