Skip to content

[backport 1.5] AGW: datapath: Fix iptable rule config.#7151

Merged
pshelar merged 1 commit intomagma:v1.5from
pshelar:fix-iptable-rules-v1.5
May 25, 2021
Merged

[backport 1.5] AGW: datapath: Fix iptable rule config.#7151
pshelar merged 1 commit intomagma:v1.5from
pshelar:fix-iptable-rules-v1.5

Conversation

@pshelar
Copy link
Copy Markdown
Contributor

@pshelar pshelar commented May 25, 2021

Rather than adding rule on every restart, check for rule.
this avoids accumulation of iptable rules on every service restart.

Signed-off-by: Pravin B Shelar [email protected]

Summary

Test Plan

tested on lab AGW.

Additional Information

  • This change is backwards-breaking

Rather than adding rule on every restart, check for rule.
this avoids accumulation of iptable rules on every service restart.

Signed-off-by: Pravin B Shelar <[email protected]>
@pshelar pshelar requested review from ardzoht and koolzz as code owners May 25, 2021 23:29
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines. label May 25, 2021
@magmabot magmabot added component: agw Access gateway-related issue component: cwag CWAG related issues labels May 25, 2021
@pshelar pshelar merged commit 6c307aa into magma:v1.5 May 25, 2021
m-trojanowski pushed a commit to openEPC/magma that referenced this pull request Oct 20, 2021
Rather than adding rule on every restart, check for rule.
this avoids accumulation of iptable rules on every service restart.

Signed-off-by: Pravin B Shelar <[email protected]>
m-trojanowski pushed a commit to openEPC/magma that referenced this pull request Oct 20, 2021
Rather than adding rule on every restart, check for rule.
this avoids accumulation of iptable rules on every service restart.

Signed-off-by: Pravin B Shelar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: agw Access gateway-related issue component: cwag CWAG related issues size/S Denotes a PR that changes 10-29 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants