Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Fix remote relation register #6773
Conversation
wallyworld
and others
added some commits
Jan 6, 2017
| - remoteEndpointName string | ||
| + applicationId params.RemoteEntityId | ||
| + localEndpoint params.RemoteEndpoint | ||
| + remoteApplicationLocalAlias string |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
jujubot
merged commit 6a2a2f3
into
juju:develop
Jan 6, 2017
1 check passed
github-check-merge-juju
Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
wallyworld commentedJan 6, 2017
•
Edited 1 time
-
wallyworld
Jan 6, 2017
The name of the remote application as recorded in the consuming model is set to user-model-appname.
But when we register with the offering model, we need to pass in the app name from the offering side.
Also fixup some error handing in the remote relation worker.
QA: bootstrap and add a remote relation