-
Notifications
You must be signed in to change notification settings - Fork 359
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
feat: Creating a Custom Resource Definition for a pod IP to metadata mapper. #1071
Closed
Closed
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
c31ed60
feat: Creating a Custom Resource Definition for the PodInfo operator.
prateek041 41de71f
feat: Creating a Custom Resource Definition for the PodInfo operator.
prateek041 67556ff
feat: Creating a Custom Resource Definition for the PodInfo operator.
prateek041 d87117d
Remove go.Mod file from podinfo directory.
prateek041 cca9008
Cleared the file since the tests will be written using testify, and n…
prateek041 ad830c1
Mirror k8s pod resource
prateek041 a1f2459
Renamed the project from podinfo to tetragonpod.
prateek041 52e59e3
Created an exporter, that will create the CRD object and return to th…
prateek041 f02a0ee
Integrated the registration of tetragonPod CRD into the tetragon oper…
prateek041 3ef1e9c
Replaced podinfo with tetragonpods.
prateek041 8b682c4
Updated the Logic for registering TetragonPod CustomResourceDefinitio…
prateek041 bf134c7
cleanup
prateek041 58ce306
Added cluster role for tetragon pod controller
prateek041 6d25981
added role binding and service account for tetragonpod controller
prateek041 336aaf3
Added deployment for the tetragonpod controller
prateek041 dbb2856
Modified files for adding labels and default values
prateek041 85b5baf
Modified tetragonpod controller to check if it gets pod resources
prateek041 087ef7a
Changed github workflows to build image of the tetragonpod controller
prateek041 d9b4019
Added flag and logic for skipping the tetragonPod CRD
prateek041 d8271a4
Added flag in the default value
prateek041 c12fd65
Cleanup of unused files
prateek041 ddfd1d9
Resolving merge conflicts
prateek041 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe my comment is stupid: doc says "the ports that will be used must be specified" but doesn't specify where or how? maybe if it makes sense add it here too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for pointing that out. I will make the changes.
@tixxdz
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't mad the changes there yet, will do right before the PR is ready.