buildah: BUG: buildah does not use layer cache when building dockerfiles with --layers on COPY commands
BUG REPORT INFORMATION
Description
buildah bud does not use layer cache on COPY if the build does not occur in the same directory as it did previously.
In build environments the github repo is checked out into a different location for each build. But the layer cache for COPY commands gets ignored when the build command occurs in a different directory then the original one
Steps to reproduce the issue:
- Have a dockerfile with a copy command and run buildah bud --layers <folder>
- After it builds rerun the command to see that the caching worked.
- Copy the folder to a new location and see that the cache is not used on the COPY commands
Describe the results you received:
No cache is used
Describe the results you expected:
Cache should be used
Output of rpm -q buildah
or apt list buildah
:
buildah-1.11.6-4.module+el8.1.1+5259+bcdd613a.x86_64
Output of buildah version
:
Version: 1.11.6
Go Version: go1.12.12
Image Spec: 1.0.1-dev
Runtime Spec: 1.0.1-dev
CNI Spec: 0.4.0
libcni Version:
image Version: 5.0.0
Git Commit:
Built: Wed Dec 31 16:00:00 1969
OS/Arch: linux/amd64```
**Output of `podman version` if reporting a `podman build` issue:**
(paste your output here)
**Output of `cat /etc/*release`:**
NAME=“Red Hat Enterprise Linux” VERSION=“8.1 (Ootpa)” ID=“rhel” ID_LIKE=“fedora” VERSION_ID=“8.1” PLATFORM_ID=“platform:el8” PRETTY_NAME=“Red Hat Enterprise Linux 8.1 (Ootpa)” ANSI_COLOR=“0;31” CPE_NAME=“cpe:/o:redhat:enterprise_linux:8.1:GA” HOME_URL=“https://www.redhat.com/” BUG_REPORT_URL=“https://bugzilla.redhat.com/”
REDHAT_BUGZILLA_PRODUCT=“Red Hat Enterprise Linux 8” REDHAT_BUGZILLA_PRODUCT_VERSION=8.1 REDHAT_SUPPORT_PRODUCT=“Red Hat Enterprise Linux” REDHAT_SUPPORT_PRODUCT_VERSION=“8.1” Red Hat Enterprise Linux release 8.1 (Ootpa) Red Hat Enterprise Linux release 8.1 (Ootpa)```
Output of uname -a
:
Linux terefah1.fyre.ibm.com 4.18.0-147.5.1.el8_1.x86_64 #1 SMP Tue Jan 14 15:50:19 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Output of cat /etc/containers/storage.conf
:
# This file is is the configuration file for all tools
# that use the containers/storage library.
# See man 5 containers-storage.conf for more information
# The "container storage" table contains all of the server options.
[storage]
# Default Storage Driver
driver = "overlay"
# Temporary storage location
runroot = "/var/run/containers/storage"
# Primary Read/Write location of container storage
graphroot = "/var/lib/containers/storage"
[storage.options]
# Storage options to be passed to underlying storage drivers
# AdditionalImageStores is used to pass paths to additional Read/Only image stores
# Must be comma separated list.
additionalimagestores = [
]
# Size is used to set a maximum size of the container image. Only supported by
# certain container storage drivers.
size = ""
# Path to an helper program to use for mounting the file system instead of mounting it
# directly.
#mount_program = "/usr/bin/fuse-overlayfs"
# OverrideKernelCheck tells the driver to ignore kernel checks based on kernel version
override_kernel_check = "true"
# mountopt specifies comma separated list of extra mount options
# mountopt = "nodev,metacopy=on"
# Remap-UIDs/GIDs is the mapping from UIDs/GIDs as they should appear inside of
# a container, to UIDs/GIDs as they should appear outside of the container, and
# the length of the range of UIDs/GIDs. Additional mapped sets can be listed
# and will be heeded by libraries, but there are limits to the number of
# mappings which the kernel will allow when you later attempt to run a
# container.
#
# remap-uids = 0:1668442479:65536
# remap-gids = 0:1668442479:65536
# Remap-User/Group is a name which can be used to look up one or more UID/GID
# ranges in the /etc/subuid or /etc/subgid file. Mappings are set up starting
# with an in-container ID of 0 and the a host-level ID taken from the lowest
# range that matches the specified name, and using the length of that range.
# Additional ranges are then assigned, using the ranges which specify the
# lowest host-level IDs first, to the lowest not-yet-mapped container-level ID,
# until all of the entries have been used for maps.
#
# remap-user = "storage"
# remap-group = "storage"
[storage.options.thinpool]
# Storage Options for thinpool
# autoextend_percent determines the amount by which pool needs to be
# grown. This is specified in terms of % of pool size. So a value of 20 means
# that when threshold is hit, pool will be grown by 20% of existing
# pool size.
# autoextend_percent = "20"
# autoextend_threshold determines the pool extension threshold in terms
# of percentage of pool size. For example, if threshold is 60, that means when
# pool is 60% full, threshold has been hit.
# autoextend_threshold = "80"
# basesize specifies the size to use when creating the base device, which
# limits the size of images and containers.
# basesize = "10G"
# blocksize specifies a custom blocksize to use for the thin pool.
# blocksize="64k"
# directlvm_device specifies a custom block storage device to use for the
# thin pool. Required if you setup devicemapper.
# directlvm_device = ""
# directlvm_device_force wipes device even if device already has a filesystem.
# directlvm_device_force = "True"
# fs specifies the filesystem type to use for the base device.
# fs="xfs"
# log_level sets the log level of devicemapper.
# 0: LogLevelSuppress 0 (Default)
# 2: LogLevelFatal
# 3: LogLevelErr
# 4: LogLevelWarn
# 5: LogLevelNotice
# 6: LogLevelInfo
# 7: LogLevelDebug
# log_level = "7"
# min_free_space specifies the min free space percent in a thin pool require for
# new device creation to succeed. Valid values are from 0% - 99%.
# Value 0% disables
# min_free_space = "10%"
# mkfsarg specifies extra mkfs arguments to be used when creating the base.
# device.
# mkfsarg = ""
# use_deferred_removal marks devicemapper block device for deferred removal.
# If the thinpool is in use when the driver attempts to remove it, the driver
# tells the kernel to remove it as soon as possible. Note this does not free
# up the disk space, use deferred deletion to fully remove the thinpool.
# use_deferred_removal = "True"
# use_deferred_deletion marks thinpool device for deferred deletion.
# If the device is busy when the driver attempts to delete it, the driver
# will attempt to delete device every 30 seconds until successful.
# If the program using the driver exits, the driver will continue attempting
# to cleanup the next time the driver is used. Deferred deletion permanently
# deletes the device and all data stored in device will be lost.
# use_deferred_deletion = "True"
# xfs_nospace_max_retries specifies the maximum number of retries XFS should
# attempt to complete IO when ENOSPC (no space) error is returned by
# underlying storage device.
# xfs_nospace_max_retries = "0"
# If specified, use OSTree to deduplicate files with the overlay backend
ostree_repo = ""
# Set to skip a PRIVATE bind mount on the storage home directory. Only supported by
# certain container storage drivers
skip_mount_home = "false"
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Reactions: 3
- Comments: 21 (9 by maintainers)
Commits related to this issue
- Do not use layers, since layer caching depends on timestamps (unreliable in a CI/CD) See discussion in https://github.com/tektoncd/catalog/issues/265. Although using layer caching in the buildah buil... — committed to holly-cummins/catalog by holly-cummins 4 years ago
- Do not use layers, since layer caching depends on timestamps (unreliable in a CI/CD) See discussion in https://github.com/tektoncd/catalog/issues/265. Although using layer caching in the buildah buil... — committed to tektoncd/catalog by holly-cummins 4 years ago
- Fail if using Buildah with --multi-layer flag Until https://github.com/containers/buildah/issues/2215 is fixed, that is. Signed-off-by: Christopher Maier <cmaier@chef.io> — committed to habitat-sh/habitat by christophermaier 4 years ago
- Fail if using Buildah with --multi-layer flag Until https://github.com/containers/buildah/issues/2215 is fixed, that is. Signed-off-by: Christopher Maier <cmaier@chef.io> — committed to habitat-sh/habitat by christophermaier 4 years ago
- Fail if using Buildah with --multi-layer flag Until https://github.com/containers/buildah/issues/2215 is fixed, that is. Signed-off-by: Christopher Maier <cmaier@chef.io> — committed to habitat-sh/habitat by christophermaier 4 years ago
- Merge #2546 2546: newTarDigester: zero out timestamps in tar headers r=rhatdan a=nalind #### What type of PR is this? /kind feature #### What this PR does / why we need it: When digesting t... — committed to containers/buildah by bors[bot] 4 years ago
- Merge #2546 2546: newTarDigester: zero out timestamps in tar headers r=rhatdan a=nalind #### What type of PR is this? /kind feature #### What this PR does / why we need it: When digesting t... — committed to containers/buildah by bors[bot] 4 years ago
the workaround is to use docker instead.
This is intentional. if you want to change the default behaviour you can set the BUILDAH_LAYERS environment variable. export BUILDAH_LAYERS=true
This feels like a big barrier to using buildah in a CI/CD system. I’ve been wondering why only the first layer in my build file was getting cached, and now I think I see why.