"relation-set --file -" doesn't seem to work

[Björn Tillenius]

With Juju 1.23.2, I'm not able to get "relation-set --file -" to read the file from stdin.

As a test, I have a file settings.yaml:

  {foo: bar}

To begin with the values isn't set:

  landscape:~> juju run --unit landscape-server/0 'relation-get -r amqp:4 - landscape-server/0'
  private-address: 10.0.3.61

Now we try with --file -:

  landscape:~> juju run --unit landscape-server/0 'cat /tmp/settings.yaml | relation-set -r amqp:4 --file -'

The value still isn't set:

  landscape:~> juju run --unit landscape-server/0 'relation-get -r amqp:4 - landscape-server/0'
  private-address: 10.0.3.61

Now, using --file settings.yaml seems to work, though:

  landscape:~> juju run --unit landscape-server/0 'relation-set -r amqp:4 --file /tmp/settings.yaml'
  landscape:~> juju run --unit landscape-server/0 'relation-get -r amqp:4 - landscape-server/0'
  foo: bar
  private-address: 10.0.3.61

Am I doing something wrong?

[Jesse Meek (waigani)]

The fix has been reverted as it introduced this bug: https://bugs.launchpad.net/juju-core/+bug/1463117. Please see the comments on that bug.

[Andrew Wilkins (axwalk)]

New fix proposed on master: https://github.com/juju/juju/pull/2594

[Andrew Wilkins (axwalk)]

I've canned PR2594 because it would break gocharm. I'll repropose later, similar to what I originally proposed but with some changes to make reading of stdin conditional.

[Andrew Wilkins (axwalk)]

Attempt 3: https://github.com/juju/juju/pull/2603