Merge pull request #46429 from cpuguy83/swap_to_otel_action

CI: Switch to use tracing action
This commit is contained in:
CrazyMax
2023-09-08 08:22:20 +02:00
committed by GitHub
2 changed files with 9 additions and 9 deletions

View File

@@ -25,12 +25,3 @@ runs:
- run: |
docker info
shell: bash
# TODO: Remove this step once the separate action is merged
# Github doesn't let you add a composite action and use it in the same PR (apparently)
# Ref: https://github.com/moby/moby/actions/runs/5581571995/jobs/10199909170?pr=45652#step:9:1
- run: |
set -e
docker run -d --net=host --name jaeger -e COLLECTOR_OTLP_ENABLED=true jaegertracing/all-in-one:1.46
docker0_ip="$(ip -f inet addr show docker0 | grep -Po 'inet \K[\d.]+')"
echo "OTEL_EXPORTER_OTLP_ENDPOINT=http://${docker0_ip}:4318" >> "${GITHUB_ENV}"
shell: bash

View File

@@ -203,6 +203,9 @@ jobs:
-
name: Set up runner
uses: ./.github/actions/setup-runner
-
name: Set up tracing
uses: ./.github/actions/setup-tracing
-
name: Set up Docker Buildx
uses: docker/setup-buildx-action@v2
@@ -293,6 +296,9 @@ jobs:
-
name: Set up runner
uses: ./.github/actions/setup-runner
-
name: Set up tracing
uses: ./.github/actions/setup-tracing
-
name: Prepare
run: |
@@ -437,6 +443,9 @@ jobs:
-
name: Set up runner
uses: ./.github/actions/setup-runner
-
name: Set up tracing
uses: ./.github/actions/setup-tracing
-
name: Set up Docker Buildx
uses: docker/setup-buildx-action@v2