Re: Anaconda Docker addon review needed
Colin Walters <[email protected]>
| Newsgroups | gmane.linux.redhat.anaconda.devel |
|---|---|
| Message-ID | <1453472401.2323761.499622522.213A7B14@webmail.messagingengine.com> |
On Thu, Jan 21, 2016, at 04:19 PM, Brian C. Lane wrote: > This is an anaconda kickstart, you setup storage with anaconda and tell > the addon which VG you're using so it can be passed on to the daemon. If > anything is duplicating something it's docker-storage-setup :) It's not really interesting to debate which component was "first" here, but the point I'm trying to make is more that this patch adds *more* duplication of code that already did exist in docker-storage-setup. (On the reverse end, certainly d-s-s' partitioning code obviously long postdates anaconda) For what it's worth, I would lean in favor of trying to use blivet in docker-storage-setup (which would be a total rewrite, but it's not a lot of code now). Then this addon could import the d-s-s python code. > > For example, the addon doesn't seem to support overlayfs right now. > > Is overlayfs supported by anything yet? https://github.com/projectatomic/docker-storage-setup/issues/49