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.

Sign up for GitHub

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

Jump to bottom

[19.03] revert controller: Check if IPTables is enabled for arrangeUserFilterRule ENGCORE-1114 #2470

Merged
merged 2 commits into from
Oct 11, 2019
Merged

[19.03] revert controller: Check if IPTables is enabled for arrangeUserFilterRule ENGCORE-1114 #2470

merged 2 commits into from
Oct 11, 2019

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Oct 11, 2019 •
edited by jira bot
Loading

@thaJeztah
[19.03] Revert "Move hasIPTablesEnabled check into firewall_linux.go"
8cdd5a3 
This reverts commit a232658.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
[19.03] Revert "controller: Check if IPTables is enabled for arrangeU…
5cbb5de 
…serFilterRule"

This reverts commit 5e34060.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@arkodg
Copy link
Contributor

arkodg commented Oct 11, 2019

PTAL @selansen @mavenugo

@kolyshkin
Copy link
Contributor

This is not a backport of #2466 to 19.03 branch, but it's identical, right? I look at the changes and they seem to be the same.

@kolyshkin
Copy link
Contributor

unit-tests CI fails:

read tcp 172.27.0.3:34670->104.196.25.67:2376: read: connection timed out
make: *** [Makefile:123: unit-tests] Error 1
Exited with code 2

not sure where it's coming from

@selansen
Copy link
Collaborator

Re running CI

@thaJeztah
Copy link
Member Author

This is not a backport of #2466 to 19.03 branch, but it's identical, right? I look at the changes and they seem to be the same.

@kolyshkin ah, yes, you're right; also see #2467. Perhaps you have thought on that

@andrewhsu andrewhsu changed the title [19.03] revert controller: Check if IPTables is enabled for arrangeUserFilterRule [19.03] revert controller: Check if IPTables is enabled for arrangeUserFilterRule ENGCORE-1114 Oct 11, 2019
@selansen
Copy link
Collaborator

@kolyshkin , All green now

selansen
selansen approved these changes Oct 11, 2019
View reviewed changes
Copy link
Collaborator

@selansen selansen left a comment

Choose a reason for hiding this comment

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

LGTM

kolyshkin
kolyshkin approved these changes Oct 11, 2019
View reviewed changes
Copy link
Contributor

@kolyshkin kolyshkin left a comment

Choose a reason for hiding this comment

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

LGTM

@justincormack justincormack merged commit 3eb3938 into moby:bump_19.03 Oct 11, 2019
@thaJeztah thaJeztah deleted the 19.03_revert_iptables_check branch October 11, 2019 19:03
This was referenced Oct 11, 2019
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@selansen selansen selansen approved these changes

@arkodg arkodg arkodg approved these changes

@kolyshkin kolyshkin kolyshkin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@thaJeztah @arkodg @kolyshkin @selansen @justincormack