Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Relate via #7813
Conversation
| @@ -218,6 +219,13 @@ func (s stateShim) AddRelation(eps ...state.Endpoint) (Relation, error) { | ||
| return stateRelationShim{r}, nil | ||
| } | ||
| +type RelationNetworks state.RelationNetworks |
axw
Aug 31, 2017
Member
I don't see any point to this, state.RelationNetworks is already an interface.
|
$$merge$$ |
|
Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju |
jujubot
merged commit 4e1be75
into
juju:develop
Aug 31, 2017
1 check failed
continuous-integration/jenkins/pr-merge
This commit cannot be built
Details
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
wallyworld commentedAug 31, 2017
Description of change
We now support adding a remote relation and specifying the egress subnets from which outbound traffic will originate. This compliments the existing model wide config attribute "egress-subnets".
The first commit renames the model attribute "egress-cidrs" to "egress-subnets".
The second commit addes the --via support.
The UX is "juju add-relation foo bar --via cidr1,cidr2,..."
QA steps
juju bootstrap aws dbctrl
juju deploy mysql
juju offer mysql:db
From behind a firewall or cgnat:
juju bootstrap lxd mwctrl
juju deploy mediawiki
juju relate mediawiki:db dbctrl:defauly.mysql --via <what's-my-ip>/32
check aws firewall ports and check that media wiki is operational