geneontology / pipeline

Declarative pipeline for the Gene Ontology.
https://build.geneontology.org/job/geneontology/job/pipeline/
BSD 3-Clause "New" or "Revised" License
5 stars 5 forks source link

For #1199 #347

Closed mugitty closed 7 months ago

mugitty commented 7 months ago

Note, this PR is associated with https://github.com/geneontology/go-site/pull/2207

dustine32 commented 7 months ago

@mugitty Change here is to rsync a aggregate-rule-violation-report.md file but where is this file generated? I don't see the command to generate in the Jenkinsfile. Does this line need to be updated too? https://github.com/geneontology/pipeline/blob/995bd712499a38a3bb7180578016461500bef1e0/Jenkinsfile#L762

mugitty commented 7 months ago

aggregate-rule-violation-report.md is an additional file that is generated by combined_assigned_by.py

dustine32 commented 7 months ago

@mugitty Oh I see. Filename aggregate-rule-violation-report.md is hard-coded in combined_assigned_by.py.

This works but it might be nicer to parameterize that new output filename so that it's explicitly mentioned in the Jenkinsfile. I'll let @kltm decide whether that's really required.

kltm commented 7 months ago

When is the file going to start being produced and how? I want to make sure there is no lag so that rsync does not/cannot fail here. Ideally, the file would be produced and in the branches before the move itself was updated.

mugitty commented 7 months ago

@kltm , It is already being produced in https://build.geneontology.org/job/geneontology/job/pipeline/job/issue-go-site-2093-metadata/ https://github.com/geneontology/go-site/pull/2207#issuecomment-1841635784 has to go in first.

kltm commented 7 months ago

@mugitty it's not being produced on master, snapshot, or release, which are the ones I care about here. A mismatch would cause pipeline failure.

mugitty commented 7 months ago

@kltm , I will update on master

kltm commented 7 months ago

@mugitty To clarify, the go-site merge needs to happen before the pipeline merge.