Reviewing SPML 2.0 Spec Draft 12

[Gary P Cole <Gary.P.Cole@Sun.COM>]

If you must target your reviews to the most significantly modified 
sections, I suggest the following:
2.1.3.3 Capabilities (changed)
3.1.2.3 Error Message (new)
3.4 CapabilityData (new)
3.6.6.1 listTargets (new "profile" argument)
3.6.1.2 add (changed CapabilityData and psoID)
3.6.1.3 modify (changed CapabilityData )
3.6.2 Async Capability (added Resource Considerations)
3.6.6.4-7 Reference CapabilityData * (new)
3.6.9.1 updates (added maxSelect)
3.6.9.3 closeIterator (new)

I'll also attach the spec_issues for Draft 11 so you can see everything 
that was touched.

Jeff_Bohren@bmc.com wrote:

>The document revision named SPML 2.0 Spec Draft 12
>(draft_pstc_spmlv2_d12.doc) has been submitted by Jeff Bohren to the OASIS
>Provisioning Services TC document repository.  This document is revision
>#10 of draft_pstc_spmlv2.zip.
>
>Document Description:
>SPML 2.0 Spec Draft 12
>
>View Document Details:
>http://www.oasis-open.org/apps/org/workgroup/provision/document.php?document_id=14183
>
>Download Document:  
>http://www.oasis-open.org/apps/org/workgroup/provision/download.php/14183/draft_pstc_spmlv2_d12.doc
>
>Revision:
>This document is revision #10 of draft_pstc_spmlv2.zip.  The document
>details page referenced above will show the complete revision history.
>
>
>PLEASE NOTE:  If the above links do not work for you, your email application
>may be breaking the link into two pieces.  You may be able to copy and paste
>the entire link address into the address field of your web browser.
>
>-OASIS Open Administration
>  
>

|1-5 CLOSED.

6. Update "Security and privacy considerations".
20050215 - Darran says that we should update this.  Wants a volunteer.
20050222 - Hal Lockhart volunteered to update this. Keep this in the spec.
20050329 - Hal thinks he can do this during this week.
20050607 - Hal thinks he can do this next day or two.
20050621 - Hal will send this by Thursday, 6/23.
20050823 - Jeff Bohren volunteered to take this over.

7. Appendix I (throughout): Document references need validation and update.
20050329 - Raj volunteered during the con-call to do this.
20050510 - Hal: Beware normative references to non-std docs.
20050621 - Raj will send this by Thursday, 6/23.
20050823 - Jeff Bohren volunteered to take this over.

8. Appendix L: Glossary.  Definitions need review and update.
20050222 - Point to a separate document for Glossary.
20050329 - Raj volunteered during the con-call to do this.
20050621 - Raj will do this by Thursday, 6/23.
20050823 - Jeff Bohren volunteered to take this over.

|9-44 CLOSED.

|45. (3.5.4.1.3) Modifying capability data may require capability-specific mechanism.
|20050614 - Modifying a subset of capabilityData may require capability-specific mechanism.
|20050615 - Each item should be wrapped in a separate <capabilityData> element.
|20050620 - (d10) Wrapped each <reference> in a separate <capabilityData> element.
|20050621 - Agreed during con-call to discuss this on the LIST.
|20050811 - Sent email: "Spec issue 45..." (no replace, no wildcard-delete).
|20050812 - JeffB says 1) cap-spec.  2) references a la LDAP attributes.
|20050812 - JeffB sent examples.  Basically, key=typeOfReference+toPsoID.
|20050815 - Slight PROBLEM: ReferenceType requires <toPsoID>.
|20050815 - Specify that ReferenceType requires <toPsoID>; optional in XSD.
|20050818 - Must address XSD issues #122-126.  Need for spec and examples.
|20050822 - Agreed that each <capabilityData> wraps all data for a single capability.
|20050822 - Agreed that "mustUnderstand" is an attribute of the wrapper.
|20050822 - (xsd123). Add "capabilityURI" attribute to CapabilityDataType.
|20050822 - Do we need/want a general mechanism for capability data?
|20080823 - Con-call participants showed no interest in a general mechanism.
|20050823 - (d12) Reworked specification of capability data processing.
 
|46-57 CLOSED.

|58. Provider may need to know requestor's profile.
|20050513 - Sent email: "Provider may need to know requestor's profile".
|20050513 - Jeff Bohren agreed (in a reply to the list).
|20050608 - Reminded Jeff Bohren (in another email) that we forgot this.
|20050614 - DEFER past COMMITTEE DRAFT.  Jeff Bohren wants to ponder it.
|20050812 - Sent email to list: "Spec issue #58..."
|           Add profile attr to ListTargetsRequestType and TargetType.
|           If profile specified, response contains only suitable targets.
|20050815 - AGREE.  Draft 27 adds profile attribute(s) and ErrorCode.
|20050818 - (d12) Added "profile" topic to listTargetsRequest.
|		Added "Target profile" in listTargetsResponse.
|           Enhanced listTargets Examples to discuss target profile.

|59-61 CLOSED.

|62.  SearchRequestType#query should be optional. Omitted means "all".
|20050818 - DISPUTED. Weird to need <query> to specify targetID. 
|           Move "targetID" to SearchRequestType?
|20050818 - Nowhere else can you omit targetID if multiple targets
|          (for <supportedSchemaEntity>, context implies containing target).
|           Search operation should not span multiple targets.
|           Each target could support a different profile.
|20050819 - (d12) FIXED. <query> is optional in search and updates requests.
|           If provider exposes more than one target, 
|           then <query> is required and must specify "targetID".

