Re: [Pki-devel] [PATCH] 623 Added pki-audit man page.
Thanks for the comments. Please take a look at the updated patch.
On 7/14/2015 12:17 PM, Matthew Harmsen wrote:
This was a little confusing as the ticket is for a kra-audit man
page.
I think I understand a bit better after reading comment #4,
Yes, the command is removed from all subsystems except TPS because of
the database upgrade requirement.
however, I
still have the following questions/comments:
* What is '<TPS admin authentication>', and how does one obtain this?
I added a reference to the Authentication section in pki(1) and an
explanation that the user must be in the Administrators group.
* The 'pki tps-audit-mod --help' command shows the
following:
usage: tps-audit-mod [OPTIONS...]
--action <action> Action: update (default), enable, disable.
--help Show help options
--input <file> Input file containing audit configuration.
--output <file> Output file to store audit configuration.
where the man page does not talk about the 'update (default)' switch
for the '--action <action>' option.
One or more EXAMPLES would be nice, although this will probably suffice
for the first pass at a man page.
To clarify this I created a separate description for each usage:
* pki tps-audit-mod --action <action>
* pki tps-audit-mod --input <file path>
I removed the "update (default)" from the CLI since it can be confusing.
--
Endi S. Dewata
Attachments:
- pki-edewata-0623-1-Added-pki-audit-man-page.patch (text/plain — 22.2 KB)