Trivial change to add logging around scale from zero events in scaling.go.
Previously scale from zero events were not logged in the same way that normal
scaling events are.  This change adds log writes to show when a scale from zero
was requested and when a function successfully moved to > 0 replicas.

Signed-off-by: Richard Gee <richard@technologee.co.uk>

Signed-off-by: qinpengfei <qinpengfei@jd.com>

We now send Accepted not OK for creating functions.

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

- Covers part of 919 by making the HTTP client used for proxying
stop following redirects. Tested with a stateless microservice,
but additional code changes may be requierd in the queue-worker,
the watchdogs and other areas.

Tested on Swarm with stateless microservice (Node.js) issuing
a redirect via Location header.

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

Signed-off-by: Radoslav Dimitrov <rdimitrow@gmail.com>

Signed-off-by: Radoslav Dimitrov <rdimitrow@gmail.com>

Signed-off-by: Radoslav Dimitrov <rdimitrow@gmail.com>

Signed-off-by: Radoslav Dimitrov <rdimitrow@gmail.com>

Signed-off-by: John McCabe <john@johnmccabe.net>

Signed-off-by: John McCabe <john@johnmccabe.net>

Signed-off-by: Patricio Diaz <padiazg@gmail.com>

Signed-off-by: Lucas Roesler <roesler.lucas@gmail.com>

**What**
- Split the table into official providers and community providers
- Move the site links to the site column

Signed-off-by: Lucas Roesler <roesler.lucas@gmail.com>

**What**
- Add openfaas-operator as a provider with status Incubation

Signed-off-by: Lucas Roesler <roesler.lucas@gmail.com>

Signed-off-by: Radoslav Dimitrov <rdimitrow@gmail.com>

Signed-off-by: Ken Fukuyama <kenfdev@gmail.com>

Signed-off-by: Alex Ellis <alexellis2@gmail.com>

Signed-off-by: Alex Ellis <alexellis2@gmail.com>

Signed-off-by: Alex Ellis <alexellis2@gmail.com>

Signed-off-by: Edward Wilde <ewilde@gmail.com>

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

Signed-off-by: Vivek Singh <vivekkmr45@yahoo.in>

- Removes use of "our" from CONTRIBUTING guide
- Updates/adds README.md files
- Commnents and typo fix in watchdog
- Adds good/bad examples of commit messages

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

- updates comments and adds where missing
- updates locks so that unlock is done via defer instead of
at the end of the statement
- extracts timeout variable in two places
- remove makeClient() unused method from metrics package

No-harm changes tested via go build.

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

Signed-off-by: Alex Ellis (VMware) <alexellis2@gmail.com>

Signed-off-by: Burton Rheutan <rheutan7@gmail.com>

Signed-off-by: Burton Rheutan <rheutan7@gmail.com>

This commit adds a line to the issue template
to request the faas-cli version and provider
information to the issue template to help diagnose
issues faster

Signed-off-by: Burton Rheutan <rheutan7@gmail.com>

Signed-off-by: Frank Pommerening <frank@pommerening-online.de>

Proceed with building the image only if it's not already available.

Signed-off-by: Radoslav Dimitrov <rdimitrow@gmail.com>

Signed-off-by: Richard Gee <richard@technologee.co.uk>

Within MakeScalingHandler() there is a call to GetReplicas() which was not returning an error when a non-200 http response was received from /system/function/.  The call would also return a populated struct, so the perception was that a function existed an had been scaled to zero.  This meant that the function would be added to the function cache and the code would continue into SetReplicas() where an attempt would be made to scale up a non-existent function.

This change amends GetReplicas() so that it will return an error if the gateway returns anything other than a 200 reponse code from the /system/function/ endpoint.  This causes MakeScalingHandler() to return earlier with an error indicating that the function could not be found.  The cache.Set call is also moved to after the error check so that the cache is only updated to include existent functions.

During investigations as to the cause of #876 tests were added to function_cache to check that Get() is behaving as intended when function exists and when not.  Tests are also added to plugin/external to test that GetReplicas() and SetReplicas() are following their intended modes of operation when 200 and non-200 responses are received from the gateway.

Signed-off-by: Richard Gee <richard@technologee.co.uk>

Signed-off-by: Marko Mudrinić <mudrinic.mare@gmail.com>