summaryrefslogtreecommitdiffstats
path: root/node-admin/scripts
diff options
context:
space:
mode:
authorOyvind Bakksjo <bakksjo@yahoo-inc.com>2016-06-24 13:39:10 +0200
committerOyvind Bakksjo <bakksjo@yahoo-inc.com>2016-06-27 12:44:10 +0200
commit526ceebb348295e9ae4adec05efe1b175e752be0 (patch)
tree31ce908f50abcc46e24c4d6dcdcadec88ba0c4ba /node-admin/scripts
parent4ab643f40ccf3eda596067c1120eebf84d57abe7 (diff)
Rename local variable for explicitness.
Diffstat (limited to 'node-admin/scripts')
-rwxr-xr-xnode-admin/scripts/configure-container-networking.py10
1 files changed, 5 insertions, 5 deletions
diff --git a/node-admin/scripts/configure-container-networking.py b/node-admin/scripts/configure-container-networking.py
index 3776662f70d..7089bdae530 100755
--- a/node-admin/scripts/configure-container-networking.py
+++ b/node-admin/scripts/configure-container-networking.py
@@ -237,14 +237,14 @@ if not container_interface_index:
# 'vespa'. 'vespa' is itself a macvlan bridge linked to the default route's
# interface (typically eth0 or em1). So could we link against eth0 or em1
# (or whatever) instead here? What's the difference?
- interface_index = create_interface_in_namespace(network_namespace=host_ns,
- ip_address_textual=container_ip_arg,
- interface_name=temporary_interface_name_while_in_host_ns,
- link_device_index=host_device_index_for_container)
+ temporary_interface_index = create_interface_in_namespace(network_namespace=host_ns,
+ ip_address_textual=container_ip_arg,
+ interface_name=temporary_interface_name_while_in_host_ns,
+ link_device_index=host_device_index_for_container)
# Move interface from host namespace to container namespace, and change name from temporary name.
# exploit that node_admin docker container shares net namespace with host:
- move_interface(src_interface_index=interface_index,
+ move_interface(src_interface_index=temporary_interface_index,
dest_namespace_pid=container_pid,
dest_interface_name=container_interface_name)