version 1.1.2.35, 2014/06/01 19:26:51
|
version 1.1.2.48, 2014/08/19 21:06:15
|
|
|
servers. In any case we can do only minimal tests because most real | servers. In any case we can do only minimal tests because most real |
testing involves both open and pull and the static tests are by their | testing involves both open and pull and the static tests are by their |
nature single operation. | nature single operation. |
7. Clean up the CIMOperationRequestDecoder.cpp reject CIMException |
|
further since there is some overlap. The only overlap is that a |
|
couple of the specific functions duplicate. VERY LOW priority. |
|
8. Correct issue between operations and HTTP where we are sending | 8. Correct issue between operations and HTTP where we are sending |
trailers with exceptions. Modify response mechanisms so that we | trailers with exceptions. Modify response mechanisms so that we |
set non-chunked for all responses where we send error responses to | set non-chunked for all responses where we send error responses to |
|
|
the associators(i.e. assoc, ref, etc.) functions and the parallel | the associators(i.e. assoc, ref, etc.) functions and the parallel |
openAssoc functions. It would be nice to consolidate that logic and | openAssoc functions. It would be nice to consolidate that logic and |
further to try to create a non-template form for those functions. LOW | further to try to create a non-template form for those functions. LOW |
PRIORITY. |
PRIORITY and not for bug 9676 ACTION: Define in new bug |
11. External runtime variables. This bug leaves these variables as |
11. External runtime variables. bug 9676 leaves these variables as |
internal to CIMOperationRequestDispatcher.cpp. | internal to CIMOperationRequestDispatcher.cpp. |
See bug 9819 for the changes to externalize these variables, both | See bug 9819 for the changes to externalize these variables, both |
the runtime and buildtime variables. The Work group discussion | the runtime and buildtime variables. The Work group discussion |
documents what the variables are and how they tie to statistics info. | documents what the variables are and how they tie to statistics info. |
See the OpenPegasus Wiki, Pull operations work group. | See the OpenPegasus Wiki, Pull operations work group. |
12. Trace functions in CIMResponseData,h & cpp should be PEGASUS_DEBUG. | 12. Trace functions in CIMResponseData,h & cpp should be PEGASUS_DEBUG. |
Same for trace function in EnumerationContext and EnumerationContextTable |
Same for trace function in EnumerationContext and |
|
EnumerationContextTable |
13. Question. We added trace in CIMRequestOperationDispatcher.cpp if | 13. Question. We added trace in CIMRequestOperationDispatcher.cpp if |
query enum is recevied with no object path. However, since this is | query enum is recevied with no object path. However, since this is |
a provider provblem should we be doing something other than a trace |
a provider problem should we be doing something other than a trace |
(ex. log) since traces are often not really used and they do generate | (ex. log) since traces are often not really used and they do generate |
lots of data whereby something like this could get lost. Also, need | lots of data whereby something like this could get lost. Also, need |
to review what level of trace if we keep trace. | to review what level of trace if we keep trace. |
|
14. Note that the build-a-class argument on OpenQueryInstances always |
|
returns error since we do not build the class. We propose to deprecate |
|
this in DMTF and not implement in in OpenPegasus. TODO add a bug |
|
defining this limitation. This is not a TODO, just a warning for |
|
the future. TODO add bug about this and add to release notes |
|
15. Clarify trace of completion of each enumeration sequence. |
|
16. Statistics keeping for open, etc. through cimperf. |
| |
| |
NEXT TASKS: | NEXT TASKS: |
a. finish 8, 12 above ( Finish by about 1 June). Do next mergeout and |
a. finish 8, 12 above. Test against cimperf since there was a bug |
mergein to create a complete patch |
in cimperf from before. |
|
|
|
14 August 2014 |
|
1. Issue with the pullop test in some systems causing an HTTP 413 error ( |
|
too much data). We confirmed that this is probably actually a limit in the |
|
client with enumerate instances (the response is simply to big in the case |
|
where we are asking it to deliver 100,000 responses instances on some systems) |
|
Modified the test for the stress test to be 30,000 instances and added a |
|
manual test testSizeLimit to the make file to try to deliver up to 600,000 |
|
instances to determine limits to the environment. |
|
|
|
Note that this is not actually a failure of the pull operations. In fact |
|
where cimcli ei ... fails with 600,000 instances in the response cimcli pei |
|
works correctly. |
|
|
|
Update the Client/tests/pullop/Makefile stresstests to hopefully pass the |
|
nightly test on more systems. |
|
|
|
2. Eliminated a number of KS_TODOs and code that we had documented out with |
|
//// . Note that there are still a number of the KS_TODOs but most of them |
|
are really isolated around |
|
|
|
19 August 2014 |
|
1. Mergeout to head of tree. We picked up one issue that for some reason |
|
the cvs mergeout was missing, a change to Server/tests/TLS... |
|
|
|
19 August 2014 |
|
1. Remove a number of KS_TODO notes either by cleaning them up or by |
|
change the keyword to FUTURE if they are things that we might consider |
|
for the future |
|
2. Some efficiency improvements in dealing with strings primarily get |
|
rid of use of String::EMPTY in the new code. |
|
|
|
3. Removed some diagnostic trace output that was used during development. |
|
|
|
13 August 2014 |
|
1. Mergeout to head of tree. |
|
|
|
11. August 2014 |
|
1. Correct error in compile in windows (bad reference to sleep(...) |
|
|
|
29 July 2014 |
|
1. Cleanup functionality in the unittests for EnumerationContext and |
|
its table. |
|
|
|
23 July 2014 |
|
1. Mergeout from head of tree to Task branch |
|
|
|
21 July 2014 |
|
1. Removed strings.h from pullop.cpp |
|
2. Minor cleanup per testing |
|
|
|
17 July 2014 |
|
1. Correct error in CIMOperationRequestDispatcher.cpp OpenQueryInstances |
|
when PEGASUS_DISABLE_QUERY set. |
|
2. Correct issue in diagnostic code in CIMResponseData.cpp that used |
|
snprintf function not available in windows. |
|
3. Clean up some miscellaneous code in pullop and Makefile. |
|
4. Minor change to PullErrosrs. |
|
|
|
2 July 2014 |
|
1. Corrected error in the new getoopt functions that caused it to fail |
|
in build and also in it unit test with the --help option. |
|
|
|
28 June 2014 |
|
1. Corrected issue where ZOs was getting errors from pullop because we |
|
used getopt to parse command line and the linux getopt is not behavior |
|
consistent with ZOs. Note that the ZOs getopt matches the posix expect and |
|
the linux one has extensions (notably it permutes non option arguments to |
|
the end of the arguments array. The easiest and most logical was to move |
|
the whole parser to use the internal getoopt which is what we should be |
|
using all the time in any case. |
|
18 June 2014 |
|
1. Correct errors in pullop and CMIMessage.cpp identified by Ji Lee as comments 58,59 |
|
2. Correct error in Makefile identified by Ji Lee as bug comment 60 |
|
|
|
14 June 2014 |
|
1. Mergeout from head of tree. |
|
|
|
12 June 2014 |
|
1. Implement the basic code for the CIMOperationDispatcher portion of bug 9819 |
|
Note that the CIMConfig calls are disabled with comments. |
|
2. Corrected a number of differences in const declaration and definition for |
|
continueOnError and maxObjectCount in CIMClient. This corrects issue |
|
documented in bug 9676 comment 53 |
|
3. Corrected some documentation for Pull operations in CIMClient. |
|
4. Corrected similar issue In EnumerationContext class. See comment 54 |
|
5. Removed ExecInfo.h header from Client/tests/PullErrors/PullErrors.cpp |
| |
30 May 2014 | 30 May 2014 |
1. Correct issue in UintArgs. Modified to use smart pointers, consistent | 1. Correct issue in UintArgs. Modified to use smart pointers, consistent |