kmlabz/sentry-onpremise
kmlabz/sentry-onpremise
Archived
2
0
Fork 0
Code Pull Requests Releases Activity

fix(install): Remove unnecessary infinite loop from snuba bootst… (#354)

Browse Source 
With getsentry/snuba#709 merged, we no longer need to randomly try forever creating Kafka topics, which were a prerequisite for creating Clickhouse tables. This patch also removes the preceding (and obsolete) `docker-compose up` statement which was used to "speed up" the `snuba bootstrap` command in the hopes to fix it.
This commit is contained in:
Burak Yigit Kaya 2020-01-21 22:56:45 +03:00 committed by GitHub
parent b8405fca76
commit 1292a125b8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
install.sh
View File

@ -104,13 +104,10 @@ echo ""
echo "Docker images built."
echo "Bootstrapping Snuba..."
$dc up -d kafka redis clickhouse
until $($dcr clickhouse clickhouse-client -h clickhouse --query="SHOW TABLES;" | grep -q sentry_local); do
# `bootstrap` is for fresh installs, and `migrate` is for existing installs
# Running them both for both cases is harmless so we blindly run them
$dcr snuba-api bootstrap --force || true;
$dcr snuba-api migrate || true;
done;
$dcr snuba-api bootstrap --force
$dcr snuba-api migrate
echo ""
# Very naively check whether there's an existing sentry-postgres volume and the PG version in it