From debb21aaf577fd2b50c96a8ee34d42ea61a12289 Mon Sep 17 00:00:00 2001 From: Olaf Alders Date: Sun, 1 Sep 2024 23:52:16 -0400 Subject: [PATCH] Don't wait for ES until we absolutely have to --- .circleci/config.yml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/.circleci/config.yml b/.circleci/config.yml index e4d401408..18c817dfc 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -40,14 +40,6 @@ jobs: ./bin/metacpan-docker init docker-compose --verbose up -d api_test name: compose up - # Since we're running docker-compose -d, we don't actually know if - # Elasticsearch is available at the time this build step begins. We - # probably need to wait for it here, so we'll add our own check. - - run: - command: | - pushd metacpan-docker - ./src/metacpan-api/wait-for-es.sh http://localhost:9200 elasticsearch_test -- - name: wait for ES # We are relying on environment variables from the host to be available when # we publish the report, so we publish from the host rather than trying # to propagate env variables to the container. @@ -56,6 +48,14 @@ jobs: pushd metacpan-docker docker-compose exec -T api_test cpm install -g Devel::Cover name: install Devel::Cover + # Since we're running docker-compose -d, we don't actually know if + # Elasticsearch is available at the time this build step begins. We + # probably need to wait for it here, so we'll add our own check. + - run: + command: | + pushd metacpan-docker + ./src/metacpan-api/wait-for-es.sh http://localhost:9200 elasticsearch_test + name: wait for ES - run: command: | pushd metacpan-docker