Skip to content

Commit 9873ce7

Browse files
Review feedback
1 parent 3e52f90 commit 9873ce7

File tree

3 files changed

+8
-36
lines changed

3 files changed

+8
-36
lines changed

.buildkite/pipeline.package.mbp.yml

+4-32
Original file line numberDiff line numberDiff line change
@@ -104,22 +104,10 @@ steps:
104104
depends_on:
105105
- step: "package-x86-64-snapshot"
106106
allow_failure: false
107-
- step: "package-arm-snapshot"
108-
allow_failure: false
109-
110-
- label: "DRA FIPS snapshot"
111-
if: "${FILE_EXISTS_IN_REPO} && build.env('VERSION_QUALIFIER') == null"
112-
key: "dra-fips snapshot"
113-
command: ".buildkite/scripts/dra_release.sh snapshot"
114-
env:
115-
FIPS: "true"
116-
agents:
117-
provider: "gcp"
118-
image: "${IMAGE_UBUNTU_X86_64}"
119-
machineType: "c2-standard-16"
120-
depends_on:
121107
- step: "package-fips-x86-64-snapshot"
122108
allow_failure: false
109+
- step: "package-arm-snapshot"
110+
allow_failure: false
123111
- step: "package-fips-arm-snapshot"
124112
allow_failure: false
125113

@@ -138,25 +126,9 @@ steps:
138126
depends_on:
139127
- step: "package-x86-64-staging"
140128
allow_failure: false
141-
- step: "package-arm-staging"
142-
allow_failure: false
143-
144-
- label: "DRA FIPS release staging"
145-
# we don't usually build staging from the main branch, but we exceptionally allow it for prereleases
146-
# details in https://github.com/elastic/ingest-dev/issues/4855
147-
if: "${FILE_EXISTS_IN_REPO} == true && (build.env('BUILDKITE_BRANCH') != 'main' || build.env('VERSION_QUALIFIER') != null)"
148-
key: "dra-fips-staging"
149-
command: |
150-
source .buildkite/scripts/version_qualifier.sh
151-
.buildkite/scripts/dra_release.sh staging
152-
env:
153-
FIPS: "true"
154-
agents:
155-
provider: "gcp"
156-
image: "${IMAGE_UBUNTU_X86_64}"
157-
machineType: "c2-standard-16"
158-
depends_on:
159129
- step: "package-fips-x86-64-staging"
160130
allow_failure: false
131+
- step: "package-arm-staging"
132+
allow_failure: false
161133
- step: "package-fips-arm-staging"
162134
allow_failure: false

.buildkite/scripts/dra_release.sh

-4
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,6 @@ if [[ ${TYPE} == "snapshot" ]]; then
4141
VERSION="${VERSION}-SNAPSHOT"
4242
fi
4343

44-
if [[ ${FIPS} == "true" ]]; then
45-
export FIPS=true
46-
fi
47-
4844
mkdir -p ${BASE_DIR}/reports
4945
./dev-tools/dependencies-report --csv ${BASE_DIR}/reports/dependencies-${VERSION}.csv
5046
cd ${BASE_DIR}/reports && shasum -a 512 dependencies-${VERSION}.csv > dependencies-${VERSION}.csv.sha512

.buildkite/scripts/package.sh

+4
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,10 @@ if [[ ${PLATFORM_TYPE} == "arm" || ${PLATFORM_TYPE} == "aarch64" ]]; then
2020
PACKAGES="docker"
2121
fi
2222

23+
if [[ ${FIPS} == "true" ]]; then
24+
export FIPS=true
25+
fi
26+
2327
add_bin_path
2428
with_go
2529
with_mage

0 commit comments

Comments
 (0)