[autobackport: sssd-2-13] Fix: sss_analyze.py modified to print correct command in usage field#8734
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the argparse.ArgumentParser initialization in src/tools/analyzer/sss_analyze.py to explicitly set the program name (prog) to 'sssctl analyze'. This ensures that the help output displays the correct command name. There are no review comments, so I have no additional feedback to provide.
Explanation: This patch will make sure to print correct command on running 'sssctl analyze --help' command avoiding confusion. Resolves: SSSD#8718 Reviewed-by: Justin Stephenson <[email protected]> (cherry picked from commit 30a4940)
|
The pull request was accepted by @justin-stephenson with the following PR CI status: 🟢 rpm-build:centos-stream-10-x86_64:upstream (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
16a0cdf to
effea8a
Compare
3 participants
This is an automatic backport of PR#8723 Fix: sss_analyze.py modified to print correct command in usage field to branch sssd-2-13, created by @asakure.
Please make sure this backport is correct.
Note
The commits were cherry-picked without conflicts.
You can push changes to this pull request
Original commits
30a4940 - Component: sssd-tools
Backported commits
Original Pull Request Body
Explanation: This patch will make sure to print correct command i.e.
sssctl analyzein usage field on runningsssctl analyze --helpcommand avoiding confusion.Resolves: #8718