Skip to content

Commit b95bf7f

Browse files
committed
Makefile: remove python3 interpreter
We can now set the Python Interpreter globally in the ansible.cfg and with Kconfig or cli. Drops support for the calls where ansible-playbook was specifying the interpreter with ansible_python_interpreter inventory variable. Signed-off-by: Daniel Gomez <[email protected]>
1 parent f25f3de commit b95bf7f

File tree

6 files changed

+7
-24
lines changed

6 files changed

+7
-24
lines changed

Makefile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -228,15 +228,13 @@ $(KDEVOPS_HOSTS): .config ansible.cfg $(KDEVOPS_HOSTS_TEMPLATE)
228228
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
229229
--inventory localhost, \
230230
$(KDEVOPS_PLAYBOOKS_DIR)/gen_hosts.yml \
231-
-e 'ansible_python_interpreter=/usr/bin/python3' \
232231
--extra-vars=@./extra_vars.yaml
233232

234233
DEFAULT_DEPS += $(KDEVOPS_NODES)
235234
$(KDEVOPS_NODES) $(KDEVOPS_VAGRANT): .config ansible.cfg $(KDEVOPS_NODES_TEMPLATE)
236235
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
237236
--inventory localhost, \
238237
$(KDEVOPS_PLAYBOOKS_DIR)/gen_nodes.yml \
239-
-e 'ansible_python_interpreter=/usr/bin/python3' \
240238
--extra-vars=@./extra_vars.yaml
241239

242240
DEFAULT_DEPS += $(LOCALHOST_SETUP_WORK)

Makefile.build_qemu

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,31 +23,27 @@ qemu: $(KDEVOPS_EXTRA_VARS)
2323
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
2424
--inventory localhost, \
2525
$(KDEVOPS_PLAYBOOKS_DIR)/build_qemu.yml \
26-
-e 'ansible_python_interpreter=/usr/bin/python3' \
2726
--extra-vars=@./extra_vars.yaml
2827
PHONY += qemu
2928

3029
qemu-install: $(KDEVOPS_EXTRA_VARS)
3130
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
3231
--inventory localhost, \
3332
$(KDEVOPS_PLAYBOOKS_DIR)/build_qemu.yml \
34-
-e 'ansible_python_interpreter=/usr/bin/python3' \
3533
--extra-vars=@./extra_vars.yaml --tags vars,install
3634
PHONY += qemu-install
3735

3836
qemu-configure: $(KDEVOPS_EXTRA_VARS)
3937
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
4038
--inventory localhost, \
4139
$(KDEVOPS_PLAYBOOKS_DIR)/build_qemu.yml \
42-
-e 'ansible_python_interpreter=/usr/bin/python3' \
4340
--extra-vars=@./extra_vars.yaml --tags vars,configure
4441
PHONY += qemu-configure
4542

4643
qemu-build: $(KDEVOPS_EXTRA_VARS)
4744
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
4845
--inventory localhost, \
4946
$(KDEVOPS_PLAYBOOKS_DIR)/build_qemu.yml \
50-
-e 'ansible_python_interpreter=/usr/bin/python3' \
5147
--extra-vars=@./extra_vars.yaml --tags vars,build
5248
PHONY += qemu-build
5349

Makefile.hypervisor-tunings

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ kdevops_hypervisor_tunning: $(KDEVOPS_EXTRA_VARS)
1717
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
1818
--inventory localhost, \
1919
$(KDEVOPS_PLAYBOOKS_DIR)/hypervisor-tuning.yml \
20-
-e 'ansible_python_interpreter=/usr/bin/python3' \
2120
--extra-vars=@./extra_vars.yaml
2221
PHONY += kdevops_hypervisor_tunning
2322

Makefile.kdevops

Lines changed: 7 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -11,28 +11,24 @@ PHONY := kdevops_all
1111
kdevops_terraform_deps:
1212
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
1313
--inventory localhost, \
14-
$(KDEVOPS_PLAYBOOKS_DIR)/install_terraform.yml \
15-
-e 'ansible_python_interpreter=/usr/bin/python3'
14+
$(KDEVOPS_PLAYBOOKS_DIR)/install_terraform.yml
1615
PHONY += kdevops_terraform_deps
1716

