cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r...@apache.org
Subject [6/6] git commit: updated refs/heads/master to 791f9df
Date Thu, 19 Nov 2015 16:59:41 GMT
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 <github@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 <github@remi.nl>
Authored: Thu Nov 19 17:58:42 2015 +0100
Committer: Remi Bergsma <github@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(-)
----------------------------------------------------------------------



Mime
View raw message