Update docker image - Githubissues

EGI-Federation / fedcloud-vmi-templates

Virtual Machine Image of EGI Fedcloud

Apache License 2.0

6 stars 4 forks source link

Update docker image #43

Closed enolfc closed 2 years ago

enolfc commented 2 years ago

Moved docker configuration to a role
Updated default configuration:
- MTU set to 1376
- Create docker group
- Add default cloud-init user to docker group
New version of the docker image and update of AppDB metadata

enolfc commented 2 years ago

@sebastian-luna-valero please take a look

sebastian-luna-valero commented 2 years ago

Looks great to me, thanks @enolfc

I launched a test VM using https://portal.cloud.ifca.es/project/images/dcb862bc-ee5b-4edd-bbb9-b209c06215d8/ which I think is updated with the changes on this PR?

I get:

$ sudo docker network inspect bridge  | grep mtu | awk '{print $2}'
"1500"

$ sudo cat /etc/docker/daemon.json 
{
  "exec-opts": [
    "native.cgroupdriver=systemd"
  ],
  "log-driver": "json-file",
  "log-opts": {
    "max-size": "100m"
  },
  "storage-driver": "overlay2"
}

enolfc commented 2 years ago

Looks great to me, thanks @enolfc

I launched a test VM using https://portal.cloud.ifca.es/project/images/dcb862bc-ee5b-4edd-bbb9-b209c06215d8/ which I think is updated with the changes on this PR?

I get:
$ sudo docker network inspect bridge  | grep mtu | awk '{print $2}'
"1500"

$ sudo cat /etc/docker/daemon.json 
{
  "exec-opts": [
    "native.cgroupdriver=systemd"
  ],
  "log-driver": "json-file",
  "log-opts": {
    "max-size": "100m"
  },
  "storage-driver": "overlay2"
}

Image is not yet at sites, this needs to be merged first