-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sync all - improvements #709
Comments
Summary of how this went: - ampliseq [ PR existed ]
- artic [ PR existed ]
+ atacseq
+ bacass
+ bactmap
- bcellmagic [ PR existed ]
+ cageseq
+ chipseq
- clinvap [ archived ]
+ coproid
+ crisprvar
+ ddamsproteomics
- deepvariant [ PR error 404 ]
+ demultiplex
+ denovohybrid
+ diaproteomics
+ eager
+ epitopeprediction
- exoseq [ archived ]
+ gwas
+ hic
- hlatyping [ PR error 404 ]
+ imcyto
- kmermaid [ TEMPLATE branch problem ]
+ lncpipe
- mag [ PR error 404 ]
+ metaboigniter
+ methylseq
- mhcquant [ PR error 404 ]
+ mnaseseq
+ nanoseq
- nascent [ PR error 404 ]
+ neutronstar
+ proteomicslfq
+ rnafusion
+ rnaseq
+ sarek
+ scrnaseq
+ slamseq
+ smartseq2
+ smrnaseq
- vipr [ archived ]
+ viralrecon PR Error 422Failure for:
ERROR Sync failed for nf-core/ampliseq: sync.py:385
GitHub API returned code 422:
{
"message": "Validation Failed",
"errors": [
{
"resource": "PullRequest",
"code": "custom",
"message": "A pull request already exists for nf-core:TEMPLATE."
}
],
"documentation_url": "https://developer.github.com/v3/pulls/#create-a-pull-request"
} This isn't really an error - a PR already exists and the new commit is just appended. We should catch these and log them properly and report them as a success / warning. PR error 404Failure for:
ERROR Sync failed for nf-core/deepvariant: sync.py:385
GitHub API returned code 404:
{
"message": "Not Found",
"documentation_url": "https://developer.github.com/v3/pulls/#create-a-pull-request"
} Most worrying, as no reason for this that I can see..? Best I can think of is to try to catch them and just try again a few times with a 10 second sleep. Problem with repoFailure for:
Pipeline needs fixing. Archived pipelinesFailure for:
Miscellaneous errors, but these pipelines should never have been synced because they are archived. I thought that this was fixed after the last sync (#548 (comment)) but apparently not.. |
Sync worked beautifully last time, so I think we're done here. |
We need to go through the the v1.10.1 sync log and see what to fix / improve.
Final summary:
Learning from my mistakes and posting here as well in case the above action logs expire and are deleted:
logs_2307.zip
6_Run synchronisation.txt
The artifact upload failed with error
##[error]read ECONNRESET
The text was updated successfully, but these errors were encountered: