Skip to content

Commit 142ad7b

Browse files
committed
do it from the code
1 parent a5927f5 commit 142ad7b

4 files changed

Lines changed: 30 additions & 14 deletions

File tree

airbyte_cdk/manifest_server/Dockerfile

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,12 @@ RUN --mount=type=cache,target=$POETRY_CACHE_DIR \
3434
RUN groupadd --gid 1000 airbyte && \
3535
useradd --uid 1000 --gid airbyte --shell /bin/bash --create-home airbyte
3636

37-
# Change ownership and make entrypoint executable
38-
RUN chown -R airbyte:airbyte /app && \
39-
chmod +x /app/airbyte_cdk/manifest_server/entrypoint.sh
37+
# Change ownership
38+
RUN chown -R airbyte:airbyte /app
4039

4140
# Run app as non-root user
4241
USER airbyte:airbyte
4342

4443
EXPOSE 8080
4544

46-
ENTRYPOINT ["/app/airbyte_cdk/manifest_server/entrypoint.sh"]
4745
CMD ["uvicorn", "airbyte_cdk.manifest_server.app:app", "--host", "0.0.0.0", "--port", "8080"]

airbyte_cdk/manifest_server/README.md

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,3 +154,25 @@ docker run -p 8080:8080 manifest-server
154154
```
155155

156156
Note: The container runs on port 8080 by default.
157+
158+
## Datadog APM
159+
160+
The manifest server supports Datadog APM tracing for monitoring and observability:
161+
162+
### Configuration
163+
164+
To enable Datadog tracing, set the environment variable:
165+
166+
```bash
167+
export DD_ENABLED=true
168+
```
169+
170+
This requires the `ddtrace` dependency, which is included in the `manifest-server` extra. For additional configuration options via environment variables, see [ddtrace configuration](https://ddtrace.readthedocs.io/en/stable/configuration.html).
171+
172+
### Usage
173+
174+
```bash
175+
# Run with Datadog tracing enabled
176+
DD_ENABLED=true manifest-server start
177+
```
178+

airbyte_cdk/manifest_server/app.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
1+
import os
2+
3+
if os.getenv("DD_ENABLED") == "true":
4+
# Auto-instrumentation should be imported as early as possible.
5+
import ddtrace.auto
6+
17
from fastapi import FastAPI
28

39
from .routers import capabilities, health, manifest

airbyte_cdk/manifest_server/entrypoint.sh

Lines changed: 0 additions & 10 deletions
This file was deleted.

0 commit comments

Comments
 (0)