-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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.
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
Run jaeger-es-index-cleaner and jaeger-es-rollover locally #5714
Changes from 14 commits
8420368
808d331
ec5ae1b
8f5f4a2
45730e3
289b0ab
c51a745
ebd8dbe
5b9dda7
57dd378
3f6940f
aeb00f7
913f952
387dd49
c918928
b475796
46c7928
85d373c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -96,14 +96,32 @@ teardown_storage() { | |||||
docker compose -f "${compose_file}" down | ||||||
} | ||||||
|
||||||
build_local_img(){ | ||||||
make "build-binaries-linux" | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you don't need to build ALL binaries, only the 2 specific ones There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think you want
|
||||||
make create-baseimg | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
#build es-index-cleaner | ||||||
docker build \ | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This worked for me: Build with a fake
Run as
|
||||||
--build-arg base_image=localhost:5000/baseimg_alpine:latest \ | ||||||
--file cmd/es-index-cleaner/Dockerfile \ | ||||||
-t jaegertracing/jaeger-es-index-cleaner:local-test \ | ||||||
cmd/es-index-cleaner | ||||||
|
||||||
#build es-rollover | ||||||
docker build \ | ||||||
--build-arg base_image=localhost:5000/baseimg_alpine:latest \ | ||||||
--file cmd/es-rollover/Dockerfile \ | ||||||
-t jaegertracing/jaeger-es-rollover:local-test \ | ||||||
cmd/es-rollover | ||||||
} | ||||||
|
||||||
main() { | ||||||
check_arg "$@" | ||||||
local distro=$1 | ||||||
local es_version=$2 | ||||||
local j_version=$2 | ||||||
|
||||||
bring_up_storage "${distro}" "${es_version}" | ||||||
|
||||||
build_local_img | ||||||
if [[ "${j_version}" == "v2" ]]; then | ||||||
STORAGE=${distro} SPAN_STORAGE_TYPE=${distro} make jaeger-v2-storage-integration-test | ||||||
else | ||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.