Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reload UFW #170

Merged
merged 1 commit into from May 3, 2018
Merged

Reload UFW #170

merged 1 commit into from May 3, 2018

Conversation

thedac
Copy link
Contributor

@thedac thedac commented May 2, 2018

Add the reload command for UFW as part of the fix for LP Bug #1757564

Add the reload command for UFW as part of the fix for LP Bug #1757564
Copy link
Contributor

@gnuoy gnuoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gnuoy gnuoy merged commit e12fe11 into juju:master May 3, 2018
openstack-gerrit pushed a commit to openstack/charm-swift-storage that referenced this pull request May 4, 2018
In a converged architecture with storage and compute on the same
host, UFW can get in the way of tunneled traffic interpreting it as
INVALID. UFW makes solving this more difficult than it needs to be.
See http://northernmost.org/blog/gre-tunnels-and-ufw/index.html for
context.

This change updates /etc/ufw/before.rules to add GRE as an allowed
input.

Also, guarantee ufw is installed for LP #1763716

Please review and merge charm-helpers first:
juju/charm-helpers#170

Change-Id: I789854c33e3af12f7412633dbf7c921beb0ed2b5
Closes-Bug: #1757564
Closes-Bug: #1763716
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants