Implements relation-ids command

Implements relation-ids command.

There are both API tests and usage in hooks.

https://code.launchpad.net/~jimbaker/juju/relation-ids-command/+merge/99150

Requires: https://code.launchpad.net/~jimbaker/juju/relation-hook-context/+merge/99149

(do not edit description out of merge proposal)

[jimbaker, 2012-03-26 01:50:46]

Please take a look.

[jimbaker, 2012-03-30 06:07:03]

Please take a look.

[hazmat, 2012-04-02 15:44:08]

I'd like to see some more real testing beyond the mock implementations.

https://codereview.appspot.com/5900068/diff/2001/juju/hooks/commands.py
File juju/hooks/commands.py (right):

https://codereview.appspot.com/5900068/diff/2001/juju/hooks/commands.py#newco...
juju/hooks/commands.py:95: default=os.environ.get("JUJU_RELATION"),
if we're in a non relation hook this arg should be required or at least its
behavior documented (afaics it would end up returning all relation ids). That's
unclear though given the string serialization of the None value. At a min it
should have a test.

[jimbaker, 2012-04-05 01:17:38]

Please take a look.

[jimbaker, 2012-04-05 03:30:51]

For some reason, it's now at https://codereview.appspot.com/5988043
 after I tried updating the summary/description with lbox propose -cr -edit

[jimbaker, 2012-04-05 03:33:19]

I have added testing directly using the relation-ids command in hooks.

[jimbaker, 2012-04-05 03:33:28]

https://codereview.appspot.com/5900068/diff/2001/juju/hooks/commands.py
File juju/hooks/commands.py (right):

https://codereview.appspot.com/5900068/diff/2001/juju/hooks/commands.py#newco...
juju/hooks/commands.py:95: default=os.environ.get("JUJU_RELATION"),
The default of JUJU_RELATION is wrong and has been fixed - the default should be
an empty string to indicate it's querying for all relation names.

[hazmat, 2012-04-05 11:05:11]

looks good, +1