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

Make iptables path consistent across dists #2266

Merged
merged 3 commits into from
May 21, 2020
Merged

Conversation

zolia
Copy link
Contributor

@zolia zolia commented May 19, 2020

fixes: #2146 and #2241

/usr/sbin/iptables is a preferred way according recent distributions. Ensuring that this path exist.

  • make links to iptables in deb post install
  • ensure myst finds iptables in docker image

@zolia zolia changed the title Make iptables path consistent acrosss dists Make iptables path consistent across dists May 19, 2020
@zolia zolia marked this pull request as draft May 19, 2020 15:27
install.sh Outdated Show resolved Hide resolved
install.sh Outdated Show resolved Hide resolved
@zolia zolia force-pushed the bugfix/fix-iptables-path branch 3 times, most recently from a169031 to 1fe4ca5 Compare May 20, 2020 08:43
@zolia zolia marked this pull request as ready for review May 20, 2020 08:44
install.sh Outdated Show resolved Hide resolved
@zolia zolia force-pushed the bugfix/fix-iptables-path branch from 1fe4ca5 to 8edf16c Compare May 20, 2020 10:24
@codecov-commenter
Copy link

codecov-commenter commented May 20, 2020

Codecov Report

Merging #2266 into master will decrease coverage by 0.04%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2266      +/-   ##
==========================================
- Coverage   48.01%   47.96%   -0.05%     
==========================================
  Files         273      273              
  Lines       12757    12757              
==========================================
- Hits         6125     6119       -6     
- Misses       6044     6047       +3     
- Partials      588      591       +3     
Impacted Files Coverage Δ
nat/service_iptables.go 0.00% <0.00%> (ø)
core/policy/oracle.go 86.48% <0.00%> (-5.41%) ⬇️
nat/traversal/pinger.go 72.33% <0.00%> (-1.95%) ⬇️
core/discovery/discovery.go 64.75% <0.00%> (-0.82%) ⬇️
tequilapi/endpoints/sse_handler.go 74.78% <0.00%> (+2.60%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d3aff1...9cd7bf6. Read the comment docs.

@zolia zolia force-pushed the bugfix/fix-iptables-path branch from 8edf16c to 0e388a4 Compare May 20, 2020 12:24
bin/helpers/prepare-run-env.sh Outdated Show resolved Hide resolved
bin/helpers/prepare-run-env.sh Outdated Show resolved Hide resolved
bin/package/installation/post-install.sh Outdated Show resolved Hide resolved
@zolia zolia force-pushed the bugfix/fix-iptables-path branch from 0e388a4 to 8b1a0e1 Compare May 21, 2020 06:20
@zolia zolia requested a review from soffokl May 21, 2020 06:21
@zolia zolia force-pushed the bugfix/fix-iptables-path branch from 8b1a0e1 to 745e6c3 Compare May 21, 2020 07:03
@zolia zolia force-pushed the bugfix/fix-iptables-path branch from 828ef6b to 9cd7bf6 Compare May 21, 2020 09:59
@zolia zolia merged commit bf4b0b9 into master May 21, 2020
@zolia zolia deleted the bugfix/fix-iptables-path branch May 21, 2020 10:41
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

4 participants