Skip to content

add ability to set all firewall rules atomically #143

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

Closed
rzezeski opened this issue May 6, 2022 · 1 comment
Closed

add ability to set all firewall rules atomically #143

rzezeski opened this issue May 6, 2022 · 1 comment

Comments

@rzezeski
Copy link
Contributor

rzezeski commented May 6, 2022

From what I recall and what others have said in the past it sounds like we want the ability to be able to set firewall rules atomically. That is, instead of adding/removing individual rules, the sled agent will always present the entire set of rules it wants, expecting that OPTE puts them all in place atomically. Whether or not this scales into the future I am not sure. And I imagine we could at least do some type of delta-based update in the future. But for now the easiest thing is to just give the large atomic hammer of setting/replacing all firewall rules at once.

rzezeski added a commit that referenced this issue May 11, 2022
Need UFT invalidation on layer/rule changes (#21)
rzezeski added a commit that referenced this issue May 11, 2022
Need UFT invalidation on layer/rule changes (#21)
@rzezeski
Copy link
Contributor Author

Addressed in 70785d9.

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

No branches or pull requests

1 participant