Add memory limits to functions

[martindekov]

Adding memory limits to functions all except
the dashboard are set to limits.memory 128Mi
requests.memory 64Mi the dashboard's limits
are doubled. All functions have requests.cpu
limit of 50m

Signed-off-by: Martin Dekov [email protected]

Description

Closes #614

How Has This Been Tested?

Tested to verify limits are applied by faas deploy -f stack.yml in base folder and in dashboard:

$ kubectl describe deploy system-github-event -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy audit-event -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy buildshiprun -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy echo -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy function-logs -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy garbage-collect -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy git-tar -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy github-push -n openfaas-fn
...  
  Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy github-status -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy import-secrets -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy list-functions -n openfaas-fn
... 
   Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy metrics -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy pipeline-log -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy system-dashboard -n openfaas-fn
...
    Limits:
      memory:  256Mi
    Requests:
      cpu:      50m
      memory:   64Mi
...
$ kubectl describe deploy system-github-event -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...

For gitlab ones:

$ kubectl describe deploy gitlab-push -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
...
$ kubectl describe deploy gitlab-status -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi
$ kubectl describe deploy system-gitlab-event -n openfaas-fn
...
    Limits:
      memory:  128Mi
    Requests:
      cpu:      50m
      memory:   32Mi

How are existing users impacted? What migration steps/scripts do we need?

They will have default memory limits when install the cloud

Checklist:

I have:

• updated the documentation and/or roadmap (if required)
• read the CONTRIBUTION guide
• signed-off my commits with git commit -s
• added unit tests

[alexellis]

How Has This Been Tested?

TODO

[Waterdrips]

Code looks ok, just the TODO on testing in PR comment

[alexellis]

I can't merge this without it having been tested.

@martindekov the deployment should work relatively quickly via https://blog.alexellis.io/openfaas-cloud-for-development/

@Waterdrips if you have some spare time and can help, maybe we can get this closed out?

[martindekov]

Changed the description with the testing steps. This is ready to be merged

[alexellis]

Approved