ci-operator icon indicating copy to clipboard operation
ci-operator copied to clipboard

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

Open stevekuznetsov opened this issue 6 years ago • 2 comments

We ignore the output of clonerefs due to https://github.com/openshift/ci-operator/commit/7ad9489ab5f502d7b40fdd33df6ca508a84f7697 and https://github.com/openshift/ci-operator/commit/9aa7a8a12785307fb33d22920d53fced467187df.

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

stevekuznetsov avatar Oct 15 '18 11:10 stevekuznetsov

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

smarterclayton avatar Oct 15 '18 13:10 smarterclayton

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

stevekuznetsov avatar Oct 15 '18 14:10 stevekuznetsov