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
Kelven Yang
3c6b3b3e3d
Add more volume info to BackupSnapshotCommand, to generalize multi-hypervisor usage
2010-11-02 17:04:07 -07:00
abhishek
473c3d33c3
bug 6870: during updating of pf rules, it is possible one might try to update a non existing rule via the api cmd. hitherto, we were returning a null which was assumed as a success by the cmd api, which failed with a npe. instead, we ought to throw back an error since we did not find any rule to be updated. i am checking in the same change. this also will fix the npe.
...
status 6870: resolved fixed
2010-11-02 16:07:15 -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
abhishek
16438f3470
we should not be ignoring case whilst updating account, reverting to .equals instead of .equalsIgnoreCase
2010-11-02 13:45:10 -07:00
abhishek
7354116517
bug 6888: there were multiple issues here. firstly, we were checking against null newaccountnames, whereas that field is a reqd val. Also, we were allowing the same name to exist across multiple accounts in the same domain. I have fixed both these issues.
...
status 6888: resolved fixed
2010-11-02 13:09:37 -07:00
Alex Huang
560d7a275e
Removed InternalErrorException. Renamed the lock methods on the DAO objects to be more understandable.
2010-11-02 12:25:08 -07:00
Alex Huang
3279c26796
Removed a bunch of InternalErrorException
2010-11-02 12:25:08 -07:00
Alex Huang
6311644e08
Successful creation of domr and uservm with all programming done correctly.....yahooo
2010-11-02 12:25:08 -07:00
alena
0c044a10d1
Moved response building for ServiceOffering/ResourceLimit commands to ApiResponseHelper class.
2010-11-02 11:59:10 -07:00
abhishek
b6aeb6746c
bug 6889: fixing the issue where we were able to create a disk offering of size 0. The check was incorrect, and now we ensure we only create the right sized volumes.
...
status 6889: resolved fixed
2010-11-02 11:30:43 -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
3bab616478
After discussing with Alex, it was agreed that there is no need to mgmt server level locking using the mgmt_server_id. This makes the code easier, as we have one parent acquire and release mechanism, and the complexity is reduced. In accordance, we do not need to maintain cleanup tasks for mgmt server crashes, as the async mechanism deals with this. Also, the schema becomes a tad simpler as the mgmt_server_id column is not needed anymore
2010-11-01 17:04:28 -07:00
edison
d9ba39bd8b
enable user to config default hypervisor type as vmware
2010-11-01 15:25:32 -07:00
Alex Huang
394af1b29c
more changes on the way
2010-11-01 15:01:28 -07:00
Alex Huang
cd1c903260
Better integration between it and virtual machine gurus
2010-11-01 15:01:28 -07:00
Alex Huang
4db0b7d156
Better integration between it and virtual machine gurus
2010-11-01 15:01:28 -07:00
abhishek
8e9d74c7f6
bug 5190: This covers the case of unforseen exceptions (although a corner case), which might pop up. We introduce a finally block which will release the cert db record for other ms to process, in case the owning ms errors out (not crash), whilst running the cert update process
2010-11-01 14:36:13 -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
Chiradeep Vittal
76c7b54fc5
fix problems after merging in bug fixes from 2.1.x
2010-11-01 12:27:15 -07:00
Chiradeep Vittal
40f15106be
when user vm migrates, update rules on destination host
2010-11-01 12:21:56 -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
fe46ddfd63
bug 6806: changing the 'iso' to 'ISO' in all the error messages being thrown back from the API
...
status 6806: resolved fixed
2010-11-01 11:12:03 -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
ff91abbb86
fix build break
2010-11-01 11:05:22 -07:00
Chiradeep Vittal
1e0eb04442
Add VPN apis, some systemvm changes for vpn
2010-11-01 10:27:49 -07:00
abhishek
71f37ff1c7
This is another improvement over the existing code for custom certs. We should ideally validate the cert for being a valid cert file (X.509 spec), instead of persisting it to the db and then validating it whilst constructing the ssl context. I am adding some validation around the same.
2010-11-01 10:07:53 -07:00
abhishek
9b684443bb
bug 5190: Final commit to clean up an unused import, and to merge the changes with the master branch.
...
status 5190: resolved fixed
2010-11-01 09:09:18 -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
fbe2d288d7
adding some code comments
2010-11-01 09:06:32 -07:00
abhishek
b9cd3e2522
some more fine tuning around corner cases
2010-11-01 09:06:32 -07:00
abhishek
f3b4dd8a1c
further code modularization, extracting redundant code to common methods
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
abhishek
7b87aad660
fixed a bug in the listener flow; now handling the application of certs to existing cpvms and new cpvms based on a fork in the logic in the console proxy resource
2010-11-01 09:04:51 -07:00
abhishek
abd64eddd1
adding more exception handling and debugging support
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
abhishek
9c80911e52
more changes
2010-11-01 09:00:58 -07:00
abhishek
d9ebb7147c
improving efficiency by reducing the number of db lookups, whilst updating the custom certificates across multiple console proxies when the command is issued. Now, we use in memory hashmaps for hosts to get the hostids for cphosts, which eliminates the need to make a db lookup for every cpvm that needs to be rebooted
2010-11-01 09:00:58 -07:00
abhishek
4d2e126ea1
more cpvm refactoring
2010-11-01 09:00:58 -07:00
kishan
b08037d22c
bug 6851: use specified account properties instead of admin, resourceType enum instead of type
...
status 6851: resolved fixed
2010-11-01 19:19:33 +05:30
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
nit
1b88b0eae4
bug 6748: Correcting secondary storage space utilization on the dashboard. The fix will update the existing db entry and will create a new entry for the first time only.
...
status 6748: resolved fixed
2010-11-01 12:16:07 +05:30
edison
5c88bb936b
Oops, == doesn't work...
2010-10-29 19:51:24 -07:00
edison
d0c68cd4a7
add a timeout (5 minutes) to wait for KVM agent host connecting to mgt svr, during adding host->discover
2010-10-29 19:18:07 -07:00
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