etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
27,619 Commits 323 Branches 325 Tags 836 MiB
Commit Graph

27619 Commits

Author SHA1 Message Date
Kishan Kavala 39d51bc021 CLOUDSTACK-5409: Include projectid during ACL check while listing S2S Vpn gateways 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

This closes #374
 2015-06-12 14:54:44 +03:00
Rajani Karuturi 4900d15e6c Merge branch 'CLOUDSTACK-8551' of https://github.com/karuturi/cloudstack 
This closes #388
 2015-06-12 15:53:08 +05:30
pritisarap12 fd1ef5220b CLOUDSTACK-8553: Unable to launch VM from template because of permission issue 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #387
 2015-06-12 11:54:23 +05:30
pritisarap12 09abb193a1 CLOUDSTACK-8550: Fixed cleanup issue related to VM 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #385
 2015-06-12 11:47:25 +05:30
Priti Sarap f61723dcfb CLOUDSTACK-8547: Modify hypervisor check in testpath_snapshot_hardning.py 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #378
 2015-06-12 11:25:03 +05:30
Rajani Karuturi d504305a98 Fixed CLOUDSTACK-8551 findbugs issue in LdapImportUsersCmd.java 
DM_DEFAULT_ENCODING issue. Used UTF-8
 2015-06-11 17:07:03 +05:30
