There should be two blank lines between items at the lowest indentation level. Here and below.

This should probably propagate up the message in the exception.

This should probably check that the command was sucessful.

Should it be up to the caller to pass in a compiled regex with the options that they care to use?

This can just return directly instead of assigning to a variable and returning that.

Two blank lines between top-level blocks. Here and below.

It might be nice to pass in arguments as a list instead of relying on shell=True to split it.

Relying on the pipe here makes it so this only works on unix-type systems (and even then relies a lot on shell=True parsing this). You could do this filtering pretty easily in python instead.

You could use '0' * 40 here.

No need for this line.

No need for this line.

Guess I never published this. Meant to :/

Just curious. Why add this function instead of just instantiating RBClient where needed?

"URL"

So I believe that Christian's comment about _make_api_client was that it's only used in this one place, so instead of wrapping it in a function you can just replace this line with:

api_client = RBClient(server_url, username=username,
                      password=password)

This looks good to me. Are there any other changes you want to make before I push it?

Ship It!