Skip to content

Commit bc03bde

Browse files
committed
Code review refactor
Rename INCLUDE_SHA to INCLUDE_COMMIT_ID in qualified-version script. Confine use of this argument to individual invocations instead at top level in scripts.
1 parent 676bf86 commit bc03bde

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

Diff for: .buildkite/scripts/common/qualified-version.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ export QUALIFIED_VERSION="$(
1818
printf '%s' "${VERSION_QUALIFIER:+-${VERSION_QUALIFIER}}"
1919
2020
# Include git SHA if requested
21-
if [[ -n "${INCLUDE_SHA:+x}" ]]; then
21+
if [[ -n "${INCLUDE_COMMIT_ID:+x}" ]]; then
2222
printf '%s' "-$(git rev-parse --short HEAD)"
2323
fi
2424

Diff for: .buildkite/scripts/dra/build-and-push-observability-sre.sh

+1-3
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,10 @@ echo "Pushing ObservabilitySRE container to Docker repository"
1717
docker_login
1818

1919
# Get qualified version without SHA (this is what the gradle task will produce)
20-
export INCLUDE_SHA=""
2120
QUALIFIED_VERSION="$(.buildkite/scripts/common/qualified-version.sh)"
2221

2322
# Set environment variable to include SHA and get version with SHA
24-
export INCLUDE_SHA=1
25-
QUALIFIED_VERSION_WITH_SHA="$(.buildkite/scripts/common/qualified-version.sh)"
23+
QUALIFIED_VERSION_WITH_SHA="$(INCLUDE_COMMIT_ID=1 .buildkite/scripts/common/qualified-version.sh)"
2624

2725
REGISTRY_PATH=docker.elastic.co/logstash/logstash-observability-sre
2826

Diff for: .buildkite/scripts/dra/multi-architecture-observability-sre.sh

+2-4
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,8 @@ source .buildkite/scripts/dra/docker-env-setup.sh
77

88
docker_login
99

10-
# Set INCLUDE_SHA to include git SHA in version
11-
export INCLUDE_SHA=1
12-
QUALIFIED_VERSION="$(.buildkite/scripts/common/qualified-version.sh)"
10+
# Set INCLUDE_COMMIT_ID to include git SHA in version
11+
QUALIFIED_VERSION="$(INCLUDE_COMMIT_ID=1 .buildkite/scripts/common/qualified-version.sh)"
1312
REGISTRY_PATH=docker.elastic.co/logstash/logstash-observability-sre
1413

1514
# Architecture-specific tags (created by the build steps)
@@ -35,7 +34,6 @@ echo "Pushing manifest: ${REGISTRY_PATH}:${VERSION_MANIFEST_TAG}"
3534
docker manifest push ${REGISTRY_PATH}:${VERSION_MANIFEST_TAG}
3635

3736
# Also create version without SHA for effective "latest" tag
38-
export INCLUDE_SHA=""
3937
BASE_VERSION="$(.buildkite/scripts/common/qualified-version.sh)"
4038
echo "Creating manifest list for: ${REGISTRY_PATH}:${BASE_VERSION}"
4139
docker manifest create ${REGISTRY_PATH}:${BASE_VERSION} \

0 commit comments

Comments
 (0)