[atomic-wg] Issue #186 `switch to overlay2`

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



vgoyal added a new comment to an issue you are following:
``
> I would vote that we keep it easy for the user. Why don't we have CONTAINER_ROOT_VOLUME=yes instead of DOCKER_ROOT_VOLUME=yes.

Problem with that is that we don't know where to mount the volume after creation. Right now, we mount it on /var/lib/docker. Also we scan /etc/sysconfig/docker for option -g and try to mount it
there. But all that soon becomes very specific to docker.

Hence, instead we wanted user to pass in the directory where volume should be mounted. And if user is passing in directory, that itself means create extra volume.

I was thinking that EXTRA_VOLUME_MOUNT_DIR is more generic because we don't care how container runtime uses it. Whether it is their ROOT or they use it to carve out volumes or for
something else. That's the reason I suggested that instead this naming scheme. But I am not too particular and I will be fine with CONTAINER_ROOT_VOLUME_MOUNT_DIR as well.

Dusty, shishir has created a new PR for this change. Feel free to review it.

https://github.com/projectatomic/docker-storage-setup/pull/181

> You can still have variables like EXTRA_VOLUME_NAME and EXTRA_VOLUME_MOUNT_DIR but just make them have default values that only need to be specified if the user doesn't like the defaults.

By default these will be nil. That is no extra volume will be set. Users will have to specify values if
they want extra volume to be set.

> small nit: I would rather have them be CONTAINER_VOLUME_NAME and CONTAINER_VOLUME_MOUNT_DIR.

I am fine with above naming.

> take it a step farther back and we should call it container storage setup?

That's the plan. We are doing changes slowly and ultimately will rename it to container-storage-setup.
``

To reply, visit the link below or just reply to this email
https://pagure.io/atomic-wg/issue/186
_______________________________________________
cloud mailing list -- cloud@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to cloud-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Big List of Linux Books]     [Yosemite News]     [Linux Apps]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [Asterisk PBX]

  Powered by Linux