Skip to content
This repository has been archived by the owner on Jun 14, 2019. It is now read-only.

clonerefs output is ignored, so tests can run with with incorrect refs #183

Open
stevekuznetsov opened this issue Oct 15, 2018 · 2 comments
Assignees

Comments

@stevekuznetsov
Copy link
Contributor

We ignore the output of clonerefs due to 7ad9489 and 9aa7a8a.

That's simply wrong and must be fixed.

We should either run initupload or write the log and read it ourselves, the output is simple.

If we had done this, the master-dockerless-build-temporary issue would not have happened.

/assign @smarterclayton

@smarterclayton
Copy link
Contributor

You mean, we should have read the log to find out that clone failed?

@stevekuznetsov
Copy link
Contributor Author

Thats exactly the (well documented) API for the tool...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants