Commit Graph

661 Commits

Author SHA1 Message Date
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
abhishek aebf02b707 removing some debug params 2010-10-25 11:52:08 -07:00
root c98b2a7fa2 incremental checkin around the console proxy config enh 2010-10-25 11:52:08 -07:00
abhishek a7b1e0a539 more changes for console proxy 2010-10-25 11:52:08 -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 8ae5a70b06 bug 6395: corrected net rule event error description
status 6395: resolved fixed
2010-10-25 13:11:50 +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 1728606e51 use the correct account id when saving the event after assigning a port forwarding service 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
anthony da58ade720 fixed addding the secondary storage 2010-10-22 11:06:00 -07:00
Alex Huang 88f446f9d4 checkin for windows 2010-10-22 09:21:47 -07:00
Alex Huang f1a431714c checkin to fix windows 2010-10-22 09:19:53 -07:00
Kelven Yang 1c702a1d69 1) As the lack of dom0 scripting support in vmware hyperviors, domR, and other system VM will fall back to private network interface instead of link local interface, accordingly,
domR scripting will be running directly inside domR and be driven from management server instead of dom0 host
2) firewall/sshd setting changes for #1
2010-10-21 21:46:41 -07:00
anthony 18ffd7e7a8 fixed build 2010-10-21 19:25:52 -07:00
anthony 141250714e fixed the break of adding standalone host 2010-10-21 19:00:11 -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 3e2c92e0db bug 3386: refining the comments inside the code 2010-10-21 14:29:07 -07:00
root 1c0c753342 bug 3386: final part of the enh for tagging zones with domains. We make the same check for childdomains as exist in the current code, to ensure the right domain permissions are enforced. By default, we will use the domainId passed by the user, if this is not passed, we will try to get the id from the dc. If it exists (private dc), if it doesn't then it is (public dc) 2010-10-21 14:27:07 -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
anthony 3260001b65 bug 6665: don't need to send poolEject command for the last host, and mark it as success
status 6665: resolved fixed
2010-10-21 10:21:32 -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
Kelven Yang 58cc90c10f 1) Modify domain router manager to support vmware
2) Add python and shell scripts for domr to support vmware
2010-10-20 18:49:18 -07:00
Kris McQueen f4e21787b6 bug 6658: fix managing load balancer instances by removing some of the 'pending' status code.
status 6658: resolved fixed
2010-10-20 17:27:08 -07:00
Kris McQueen 2f8aad2691 The listSnapshots API has changed in 2.2 a bit, so rather than merge my fix for bug 6680 from 2.1.x, just applied it manually...the fix was to apply the domain search to listSnapshots so that domain admins can see snapshots in their domain and all sub domains, but nothing outside their domain hierarchy. 2010-10-20 16:48:20 -07:00
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
Kelven Yang d8a5e84e21 If default hypervisor is vmware, use private network instead of link local network 2010-10-19 16:35:32 -07:00
Kris McQueen 47e26af419 bug 6643: when configuring vlan IP ranges, account might not be specified (zone wide IP range), so assign the configuration event to the SYSTEM account in those cases.
status 6643: resolved fixed
2010-10-19 16:27:08 -07:00
Kris McQueen 2bd5b6a29e bug 6654: virtualmachinename should be serialized as vmname, fix handling of the createPortForwardingRule response
status 6654: resolved fixed
2010-10-19 15:59:37 -07:00
abhishek 7c11c04427 reverting the change for bug 6361; this was an accidental push instead of commit 2010-10-19 15:21:01 -07:00
abhishek 7cc8087fa6 bug 6361: more name changes from network group to security group 2010-10-19 15:21:01 -07:00
abhishek 523862f571 bug 6361: further name changes from network to security group 2010-10-19 14:35:17 -07:00
abhishek 6bd7caa9c2 bug 6361: first part of incremental checkin to rename network group to security grou[ 2010-10-19 14:35:17 -07:00
Kris McQueen 0267a306c8 bug 6649: when listing domains, accept -1 as 'list all.' Fix listDomainChildren response to return haschild, handle haschild as a boolean rather than a string.
status 6649: resolved fixed
2010-10-19 13:49:00 -07:00
Alex Huang 5466e10fa3 simple variable rename 2010-10-19 12:03:40 -07:00
Alex Huang 2da8035bd1 more changes for the rewrite 2010-10-19 11:52:02 -07:00
Alex Huang 794ca16a65 Added code to start domR 2010-10-19 11:52:01 -07:00
abhishek 04c8c0f91c bug 6041: returning the complete embedded reponse for a destroyvm async call
status 6041: resolved fixed
2010-10-19 11:46:42 -07:00
Kris McQueen f4f00cbe2d bug 6655: when verifying request signature for API requests that use keys, pass the userId of the requester as a Long to avoid NPEs when converting it to a string. Also set up the UserContext for both normal users and admins.
status 6655: resolved fixed
2010-10-19 11:41:09 -07:00
abhishek 408822e358 bug 6037: last part of the checkin for this bug
status 6037: resolved fixed
2010-10-19 11:36:54 -07:00
abhishek 89e9a85902 bug 6037: adding response obj of type uservmmanager to recovervm response 2010-10-19 11:00:23 -07:00
Kris McQueen 68eb09114a there is more than one routing template now (VMWare has its own) so change the listVolumes filter to remove volumes for routing* templates 2010-10-19 10:40:52 -07:00
Chiradeep Vittal 947c2ca16d fix problems in api post-refactor 2010-10-19 09:53:51 -07:00
Chiradeep Vittal 4d37269961 fix local disk for system vms 2010-10-19 09:53:51 -07:00
alena bb5399c2bc Fixed PodResponse: endip is all lowercase now. 2010-10-18 14:11:21 -07:00
Kris McQueen 603874605d The serialized names for the old VIRTUAL_MACHINE_* constants weren't quite right. Fix these up. This is a good example for why there needs to be a pass through the API commands and responses and constants should be used in the annotations rather than the current strings that are in place. 2010-10-18 14:10:07 -07:00
Kris McQueen 3be625ca6e The stopVirtualMachine API should return the vm instance rather than a boolean indicating success. The response for async jobs is now under the jobresult property of the JSON response, handle the start/stop virtual machine responses using data from the jobresult rather than expecting some embedded object. 2010-10-18 11:15:07 -07:00
edison e41729cc43 fix rpm build 2010-10-16 01:03:31 -07:00
Kelven Yang 915d29f47b Skip starting of console proxy when storage pool is not ready 2010-10-15 18:57:28 -07:00
Kelven Yang 2c919c5c01 Make unique name conforming with the name generated in template installation for routing template, some other minor fixes 2010-10-15 16:36:02 -07:00
anthony d41f7d78d9 DeleteStoragePool command fix
1. not delete SR in xenserver side, only delete it in database.
    2. remove a storagepool,  adding it back fails
2010-10-15 14:15:55 -07:00
Kelven Yang 8555eb1913 Fix the broken ConsoleProxyServlet after the merge of API refactoring changes 2010-10-15 11:17:27 -07:00
Alex Huang 84d2a0c842 reverted back most of my changes to merge in Kris stuff. Thanks much to Anthony who caused my XenServer to reboot all the time which caused me unable to unit test stuff and unlike Kris, I do not check in code untested so now I have to do all this stuff. 2010-10-15 07:10:16 -07:00
Alex Huang bebf9dbc3f Adding stuff 2010-10-15 06:26:24 -07:00
Alex Huang b3da4cea1b Forcing me to commit because of stupid git 2010-10-15 06:22:59 -07:00
Kris McQueen 80090990a6 Fix configuration in light of new API framework. Move code from ConfigurationManager into ConfigurationServer for creating disk/service offerings, pods, and zones. This prevents the ComponentLocator from pulling in a lot of extra DAOs and Managers now that ConfigurationManager has a lot more dependencies due to API changes. Without making this change, there would be a dependency on ClusteredAgentManager in both config-server and mgmt-server, which would try to start two listeners on port 8250, which is bad. Now that the configuration happens in ConfigurationServer, things should be kept simple. 2010-10-14 19:11:22 -07:00
Kris McQueen 88814d95f0 A zoneId of -1 means all zones, so if zoneId is passed in as -1 just list all zones for registering the template 2010-10-14 19:11:21 -07:00
Kelven Yang f6e0d2dd41 Make system VM work on vmware 2010-10-14 17:23:13 -07:00
Kris McQueen e168760295 The code implies resource could be null, and yet de-references resource before the null check. As I was experiencing a NPE because if this exact scenario, put the remote agent check inside the check for null to avoid problems. If resource is null we log that we were unable to send ping due to agent disconnected... 2010-10-14 15:02:43 -07:00
anthony 39d30b9bf1 only send commands to host which is Up and Connecting 2010-10-14 11:35:48 -07:00
anthony 7115aa06bd send commands to host which is UP and connecting 2010-10-14 11:08:35 -07:00
anthony 9b839d749b only send command to host which is UP 2010-10-14 10:24:52 -07:00
anthony f7ec2ebc48 bug 6521: for add host, return api response, even if the host is in disconnected state
status 6521: resolved fixed
2010-10-14 10:20:48 -07:00
Kris McQueen d928a6f3d9 If storage pool is null after adding a pool, throw an exception so the user sees a problem adding the pool. This is better than the NPE that was happening... 2010-10-13 16:08:49 -07:00
Kris McQueen 5960911511 Fix processing of session parameters in logout case 2010-10-12 19:42:53 -07:00
Kris McQueen 8e086d7a21 resolving merge conflicts between 2.1.refactor and master 2010-10-12 16:28:16 -07:00
Kris McQueen 0f4a245f0a resolving merge conflicts before pushing to the remote repository 2010-10-12 15:37:50 -07:00
Kris McQueen 2740ba6c40 final merge of master->2.1.refactor preparing for merge back into master 2010-10-12 14:46:24 -07:00
Alex Huang ce091de3d2 more changes for refactor 2010-10-12 07:17:47 -07:00
anthony 09cefaf3e9 bug 6500: add back the special handle for secodary storage host, it is in the same path of handling xenserver host, it is easy to break that, so I add some comments there
status 6500: resolved fixed
2010-10-11 12:05:13 -07:00
alena fb50fe624c * Added descriptions for Api command's request and response parameters.
* Changed response type for DeleteDomain and DeletePreallocatedLun commands to SuccessResponse
* Implemented small tool (ApiXmlDocReader) for comparing api commands of different product versions.
2010-10-11 09:21:42 -07:00
anthony 0c750aa8fa bug 6462: notifyMonitorsOfConnection of handleDirectConnect may fail, but it still returns the removed attach, and lauch ping task
status 6462: resolved fixed
2010-10-09 17:34:36 -07:00
anthony a93d36d81a should use _hostDao.listByCluster instead of _hostDao.listByPodId in some places 2010-10-08 18:10:15 -07:00
edison 2f39664f94 maybe it was an incorrect merge that snapshot was disabled for KVM
Re-enable it.
2010-10-07 18:56:42 -07:00
Kelven Yang 0d64b460f8 add vmware network configuration variables 2010-10-07 18:23:26 -07:00
edison 3028dab141 fix systemvm template in direct attached mode, guest vm can't get gateway and dns server 2010-10-07 16:32:46 -07:00
anthony 2861895a84 bug 6450: list all snapshot for admin
status 6450: resolved fixed
2010-10-07 15:35:58 -07:00
Kris McQueen 9e03518c94 fix return value of stopSystemVm (should be the vm, not a boolean); add the missing disableAccount API in management server to avoid a NoSuchMethod exception when disabling an account 2010-10-07 15:10:03 -07:00
Kris McQueen 45c611100a fix NPE when updating a resource limit using the internal API port 2010-10-07 15:10:03 -07:00
Kris McQueen 55201432f6 createServiceOffering had cpuSpeed passed twice when creating the offering rather than cpuNumber then cpuSpeed...fixed 2010-10-07 15:10:03 -07:00
Kris McQueen ea84f3bde2 registerTemplate and registerIso expect the template to be returned, not the template id...fixed 2010-10-07 15:10:02 -07:00
Kris McQueen 7c4a06a764 fix the method name for listIsos command 2010-10-07 15:10:02 -07:00