set COSA_DIR in Dockerfile#827
Conversation
|
/lgtm |
6220d6d to
2fb01b0
Compare
|
Looks like we need to create it in the |
2fb01b0 to
46deaad
Compare
We can also create it inside the container image. I ran into the same permissions issues I ran into when I was setting up the CI. I fixed it though. |
|
This is still showing the same failure mode of EDIT: The kola tests need to be executed from |
yet...we didn't exit out? Oh no, oh no. coreos/coreos-assembler#2898 strikes again. |
This way it should be inherited more consistently by further jobs.
I noticed in CI we just plowed forward and yeah...bash is just a language that seems superficially convenient and simple. But then you pay it back later spending hours on inane problems.
|
I pushed two followup commits here, hope that's OK! |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cheesesashimi, travier The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/retest |
|
@cheesesashimi: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
OK cool, so it's just the last bit of |
I was about to add |
With the introduction of openshift/release#29187, the test stages no longer have an explicitly set COSA_DIR value. Instead, they are calling
$ mktemp -d.