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>
1. correcting the log_test_exceptions decorator so nose testrunner can
detect and document the running test
2. correct the wait for network cleanup -> network.gc
3. Remove commented skips
4. Sanitize host passwords
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 04df5d289ca9394c7f647fb94e5289160a2d3467)
use a vlan outside of the vlans managed by cloudstack.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 7eefeab957e7b2a904a2f5ec1ce2a2624587eb9e)
this suite performs enable/disable of pods/clusters in the zone causing
other tests to fail deploying VMs. Move it to sequential execution suite
in the maintenance tests
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
There is no need to attach a volume to the VM when doing the
changeserviceOffering test. Condensed the test to verify that the VM's
offering is changed successfully when stopped and reflects when VM is
started.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
VirtualMachine.list( only gives the list of vms but not the vm object as
returned by VirtualMachine.create( any operations on the listed VMs then
fails. Explicit call to startVirtualMachine as in setUpClass fixes this.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
Currently we're using enable/disable static nat rather than create IP forwarding
command. The list of StaticNAT won't work.
Also change to use API to verify the rules rather than look into db directly.
1. You cannot check routers after your destroy them
2. You cannot use the old value in checking migration status.
3. You cannot upgrade VR to a non-system offering.
test_reboot_router.py was trying to ssh to vm using private IP
and hence it failed with error "No route to host."
Fixed the testcase to ssh using public IP.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 003721a085210afe167fbc734abe4cc8e2d3d23f)
ISO size can differ from the mounted packages within the ISO. Remove the
size check and only perform attach and detach in the guest.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit d875a36e5c)
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>
(cherry picked from commit 4054a8e2ac)
- Fix the test that ensures vpc offerings cannot be created with
firewall service
- CLOUDSTACK-3902 blocks the fix for vpc offerings created without
source nat service. Without source NAT VPC creation should fail.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit e4de46e4b1)
One of the VPC offerings (4) is deleted in the test before moving into
teardown. This offering should be removed from cleanup.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
PublicIpAddresses.list takes an associatednetworkid which lists all the
public ip addresses in the deployment's physical network associated with
the vnet.
Various tests failed because of `networkid` used in place of
`associatednetworkid`
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
Provided a utility method to ensure the config is suitable for running
the test. If vm.instancename.flag is unset then skip the tests.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
We delete the accounts within the domain thereby cleaning up resources.
Also corrected the typo in the test description. ForceDelete test was
called without force option and vice versa
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
We shouldn't clean up class-scope resources(_cleanup). We should create local
ones(cleanup) then clean it.
So:
cls._cleanup would be used for class-scope resources.
self.cleanup would be used for each unit.
This result in countless test case failure because many test case units didn't
clean up after it.
The regression test result should be improved quite a bit after this change.
- No need to cleanup offerings as they are lightweight entities
- Cleanup order changed to clean accounts, networks and then the domains
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
since random_gen can take an id to prepend a testcase identifier to the
created cloud resource, we need to explicitly specify the size param. In
vmsnapshot test create the random_data string of size (100)
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
Now password is sent as clear text as per CLOUDSTACK-1734
So changed marvin to handle this. Plus domainid was not
passed in the testcase and marvin used "domainid" instead of
"domainId" as a parameter. Fixed these two errors.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
During switchover of master->backup->master, the rvr goes through
UNKNOWN, FAULT, MASTER/BACKUP. In case of improper transition the state
is FAULT. Include additional state check in the tests.
The guestcidr is no longer specified directly. Use the netmask and
gateway combination to create custom cidr network
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
Also do not wait for cleanup. Just send cleanup to server and proceed
further.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 04d9d209ad)
We don't need to delete the volume resource as part of cleanup as the
storage gc is expected to cleanup orphaned volumes in deleted
accounts/projects.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
API call listprojectaccounts failed, and test case
test_projects.py:test_08_cleanup_after_project_delete failed due to
this
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
Emails belong to user's within accounts. Refer to the first user within
the account and use his email for account invites.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 22cb5be8b6)
VLAN ranges beyond 4095 are properly handled. Fix the test to accomdate
this change
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 3f5c1c6da3)
The network GC test is a long running test as it waits for
network.gc.wait + network.gc.interval * 2 to verify that the network
cleanups happen corectly. This test is also part of the regression suite
and is redundant in the smoke. The test run takes 30m longer because of
this test included in smoke.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 915f7b9c12fb816ff87558e2b95599e74b8e9462)
Added method to compare if two digits are almost equal within given range
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 048cf5aae8)
After you scale the vm, user is expected (oddly) to update the virtual
machine to signal that the PV tools are installed. This tells cloudstack
that the VM has been successfully scaled up/down.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
updateVirutalMachine fails if no args are passed. this test also further
verifies that the displayname is changed when vm is started up
successfully.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 9f0c9ee20c)
User Dispersion is a best effort of deployment planning and cannot
always guarantee the dispersion logic will work. However this does not
fail the deployment. Adjust the test so as to print a debug and exit
instead of fail.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 43f31dcc66)
Object is being used in one of the Assert statements however it is not
callable. Fixed it.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 479f89ca3c)
the nosetests runner will not discover tests unless the tests start with
the test_ prefix. When wrapping by the log_test_exceptions method the
name of the method changes and is not test_ anymore causing the runner
to skip the test. Included the __doc__ string as well so the runner can
report results correctly.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 2a51c3e2c9)
Prasanna Santhanam
Alena Prokharchyk
rayeesn
SrikanteswaraRao Talluri
Chirag Jog
Gaurav Aradhye
ashutoshkelkar
Sheng Yang
Bharat Kumar
Rayees Namathponnan
SrikanteswaraRao Talluri
Girish Shilamkar
Sowmya Krishnan
Sanjay Tripathi
Likitha Shetty
sailajam
sanjeevneelarapu
suresh sadhu