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

adding fake ip for pod from core #127

Merged
merged 14 commits into from
May 9, 2023
Merged

adding fake ip for pod from core #127

merged 14 commits into from
May 9, 2023

Conversation

shireenf-ibm
Copy link
Contributor

No description provided.

@shireenf-ibm shireenf-ibm requested a review from adisos May 1, 2023 09:09
@adisos
Copy link
Collaborator

adisos commented May 1, 2023

please read this issue, as it is relevant: #44
does your change assign fake ip only to pods read from files?

@shireenf-ibm
Copy link
Contributor Author

please read this issue, as it is relevant: #44 does your change assign fake ip only to pods read from files?

not sure, will re-check and update

pkg/netpol/eval/internal/k8s/pod.go Outdated Show resolved Hide resolved
pkg/netpol/eval/internal/k8s/pod.go Outdated Show resolved Hide resolved
pkg/netpol/eval/internal/k8s/pod.go Show resolved Hide resolved
pkg/netpol/scan/scan.go Outdated Show resolved Hide resolved
@shireenf-ibm shireenf-ibm requested a review from adisos May 8, 2023 12:49
pkg/netpol/scan/scan.go Outdated Show resolved Hide resolved
pkg/netpol/scan/scan.go Outdated Show resolved Hide resolved
@shireenf-ibm shireenf-ibm requested a review from adisos May 8, 2023 13:10
pkg/netpol/scan/scan.go Outdated Show resolved Hide resolved
pkg/netpol/scan/scan.go Show resolved Hide resolved
shireenf-ibm and others added 2 commits May 8, 2023 16:13
@shireenf-ibm shireenf-ibm requested a review from adisos May 8, 2023 13:24
@shireenf-ibm shireenf-ibm merged commit 7f8df93 into main May 9, 2023
@shireenf-ibm shireenf-ibm deleted the fake_pod_id branch May 9, 2023 06:01
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.

2 participants