----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17888/#review35466 -----------------------------------------------------------
Also please change the level of the log from WARN to DEBUG. When incorrect parameter is passed in, there is no harm as its basically ignored at the end. WARN is used when something harmful is going on. - Alena Prokharchyk On Feb. 25, 2014, 12:11 p.m., Antonio Fornie wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17888/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2014, 12:11 p.m.) > > > Review request for cloudstack, Alena Prokharchyk, daan Hoogland, and Hugo > Trippaers. > > > Repository: cloudstack-git > > > Description > ------- > > Dispatcher corrections, refactoring and tests. Corrects problems from > previous attempts that were reverted by Alena. Most of the changes are the > same, but this one is not creating conflicts of Map types for Aync Commands > or for parameters as Lists or Maps. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/ApiConstants.java 7b7f9ca > api/src/org/apache/cloudstack/api/BaseCmd.java 0e83cee > api/src/org/apache/cloudstack/api/BaseListCmd.java c1a4b4c > > api/src/org/apache/cloudstack/api/command/user/autoscale/CreateAutoScaleVmProfileCmd.java > 06d86ec > > server/resources/META-INF/cloudstack/core/spring-server-core-misc-context.xml > fd2f5fb > server/src/com/cloud/api/ApiAsyncJobDispatcher.java 71ac616 > server/src/com/cloud/api/ApiDispatcher.java ed95c72 > server/src/com/cloud/api/ApiServer.java d715db6 > server/src/com/cloud/api/ApiServlet.java 46f7eba > server/src/com/cloud/api/dispatch/CommandCreationWorker.java PRE-CREATION > server/src/com/cloud/api/dispatch/DispatchChain.java PRE-CREATION > server/src/com/cloud/api/dispatch/DispatchChainFactory.java PRE-CREATION > server/src/com/cloud/api/dispatch/DispatchTask.java PRE-CREATION > server/src/com/cloud/api/dispatch/DispatchWorker.java PRE-CREATION > server/src/com/cloud/api/dispatch/ParamGenericValidationWorker.java > PRE-CREATION > server/src/com/cloud/api/dispatch/ParamProcessWorker.java PRE-CREATION > server/src/com/cloud/api/dispatch/ParamSemanticValidationWorker.java > PRE-CREATION > server/src/com/cloud/api/dispatch/ParamUnpackWorker.java PRE-CREATION > server/src/com/cloud/network/as/AutoScaleManagerImpl.java ff2b2ea > server/src/com/cloud/storage/snapshot/SnapshotSchedulerImpl.java 3159059 > server/test/com/cloud/api/ApiDispatcherTest.java 7314a57 > server/test/com/cloud/api/dispatch/CommandCreationWorkerTest.java > PRE-CREATION > server/test/com/cloud/api/dispatch/DispatchChainFactoryTest.java > PRE-CREATION > server/test/com/cloud/api/dispatch/ParamGenericValidationWorkerTest.java > PRE-CREATION > server/test/com/cloud/api/dispatch/ParamProcessWorkerTest.java PRE-CREATION > server/test/com/cloud/api/dispatch/ParamSemanticValidationWorkerTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/17888/diff/ > > > Testing > ------- > > Full build and test plus manually testing many features. Also including > CreateTagsCommand that failed in previous commit. > > All unit and integration tests. > > Test CS Web UI with the browser going through several use cases. > > Also use the CS API by sending HTTP requests generated manually including > requests for Async Commands with Map parameters and during these tests apart > fromtesting correct functionality I also debugged to check that Maps created > correctly where they should but also that in the cases where the async > command must be persisted and later on retrieved and deserialized by gson > everything works ok and does what and where is expected. An example based on > the comment by Alena: > http://localhost:8096/client/api?command=createTags&resourceids=ids&resourcetype=type&tags[0].key=region&tags[0].value=canada > Also other examples like > http://localhost:8096/client/api?command=createSecondaryStagingStore&url=httpbla&details[0].key=region&details[0].value=canada&details[1].key=element&details[1].value=fire > > > Thanks, > > Antonio Fornie > >
