-
Notifications
You must be signed in to change notification settings - Fork 9
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
RFC for K8S Commons library #28
base: main
Are you sure you want to change the base?
Conversation
Commons method for geting pods Signed-off-by: Vicente Zepeda Mas <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #28 +/- ##
=======================================
Coverage 85.13% 85.13%
=======================================
Files 6 6
Lines 343 343
=======================================
Hits 292 292
Misses 37 37
Partials 14 14 |
Signed-off-by: Vicente Zepeda Mas <[email protected]>
Signed-off-by: Vicente Zepeda Mas <[email protected]>
Signed-off-by: Vicente Zepeda Mas <[email protected]>
Thanks for this initial draft Vicente! |
@chentex Design looks very complex. Lets follow |
Type of change
Description
Trying to create Commons methods that could be used simply by other project in our Orgs.
This is an RFC please be sure to review the code ask questions and comment.
I went with the simplest way to create a client at this point it is not set in stone, basing my methods of what I've seen mainly in K8S-netperf repo.
I'll be adding methods during this development.
Code might change during the development, so marking this as a draft.
Main Feedback Requested
Related Tickets & Documents
Draft analysis of the methods: https://docs.google.com/document/d/1ZlB7k1V6jpOP_Sy9qP6GDfVasg7k8DEuQPIPlaIwghw/edit
Checklist before requesting a review
Testing