Alex Huang
16caf1954b
New ip address states and mode
2010-12-04 17:32:32 -08:00
Alex Huang
daa9add158
removed router stuff from network manager
2010-12-04 17:32:31 -08:00
alena
150d684e05
1) isShared attribute belongs to Network now, not NetworkOffering
...
2) Added list by networkId support to listVirtualMachines command.
3) Implemented search by zoneId in listNetworks command
2010-12-03 19:07:08 -08:00
alena
2759e422f5
Return type/trafficType in the listNetworks response
2010-12-03 13:48:43 -08:00
alena
9f11f52611
listNetworks - don't return Networks created from systemOnly network offerings
2010-12-03 12:31:30 -08:00
Alex Huang
96ac136b8c
changed getAccountId to getEntityOwnerId
2010-12-03 11:59:45 -08:00
Alex Huang
b859e103d8
ip nat rule forwarding done
2010-12-03 08:48:03 -08:00
Alex Huang
223688d0bf
Port forwarding rules and load balancing rules rewrite
2010-12-03 08:48:02 -08:00
nit
94cb30a745
bug 7412 : UpdateTemlate - Adding some fields keeping it consistent with listTemplates
...
status 7412: resolved fixed
2010-12-03 20:06:16 +05:30
nit
b543faeeee
bug 7366 : Removing the bootable field from "createTemplate","registerTemplate","listTemplate" and "updateTemplate" commands.
...
Making some db changes for a successful ant deploydb
status 7366: resolved fixed
2010-12-03 18:31:08 +05:30
nit
d7b8e93f22
Merge branch 'master' of ssh://git.cloud.com/var/lib/git/cloudstack-oss
2010-12-03 14:32:22 +05:30
nit
f5279e74a6
bug 7403 : Correct NPE for ListIso command.
...
status 7403: resolved fixed
2010-12-03 14:31:38 +05:30
alena
f8a687ab23
1) Implemented create/list/deleteNetworkOffering create/delete/listNetwork APIs.
...
2) Added networkIds parameter to deployVMCmd - accepts list of networks ids separated by coma.
3) Changed domainRouter/systemVm/userVm response to return list of Nics associated with the vm.
2010-12-02 19:30:17 -08:00
abhishek
d0fc07496f
search functionality for ip forwarding rules, with covering of cerner cases
2010-12-02 09:46:32 -08:00
will
cd5afb3d7e
bug 6969: Fixed snapshots, templates, isos, routers, volumes, and system vms async commands to make use of the new job id/job status when listing objects with pending jobs.
2010-12-01 16:50:56 -08:00
anthony
17f3239f07
bug 7368: should get volumeVO even if it is removed
...
status 7368: resolved fixed
2010-12-01 16:11:47 -08:00
abhishek
a477d6c33c
adding support for updating service and disk offerings, also adding domain and domainid back in the responses to the CRUD methods for both service and disk offerings
2010-12-01 10:40:08 -08:00
abhishek
9a6eaaf21f
this is a good point to commit the code; have the basic domain specific service offering working
2010-12-01 09:53:13 -08:00
will
c67d26cec4
bug 6969: First step of now displaying pending async jobs for listXXXCommands. There is a lot more cleanup and fixing to do but all commands acting against VirtualMachines now work.
2010-11-30 19:12:09 -08:00
alena
680812f675
bug 7223: page and pagesize parameters must be specified together "list" api commands
...
status 7223: resolved fixed
2010-11-30 11:31:34 -08:00
Kelven Yang
54f177cacc
1) Make generic dao be able to persist UTF-8 string for internationalization support
...
2) Undo gson Unicode escape in API response object to avoid double escaping which can break Javascript from getting correct text content
3) Correct API layer in dealing with character encoding
4) Remove double escape in cloud.core.js
2010-11-30 09:40:54 -08:00
anthony
d3325de5ae
1. limit xs-tools in xenserver related java file
...
2. don't populate xs-tools to template_host_ref
3. introduce a new template type PERHOST, which doesn't need to be populated to secondary storage
2010-11-29 18:21:08 -08:00
abhishek
bdf4426609
fixing the same issue in another command (found whilst checking references)
2010-11-29 13:38:06 -08:00
abhishek
1a0ec2a00f
bug 7346: fixing the corner case where list ip forwarding rules is called for a rule, which is yet to be expunged, but the vm is destroyed. if you call list here, an npe results
...
status 7346: resolved fixed
2010-11-29 13:35:04 -08:00
alena
aa8aa6027a
Modified UserContext - got rid of accountName, domainId fields as we can get this information from Account field.
2010-11-24 14:29:08 -08:00
alena
44737d24c5
bug 7196: fixed response object generation for authorizeNetworkGroupIngress command (response type is Network group response - the way it was in 2.1.x - instead of IngressRule response)
...
status 7196: resolved fixed
Also fixed revokeNetworkGroup command - didn't work via 8096 port
2010-11-24 09:50:04 -08:00
nit
8ea124ae79
bug 7046 : extracting templates should allow for exclusion of some templates. An attribute "extractable" would be set on templates by the template owner at upload time. It would specify if we allow the template to be extracted from the system. Default would be true ie "allow extraction". If set to false, it would not be possible to extract the template or any derivatives of the template. For example, volumes, snapshots or templates from that template.
...
status 7046: resolved fixed
2010-11-24 18:37:54 +05:30
alena
cb74030d01
bug 7298: throw an Exception when invalid job Id is specified in queryAsyncJobResult
...
status 7298: resolved fixed
2010-11-23 15:06:29 -08:00
alena
a646745516
bug 7227: applying Alex's fix - different way of desirializing List objects.
...
status 7227: resolved fixed
2010-11-22 14:08:43 -08:00
alena
6765e97976
Made ApiResponseHelper to use Long instead of long for response parameters to eliminate possible NPEs during building an api response.
2010-11-22 11:27:15 -08:00
alena
cdea2d210d
Better logging for api exceptions.
2010-11-22 11:27:15 -08:00
Alex Huang
1e44925564
api refactoring
2010-11-21 12:00:21 -08:00
alena
138e0914b6
bug 7245: don't try to set vm name for detached volume during expunge process.
...
status 7245: resolved fixed
Fixed couple of other problems:
* cleanup corresponding records in load_balancer_vm_map table when loadBalancerRule is removed.
* don't log "java.lang.IllegalStateException: getAttribute: Session already invalidated" exception when UI session expires
2010-11-19 20:08:18 -08:00
alena
cec8ebd5cd
bug 7236: don't catch AsyncCommandQueued exception in ApiDispatcher
...
status 7236: resolved fixed
2010-11-19 18:40:32 -08:00
alena
a5dfaaf31b
API exception handling - print api command name to the log.
2010-11-19 15:48:58 -08:00
Alex Huang
5b4a6f27db
Changed networkconfigurations to network as per Sheng's suggestion
2010-11-19 15:44:42 -08:00
alena
8362d0feba
Reverting back commit e5c137e303 as it wasn't meant to be in 2.2 RC1.
2010-11-19 14:23:39 -08:00
alena
e5c137e303
bug 7223: pagesize and page should be specified together in "list" api commands
...
status 7223: resolved fixed
2010-11-19 14:03:44 -08:00
Chiradeep Vittal
394554770e
Enforce user limit on number of vpn users
2010-11-18 18:38:29 -08:00
alena
0b2b8e31d8
Fixed updateUser command - didn't work correctly via 8096 port (user account info was taken from UserContext instead of User object).
2010-11-18 17:07:20 -08:00
abhishek
6ae21932eb
realized that the var name used in response code might be confusing to a new reader. changing the name to be more intuitive
2010-11-18 11:20:01 -08:00
abhishek
18ff23c839
added a list method to list ip forwarding rules, optionally list a rule based on its public ip address
2010-11-18 11:15:13 -08:00
abhishek
020afa6e33
adding a list command for ip forwarding rules for 1:1 nat, to be consumed by the ui
2010-11-18 11:15:13 -08:00
abhishek
c4f1b35aef
returning the public ip address in the creation of firewall rule, for the UI to consume
2010-11-18 09:42:47 -08:00
alena
91a87f9d8f
bug 7155: fixed NPE in AuditTrial which used to happen when logout command was executed when session was already invalidated.
...
status 7155: resolved fixed
2010-11-17 21:42:51 -08:00
alena
a95b853a17
bug 6898: updateResourceLimit command requires domainId or account/domainId now if executed via 8096 port
...
status 6898: resolved fixed
2010-11-17 19:43:06 -08:00
alena
97a8349110
bug 6894: removed "account" request parameter in listAccounts command. Should use "name" if need to make search by account name
...
status 6894: resolved fixed
2010-11-17 19:21:02 -08:00
Alex Huang
a73e3dc9ac
new networking can be set by use.new.networking
2010-11-17 17:51:17 -08:00
Chiradeep Vittal
9640f12cce
bug 6971: revert some changes
2010-11-17 15:33:10 -08:00
Chiradeep Vittal
31e17b907d
bug 6971: ensure no conflicts with portforwarding rules for vpn udp ports
...
also, do not open up port 1701 -- only used on ppp interface not public interface
also clean up password generator, make it easier to use and more secure
moved some constants to NetUtils
heuristic to determine if an ip is one-to-one nat'ted
2010-11-17 15:32:50 -08:00
alena
2184dd1af2
bug 7195: introduced search by forVirtualNetwork parameter in listVmsCmd. Added "forVirtualNetwork" param to responses for all VM's apis.
...
status 7195: resolved fixed
2010-11-17 14:11:16 -08:00
alena
dae39942f1
bug 7192: Added networktype parameter for the zone
...
status 7192: resolved fixed
1) Zone has networktype parameter now, 2 values are supported: Basic and Advanced. "networktype" field is created in data_center table. The parameter is being set at creation time; we don't allow to modify it with updateZone command.
2) Only vlan of Untagged Direct type can be created in Basic network zone; any other vlan except for Untagged Direct can be created in Advanced zone
3) Allow NULL guest vlan range for the zone. Only vlan of Direct type can be created in zone with NULL guest vlan.
4) "Default" zone is Basic by default.
5) Changed "vnet" parameter to "vlan" in updateZone command to be consistent with createZone
2010-11-16 16:54:44 -08:00
abhishek
10cfe07970
bug 6793: returning the right msg to the ui when we try to log in to a locked account
2010-11-16 14:15:08 -08:00
abhishek
e5e128545d
removing the tags support in the update commands for disk and service offerings
2010-11-16 13:25:25 -08:00
alena
c5d78a726e
Modified exception handing in API. Following exceptions are supported now (location - BaseCmd.java):
...
* MALFORMED_PARAMETER_ERROR - when type of the request parameter is invalid (String instead of Long for id for example)
* PARAM_ERROR - when invalid parameter value is specified in api request. For example, id of non existing vm for StartVmCmd
* ACCOUNT_RESOURCE_LIMIT_ERROR - when user tries to exceed his resource limits by executing the api command.
* INSUFFICIENT_CAPACITY_ERROR - when resource fails to create/start due to insufficient capacity.
* RESOURCE_UNAVAILABLE_ERROR - when user tries to create a vm when storage is not available.
* RESOURCE_IN_USE_ERROR - when user tries to delete/modify resource while it's in use. For example, when we try to delete a network group when it contains ingress rules.
* NETWORK_RULE_CONFLICT_ERROR - when LB/PF rule to add conflicts with existing rule
* ACCOUNT_ERROR - when user is not authorized to execute operation on the resource.
* INTERNAL_ERROR
2010-11-16 11:56:34 -08:00
abhishek
92e3dc91c4
found a bug in the code where we were not decoding the params sent in eg. "a b" was being stored in the db as 'a%20b'. Fixing the same to have the right val persisted to the db
2010-11-16 11:46:55 -08:00
Alex Huang
ed31497188
ComponentLocator changes
2010-11-16 04:28:15 -08:00
Alex Huang
17afeade00
better componentslocator
2010-11-16 04:28:14 -08:00
alena
d43a8e983c
Fixed authorizeNetworkGroupIngress api command (it stopped working after API refactor)
2010-11-15 17:44:51 -08:00
alena
1a8d9a92a7
Modified all api calls to throw ApiServerException if null object is returned by the manager.
2010-11-15 17:44:51 -08:00
abhishek
d5ab597fcf
bug 6474,6475: porting over the expunge fixes for port forwarding rules to 2.2
...
status 6474,6475: resolved fixed
2010-11-15 17:33:05 -08:00
abhishek
c115a2e401
bug 7189: returning vmdisplayname if available for all port fwding crud cmds
...
status 7189: resolved fixed
2010-11-15 14:55:03 -08:00
abhishek
d520f0af5b
adding a column in user_ip_address table called one_to_one_nat; which is a boolean to tell us if the ip is a nat ip. This is false by default, and is set to true when we nat the ip. Also, this value is returned in the list public ip addrsses api call.
...
finally, the value is set to false when we delete the ip forwarding rule.
2010-11-15 12:39:24 -08:00
abhishek
6d943abf55
bug 7119: code refactor away from the ms calls, to the share and prepare methods based on alex's recommendations
...
status 7119: resolved fixed
2010-11-15 11:17:36 -08:00
alena
a01aad84ed
bug 7162: added search by id to listZones command
...
status 7162: resolved fixed
2010-11-15 10:48:36 -08:00
alena
050eed742f
1) Removed account/domainId info from deleteSnapshot and deleteSnapshotPolicy commands as we get account info from the snapshot/policy object.
...
2) Introduced responseObject field in @Implementation for all apis. This field contains information about the response object class, so we can used it for generating API documentation.
2010-11-12 17:36:17 -08:00
abhishek
fc461abed1
bug 6755: adding checks for commands before they hit the ms; to ensure they are valid. Also returning the right error msg if the command is not valid
...
status 6755: resolved fixed
2010-11-12 13:55:06 -08:00
abhishek
c91c977fba
bug 6290: we will be returning the service offering details for a root disk in the list volumes cmd, as opposed to the disk offering which was misleading
...
status 6290: resolved fixed
2010-11-12 11:24:34 -08:00
abhishek
673ddb246a
bug 6832: returning the missing api value for destroyed volumes
...
status 6832: resolved fixed
2010-11-12 10:55:13 -08:00
nit
dd8f02589d
bug 6837 : UpdateIso and UpdateTemplate API should return ostypename. Removing incorrect domainId creeping in because it was of primitive type.
...
status 6837: resolved fixed
2010-11-12 18:08:43 +05:30
nit
5215f6f817
bug 7101 : DetachISO API and AttachISO API would start returning an embedded object of virtualmachine so that the UI could use that to update the vm.
...
status 7101: resolved fixed
2010-11-12 16:40:55 +05:30
alena
6e0b500a83
bug 4964: made listDomainChildren work with "isRecursive" flag
...
status 4964: resolved fixed
2010-11-11 14:32:36 -08:00
abhishek
1faaba8fb0
fixed a corner case; also changed the commands to be async and changed the method signatures to conform with the master refactor
2010-11-11 13:07:26 -08:00
abhishek
26b75fc05d
this is a good point to check the code in; have completed a part of the enhancement's code; with create and delete ip forwarding rule commands and the skeleton for port to rule map with associated table
2010-11-11 13:07:26 -08:00
abhishek
e5ca5dddf9
renaming the ip forwarding commands to port forwarding commands, and beginning to work on Keshav's request to modify the current 1-1 NAT to be compatible with the Amazon apis
2010-11-11 13:07:26 -08:00
Alex Huang
05ce97f918
Merged in alena's changes
2010-11-11 12:53:19 -08:00
Alex Huang
284720bf00
compilation errors fixed
2010-11-11 12:53:19 -08:00
Alex Huang
c6033baa78
system vm start and stop added
2010-11-11 12:53:19 -08:00
alena
86c8f86a56
bug 6192: use "ipaddress" instead of "publicip" in updatePortForwarding command
...
status 6192: resolved fixed
2010-11-11 12:12:29 -08:00
alena
86e83bc4bd
Created new response objects for userVm/domainRouter/systemVm that going to be used by API after network refactorig is in place.
2010-11-11 10:43:18 -08:00
alena
b1194e68f7
Fixed the build
2010-11-10 10:14:08 -08:00
nit
0597c0cf4b
bug 4177: RegisterIsoCmd and RegisterTemplateCmd API - Added account and domainId params so that one could register them on behalf of other accounts.
...
status 4177: resolved fixed
2010-11-10 17:47:59 +05:30
alena
3954c7550a
bug 6914: return password in deployVirtualMachine/resetVmPassword response.
...
status 6914: resolved fixed
2010-11-09 11:18:54 -08:00
alena
5fafef9970
More api refactor - build api response is execute() method instead of getResponse method.
2010-11-09 09:58:05 -08:00
abhishek
7262b854a5
bug 7121: made the disk offering id optional, and added a check to ensure we never have both disk offering and snapshot ids to be null
...
status 7121: resolved fixed
2010-11-08 17:59:26 -08:00
alena
47e365f5ea
bug 7089: return embedded object name in xml response
...
status 7089: resolved fixed
2010-11-08 13:59:14 -08:00
will
551e7e4b82
bug 6812: fixed default pagesize of all listXXX command to unlimited for now.
2010-11-08 13:15:02 -08:00
Alex Huang
c8a3ea3080
merge conflict with alena's changes
2010-11-08 12:40:38 -08:00
Alex Huang
734cd0b1b1
fixed some bugs
2010-11-08 12:29:37 -08:00
alena
1c24deda34
1) Refactored current Apis not to use reflection for method invocation. In new version following has to be done:
...
* Implement callCreate() and execute() methods in apis extending BaseAsyncCreateCmd (example - CreateVolumeCmd)
* Implement execute() method in apis extending BaseCmd and BaseAsyncCmd (example - deleteVolumeCmd and deleteUserCmd)
We no longer need createMethod(), method(), manager() fields in @Implementation annotation, cleanup has to be done.
2) Moved Account/User related methods to AccountManagerImpl. Methods are exposed via AccountService interface.
3) Enhanced exception handing for Api calls.
4) Created ConfigurationService interface, all Api config methods are exposed via it.
2010-11-08 11:27:36 -08:00
nit
02781422b5
bug 6055: createDiskOffering: remove domainId parameter.
...
status 6055: resolved fixed
2010-11-08 13:41:33 +05:30
nit
4dc8837dc8
bug 6952: listISO should return "ispublic" property.
...
status 6952: resolved fixed
2010-11-08 12:33:23 +05:30
will
89c717703f
Fixed API commands that extend BaseCreateAsyncCmd to return the correct API response of id, and jobid.
...
Added the default public constructor back in the SuccessResponse to make sure deserialization works from the asyncjob table.
Fixed a issues with the VPN tab. It should now make use of the corrected API format.
2010-11-06 16:30:08 -07:00
will
ca0ce93fde
Fixed all sync commands that can possibly return a <success> tag. If success, it will return true, otherwise an HTTP error will be returned.
2010-11-05 21:25:19 -07:00
will
563b45a220
bug 7059: Fixed the async output to use the object name
2010-11-05 20:58:00 -07:00
anthony
224da7e775
several snapshot fixes
2010-11-05 20:55:19 -07:00
will
bcb53ec363
bug 7059: Fixing adding primary and secondary storage to use new api output.
2010-11-05 20:30:55 -07:00
will
a3dbc34857
bug 7059: refactored the API to now add the single embedded object tag after a response
...
- Changed networkgroups to security groups
- a whole bunch of other API bugs. Unfortunately, I may have broken more.
2010-11-05 20:10:57 -07:00
will
05ab3bf673
bug 7063: AsyncJobResult should now return the correct success and error tags upon the completion of a async job. It should now follow either the format ... jobresult.success or jobresult.errorcode and jobresult.errortext
...
- Fixed a lot of other Async related packaging issues. It's not all done but will continue to refactor this later.
2010-11-05 17:30:03 -07:00
Alex Huang
4555fcd73f
changed everything back so everyone can test
2010-11-05 16:10:22 -07:00
Alex Huang
5f90a8adb0
Checking in to do git pull...stupid git
2010-11-05 16:10:22 -07:00
alena
e2aa4738ec
listVirtualMachine api: return "Not created" as a root volume type when volume's storage pool id is NULL
2010-11-05 16:03:37 -07:00
alena
c0e2986766
bug 6958: return state for Account api commands
...
status 6958: resolved fixed
2010-11-05 15:48:05 -07:00
alena
a9166929f0
Fixed NPE in listVolumes/listVirtualMachines commands - do searchIncludingRemoved when get diskOffering/serviceOffering info.
2010-11-05 14:46:08 -07:00
alena
b9ad299ac6
bug 6998: return embedded job result object in queryAsyncJobResult command
...
status 6998: resolved fixed
2010-11-05 12:03:01 -07:00
will
b9fc4137f7
Bug 6988: Fixed issue with queryAsyncJobResult returning jobResults in all String format. This is to mirror the same way that the async commands are being serialized.
...
- Changed client.war to now include all the thirdparty files
- Fixed addVpnUser command to return a vpnuser instead of the response
- Added vmware jar files to the development build.
2010-11-04 20:56:28 -07:00
will
70b4c4d181
Bug 6988: addVpnUser did not camel case the "username" which resulted in that particular value not being returned.
2010-11-04 20:56:23 -07:00
Chiradeep Vittal
5c445bd99e
bug 6991: add presharedkey to response
...
status 6991: resolved fixed
2010-11-04 15:40:24 -07:00
abhishek
03ea885dcd
making disk offering id mandatory for volume creation
2010-11-04 15:30:23 -07:00
abhishek
d96a1a841d
adding logic to prevent commands from being sent to the pool when it is NOT up; also adding better thread safety and db locking, along with a general refactor
2010-11-04 13:35:57 -07:00
nit
7dc7a4838a
bug 4286: Fixing the extract functionality for Upload after new API framework. Also added a check to avoid system created templates and ISO's from getting extracted as per Mike.
...
status 4286: resolved fixed
2010-11-04 19:09:38 +05:30
will
f907e370d1
- Added the rest of the VPN UI feature with the exception of displaying the preshared key.
...
- AddVpnUserCmd will now return the embedded object.
- Fixed RemoteAccssVpnResponse to return "iprange" instead of "ipRange"
2010-11-03 22:24:22 -07:00
abhishek
84ee08430a
making changes to the list zones flow, to add the domain name to the table, and return it
2010-11-03 22:08:32 -07:00
Alex Huang
9bb05d3ef8
harmony amongst joins
2010-11-03 21:18:31 -07:00
Alex Huang
e27bb550fe
Harmony among gurus
2010-11-03 21:18:31 -07:00
Jessica Wang
3deb7580bb
Create Disk Offering - make disksize optional
2010-11-03 19:48:35 -07:00
abhishek
2303afe55c
making changes for disk offerings with a new column called customized being added, for the UI to cosume whilst deploying vm with an arbitrary volume size. The UI will use this value to set the right size and the offering will ensure the right tags are taken.
2010-11-03 17:45:52 -07:00
alena
2939e90d66
Build response the same way for create/list/update IPForwardingRule.
2010-11-03 15:49:51 -07:00
abhishek
7674df6c7a
fixing the merge conflicts
2010-11-03 15:39:16 -07:00
abhishek
46497e3aad
making changes for the private/public zones enhancement, for the UI to consume. The list method has been changed to return visible zones based on the user's credentials, or if the domainId for private zone listing is specified. In the update zone cmd, we do not allow updating private zone once it has been associated with a domain. This might change in the coming releases.
2010-11-03 15:39:16 -07:00
alena
6bd87ca161
Return "status" field in StoragePoolResponse. Status can be Up, Maintenance, ErrorInMaintenance
2010-11-03 14:59:00 -07:00
alena
52a8370102
bug 6955: return embedded object for PreparePrimaryStorageForMaintenance
...
status 6955: resolved fixed
2010-11-03 14:30:35 -07:00
Chiradeep Vittal
e1cb94831f
VPN users feature complete
2010-11-03 14:02:13 -07:00
Chiradeep Vittal
0dd19c197b
VPN users API -- WIP
2010-11-03 14:02:13 -07:00
Chiradeep Vittal
14613415e1
VPN users API -- WIP
2010-11-03 14:02:13 -07:00
Chiradeep Vittal
b6d3b3c184
users for vpn -- data model
2010-11-03 14:02:12 -07:00
alena
4a645cdeb0
Return embedded Volume object for create/attach/detach Volume commands.
2010-11-03 13:07:06 -07:00
alena
8c0e2fa7ec
Fixed updatePod Api to return embedded object instead of "success" response
2010-11-03 11:18:51 -07:00
alena
886fd4e754
More api refactoring - move response building for LoadBalancer/IpAddress commands to ApiResponseHelper.
2010-11-03 10:11:59 -07:00
alena
ebdc1f069d
1) Return embedded Host object in PrepareForMaintenance command.
...
2) Build HostResonse object in ApiResponseHelper for add/reconnect/update/prepareForMaintenance/cancelMaintenance commands
2010-11-02 20:30:41 -07:00
alena
abb8f45667
1) Return embedded object for upgradeRouter command
...
2) Build response objects in ApiResponseHelper for stop/start/reboot/list router/systemVm commands
2010-11-02 19:47:14 -07:00
Alena Prokharchyk
694a8231ef
Build responses in ApiResponseHelper for the deploy/start/stop/reboot/recover/listVm create/list snapshot/snapshotPolicies
2010-11-02 18:16:45 -07:00
abhishek
c6809f89d4
bug 6890: preventing users from searching for system account details via the list api
...
status 6890: resolved fixed
2010-11-02 17:59:24 -07:00
alena
571ca42c44
1) Return embedded object for addCfg/updateCfg/listCfg.
...
2) Fixed NPE in updateCfg command (used to happen when no value was specified)
3) Fixed addCfg command to call correct getName method while setting config name (used to call the method returning the command name)
2010-11-02 15:26:33 -07:00
Alex Huang
5fe5450abc
even more refactoring
2010-11-02 15:25:07 -07:00
abhishek
d4b33f523d
bug 5190: changes from the ui perspective; we upload the entire cert as a string, instead of reading it from a file. Then, we process the cert via ip streams instead of fileinputstreams. Moved the cert validation to before cert persistance, and also added more logging and exception catch blocks to deal with IO exceptions that might arise out of the ip stream processing for the new change.
2010-11-02 14:48:05 -07:00
alena
0c044a10d1
Moved response building for ServiceOffering/ResourceLimit commands to ApiResponseHelper class.
2010-11-02 11:59:10 -07:00
abhishek
85f78d8cb3
bug 6886: fixing the npe which occurs when we try to delete a non existant user. We were trying to return user details for an async job desc assuming the user always existed. Changing that logic to ensure when we don't have such details, we log the apt message
...
status 6886: resolved fixed
2010-11-02 11:20:52 -07:00
alena
6c3256d1e4
1) Return embedded objects for enable/disable/lock/update Account/User and updateDomain
...
2) Added "haschild" return parameter to Domain response.
3) Fixed updateDomain command - used to set domainname to "updatedomainnameresponse" value if no name was specified in the request. Added more logging for updateDomain command.
2010-11-01 23:49:57 -07:00
alena
5d4e4be974
1) Created ApiResponseHelper class - all api response objects should be built there.
...
2) Removed isMirrored tag from the update/create/listDiskOffering commands
2010-11-01 23:49:57 -07:00
abhishek
45f4499c14
bug 6836: fixing the api responses for disk offerings. We were using the wrong objext name, and were only returning success responses. Now, we are constructing the apt object and returning the entire object back.
...
status 6836: resolved fixed
2010-11-01 13:24:28 -07:00
abhishek
f70b55daa2
bug 6380: removing the code which returns hostid and hostname, as on a stopped vm, we do not have any association of the vm to the host
2010-11-01 12:16:53 -07:00
root
bee976b5f5
bug 6779: returning the deviceId wherever possible, in the xxxVolume() calls
...
status 6779: resolved fixed
2010-11-01 11:12:03 -07:00
Chiradeep Vittal
1e0eb04442
Add VPN apis, some systemvm changes for vpn
2010-11-01 10:27:49 -07:00
abhishek
c74fd77730
putting in final comments, so that the code to be checked in is better readable for others trying to understand it
2010-11-01 09:06:32 -07:00
abhishek
362e67d724
adding failover protection case for when the mgmt svr crashes in the middle of executing something; along with custom response for cert
2010-11-01 09:04:51 -07:00
root
3752a4eb31
making the logic more robust, testing more corner cases, improving efficiency and changing the schema
2010-11-01 09:04:51 -07:00
kishan
3a0f30a049
bug 6408: remove security groups/port forwarding service in virtual network case
...
status 6408: resolved fixed
2010-11-01 14:28:03 +05:30
Kris McQueen
24551eea3a
Add constants for API command parameter names and for API response parameter names. Update the commands with the new constants rather than custom strings. Update several of the responses with the constants. Not all the responses have been updated, this is still work in progress...
2010-10-29 17:20:40 -07:00
Kris McQueen
a9349cb815
bug 6857: if the dispatcher recieves an IllegalArgumentException, bubble that up to the response so the caller gets a useful message
2010-10-29 14:07:23 -07:00
Kris McQueen
88dac13bb0
use the API response date format when serializing response objects
2010-10-29 13:23:29 -07:00
will
9cc5aaa978
Fixed listTemplates to use the template type.
2010-10-29 13:17:32 -07:00
Kris McQueen
ebac4539d0
it's possible createSnapshotDB will return null, handle that case. However, if createSnapshotDB returns null because there was no work to do (empty snapshot) we should probably throw an exception that reports that fact instead of returning null and throwing an internal error
2010-10-28 17:52:37 -07:00
Kris McQueen
642b8a56f4
If the create command returns null for whatever reason, throw an exception so it aborts scheduling the async job to act on the created object
2010-10-28 17:52:37 -07:00
Kris McQueen
2f9e91a9ae
bug 6738: honor groupid when listing vms (restrict the list to just that group)
...
status 6738: resolved fixed
2010-10-28 16:27:07 -07:00
Kris McQueen
7531525252
Add commented code out for the old style (2.1.x) JSON async job responses. The XML responses are still a TODO. In order to get this to fully revert back to 2.1.x style responses, some commands will need to be modified to set the responseName to be something like getResultObjectName(), and there's an example of this in StopVMCmd.
2010-10-28 14:32:13 -07:00
Kris McQueen
9eb851db60
bug 6809, 6712: standardize the domain router response to use guestmacaddress instead of macaddress
...
status 6809, 6712: resolved fixed
2010-10-28 14:32:13 -07:00
will
c5fef26f11
ListTemplates will no longer return any SystemVM templates. I am using a startsWith() string match for now since we used to make this comparision by ID with a single hypervisor.
2010-10-28 13:28:33 -07:00
will
f0a4449ec3
ListTemplates now return the hypervisor type.
2010-10-27 19:50:36 -07:00
Kris McQueen
50c175d6c1
bug 6799: forgot to add the exception response in my commit for this bug...
2010-10-27 18:23:23 -07:00
Kris McQueen
9056653e17
Change the API Implementation annotation to use the manager's class rather than an enum. This allows methods for new managers to be added trivially rather than having to modify the dispatcher to load a new manager class.
2010-10-27 12:23:31 -07:00
Kris McQueen
ae9b6d4fe1
bug 6776: for some reason, we checked for zoneId = -1 in registerTemplate and applied it to all zones, but this wasn't done for registerIso. Oh well, make the change for registerIso and things seem happy again
...
status 6776: resolved fixed
2010-10-26 17:55:00 -07:00
Kris McQueen
a788d2cb2e
bug 6736: rebootSystemVm should return the vm properties, not success true/false.
...
status 6736: resolved fixed
2010-10-26 17:19:55 -07:00
Kris McQueen
95da86ef1a
bug 6733: re-throw ServerApiExceptions when dispatching API commands rather than logging them and throwing a generic INTERNAL_ERROR exception
...
status 6733: resolved fixed
2010-10-26 16:54:42 -07:00
abhishek
c6588c0e25
adding some error return handling to api dispatcher, and cleaning up some of the code paths to return the right error messages
2010-10-26 14:27:26 -07:00
alena
afd79a9b1b
Code cleanup
2010-10-26 13:59:11 -07:00
Alex Huang
f58063911e
deploy vm completed
2010-10-26 09:35:24 -07:00
nit
497c60d63d
Extending extract functionlity for volumes to allow download. The
...
extraction can have two modes FTP_UPLOAD and HTTP_DOWNLOAD. In the former one
the user would provide the ftp url where the entity needs to be uploaded and in
the later the user would be provided a HTTP URL where from he/she can download
the entity. This url would be exposed for a specific time limit and would not
function after the time limit
2010-10-26 18:49:43 +05:30
Kris McQueen
b3c7821b44
bug 6752: fix updating resource limits for a domain by looking at the given accountName/domainId parameters rather than looking at the caller's account. If updating limits for an account, e.g. account 'foo' in domain 4, then execute updateResourceLimit&account=foo&domainid=4&..., but if updating the limit for a domain, don't specify the account parameter. When processing the command, if accountName is null, the limit for the domain will be properly updated.
...
status 6752: resolved fixed
2010-10-25 17:32:28 -07:00
Kris McQueen
ef131bbc18
bug 6724: set useVirtualNetwork in the createServiceResponse
...
status 6724: resolved fixed
2010-10-25 17:08:47 -07:00
Kris McQueen
f9c1f12778
bug 6725: use serviceOfferingName rather than the API command name when updating the service offering; set memory in the response for UpdateServiceOffering
...
status 6725: resolved fixed
2010-10-25 17:08:47 -07:00
Kris McQueen
27cafb2957
bug 6727: ostypename was being set from guest_os.name which seems to be null most of the time. Instead, use guest_os.display_name.
...
status 6727: resolved fixed
2010-10-25 17:08:47 -07:00
Kris McQueen
fcb0ff83ac
bug 6712: for router responses, change guestmacaddress -> macaddress to preserve behavior from 2.1.x and before.
...
status 6712: resolved fixed
2010-10-25 16:37:42 -07:00
Kris McQueen
869f3587c3
bug 6714: add displaytext to SuccessResponse so commands that return success true/false can also return a brief description of what took place.
...
status 6714: resolved fixed
2010-10-25 16:37:42 -07:00
Kris McQueen
3660d6a3d1
bug 6717: updateDiskOffering is supposed to return success true/false rather than the updated disk offering
...
status 6717: resolved fixed
2010-10-25 16:37:42 -07:00
Kris McQueen
c8e9531842
bug 6720: add 'created' to the ListHosts response
...
status 6720: resolved fixed
2010-10-25 16:37:42 -07:00
Kris McQueen
83e4922b94
bug 6703, 6704: use the correct method signatures for invoking updateTemplate/updateIso and updateTemplatePermissions/updateIsoPermissions APIs. It's possible that the reflection should just use 'isAssignableFrom' when getting methods since the manager methods were using a base class as the parameter rather than UpdateTemplateCmd/UpdateIsoCmd and UpdateTemplatePermissions/UpdateIsoPermissions, but for now the explicit methods are added for simplicity.
...
status 6703, 6704: resolved fixed
2010-10-25 15:01:50 -07:00
Kris McQueen
d229877cd8
bug 6710: rebootRouter was supposed to return the rebooted router, not success/failure
...
status 6710: resolved fixed
2010-10-25 15:01:50 -07:00
Kris McQueen
18f6d67121
bug 6705, 6708: use the UserVmResponse for changeServiceForVirtualMachine API (UpgradeVMCmd) since the response is a vm. There was no need for a specific UpgradeVmResponse which had scalars instead of objects and the serializer skips null, but will serialize scalars with their default values, in this case 0 for id and host id.
...
status 6750, 6708: resolved fixed
2010-10-25 15:01:49 -07:00
nit
511a095c26
bug 6487: AttachISO API would return an embedded object on success.
...
status 6487: resolved fixed
2010-10-25 18:26:39 +05:30
nit
5774fd163d
bug 6480: detachVolume API would start returning an embedded object, like attachVolume API does.
...
status 6480: resolved fixed
2010-10-25 17:19:22 +05:30
nit
554e360d21
Fixing a typo in the HostResponse of the word custer.
...
Fixing the names of the ISO and templates.
2010-10-25 14:54:49 +05:30
nit
035810d60a
bug 6481: embedded object returned by attachVolume API would have the "type" property indicating the volume type of the disk volume (ROOT or DATADISK)
...
status 6481: resolved fixed
2010-10-25 14:14:40 +05:30
kishan
3530daf3ff
bug 6697: make resourcetype optional
...
status 6697: resolved fixed
2010-10-25 12:22:06 +05:30
Jessica Wang
958aaaaf87
new UI - instance page - refactor submenu.
2010-10-22 20:23:05 -07:00
Kris McQueen
e6f7a480ae
bug 6674: Fix VM responses
...
- ostypeid -> guestosid
- findRootVolume is now in ApiDBUtils (from mgmt server) since it's for API responses only
- add rootdeviceid and rootdevicetype to various VM respones (listVirtualMachines, deployVirtualMachine, start/stopVirtualMachine)
status 6674: resolved fixed
2010-10-22 13:57:03 -07:00
Kris McQueen
4a96e1f633
bug 6662: handle a null ip forwarding rule in the API response as 'the rule already exists' since other errors will result in thrown exceptions. Also, fix up detection of network conflicts and duplicate rules by adding a list of used protocols to the port mappings
...
status 6662: resolved fixed
2010-10-22 11:47:52 -07:00
Alex Huang
f1a431714c
checkin to fix windows
2010-10-22 09:19:53 -07:00
anthony
18ffd7e7a8
fixed build
2010-10-21 19:25:52 -07:00
NIKITA
2f70032972
conflict
2010-10-21 17:57:56 -07:00
Alex Huang
25e4ed6997
more changes in the api
2010-10-21 16:53:42 -07:00
abhishek
a957a679ed
bug 5190: modifying a few minor details in the upload cert part, to conform with the new api refactor design
2010-10-21 15:51:46 -07:00
Kris McQueen
365ac7501f
bug 6672: Fix up volume commands. For createVolume, the command is supposed to create a database object, but not assign it to a storage pool until the user first attaches it to a vm instance. That allows the volume to start off hypervisor agnostic. For attachVolume, detachVolume, and listVolumes, the responses had minor issues (sourceType could be null, for example) in either the response generation, or the response handling by the UI.
...
status 6672: resolved fixed
2010-10-21 15:15:01 -07:00
Alex Huang
d5d8f27cad
Added more changes to the api
2010-10-21 15:11:49 -07:00
abhishek
9a9146c07b
whilst testing my enh, found that this call was using the wrong method name. Checking in the fix for the same.
2010-10-21 11:43:06 -07:00
abhishek
2b9ab00cc2
bug 3386: first part of the checkin, wherein we have an optional param domainId for create and modify zones. If this domain exists in the system, then the zone is tagged with this domain id, so that subsequent calls are checked for sanity against this domain.
2010-10-21 11:43:06 -07:00
kishan
bec09a4323
bug 6486: added VirtualMachine to stopVm response
...
status 6486: resolved fixed
2010-10-21 14:52:27 +05:30
nit
b7d430cf86
bug 6399: deployVirtualMachine API should return an embedded object (named "virtualmachine").
...
status 6399: resolved fixed
2010-10-21 13:48:29 +05:30
kishan
fa4a16cb68
bug 5592: Added configurable domain for hosts.
...
status 5592: resolved fixed
2010-10-21 11:51:09 +05:30
nit
6723549fc7
bug 6399: extending extract functionlity for templates and ISO's to allow download. The extraction can have two modes FTP_UPLOAD and HTTP_DOWNLOAD. In the former one the user would provide the ftp url where the entity needs to be uploaded and in the later the user would be provided a HTTP URL where from he/she can download the entity. This url would be exposed for a specific time limit and would not function after the time limit.
2010-10-20 11:40:03 +05:30