listVirtualMachines queried using tags needs the tags (key,value) to be
passed as a dictionary.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit c2840b93f5)
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
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)
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>
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>
(cherry picked from commit 73de7646ffd289b779d5c45cfee84e07551c3a0c)
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>
(cherry picked from commit 35492a3668)
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>
(cherry picked from commit f064ad22dc)
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.
Skip is done using cloudstackTestCase inheriting from unittest as
self.SKipTest
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 797450a5ba)
- 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>
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>
(cherry picked from commit d66edb5e2b)
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>
(cherry picked from commit b88fe8233af699786de6d08a6bc981546538898d)
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>
(cherry picked from commit 8b8cee52f09f9f2c1b612ece16fc7ffc4b2c9176)
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>
(cherry picked from commit a3c21b3682db51986e7cc54d8f25b1dbf4ae9eaa)
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>
(cherry picked from commit c44fa2cc20)
Make the size explicit parameter to random_gen() for snapshot data
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 874d59847aa6bfd12e6967891a21f1746987b566)
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>
(cherry picked from commit dba09791fc99c342974d8ca1c69609c04f1a5512)
- SourceHost is an id
- Invalid format in debug() message corrected
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit ce4dd723ab486894523f08f48b6054f614be4d85)
when an account is deleted listAccounts raises an exception.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit aa9bc4413efad4eccdf8e336f66ea8c8168fbbda)
Test VPC Release IP assigned to LB rule fails because we don't pass in
the apiclient to it.
Signed-off-by: Prasanna Santhanam <tsp@apache.org>
(cherry picked from commit 2f0a0a61363074ffa959d7ff9596a1b07b2de40a)
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.
(cherry picked from commit 0c699394f246bc2b052a22d88b3f938ba8a72770)
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>
(cherry picked from commit a4d902fc9b48e963d5987965207e3745e1728741)