[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Pki-devel] [PATCH] 623 Added pki-audit man page.



On 7/17/2015 6:12 PM, Christina Fu wrote:
1. perhaps instead of saying:
Currently the only valid sub‐ system is tps.
we say
Currently the only supported sub‐ system is tps.

Fixed.

2.pki [CLI options] <subsystem>-audit
            This command is to list the available audit commands the
subsystem.
do you mean
            This command is to list the available audit commands *of*
the subsystem
?

Fixed. I used "for" to be consistent with other man pages.

3.pki [CLI  options]  <subsystem>-audit-mod  --action <action>  [command
        options]
            This command is to the audit status in the subsystem.
do you mean
            This command is to *modify* the audit status in the subsystem.
?

Fixed. I meant to say "change".

also, what is the audit status?  if its enable/disable status, how about:
            This command is to *modify* the audit system
(enabled/disabled) status in the subsystem.
?

Fixed. Audit status is referring to the log.instance.SignedAudit.enable property in the CS.cfg.

4.        To enable/disable audit in TPS execute the following command:
try with a punctuation
        To enable/disable audit in TPS*,* execute the following command:

Fixed.

thanks,
Christina

Conditionally ACKed by cfu. Pushed to master.

--
Endi S. Dewata


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]