URL: https://github.com/SSSD/sssd/pull/5712
Title: #5712: Health and Support Analyzer - Add request log parsing utility

pbrezina commented:
"""
`--list` output has `(` that looks like should not be there at the beginning:
```
(2021-09-21 10:31:47: CID #21: /usr/bin/bash
```

```
(2021-09-21 10:31:47: CID #23: /usr/bin/sh
   - User by ID
       - UID:0...@ldap.vm
   - Service by name
       - https
   - Service by name
       - https
   - User by ID
       - UID:0...@ldap.vm
   - User by ID
       - UID:0...@ldap.vm
```
I would expect the output to only list the timestampe, id and command. There 
might be additional parameter to make it more verbose and print the rest of the 
information.

For better usage, it may be good to introduce nested subcommands:
```
sssctl analyze request list --verbose
sssctl analyze request show $id [options]
```

This is again something that is handled by argparse in python-nutcli and can be 
extracted (code is 
[here](https://github.com/pbrezina/python-nutcli/blob/master/nutcli/commands.py)
 and 
[here](https://github.com/pbrezina/python-nutcli/blob/master/nutcli/parser.py)).
 Or there are some other alternatives that might be used to create nested 
commands interface and already packaged (click perhaps?)

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5712#issuecomment-924854691
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to