Split kube-burner repository #528
Replies: 4 comments
-
Works for me. My only comment would be we need to properly doc and probably add an alert/help notification if a user tries to run kube-burner ocp .... for the time being |
Beta Was this translation helpful? Give feedback.
-
This is a good idea. kube-burner will focus on implementing a spec (yaml) which has "jobs". It can focus on optimizing k8s requests etc. |
Beta Was this translation helpful? Give feedback.
-
+1 on the split. I would also like to advocate for moving both of these repos into a new kube -burner org to manage the growth and traction kube-burner is seeing. That would also help us add additional repositories like those that house the grafonnet code for kube-burner specific dashboards and pave the way to build a wider ecosystem around kube-burner. |
Beta Was this translation helpful? Give feedback.
-
I like this direction. Instead of focusing on OCP wrapper and workload (job) definitions, make kube-burner a tool that runs defined jobs on a Kubernetes cluster and collects the metrics. 👍 |
Beta Was this translation helpful? Give feedback.
-
The quantify of code of the kube-burner repository has grown considerably in the latest months. Part of this growth is thanks to the contributions made to OCP wrapper code and workloads.
Holding the whole OCP wrapper code in this repository is becoming an issue, for several reasons I'm gonna try to summarize along these lines:
My proposal to make this split is to take the OCP wrapper code to a new repository
kube-buner-ocp
. The code of this new repository will be:An overview of how this repository would look like can be found at https://github.com/rsevilla87/kube-burner-ocp, compiling the code from this repository results in having a new
kube-burner-ocp
binary with only the ocp wrapper options:There're several things we need to consider like:
kube-burner-ocp
is made into kube-burner core, we will need to cut releases in both kube-burner and kube-burner-ocp to get a binary with that change.cc: @smalleni @jtaleric @vishnuchalla @dry923 @josecastillolema @paigerube14 @afcollins
Beta Was this translation helpful? Give feedback.
All reactions