1817
kdevops_vagrant_install_vagrant:
1918
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
2019
--inventory localhost, \
21-
$(KDEVOPS_PLAYBOOKS_DIR)/install_vagrant.yml \
22-
-e 'ansible_python_interpreter=/usr/bin/python3'
20+
$(KDEVOPS_PLAYBOOKS_DIR)/install_vagrant.yml
2321

2422
kdevops_install_libvirt:
2523
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
2624
--inventory localhost, \
27-
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_user.yml -e "skip_configuration=True" \
28-
-e 'ansible_python_interpreter=/usr/bin/python3'
25+
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_user.yml -e "skip_configuration=True"
2926

3027
kdevops_configure_libvirt:
3128
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
3229
--inventory localhost, \
3330
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_user.yml -e "skip_install=True" \
34-
-e 'running_user=$(USER)' \
35-
-e 'ansible_python_interpreter=/usr/bin/python3'
31+
-e 'running_user=$(USER)'
3632

3733
kdevops_vagrant_deps: \
3834
kdevops_vagrant_install_vagrant \
@@ -45,25 +41,22 @@ KDEVOPS_VAGRANT_WORK := kdevops_vagrant_deps
4541
kdevops_vagrant_boxes:
4642
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
4743
--inventory localhost, \
48-
$(KDEVOPS_PLAYBOOKS_DIR)/install_vagrant_boxes.yml \
49-
-e 'ansible_python_interpreter=/usr/bin/python3'
44+
$(KDEVOPS_PLAYBOOKS_DIR)/install_vagrant_boxes.yml
5045
PHONY += kdevops_vagrant_boxes
5146
KDEVOPS_VAGRANT_WORK += kdevops_vagrant_boxes
5247

5348
kdevops_verify_libvirt_user:
5449
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
5550
--inventory localhost, \
56-
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_user.yml -e "only_verify_user=True" \
57-
-e 'ansible_python_interpreter=/usr/bin/python3'
51+
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_user.yml -e "only_verify_user=True"
5852

5953
PHONY += kdevops_verify_libvirt_user
6054
KDEVOPS_VAGRANT_WORK += kdevops_verify_libvirt_user
6155

6256
kdevops_libvirt_storage_pool_create:
6357
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
6458
--inventory localhost, \
65-
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_storage_pool_create.yml \
66-
-e 'ansible_python_interpreter=/usr/bin/python3'
59+
$(KDEVOPS_PLAYBOOKS_DIR)/libvirt_storage_pool_create.yml
6760

6861
PHONY += kdevops_libvirt_storage_pool_create
6962
KDEVOPS_VAGRANT_WORK += kdevops_libvirt_storage_pool_create

Makefile.linux-mirror

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ kdevops_linux_mirror: $(KDEVOPS_EXTRA_VARS)
2929
--inventory localhost, \
3030
--tags vars,mirror \
3131
$(KDEVOPS_PLAYBOOKS_DIR)/linux-mirror.yml \
32-
-e 'ansible_python_interpreter=/usr/bin/python3' \
3332
--extra-vars=@./extra_vars.yaml
3433
PHONY += kdevops_linux_mirror
3534

@@ -42,7 +41,6 @@ mirror-status: $(KDEVOPS_EXTRA_VARS)
4241
--inventory localhost, \
4342
--tags vars,mirror-status \
4443
$(KDEVOPS_PLAYBOOKS_DIR)/linux-mirror.yml \
45-
-e 'ansible_python_interpreter=/usr/bin/python3' \
4644
--extra-vars=@./extra_vars.yaml
4745

4846
PHONY += mirror-status

Makefile.postfix

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ kdevops_postfix_setup_relay: $(KDEVOPS_EXTRA_VARS)
1010
$(Q)ansible-playbook $(ANSIBLE_VERBOSE) --connection=local \
1111
--inventory localhost, \
1212
$(KDEVOPS_PLAYBOOKS_DIR)/postfix_relay_host.yml \
13-
-e 'ansible_python_interpreter=/usr/bin/python3' \
1413
--extra-vars=@./extra_vars.yaml
1514
PHONY += kdevos_postfix_setup_relay
1615
ANSIBLE_EXTRA_ARGS += $(POSTFIX_SETUP_ARGS)

0 commit comments

Comments
 (0)