Add option to disable users

[jcpunk]

Rather than just setting the password to a random value, this extends the user management code to let a user be fully disabled.

[jcpunk]

I do not understand the test failures...

[jhoblitt]

@jcpunk I am able to reproduce the errors.

[jcpunk]

I think I've got the tests passing. You can check the dep loop by pulling down this branch and dropping the require ipmi::install for require ipmi and running the test suite.

I'm showing:

dependency cycles found: (Exec[ipmi_user_add_newuser] => Ipmi::User[newuser] => Class[Ipmi] => Ipmi::User[newuser] => Exec[ipmi_user_add_newuser])

[jcpunk]

Just checking back in.

[jhoblitt]

Sorry for the slow response. I wanted to try to test this a box with a bmc before merging it. It looks like the gha failure on centos 8 is bogus. Could you touch the branch to retrigger it? It isn't giving me the re-run button.

[jcpunk]

Any guesses why it doesn't like REFERENCE.md? I'm showing it is the current copy.

[jhoblitt]

@jcpunk running bundle exec rake reference on this branch does result in changes for me.

[jcpunk]

I've been running puppet strings generate --format markdown --out REFERENCE.md is that not the way?

I've run the other command and it is indeed different?

[jhoblitt]

LGTM