minishift-addons icon indicating copy to clipboard operation
minishift-addons copied to clipboard

Check current minishift config when enabling/applying che addon.

Open rhopp opened this issue 8 years ago • 7 comments

We are suggesting user to increase minishift memory to 5GB before enabling/applying che addon (in readme?). It would be nice, if "minishift addon enable che" would check current minishift config and if it is below 5GB, it would print message like:

Che addon required at least 5GB RAM of minishift virtual machine. Your current config is at 2GB. Do you want to increase this to 5GB? [y/n]

Or at least print how it could be done.

rhopp avatar Oct 26 '17 12:10 rhopp

@rhopp

I think we can't do much from minishift side as we can't track which addon has which configuration.

I think the addon can itself check initially if VM has enough memory or not and then exit with message.

brgnepal avatar Oct 27 '17 03:10 brgnepal

Any opinion @minishift/minishift-dev ?

brgnepal avatar Oct 27 '17 03:10 brgnepal

I think the addon can itself check initially if VM has enough memory or not and then exit with message. That's exactly what I had on my mind ;-)

rhopp avatar Oct 27 '17 07:10 rhopp

This is not Che addon specific. this could be part of the meta-data, just like minimum OpenShift version, etc.

gbraad avatar Oct 27 '17 08:10 gbraad

I think the addon can itself check initially if VM has enough memory or not and then exit with message.

Do you mean through add-on ssh command.

LalatenduMohanty avatar Oct 27 '17 08:10 LalatenduMohanty

through add-on ssh command

I hope not... as this moves logic for something generic into an add-on install... a think a 'Minimum Memory' variable in the ad-on header would be sufficient/possible.

gbraad avatar Oct 27 '17 08:10 gbraad

@LalatenduMohanty @gbraad This seems like feature request for Minishift, shall we migrate the issue to appropriate issue tracker?

agajdosi avatar Aug 14 '18 12:08 agajdosi