goldilocks icon indicating copy to clipboard operation
goldilocks copied to clipboard

Support LimitRange for current settings.

Open MattPOlson opened this issue 4 years ago • 5 comments

I've deployed this to an OpenShift 4.6 cluster and if a pods limit/request comes from a LimitRange instead of being defined at the pod/deployment level it's current settings on the dashboard shows as "Not Set"

MattPOlson avatar Jun 01 '21 14:06 MattPOlson

I converted this to a feature request. LimitRanges are not currently something we support.

sudermanjr avatar Jun 01 '21 17:06 sudermanjr

Thanks, would it be a big change? Could you point me where in the current code base it would need to be added and I could maybe take a look.

MattPOlson avatar Jun 01 '21 17:06 MattPOlson

I honestly don't know how hard it would be. I haven't used LimitRanges much at all.

Here is where we pull the current requests and limits - https://github.com/FairwindsOps/goldilocks/blob/a6988c29f15ff5472d147a4496dce76893d76d0f/pkg/summary/summary.go#L184-L190

Since it's based on the pod spec, I'm not sure how difficult it is going to be to bubble up the settings from the namespace-level object.

sudermanjr avatar Jun 01 '21 17:06 sudermanjr

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jul 21 '21 03:07 stale[bot]

@MattPOlson We are looking for a similar solution. Is there any workaround if the limits are taken from Namespace limitranges ?

divyanjali1 avatar Mar 26 '24 07:03 divyanjali1