Skip to content

Commit c9cedf3

Browse files
committed
Use a patched ogccite/ets-ogcapi-features10 Docker image with additional fixes
Use a patched version of `ogccite/ets-ogcapi-features10:1.8-SNAPSHOT-teamengine-5.4.1`. Currently, we require the following pull request to be applied: [Fix global crs codes lookup in discovery collections crs tests](opengeospatial/ets-ogcapi-features10#255) to the test suite. This may happen again, hence we're introducing a build step to generate the required `ogccite/ets-ogcapi-features10` image out of the `geoserver/ets-ogcapi-features10` fork, using a branch named `geoserver/integration`. The procedure to maintain that branch is to rebase on top of the upstream master branch as required, and apply the additional branches with `git merge --no-ff`.
1 parent 564a275 commit c9cedf3

File tree

5 files changed

+46
-2
lines changed

5 files changed

+46
-2
lines changed

Diff for: .github/workflows/cite.yml

+5-1
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,11 @@ jobs:
7171

7272
- name: Build ogccite/geoserver:${{ matrix.suite }} docker image
7373
working-directory: build/cite
74-
run: make build suite=${{ matrix.suite }}
74+
run: make build-geoserver-image suite=${{ matrix.suite }}
75+
76+
- name: Build the ETS docker image
77+
working-directory: build/cite
78+
run: make build-ets-image suite=${{ matrix.suite }}
7579

7680
- name: Print services and images used for the ${{ matrix.suite }} test run
7781
# in case there's a jq syntax error

Diff for: build/cite/Makefile

+25
Original file line numberDiff line numberDiff line change
@@ -32,17 +32,42 @@ war: ## Build the geoserver.war file to use for testing and place it in ./g
3232
@$(MAKE) build-war
3333
@$(MAKE) copy-war
3434

35+
# calls build-ets-image and build-geoserver-image
36+
# the CI job may call them independently for the sake of using separate job steps
3537
.PHONY: build
3638
build: ## suite=<suite> Build the GeoServer Docker Image for the Environment.
3739
ifndef suite
3840
@echo "Error: suite is not specified. Please specify the suite to build the GeoServer Docker image for."
3941
@$(MAKE) help
4042
@exit 1
43+
endif
44+
@$(MAKE) build-ets-image
45+
@$(MAKE) build-geoserver-image
46+
47+
# preparing the geoserver image is performed with docker-compose build including $suite/compose-override.yml
48+
.PHONY: build-geoserver-image
49+
build-geoserver-image:
50+
ifndef suite
51+
@echo "Error: suite is not specified. Please specify the suite to build the GeoServer Docker image for."
52+
@exit 1
4153
endif
4254
@echo Building the GeoServer Docker Image for the $(suite) test suite
4355
@docker compose -f compose.yml -f ./$(suite)/compose.override.yml \
4456
build --no-cache --build-arg SOURCE_FILE=${war_url} geoserver
4557

58+
# If there's a $suite/build-ets.sh file calls it, otherwise exists silently
59+
.PHONY: build-ets-image
60+
build-ets-image:
61+
ifndef suite
62+
@echo "Error: suite is not specified. Please specify the suite to build the GeoServer Docker image for."
63+
@exit 1
64+
endif
65+
@if [ -f "$(suite)/build-ets.sh" ]; then \
66+
(cd $(suite) && ./build-ets.sh); \
67+
else \
68+
echo "$(suite) has no special script to build the ETS docker image."; \
69+
fi
70+
4671
.PHONY: test
4772
test: ## suite=<suite> Run the Test Suite with teamengine and GeoServer on docker compose.
4873
ifndef suite

Diff for: build/cite/ogcapi-features10/.gitignore

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
ets-ogcapi-features10

Diff for: build/cite/ogcapi-features10/build-ets.sh

+13
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
#!/bin/sh
2+
3+
# build the ets-ogcapi-features10 Docker image from the
4+
# (at the time of writing, ogccite/ets-ogcapi-features10:1.8-SNAPSHOT-teamengine-5.4.1)
5+
# from the geoserver's fork `geoserver/integration` branch, which applies
6+
# patches to the ets that have not yet being released upstream
7+
8+
echo "Building the ogccite/ets-ogcapi-features10 docker image from the geoserver/integration branch"
9+
rm -rf ets-ogcapi-features10
10+
git clone https://github.com/geoserver/ets-ogcapi-features10.git
11+
cd ets-ogcapi-features10
12+
git checkout geoserver/integration && mvn clean install -Pdocker -DskipTests -ntp
13+
cd ..

Diff for: build/cite/ogcapi-features10/compose.override.yml

+2-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@ services:
1212
# see https://github.com/opengeospatial/teamengine-docker
1313
# there's ogccite/teamengine-production and ogccite/teamengine-beta
1414
# that we could use to migrate the other OWS tests to newer teamengine images
15-
image: ogccite/ets-ogcapi-features10:1.7.1-teamengine-5.4.1
15+
# image: ogccite/ets-ogcapi-features10:1.7.1-teamengine-5.4.1
16+
image: ogccite/ets-ogcapi-features10:1.8-SNAPSHOT-teamengine-5.4.1
1617
healthcheck:
1718
test: "curl -f http://localhost:8080/teamengine/ || exit 1"
1819
interval: 15s

0 commit comments

Comments
 (0)