Skip to content

Conversation

@aravindhp
Copy link
Contributor

Problem: hack/run-ci-e2e-test.sh fails in the following manner as observed when working on openshift/release#8323:

failed to initialize build cache at /.cache/go-build: mkdir /.cache: permission denied
Error: failed to build test binary: ...

Solution: Preemptively create the go cache directory in the image being built by CI.

Problem: hack/run-ci-e2e-test.sh fails in the following manner as
observed when working on openshift/release#8323:
```
 failed to initialize build cache at /.cache/go-build: mkdir /.cache: permission denied
Error: failed to build test binary: ...
```

Solution: Preemptively create the go cache directory in the image being
built by CI.
@aravindhp
Copy link
Contributor Author

/hold

@openshift-ci-robot openshift-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Apr 16, 2020
@aravindhp
Copy link
Contributor Author

/approve cancel

@aravindhp
Copy link
Contributor Author

/cc @openshift/openshift-team-windows-containers

@openshift-ci-robot openshift-ci-robot requested a review from a team April 16, 2020 19:31
@openshift-ci-robot openshift-ci-robot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 16, 2020
@PratikMahajan
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 16, 2020
@ravisantoshgudimetla
Copy link
Contributor

/approve

Hoping that this passes the tests.

@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ravisantoshgudimetla

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [ravisantoshgudimetla]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 16, 2020
@aravindhp
Copy link
Contributor Author

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 16, 2020
@aravindhp
Copy link
Contributor Author

/retest

@openshift-merge-robot openshift-merge-robot merged commit b5c09a2 into openshift:master Apr 17, 2020
@aravindhp aravindhp deleted the fix-container-build branch April 20, 2020 22:57
wgahnagl pushed a commit to wgahnagl/windows-machine-config-operator that referenced this pull request Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants