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.

Sign up for GitHub

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

Jump to bottom

Skip building HotROD for all platforms for pull requests #5765

Merged
merged 8 commits into from
Jul 19, 2024
Merged

Skip building HotROD for all platforms for pull requests #5765

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d81ab88
Enable build_flags for hotrod workflow
Manoramsharma Jul 19, 2024
9e7f0ef
Merge remote-tracking branch 'upstream/main' into hotrod-image-build
Manoramsharma Jul 19, 2024
dc68116
Need local flag only when the test runs on a PR
Manoramsharma Jul 19, 2024
d847dff
Publish Image with LOCAL_FLAG after successful tests
Manoramsharma Jul 19, 2024
6f855c4
fix
yurishkuro Jul 19, 2024
e1fb84c
fix
yurishkuro Jul 19, 2024
a8b1d1b
fix
yurishkuro Jul 19, 2024
7d6539f
fix
yurishkuro Jul 19, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion .github/workflows/ci-docker-hotrod.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,19 @@ jobs:

- uses: docker/setup-qemu-action@5927c834f5b4fdf503fca6f4c7eccda82949e1ee # v3.1.0

- name: Define BUILD_FLAGS var if running on a Pull Request
run: |
case ${GITHUB_EVENT_NAME} in
pull_request)
echo "BUILD_FLAGS=-l -D -p linux/amd64" >> ${GITHUB_ENV}
;;
*)
echo "BUILD_FLAGS=" >> ${GITHUB_ENV}
;;
esac

- name: Build, test, and publish hotrod image
run: bash scripts/hotrod-integration-test.sh
run: bash scripts/hotrod-integration-test.sh ${{ env.BUILD_FLAGS }}
env:
DOCKERHUB_TOKEN: ${{ secrets.DOCKERHUB_TOKEN }}
QUAY_TOKEN: ${{ secrets.QUAY_TOKEN }}
Expand Down
31 changes: 29 additions & 2 deletions scripts/hotrod-integration-test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,33 @@

set -euxf -o pipefail

print_help() {
echo "Usage: $0 [-l] [-p platforms] [-h]"
echo "-h: Print help"
echo "-l: Enable local-only mode that only pushes images to local registry"
echo "-p: Comma-separated list of platforms to build for (default: all supported)"
exit 1
}

docker_compose_file="./examples/hotrod/docker-compose.yml"
platforms="linux/amd64,linux/s390x,linux/ppc64le,linux/arm64"
BINARY='example-hotrod'
LOCAL_FLAG=''

while getopts "lp:h" opt; do
case "${opt}" in
l)
# in the local-only mode the images will only be pushed to local registry
LOCAL_FLAG='-l'
;;
p)
platforms=${OPTARG}
;;
?)
print_help
;;
esac
done

teardown() {
echo "Tearing down..."
Expand All @@ -21,13 +46,15 @@ make create-baseimg

# Loop through each platform (separated by commas)
for platform in $(echo "$platforms" | tr ',' ' '); do
# Extract the operating system from the platform string
os=${platform%%/*} #remove everything after the last slash
# Extract the architecture from the platform string
arch=${platform##*/} # Remove everything before the last slash
make "build-all-in-one" GOOS=linux GOARCH="${arch}"
make "build-all-in-one" GOOS=${os} GOARCH="${arch}"
done

# Build image locally (-l) for integration test
bash scripts/build-upload-a-docker-image.sh -l -c example-hotrod -d examples/hotrod -p "${platforms}"
bash scripts/build-upload-a-docker-image.sh ${LOCAL_FLAG} -c ${BINARY} -d examples/hotrod -p "${platforms}"
Manoramsharma marked this conversation as resolved.
Show resolved Hide resolved
bash scripts/build-upload-a-docker-image.sh -l -b -c all-in-one -d cmd/all-in-one -p "${platforms}" -t release

JAEGER_VERSION=$GITHUB_SHA REGISTRY="localhost:5000/" docker compose -f "$docker_compose_file" up -d
Expand Down
Loading