Commit Graph

21208 Commits

Author SHA1 Message Date
Koushik Das 269a4ef11e CLOUDSTACK-4855: Throttle based on the # of outstanding requests to the directly managed HV host (direct agents)
Cloudstack sends requests to directly managed HV hosts (direct agents) using the direct agent thread pool. The size of the pool is determined by global config direct.agent.pool.size defaulted to 500.

Currently there is no restriction on the number of threads a direct agent can use from this shared thread pool to send requests to the host. This is fine as long as the host is responding to requests
in a reasonable amount of time. But if there is a considerable delay in getting response, the thread remain blocked for that much time. As more commands are send to the slow host threads keep getting
blocked. This can eventually lead to a situation where requests to healthy hosts cannot be processed as there are not enough free threads.

The problem being addressed here is to localize the impact of few bad hosts, so that entire management server is not affected.

One such way is to throttle based on the # of outstanding requests on per host basis. The outstanding requests to a host will be a % of direct agent pool size. This is configurable based on
direct.agent.thread.cap. The default value is 0.1 or 10%, a value of 1 would mean the old behavior where there is no upper cap. This will ensure that the impacted host will be bound by a upper cap on the number of threads it can use to process requests and not the entire pool.
2013-11-04 14:52:26 +05:30
Min Chen 99ead3419c Fix S3 system vm start issues caused by recent storage refactor work. 2013-11-02 22:30:10 -07:00
Alena Prokharchyk 2ff791b0b8 CLOUDSTACK-5025: volumes.display_volume field should be set to 1 by default. DB used to handle this, but VO was missing the default field assignment. Fixed th
e VO and db upgrade to update all existing volumes with 1 value
2013-11-01 22:48:11 -07:00
Alena Prokharchyk 7a26894eb0 Changes to @Parameter - added RoleType[] authorized() field. The field defines who is authorized to put the parameter to the call. If the person is not authorized, the parameter will be ignored.
Also fixed CLOUDSTACK-5024 - added storageId to the list of the request parameters (available to the Root admin only)
2013-11-01 22:48:11 -07:00
Darren Shepherd c1bae7984a Revert "Commit 1460196496 changed db.properties"
This reverts commit a3cec3802c.
2013-11-01 22:25:53 -07:00
Min Chen 24af28290e CLOUDSTACK-5017: use LocalHostEndPoint in copying from S3 to cache store
during ssvm launch.
2013-11-01 18:07:00 -07:00
Min Chen 6eecb0b3b5 CLOUDSTACK-5017: If SSVM is unavailable DownloadCommands will be routed
to mgmt server.
2013-11-01 18:07:00 -07:00
Marcus Sorensen c2e2f6ce63 relax KVM agent setup to allow non-bridge traffic labels
(guest bridges are created on physical device, no need for bridge traffic label)
2013-11-01 18:16:54 -06:00
Edison Su 918bc74050 fix compile 2013-11-01 16:16:03 -07:00
Edison Su 143fbc37a6 fix build 2013-11-01 16:02:24 -07:00
Edison Su 31ec4191c0 Fix compile 2013-11-01 15:30:20 -07:00
Edison Su 7c99b88926 Don't use cast to PrimarydataStoreTO on the resource code 2013-11-01 15:13:16 -07:00
Edison Su b6576d7c98 fix: snapshot_detail table is missing, due to wrong merge 2013-11-01 12:45:05 -07:00
Pedro Marques 6b5fab2f5c OpenContrail network plugin
Signed-off-by: Hugo Trippaers <htrippaers@schubergphilis.com>
2013-11-01 17:46:52 +01:00
Girish Shilamkar 1e417994df CLOUDSTACK-3961: Added a function to get free vlan 2013-11-01 17:27:46 +05:30
Santhosh Edukulla 58cd71f5c1 Added a verifyElementInList utility function
The purpose is to verify a given element in list
at a given position with few error checks like
list type,empty list and position
Returns appropriate codes based upon the inputs.
Can be used under tests instead of multiple asserts

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-11-01 16:55:07 +05:30
John Kinsella 47641637f6 Summary: LibvirtStorageAdaptor.getPhysicalDisk() to recogize .raw extension
Detail: getPhysicalDisk() was not matching on volumes with .raw, so
instead setting disk format to QCOW2.

