Commit Graph

277 Commits

Author SHA1 Message Date
Santhosh Edukulla 03830c570e Adding few changes and cleaning up the code
- Added few common naming conventions
- Cleanedup code
- Added a simple utility function

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-10-28 12:49:41 +05:30
Santhosh Edukulla b4ceefa469 Adding readable start and end time stamps
Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-10-28 11:29:27 +05:30
Santhosh Edukulla 0780604503 CLOUDSTACK-4885: basic timing profiler for tests
This will add time taken in seconds along with start and end timestamp
logging facility for test cases.Currently this is not
available for console logs

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-10-24 14:52:53 +05:30
Santhosh Edukulla d4072c6daa Added few changes related to bug 4920. Removed unwanted code
There were few unwanted calls as part of test client, did some clean up
Made the test client API uniform to accept both mgmt and dbsvr details
Did some minor bug fixes as well.

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-10-23 13:53:43 +05:30
Santhosh Edukulla 3f656b0434 The code module represents enum for constant comparison
This module provides facilitation to use uniform codes across various test
features. Users can import this  module and use these codes to maintain
readability and one point of change.

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-10-22 15:38:48 +05:30
Prasanna Santhanam dfa0678fc6 marvin: a1b979da8 breaks formatting and pep8.
setup/dev/advanced.cfg is used by the simulator deployments that are
usually not https. disabled the http within this config file.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-10-22 11:59:14 +05:30
Santhosh Edukulla a1b979da86 CLOUDSTACK-4832. Added support for https to marvin.
advanced.cfg: Contains three additional flags "useHttps,certCAPath,certPath"
for https usage in marvin for establishing cs connection. We will use the
configuraiton under advanced.cfg provided by user to establish connection over
https. If establishing the connection over https failed, then the default certs
will be used.  or else raise the exception, the existing http will work as it
is when useHttps flag set to "False"

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-10-22 11:32:10 +05:30
Gaurav Aradhye 0a8bce5cf6 CLOUDSTACK-4647: Fixed snapshot_gc test case and common util function is_snapshot_on_nfs
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-10-15 13:37:08 +05:30
Anshul Gangwar 7b5daaea20 added support for ispersistent parameter for networkoffering
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-10-15 13:22:42 +05:30
Prasanna Santhanam 52c2bc0baf pep8: fix regression from 10938612cc
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-10-10 17:47:52 +05:30
Daan Hoogland 10938612cc [CLOUDSTACK-4832] no cert verification on https 2013-10-08 17:35:46 +02:00
Prasanna Santhanam 5ed493c6e6 pep8: fix recent pep8 violations
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-10-04 14:13:37 +05:30
Gaurav Aradhye b7e00d3700 CLOUDSTACK: 4706 - Adding missing method get_region in common.py
Signed-off-by: venkataswamybabu budumuru <venkataswamybabu.budumuru@citrix.com>
(cherry picked from commit 503fe75dc9)
2013-09-30 15:19:58 +05:30
SrikanteswaraRao Talluri 65da24c35e CLOUDSTACK-4691: move egress rules host maintenance tests to maint folder
Signed-off-by: venkataswamybabu budumuru <venkataswamybabu.budumuru@citrix.com>
(cherry picked from commit 6b28e03d6d)
2013-09-30 15:12:20 +05:30
Edison Su b3ac12e171 delete snapshot_store_ref if the snapshot is in error state 2013-09-25 15:51:55 -07:00
Hugo Trippaers 15a97ec412 Cleanup pom.xml in various projects to properly use managed versions and other defaults 2013-09-21 17:21:06 +08:00
Anshul Gangwar 955408166c CLOUDSTACK-2266: marvin tests for IP Address reservation within a network it also adds the ipaddress parameter to virtual machine's create method in base.py
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-09-18 09:21:24 +05:30
Gaurav Aradhye ae1cebf4e9 Automation - Portable Ip feature test cases
Signed-off-by: venkataswamybabu budumuru <venkataswamybabu.budumuru@citrix.com>
(cherry picked from commit 23c759ec35)
2013-09-13 17:35:08 +05:30
Prasanna Santhanam 433d5839a1 CLOUDSTACK-4608: Resolve the hostname to fqdn from marvin config
In case the marvin configuration does not contain fqdn information fetch
and compare the fqdns of hostip from `cloud`.`host` with the fqdn
available on the client.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-09-09 11:41:13 +05:30
Gaurav Aradhye eb390fba22 CLOUDSTACK-4531: Fixed indent problem with the patch.
Signed-off-by: venkataswamybabu budumuru <venkataswamybabu.budumuru@citrix.com>
(cherry picked from commit 751fe8e339)
2013-09-06 21:36:42 +05:30
Sowmya Krishnan 8b5cb374b9 CLOUDSTACK-4487 Fix adding Netscaler service provider if not already done
Signed-off-by: venkataswamybabu budumuru <venkataswamybabu.budumuru@citrix.com>
(cherry picked from commit c1ea61a0e7)
2013-09-05 15:08:43 +05:30
Girish Shilamkar 0fb2014d19 CLOUDSTACK-4531: Resolved ssh error for basic zone. Public ip should be used for ssh instead of ipaddress of nic
Signed-off-by: venkataswamybabu budumuru <venkataswamybabu.budumuru@citrix.com>
(cherry picked from commit 4f3b411d4c)
2013-09-05 10:20:17 +05:30
Prasanna Santhanam 5326833971 Fix pep8 formatting to fix cloudstack-marvin build 2013-08-30 12:40:42 +05:30
Gaurav Aradhye 95e4b7afa8 Automation: Limit Resources - Adding tests related to CPU limits
Reviewed-by: Sanjay Tripathi <Sanjay.Tripathi@citrix.com>
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-08-29 14:13:39 +05:30
Girish Shilamkar c6f0d46911 CLOUDSTACK-4407: Use extractTemplate API to get hypervisor specific template information
Template url, hypervisor and format were defined in Service class to be Xenserver specific
and therefore registering a new template failed on Vmware and KVM.
Fixed this to get hypervisor specific info for registering new template.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 20256706b376551fe8993ee2e73c61df31dcb6de)
2013-08-29 14:10:54 +05:30
Prasanna Santhanam 1058654cb4 CLOUDSTACK-4499: Wait until hosts are up before adding storage pools
This works around the delay caused in adding Xen 6.1/6.2 hosts where
host takes two attempts to transition to Up state. We will wait for one
minute per cluster before attempting to add storage pool in the cluster.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 22ee499c3571b2a6b6921abb36c679128893c415)
2013-08-29 14:10:54 +05:30
Ashutosh Kelkar a7833cdfb4 CLOUDSTACK-4303: Add missing base.py changes
egressdefaultpolicy parameter was not set in base.py. Therefore
egress test failed. Added this missing change.

