Skip to content

Commit 712dc4b

Browse files
phermann1988claude
andcommitted
fix: point kind/helmfile/helm/k9s/yq bin_dir at /usr/bin
bin_dir was '/usr/bin/<name>', which the download_install_binary role turns into '/usr/bin/<name>/<name>'. When the binary is already preinstalled as a file at /usr/bin/<name> (e.g. ubuntu26 base-os template 144), the role fails with '[Errno 20] Not a directory'. Point bin_dir at the directory /usr/bin like every other tool. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 5b0a3f1 commit 712dc4b

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

defaults/main.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ bin:
116116
source_url: "https://github.com/kubernetes-sigs/kind/releases/download/v{{ kind_version }}/kind-linux-amd64"
117117
bin_to_copy: kind-linux-amd64
118118
to_remove: ""
119-
bin_dir: "/usr/bin/kind"
119+
bin_dir: "/usr/bin"
120120
version_cmd: "version"
121121
target_version: "{{ kind_version }}"
122122
helmfile:
@@ -126,7 +126,7 @@ bin:
126126
source_url: "https://github.com/helmfile/helmfile/releases/download/v{{ helmfile_version }}/helmfile_{{ helmfile_version }}_linux_amd64.tar.gz"
127127
bin_to_copy: helmfile
128128
to_remove: "helmfile"
129-
bin_dir: "/usr/bin/helmfile"
129+
bin_dir: "/usr/bin"
130130
version_cmd: "version"
131131
target_version: "{{ helmfile_version }}"
132132
helm:
@@ -136,7 +136,7 @@ bin:
136136
source_url: "https://get.helm.sh/helm-v{{ helm_version }}-linux-amd64.tar.gz"
137137
bin_to_copy: "linux-amd64/helm"
138138
to_remove: "helm"
139-
bin_dir: "/usr/bin/helm"
139+
bin_dir: "/usr/bin"
140140
version_cmd: "version"
141141
target_version: "{{ helm_version }}"
142142
k9s:
@@ -146,7 +146,7 @@ bin:
146146
source_url: "https://github.com/derailed/k9s/releases/download/v{{ k9s_version }}/k9s_Linux_amd64.tar.gz"
147147
bin_to_copy: "k9s"
148148
to_remove: "k9s"
149-
bin_dir: "/usr/bin/k9s"
149+
bin_dir: "/usr/bin"
150150
version_cmd: "version"
151151
target_version: "{{ k9s_version }}"
152152
yq:
@@ -156,6 +156,6 @@ bin:
156156
source_url: "https://github.com/mikefarah/yq/releases/download/v{{ yq_version }}/yq_linux_amd64.tar.gz"
157157
bin_to_copy: yq_linux_amd64
158158
to_remove: ""
159-
bin_dir: "/usr/bin/yq"
159+
bin_dir: "/usr/bin"
160160
version_cmd: " version"
161161
target_version: "v{{ yq_version }}"

molecule/docker_test/converge.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
source_url: "https://github.com/kubernetes-sigs/kind/releases/download/v{{ kind_version }}/kind-linux-amd64"
4444
bin_to_copy: kind-linux-amd64
4545
to_remove: ""
46-
bin_dir: "/usr/bin/kind"
46+
bin_dir: "/usr/bin"
4747
version_cmd: "version"
4848
target_version: "{{ kind_version }}"
4949
helmfile:
@@ -53,7 +53,7 @@
5353
source_url: "https://github.com/helmfile/helmfile/releases/download/v{{ helmfile_version }}/helmfile_{{ helmfile_version }}_linux_amd64.tar.gz"
5454
bin_to_copy: helmfile
5555
to_remove: "helmfile"
56-
bin_dir: "/usr/bin/helmfile"
56+
bin_dir: "/usr/bin"
5757
version_cmd: "version"
5858
target_version: "{{ helmfile_version }}"
5959
helm:
@@ -63,7 +63,7 @@
6363
source_url: "https://get.helm.sh/helm-v{{ helm_version }}-linux-amd64.tar.gz"
6464
bin_to_copy: "linux-amd64/helm"
6565
to_remove: "helm"
66-
bin_dir: "/usr/bin/helm"
66+
bin_dir: "/usr/bin"
6767
version_cmd: "version"
6868
target_version: "{{ helm_version }}"
6969
k9s:
@@ -73,7 +73,7 @@
7373
source_url: "https://github.com/derailed/k9s/releases/download/v{{ k9s_version }}/k9s_Linux_amd64.tar.gz"
7474
bin_to_copy: "k9s"
7575
to_remove: "k9s"
76-
bin_dir: "/usr/bin/k9s"
76+
bin_dir: "/usr/bin"
7777
version_cmd: "version"
7878
target_version: "{{ k9s_version }}"
7979

0 commit comments

Comments
 (0)