Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

openbmctool: help text #42

Open
ssombar opened this issue Sep 28, 2018 · 2 comments
Open

openbmctool: help text #42

ssombar opened this issue Sep 28, 2018 · 2 comments
Assignees

Comments

@ssombar
Copy link

ssombar commented Sep 28, 2018

openbmctool.py version = 1.07

  1. Is "blink "a valid option in chassis identify?
    bash-4.1$ python3 openbmctool.py -H $BMC -U root -P 0penBmc chassis identify -h
    usage: openbmctool.py chassis identify [-h] {on,off,status}
    positional arguments:
    {on,off,status} The control option for the led: on, off, blink, status <------ is blink valid?

  2. Is "info" a valid option for bmc?
    bash-4.1$ python3 openbmctool.py -H $BMC -U xxxx -P xxx bmc -h
    usage: openbmctool.py bmc [-h] {reset} ...
    positional arguments:
    info Displays information about the BMC hardware, including device
    revision, firmware revision, IPMI version supported,
    manufacturer ID, and information on additional device support.

@ssombar ssombar changed the title openbmctool: chassis identify help text blink option openbmctool: help text Sep 28, 2018
@amboar
Copy link
Member

amboar commented Sep 28, 2018

@ssombar thanks for continuing to identify issues in openbmctool. In the future though, please take advantage of GitHub's markdown formatting: https://guides.github.com/features/mastering-markdown/

Particularly, if you are pasting command line output, the triple-backtick markdown will make the output easy for everyone to identify and ensures GitHub will render the output correctly. Here's an example:

bash-4.1$ python3 openbmctool.py -H $BMC -U root -P 0penBmc chassis identify -h
usage: openbmctool.py chassis identify [-h] {on,off,status}
positional arguments:
{on,off,status} The control option for the led: on, off, blink, status

@thalerj
Copy link

thalerj commented Nov 16, 2018

@ssombar bmc info is a function that needs to be implemented. Will need to pull firmware levels and hardware info from the inventory.

for the chassis identify blink, I think this needs to be removed. Early discussion and design allowed for blink versus solid on. It is my understanding that only blink is supported now for the identify LED.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants