diff options
| author | Valentin Rothberg <rothberg@redhat.com> | 2021-01-19 17:19:46 +0100 |
|---|---|---|
| committer | Matthew Heon <matthew.heon@pm.me> | 2021-01-29 15:27:19 -0500 |
| commit | edd9a06dc7ea3661f8fd581ecc2086a6c0111ab3 (patch) | |
| tree | 24521c806049df7831a42fcd6f6f04ed99b7a0d6 | |
| parent | 6bf4d6195a02fcd39b6d1f7220b4e8ab4f3c2cb5 (diff) | |
| download | podman-edd9a06dc7ea3661f8fd581ecc2086a6c0111ab3.tar.gz podman-edd9a06dc7ea3661f8fd581ecc2086a6c0111ab3.tar.bz2 podman-edd9a06dc7ea3661f8fd581ecc2086a6c0111ab3.zip | |
Cirrus: add bindings checks
Make sure that bindings are in sync with the code. The check is similar
to what's already being done with `make vendor`, so integrate the two.
[NO TESTS NEEDED]
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| -rw-r--r-- | .cirrus.yml | 14 | ||||
| -rw-r--r-- | Makefile | 1 | ||||
| -rwxr-xr-x | contrib/cirrus/runner.sh | 6 | ||||
| -rwxr-xr-x | contrib/cirrus/setup_environment.sh | 2 |
4 files changed, 14 insertions, 9 deletions
diff --git a/.cirrus.yml b/.cirrus.yml index 0abb71146..7c797cdf2 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -274,17 +274,19 @@ swagger_task: # Check that all included go modules from other sources match -# what is expected in `vendor/modules.txt` vs `go.mod`. -vendor_task: - name: "Test Vendoring" - alias: vendor +# what is expected in `vendor/modules.txt` vs `go.mod`. Also +# make sure that the generated bindings in pkg/bindings/... +# are in sync with the code. +consistency_task: + name: "Test Code Consistency" + alias: consistency skip: *tags depends_on: - build container: *smallcontainer env: <<: *stdenvars - TEST_FLAVOR: vendor + TEST_FLAVOR: consistency TEST_ENVIRON: container CTR_FQIN: ${FEDORA_CONTAINER_FQIN} clone_script: *full_clone # build-cache not available to container tasks @@ -642,7 +644,7 @@ success_task: - validate - bindings - swagger - - vendor + - consistency - alt_build - static_alt_build - osx_alt_build @@ -460,6 +460,7 @@ podman-remote-%-release: rm -f release.txt $(MAKE) podman-remote-release-$*.zip +.PHONY: generate-bindings generate-bindings: ifneq ($(shell uname -s), Darwin) GO111MODULE=off $(GO) generate ./pkg/bindings/... ; diff --git a/contrib/cirrus/runner.sh b/contrib/cirrus/runner.sh index 572f0b44a..6e6747f28 100755 --- a/contrib/cirrus/runner.sh +++ b/contrib/cirrus/runner.sh @@ -142,9 +142,11 @@ function _run_swagger() { cp -v $GOSRC/pkg/api/swagger.yaml $GOSRC/ } -function _run_vendor() { +function _run_consistency() { make vendor - ./hack/tree_status.sh + SUGGESTION="run 'make vendor' and commit all changes" ./hack/tree_status.sh + make generate-bindings + SUGGESTION="run 'make generate-bindings' and commit all changes" ./hack/tree_status.sh } function _run_build() { diff --git a/contrib/cirrus/setup_environment.sh b/contrib/cirrus/setup_environment.sh index 5c6f05ac0..7b49caba0 100755 --- a/contrib/cirrus/setup_environment.sh +++ b/contrib/cirrus/setup_environment.sh @@ -214,7 +214,7 @@ case "$TEST_FLAVOR" in install_test_configs ;; - vendor) make clean ;; + consistency) make clean ;; release) ;; *) die_unknown TEST_FLAVOR esac |