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

Re: [atomic-devel] Using docker run -ti for interactive INSTALL?

On 03/03/2016 10:43 AM, Jan Pazdziora wrote:
On Thu, Mar 03, 2016 at 10:31:08AM -0500, Daniel J Walsh wrote:
On 03/03/2016 10:25 AM, Jan Pazdziora wrote:
Thinking about it a bit more, one way that seems to do the trick would
be to use

	LABEL INSTALL 'docker run $( test -t 0 && echo "-ti" ) ...'
That will not work in future versions of atomic, since it will not run the
command in a shell.  :^(
That is unfortunate. I actually tested and planned to push live

	LABEL INSTALL 'mkdir -p /var/lib/${NAME} && docker run -ti
		--rm=true -v /sys/fs/cgroup:/sys/fs/cgroup:ro -v
		/var/lib/${NAME}:/data:Z -h "${IPA_HOSTNAME}" -e
		${IMAGE} /bin/install.sh'
	LABEL RUN 'docker run -d --stop-signal=37
		-v /sys/fs/cgroup:/sys/fs/cgroup:ro
		-v /var/lib/${NAME}:/data:Z
		-h "${IPA_HOSTNAME:-$(cat /var/lib/${NAME}/hostname)}"
		--name ${NAME} ${IMAGE}'

as a rather flexible way of avoiding giving the container the whole

	-v /:/host

when precreating the /var/lib/${NAME} and just using that subdirectory
worked fine.

Plus I was able to retrieve hostname in atomic run with

	$(cat /var/lib/${NAME}/hostname)

from the container's data directory, thus making

	IPA_HOSTNAME=ipa.example.com atomic install freeipa --options abc

and plain

	atomic run freeipa

work and preserve the hostname across invocations without having to use
--privileged or giving the container SYS_ADMIN capabilities.

Can't we have shell-based approach in atomic preserved, perhaps via
INSTALL-SH/RUN-SH LABEL variants? Otherwise we might need to recreate
a lot of the functionality in atomic CLI that can be hacked in shell.

Take a look at the shell=true issues on atomic, Not sure if we need to disable this or not.

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