diff options
author | Oyvind Bakksjo <bakksjo@yahoo-inc.com> | 2016-06-24 13:39:10 +0200 |
---|---|---|
committer | Oyvind Bakksjo <bakksjo@yahoo-inc.com> | 2016-06-27 12:44:10 +0200 |
commit | 526ceebb348295e9ae4adec05efe1b175e752be0 (patch) | |
tree | 31ce908f50abcc46e24c4d6dcdcadec88ba0c4ba /node-admin/scripts | |
parent | 4ab643f40ccf3eda596067c1120eebf84d57abe7 (diff) |
Rename local variable for explicitness.
Diffstat (limited to 'node-admin/scripts')
-rwxr-xr-x | node-admin/scripts/configure-container-networking.py | 10 |
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) |