Hide Forgot
Description of problem: One is unable to set more than 25 user env variables (https://twitter.com/developercorey/status/405456556772171776) How reproducible: Consistent Steps to Reproduce: 1. Set more than 25 user env variables Actual results: A warning is thrown indicating that the limit of user env variables has been exceeded (also, I had expected the existing variables to be updated - I don't recall if this was the case) Expected results: User env variables are set Additional info: Part of my application was being served from Heroku which, as you may be aware, relies heavily on the use of environment variables to allow the dyno to communicate with external services (Neo4j, PostgreSQL, etc.) In moving this part of the app to OpenShift Online, I quickly hit the env limit of 25 (in part because some are more or less pointers to others, but I digress). I freely admit that I'm new to OpenShift and that this may be a non-issue, however I reference a number of external services and credentials using environment variables and it's entirely possible that I could end up blowing the limit of 25.
Hi Greg, I don't want to remove the limit entirely. Can you give me an idea of what number might make you happy? -Dan
Increased to 50: https://github.com/openshift/origin-server/pull/4709
Commits pushed to master at https://github.com/openshift/origin-server https://github.com/openshift/origin-server/commit/49de4a6c3fde6d47fb244475514e4680079d2fd5 Bug 1035046 - Increase user set env vars to 50 https://github.com/openshift/origin-server/commit/150e8320f6f6d7863206516965cd86d2243e34b5 Merge pull request #4709 from danmcp/dev/bug1035046 Merged by openshift-bot
Hi - thanks for this, I was out of reach until now so apologies for not replying sooner. 50 should be sufficient I imagine
(In reply to Dan McPherson from comment #3) > Increased to 50: > > https://github.com/openshift/origin-server/pull/4709 I have sync the code for the bug, for the changed code not in devenv_4356(I do not know why), and after synced the number changed to 50. After the code have been really merged, I will verified this "issue".
Tried on devenv_4357 and the number has been increased to 50.