Missing content from operator directories
yselkowitz opened this issue · 5 comments
An erroneous fix for #226 has removed almost everything from the operator directories:
https://github.com/openshift/library/tree/master/operator/ocp-x86_64/official
/assign @coreydaley
When I look at https://deck-ci.apps.ci.l2s4.p1.openshiftapps.com/log?job=periodic-openshift-library-import&id=1276546179903000576
that came from the run @coreydaley that started at 12:01:41
which I think was about 20 minutes before your openshift/release PR merged
and yep that deleted a bunch of stuff
to your question openshift/release#9943 (comment) any samples operator PR will run image-eco, jenkins, and build e2e's .... which touch upon a number of samples
openshift/release#9943 was merged at 11:02:39 at which time @coreydaley said that the import would run within the hour, which it did.
thanks for the confirm @yselkowitz
@coreydaley is working on a fix now .... I'm having him run the periodic job command in his fix openshift/library fix branch and post the results as a separate commit for us to review
and if the batch/periodic commands have to change to get the content back, post that as well in the PR for reference, so we can cross ref with any openshift/release change he has to make
ok the periodic just ran ( https://deck-ci.apps.ci.l2s4.p1.openshiftapps.com/log?job=periodic-openshift-library-import&id=1276607614913875968 ) and no deletes
along with the (re)additions from #228 I think we are good to go but will leave this open and wait for @yselkowitz confirmation
Comparing the current contents of master to 9819966 (just before the python->golang transition), the only remaining deletions in operator/ocp-$ARCH seem to be the index.json and README.md files. The imagestreams and templates themselves are radically reformatted, so it's difficult to verify the differences, but at least the content appears to be present again.