BUG-ID: CLOUDSTACK-5018
Bugfix-for:
Reviewed-by:
Reported-by:
Signed-off-by: John Kinsella <jlk@stratosec.co> 1383287538 -0700
2013-10-31 23:32:18 -07:00
Brian Federle 61b77086f4 VM snapshot action: add quiescevm option as checkbox 2013-10-31 16:55:15 -07:00
Edison Su c5bec6b336 add quiescevm in createvmsnapshotcmd 2013-10-31 16:45:05 -07:00
Brian Federle 2e07c452e3 Alerts UI: Implement multi-select delete/archive alerts 2013-10-31 16:40:01 -07:00
Brian Federle e8f8185393 Events UI: Implement multi-select delete/archive actions 2013-10-31 16:40:00 -07:00
Brian Federle ccab740129 List view: Fix refresh not loading multi-select column 2013-10-31 16:40:00 -07:00
Edison Su b704d5b8d3 add quiesceVm vm helper function 2013-10-31 16:24:48 -07:00
Brian Federle 0019e42ce0 Notification area: Show more detail in errors
-Show actual error message (if available) in the notification item

-Minor layout/padding tweaks
2013-10-31 16:17:52 -07:00
Brian Federle 1a7872d189 Notifications: New alert style
-Move corner alert to pop up under notification area, intead of in the
 window corner, for better visibility

-Give alert box tooltip appearance
2013-10-31 15:55:26 -07:00
Brian Federle 707419670a Region switcher: Increase max-width to prevent cutoff 2013-10-31 15:55:26 -07:00
Alena Prokharchyk 78b1ac4126 CLOUDSTACK-3868: fixed "count" param in listUsageRecords API to return the actual number of records matching the search criteria. Used to return the wrong value - the number of records per page
Conflicts:
	server/src/com/cloud/usage/UsageServiceImpl.java
	utils/src/com/cloud/utils/db/GenericDaoBase.java
2013-10-31 15:25:57 -07:00
Alena Prokharchyk 9c773fbd68 DestroyNetwork: fixed regression bug - the call always returns false after finishing the transaction. Return true instead. 2013-10-31 15:25:57 -07:00
Rajani Karuturi b436a82392 added group and domain params to importLdapUsers api call
Signed-off-by: Ian Duffy <ian@ianduffy.ie>
2013-10-31 22:06:32 +00:00
Jessica Wang 856703cc1c CLOUDSTACK-4758: UI > Instance Wizard > (1) pass cpuNumber, cpuSpeed, memory to API when custom area is shown (i.e. when selected compute offering is dynamic).
(2) pass rootDiskSize to API when custom area is shown (i.e. when selected item in step 2 is a template).
2013-10-31 13:56:42 -07:00
Nitin Mehta 03b27c02a3 CLOUDSTACK-5013:
Add the format for volumes table for uploaded volumes. Since volume_host_ref still exists, using that for populating it
Signed off by : nitin mehta<nitin.mehta@citrix.com>
2013-10-31 13:53:24 -07:00
Brian Federle 0db10ad55c VM snapshot: Make multi-select action 2013-10-31 13:39:40 -07:00
Brian Federle f930ccda99 Multiselect: Fix size and alignment on select column 2013-10-31 13:28:06 -07:00
Brian Federle b4b34edf15 Quick view tooltip: Fix offset 2013-10-31 13:18:59 -07:00
Brian Federle bfae24db1d Socket UI: remove old API calls 2013-10-31 12:10:04 -07:00
Brian Federle 16bb286729 Sockets: new UI flow
-Removes socket display from the main infra. dashboard and onto a
 separate page. This is due to performance issues with the API calls
 that query each hypervisor's socket data.

