/juju

Add support for relation egress networks in state model #7808

Merged
merged 1 commit into from Aug 30, 2017

Conversation

Reviewers

@axw axw

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@wallyworld @jujubot @axw
@wallyworld
Owner

wallyworld commented Aug 30, 2017

Description of change

This is smaller than it looks - the PR doesn't reflect that state/relationingress.go is renamed to state/relationnetworks.go.

The relation ingress model in state is extended to support egress networks as well. Tweaks to the ingress network watcher were needed to filter out egress networks.

QA steps

Run up a CMR scenario and ensure the expected ports are opened in the offering model.
@axw

axw approved these changes Aug 30, 2017
View changes

@wallyworld
Owner

wallyworld commented Aug 30, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Aug 30, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju

@jujubot jujubot merged commit 2740410 into juju:develop Aug 30, 2017
1 check passed

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment