mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-11-17 11:50:18 +00:00
ci: generate golangci.yml with correct CEPH_VERSION
When building against go-ceph, the most recent version of Ceph is assumed to be available (currently Octopus). In case an older version of the development packages is installed, building go-ceph will fail. Golangci-lint does not accept the `-tags nautilus` parameter like other Golang tools. Instead, the build-constraints need to be configured in a confguration file. This change takes care of the following: - move the current scripts/golangci.yml to a template - add the @@CEPH_VERSION@@ substitute - generate the configuration file when needed Signed-off-by: Niels de Vos <ndevos@redhat.com>
This commit is contained in:
parent
7a87db39e6
commit
f9f9fba26c
3
.gitignore
vendored
3
.gitignore
vendored
@ -14,3 +14,6 @@ deploy/cephcsi/image/cephcsi
|
||||
*.orig
|
||||
*.patch
|
||||
*.rej
|
||||
|
||||
# generated golangci-lint configuration
|
||||
scripts/golangci.yml
|
||||
|
6
Makefile
6
Makefile
@ -86,7 +86,10 @@ mod-check: check-env
|
||||
@echo 'running: go mod verify'
|
||||
@go mod verify && [ "$(shell sha512sum go.mod)" = "`sha512sum go.mod`" ] || ( echo "ERROR: go.mod was modified by 'go mod verify'" && false )
|
||||
|
||||
go-lint:
|
||||
scripts/golangci.yml: scripts/golangci.yml.in
|
||||
sed "s/@@CEPH_VERSION@@/$(CEPH_VERSION)/g" < scripts/golangci.yml.in > scripts/golangci.yml
|
||||
|
||||
go-lint: scripts/golangci.yml
|
||||
./scripts/lint-go.sh
|
||||
|
||||
lint-extras:
|
||||
@ -163,6 +166,7 @@ clean:
|
||||
go clean -mod=vendor -r -x
|
||||
rm -f deploy/cephcsi/image/cephcsi
|
||||
rm -f _output/cephcsi
|
||||
$(RM) scripts/golangci.yml
|
||||
$(RM) e2e.test
|
||||
[ ! -f .devel-container-id ] || $(CONTAINER_CMD) rmi $(CSI_IMAGE_NAME):devel
|
||||
$(RM) .devel-container-id
|
||||
|
@ -5,6 +5,9 @@
|
||||
|
||||
# options for analysis running
|
||||
run:
|
||||
build-tags:
|
||||
- @@CEPH_VERSION@@
|
||||
|
||||
# default concurrency is a available CPU number
|
||||
concurrency: 4
|
||||
|
Loading…
Reference in New Issue
Block a user