-Convert socket display to a list view, for simplicity/consistency.
2013-10-31 12:03:20 -07:00
Jessica Wang c090a2df7a CLOUDSTACK-730: UI > VPC > Router > site-to-site VPN > VPN Connection > Create VPN Connection - add new checkbox "passive", default it as unchecked. 2013-10-31 11:34:53 -07:00
Brian Federle 152e23a00b UI plugins: Fix side nav behavior
Due to usability issues, any UI plugins that add a main section (via the
addSection function) will now be shown in the 'Plugins' area by clicking
on the relevant tile. This is to prevent overflow of the side nav bar
caused by too many plugins being loaded.
2013-10-31 11:27:50 -07:00
Girish Shilamkar addf5b0f8c CLOUDSTACK-4256: Fixed Shared Network test cases using random private
ips for shared networks
2013-10-31 22:21:00 +05:30
Gaurav Aradhye 23499efe42 CLOUDSTACK-4995: Fixed test_reset_ssh_keypair which checks the state of the VM
Signed-off-by: Girish Shilamkar <girish@clogeny.com>
2013-10-31 21:57:57 +05:30
Darren Shepherd afaf6370dd Fix change that broke db.properties.override 2013-10-31 09:21:34 -07:00
Mike Tutkowski 449b5daa70 VMware changes related to managed storage 2013-10-31 08:30:50 -06:00
Hugo Trippaers a3cec3802c Commit 1460196496 changed db.properties
loading, this also affects the database creator. This make the database
creator use the new loading completely instead of partially.
2013-10-31 15:08:25 +01:00
Girish Shilamkar 58179d2884 CLOUDSTACK-4335: Delete test_deployVmSharedNetworkWithoutIpRange
test_deployVmSharedNetworkWithoutIpRange creates a shared network
without start and end ip. This is no longer permitted and creation
of shared network will fail with error "insufficient capacity". So
remove this test which is invalid.

Signed-off-by: Girish Shilamkar <girish@clogeny.com>
2013-10-31 18:26:14 +05:30
Santhosh Edukulla 0b617a13fd Fixed Bug: 4899
Added Configuration Support to Marvin.

1. It provides the basic configuration facilities to marvin.

2. User can just add configuration files for his tests, deployment
              etc, under one config folder before running their tests.
              cs/tools/marvin/marvin/config.
              They can remove all hard coded values from code and separate
              it out as config at this location.
              Either add this to the existing setup.cfg as separate section
              or add new configuration.
3. This will thus removes hard coded tests and separate
              data from tests.

4. This API is provided as an additional facility under
              cloudstackTestClient and users can get the
              configuration object as similar to apiclient,dbconnection
              etc to drive their test.

5. They just add their configuration for a test,
              setup etc,at one single place under configuration dir
              and use "getConfigParser" API of cloudstackTestClient
              It will give them "configObj".They can either pass their own
              config file for parsing to "getConfig" or it will use
              default config file @ config/setup.cfg.
6. They will then get the dictionary of parsed
              configuration and can use it further to drive their tests or
              config drive
7. Test features, can  drive their setups thus removing hard coded
          values. Configuration default file will be under config and as
              setup.cfg.
8. Users can use their own configuration file passed to
              "getConfig" API,once configObj is returned.

Signed-off-by: Santhosh Edukulla <Santhosh.Edukulla@citrix.com>
2013-10-31 17:54:13 +05:30
Wei Zhou 4495d456ed CLOUDSTACK-4830: fix build error 2013-10-31 11:50:01 +01:00
Wei Zhou 382391f270 CLOUDSTACK-4987: only shared network and isolated network with same account can be assigned to a VM
(cherry picked from commit 743eed8771)
2013-10-31 11:37:31 +01:00
Wei Zhou bad23908c2 remove unused file lock codes from LibvirtStorageAdaptor.java
(cherry picked from commit 4b8d636ce6)
2013-10-31 11:37:25 +01:00
Wei Zhou 57036e379b CLOUDSTACK-4831: allow create network by domain admin
(cherry picked from commit 46e4cfd3c3)
2013-10-31 11:37:18 +01:00
Wei Zhou 5109498783 CLOUDSTACK-4830: allow create account and user by domain admin
(cherry picked from commit 0d12e3eb9d)
2013-10-31 11:36:54 +01:00