Skip to content
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

Add healthcheck for shibboleth-idp in idp-fixture #100369

Merged
merged 7 commits into from
Oct 6, 2023
1 change: 1 addition & 0 deletions x-pack/test/idp-fixture/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ apply plugin: 'elasticsearch.test.fixtures'

dockerCompose {
composeAdditionalArgs = ['--compatibility']
upAdditionalArgs = ["--wait"]
}

tasks.named("preProcessFixture").configure {
Expand Down
8 changes: 7 additions & 1 deletion x-pack/test/idp-fixture/docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
version: '3.7'
version: "3.7"
services:
openldap:
command: --copy-service --loglevel debug
Expand Down Expand Up @@ -37,6 +37,12 @@ services:
links:
- openldap:openldap
restart: always #ensure ephemeral port mappings are properly updated
healthcheck:
test: curl -f -s --http0.9 http://localhost:4443 --connect-timeout 10 --max-time 10 --output - > /dev/null
interval: 5s
timeout: 20s
retries: 60
start_period: 10s

oidc-provider:
build:
Expand Down
17 changes: 16 additions & 1 deletion x-pack/test/idp-fixture/idp/bin/run-jetty.sh
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,19 @@ fi
export JETTY_ARGS="jetty.sslContext.keyStorePassword=$JETTY_BROWSER_SSL_KEYSTORE_PASSWORD jetty.backchannel.sslContext.keyStorePassword=$JETTY_BACKCHANNEL_SSL_KEYSTORE_PASSWORD"
sed -i "s/^-Xmx.*$/-Xmx$JETTY_MAX_HEAP/g" /opt/shib-jetty-base/start.ini

exec /opt/jetty-home/bin/jetty.sh run
# For some reason, this container always immediately (in less than 1 second) exits with code 0 when starting for the first time
# Even with a health check, docker-compose will immediately report the container as unhealthy when using --wait instead of waiting for it to become healthy
# So, let's just start it a second time if it exits quickly
set +e
start_time=$(date +%s)
/opt/jetty-home/bin/jetty.sh run
exit_code=$?
end_time=$(date +%s)

duration=$((end_time - start_time))
if [ $duration -lt 5 ]; then
/opt/jetty-home/bin/jetty.sh run
exit_code=$?
fi

exit $exit_code
Loading