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

Explainability #416

Merged
merged 47 commits into from
Dec 9, 2024
Merged

Explainability #416

merged 47 commits into from
Dec 9, 2024

Conversation

tanyaveksler
Copy link
Contributor

No description provided.

@tanyaveksler
Initial implementation of explainability - adding ImplyingRules to th…
c562b5a 
…e infrastructure.
@tanyaveksler
More infrastructure changes for explaibability
451000c
@tanyaveksler
Merge branch 'support_admin_netpolicy' into explainability
6a120c0
@tanyaveksler
Added explainability data from anp/banp and ingress policies
898c672
@tanyaveksler
Fixing linter errors
f78442f
@tanyaveksler
Moved to models v0.5.0 release (netset instead of iplock package).
fe063fe 
More infrastructure changes for explainability
@tanyaveksler
Added initiation of CommonImplyingRules
b299c17
@tanyaveksler
Minor fixes
fc5b9d0
@tanyaveksler
Fixing corner case in AddAugmentedInterval
1e4c8d6
@tanyaveksler
Merge branch 'support_admin_netpolicy' into explainability
d076eaf
@tanyaveksler
Corner case fix in AddAugmentedInterval;
26bd02b 
Writing connection results in canonical form
@tanyaveksler
Fixing bug in AugmentedCanonicalSet.Union (adding only 'in-set' inter…
0dec8a5 
…vals
@tanyaveksler
Merge branch 'support_admin_netpolicy' into explainability
8b90271
@tanyaveksler
Canonize port set and filter out blocked ports before comparing to ex…
b7dbf1f 
…pected results.
@tanyaveksler
preserving 'inSet' information of PortSet
39ebe99
@tanyaveksler
Fixing AugmentedCanonicalSet.String() - better handling semantically …
0729265 
…empty intervals
@tanyaveksler
Fixed comparsion of ConnectionSets
9dc0d7f
@tanyaveksler
Using canonical connection set for returning exposure analysis results.
faad5f1 
Fixing a bug in ConnectionSet.ContainedIn function.
@tanyaveksler
Merge branch 'support_admin_netpolicy' into explainability
3711344
@tanyaveksler
Removed unused function
d203565
@tanyaveksler
Merge branch 'main' into explainability
dba26dd
@tanyaveksler
Fix typo
50fefb5
@tanyaveksler
Avoid using require.Contains, since it fails when capacities are not …
a19be86 
…equal.
@tanyaveksler
Fixed a bug in GetEquivalentCanonicalConnectionSet - handling NamedPo…
d2ae852 
…rts as well.
@tanyaveksler
make linter happy
2c99082
@tanyaveksler
make linter even more happy
d698c96
@tanyaveksler
Added command-line option --explain.
dcc5bda 
More delicate handling of CommonImplyingRules.
@tanyaveksler
Merge branch 'main' into explainability
a7c06e8
@tanyaveksler
Adding 'explain' flag to all conns formatters
58266f0
@tanyaveksler
Make linter happy
52d3235
@tanyaveksler
Make linter more happy
45c7789
@tanyaveksler
Merge branch 'support_admin_netpolicy' into explainability
c868140
@tanyaveksler
Splitting explainability data into Ingress and Egress directions data.
7a38cb7
@tanyaveksler
make linter happy
9c0ab85
@tanyaveksler
Better handling explainability data for system default
feca3a3
@tanyaveksler
Another fix in handling system default connections
010a007
@tanyaveksler
Improving explainability print format.
010ac87
@tanyaveksler
Unifying similar functions. Makeing linter happy
f945d5b
@tanyaveksler
Make linter more happy
8131e85
@tanyaveksler
Better formatting explainability info for ingress
9aa2ea2
@tanyaveksler
More readable formatting of explainability output;
b484b33 
explaining denied connections as well.
@tanyaveksler
make linter happy
bf30935
@tanyaveksler
Added printing Allow/Deny result separtely for Ingress and Egress dir…
e6b8a41 
…ections.
@tanyaveksler
make linter happy
e09d704
@tanyaveksler
More accurate accumulation of implying rules.
a6f81fb
@tanyaveksler
Different handling os pass connections (to allow preserving explainab…
7c7bb5a 
…ility data)
@tanyaveksler
Added explainability tests method and initial explainability tests
e742b6c
@tanyaveksler tanyaveksler changed the base branch from main to support_admin_netpolicy December 9, 2024 12:18
@tanyaveksler tanyaveksler marked this pull request as ready for review December 9, 2024 12:19
@tanyaveksler tanyaveksler merged commit d738af9 into support_admin_netpolicy Dec 9, 2024
@tanyaveksler tanyaveksler deleted the explainability branch December 9, 2024 12:25
tanyaveksler added a commit that referenced this pull request Dec 9, 2024
@tanyaveksler
Revert "Explainability (#416)"
98e2267 
This reverts commit d738af9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size/XXL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tanyaveksler