artin/sourcegraph
1
0
Fork 0
mirror of https://github.com/sourcegraph/sourcegraph.git synced 2026-02-06 17:31:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

sg: correctly download jaeger (#26984)

Browse Source 
Currently jaeger just doesn't start in our dev environment since it
used "version" as an envvar. Additionally, we didn't notice this was
broken since "-e" bash option was not set. This is now set. We
should consider it for all shell commands in sg.

The log level is also lowered for Jaeger, since it is quiet spammy.
This commit is contained in:
Keegan Carruthers-Smith 2021-11-03 12:15:18 +02:00 committed by GitHub
parent 23c699135c
commit 9bc13b83d6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
sg.config.yaml
View File

@ -551,11 +551,12 @@ commands:
cmd: |
.bin/jaeger-all-in-one-${JAEGER_VERSION}-$(go env GOOS)-$(go env GOARCH) --log-level ${JAEGER_LOG_LEVEL}
install: |
set -e
mkdir -p "${JAEGER_DISK}"
suffix="${JAEGER_VERSION}-$(go env GOOS)-$(go env GOARCH)"
target="$PWD/.bin/jaeger-all-in-one-${suffix}"
url="https://github.com/jaegertracing/jaeger/releases/download/v${version}/jaeger-${suffix}.tar.gz"
url="https://github.com/jaegertracing/jaeger/releases/download/v${JAEGER_VERSION}/jaeger-${suffix}.tar.gz"
if [ ! -f "${target}" ]; then
echo "downloading ${url}" 1>&2
@ -567,7 +568,7 @@ commands:
env:
JAEGER_VERSION: 1.18.1
JAEGER_DISK: $HOME/.sourcegraph-dev/data/jaeger
JAEGER_LOG_LEVEL: info
JAEGER_LOG_LEVEL: error
QUERY_BASE_PATH: /-/debug/jaeger
grafana: