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

Evaluate using dynamic groups inside of AddressObject #101

Open
Kircheneer opened this issue Oct 20, 2022 · 3 comments
Open

Evaluate using dynamic groups inside of AddressObject #101

Kircheneer opened this issue Oct 20, 2022 · 3 comments
Labels
status: action required This issue requires additional information to be actionable status: blocked Another issue or external requirement is preventing implementation

Comments

@Kircheneer
Copy link
Contributor

Environment

  • Nautobot version: 1.4.4
  • nautobot-plugin-firewall-model version: 1.1.0

Proposed Functionality

Allow AddressObjectGroup to source its members from a dynamic group. This could for example be a new ForeignKey field on the model pointing to DynamicGroup.

Use Case

All prefixes with the role user-lan should have access to a set of services. Instead of manually updating the policy rule (or NAT policy rule) whenever there are changes, we could instead use dynamic groups to automatically accomplish that.

@whitej6
Copy link
Contributor

whitej6 commented Oct 24, 2022

At this time the performance limitations with dynamic groups makes this a non-starter at scale. We should keep the issue and reevaluate pending performance improvements for dynamic groups.

Dynamic Groups start having performance degradation over the 500 groups count on a single content type.

@whitej6 whitej6 added status: action required This issue requires additional information to be actionable status: blocked Another issue or external requirement is preventing implementation labels Nov 18, 2022
@nniehoff
Copy link

I feel like the development and added functionality this would bring to this plugin would be worth it to keep moving forward, even if there is a warning or a opt in configuration parameter to enable this. Hopefully when the feature is ready in this plugin Nautobot Core would have an update... Because I really want this feature

@whitej6
Copy link
Contributor

whitej6 commented Nov 22, 2022

I have some work on logical grouping which is solving this from a different PoV. I see this really relevant when it comes to controller based systems and how each group is treated as unique and how the group has access to certain items depending on where it's at within the tree. From a firewall object modeling perspective some of the patterns in dynamic groups does not solve this problem case and introduces a level of complexity in managing uniqueness.

I'm open to discussions around the topic and personally would love to swap out <type>ObjectGroup to a generic upstream group & how we can assign X content types to the same group. I am committed to keeping this issue open until we have a more elegant solution available in the plugin/core.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: action required This issue requires additional information to be actionable status: blocked Another issue or external requirement is preventing implementation
Projects
None yet
Development

No branches or pull requests

3 participants