Prevent slow-to-start containers from trying to run in the same slot (#114)

* Reserve slot for container once identified. Prevents race condition where containers start in the same slot if slow to start.

---------

Signed-off-by: Dan Kouba <dan@atym.io>
Signed-off-by: Krisztian Szilvasi <34309983+kr-t@users.noreply.github.com>
Co-authored-by: Krisztian Szilvasi <34309983+kr-t@users.noreply.github.com>

Author: dkouba-atym

src/ocre/components/container_supervisor/cs_sm_impl.c

@@ -354,7 +354,8 @@ ocre_container_status_t CS_create_container(ocre_container_t *container)
 	}
 
 	if (container->container_runtime_status != CONTAINER_STATUS_UNKNOWN &&
-	    container->container_runtime_status != CONTAINER_STATUS_DESTROYED) {
+	    container->container_runtime_status != CONTAINER_STATUS_DESTROYED &&
+	    container->container_runtime_status != CONTAINER_STATUS_RESERVED) {
 		LOG_ERR("Cannot create container again container with ID: %d, already exists", curr_container_ID);
 		return CONTAINER_STATUS_ERROR;
 	}

src/ocre/ocre_container_runtime/ocre_container_runtime.c

@@ -54,6 +54,7 @@ ocre_container_status_t ocre_container_runtime_create_container(ocre_cs_ctx *ctx
 		    (ctx->containers[i].container_runtime_status == CONTAINER_STATUS_DESTROYED)) {
 			*container_id = i;
 			ctx->containers[i].container_ID = i;
+			ctx->containers[i].container_runtime_status = CONTAINER_STATUS_RESERVED;
 			validity_flag = true;
 			break;
 		}

src/ocre/ocre_container_runtime/ocre_container_runtime.h

@@ -56,6 +56,7 @@ typedef enum {
  */
 typedef enum {
 	CONTAINER_STATUS_UNKNOWN,      ///< Status is unknown.
+	CONTAINER_STATUS_RESERVED,     ///< Container slot has been reserved.
 	CONTAINER_STATUS_CREATED,      ///< Container has been created.
 	CONTAINER_STATUS_RUNNING,      ///< Container is currently running.
 	CONTAINER_STATUS_STOPPED,      ///< Container has been stopped.