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

Fix for acl server #461

Merged
merged 1 commit into from
Nov 26, 2021

Conversation

Mixaster995
Copy link
Contributor

Signed-off-by: Mikhail Avramenko avramenkomihail15@gmail.com

Description

Removed recreation of acl rules on each request, and added additional cleanup on close
This prevents acl rules creation on each and every request (refresh/heal)

Issue

Closes networkservicemesh/deployments-k8s#2879
Unblocks networkservicemesh/deployments-k8s#3663

… cleanup on close

Signed-off-by: Mikhail Avramenko <avramenkomihail15@gmail.com>
@denis-tingaikin denis-tingaikin merged commit b21fcd1 into networkservicemesh:main Nov 26, 2021
nsmbot pushed a commit to networkservicemesh/cmd-nse-firewall-vpp that referenced this pull request Nov 26, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#461

Commit: b21fcd1
Author: Авраменко Михаил
Date: 2021-11-26 19:35:30 +0700
Message:
  - removed recreation of acl rules on each request, and added additional cleanup on close (#461)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vlan-vpp that referenced this pull request Nov 26, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#461

Commit: b21fcd1
Author: Авраменко Михаил
Date: 2021-11-26 19:35:30 +0700
Message:
  - removed recreation of acl rules on each request, and added additional cleanup on close (#461)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-icmp-responder-vpp that referenced this pull request Nov 26, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#461

Commit: b21fcd1
Author: Авраменко Михаил
Date: 2021-11-26 19:35:30 +0700
Message:
  - removed recreation of acl rules on each request, and added additional cleanup on close (#461)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-forwarder-vpp that referenced this pull request Nov 26, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#461

Commit: b21fcd1
Author: Авраменко Михаил
Date: 2021-11-26 19:35:30 +0700
Message:
  - removed recreation of acl rules on each request, and added additional cleanup on close (#461)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-vpp that referenced this pull request Nov 26, 2021
…k-vpp@main

PR link: networkservicemesh/sdk-vpp#461

Commit: b21fcd1
Author: Авраменко Михаил
Date: 2021-11-26 19:35:30 +0700
Message:
  - removed recreation of acl rules on each request, and added additional cleanup on close (#461)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
@Mixaster995 Mixaster995 deleted the fix/acl-rules branch November 30, 2021 04:10
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.

wget check in nse-composition example is unstable
2 participants