Re: [PATCH server-a001 v2 4/4] Create start.sh to start the docker container

[Hazmi Alfarizqi <hzmi@gnuweeb.org>]

On 01/05/2022 00:08, Ammar Faizi wrote:
> Create a file named start.sh. This is a script to start the container
> with prepared configuration. Only run this script on the host.
>
> Signed-off-by: Ammar Faizi <ammarfaizi2@gnuweeb.org>

It seems that `server-a001` gonna be the image name? I would assume we 
will build the Docker image from Dockerfile directly on the host.


If not and the image will be pulled from some registry, the image name 
format would look like: [registry/]Username/Image[:tag] (Example: 
docker.io/username/image:latest).

The tag is optional, the default tag would be latest.
For the registry part, if it's docker.io (which is the default registry 
for Docker) it will be omitted on Docker, but not on Podman.


Other than that, LGTM.

Reviewed-by: Hazmi Alfarizqi <hzmi@gnuweeb.org>

-- hzmi