Bug 1712245
Summary: | Unable to use shell variable in build config environment variable section when variable name includes a '.' | ||||||
---|---|---|---|---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | mchoma | ||||
Component: | Build | Assignee: | Nalin Dahyabhai <nalin> | ||||
Status: | CLOSED ERRATA | QA Contact: | wewang <wewang> | ||||
Severity: | high | Docs Contact: | |||||
Priority: | unspecified | ||||||
Version: | 4.1.0 | CC: | adam.kaplan, aos-bugs, dwalsh, jokerman, lxia, maschmid, mmccomas, mnovak, nalin, nstielau, pthomas, pweil, rjanik, wewang, wsun, wzheng, xiuwang, xtian | ||||
Target Milestone: | --- | Keywords: | Regression | ||||
Target Release: | 4.2.0 | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: |
Cause: When shell variables were referenced in build configurations which used the source-to-image build strategy, logic which attempted to produce a Dockerfile which could be used to perform the source-to-image build would incorrectly attempt to evaluate those variables.
Consequence: Some shell variables would be erroneously evaluated as empty values, leading to build errors, and other variables would trigger error messages from failed attempts to evaluate them.
Fix: Shell variables referenced in build configurations are now properly escaped, so that they are evaluated at the expected time.
Result: These errors should no longer be observed.
|
Story Points: | --- | ||||
Clone Of: | |||||||
: | 1713681 (view as bug list) | Environment: | |||||
Last Closed: | 2019-10-16 06:29:13 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 1713681 | ||||||
Attachments: |
|
Description
mchoma
2019-05-21 07:16:42 UTC
Created attachment 1571511 [details]
build.log
Could reproduce this with customer case in 4.1.0-0.nightly-2019-05-21-005558 version. And works with v3.11.115 and v3.10.145. Moving to the Containers component - bug appears to be in buildah processing the Dockerfile. We can't call this issue closed until the issue is fixed in buildah and merged into openshift/builder. We will also want to backport the fix to 4.1.z. Hmm, it looks like it's producing an error when the variable name includes a '.'. When I try the logged instructions using a Dockerfile with 'docker build' (both moby-engine-18.06.3-2.ce.gitd7080c1.fc30.x86_64 or docker-1.13.1-96.gitb2f74b2.el7.x86_64), I get the same `Missing ':' in substitution: "${user.home}"` error: FROM registry.access.redhat.com/jboss-eap-7/eap72-openshift:1.0 LABEL "io.openshift.build.image"="registry.access.redhat.com/jboss-eap-7/eap72-openshift:1.0" "io.openshift.build.source-location"="/tmp/build/inputs" ENV OPENSHIFT_BUILD_NAME="env-reproducer-bc-2" OPENSHIFT_BUILD_NAMESPACE="default" TEST="${user.home}" A bit of experimenting with 3.11.98 suggests that for s2i builds, environment variables set in the build config weren't being expanded, and ended up being set as specified in the resulting image's config blob, and the trigger for the build failing is that we're attempting to do so now. This looks like a combination of https://github.com/openshift/source-to-image/pull/969 and https://github.com/containers/buildah/pull/1607. *** Bug 1712855 has been marked as a duplicate of this bug. *** Wen - is this ON_QA yet? Cloned BZ for 4.1.z has been verified. Verified version: 4.2.0-0.nightly-2019-06-24-160709 Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2019:2922 |