Commit graph

1186 commits

Author SHA1 Message Date
Debarshi Ray
1b08b259fe Fix typo
Fallout from 169f91669f
2018-09-27 11:59:24 +02:00
Debarshi Ray
bcd55f84fb Make su(1) work
... by deleting the root password.
2018-09-26 17:39:33 +02:00
Debarshi Ray
f30dd246dc Use a more direct way to check for the customized toolbox image
... than grepping the output of something else. One nice side-effect
is that the --verbose output becomes more meaningful.
2018-09-26 17:35:29 +02:00
Debarshi Ray
65ce3f76e8 Remove redundant if branch
Trust the exit status of 'buildah from ...' to have created a working
container instead of checking for its presence once again.
2018-09-26 17:10:42 +02:00
Debarshi Ray
07b37dbc2a Style fix
This reduces one level of indentation and makes the code more readable.
2018-09-26 16:59:38 +02:00
Debarshi Ray
573b156b92 Add fedora-toolbox image definitions for Fedoras 29 and 30 2018-09-26 16:37:46 +02:00
Debarshi Ray
169f91669f Make room for Dockerfiles for multiple operating system versions
It would be good to have Dockerfiles for multiple Fedora versions, and
possibly even other operating systems like Red Hat Enterprise Linux or
CentOS, so that people can use the toolbox on different host OSes.

In the future, it would be nice to have multiple toolbox containers
corresponding to different OSes exist side by side. One could then
have a Fedora 29 environment on a Fedora 28 host and so on.
2018-09-26 16:33:17 +02:00
Debarshi Ray
b103571d77 Dockerfile: Simplify the package installation 2018-09-26 12:07:13 +02:00
Debarshi Ray
913e3ea5e7 Dockerfile: Rely on an updated base image build for package updates
... instead of updating them while building the fedora-toolbox image.

https://bugzilla.redhat.com/show_bug.cgi?id=1628914
2018-09-24 18:01:56 +02:00
Debarshi Ray
188e625dcb Dockerfile: Add a usage label
... as mandated by the Fedora Container Guidelines:
https://fedoraproject.org/wiki/Container:Guidelines#LABELS
2018-09-24 17:16:13 +02:00
Debarshi Ray
31a3469fa9 Dockerfile: Fix the build on Fedora OpenShift Build Service
... by removing the docker:// scheme prefix.

https://bugzilla.redhat.com/show_bug.cgi?id=1628914
2018-09-24 17:11:11 +02:00
Debarshi Ray
68a19bcfd7 Clarify the error message if the container already exists 2018-09-21 12:31:41 +02:00
Debarshi Ray
66ab4da724 Allow running under sudo(8) when desired
This adds a --sudo flag, and the corresponding /etc/sudoers.d
configuration snippet, to run everything under sudo(8) for debugging.
2018-09-20 21:15:09 +02:00
Debarshi Ray
4bda42d414 Ensure that the toolbox user runs with 'wheel' as an additional group 2018-09-20 20:37:00 +02:00
Debarshi Ray
cfcf4eb31e Map the host UID into the container
Assuming a host UID of 1000, the UID mapping inside the user namespace
created by rootless podman for the toolbox container was:
         0       1000          1
         1     100000      65536

... which was the same as seen from the host:
         0       1000          1
         1     100000      65536

Therefore, when running with an UID of 1000 inside the container, it
got mapped to UID 100999 on the host. That means, for example, files
created by the user inside the container end up looking funny from the
host.

This is addressed by creating another user namespace that's a child of
the initial user namespace created by rootless podman. Assuming a host
UID of 1000, the UID mapping inside this child namespace is:
      1000          0          1
         0          1       1000
      1001       1001      64536

... which when seen from the host is:
      1000       1000          1
         0     100000       1000
      1001     101000      64536

This means that UID 1000 inside the child namespace is mapped to the
same UID 1000 on the host via the intermediate namespace created by
rootless podman. UIDs 0 to 999 inside the child namespace are mapped
to UIDs 100000 to 100999 in the host.

This change requires this runc pull request to work:
https://github.com/opencontainers/runc/pull/1862

As suggested by Giuseppe Scrivano.
2018-09-20 16:27:19 +02:00
Debarshi Ray
a878a1fe40 Dockerfile: Add a maintainer label
... as mandated by the Fedora Container Guidelines:
https://fedoraproject.org/wiki/Container:Guidelines#LABELS
2018-09-18 16:06:55 +02:00
Debarshi Ray
c9e1d585f0 Remove extra whitespaces
Fallout from d7219ba512
2018-09-14 20:03:54 +02:00
Debarshi Ray
4581537baf Localize the working container name to 'create'
The working container and its name is a temporary and internal detail
of the 'create' command. It's not something that someone would want to
manually tweak.

