@kapeta/local-cluster-service 0.9.1 → 0.10.1

package/dist/cjs/src/utils/BlockInstanceRunner.js

@@ -100,7 +100,7 @@ class BlockInstanceRunner {
                 processInfo = await this._startLocalProcess(blockInstance, blockUri, env, assetVersion);
             }
             else {
-                processInfo = await this._startDockerProcess(blockInstance, blockUri, env);
+                processInfo = await this._startDockerProcess(blockInstance, blockUri, env, assetVersion);
             }
             if (portTypes.length > 0) {
                 processInfo.portType = portTypes[0];
@@ -134,57 +134,16 @@ class BlockInstanceRunner {
             throw new Error(`Missing docker image information: ${JSON.stringify(localContainer)}`);
         }
         const containerName = (0, utils_1.getBlockInstanceContainerName)(blockInstance.id);
-        const logs = new LogData_1.LogData();
-        logs.addLog(`Starting block ${blockInstance.ref}`);
-        let containerInfo = await containerManager_1.containerManager.getContainerByName(containerName);
-        let container = containerInfo?.native;
-        console.log('Starting dev container', containerName);
-        if (containerInfo) {
-            console.log(`Dev container already exists. Deleting...`);
-            try {
-                await containerInfo.remove({
-                    force: true,
-                });
-            }
-            catch (e) {
-                throw new Error('Failed to delete existing container: ' + e.message);
-            }
-            container = undefined;
-            containerInfo = undefined;
-        }
-        logs.addLog(`Creating new container for block: ${containerName}`);
-        console.log('Creating new dev container', containerName, dockerImage);
-        await containerManager_1.containerManager.pull(dockerImage);
         const startCmd = localContainer.handlers?.onCreate ? localContainer.handlers.onCreate : '';
         const dockerOpts = localContainer.options ?? {};
         const homeDir = localContainer.userHome ? localContainer.userHome : '/root';
         const workingDir = localContainer.workingDir ? localContainer.workingDir : '/workspace';
-        const bindHost = (0, utils_1.getBindHost)();
-        const ExposedPorts = {};
-        const addonEnv = {};
-        const PortBindings = {};
-        const portTypes = getProviderPorts(assetVersion);
-        let port = 80;
-        const promises = portTypes.map(async (portType) => {
-            const publicPort = await serviceManager_1.serviceManager.ensureServicePort(this._systemId, blockInstance.id, portType);
-            const thisPort = port++; //TODO: Not sure how we should handle multiple ports or non-HTTP ports
-            const dockerPort = `${thisPort}/tcp`;
-            ExposedPorts[dockerPort] = {};
-            addonEnv[`KAPETA_LOCAL_SERVER_PORT_${portType.toUpperCase()}`] = '' + thisPort;
-            PortBindings[dockerPort] = [
-                {
-                    HostIp: bindHost,
-                    HostPort: `${publicPort}`,
-                },
-            ];
-        });
-        await Promise.all(promises);
+        const { PortBindings, ExposedPorts, addonEnv } = await this.getDockerPortBindings(blockInstance, assetVersion);
         let HealthCheck = undefined;
         if (localContainer.healthcheck) {
             HealthCheck = containerManager_1.containerManager.toDockerHealth({ cmd: localContainer.healthcheck });
         }
-        console.log('Starting dev container', containerName, dockerImage);
-        container = await containerManager_1.containerManager.startContainer({
+        return this.ensureContainer({
             Image: dockerImage,
             name: containerName,
             WorkingDir: workingDir,
@@ -211,8 +170,163 @@ class BlockInstanceRunner {
             },
             ...dockerOpts,
         });
+    }
+    async _startDockerProcess(blockInstance, blockInfo, env, assetVersion) {
+        const { versionFile } = local_cluster_config_1.default.getRepositoryAssetInfoPath(blockInfo.handle, blockInfo.name, blockInfo.version);
+        const versionYml = versionFile;
+        if (!node_fs_1.default.existsSync(versionYml)) {
+            throw new Error(`Did not find version info at the expected path: ${versionYml}`);
+        }
+        const versionInfo = (0, utils_1.readYML)(versionYml);
+        if (versionInfo?.artifact?.type !== 'docker') {
+            throw new Error(`Unsupported artifact type: ${versionInfo?.artifact?.type}`);
+        }
+        const dockerImage = versionInfo?.artifact?.details?.primary;
+        if (!dockerImage) {
+            throw new Error(`Missing docker image information: ${JSON.stringify(versionInfo?.artifact?.details)}`);
+        }
+        const { PortBindings, ExposedPorts, addonEnv } = await this.getDockerPortBindings(blockInstance, assetVersion);
+        const containerName = (0, utils_1.getBlockInstanceContainerName)(blockInstance.id);
+        // For windows we need to default to root
+        const innerHome = process.platform === 'win32' ? '/root/.kapeta' : local_cluster_config_1.default.getKapetaBasedir();
+        return this.ensureContainer({
+            Image: dockerImage,
+            name: containerName,
+            ExposedPorts,
+            Labels: {
+                instance: blockInstance.id,
+            },
+            Env: [
+                ...DOCKER_ENV_VARS,
+                `KAPETA_LOCAL_CLUSTER_PORT=${clusterService_1.clusterService.getClusterServicePort()}`,
+                ...Object.entries({
+                    ...env,
+                    ...addonEnv
+                }).map(([key, value]) => `${key}=${value}`),
+            ],
+            HostConfig: {
+                Binds: [`${(0, containerManager_1.toLocalBindVolume)(local_cluster_config_1.default.getKapetaBasedir())}:${innerHome}`],
+                PortBindings,
+            },
+        });
+    }
+    /**
+     *
+     * @param blockInstance
+     * @param blockUri
+     * @param providerDefinition
+     * @param {{[key:string]:string}} env
+     * @return {Promise<ProcessDetails>}
+     * @private
+     */
+    async _startOperatorProcess(blockInstance, blockUri, providerDefinition, env) {
+        const { assetFile } = local_cluster_config_1.default.getRepositoryAssetInfoPath(blockUri.handle, blockUri.name, blockUri.version);
+        const kapetaYmlPath = assetFile;
+        if (!node_fs_1.default.existsSync(kapetaYmlPath)) {
+            throw new Error(`Did not find kapeta.yml at the expected path: ${kapetaYmlPath}`);
+        }
+        const spec = providerDefinition.definition.spec;
+        const providerRef = `${providerDefinition.definition.metadata.name}:${providerDefinition.version}`;
+        if (!spec?.local?.image) {
+            throw new Error(`Provider did not have local image: ${providerRef}`);
+        }
+        const dockerImage = spec?.local?.image;
+        const containerName = (0, utils_1.getBlockInstanceContainerName)(blockInstance.id);
+        const logs = new LogData_1.LogData();
+        const bindHost = (0, utils_1.getBindHost)();
+        const ExposedPorts = {};
+        const addonEnv = {};
+        const PortBindings = {};
+        let HealthCheck = undefined;
+        let Mounts = [];
+        const promises = Object.entries(spec.local.ports).map(async ([portType, value]) => {
+            const dockerPort = `${value.port}/${value.type}`;
+            ExposedPorts[dockerPort] = {};
+            addonEnv[`KAPETA_LOCAL_SERVER_PORT_${portType.toUpperCase()}`] = value.port;
+            const publicPort = await serviceManager_1.serviceManager.ensureServicePort(this._systemId, blockInstance.id, portType);
+            PortBindings[dockerPort] = [
+                {
+                    HostIp: bindHost,
+                    HostPort: `${publicPort}`,
+                },
+            ];
+        });
+        await Promise.all(promises);
+        if (spec.local?.env) {
+            Object.entries(spec.local.env).forEach(([key, value]) => {
+                addonEnv[key] = value;
+            });
+        }
+        if (spec.local?.mounts) {
+            const mounts = containerManager_1.containerManager.createMounts(blockUri.id, spec.local.mounts);
+            Mounts = containerManager_1.containerManager.toDockerMounts(mounts);
+        }
+        if (spec.local?.health) {
+            HealthCheck = containerManager_1.containerManager.toDockerHealth(spec.local?.health);
+        }
+        // For windows we need to default to root
+        const innerHome = process.platform === 'win32' ? '/root/.kapeta' : local_cluster_config_1.default.getKapetaBasedir();
+        logs.addLog(`Creating new container for block: ${containerName}`);
+        const out = await this.ensureContainer({
+            Image: dockerImage,
+            name: containerName,
+            ExposedPorts,
+            HealthCheck,
+            HostConfig: {
+                Binds: [
+                    `${(0, containerManager_1.toLocalBindVolume)(kapetaYmlPath)}:/kapeta.yml:ro`,
+                    `${(0, containerManager_1.toLocalBindVolume)(local_cluster_config_1.default.getKapetaBasedir())}:${innerHome}`,
+                ],
+                PortBindings,
+                Mounts,
+            },
+            Labels: {
+                instance: blockInstance.id,
+            },
+            Env: [
+                `KAPETA_INSTANCE_NAME=${blockInstance.ref}`,
+                `KAPETA_LOCAL_CLUSTER_PORT=${clusterService_1.clusterService.getClusterServicePort()}`,
+                ...DOCKER_ENV_VARS,
+                ...Object.entries({
+                    ...env,
+                    ...addonEnv,
+                }).map(([key, value]) => `${key}=${value}`),
+            ],
+        });
+        const portTypes = spec.local.ports ? Object.keys(spec.local.ports) : [];
+        if (portTypes.length > 0) {
+            out.portType = portTypes[0];
+        }
+        return out;
+    }
+    async getDockerPortBindings(blockInstance, assetVersion) {
+        const bindHost = (0, utils_1.getBindHost)();
+        const ExposedPorts = {};
+        const addonEnv = {};
+        const PortBindings = {};
+        const portTypes = getProviderPorts(assetVersion);
+        let port = 80;
+        const promises = portTypes.map(async (portType) => {
+            const publicPort = await serviceManager_1.serviceManager.ensureServicePort(this._systemId, blockInstance.id, portType);
+            const thisPort = port++; //TODO: Not sure how we should handle multiple ports or non-HTTP ports
+            const dockerPort = `${thisPort}/tcp`;
+            ExposedPorts[dockerPort] = {};
+            addonEnv[`KAPETA_LOCAL_SERVER_PORT_${portType.toUpperCase()}`] = '' + thisPort;
+            PortBindings[dockerPort] = [
+                {
+                    HostIp: bindHost,
+                    HostPort: `${publicPort}`,
+                },
+            ];
+        });
+        await Promise.all(promises);
+        return { PortBindings, ExposedPorts, addonEnv };
+    }
+    async ensureContainer(opts) {
+        const logs = new LogData_1.LogData();
+        const container = await containerManager_1.containerManager.ensureContainer(opts);
         try {
-            if (HealthCheck) {
+            if (opts.HealthCheck) {
                 await containerManager_1.containerManager.waitForHealthy(container);
             }
             else {
@@ -274,202 +388,5 @@ class BlockInstanceRunner {
             },
         };
     }
-    async _startDockerProcess(blockInstance, blockInfo, env) {
-        const { versionFile } = local_cluster_config_1.default.getRepositoryAssetInfoPath(blockInfo.handle, blockInfo.name, blockInfo.version);
-        const versionYml = versionFile;
-        if (!node_fs_1.default.existsSync(versionYml)) {
-            throw new Error(`Did not find version info at the expected path: ${versionYml}`);
-        }
-        const versionInfo = (0, utils_1.readYML)(versionYml);
-        if (versionInfo?.artifact?.type !== 'docker') {
-            throw new Error(`Unsupported artifact type: ${versionInfo?.artifact?.type}`);
-        }
-        const dockerImage = versionInfo?.artifact?.details?.primary;
-        if (!dockerImage) {
-            throw new Error(`Missing docker image information: ${JSON.stringify(versionInfo?.artifact?.details)}`);
-        }
-        const containerName = (0, utils_1.getBlockInstanceContainerName)(blockInstance.id);
-        const logs = new LogData_1.LogData();
-        const containerInfo = await containerManager_1.containerManager.getContainerByName(containerName);
-        let container = containerInfo?.native;
-        // For windows we need to default to root
-        const innerHome = process.platform === 'win32' ? '/root/.kapeta' : local_cluster_config_1.default.getKapetaBasedir();
-        if (container) {
-            const containerData = container.data;
-            if (containerData.State === 'running') {
-                logs.addLog(`Found existing running container for block: ${containerName}`);
-            }
-            else {
-                logs.addLog(`Found existing container for block: ${containerName}. Starting now`);
-                await container.start();
-            }
-        }
-        else {
-            logs.addLog(`Creating new container for block: ${containerName}`);
-            container = await containerManager_1.containerManager.startContainer({
-                Image: dockerImage,
-                name: containerName,
-                Labels: {
-                    instance: blockInstance.id,
-                },
-                Env: [
-                    ...DOCKER_ENV_VARS,
-                    `KAPETA_LOCAL_CLUSTER_PORT=${clusterService_1.clusterService.getClusterServicePort()}`,
-                    ...Object.entries(env).map(([key, value]) => `${key}=${value}`),
-                ],
-                HostConfig: {
-                    Binds: [`${(0, containerManager_1.toLocalBindVolume)(local_cluster_config_1.default.getKapetaBasedir())}:${innerHome}`],
-                },
-            });
-            try {
-                await containerManager_1.containerManager.waitForReady(container);
-            }
-            catch (e) {
-                logs.addLog(e.message, 'ERROR');
-            }
-        }
-        return this._handleContainer(container, logs);
-    }
-    /**
-     *
-     * @param blockInstance
-     * @param blockUri
-     * @param providerDefinition
-     * @param {{[key:string]:string}} env
-     * @return {Promise<ProcessDetails>}
-     * @private
-     */
-    async _startOperatorProcess(blockInstance, blockUri, providerDefinition, env) {
-        const { assetFile } = local_cluster_config_1.default.getRepositoryAssetInfoPath(blockUri.handle, blockUri.name, blockUri.version);
-        const kapetaYmlPath = assetFile;
-        if (!node_fs_1.default.existsSync(kapetaYmlPath)) {
-            throw new Error(`Did not find kapeta.yml at the expected path: ${kapetaYmlPath}`);
-        }
-        const spec = providerDefinition.definition.spec;
-        const providerRef = `${providerDefinition.definition.metadata.name}:${providerDefinition.version}`;
-        if (!spec?.local?.image) {
-            throw new Error(`Provider did not have local image: ${providerRef}`);
-        }
-        const dockerImage = spec?.local?.image;
-        try {
-            await containerManager_1.containerManager.pull(dockerImage);
-        }
-        catch (e) {
-            console.warn('Failed to pull image. Continuing...', e);
-        }
-        const containerName = (0, utils_1.getBlockInstanceContainerName)(blockInstance.id);
-        const logs = new LogData_1.LogData();
-        const containerInfo = await containerManager_1.containerManager.getContainerByName(containerName);
-        let container = containerInfo?.native;
-        if (container) {
-            const containerData = container.data;
-            if (containerData.State === 'running') {
-                logs.addLog(`Found existing running container for block: ${containerName}`);
-            }
-            else {
-                if (containerData.State?.ExitCode > 0) {
-                    logs.addLog(`Container exited with code: ${containerData.State.ExitCode}. Deleting...`);
-                    try {
-                        await containerManager_1.containerManager.remove(container);
-                    }
-                    catch (e) { }
-                    container = undefined;
-                }
-                else {
-                    logs.addLog(`Found existing container for block: ${containerName}. Starting now`);
-                    try {
-                        await container.start();
-                    }
-                    catch (e) {
-                        console.warn('Failed to start container. Deleting...', e);
-                        try {
-                            await containerManager_1.containerManager.remove(container);
-                        }
-                        catch (e) { }
-                        container = undefined;
-                    }
-                }
-            }
-        }
-        const bindHost = (0, utils_1.getBindHost)();
-        if (!container) {
-            const ExposedPorts = {};
-            const addonEnv = {};
-            const PortBindings = {};
-            let HealthCheck = undefined;
-            let Mounts = [];
-            const promises = Object.entries(spec.local.ports).map(async ([portType, value]) => {
-                const dockerPort = `${value.port}/${value.type}`;
-                ExposedPorts[dockerPort] = {};
-                addonEnv[`KAPETA_LOCAL_SERVER_PORT_${portType.toUpperCase()}`] = value.port;
-                const publicPort = await serviceManager_1.serviceManager.ensureServicePort(this._systemId, blockInstance.id, portType);
-                PortBindings[dockerPort] = [
-                    {
-                        HostIp: bindHost,
-                        HostPort: `${publicPort}`,
-                    },
-                ];
-            });
-            await Promise.all(promises);
-            if (spec.local?.env) {
-                Object.entries(spec.local.env).forEach(([key, value]) => {
-                    addonEnv[key] = value;
-                });
-            }
-            if (spec.local?.mounts) {
-                const mounts = containerManager_1.containerManager.createMounts(blockUri.id, spec.local.mounts);
-                Mounts = containerManager_1.containerManager.toDockerMounts(mounts);
-            }
-            if (spec.local?.health) {
-                HealthCheck = containerManager_1.containerManager.toDockerHealth(spec.local?.health);
-            }
-            // For windows we need to default to root
-            const innerHome = process.platform === 'win32' ? '/root/.kapeta' : local_cluster_config_1.default.getKapetaBasedir();
-            logs.addLog(`Creating new container for block: ${containerName}`);
-            container = await containerManager_1.containerManager.startContainer({
-                Image: dockerImage,
-                name: containerName,
-                ExposedPorts,
-                HealthCheck,
-                HostConfig: {
-                    Binds: [
-                        `${(0, containerManager_1.toLocalBindVolume)(kapetaYmlPath)}:/kapeta.yml:ro`,
-                        `${(0, containerManager_1.toLocalBindVolume)(local_cluster_config_1.default.getKapetaBasedir())}:${innerHome}`,
-                    ],
-                    PortBindings,
-                    Mounts,
-                },
-                Labels: {
-                    instance: blockInstance.id,
-                },
-                Env: [
-                    `KAPETA_INSTANCE_NAME=${blockInstance.ref}`,
-                    `KAPETA_LOCAL_CLUSTER_PORT=${clusterService_1.clusterService.getClusterServicePort()}`,
-                    ...DOCKER_ENV_VARS,
-                    ...Object.entries({
-                        ...env,
-                        ...addonEnv,
-                    }).map(([key, value]) => `${key}=${value}`),
-                ],
-            });
-            try {
-                if (HealthCheck) {
-                    await containerManager_1.containerManager.waitForHealthy(container);
-                }
-                else {
-                    await containerManager_1.containerManager.waitForReady(container);
-                }
-            }
-            catch (e) {
-                logs.addLog(e.message, 'ERROR');
-            }
-        }
-        const out = await this._handleContainer(container, logs, true);
-        const portTypes = spec.local.ports ? Object.keys(spec.local.ports) : [];
-        if (portTypes.length > 0) {
-            out.portType = portTypes[0];
-        }
-        return out;
-    }
 }
 exports.BlockInstanceRunner = BlockInstanceRunner;

package/dist/esm/src/containerManager.d.ts

@@ -41,6 +41,7 @@ interface Health {
     timeout?: number;
     retries?: number;
 }
+export declare const CONTAINER_LABEL_PORT_PREFIX = "kapeta_port-";
 export declare const HEALTH_CHECK_TIMEOUT: number;
 declare class ContainerManager {
     private _docker;
@@ -56,7 +57,7 @@ declare class ContainerManager {
     ping(): Promise<void>;
     docker(): Docker;
     getContainerByName(containerName: string): Promise<ContainerInfo | undefined>;
-    pull(image: string, cacheForMS?: number): Promise<void>;
+    pull(image: string, cacheForMS?: number): Promise<boolean>;
     toDockerMounts(mounts: StringMap): DockerMounts[];
     toDockerHealth(health: Health): {
         Test: string[];
@@ -64,13 +65,8 @@ declare class ContainerManager {
         Timeout: number;
         Retries: number;
     };
-    run(image: string, name: string, opts: {
-        ports: {};
-        mounts: {};
-        env: {};
-        cmd: string;
-        health: Health;
-    }): Promise<ContainerInfo>;
+    private applyHash;
+    ensureContainer(opts: any): Promise<Container>;
     startContainer(opts: any): Promise<Container>;
     waitForReady(container: Container, attempt?: number): Promise<void>;
     waitForHealthy(container: Container, attempt?: number): Promise<void>;

package/dist/esm/src/containerManager.js

@@ -6,9 +6,9 @@ import FSExtra from 'fs-extra';
 import { Docker } from 'node-docker-api';
 import { parseKapetaUri } from '@kapeta/nodejs-utils';
 import ClusterConfiguration from '@kapeta/local-cluster-config';
-import { getBindHost } from './utils/utils';
-import uuid from "node-uuid";
-const LABEL_PORT_PREFIX = 'kapeta_port-';
+import uuid from 'node-uuid';
+import md5 from 'md5';
+export const CONTAINER_LABEL_PORT_PREFIX = 'kapeta_port-';
 const NANO_SECOND = 1000000;
 const HEALTH_CHECK_INTERVAL = 3000;
 const HEALTH_CHECK_MAX = 20;
@@ -141,22 +141,19 @@ class ContainerManager {
         if (!tag) {
             tag = 'latest';
         }
-        if (tag !== 'latest') {
-            if (IMAGE_PULL_CACHE[image]) {
-                const timeSince = Date.now() - IMAGE_PULL_CACHE[image];
-                if (timeSince < cacheForMS) {
-                    return;
-                }
-            }
-            const imageTagList = (await this.docker().image.list())
-                .map((image) => image.data)
-                .filter((imageData) => !!imageData.RepoTags)
-                .map((imageData) => imageData.RepoTags);
-            if (imageTagList.some((imageTags) => imageTags.indexOf(image) > -1)) {
-                console.log('Image found: %s', image);
-                return;
+        if (IMAGE_PULL_CACHE[image]) {
+            const timeSince = Date.now() - IMAGE_PULL_CACHE[image];
+            if (timeSince < cacheForMS) {
+                return false;
             }
-            console.log('Image not found: %s', image);
+        }
+        const imageTagList = (await this.docker().image.list())
+            .map((image) => image.data)
+            .filter((imageData) => !!imageData.RepoTags)
+            .map((imageData) => imageData.RepoTags);
+        if (imageTagList.some((imageTags) => imageTags.indexOf(image) > -1)) {
+            console.log('Image found: %s', image);
+            return false;
         }
         console.log('Pulling image: %s', image);
         await this.docker()
@@ -167,6 +164,7 @@ class ContainerManager {
             .then((stream) => promisifyStream(stream));
         IMAGE_PULL_CACHE[image] = Date.now();
         console.log('Image pulled: %s', image);
+        return true;
     }
     toDockerMounts(mounts) {
         const Mounts = [];
@@ -189,51 +187,58 @@ class ContainerManager {
             Retries: health.retries || 10,
         };
     }
-    async run(image, name, opts) {
-        const PortBindings = {};
-        const Env = [];
-        const Labels = {
-            kapeta: 'true',
-        };
-        await this.pull(image);
-        const bindHost = getBindHost();
-        const ExposedPorts = {};
-        _.forEach(opts.ports, (portInfo, containerPort) => {
-            ExposedPorts['' + containerPort] = {};
-            PortBindings['' + containerPort] = [
-                {
-                    HostPort: '' + portInfo.hostPort,
-                    HostIp: bindHost,
-                },
-            ];
-            Labels[LABEL_PORT_PREFIX + portInfo.hostPort] = portInfo.type;
-        });
-        const Mounts = this.toDockerMounts(opts.mounts);
-        _.forEach(opts.env, (value, name) => {
-            Env.push(name + '=' + value);
-        });
-        let HealthCheck = undefined;
-        if (opts.health) {
-            HealthCheck = this.toDockerHealth(opts.health);
-        }
-        const dockerContainer = await this.startContainer({
-            name: name,
-            Image: image,
-            Hostname: name + '.kapeta',
-            Labels,
-            Cmd: opts.cmd,
-            ExposedPorts,
-            Env,
-            HealthCheck,
-            HostConfig: {
-                PortBindings,
-                Mounts,
-            },
-        });
-        if (opts.health) {
-            await this.waitForHealthy(dockerContainer);
+    applyHash(dockerOpts) {
+        if (dockerOpts?.Labels?.HASH) {
+            delete dockerOpts.Labels.HASH;
         }
-        return new ContainerInfo(dockerContainer);
+        const hash = md5(JSON.stringify(dockerOpts));
+        if (!dockerOpts.Labels) {
+            dockerOpts.Labels = {};
+        }
+        dockerOpts.Labels.HASH = hash;
+    }
+    async ensureContainer(opts) {
+        let imagePulled = false;
+        try {
+            imagePulled = await this.pull(opts.Image);
+        }
+        catch (e) {
+            console.warn('Failed to pull image. Continuing...', e);
+        }
+        this.applyHash(opts);
+        if (!opts.name) {
+            console.log('Starting unnamed container: %s', opts.Image);
+            return this.startContainer(opts);
+        }
+        const containerInfo = await this.getContainerByName(opts.name);
+        if (imagePulled) {
+            console.log('New version of image was pulled: %s', opts.Image);
+        }
+        else {
+            // If image was pulled always recreate
+            if (!containerInfo) {
+                console.log('Starting new container: %s', opts.name);
+                return this.startContainer(opts);
+            }
+            const containerData = containerInfo.native.data;
+            if (containerData?.Labels?.HASH === opts.Labels.HASH) {
+                if (!(await containerInfo.isRunning())) {
+                    console.log('Starting previously created container: %s', opts.name);
+                    await containerInfo.start();
+                }
+                else {
+                    console.log('Previously created container already running: %s', opts.name);
+                }
+                return containerInfo.native;
+            }
+        }
+        if (containerInfo) {
+            // Remove the container and start a new one
+            console.log('Replacing previously created container: %s', opts.name);
+            await containerInfo.remove({ force: true });
+        }
+        console.log('Starting new container: %s', opts.name);
+        return this.startContainer(opts);
     }
     async startContainer(opts) {
         const extraHosts = getExtraHosts(this._version);
@@ -411,10 +416,10 @@ export class ContainerInfo {
         const portTypes = {};
         const ports = {};
         _.forEach(inspectResult.Config.Labels, (portType, name) => {
-            if (!name.startsWith(LABEL_PORT_PREFIX)) {
+            if (!name.startsWith(CONTAINER_LABEL_PORT_PREFIX)) {
                 return;
             }
-            const hostPort = name.substr(LABEL_PORT_PREFIX.length);
+            const hostPort = name.substr(CONTAINER_LABEL_PORT_PREFIX.length);
             portTypes[hostPort] = portType;
         });
         _.forEach(inspectResult.HostConfig.PortBindings, (portBindings, containerPortSpec) => {

package/dist/esm/src/instanceManager.js

@@ -447,7 +447,8 @@ export class InstanceManager {
                         changed = true;
                     }
                 }
-                if (instance.desiredStatus === DesiredInstanceStatus.RUN && newStatus === InstanceStatus.STOPPED) {
+                if (instance.desiredStatus === DesiredInstanceStatus.RUN &&
+                    [InstanceStatus.STOPPED, InstanceStatus.FAILED, InstanceStatus.STOPPING].includes(newStatus)) {
                     //If the instance is stopped but we want it to run, start it
                     try {
                         await this.start(instance.systemId, instance.instanceId);
@@ -457,7 +458,8 @@ export class InstanceManager {
                     }
                     return;
                 }
-                if (instance.desiredStatus === DesiredInstanceStatus.STOP && newStatus === InstanceStatus.READY) {
+                if (instance.desiredStatus === DesiredInstanceStatus.STOP &&
+                    [InstanceStatus.READY, InstanceStatus.STARTING, InstanceStatus.UNHEALTHY].includes(newStatus)) {
                     //If the instance is running but we want it to stop, stop it
                     try {
                         await this.stop(instance.systemId, instance.instanceId);