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

[Endi Sukma Dewata]

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