Since 'local' is not mandated by POSIX, use a subshell for the
function instead.

This is a step towards making the commands more parametrized so that
one can use different toolbox containers.
2018-09-14 20:03:54 +02:00
Debarshi Ray
29b8471f6c Unhide the error output when unable to create a working container
Fallout from d7219ba512
2018-09-14 17:14:38 +02:00
Debarshi Ray
5cc2678a36 Dockerfile: Specify a CMD
... as mandated by the Fedora Container Guidelines:
https://fedoraproject.org/wiki/Container:Guidelines#CMD_.2F_ENTRYPOINT

The same entrypoint is also specified in 'create' because since
podman-0.9.1 'podman create ...' doesn't work without a command. See
commit fd697274a3.
2018-09-14 16:38:59 +02:00
Debarshi Ray
419bb1edeb README.md: Be consistent with the hyphenation 2018-09-14 13:11:53 +02:00
Debarshi Ray
edfb736517 Dockerfile: COPY the README.md into the fedora-toolbox image
... as mandated by the Fedora Container Guidelines:
https://fedoraproject.org/wiki/Container:Guidelines#Help_File
2018-09-14 11:49:24 +02:00
Debarshi Ray
b85fd7fccc README.md: Link to the GitHub page
The Fedora Container Guidelines [1] mandate that a README.md must be
COPYed into the image, and that it should have a link to the project
page.

[1] https://fedoraproject.org/wiki/Container:Guidelines#Help_File
2018-09-14 11:47:11 +02:00
Debarshi Ray
89bd833f44 README.md: Add description 2018-09-13 14:26:58 +02:00
Debarshi Ray
fd697274a3 Use a lighter PID 1 than $SHELL
When used as PID 1, /bin/sh takes 3248 kB compared to the 4136 kB
taken by /bin/bash. It's not a lot, but is memory that can be saved
for free.

As a nice side-effect, this unbreaks 'create' with podman-0.9.1 because
'podman create ...' doesn't work without a command.

https://github.com/containers/libpod/issues/1452
2018-09-13 11:49:26 +02:00
Debarshi Ray
d7219ba512 Allow unhiding the error output when desired
This adds a --verbose flag which will prevent the error stream of the
child commands from being redirected to /dev/null. The intermediate
file descriptor is arbitrarily hard coded as 42 because shell
variables and redirection don't appear to play well together.
2018-09-12 20:44:24 +02:00
Debarshi Ray
678bdbaf4e Make room for specifying global options and commands
A subsequent commit will add a global --verbose option that's can be
used with all commands. This will make it possible to do so.
2018-09-12 20:44:20 +02:00
Debarshi Ray
7cd0b3ce97 Be more explicit when the number of arguments is wrong
This is more in line with how unrecognized commands and options are
reported.
2018-09-12 20:44:16 +02:00
Debarshi Ray
fb3b0f76cb Add missing 'enter' command to --help
Fallout from 990e9eb5b7
2018-09-12 20:44:14 +02:00
Debarshi Ray
e02ea68921 README.md: Update
Fallout from 6c3031c9d0
2018-09-12 16:29:15 +02:00
Debarshi Ray
6c3031c9d0 Prefix the toolbox PS1 with an emoji for differentiation
Flatbox already uses 📦, so got to try something different. 🔹 could be
a good option. It's small, subtle and blue.
2018-09-12 15:19:55 +02:00
Debarshi Ray
f100cdaf77 Use $XDG_RUNTIME_DIR instead of /run/user/$UID 2018-09-11 19:25:28 +02:00
Debarshi Ray
34bfd0317d extra-packages: Add more things present in the Workstation image 2018-09-11 19:09:10 +02:00
Debarshi Ray
9e9a373d97 extra-packages: Add dnf-plugins-core
It's installed by default on Fedora Workstation.
2018-09-04 16:35:54 +02:00
Michael Scherer
0dedc0140d Avoid repeated prompts to unlock SSH keys
... by giving access to the SSH agent.

https://github.com/debarshiray/fedora-toolbox/pull/1
2018-09-03 16:39:51 +02:00
Debarshi Ray
990e9eb5b7 Initial revision 2018-08-31 18:38:01 +02:00