Signed-off-by: Jayapal <jayapal@apache.org>
2013-08-27 16:08:37 +05:30
Prasanna Santhanam de770d37ec CLOUDSTACK-4452: Fix the mount format to host:dir
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit eabd53b9719fe38b933efe1169c420c88ad5ecf5)
2013-08-27 15:28:20 +05:30
Sowmya Krishnan f1c6535f6e CLOUDSTACK-4510 Move NS scripts to appropriate suites and related common.py fixes
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit f7df3ef9f1)
2013-08-27 10:59:13 +05:30
Prasanna Santhanam 02e7630f1c CLOUDSTACK-4452: Cleaning up the is_snapshot_on_nfs utility
- Pass in config and determine the mgmtsvr credentials
- remove the dir_paths param which is not reqd.
- pass all entities as uuids and convert to ids within the method

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-08-26 11:10:06 +05:30
Gaurav Aradhye 684573939f CLOUDSTACK-4452 - Fixed method is_snapshot_on_nfs and moved it to utils
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 35ee38385c)
2013-08-26 10:40:44 +05:30
Prasanna Santhanam 4a96740681 CLOUDSTACK-4473: allow_egress referenced before assignment
When deploying with networkids allow_egress gets no default value. This
is a regression caused by fix for CLOUDSTACK-4418

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 239ba9447109412cf48cfefc547d6b5fef83a4a9)
2013-08-23 17:02:34 +05:30
Prasanna Santhanam b3306497fe CLOUDSTACK-4453: fetch host credentials from marvin config
Tests would fetch the credentials for the host to hop into router to
check for essential services. Each test would require to put in the host
information into the test data. Instead fetch the credential information
from the marvin configuration file.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 4b546ce85d40098ade69c575316e76e25a422a12)
2013-08-22 20:20:16 +05:30
Sowmya Krishnan e4b5d4d62e CLOUDSTACK-3927, CLOUDSTACK-3928: Fix NS scripts to add/remove NS device in setupClass/tearDown
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 1c31ee22d4)
2013-08-22 10:27:56 +05:30
Prasanna Santhanam 7e36dd6be8 CLOUDSTACK-4418: allow egress access to all networks for VMs on NAT
Tests doing operations within guests post VirtualMachine.create w. default
network offering fail since the egress policy with isolated networks and
sourcenat services is DENY.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 81b5146c8e)
2013-08-21 15:20:27 +05:30
Prasanna Santhanam 61b787975c CLOUDSTACK-4283: remove hardcoded shared_vlan from test_shared_networks
SharedNetwork creation requires a pre-setup VLAN outside of the guest
VLANs defined during zone creation. Use a VLAN tag outside the guest
range as a shared network vlan to create shared networks.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 35867ccbd90c824b4faae438dac99400babeff95)
2013-08-13 15:31:43 +05:30
Prasanna Santhanam fb58addc35 Fix snapshot GC test
Changes to object store removes SecondaryStorage host from the host
table. This affects the test where we identify the secondary storage nfs
host to login to where we verify if the snapshot is present on the
physical resource.

