[cifs-protocol] [REG: 115021812413073] MS-DRSR ExecuteKCCTasks empty

[Tarun Chopra]

Hello Andrew - I had a discussion internally and we have come up with following addition - one in quotes, bold and underscored. Appreciate if you can share your inputs on this addition.

4.1.6.2.1  ExecuteKCCTasks

procedure ExecuteKCCTasks(): ULONG
This procedure executes the tasks necessary for maintaining the replication topology between DCs.

"These tasks include activities such as maintenance of kCCFailedLinks and kCCFailedConnections, maintenance of intra- and inter-site connections, and updates of RODC objects (as appropriate). See [MS-ADTS] section 6.2.2 for a full list of these tasks."

If an error occurs, a Windows error code is returned. If successful, the method returns 0.

Thanks
Tarun chopra.

On Feb 19, 2015, at 6:54 AM, Andrew Bartlett <abartlet at samba.org<mailto:abartlet at samba.org>> wrote:

On Thu, 2015-02-19 at 00:37 +0000, Tarun Chopra wrote:
Hello Andrew -

Thanks for the feedback.

Just below this section 4.1.6.2.1  in section 4.1.6.3, we have provided forward reference to section 6.2.2 of MS-ADTS. In addition,  last step of pseudo code provided in section 4.1.6.3 calls into  'ExecuteKCCTasks()' which gives an impression that both sections are linked. Is it sufficient from implementation\clarity point of view to have reference from section 4.1.6.3  only ? If not, we will go ahead and add the reference in section 4.1.6.2.1  too.

    4.1.6.3 Server Behavior of the IDL_DRSExecuteKCC Method
    ........    See [MS-ADTS] section 6.2.2 for more information related to the tasks performed by the KCC upon receipt of an IDL_DRSExecuteKCC request.

    ULONG
    IDL_DRSExecuteKCC(
     ...
     ...
     return ExecuteKCCTasks()

I did miss that reference.  Thanks.

However, I do think we need a bit more - the transition between
paragraphs and pseudo-code in the docs is always a bit jarring, but it
is particularly so when it is an inter-doc reference to something that
starts with a much more general description.  Perhaps fill in
ExecuateKCCTasks with this list from 6.2.2, referencing each of the
tasks by number individually?

??Refresh kCCFailedLinks and kCCFailedConnections.
??Create intra-site connections.
??Create inter-site connections.
??Remove unnecessary connections.
??Translate connections.
??Remove unnecessary kCCFailedLinks and kCCFailedConnections.

Thanks,

Andrew Bartlett
--
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.samba.org/pipermail/cifs-protocol/attachments/20150228/f0665f36/attachment.html>