|63.  deleteResponse element is of type ResponseType (not DeleteResponseType).
|20050812 - FIXED in Draft 12.

|64.  (Examples) "path" must precede "namespaceURI" in any <select> element.
|20050818 - Draft 12: Updated <select> and <component> elements in examples.

|65.  XSD 26: SearchQueryType#basePsoID minOccurs=0.
|20050818 - Draft 12: Updated XSD snippets and Appendices.

|66.  (=XSD#119) LookupResponse contains at most one <pso>.  Clarify text.
|20050815 - (d12) Also updated XSD snippets and Appendix A.

|67.  Line 2044 "Batch Capability" should say "Bulk Capability".
|20050812 - FIXED in Draft 12.

|68.  Can modify add/replace/delete an XML *attribute* (or only an element)?
|     Jeff Bohren review comments on examples say only an XML *element*.
|     Careful: Refer to profile-spec for normative specifics.
|20050818 - (d12) FIXED. Added "Depends on profile" topic to normative sections 
| "SelectionType in a Request", "Selection Type Processing" and "SelectionType Errors".

|69.  Async Capability needs Resource Considerations (like those in Search).
|20050818 - (d12) FIXED. Async Intro has "Resource Considerations" topic.

|70.  Discuss ErrorMessages after Status and Error.
|20050813 - Added section entitled "ErrorMessage (normative)".

|71.  Add SPMLv2-UPDATES to "Namespaces".
|20050815 - (d12). FIXED in Draft 12.

|72.  Add SPMLv2-SUSPEND and SPMLv2-UPDATES to "References".
|20050815 - (d12). FIXED in Draft 12.

|73.  Mention Updates Capability at the top of "Operations" section.
|20050815 - (d12). FIXED in Draft 12.

|74.  Replace references to "execution" attribute with "executionMode".
|20050815 - (d12). FIXED in Draft 12.

|75.  Provider must honor any <psoID> supplied in <addRequest>.
|20050815 - (d12) FIXED. Reworked addRequest and addResponse sections to discuss.
|20050815 - XSD 27 adds ErrorCode#invalidIdentifier and #alreadyExists.

|76.  Discuss new maxSelect attribute of UpdatesRequestType (new in XSD 27).
|20050818 - (d12) FIXED. updatesRequest and updatesResponse discuss maxSelect.

|77.  Document new closeIterator operation of UpdatesCapability.
|20050818 - (d12) FIXED. Draft 12 has new "closeIterator" section.

|78.  Consistently use XML element font for enclosing brackets.
|79. line 661: "existing object that exists on a target" is redundant.
|80. line 1705: deleteResponse returns "requestID" (not "asyncRequestID").
|81. line 1739: statusRequest MUST return status of (not cancel) the operation.
|82. line 1799: response for pending must contain *initial* subset of output.
|83. line 1908: Section 7.3.6 does not exist. Paragraph is redundant.  Rewrite.
|84. line 1917: No such as batchableRequest.  Relabel as "Nested requests".
|85. line 1919: batchRequest MUST contain at least one request. 
|    Combine this sentence with the previous sentence.
|86. line 1928: Add spmlupdates:UpdatesRequestType, IterateRequestType
|    and CloseIteratorRequestType to the list that must not be batched.
|87. line 2007: delete unmatched parenthesis in "described by a) nested".
|88. line 2216: setPassword must change password for (not enable) object.
|89. line 2268: expirePassword invalidates password for (not enables) object.
|90. line 2316: must should be all capitals (because it is normative).
|91. line 2360: validatePassword tests the value (not changes password).
|92. line 2363: delete paragraph (copied from resetPassword).
|93. line 2528: complexType RacfGroupMembership should be RacfGroupMembershipType.
|94. line 2571: complexType RacfGroupMembership should be RacfGroupMembershipType.
|95. line 2585: complexType RacfGroupMembership should be RacfGroupMembershipType.
|96. line 2784 (and throughout): "status set" should be "result set".
|97. line 2834 (and in section): iterate returns next set of items (not next item).
|98. line 2869: Clarify time when "one object remains".  Tense is ambiguous.
|99. line 2872: Clarify time when "no object remains".  Tense is ambiguous.
|100. line 2919 (and in section): searchResponse should be iterateResponse.
|101. line 2933: Remove "requestor cannot specify "ID". Confusing.
|102. line 3427: Remove "requestor cannot specify "ID". Confusing.
|103. line 3464 (and in section): iterateResponse returns updates (not objects).
|104. line 3484 (and in section): "updates set" should be "result set".
|105. line 3507 (and in section): iterate returns next set of items (not next item).
|106. line 3516: Clarify time when "one object (update) remains".  Tense is ambiguous.
|107. line 3519: Clarify time when "no object (update) remains".  Tense is ambiguous.
|108. line 3524: Remove "PSO and ReturnData" section. Copied from search iteration.
|109. line 3524: Remove "PSO capability-specific Data..." section.
|110. line 3580: Remove "requestor cannot specify "ID". Confusing.
|20050817 - Draft 12 addresses issues 78-110.

|111. Add contributors Robert Boucher, Cal Collingham, James Hu, Prateek Mishra.
|	Alphabetize contributors by last name.
|20050822 - (d12) Draft 12 adds these (and alphabetizes) contributors.