Rajani Karuturi d46b658ec0 Fixed CLOUDSTACK-8551 Findbugs warning in LdapCreateAccountCmd.java 
byte[].toString() would give reference to the array (ex: [B@6c521576 )
but not the original string.  used new String() to get the text.
 2015-06-11 14:05:03 +05:30
Ian Southam 7ba2f2274a Allow test to run on tiny linux templates using busybox 
Signed-off-by: wilderrodrigues <wrodrigues@schubergphilis.com>

This closes #381
 2015-06-10 22:00:54 +02:00
Sanjeev Neelarapu de6538c9e7 SKIP vm migration tests on LXC since it is not a supported hypervisor 
This closes #383
 2015-06-10 17:29:18 +05:30
Gaurav Aradhye 54a664730d CLOUDSTACK-8515: Skipping snapshot test cases on HyperV and LXC 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #376
 2015-06-10 16:11:34 +05:30
Gaurav Aradhye e3cfe89053 CLOUDSTACK-8546: Correcting skip test code in testpath_vMotion_vmware.py 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #377
 2015-06-10 16:04:10 +05:30
pritisarap12 84bc2e6127 CLOUDSTACK-8549: Update assert statements in testpath_disable_enable_zone.py 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #382
 2015-06-10 15:59:45 +05:30
Sateesh Chodapuneedi 5e41a830a5 This closes #373 
CLOUDSTACK-3317 - DVS does not support management\storage network

Added support for Management and Storage Network traffic over VMware DVS in CloudStack deployments. Also added support for storage VLAN over dvPortGroup.

Signed-off-by: Sateesh Chodapuneedi <sateesh@apache.org>

CLOUDSTACK-3317 - DVS does not support management\storage network

Use non-zero dvport count while updating dvportgroups of system traffic.
Updated configuration compare logic to avoid update dvportgroup operation unless required.
This would help improve speed in vm/network deployment as the update calls would reduce. Also improved logging.

Signed-off-by: Sateesh Chodapuneedi <sateesh@apache.org>

Adding unit tests in class HypervisorHostHelperTest

Signed-off-by: Sateesh Chodapuneedi <sateesh@apache.org>

Added license header to new file being added to repo/branch.

Signed-off-by: Sateesh Chodapuneedi <sateesh@apache.org>
 2015-06-10 02:07:40 +05:30
Sanjeev Neelarapu 8f1e0510cd Skipping all unsupported tests on LXC 
This closes #379
 2015-06-10 13:47:28 +05:30
Gaurav Aradhye 6140db50b3 CLOUDSTACK-8542: Correcting test case in test_vpc_on_host_maintenance.py 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #360
 2015-06-08 13:37:37 +05:30
Daan Hoogland b272d7717e Revert "Fix 2 findbugs high priority warnings" 
This reverts commit 1ca74dac26.
 2015-06-08 10:01:19 +02:00
Rafael da Fonseca c5437d216f Fix malformed closing tag on comments in cisco .xml files 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #369
 2015-06-08 09:41:13 +02:00
Rafael da Fonseca b9b13d764d Fix another case of unnecessary boxing JuniperSrxResource.java:1280, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High Boxing/unboxing to parse a primitive com.cloud.network.resource.JuniperSrxResource.removeDestinationNatRules(Long, Map, List) 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #367
 2015-06-08 09:41:07 +02:00
Rafael da Fonseca 039d632ad3 Fix 3 more findbugs performance alerts, creating Long and Integer unnecessarily F5BigIpResource.java:297, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High F5BigIpResource.java:703, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High F5BigIpResource.java:889, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High Boxing/unboxing to parse a primitive com.cloud.network.resource.F5BigIpResource 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #366
 2015-06-08 09:40:59 +02:00
Rafael da Fonseca e8c7069f73 Fix findbugs high priority warning VmwareStorageManagerImpl.java:1023, RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE, Priority: High Redundant nullcheck of datastoreVolumePath, which is known to be non-null in com.cloud.hypervisor.vmware.manager.VmwareStorageManagerImpl.getVolumePathInDatastore(DatastoreMO, String) 
Assertion is not used in runtime, correct way is throw and handle exception without killing app

Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #362
 2015-06-08 09:40:50 +02:00
Rafael da Fonseca 9bac84a3f7 Fix last ocurrence of unnecessary boxing detected by findbugs CiscoVnmcResource.java:266, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High Boxing/unboxing to parse a primitive com.cloud.network.resource.CiscoVnmcResource.getIpRangeFromCidr(String) 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #368
 2015-06-08 09:39:02 +02:00
Rafael da Fonseca 38c269d71e Fix performance issue reported by findbugs (unnescessary boxing/unboxing) VmwareResource.java:693, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High VmwareResource.java:4769, DM_BOXED_PRIMITIVE_FOR_PARSING, Priority: High Boxing/unboxing to parse a primitive com.cloud.hypervisor.vmware.resource.VmwareResource.getNetworkStats(String) 
Now op is faster and takes up less memory

Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #365
 2015-06-08 09:32:52 +02:00
Rafael da Fonseca 65383fb8fa Fix another findbugs high priority warning JuniperSrxResource.java:2617, ES_COMPARING_STRINGS_WITH_EQ, Priority: High Comparison of String objects using == or != in com.cloud.network.resource.JuniperSrxResource.manageApplication(JuniperSrxResource$SecurityPolicyType, JuniperSrxResource$SrxCommand, JuniperSrxResource$Protocol, int, int) 
This now correctly compares strings

Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #364
 2015-06-08 09:30:38 +02:00
Rafael da Fonseca 1ca74dac26 Fix 2 findbugs high priority warnings 
VmwareResource.java:315, MS_SHOULD_BE_FINAL, Priority: High
com.cloud.hypervisor.vmware.resource.VmwareResource.s_serviceContext isn't final but should be

VmwareResource.java:331, MS_SHOULD_BE_FINAL, Priority: High
com.cloud.hypervisor.vmware.resource.VmwareResource.s_powerStatesTable isn't final but should be

Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #363
 2015-06-08 09:24:07 +02:00
jeff db69c8e82b Allow PropertiesUtil to read from jar files. 
PropertiesUtil has code for reading from jar files, but the
findConfigFile method will prevent it from ever returning a file in a
jar on the classpath since it always wants to have a "file:" URL and
use the File class.

This commit moves the jar file loading attempt from a catch block to
an else clause, executed if a config file:// URL could not be found.

Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #358
 2015-06-08 09:19:21 +02:00
pritisarap12 0326fb3b5c CLOUDSTACK-8308: Updating Disable/enable host testpath as admin user should not be able to deploy vm on disabled host 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #356
 2015-06-05 14:08:56 +05:30
wilderrodrigues 93845afd53 Using a try-wioth resrouce block as suggested in @DaanHoogland review. 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #355
 2015-06-04 12:34:05 +02:00
wilderrodrigues 7f4e2c7cf0 Renaming the variable from "s" to "script" 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:05 +02:00
wilderrodrigues 6138d9a690 Coverity issue 1116812 - Replacing concatenation with optionsBuffer.append(option.getKey()).append('=').append(option.getValue()).append(','); 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:05 +02:00
wilderrodrigues 9ff38486a1 Coverity issue 1116677 - Avoiding catching only Exception. Makes the code too britle. - Catching the QemuImgException and throwing it to be caught further in the code - Surrounding the output stream with try/catch and throwing it to be further handled in the code. Closing the output stream quietly. 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:05 +02:00
wilderrodrigues 6271663682 Formatting the code - Adding final modifier and indenting the code 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:04 +02:00
wilderrodrigues c3b4c7a9cf Coverity issue 1116509 - Assigning the the new returned ResultSet to the rs variable in order to get it closed in the finally block 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:04 +02:00
wilderrodrigues abe0990259 Formatting the code - Adding final modifier and indenting the code 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:04 +02:00
wilderrodrigues fe78c76691 Coverity issue: 1012179 - Commenting out unused variable. 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:03 +02:00
wilderrodrigues 159d8c2c97 Formatting the code - Adding final modifier to attributes and indenting the code. 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>
 2015-06-04 12:34:03 +02:00
Gaurav Aradhye f92a50363c CLOUDSTACK-8515: Skipping snapshots tests on HyperV and LXC hypervisors 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #350
 2015-06-04 10:56:28 +05:30
Rafael da Fonseca 00884b3b0b Fixes LB certificate tests failure when the full path has a special character like whitespace or many others. 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

This closes #336
 2015-06-04 00:11:09 +02:00
Daan Hoogland 3d4d152753 CID 1302976 Scanner in try-with-resource 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

This closes #352
 2015-06-04 00:09:40 +02:00
Daan Hoogland 109b6e94d3 CID 1302974: Scanner in try-with-resource 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

This closes #353
 2015-06-04 00:09:16 +02:00
pritisarap12 9c2a1ea81e CLOUDSTACK-8532: Modification in setupClass to skip testcases rather than throwing an exception on "Exceeding connection limit to Netscaler device" 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #343
 2015-06-03 15:19:04 +05:30
Rajani Karuturi b31b8425df CLOUDSTACK-8525: NPE while updating the state of the volume after deletion 
The volume is already deleted (may be by the cleanup thread) and hence
the NPE. Added a not null check for the volumevo and returning false
from the state transition

This closes #321
 2015-06-03 11:45:02 +05:30
Rafael da Fonseca bec44bffb3 Upgrade Reflections to 0.9.9 
Signed-off-by: Rajani Karuturi <rajanikaruturi@gmail.com>

This closes #333
 2015-06-03 11:01:32 +05:30
Rajani Karuturi c2f2f77ab1 Merge branch 'fast_tests' of https://github.com/rsafonseca/cloudstack 
This closes #330
 2015-06-03 10:08:11 +05:30
miguelaferreira e983246cd4 Set the url accordingly when installing a system vm template 
The script that installs the system vm templates sets the uuid column
for the template being installed, however it does not set the respective
url column. This commit changes that.

Signed-off-by: Remi Bergsma <apache@remi.nl>

This closes #348
 2015-06-02 16:00:15 +02:00
Rohit Yadav f341246888 CLOUDSTACK-8530: KVM hosts without active agents should be in Disconnected state 
KVM hosts which are actuall up, but if their agents are shutdown should be put
in disconnected state. This would avoid getting the VMs HA'd and other commands
such as deploying a VM will exclude that host and save us from errors.

The improvement is that, we first try to contact the KVM host itself. If it fails
we assume that it's disconnected, and then ask its KVM neighbours if they can
check its status. If all of the KVM neighbours tell us that it's Down and we're
unable to reach the KVM host, then the host is possibly down. In case any of the
KVM neighbours tell us that it's Up but we're unable to reach the KVM host then
we can be sure that the agent is offline but the host is running.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: wilderrodrigues <wrodrigues@schubergphilis.com>

This closes #340
 2015-06-02 13:27:16 +02:00
Gaurav Aradhye f2b1ec2c7d CLOUDSTACK:8515: Skipping snapshot test case for HyperV and LXC 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #347
 2015-06-02 16:18:51 +05:30
pritisarap12 d6052a31a3 Modified test_lb_secondary_ip.py file: Local variable accessed as class variable 
Signed-off-by: Daan Hoogland <daan.hoogland@gmail.com>

This closes #344
 2015-06-02 12:33:06 +02:00
Gaurav Aradhye 06f9ee63bb CLOUDSTACK-8534: Fixed import and pep8 issues in test_redundant_router_upgrades.py 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #346
 2015-06-02 15:52:11 +05:30
Gaurav Aradhye 8bf59b5393 CLOUDSTACK-8521: Improving logging in test case - test_egress_fw_rules.py 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #345
 2015-06-02 13:31:29 +05:30
Abhinav Roy 108a74a6b8 CLOUDSTACK-8487 : Add vMotion related tests 
Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes #268
 2015-06-02 12:01:12 +05:30