openstack/openstack-ansible-tests
Code Issues Proposed changes

network_interfaces: Remove useless 'break' command from case statement

Browse Source 
'break' is pointless in a case statement. This fixes the following
warning:

"break: only meaningful in a `for', `while', or `until' loop"

Change-Id: Id8dbb96f1a918a378c3ac16862c3bd4fea3c94d3
This commit is contained in:
Markos Chandras 2017-09-02 12:48:50 +01:00
parent aaa6825c14
commit e8b9340567
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
network_interfaces/rpm_interface_ifdown-post.cfg.j2
View File

@ -16,8 +16,8 @@
source /etc/os-release || source /usr/lib/os-release
case "${ID}" in
*suse*) INTERFACE="${1}"; break ;;
centos|rhel|fedora) INTERFACE="${DEVICE}"; break ;;
*suse*) INTERFACE="${1}"; ;;
centos|rhel|fedora) INTERFACE="${DEVICE}"; ;;
*) echo "Unsupported distribution ${ID}"; exit 1;
esac

4
network_interfaces/rpm_interface_ifup-post.cfg.j2
View File

@ -17,8 +17,8 @@
source /etc/os-release || source /usr/lib/os-release
case "${ID}" in
*suse*) INTERFACE="${1}"; break ;;
centos|rhel|fedora) INTERFACE="${DEVICE}"; break ;;
*suse*) INTERFACE="${1}"; ;;
centos|rhel|fedora) INTERFACE="${DEVICE}"; ;;
*) echo "Unsupported distribution ${ID}"; exit 1;
esac