Minor fixes

Author: grussorusso

README.md

@@ -73,8 +73,8 @@ Register a function `func` from example python code (the handler is formatted li
 
 Register a function `func` from example javascript code (the handler is formatted like this: $(filename) and the name of the function is "handler"):
 
-	$ bin/serverledge-cli create -f func --memory 600 --src examples/hello.js --runtime nodejs17 --handler "hello"
-    $ bin/serverledge-cli create -f func --memory 600 --src examples/inc.js --runtime nodejs17 --handler "inc"
+	$ bin/serverledge-cli create -f func --memory 600 --src examples/hello.js --runtime nodejs17ng --handler "hello"
+    $ bin/serverledge-cli create -f func --memory 600 --src examples/inc.js --runtime nodejs17ng --handler "inc"
 
 Invoke `func` with arguments `a=2` and `b=3`:
 

internal/api/api.go

@@ -140,16 +140,20 @@ func CreateOrUpdateFunction(c echo.Context) error {
 		return err
 	}
 
+	var isUpdate bool
+
 	if c.Path() != "/update" {
 		_, ok := function.GetFunction(f.Name) // TODO: we would need a system-wide lock here...
 		if ok {
 			log.Printf("Dropping request for already existing function '%s'\n", f.Name)
 			return c.String(http.StatusConflict, "")
 		}
 
+		isUpdate = false
 		log.Printf("New request: creation of %s\n", f.Name)
 	} else {
 		log.Printf("New request: creation/update of %s\n", f.Name)
+		isUpdate = true
 	}
 
 	// Check that the selected runtime exists
@@ -182,6 +186,12 @@ func CreateOrUpdateFunction(c echo.Context) error {
 		log.Printf("Failed creation: %v\n", err)
 		return c.JSON(http.StatusServiceUnavailable, "")
 	}
+
+	if isUpdate {
+		// terminate any warm container after the update
+		node.ShutdownWarmContainersFor(&f)
+	}
+
 	response := struct{ Created string }{f.Name}
 	return c.JSON(http.StatusOK, response)
 }