<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    1. perhaps instead of saying:<br>
    Currently the only valid sub‐ system is tps.<br>
    we say<br>
    Currently the only supported sub‐ system is tps.<br>
    <br>
    2.pki [CLI options] <subsystem>-audit<br>
               This command is to list the available audit commands the
    subsystem.<br>
    do you mean<br>
               This command is to list the available audit commands <b>of</b>
    the subsystem<br>
    ?<br>
    <br>
    3.pki [CLI  options]  <subsystem>-audit-mod  --action 
    <action>  [command<br>
           options]<br>
               This command is to the audit status in the subsystem.<br>
    do you mean<br>
               This command is to <b>modify</b> the audit status in the
    subsystem.<br>
    ?<br>
    also, what is the audit status?  if its enable/disable status, how
    about:<br>
               This command is to <b>modify</b> the audit system
    (enabled/disabled) status in the subsystem.<br>
    ?<br>
    <br>
    4.        To enable/disable audit in TPS execute the following
    command:<br>
    try with a punctuation<br>
           To enable/disable audit in TPS<b>,</b> execute the following
    command:<br>
    <br>
    thanks,<br>
    Christina<br>
    <br>
    <div class="moz-cite-prefix">On 07/17/2015 03:29 PM, Endi Sukma
      Dewata wrote:<br>
    </div>
    <blockquote cite="mid:55A981CB.1070603@redhat.com" type="cite">On
      7/15/2015 5:13 PM, Endi Sukma Dewata wrote:
      <br>
      <blockquote type="cite">Thanks for the comments. Please take a
        look at the updated patch.
        <br>
        <br>
        On 7/14/2015 12:17 PM, Matthew Harmsen wrote:
        <br>
        <blockquote type="cite">This was a little confusing as the
          ticket is for a kra-audit man page.
          <br>
          I think I understand a bit better after reading comment #4,
          <br>
        </blockquote>
        <br>
        Yes, the command is removed from all subsystems except TPS
        because of
        <br>
        the database upgrade requirement.
        <br>
        <br>
        <blockquote type="cite">however, I
          <br>
          still have the following questions/comments:
          <br>
          <br>
            * What is '<TPS admin authentication>', and how does
          one obtain this?
          <br>
        </blockquote>
        <br>
        I added a reference to the Authentication section in pki(1) and
        an
        <br>
        explanation that the user must be in the Administrators group.
        <br>
        <br>
        <blockquote type="cite">  * The 'pki tps-audit-mod --help'
          command shows the following:
          <br>
          <br>
              usage: tps-audit-mod [OPTIONS...]
          <br>
                   --action <action>   Action: update (default),
          enable, disable.
          <br>
                   --help              Show help options
          <br>
                   --input <file>      Input file containing audit
          configuration.
          <br>
                   --output <file>     Output file to store audit
          configuration.
          <br>
          <br>
              where the man page does not talk about the 'update
          (default)' switch
          <br>
              for the '--action <action>' option.
          <br>
          <br>
          One or more EXAMPLES would be nice, although this will
          probably suffice
          <br>
          for the first pass at a man page.
          <br>
        </blockquote>
        <br>
        To clarify this I created a separate description for each usage:
        <br>
        * pki tps-audit-mod --action <action>
        <br>
        * pki tps-audit-mod --input <file path>
        <br>
        <br>
        I removed the "update (default)" from the CLI since it can be
        confusing.
        <br>
      </blockquote>
      <br>
      Please take a look at the new patch (#623-2). The patch now
      contains just the new pki-audit man page and line wrapping in the
      pki man page.
      <br>
      <br>
      The CLI changes to remove the non-working command from non-TPS
      subsystems have been posted as a separate patch (#636).
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>