* added base classes for imagestore/stagingstore operations
* fixed the test to check only if NFS is used as backing store

TODO: Fix for s3 based storage layer

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 4f973778bf6c2bfc7da2b8c785557e86f81af9d5)
2013-08-07 14:33:57 +05:30
SrikanteswaraRao Talluri c853426c76 CLOUDSTACK-4098: Fix the test which deploys a VM from password enabled template.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 64591e65be)
2013-08-06 18:46:44 +05:30
Prasanna Santhanam f37417e365 Fix pep8
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 1060997bcd40282656503a15191abd1814bfe064)
2013-08-05 14:06:51 +05:30
Prasanna Santhanam ccbcee5e00 isportable defaults to False
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-08-02 22:23:44 +05:30
Gaurav Aradhye 6ef51b4159 CLOUDSTACK:3594: Fixes related to affinity groups tests. Included two test suites those were missed because there was no decorator for test cases
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 3028f4223b2a00f80159b387e86211effcfeda11)
2013-08-02 16:00:05 +05:30
Chip Childers 8225374138 Updating pom.xml version numbers for release 4.3.0-SNAPSHOT
Signed-off-by: Chip Childers <chipchilders@apache.org>
2013-08-01 10:35:00 -04:00
Prasanna Santhanam d714ce54b5 CLOUDSTACK-3921: Don't change order of params in PublicIpAddress.create()
Or existing tests that depend on the order barf and fail.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit ec054cde23113baa40ae35c6195bc666d3c1b8d0)
2013-08-01 19:25:37 +05:30
Prasanna Santhanam d31fe92169 Remove unused code
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-08-01 16:10:57 +05:30
Sheng Yang a3df696fd6 Automation: Fix SSH connection test
And increase default retry to 10 times, each 30 seconds.

Comment out auth exception part because sometime it caused failure when VM
is starting up.

Now we just wait anyway for 5 minutes if it's failing.
2013-07-31 18:28:33 -07:00
Sheng Yang 536c377752 Marvin: Add missing __init__ to StaticRoute 2013-07-31 15:10:39 -07:00
Prasanna Santhanam cce14f3974 Include SSH retry logic when encountering channel failures.
Only on SSHExceptions we attempted retries, but during socket failures,
like Network Unreachable we failed the ssh connection immediately.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit dcbb2fcaa18e3faaf1c0765633300f7d0fa46d2c)
2013-07-31 17:20:35 +05:30
Prasanna Santhanam 476bd93e43 Cleanup the marvin logger
It is useful to have the same logger used in the marvinplugin into the
testcase. The testcase can then append its name to the logger making
the logs specify exactly which test is executing when the logs are
generated

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit c84a2d8801b7898dbfee18e17950cb9fca7d36e7)
2013-07-31 17:20:35 +05:30
Prasanna Santhanam 28835054e6 CLOUDSTACK-3921: AssociateIpAddress fails
When you have a method with named parameters, all calls to it should
explicitly pass name=value pairs. Not doing so can result in the wrong
params being passed in the method call. This is a result of the
"services" dictionary being passed into the isportable flag failing the
API signature generation and hence the API call.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-07-30 15:01:34 +05:30
Girish Shilamkar 4054a8e2ac CLOUDSTACK-3454: Fix test_portable_publicip
Added isportable param to associateIP API. Fixed base class
for PortableIP tio call portableip APIs.
Removed test_createPortablePublicIPAcquire from basic zone run
requires additional network creation handling which can be done
in component tests.

Signed-off-by: Prasanna Santhanam <tsp@apache.org>
2013-07-29 20:00:08 +05:30