You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/11/19 17:59:41 UTC
[6/6] git commit: updated refs/heads/master to 791f9df
Merge pull request #1084 from ekholabs/improvement/remove_scripts-CLOUDSTACK-9067
CLOUDSTACK-9067 - As I developer I want to remove all the unused router-shell scripts from ACSThis PR removes the unused shell scripts that were present in the ACS project. Those script were replaced by the.
Some of the scripts are used by the HyperV Resource, which were hardcoded. I took the opportunity to use the Java constants over there as well, so the next one touching the code will know they exist and won't hardcode anything.
The following task were applied:
* Remove the shell files and the Java constants that were mapping them;
* Apply the use of the Java constants to the HyperV Resource class;
* Wrap the String.format() method in the StringUtils so we can test the changes in the HyperV Resource class.
The last point was added because I do not have a HyperV test environment. Hence, I wanted to make sure the tiny code I changed is covered at least by unit tests.
* pr/1084:
CLOUDSTACK-9067 - Replaces hardcoded paths with the VRScripts constants.
CLOUDSTACK-9067 - Fomatting the code of HypervDirectConnectResource class
CLOUDSTACK-9067 - Remove old script file from the project
Signed-off-by: Remi Bergsma <gi...@remi.nl>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/791f9dfe
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/791f9dfe
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/791f9dfe
Branch: refs/heads/master
Commit: 791f9dfe5f97c3650a52b16bb7e7c50b553cd25b
Parents: fe2917e 942aa17
Author: Remi Bergsma <gi...@remi.nl>
Authored: Thu Nov 19 17:58:42 2015 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Thu Nov 19 17:58:42 2015 +0100
----------------------------------------------------------------------
.../resource/virtualnetwork/VRScripts.java | 20 +-
.../VirtualRoutingResourceTest.java | 217 +----
.../resource/HypervDirectConnectResource.java | 823 ++++++++++---------
.../kvm/resource/LibvirtComputingResource.java | 13 +-
.../config/opt/cloud/bin/createIpAlias.sh | 129 ---
.../config/opt/cloud/bin/deleteIpAlias.sh | 62 --
.../config/opt/cloud/bin/firewall_egress.sh | 188 -----
.../config/opt/cloud/bin/firewall_ingress.sh | 202 -----
.../debian/config/opt/cloud/bin/firewall_nat.sh | 358 --------
.../config/opt/cloud/bin/getRouterAlerts.sh | 55 --
.../debian/config/opt/cloud/bin/vpc_acl.sh | 250 ------
.../debian/config/opt/cloud/bin/vpc_guestnw.sh | 316 -------
.../debian/config/opt/cloud/bin/vpc_ipassoc.sh | 223 -----
.../config/opt/cloud/bin/vpc_loadbalancer.sh | 229 ------
.../config/opt/cloud/bin/vpc_portforwarding.sh | 126 ---
.../config/opt/cloud/bin/vpc_privateGateway.sh | 98 ---
.../config/opt/cloud/bin/vpc_privategw_acl.sh | 229 ------
.../config/opt/cloud/bin/vpc_staticnat.sh | 124 ---
18 files changed, 439 insertions(+), 3223 deletions(-)
----------------------------------------------------------------------