Skip to content

Commit

Permalink
install/kubernetes/Makefile: use CURDIR
Browse files Browse the repository at this point in the history
./tetragon does not seem to work for my version of docker, and I'm
getting:

docker: Error response from daemon: create ./tetragon: "./tetragon" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed. If you intended to pass a host directory, use absolute path.

Use CURDIR to pass the full path.

Other versions of docker seem to work fine.

Signed-off-by: Kornilios Kourtis <[email protected]>
  • Loading branch information
kkourt committed Jun 11, 2024
1 parent 9348854 commit 9ee2be0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions install/kubernetes/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ REPO_ROOT := $(shell git rev-parse --show-toplevel)
TETRAGON_CHART := tetragon
CRDS := $(REPO_ROOT)/pkg/k8s/apis/cilium.io/client/crds/v1alpha1

HELM ?= docker run --rm -v ./$(TETRAGON_CHART):/apps $(HELM_IMAGE)
HELM ?= docker run --rm -v $(CURDIR)/$(TETRAGON_CHART):/apps $(HELM_IMAGE)

.PHONY: all
all: deps $(TETRAGON_CHART)/crds-yaml lint docs
Expand All @@ -28,7 +28,7 @@ lint:

.PHONY: docs
docs:
docker run --rm -v ./$(TETRAGON_CHART):/helm-docs -u $$(id -u) $(HELMDOCS_IMAGE)
docker run --rm -v $(CURDIR)/$(TETRAGON_CHART):/helm-docs -u $$(id -u) $(HELMDOCS_IMAGE)
./export-doc.sh $(REPO_ROOT)/docs/content/en/docs/reference/helm-chart.md

# NB: Helm has an "official" way to install CRDs which requires simply putting
Expand Down

0 comments on commit 9ee2be0

Please sign in to comment.