adding a few tweaks and fixes/workarounds
- looks like the ssh url for gerrit is not working from the releng VMs
so trying with https now
- the CommandFailed class was broken and it appears to be that there
was a missing docstring for that class, so self.__doc__ was None
and doing a return with something None + something else was crashing.
So, I added a basic docstring there. I know this is code similar to
another project (e.g. git-review), so I'm not so confident it's
the right thing, but its working for me. It was noticed because the
ssh url to gerrit was being rejected so that class was getting
excercised
- I did a few modifications based on what my IDE told me was good
python
- I removed one and added one print output to make it more clear
what's going on (to me, at least)
- fixed one variable naming typo
- defining the ssh URL in a single location
- verbose cmdline arg was overriding VERBOSE to 0 even if it was modified inline
Change-Id: I623e67ffb71660b3d9c5a55b0b31bc93dab4df55
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>