Skip to content

Commit 8f9ecff

Browse files
authored
Cleanup ci refresh rollout (#1193)
This commit updates the `docker-setup.sh` and `.travis.yml` files to follow the current patterns in the CI refresh rolled out to accomidate the new release cadence. The specific changes are explicitly passing a parameter to the docker pull function in the setup script and ensuring that the travis config segments steps correctly.
1 parent b558150 commit 8f9ecff

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

.ci/docker-setup.sh

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ set -e
66

77
pull_docker_snapshot() {
88
project="${1?project name required}"
9+
stack_version_alias="${2?stack version alias required}"
910
local docker_image="docker.elastic.co/${project}/${project}${DISTRIBUTION_SUFFIX}:${ELASTIC_STACK_VERSION}"
1011
echo "Pulling $docker_image"
1112
if docker pull "$docker_image" ; then
@@ -30,8 +31,8 @@ if [ -z "${ELASTIC_STACK_VERSION}" ]; then
3031
exit 1
3132
fi
3233

33-
# save the original arg if needed
34-
ELASTIC_STACK_VERSION_ARG="$ELASTIC_STACK_VERSION"
34+
# The ELASTIC_STACK_VERSION may be an alias, save the original before translating it
35+
ELASTIC_STACK_VERSION_ALIAS="$ELASTIC_STACK_VERSION"
3536

3637
echo "Fetching versions from $VERSION_URL"
3738
VERSIONS=$(curl -s $VERSION_URL)
@@ -64,9 +65,9 @@ export DISTRIBUTION_SUFFIX
6465
echo "Testing against version: $ELASTIC_STACK_VERSION (distribution: ${DISTRIBUTION:-"default"})"
6566

6667
if [[ "$ELASTIC_STACK_VERSION" = *"-SNAPSHOT" ]]; then
67-
pull_docker_snapshot "logstash"
68+
pull_docker_snapshot "logstash" $ELASTIC_STACK_VERSION_ALIAS
6869
if [ "$INTEGRATION" == "true" ]; then
69-
pull_docker_snapshot "elasticsearch"
70+
pull_docker_snapshot "elasticsearch" $ELASTIC_STACK_VERSION_ALIAS
7071
fi
7172
fi
7273

.travis.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,13 @@ import:
44
jobs:
55
include:
66
- stage: "Integration Tests"
7-
- env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=7.current
7+
env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=7.current
88
- env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=8.previous
99
- env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=8.current
1010
- env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=8.next
1111
- env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=8.future
12-
# - env: INTEGRATION=true SNAPSHOT=true LOG_LEVEL=info ELASTIC_STACK_VERSION=main
1312
- stage: "Secure Integration Tests"
14-
- env: SECURE_INTEGRATION=true INTEGRATION=true LOG_LEVEL=info ELASTIC_STACK_VERSION=8.current SNAPSHOT=true
13+
env: SECURE_INTEGRATION=true INTEGRATION=true LOG_LEVEL=info ELASTIC_STACK_VERSION=8.current SNAPSHOT=true
1514
- env: SECURE_INTEGRATION=true INTEGRATION=true LOG_LEVEL=info ELASTIC_STACK_VERSION=7.current
1615
- env: SECURE_INTEGRATION=true INTEGRATION=true LOG_LEVEL=info ELASTIC_STACK_VERSION=7.current ES_SSL_KEY_INVALID=true
1716
- env: SECURE_INTEGRATION=true INTEGRATION=true LOG_LEVEL=info ELASTIC_STACK_VERSION=7.current ES_SSL_SUPPORTED_PROTOCOLS=TLSv1.3

0 commit comments

Comments
 (0)