Skip to content

Conversation

@mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Feb 24, 2015

No description provided.

@mfojtik
Copy link
Contributor Author

mfojtik commented Feb 24, 2015

fyi, this works for both ONBUILD and STI strategies

@soltysh
Copy link
Contributor

soltysh commented Feb 24, 2015

LGTM

README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make it clear these env variables also end up existing in the output image.

@mfojtik
Copy link
Contributor Author

mfojtik commented Feb 24, 2015

@bparees fixed. the layered build will get the env vars same way as regular STI build. I was looking at stdlib for property file parser, but found nothing (I found some 3th party libs, but I think they are overkill for this)

@mfojtik
Copy link
Contributor Author

mfojtik commented Feb 24, 2015

[test]

@openshift-bot
Copy link
Contributor

Source To Image Test Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_sti/74/)

@bparees
Copy link
Contributor

bparees commented Feb 24, 2015

lgtm. [merge]

@openshift-bot
Copy link
Contributor

Source To Image Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_sti/69/)

@openshift-bot
Copy link
Contributor

Evaluated for source to image up to d2e8a46

openshift-bot pushed a commit that referenced this pull request Feb 24, 2015
@openshift-bot openshift-bot merged commit c0c154e into openshift:master Feb 24, 2015
@mfojtik mfojtik deleted the environment branch February 25, 2015 09:20
sayan-biswas pushed a commit to sayan-biswas/openshift-source-to-image that referenced this pull request Jan 21, 2025
OCPBUGS-21730: bump golang.org/x/net to v0.17.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants