candlepin / rho

ssh scanner for your network
GNU General Public License v2.0
32 stars 20 forks source link

Pack-scan changes #12

Closed cnsnyder closed 10 years ago

jmrodri commented 10 years ago

Seems to work. I tested it against a remote RHEL machine and a local Fedora machine. Using both the script and rho itself.

jmrodri commented 10 years ago

CONDACK, answer the question about pack_scan.sh and its relevance and we can move forward. If it isn't shipped it might be worth documenting how to emulate pack_scan in the README and remove the shell script. Or we need to add it to bin and put it in the spec file

alikins commented 10 years ago

Needs a pep8/flake8'ifying.

(And I suppose, jenkins test runners...)

alikins commented 10 years ago

"Scanner loads all cmds defined in rho_cmds and not in NONDEFAULT_CMDS."

Why?

jmrodri commented 10 years ago

@alikins that is a good catch on the NONDEFAULT. It used to be a physical list of commands we wanted listed. And it was changed to be the opposite. I.e. include all except the given list. I seem to remember we made it explicit before on purpose. I'll see if I can find out why.

jmrodri commented 10 years ago

ACK